Skip to content

test: skip gpg signing in test gitInit - #3967

Open
lkingland wants to merge 1 commit into
knative:mainfrom
lkingland:coder.functions.2/61
Open

test: skip gpg signing in test gitInit#3967
lkingland wants to merge 1 commit into
knative:mainfrom
lkingland:coder.functions.2/61

Conversation

@lkingland

@lkingland lkingland commented Jul 25, 2026

Copy link
Copy Markdown
Member
  • 🐛 Local-disable commit.gpgsign and tag.gpgsign in the config-ci e2e helper's temp git repo so a host global GPG signing config cannot fail git commit with No secret key for the synthetic test identity.

/kind bug

Functions#61

Global git config commit.gpgsign=true was leaking into the test's temp
repository, causing 'git commit' to fail with 'No secret key' when the
host has signing enabled but no matching key for the test identity.
Override both commit.gpgsign and tag.gpgsign locally so the test is
robust to host gitconfig.

Functions#61 (recovered from local rywnuxxx)
@knative-prow knative-prow Bot added the kind/bug Bugs label Jul 25, 2026
@knative-prow
knative-prow Bot requested review from dsimansk and jrangelramos July 25, 2026 15:07
@knative-prow

knative-prow Bot commented Jul 25, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lkingland

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow Bot added approved 🤖 PR has been approved by an approver from all required OWNERS files. size/XS 🤖 PR changes 0-9 lines, ignoring generated files. labels Jul 25, 2026
@lkingland lkingland changed the title test(e2e/configci): skip gpg signing in test gitInit test: skip gpg signing in test gitInit Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved 🤖 PR has been approved by an approver from all required OWNERS files. kind/bug Bugs size/XS 🤖 PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant