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

Make sure list of 'repositories' in the 'resolved.json.file' is complete. #981

Merged
merged 2 commits into from
Dec 14, 2023

Conversation

sfc-gh-ptabor
Copy link
Contributor

@sfc-gh-ptabor sfc-gh-ptabor commented Dec 11, 2023

Make sure list of 'repositories' in the 'resolved.json.file' is complete.

Prior to this PR, there were only listed used repositories used by any of the artifacts for given arch.
On the other hand the list was used to run APKO in offline environment -> and missing repos were leading to a failure
(in rules_apko).

Eventually repositories and indexes will not be needed for creation of image from resolved file, but, if we have the list in the file-format -> it should be complete.

@sfc-gh-ptabor sfc-gh-ptabor force-pushed the 20231211-all-repositories branch 3 times, most recently from 7d6fb0b to ee83d80 Compare December 11, 2023 11:20
…ete.

Prior to this PR, there were only listed used repositories by any of the artifact for given arch.
On the other hand the list was used to run APKO in off-line environment -> and missing repos were leading to failure
(in rules_apko).

Eventually repositories and indexes will not be needed for creation of image from resolved file,
but, if we have the list in the file-format -> it should be complete.

Signed-off-by: Piotr Tabor <[email protected]>
Signed-off-by: Jason Hall <[email protected]>
@imjasonh imjasonh merged commit abc45a4 into chainguard-dev:main Dec 14, 2023
20 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.

2 participants