Skip to content

Upload 503 Error and Batchjob Interaction Fix#3347

Open
daralynnrhode wants to merge 4 commits into
IMAP-Science-Operations-Center:devfrom
daralynnrhode:upload_batchjob_error
Open

Upload 503 Error and Batchjob Interaction Fix#3347
daralynnrhode wants to merge 4 commits into
IMAP-Science-Operations-Center:devfrom
daralynnrhode:upload_batchjob_error

Conversation

@daralynnrhode

Copy link
Copy Markdown
Contributor

Fix for the Upload Batchjob DeadLock Error Occurring

Overview

closes #3010
Checking the response of the Upload API before the upload_products() function fails and gives a response to batch. Also added a test to check for the upload 503 error.

File changes

  • imap_processing/cli.py
  • imap_processing/tests/test_cli.py

Testing

Added test_post_processing_upload_503_error() to test_cli.py

@daralynnrhode daralynnrhode added this to the July 2026 milestone Jul 24, 2026
@daralynnrhode daralynnrhode self-assigned this Jul 24, 2026
@daralynnrhode daralynnrhode added bug Something isn't working Student Tasks suitable for student work labels Jul 24, 2026

@tmplummer tmplummer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You implemented exactly what was suggested in the ticket.

I am wondering if we should try the upload again in the case of the specific 503 error mentioned in the ticket. It seems like if the exception is a 503 slow down (I have no idea if that is all 503 errors) then we could potentially sleep for a few seconds and try again. Just a thought.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Student Tasks suitable for student work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG - Job should fail if file upload fails

2 participants