-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Re-integrate (QA | ARCHIVER | REPORT) extensions to catalog.data.gov #3957
Comments
List of things that work and don't work
|
|
|
Note: We are on a fork of |
Do we want to attempt to merge these changes back upstream? That might be a separate ticket... Or were there more changes than just the PR above? |
So... archiver tries to use Redis to process the archival queue, but falls back to celery if there is an ImportError 😕 It looks like it's falling back to celery, but I'm not sure why... Since celery is not installed, the openness score is not computed per dataset.
@jbrown-xentity That was the only PR. We can try to merge it upstream. But I think upstream is still supporting PY2, so we'd have to do a conditional |
This is the solution to getting the queue to finish!! We need to specify the queue that has the work to do when calling ckan jobs worker bulk |
Welp... looks like they don't care about PY2 support after all, Yay!
The above PR fixes two issues. The |
Also going to note, at least we know of at least one CKAN instance that is actively use the QA extension, Finnish Digital Agency |
User Story
In order to restore dilapidated features on catalog.data.gov, the Data.gov Catalog team wants to re-integrate
ckanext-qa
,ckanext-archiver
andckanext-report
.Acceptance Criteria
[ACs should be clearly demoable/verifiable whenever possible. Try specifying them using BDD.]
WHEN I visit a dataset on catalog.data.gov
THEN I can see that the extensions are live at https://catalog.data.gov/api/action/status_show
Background
[Any helpful contextual notes or links to artifacts/evidence, if needed]
Security Considerations (required)
[Any security concerns that might be implicated in the change. "None" is OK, just be explicit here!]
Sketch
[Notes or a checklist reflecting our understanding of the selected approach]
The text was updated successfully, but these errors were encountered: