Skip to content

ci: push gateway model settings update as signed commit#14862

Merged
R-Taneja merged 1 commit intomainfrom
fix/update-model-settings-signed-commit
Apr 30, 2026
Merged

ci: push gateway model settings update as signed commit#14862
R-Taneja merged 1 commit intomainfrom
fix/update-model-settings-signed-commit

Conversation

@R-Taneja
Copy link
Copy Markdown
Contributor

@R-Taneja R-Taneja commented Apr 30, 2026

Summary

  • The Update Gateway Model Settings workflow has been failing on git push because the repo enforces "Commits must have verified signatures" and the workflow's git commit is unsigned.
  • Replaces the git checkout -b && git commit && git push step with the same createCommitOnBranch GraphQL pattern already used by .github/workflows/backport.yml. Commits submitted via the GitHub API with GITHUB_TOKEN are signed by GitHub automatically.
  • The local git commit is preserved purely as a way to compute the diff against the base branch tip — only the API-submitted commit is pushed.

Reference failure: run log

The repo enforces "Commits must have verified signatures", so the plain
`git push` from this workflow has been rejected. Switch to the same
pattern used by `.github/workflows/backport.yml`: build the local commit,
then submit the file additions/deletions through the GraphQL
`createCommitOnBranch` mutation. Commits made via the GitHub API with
GITHUB_TOKEN are signed by GitHub automatically, so they pass the rule.
@R-Taneja R-Taneja merged commit cc38d0c into main Apr 30, 2026
20 of 21 checks passed
@R-Taneja R-Taneja deleted the fix/update-model-settings-signed-commit branch April 30, 2026 19:42
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