Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add a bom test for derived images #483

Merged
merged 2 commits into from
Sep 5, 2023

Conversation

rchincha
Copy link
Contributor

What type of PR is this?

Which issue does this PR fix:

What does this PR do / Why do we need it:

If an issue # is not available please add repro steps and logs showing the issue:

Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?

Does this PR introduce any user-facing change?:


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

test/bom.bats Outdated Show resolved Hide resolved
test/bom.bats Outdated Show resolved Hide resolved
test/bom.bats Outdated Show resolved Hide resolved
@rchincha
Copy link
Contributor Author

publishing bom-good.yaml bom-parent to docker://localhost:8080/bom-parent:latest

Copying blob a1d0c7532777 skipped: already exists
Copying blob c80fd6526a2d skipped: already exists
Copying config 3b49bea002 done
Writing manifest to image destination
Storing signatures
artifact '/tmp/stacker/.stacker/artifacts/bom-parent/bom-parent.json' sucessfully uploaded to url:http://localhost:8080/v2/bom-parent/manifests/sha256:99d732817fdb2c707ff140b399bbc7981880618b4dea647d759921b069517057
artifact '/tmp/stacker/.stacker/artifacts/bom-parent/inventory.json' sucessfully uploaded to url:http://localhost:8080/v2/bom-parent/manifests/sha256:a6334f2ed74517fdc7bef5a04ffa1d31f86782547bc5551eed00a63b5610d8fa

I am able to test this locally. However, in our CI ..

2023-08-23T18:48:15.9910388Z # usernsexec-ing [u 0 1001 1 1 165537 65535 g 0 123 1 1 165537 65535 -- /home/runner/work/stacker/stacker/stacker --internal-userns --debug publish --skip-tls --url docker://localhost:5000 --tag latest]
2023-08-23T18:48:15.9910592Z # stacker version 5b6c4e1
2023-08-23T18:48:15.9910779Z # initializing stacker recipe: stacker.yaml
2023-08-23T18:48:15.9911307Z # substituting $STACKER_ROOTFS_DIR to /home/runner/work/stacker/stacker/stackertest-test_bom_tool_should_work_inside_run.nexivr/roots
2023-08-23T18:48:15.9911849Z # substituting $STACKER_STACKER_DIR to /home/runner/work/stacker/stacker/stackertest-test_bom_tool_should_work_inside_run.nexivr/.stacker
2023-08-23T18:48:15.9912357Z # substituting $STACKER_OCI_DIR to /home/runner/work/stacker/stacker/stackertest-test_bom_tool_should_work_inside_run.nexivr/oci
2023-08-23T18:48:15.9912517Z # substituting $STACKER_WORK_DIR to
2023-08-23T18:48:15.9912903Z # publishing stacker.yaml bom-parent to docker://localhost:5000/bom-parent:latest
2023-08-23T18:48:15.9912995Z #
2023-08-23T18:48:15.9913440Z # subject:http://localhost:5000/v2/bom-parent/manifests/latest doesn't exist, ignoring and proceeding
2023-08-23T18:48:15.9913880Z # unable to get descriptor details for subject:http://localhost:5000/v2/bom-parent/manifests/latest

@rchincha
Copy link
Contributor Author

Maybe zot is not up

@rchincha rchincha force-pushed the sbom branch 11 times, most recently from dd076c7 to 292b090 Compare August 24, 2023 19:10
Signed-off-by: Ramkumar Chinchani <rchincha@cisco.com>
pkg/stacker/publisher.go Outdated Show resolved Hide resolved
Signed-off-by: Ramkumar Chinchani <rchincha@cisco.com>
@rchincha rchincha merged commit fdddd5b into project-stacker:main Sep 5, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants