Skip to content

test: add a bom test for derived images #483

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 5, 2023
Merged

test: add a bom test for derived images #483

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.

@rchincha rchincha requested review from smoser and hallyn as code owners August 21, 2023 18:17
@rchincha rchincha force-pushed the sbom branch 2 times, most recently from b3568e8 to cb9cae0 Compare August 22, 2023 15:57
smoser
smoser requested changes Aug 22, 2023
View reviewed changes
smoser
smoser reviewed Aug 22, 2023
View reviewed changes
smoser
smoser requested changes Aug 22, 2023
View reviewed changes
@rchincha rchincha force-pushed the sbom branch 3 times, most recently from 8a406d4 to 2240247 Compare August 23, 2023 17:43
@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
test: add a bom test for derived images
5142e0a 
Signed-off-by: Ramkumar Chinchani <rchincha@cisco.com>
hallyn
hallyn reviewed Sep 5, 2023
View reviewed changes
hallyn
hallyn reviewed Sep 5, 2023
View reviewed changes
fix: refactor code and also upload empty config blob
496368e 
Signed-off-by: Ramkumar Chinchani <rchincha@cisco.com>
@rchincha rchincha merged commit fdddd5b into project-stacker:main Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smoser smoser smoser requested changes

@hallyn hallyn hallyn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rchincha @smoser @hallyn