Skip to content

Commit 71a78ac

Browse files
authored
fix(pr-metadata): Make PR metadata collection work with remote states (#2648)
Signed-off-by: Javier Rodriguez <javier@chainloop.dev>
1 parent c5151c0 commit 71a78ac

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

pkg/attestation/crafter/crafter.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -479,6 +479,14 @@ func (c *Crafter) ResolveEnvVars(ctx context.Context, attestationID string) erro
479479

480480
// AutoCollectPRMetadata automatically collects PR/MR metadata if running in a PR/MR context
481481
func (c *Crafter) AutoCollectPRMetadata(ctx context.Context, attestationID string, runner SupportedRunner, casBackend *casclient.CASBackend) error {
482+
if err := c.requireStateLoaded(); err != nil {
483+
return fmt.Errorf("crafting state not loaded before inspecting PR/MR metadata: %w", err)
484+
}
485+
486+
if err := c.LoadCraftingState(ctx, attestationID); err != nil {
487+
c.Logger.Warn().Err(err).Msg("failed to reload crafting state")
488+
}
489+
482490
// Detect if we're in a PR/MR context
483491
isPR, metadata, err := DetectPRContext(runner)
484492
if err != nil {

0 commit comments

Comments
 (0)