remove confusing comment

javirln · javirln · commit f82ae44fc6d2 · 2025-12-15T18:24:13.000+01:00
Signed-off-by: Javier Rodriguez &lt;javier@chainloop.dev&gt;
diff --git a/app/cli/pkg/action/attestation_init.go b/app/cli/pkg/action/attestation_init.go
@@ -234,7 +234,7 @@ func (action *AttestationInit) Run(ctx context.Context, opts *AttestationInitRun
 	if !action.dryRun && attestationID != "" {
 		connectionCloserFn, err := getCASBackend(ctx, client, attestationID, action.casCAPath, action.casURI, action.connectionInsecure, action.Logger, casBackend)
 		if err != nil {
-			// This should never happen in lenient mode, but handle it just in case
+			// We don't want to fail the attestation initialization if CAS setup fails, it's a best-effort feature for PR/MR metadata
 			action.Logger.Warn().Err(err).Msg("unexpected error getting CAS backend")
 		}
 		if connectionCloserFn != nil {

Original file line number	Diff line number	Diff line change
`@@ -234,7 +234,7 @@ func (action AttestationInit) Run(ctx context.Context, opts AttestationInitRun`
`234`	`234`	`if !action.dryRun && attestationID != "" {`
`235`	`235`	`connectionCloserFn, err := getCASBackend(ctx, client, attestationID, action.casCAPath, action.casURI, action.connectionInsecure, action.Logger, casBackend)`
`236`	`236`	`if err != nil {`
`237`		`- // This should never happen in lenient mode, but handle it just in case`
	`237`	`+ // We don't want to fail the attestation initialization if CAS setup fails, it's a best-effort feature for PR/MR metadata`
`238`	`238`	`action.Logger.Warn().Err(err).Msg("unexpected error getting CAS backend")`
`239`	`239`	`}`
`240`	`240`	`if connectionCloserFn != nil {`