Skip to content

FIREFLY-2007: Properly initialize jobs, distinguishing between UWS and Firefly-managed jobs.#1966

Merged
loitly merged 1 commit into
rc-2026.1from
FIREFLY-2007-uws-job-init
May 28, 2026
Merged

FIREFLY-2007: Properly initialize jobs, distinguishing between UWS and Firefly-managed jobs.#1966
loitly merged 1 commit into
rc-2026.1from
FIREFLY-2007-uws-job-init

Conversation

@loitly
Copy link
Copy Markdown
Contributor

@loitly loitly commented May 28, 2026

Ticket: https://jira.ipac.caltech.edu/browse/FIREFLY-2007

Also:

  • Continue polling(with timeout) for the UWS status during bad phases to ensure its survival even in transient errors like network issues.

Test: https://fireflydev.ipac.caltech.edu/firefly-2007-uws-job-init/firefly/

  • Ensure that background jobs continue to function as intended.
  • Verify that the UWS job is properly initialized and accurately reflects information from the UWS service, without introducing any default values.

An example: IRSA TAP do not have Creation Time.

…d Firefly-managed jobs.

- Continue polling(with timeout) for the UWS status during bad phases to ensure its survival even in transient errors like network issues.
@loitly loitly added this to the 2026.1 milestone May 28, 2026
@loitly loitly requested review from Copilot and robyww May 28, 2026 19:40
@loitly loitly self-assigned this May 28, 2026
@loitly loitly added the bug label May 28, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Contributor

@robyww robyww left a comment

Choose a reason for hiding this comment

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

Looks fine

@loitly loitly merged commit df82b05 into rc-2026.1 May 28, 2026
1 check failed
@loitly loitly deleted the FIREFLY-2007-uws-job-init branch May 28, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants