-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
runtime: parse conmon-rs version #15340
runtime: parse conmon-rs version #15340
Conversation
371a977
to
39b8330
Compare
changed to use the version from containers/common, but now we need containers/common#1121 before the original issue is addressed |
golang is mad 😢 |
39b8330
to
4afc20e
Compare
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
4afc20e
to
91752f8
Compare
ready for review |
it detects conmon-rs. [NO NEW TESTS NEEDED] no functionalities added. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
91752f8
to
20ad122
Compare
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.
LGTM
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flouthoc, giuseppe 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 |
/lgtm |
detect the conmon-rs version string and if detected disable version
check for now.
Closes: #15335
Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com
Does this PR introduce a user-facing change?