Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

Upgrade Runc in main from 1.0.0_rc90 to 1.0.0_rc91 #500

Closed
wants to merge 2 commits into from

Conversation

github-actions[bot]
Copy link

Upgrade Runc in main from 1.0.0_rc90 to 1.0.0_rc91

@github-actions github-actions bot added the main label Jul 30, 2020
To fix builds, adapt patches for runc 1.0.0-rc91.
@dongsupark
Copy link
Contributor

Fixed build issues.

# $ git log --oneline v1.0.0-rc90..${COMMIT_ID} | wc -l
COMMIT_ID="dc9208a3303feef5b3839f4323d9beb36df0a9dd"
# $ git log --oneline v1.0.0-rc91..${COMMIT_ID} | wc -l
COMMIT_ID="42fff598cfbc9bd49f1d9d78e1e9d78b0921ddb0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this commit ID wrong? opencontainers/runc@42fff59 gives 404. In the releases page, rc91 is at 24a3cf88a7ae5f4995f6750654c0e2ca61ef4bb2.

Copy link
Contributor

Choose a reason for hiding this comment

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

To expand on it - I've got a build failure with the following error:

--2020-08-04 19:52:30--  https://github.com/opencontainers/runc/archive/42fff598cfbc9bd49f1d9d78e1e9d78b0921ddb0.tar.gz
Resolving github.com... 140.82.118.3
Connecting to github.com|140.82.118.3|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://codeload.github.com/opencontainers/runc/tar.gz/master [following]
--2020-08-04 19:52:31--  https://codeload.github.com/opencontainers/runc/tar.gz/master
…
…
…
!!! Fetched file: docker-runc-1.0.0_rc91.tar.gz VERIFY FAILED!
!!! Reason: Filesize does not match recorded size
!!! Got:      2061612
!!! Expected: 1939849
Refetching... File renamed to '/mnt/host/source/.cache/distfiles/docker-runc-1.0.0_rc91.tar.gz._checksum_failure_.mL04zx'

!!! Couldn't download 'docker-runc-1.0.0_rc91.tar.gz'. Aborting.

Notice that github does some redirect to the archive of the master branch.

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this commit ID wrong? opencontainers/runc@42fff59 gives 404. In the releases page, rc91 is at 24a3cf88a7ae5f4995f6750654c0e2ca61ef4bb2.

Both 42fff598cfbc9bd49f1d9d78e1e9d78b0921ddb0 and 24a3cf88a7ae5f4995f6750654c0e2ca61ef4bb2 are correct. Both commits point to v1.0.0-rc91. However, the former points to a commit with GPG signature, while the latter represents a commit without GPG signature.

Github actions create the commit by running git rev-parse refs/tags/v1.0.0-rc91. That's why we see the result commit 42fff598cfbc9bd49f1d9d78e1e9d78b0921ddb0 with GPG sig. If we would want to make the commit point to the commit without GPG sig, then we need to run instead git rev-parse refs/tags/v1.0.0-rc91^{commit}.

$ git clone https://github.com/opencontainers/runc
$ cd runc
$ git rev-parse refs/tags/v1.0.0-rc91
42fff598cfbc9bd49f1d9d78e1e9d78b0921ddb0
$ git rev-parse refs/tags/v1.0.0-rc91^{commit}
24a3cf88a7ae5f4995f6750654c0e2ca61ef4bb2

!!! Fetched file: docker-runc-1.0.0_rc91.tar.gz VERIFY FAILED!
!!! Reason: Filesize does not match recorded size
!!! Got: 2061612
!!! Expected: 1939849
Refetching... File renamed to '/mnt/host/source/.cache/distfiles/docker-runc-1.0.0_rc91.tar.gz.checksum_failure.mL04zx'

Both URLs are valid, and should work fine, https://github.com/opencontainers/runc/archive/24a3cf88a7ae5f4995f6750654c0e2ca61ef4bb2.tar.gz and https://github.com/opencontainers/runc/archive/42fff598cfbc9bd49f1d9d78e1e9d78b0921ddb0.tar.gz .

The checksum error above is another issue.
It happens probably because some garbage tarballs remains under ~/trunk/.cache/distfiles/.
Please try cleaning up all files like ~/trunk/.cache/distfiles/docker-runc-1.0.0_rc91.tar.gz and reinstalling docker-runc.

@dongsupark
Copy link
Contributor

Closing in favor of #525

@dongsupark dongsupark closed this Aug 13, 2020
@dongsupark dongsupark deleted the runc-1.0.0_rc91-main branch August 13, 2020 11:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants