Skip to content
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

containerd and runc are included in two places that cause versions falling out of sync #252

Closed
2 tasks done
winkingturtle-vmw opened this issue Feb 13, 2023 · 1 comment · Fixed by cloudfoundry/guardian#344 or #253
Assignees

Comments

@winkingturtle-vmw
Copy link
Contributor

winkingturtle-vmw commented Feb 13, 2023

Description

Currently we have two versions of containerd under this repo:

  1. Submoduled under gopath
  2. Go Module under guardian

This is causing an issue whenever there is a bump for the module. We would have to make sure that the submodule is also bumped so that the two versions are in sync.

This same situation is also true for runc. As for runc, it looks like the versions have already diverged, where submoduled version is lacking behind moduled version in guardian.

Steps to reproduce

Logs

Docker image (if relevant)

Cause

Resolution

Modify guardian so that required binaries are all checked in under vendor directory. This would require changing packaging spec and ci helpers so that correct path of the binary is being built. With this change, we would get notified of any changes in upstream and able to patch both the guardian module and bosh-release to built the new version.

@winkingturtle-vmw winkingturtle-vmw self-assigned this Feb 13, 2023
winkingturtle-vmw added a commit to cloudfoundry/guardian that referenced this issue Feb 13, 2023
- ./bin/ctr
- ./bin/containerd
- ./bin/containerd-shim
- ./bin/containerd-shim-runc-v1
- ./bin/containerd-shim-runc-v2

Resolves cloudfoundry/garden-runc-release#252
winkingturtle-vmw added a commit that referenced this issue Feb 14, 2023
- Remove containerd submodule

Resolves #252
geofffranks pushed a commit to cloudfoundry/guardian that referenced this issue Feb 14, 2023
- ./bin/ctr
- ./bin/containerd
- ./bin/containerd-shim
- ./bin/containerd-shim-runc-v1
- ./bin/containerd-shim-runc-v2

Resolves cloudfoundry/garden-runc-release#252
winkingturtle-vmw added a commit that referenced this issue Feb 14, 2023
- Remove containerd submodule

Resolves #252
winkingturtle-vmw added a commit to cloudfoundry/guardian that referenced this issue Mar 3, 2023
so that it can be build from within garden-runc-release

addresses: cloudfoundry/garden-runc-release#252
winkingturtle-vmw added a commit that referenced this issue Mar 3, 2023
- build runc from guardian and bump version of runc to be the same
as what's running in containerd
- Fix sync-package-specs to include missing files

When running this script, make sure [gosub PR-12](vito/gosub#12)
if included as the minimum version

addresses: #252
geofffranks pushed a commit to cloudfoundry/guardian that referenced this issue Mar 3, 2023
so that it can be build from within garden-runc-release

addresses: cloudfoundry/garden-runc-release#252
@winkingturtle-vmw
Copy link
Contributor Author

Next releases will run runc version 1.1.4 upgraded from 1.0.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant