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
podman-provider: get actually available cgroupControllers #2511
podman-provider: get actually available cgroupControllers #2511
Conversation
Welcome @flouthoc! |
Hi @flouthoc. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
that patch seems to affect only 3.3.0 and onwards, what happen with 3.2.0 and previous versions? |
@aojea I dont think it will be backported. Marking PR as draft till we get confirmation on previous versions. |
let's run it through the CI, I know we have some checks for the podman version, I'll take a look later this week |
@aojea you have to click "approve and run" to run github actions, prow does not run podman and does not trigger github actions |
what versions? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only makes sense for Podman v4: containers/podman#11935
For Podman v3 we can't rely on podman info
(unless
containers/podman#11935 gets backported to v3)
@AkihiroSuda makes sense since i don't think it will be backported. Should let the PR dangle in draft mode or close ? |
Maybe parse Podman version, and apply the new logic only when the version is newer than 4.0.0 ? |
there are already some helpers kind/pkg/cluster/internal/providers/podman/provider.go Lines 184 to 192 in dbe461f
|
8d68103
to
9de6ece
Compare
@aojea @AkihiroSuda thanks for the suggestion made changes PTAL :) |
Signed-off-by: flouthoc <flouthoc.git@gmail.com>
9de6ece
to
151dba2
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: AkihiroSuda, aojea, flouthoc The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi Team,
Following (containers/podman#10387) got rolled out in multiple versions. I think we can remove the hard-codes.