Skip to content

Fall back to init state on deploy constraint err - #1970

Open
osoriano wants to merge 1 commit into
masterfrom
osoriano/deploy-constraint-err
Open

osoriano wants to merge 1 commit into
masterfrom
osoriano/deploy-constraint-err

Conversation

@osoriano

Copy link
Copy Markdown
Contributor

No description provided.

@osoriano
osoriano requested a review from a team as a code owner September 18, 2026 23:33
@github-actions github-actions Bot added the deploy-service Includes changes to deploy-service label Sep 18, 2026
@osoriano
osoriano requested a balanced review from Copilot September 18, 2026 23:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The existing exception test will fail, and the error log reports the wrong resulting state.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 High severity

Open (1)
What changed in this PR

Resets failed deploy-constraint jobs to INIT so they retry on the next worker run.

Changes:

  • Falls back to INIT after unexpected exceptions.
  • Logs the planned retry.
File Description
DeployTagWorker.java Resets failed jobs for retry.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

ExceptionUtils.getRootCauseMessage(e),
ExceptionUtils.getStackTrace(e));
job.setState(TagSyncState.ERROR);
job.setState(TagSyncState.INIT);
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy-service Includes changes to deploy-service

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants