Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[1.19] vendor: bump containers/storage to v1.20.5 #4337

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.

Sign up for GitHub

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

Jump to bottom
Closed
kolyshkin wants to merge 1 commit into from
Closed

[1.19] vendor: bump containers/storage to v1.20.5 #4337

kolyshkin wants to merge 1 commit into from

Conversation

@kolyshkin
Copy link
Collaborator

@kolyshkin kolyshkin commented Nov 2, 2020 •
edited
Loading

What type of PR is this?

/kind bug

What this PR does / why we need it:

(this is similar to #4334 but for 1.19 release branch)

This is mostly to have the backport of containers/storage#757 brought in
which should fix occasional CI failures (and probably real failures, too).

Which issue(s) this PR fixes:

time="2020-11-02T19:00:03Z" level=fatal msg="creating container: rpc error: code = Unknown desc = error creating an ID-mapped copy of layer "01d6f430ddfdf1203b505fbb03c6479802a3366609d5e0231a30f85f412f873d": exit status 1: error during chown: storage-chown-by-maps: chown("lib/apk/db/scripts.tar"): lchown lib/apk/db/scripts.tar: interrupted system call"

periodically caught by CI

Special notes for your reviewer:

None

Does this PR introduce a user-facing change?

Fix occasional "chown: interrupted system call" error on container creation.

@kolyshkin
[1.19] vendor: bump containers/storage to v1.23.8
a9ff8a5 
This is mostly to have containers/storage#757
which should fix occasional CI failures.

Manual backport of commit 85c665a.

Signed-off-by: Kir Kolyshkin <[email protected]>
@openshift-ci-robot openshift-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. kind/bug Categorizes issue or PR as related to a bug. labels Nov 2, 2020
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 2, 2020

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kolyshkin
To complete the pull request process, please assign haircommander after the PR has been reviewed.
You can assign the PR to them by writing /assign @haircommander in a comment when ready.

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

@kolyshkin kolyshkin mentioned this pull request Nov 2, 2020
Closed
@haircommander
Copy link
Member

haircommander commented Nov 2, 2020

I think for 1.19 we should stick to the closest stable branch: https://github.com/containers/storage/tree/v1.20-stable

@openshift-merge-robot
Copy link
Contributor

openshift-merge-robot commented Nov 2, 2020

@kolyshkin: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/openshift-jenkins/integration_crun_cgroupv2 a9ff8a5 link /test integration_cgroupv2

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@kolyshkin kolyshkin mentioned this pull request Nov 2, 2020
Merged
@kolyshkin
Copy link
Collaborator Author

kolyshkin commented Nov 2, 2020

Ah OK; we need a release for that first (1.20.5 or so).

@kolyshkin kolyshkin closed this Nov 2, 2020
@kolyshkin kolyshkin changed the title [1.19] vendor: bump containers/storage to v1.23.8 [1.19] vendor: bump containers/storage to v1.20.5 Nov 3, 2020
@kolyshkin
Copy link
Collaborator Author

kolyshkin commented Nov 3, 2020

Re-implemented with 1.20.5

@kolyshkin
Copy link
Collaborator Author

kolyshkin commented Nov 3, 2020

Ahhh, can't reopen this one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mrunalp mrunalp Awaiting requested review from mrunalp mrunalp is a code owner

@runcom runcom Awaiting requested review from runcom runcom is a code owner

@giuseppe giuseppe Awaiting requested review from giuseppe

@haircommander haircommander Awaiting requested review from haircommander

Assignees

No one assigned

Labels

dco-signoff: yes Indicates the PR's author has DCO signed all their commits. kind/bug Categorizes issue or PR as related to a bug. release-note Denotes a PR that will be considered when it comes time to generate release notes.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kolyshkin @openshift-ci-robot @haircommander @openshift-merge-robot