Skip to content

Add workflow exist check to centralized registry workflow deploy#376

Merged
j-nowak merged 11 commits intomainfrom
DEVSVCS-4625/check-workflow-exists
Apr 16, 2026
Merged

Add workflow exist check to centralized registry workflow deploy#376
j-nowak merged 11 commits intomainfrom
DEVSVCS-4625/check-workflow-exists

Conversation

@j-nowak
Copy link
Copy Markdown
Contributor

@j-nowak j-nowak commented Apr 15, 2026

Summary

  • Adds a pre-deploy workflow existence check so workflow deploy can detect existing workflows before upload and prompt for overwrite confirmation.
  • Implements private-registry existence checks via GraphQL lookup by workflow name, and threads the existing workflow status through deploy flow.
  • Refactors registry deploy strategies to share a CheckWorkflowExists path across onchain and private targets.
  • Updates private-registry client models/operations to the offchain workflow API naming and adds coverage for success, not-found, and transport-error paths.

Testing

  • Extended deploy and private-registry unit tests around existence checks, overwrite behavior, and GraphQL failure handling.
  • Updated multi-command private-registry flow test to validate the new lookup + upsert operation sequence.

@j-nowak j-nowak changed the base branch from main to jn/offchain-registry-gql-rename April 15, 2026 14:57
@j-nowak j-nowak marked this pull request as ready for review April 15, 2026 15:03
@j-nowak j-nowak requested a review from a team as a code owner April 15, 2026 15:03
Base automatically changed from jn/offchain-registry-gql-rename to main April 15, 2026 15:26
@j-nowak j-nowak requested a review from timothyF95 April 16, 2026 05:01
Copy link
Copy Markdown
Contributor

@timothyF95 timothyF95 left a comment

Choose a reason for hiding this comment

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

LGTM

@j-nowak j-nowak added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit 3c4f9c8 Apr 16, 2026
22 checks passed
@j-nowak j-nowak deleted the DEVSVCS-4625/check-workflow-exists branch April 16, 2026 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants