Skip to content

update so that project can run on latest crictl version#183

Open
swermin wants to merge 3 commits intoIBM:mainfrom
swermin:feature/upgrade
Open

update so that project can run on latest crictl version#183
swermin wants to merge 3 commits intoIBM:mainfrom
swermin:feature/upgrade

Conversation

@swermin
Copy link
Contributor

@swermin swermin commented Mar 9, 2026

The current version of core-dump-handler doesn't work with Kubernetes 1.33 and beyond.
This PR updates the musl version, which is multiarch, to the latest version of crictl and also makes the crictl version an build-arg in case someone needs to update it.

Running the docker file seems to fail on compiling, so this PR also includes fixes to latest rust compatibility.

Signed-off-by: Ermin Hrkalovic <ermin.hrkalovic@outlook.com>
@swermin
Copy link
Contributor Author

swermin commented Mar 9, 2026

We only run the musl version, so I have no way of verifying if the regular version works.

Signed-off-by: Ermin Hrkalovic <ermin.hrkalovic@outlook.com>
No9
No9 previously approved these changes Mar 10, 2026
@swermin
Copy link
Contributor Author

swermin commented Mar 10, 2026

@No9 thanks for the approval! I see no way to merge this from my side, is this something you can do?

Dockerfile Outdated
FROM registry.access.redhat.com/ubi7/ubi as rhel7builder
FROM registry.access.redhat.com/ubi7/ubi AS rhel7builder

ARG CRICTL_VERSION=1.33.0

Choose a reason for hiding this comment

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

The ARG will need to be moved to rhel8builder to be visible since that is the layer in which it is used.

Signed-off-by: Ermin Hrkalovic <ermin.hrkalovic@guerrilla-games.com>
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