Skip to content

Conversation

@kiashok
Copy link
Contributor

@kiashok kiashok commented May 1, 2023

Welcome to the v1.6.21 release of release tool!

The twenty-first patch release for containerd 1.6 contains various fixes and updates.

Notable Updates

  • **update runc binary to v1.1.7 (#8450)
  • **Remove entry for container from container store on error (#8456)
  • **oci: partially restore comment on read-only mounts for uid/gid uses (#8403)
  • **windows: Add ArgsEscaped support for CRI (#8247)
  • **oci: Use WithReadonlyTempMount when adding users/groups (#8357)
  • **update go to go1.19.9 (#8469 )
  • **archive: consistently respect value of WithSkipDockerManifest (#8345)

See the changelog for complete list of changes

Please try out the release binaries and report any issues at
https://github.com/containerd/containerd/issues.

Contributors

  • Sebastiaan van Stijn
  • Derek McGowan
  • Iceber Gu
  • Kirtana Ashok
  • Justin Chadwell
  • Phil Estes
  • Akihiro Suda
  • Djordje Lukic
  • Kazuyoshi Kato
  • Mike Brown
  • Wei Fu
  • kiashok

Changes

25 commits

  • da91c3b07 Add release notes for v1.6.21
  • [release/1.6] update go to go1.19.9 (#8469)
    • 39566aade [release/1.6] update go to go1.19.9
  • [release/1.6] fix the task setting the runtime path (#8454)
    • e8840f688 skip TestContainerStartWithAbsRuntimePath if the runtime is v1
    • 75ab094de integration: add container start test using abs runtime path
    • f49254f0b WithRuntimePath uses the TaskInfo.RuntimePath field
  • [release/1.6 backport] update runc binary to v1.1.7 (#8450)
  • [release/1.6] Remove entry for container from container store on error (#8456)
    • 95d31551d Remove entry for container from container store on error
  • [release/1.6 backport] oci: partially restore comment on read-only mounts for uid/gid uses (#8403)
    • c33eb574d oci: partially restore comment on read-only mounts for uid/gid uses
  • [release/1.6 ] Add ArgsEscaped support for CRI (#8247)
  • [release/1.6 backport] update runc binary to v1.1.6 (#8385)
  • [release/1.6 backport] oci: Use WithReadonlyTempMount when adding users/groups (#8357)
    • fb5e663d0 oci: Use WithReadonlyTempMount when adding users/groups
  • [release/1.6] update go to go1.19.8 (#8353)
    • 26efb8fd5 [release/1.6] update go to go1.19.8
  • [release/1.6] archive: consistently respect value of WithSkipDockerManifest (#8345)
    • ec13b497e export: add test for WithSkipDockerManifest
    • d1f3771c4 archive: consistently respect value of WithSkipDockerManifest

Dependency Changes

This release has no dependency changes

Previous release can be found at v1.6.20

@k8s-ci-robot
Copy link

Hi @kiashok. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@kiashok
Copy link
Contributor Author

kiashok commented May 1, 2023

@dmcgowan @dcantah could you please take a look when you have some time? Thanks!

@dcantah
Copy link
Member

dcantah commented May 1, 2023

Copy link
Member

@dcantah dcantah left a comment

Choose a reason for hiding this comment

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

LGTM after the two bits of feedback

@dcantah
Copy link
Member

dcantah commented May 1, 2023

Might wanna wait for this actually #8454

@kiashok
Copy link
Contributor Author

kiashok commented May 2, 2023

Might wanna wait for this actually #8454

This PR has been checked in. Updated this PR to include this change and contributor

@estesp
Copy link
Member

estesp commented May 2, 2023

Still need to add this file to the commit with the version bump:

This needs a bump to 1.6.21 https://github.com/containerd/containerd/blob/release/1.6/version/version.go#L26

@kiashok kiashok force-pushed the addReleaseNotes-v.1.6.21 branch from 1599600 to fbb371f Compare May 3, 2023 12:49
@kiashok
Copy link
Contributor Author

kiashok commented May 3, 2023

Still need to add this file to the commit with the version bump:

This needs a bump to 1.6.21 https://github.com/containerd/containerd/blob/release/1.6/version/version.go#L26

resolved this!

Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM just a nit

@thaJeztah
Copy link
Member

Did we update the Go version to latest? There was a release Yesterday

@thaJeztah
Copy link
Member

@kiashok
Copy link
Contributor Author

kiashok commented May 3, 2023

Do we want to block this PR on go version update?

@kiashok kiashok force-pushed the addReleaseNotes-v.1.6.21 branch from fbb371f to 66453ea Compare May 3, 2023 18:47
@thaJeztah
Copy link
Member

I would personally consider making sure not to do a release with a go version that has CVE's attached (even though the actual CVE's wouldn't affect containerd).

There's also some fixes for compiler issues in the go1.19.9 release. We just received reports of random panics in Docker, which could be related to a bug in go1.19.8; see moby/moby#45454

@estesp
Copy link
Member

estesp commented May 3, 2023

I think we can probably merge the Go update in the next few hours; in that sense I don't think it is really holding up the release if we get it in today. I agree with @thaJeztah that it's easier to just make sure we are on the latest Go release that has CVE's associated with it to keep from complaints that we are using a runtime with known CVEs (even though we know containerd is unaffected)

@dmcgowan
Copy link
Member

dmcgowan commented May 3, 2023

I normally avoid putting mention of go updates into the release notes unless it has a relevant since it can be a bit of a moving target. Without, easy to just release with or without it merged and not worrying about updating this every time. Its easier to get these PRs merged quicker so it doesn't need a constant update. Think just remove the mention and we'll merge and do the release when the Go update is merged.

@thaJeztah
Copy link
Member

Ah, yes, that's fine with me; my comment was mostly "don't release before updating"; not specifically tied to including it in the changelog

Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

@kiashok kiashok force-pushed the addReleaseNotes-v.1.6.21 branch from 66453ea to 763bb4a Compare May 4, 2023 03:58
@kiashok
Copy link
Contributor Author

kiashok commented May 4, 2023

I normally avoid putting mention of go updates into the release notes unless it has a relevant since it can be a bit of a moving target. Without, easy to just release with or without it merged and not worrying about updating this every time. Its easier to get these PRs merged quicker so it doesn't need a constant update. Think just remove the mention and we'll merge and do the release when the Go update is merged.

updated PR!

@kiashok
Copy link
Contributor Author

kiashok commented May 4, 2023

@estesp the test failures here don't seem to be related to the changes being made here and I remember they passed earlier. Could you please help restart these tests? Thanks!

@thaJeztah
Copy link
Member

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@kiashok
Copy link
Contributor Author

kiashok commented May 5, 2023

do we need more sign offs for this PR to be merged? We are trying to get a new release out based on v1.6.21, so wanted to check :)

@kiashok
Copy link
Contributor Author

kiashok commented May 5, 2023

@dcantah if you have some time would you be able to take a look please? Thanks!

Copy link
Member

@dcantah dcantah left a comment

Choose a reason for hiding this comment

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

Sorry, missed the updates. LGTM

Signed-off-by: Kirtana Ashok <[email protected]>
@kiashok kiashok force-pushed the addReleaseNotes-v.1.6.21 branch from 763bb4a to 9226c36 Compare May 5, 2023 08:51
@dmcgowan dmcgowan merged commit 3dce8eb into containerd:release/1.6 May 5, 2023
aravindhp added a commit to openshift/containerd that referenced this pull request May 24, 2023
containerd 1.6.21

Welcome to the v1.6.21 release of containerd!

The twenty-first patch release for containerd 1.6 contains various fixes and updates.

* **update runc binary to v1.1.7 ([containerd#8450](containerd#8450))
* **Remove entry for container from container store on error ([containerd#8456](containerd#8456))
* **oci: partially restore comment on read-only mounts for uid/gid uses ([containerd#8403](containerd#8403))
* **windows: Add ArgsEscaped support for CRI ([containerd#8247](containerd#8247))
* **oci: Use WithReadonlyTempMount when adding users/groups ([containerd#8357](containerd#8357))
* **archive: consistently respect value of WithSkipDockerManifest ([containerd#8345](containerd#8345))

See the changelog for complete list of changes

Please try out the release binaries and report any issues at
https://github.com/containerd/containerd/issues.

* Derek McGowan
* Sebastiaan van Stijn
* Iceber Gu
* Kirtana Ashok
* Justin Chadwell
* Phil Estes
* Akihiro Suda
* Djordje Lukic
* Kazuyoshi Kato
* Mike Brown
* Wei Fu
* kiashok

<details><summary>26 commits</summary>
<p>

* [release/1.6] Prepare release notes for v1.6.21  ([containerd#8463](containerd#8463))
  * [`9226c362a`](containerd@9226c36) Add release notes for v1.6.21
* [release/1.6] update go to go1.19.9 ([containerd#8469](containerd#8469))
  * [`39566aade`](containerd@39566aa) [release/1.6] update go to go1.19.9
* [release/1.6] fix the task setting the runtime path ([containerd#8454](containerd#8454))
  * [`e8840f688`](containerd@e8840f6) skip TestContainerStartWithAbsRuntimePath if the runtime is v1
  * [`75ab094de`](containerd@75ab094) integration: add container start test using abs runtime path
  * [`f49254f0b`](containerd@f49254f) WithRuntimePath uses the TaskInfo.RuntimePath field
* [release/1.6 backport] update runc binary to v1.1.7 ([containerd#8450](containerd#8450))
  * [`ccb51ff26`](containerd@ccb51ff) update runc binary to v1.1.7
* [release/1.6] Remove entry for container from container store on error ([containerd#8456](containerd#8456))
  * [`95d31551d`](containerd@95d3155) Remove entry for container from container store on error
* [release/1.6 backport] oci: partially restore comment on read-only mounts for uid/gid uses ([containerd#8403](containerd#8403))
  * [`c33eb574d`](containerd@c33eb57) oci: partially restore comment on read-only mounts for uid/gid uses
* [release/1.6 ] Add ArgsEscaped support for CRI ([containerd#8247](containerd#8247))
  * [`bc2e01303`](containerd@bc2e013) Fix argsEscaped tests
  * [`8b81d5acc`](containerd@8b81d5a) Add ArgsEscaped support for CRI
* [release/1.6 backport] update runc binary to v1.1.6 ([containerd#8385](containerd#8385))
  * [`57d953482`](containerd@57d9534) update runc binary to v1.1.6
* [release/1.6 backport] oci: Use WithReadonlyTempMount when adding users/groups ([containerd#8357](containerd#8357))
  * [`fb5e663d0`](containerd@fb5e663) oci: Use WithReadonlyTempMount when adding users/groups
* [release/1.6] update go to go1.19.8 ([containerd#8353](containerd#8353))
  * [`26efb8fd5`](containerd@26efb8f) [release/1.6] update go to go1.19.8
* [release/1.6] archive: consistently respect value of WithSkipDockerManifest ([containerd#8345](containerd#8345))
  * [`ec13b497e`](containerd@ec13b49) export: add test for WithSkipDockerManifest
  * [`d1f3771c4`](containerd@d1f3771) archive: consistently respect value of WithSkipDockerManifest
</p>
</details>

This release has no dependency changes

Previous release can be found at [v1.6.20](https://github.com/containerd/containerd/releases/tag/v1.6.20)
Mengkzhaoyun pushed a commit to open-beagle/containerd that referenced this pull request Jun 9, 2023
containerd 1.6.21

Welcome to the v1.6.21 release of containerd!

The twenty-first patch release for containerd 1.6 contains various fixes and updates.

* **update runc binary to v1.1.7 ([#8450](containerd/containerd#8450))
* **Remove entry for container from container store on error ([#8456](containerd/containerd#8456))
* **oci: partially restore comment on read-only mounts for uid/gid uses ([#8403](containerd/containerd#8403))
* **windows: Add ArgsEscaped support for CRI ([#8247](containerd/containerd#8247))
* **oci: Use WithReadonlyTempMount when adding users/groups ([#8357](containerd/containerd#8357))
* **archive: consistently respect value of WithSkipDockerManifest ([#8345](containerd/containerd#8345))

See the changelog for complete list of changes

Please try out the release binaries and report any issues at
https://github.com/containerd/containerd/issues.

* Derek McGowan
* Sebastiaan van Stijn
* Iceber Gu
* Kirtana Ashok
* Justin Chadwell
* Phil Estes
* Akihiro Suda
* Djordje Lukic
* Kazuyoshi Kato
* Mike Brown
* Wei Fu
* kiashok
<details><summary>26 commits</summary>
<p>

* [release/1.6] Prepare release notes for v1.6.21  ([#8463](containerd/containerd#8463))
  * [`9226c362a`](containerd/containerd@9226c36) Add release notes for v1.6.21
* [release/1.6] update go to go1.19.9 ([#8469](containerd/containerd#8469))
  * [`39566aade`](containerd/containerd@39566aa) [release/1.6] update go to go1.19.9
* [release/1.6] fix the task setting the runtime path ([#8454](containerd/containerd#8454))
  * [`e8840f688`](containerd/containerd@e8840f6) skip TestContainerStartWithAbsRuntimePath if the runtime is v1
  * [`75ab094de`](containerd/containerd@75ab094) integration: add container start test using abs runtime path
  * [`f49254f0b`](containerd/containerd@f49254f) WithRuntimePath uses the TaskInfo.RuntimePath field
* [release/1.6 backport] update runc binary to v1.1.7 ([#8450](containerd/containerd#8450))
  * [`ccb51ff26`](containerd/containerd@ccb51ff) update runc binary to v1.1.7
* [release/1.6] Remove entry for container from container store on error ([#8456](containerd/containerd#8456))
  * [`95d31551d`](containerd/containerd@95d3155) Remove entry for container from container store on error
* [release/1.6 backport] oci: partially restore comment on read-only mounts for uid/gid uses ([#8403](containerd/containerd#8403))
  * [`c33eb574d`](containerd/containerd@c33eb57) oci: partially restore comment on read-only mounts for uid/gid uses
* [release/1.6 ] Add ArgsEscaped support for CRI ([#8247](containerd/containerd#8247))
  * [`bc2e01303`](containerd/containerd@bc2e013) Fix argsEscaped tests
  * [`8b81d5acc`](containerd/containerd@8b81d5a) Add ArgsEscaped support for CRI
* [release/1.6 backport] update runc binary to v1.1.6 ([#8385](containerd/containerd#8385))
  * [`57d953482`](containerd/containerd@57d9534) update runc binary to v1.1.6
* [release/1.6 backport] oci: Use WithReadonlyTempMount when adding users/groups ([#8357](containerd/containerd#8357))
  * [`fb5e663d0`](containerd/containerd@fb5e663) oci: Use WithReadonlyTempMount when adding users/groups
* [release/1.6] update go to go1.19.8 ([#8353](containerd/containerd#8353))
  * [`26efb8fd5`](containerd/containerd@26efb8f) [release/1.6] update go to go1.19.8
* [release/1.6] archive: consistently respect value of WithSkipDockerManifest ([#8345](containerd/containerd#8345))
  * [`ec13b497e`](containerd/containerd@ec13b49) export: add test for WithSkipDockerManifest
  * [`d1f3771c4`](containerd/containerd@d1f3771) archive: consistently respect value of WithSkipDockerManifest
</p>
</details>

This release has no dependency changes

Previous release can be found at [v1.6.20](https://github.com/containerd/containerd/releases/tag/v1.6.20)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants