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.

Sign up for GitHub

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

Jump to bottom

Fix handling of options provided to the push to S3 script #964

Merged
merged 1 commit into from
Apr 27, 2023
Merged

Fix handling of options provided to the push to S3 script #964

merged 1 commit into from
Apr 27, 2023

Conversation

cmadjar
Copy link
Collaborator

@cmadjar cmadjar commented Apr 27, 2023

Description

This fixes the handling of options provided to the script run_push_imaging_files_to_s3_pipeline.py. The fixes include:

  • when no Upload ID is provided to the script, return a message that --upload_id is a required option
  • when an Upload ID that does not exist in the database is provided to the script, return a message that the Upload ID does not exist
  • remove from the help documentation the -t option as it is not a valid option for this script

@cmadjar cmadjar added this to the 24.1.11 milestone Apr 27, 2023
@cmadjar cmadjar self-assigned this Apr 27, 2023
@cmadjar cmadjar linked an issue Apr 27, 2023 that may be closed by this pull request
Push to S3 pipeline script - better handle options provided to the script #959
Closed
3 tasks
@cmadjar cmadjar merged commit cddb7fa into aces:24.1-release Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cmadjar cmadjar

Labels
Bug Python imaging pipeline
Projects
None yet
Milestone
24.1.11
Development

Successfully merging this pull request may close these issues.

Push to S3 pipeline script - better handle options provided to the script
1 participant
@cmadjar