Skip to content

Fetch app bundle index #389

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
mike-sul merged 2 commits into from
May 28, 2025
Merged

Fetch app bundle index #389

mike-sul merged 2 commits into from
May 28, 2025

Conversation

@mike-sul
Copy link
Contributor

@mike-sul mike-sul commented May 28, 2025 •
edited
Loading

  • Make sure app image manifests are fetched and stored intact, so their content hashes matches the hashes in references to them.
  • Fetch the app bundle index file if an app manifest includes a reference to it.

mike-sul added 2 commits May 28, 2025 12:46
@mike-sul
apps: Preserve app image digest
b8a6992 
Make `skopeo` preserve the original container image manifest so its hash
matches the hash specified in its digest reference in an app compose
project or in an image index.

Signed-off-by: Mike Sul <mike.sul@foundries.io>
@mike-sul
apps: Fetch app bundle index
26b74d1 
Fetch app bundle index if it is reference is found in an app manifest.
It allows checking app bundle integrity during offline update.

Signed-off-by: Mike Sul <mike.sul@foundries.io>
@mike-sul mike-sul marked this pull request as ready for review May 28, 2025 13:13
@mike-sul mike-sul requested review from detsch and doanac May 28, 2025 13:13
@mike-sul mike-sul merged commit fc9ce13 into master May 28, 2025
2 checks passed
@mike-sul mike-sul deleted the fetch-app-bundle-index branch May 28, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@doanac doanac doanac approved these changes

@detsch detsch detsch 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.

4 participants

@mike-sul @doanac @detsch