Skip to content

ci: force VM harness clone to use deploy key#82

Merged
dtkav merged 1 commit intomerge-hsmfrom
fix/e2e-standard-vm-harness-ssh
Apr 10, 2026
Merged

ci: force VM harness clone to use deploy key#82
dtkav merged 1 commit intomerge-hsmfrom
fix/e2e-standard-vm-harness-ssh

Conversation

@mgmobrien
Copy link
Copy Markdown
Member

Summary

  • fix the first real merge-hsm E2E standard suite failure after PR ci: rename e2e workflow to standard suite #81 merged
  • stop relying on stale VM ~/.ssh/config when cloning or updating the private relay-harness repo
  • force the VM-side harness git operations to use the staged ci-deploy-key via GIT_SSH_COMMAND

Root cause

The Linux VM already has ~/.ssh/config pinned to ~/.ssh/relay-e2e-deploy, and that key does not have access to No-Instructions/relay-harness. The workflow was appending a second Host github.com stanza, but that was not a reliable override, so the private harness clone failed inside the VM step.

Verification

  • git diff --check -- .github/workflows/e2e-standard.yml
  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/e2e-standard.yml")'
  • direct VM repro of the stale-key failure mode via gcloud ssh

@mgmobrien
Copy link
Copy Markdown
Member Author

@dtkav follow-up to #81. The first real merge-hsm standard-suite run failed because the Linux VM still had a stale github.com SSH identity pinned in ~/.ssh/config, so the private harness clone used the wrong key. This PR forces the VM-side harness git ops to use ci-deploy-key explicitly.

@mgmobrien mgmobrien requested a review from dtkav April 10, 2026 21:33
@dtkav
Copy link
Copy Markdown
Member

dtkav commented Apr 10, 2026

@dtkav
Copy link
Copy Markdown
Member

dtkav commented Apr 10, 2026

IdentitiesOnly yes could also be applied to the old way (so it isn't per session), but this way is fine too.

@dtkav dtkav merged commit e850fe4 into merge-hsm Apr 10, 2026
1 of 2 checks passed
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