Skip to content

Conversation

@anlexN
Copy link
Contributor

@anlexN anlexN commented Aug 13, 2025

Add "uninstall" script to get file and add "uninstall" markdown to README.md

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?

None

@openshift-ci openshift-ci bot added the dco-signoff: no Indicates the PR's author has not DCO signed all their commits. label Aug 13, 2025
@openshift-ci openshift-ci bot added dco-signoff: yes Indicates the PR's author has DCO signed all their commits. and removed dco-signoff: no Indicates the PR's author has not DCO signed all their commits. labels Aug 13, 2025
Copy link
Member

@saschagrunert saschagrunert left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution! 🙏 Do you mind fixing the CI?

@anlexN
Copy link
Contributor Author

anlexN commented Aug 13, 2025

Thank you for the contribution! 🙏 Do you mind fixing the CI?

wait for me.

@anlexN
Copy link
Contributor Author

anlexN commented Aug 13, 2025

@saschagrunert I have fixed the CI, please you reviewed again...

@anlexN anlexN requested a review from saschagrunert August 14, 2025 08:05
@anlexN
Copy link
Contributor Author

anlexN commented Aug 14, 2025

Is it Ok?

@saschagrunert
Copy link
Member

Is it Ok?

Ah no, we have a CI issue here: https://github.com/cri-o/packaging/actions/runs/16941281896/job/48071296575?pr=280

@anlexN
Copy link
Contributor Author

anlexN commented Aug 14, 2025

@saschagrunert I see. This time, I fix it again, it should pass. please you review again.

Add "uninstall" script to `get` file, "uninstall" markdown to `README.md`, "remember install path" script to `templates/latest/cri-o/bundle/install`.

Signed-off-by: anlex N <[email protected]>
@anlexN
Copy link
Contributor Author

anlexN commented Aug 14, 2025

@saschagrunert, excuse me, are you sure this fail is caused by me? it display:

Progress: 0% (Rate: 0*/s, Estimated time remaining: --:--:--)
The box failed to unpackage properly. Please verify that the box
file you're trying to add is not corrupted and that enough disk space
is available and then try again.

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Aug 14, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 14, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anlexN, saschagrunert

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 14, 2025
@anlexN
Copy link
Contributor Author

anlexN commented Aug 15, 2025

@klihub @sohankunkerkar hello, what else do I need to do?

@openshift-merge-bot openshift-merge-bot bot merged commit d7e31b5 into cri-o:main Aug 15, 2025
112 of 116 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. release-note-none

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants