chore(ci): use GraphQL createCommitOnBranch for signed commits#285
Closed
guglez wants to merge 4 commits into
Closed
chore(ci): use GraphQL createCommitOnBranch for signed commits#285guglez wants to merge 4 commits into
guglez wants to merge 4 commits into
Conversation
After a successful Docker build on a v* tag, open a PR in ChainSafe/infra-kubernetes bumping canton-middleware-api image tag for validator-dev1. Refs ChainSafe/infrastructure-general#1246 Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
force-with-lease on a freshly-created local branch with no upstream tracking ref is unreliable — use plain git push -u origin instead. Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
- Enable allow_auto_merge on infra-kubernetes repo - Add gh pr merge --auto --squash after PR creation - Split pr create and pr merge to avoid masking merge failures Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Replace git commit+push with GitHub GraphQL API to produce Verified/signed commits — satisfies required_signatures ruleset in infra-kubernetes without needing a bypass actor. Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Contributor
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
dhyaniarun1993
changed the title
Member
dhyaniarun1993
left a comment
dhyaniarun1993
left a comment
There was a problem hiding this comment.
Please fix the merge conflict.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces
git commit + git pushwith the GitHub GraphQLcreateCommitOnBranchmutation.Why: The
required_signaturesruleset onChainSafe/infra-kubernetesrequires all commits onmainto be signed. Commits created viagit pushfrom CI are unsigned and block auto-merge. Commits created via the GitHub GraphQL API are automatically signed by GitHub and show as Verified — no bypass actor or GPG key needed.Changes in
open-devnet-prjob:Accept: application/vnd.github.raw(no base64 decode)createCommitOnBranchgit commit/git push/git configblocks removed entirelyRefs ChainSafe/infrastructure-general#1246
🤖 Generated with Claude Code