feat(hydrator): sign hydrated commits (Alpha) (#28239)#28271
Open
mladjan-gadzic wants to merge 1 commit into
Open
feat(hydrator): sign hydrated commits (Alpha) (#28239)#28271mladjan-gadzic wants to merge 1 commit into
mladjan-gadzic wants to merge 1 commit into
Conversation
Signed-off-by: Mladjan Gadzic <gadzic.mladjan@gmail.com>
✅ Preview Environment deployed on Bunnyshell
See: Environment Details | Pipeline Logs Available commands (reply to this comment):
|
Bundle ReportBundle size has no change ✅ |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #28271 +/- ##
==========================================
+ Coverage 64.61% 64.83% +0.21%
==========================================
Files 423 426 +3
Lines 58320 59333 +1013
==========================================
+ Hits 37685 38468 +783
- Misses 17115 17292 +177
- Partials 3520 3573 +53 ☔ View full report in Codecov by Harness. |
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
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.
Description
Closes #28239
What does this PR do?
Adds optional GPG signing of hydrated commits to the commit server. When a signing key is
configured, every commit the source hydrator pushes is GPG-signed before being pushed, and the
commit server refuses to push any commit it cannot locally verify against that key (no unsigned
fallback).
This closes a documented limitation of the source hydrator: previously it never signed the commits
it pushed, so when signature verification was enabled the hydrated manifests failed verification at
sync time.
Feature status: Alpha (since v3.6). This builds on the (Beta) source hydrator and the (Alpha)
source-integrity verification of dry sources.
How does it work?
commitserver.signing.key.path— path to the ASCII-armored GPG private key (default"",signing off).
commitserver.signing.key.passphrase.file— optional passphrase file for the key.argocd-commit-server-gpg-signing-key) mounted intothe commit server deployment.
startup rather than silently falling back to unsigned commits.
Finer-grained (per-project/per-repo) keys are a planned additive extension; the existing
commitserver.signing.key.*params remain the default and won't be renamed.To verify the signed commits at sync time, the corresponding public key must be added to Argo
CD's trusted GPG keys and the project's
SourceIntegritypolicy configured for the hydrated branch.Checklist: