Skip to content

Updated HarmonyTransientError handling to address reviewer comments - #188

Merged
jackiryan merged 1 commit into
release/0.8.0from
issues/0.8.0-review1
Jun 9, 2026
Merged

Updated HarmonyTransientError handling to address reviewer comments#188
jackiryan merged 1 commit into
release/0.8.0from
issues/0.8.0-review1

Conversation

@jackiryan

Copy link
Copy Markdown
Contributor

Description

Addresses Andrew Johnston's comments on the release/0.8.0 PR.

Overview of work done

  • HarmonyTransientError is handled in the same retry logic as HarmonyJobIncompleteError in the state machine definition file
  • ConnectionErrors and Timeouts when checking job status are also treated as HarmonyTransientError

Overview of verification done

No need to update unit tests or changelog since this is part of issue 181 in the original PR.

@jackiryan
jackiryan requested a review from jamesfwood June 9, 2026 19:02
@tloubrieu-jpl tloubrieu-jpl moved this to needs:triage in podaac Jun 9, 2026
@jackiryan
jackiryan merged commit 94cf509 into release/0.8.0 Jun 9, 2026
7 checks passed
@jackiryan
jackiryan deleted the issues/0.8.0-review1 branch June 9, 2026 19:54
@tloubrieu-jpl tloubrieu-jpl moved this from needs:triage to closed in podaac Jun 9, 2026
@jackiryan
jackiryan restored the issues/0.8.0-review1 branch June 9, 2026 21:31
@jackiryan
jackiryan deleted the issues/0.8.0-review1 branch June 9, 2026 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: closed

Development

Successfully merging this pull request may close these issues.

2 participants