Skip to content

ci: rotate ec2-github-runner SHA + opt into EBS encryption (Phase 6.b)#192

Merged
kurok merged 1 commit into
masterfrom
fix/rotate-runner-phase6b
Apr 21, 2026
Merged

ci: rotate ec2-github-runner SHA + opt into EBS encryption (Phase 6.b)#192
kurok merged 1 commit into
masterfrom
fix/rotate-runner-phase6b

Conversation

@kurok
Copy link
Copy Markdown
Collaborator

@kurok kurok commented Apr 21, 2026

Dogfood rotation for namecheap/ec2-github-runner#27 (EBS encryption opt-in). Rotates both pins 0fdd401 → 7c6a9a7 and sets encrypt-ebs: 'true' on the start-runner step.

Risk: if this CI account can't use the default alias/aws/ebs KMS key, or if the shared AMI's snapshot is encrypted under a customer-managed key lacking a cross-account grant, Start EC2 runner will fail with a KMS/IAM error. The action throws early (doesn't time out), so diagnostics are quick via the console-output recipe if needed.

If it fails, the fix is a 1-line revert of the encrypt-ebs input — the SHA rotation itself is orthogonal and stays correct.

namecheap/ec2-github-runner#27 merged. Rotates both pins from
0fdd401 (Phase 4 retry) to 7c6a9a7 (Phase 6.b) and flips the new
encrypt-ebs input to 'true' so the acceptance-test runner's root
volume launches with SSE-EBS.

Risk: if the CI AWS account can't use the default aws/ebs KMS key
or the shared AMI's snapshot is encrypted with a customer-managed
key lacking a cross-account grant, start-runner will fail with a
KMS / IAM error. The action-side code handles that gracefully
(throws early rather than timing out on registration), so
diagnostics are easy via aws ec2 get-console-output.

If dogfood fails, revert just the encrypt-ebs line; SHA rotation
is orthogonal and can stay green.

Signed-off-by: yuriyryabikov <22548029+kurok@users.noreply.github.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.76%. Comparing base (9a21817) to head (f439e99).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   87.76%   87.76%           
=======================================
  Files           4        4           
  Lines         711      711           
=======================================
  Hits          624      624           
  Misses         52       52           
  Partials       35       35           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kurok kurok merged commit 31d8069 into master Apr 21, 2026
12 checks passed
@kurok kurok deleted the fix/rotate-runner-phase6b branch April 21, 2026 16:11
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.

1 participant