Skip to content

chore(EC-2153): adapt digest update script for pipeline-level default - #236

Open
jsmid1 wants to merge 1 commit into
conforma:mainfrom
jsmid1:EC-2153
Open

jsmid1 wants to merge 1 commit into
conforma:mainfrom
jsmid1:EC-2153

Conversation

@jsmid1

@jsmid1 jsmid1 commented Sep 17, 2026

Copy link
Copy Markdown

DO NOT MERGE BEFORE conforma/cli#3573 IS MERGED

The POLICY_BUNDLE_DIGEST default has been moved from task definitions to the enterprise-contract pipeline in conforma/cli (EC-2153). Task definitions now have an empty default.

Changes:

  • update-policy-digest.sh: generalize variable names and exit cleanly when no files carry a non-empty POLICY_BUNDLE_DIGEST default
  • konflux-policy.yaml: mark the digest update step as expected no-op since extracted tasks no longer carry the default
  • spec/update_policy_digest_spec.sh: add tests for Pipeline YAML, empty defaults, and the no-files-found case

Resolves: EC-2153

The POLICY_BUNDLE_DIGEST default has been moved from task definitions
to the enterprise-contract pipeline in conforma/cli (EC-2153). Task
definitions now have an empty default.

Changes:
- update-policy-digest.sh: generalize variable names and exit cleanly
  when no files carry a non-empty POLICY_BUNDLE_DIGEST default
- konflux-policy.yaml: mark the digest update step as expected no-op
  since extracted tasks no longer carry the default
- spec/update_policy_digest_spec.sh: add tests for Pipeline YAML,
  empty defaults, and the no-files-found case
@jsmid1
jsmid1 requested a review from a team as a code owner September 17, 2026 13:09
@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 1297503c-f722-4c68-90d1-5594e85769c9


Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant