Skip to content

github: install dependencies to run snapshot update in images#155

Merged
thozza merged 1 commit intomainfrom
images/run-in-container-with-deps
Feb 11, 2026
Merged

github: install dependencies to run snapshot update in images#155
thozza merged 1 commit intomainfrom
images/run-in-container-with-deps

Conversation

@achilleas-k
Copy link
Member

@achilleas-k achilleas-k commented Feb 11, 2026

Follow-up from #153.

When updating the repo snapshots in images, we need to regenerate manifest checksums, which requires compiling cmd/gen-manifests. Install the necessary dependencies to run the tool.

@achilleas-k achilleas-k requested a review from a team as a code owner February 11, 2026 15:34
@achilleas-k achilleas-k requested review from croissanne, ondrejbudai and thozza and removed request for a team February 11, 2026 15:34
@achilleas-k achilleas-k marked this pull request as draft February 11, 2026 15:42
@achilleas-k achilleas-k force-pushed the images/run-in-container-with-deps branch from 01e5a3a to 62f03b2 Compare February 11, 2026 15:44
When updating the repo snapshots in images, we need to regenerate
manifest checksums, which requires compiling cmd/gen-manifests.  Install
the necessary dependencies to run the tool.
@achilleas-k achilleas-k force-pushed the images/run-in-container-with-deps branch 3 times, most recently from 58b8de1 to e4a445c Compare February 11, 2026 15:52
@achilleas-k
Copy link
Member Author

@achilleas-k achilleas-k force-pushed the images/run-in-container-with-deps branch from e4a445c to 7e1e195 Compare February 11, 2026 15:55
@achilleas-k achilleas-k marked this pull request as ready for review February 11, 2026 15:55
Copy link

@tkoscieln tkoscieln left a comment

Choose a reason for hiding this comment

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

lgtm

@thozza thozza merged commit 6ed4920 into main Feb 11, 2026
5 of 6 checks passed
@thozza thozza deleted the images/run-in-container-with-deps branch February 11, 2026 16:02
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