Skip to content

Commit d357bac

Browse files
committed
fix: added minor change to address pr feedback
1 parent 0598434 commit d357bac

File tree

4 files changed

+12
-2
lines changed

4 files changed

+12
-2
lines changed

.github/actions/generate-slsa-predicate/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ runs:
2828
run: |
2929
set -euo pipefail
3030
PREDICATE="${RUNNER_TEMP}/slsa-predicate.json"
31-
cat > "$PREDICATE" <<EOF
31+
cat > "$PREDICATE" <<-EOF
3232
{
3333
"buildDefinition": {
3434
"buildType": "https://aicr.nvidia.com/binary/v1",

.github/workflows/build-attested.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,13 @@
1616
# Produces tar.gz archives with SLSA Build Provenance v1 attestation
1717
# as downloadable job artifacts.
1818

19+
## NOTE: THIS WORKFLOW IS FOR TESTING PURPOSES ONLY.
20+
## if you need something attested by ci. This does not run tests or security scans.
21+
## The only complete/valid way to attest that passes all validation is via on-tag.yaml.
22+
## Validate attestations requires to pass the following certificate identity regexp:
23+
## --certificate-identity-regexp 'https://github.com/NVIDIA/aicr/.github/workflows/on-tag\.yaml@refs/tags/.*'
24+
## so this attestation is not the same as a production release.
25+
1926
name: Build Attested Binaries
2027

2128
on:
@@ -27,6 +34,7 @@ permissions:
2734

2835
jobs:
2936
build-and-attest:
37+
needs: [tests, security-scan]
3038
name: Build and Attest Binaries
3139
runs-on: ubuntu-latest
3240
timeout-minutes: 15

.goreleaser.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,8 @@ builds:
3535
-X github.com/NVIDIA/aicr/pkg/cli.date={{.Date}}
3636
hooks:
3737
post:
38+
## cosign v1 attestation with slsa provenance v1.
39+
## NOTE: below aicrd currently attests via github attestation
3840
- cmd: >-
3941
bash -c '[ -z "${SLSA_PREDICATE:-}" ] && exit 0;
4042
cosign attest-blob

install

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ main() {
238238
"${temp_dir}/${BIN_NAME}" 2>/dev/null; then
239239
msg "Attestation verified: binary built by github.com/NVIDIA/aicr"
240240
else
241-
msg "Warning: attestation verification failed (binary may not be from an official release)"
241+
msg "Warning: attestation verification failed — cannot confirm this binary was built by the official CI pipeline"
242242
fi
243243
elif [[ -f "${temp_dir}/${BIN_NAME}-attestation.sigstore.json" ]]; then
244244
msg "Tip: install cosign to verify binary attestation (https://docs.sigstore.dev/cosign/system_config/installation/)"

0 commit comments

Comments
 (0)