Skip to content

More strict hermetic image download#1548

Merged
xsuchy merged 1 commit intorpm-software-management:mainfrom
praiskup:praiskup-architecture
Feb 19, 2025
Merged

More strict hermetic image download#1548
xsuchy merged 1 commit intorpm-software-management:mainfrom
praiskup:praiskup-architecture

Conversation

@praiskup
Copy link
Member

@praiskup praiskup commented Feb 15, 2025

hermetic: more image-pedantic work with bootstrap

When producing the buildroot lockfile, include additional bootstrap
image metadata that can be later used for a precise image pull.

The mock-hermetic-repo script is also modified here to respect the
additional metadata.  This allows us to, e.g., download bootstrap image
of a different (cross) architecture then the platform/host architecture
is.  In turn, the script is now fully arch-agnostic (any host arch may
be used for downloading files from any arch specific lockfile).

Closes: #1548

@praiskup praiskup marked this pull request as draft February 15, 2025 18:59
When producing the buildroot lockfile, include additional bootstrap
image metadata that can be later used for a precise image pull.

The mock-hermetic-repo script is also modified here to respect the
additional metadata.  This allows us to, e.g., download bootstrap image
of a different (cross) architecture then the platform/host architecture
is.  In turn, the script is now fully arch-agnostic (any host arch may
be used for downloading files from any arch specific lockfile).

Closes: rpm-software-management#1548
@praiskup praiskup force-pushed the praiskup-architecture branch from 6f693f8 to 6048fb3 Compare February 15, 2025 20:19
@praiskup praiskup marked this pull request as ready for review February 15, 2025 20:20
@praiskup
Copy link
Member Author

@tkopecek can you please take a look? This should allow us to have a separate Tekton Task for fetching rpms/boostrap image (one for all architectures, and can be run pod-natively).

@tkopecek
Copy link
Contributor

Works for me!

Copy link
Member

@xsuchy xsuchy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xsuchy xsuchy merged commit 8561d18 into rpm-software-management:main Feb 19, 2025
24 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