Skip to content
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

[QOLSVC-4689] add CLI option to process datasets immediately, #202 #82

Merged
merged 4 commits into from
Feb 2, 2024

Conversation

ThrawnCA
Copy link

  • This will allow eg support staff processing a critical resource immediately while a large job is occupying the queue

- This will allow eg support staff processing a critical resource immediately while a large job is occupying the queue
@duttonw
Copy link
Member

duttonw commented Jan 31, 2024

Can you also add the option for allowing a different queue for it to be submitted against.

@ThrawnCA
Copy link
Author

Can you also add the option for allowing a different queue for it to be submitted against.

That's possible, but by default the option would then be exposed in the public API, so any authenticated user could theoretically submit to an arbitrary queue. Is that something we want?

@duttonw
Copy link
Member

duttonw commented Jan 31, 2024

may need a comma delimited list of valid queues to stop dead letter queue issues. This only affects authors and above. This could also help in doing the front end for priority processing for sysadmins at least.

@ThrawnCA ThrawnCA force-pushed the QOLSVC-4689-oft-locations branch 6 times, most recently from f5238d4 to 56dd94e Compare January 31, 2024 07:40
- This is currently only available to sysadmins or via CLI
@ThrawnCA ThrawnCA force-pushed the QOLSVC-4689-oft-locations branch from 56dd94e to 9c79600 Compare February 1, 2024 02:23
- Queue name isn't available to the job, and if support staff are directly running it we probably don't need the automatic retry
@ThrawnCA ThrawnCA merged commit 772f0bf into develop Feb 2, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants