Skip to content

Conversation

@mtrmac
Copy link
Contributor

@mtrmac mtrmac commented Oct 21, 2022

go mod edit -replace github.com/containers/storage=github.com/mtrmac/storage@cleanup-in-readers
make vendor

What type of PR is this?

/kind bug

What this PR does / why we need it:

How to verify it

TBD

Which issue(s) this PR fixes:

None

Special notes for your reviewer:

Does this PR introduce a user-facing change?

Improves resiliency against interrupted image pulls.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 21, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mtrmac
Once this PR has been reviewed and has the lgtm label, please assign rhatdan for approval by writing /assign @rhatdan in a comment. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details 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-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 30, 2022
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 1, 2022
@mtrmac mtrmac force-pushed the cleanup-test branch 2 times, most recently from 2d1bf6c to bb7171a Compare November 10, 2022 15:23
> go mod edit -replace github.com/containers/storage=github.com/mtrmac/storage@cleanup-in-readers
> make vendor

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 13, 2022
@openshift-merge-robot
Copy link
Collaborator

@mtrmac: PR needs rebase.

Details

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.

@mtrmac
Copy link
Contributor Author

mtrmac commented Nov 14, 2022

The c/storage PR was merged.

@mtrmac mtrmac closed this Nov 14, 2022
@mtrmac mtrmac deleted the cleanup-test branch November 14, 2022 20:15
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

do-not-merge/work-in-progress locked - please file new issue/PR needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants