-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Release 1.2.0-rc.2 #4317
Release 1.2.0-rc.2 #4317
Conversation
00b81bf
to
83326a3
Compare
83326a3
to
aa78741
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for creating this PR so promptly!
It would also LGTM without listing the changes for ci, dependencies and tests. But I think like this is fine, no need to change it :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some notes.
Additionally, we need to release 1.1.13 first and then incorporate 1.1.13 release notes here, and make sure there are no overlaps.
draft until 1.1.13 is released |
Yes, I'll address all your requested changes and update the 1.1.13 release notes after v1.1.13 released. |
It's time! :) |
9a8086e
to
363b3a1
Compare
@kolyshkin Ready to go! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did some nit comments. But this mostly LGTM
363b3a1
to
8e6c2b9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!!
@lifubang can we fix the changelog to be sorted by version number, not by the release date? And change the release date, to, say, next Tuesday (25th of June). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs a rebase
8e6c2b9
to
bf448c4
Compare
c1a5325
to
93ea985
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I took some time, I was afk.
Added some nits, but this mostly LGTM.
93ea985
to
aff5fdf
Compare
Signed-off-by: lifubang <[email protected]>
Signed-off-by: lifubang <[email protected]>
aff5fdf
to
5ea7625
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Let's see if we can still make the release today, as the changelog says 🤞
|
||
* cgroup v2: do not set swap to 0 or unlimited when it's not available. (#4188) | ||
* Set the default value of CpuBurst to nil instead of 0. (#4210, #4211) | ||
* runc exec: fix setting rlimit_nofile. (#4195, #4265, #4282, #4290) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not be here since it's also in v1.1.13 but it's too late to fix this. I will remove this line in the github release notes.
* cgroup v2: do not set swap to 0 or unlimited when it's not available. (#4188) | ||
* Set the default value of CpuBurst to nil instead of 0. (#4210, #4211) | ||
* runc exec: fix setting rlimit_nofile. (#4195, #4265, #4282, #4290) | ||
* libct/cg/fs: fix setting rt_period vs rt_runtime. (#4094, #4258) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here. Sorry I overlooked this.
* Set the default value of CpuBurst to nil instead of 0. (#4210, #4211) | ||
* runc exec: fix setting rlimit_nofile. (#4195, #4265, #4282, #4290) | ||
* libct/cg/fs: fix setting rt_period vs rt_runtime. (#4094, #4258) | ||
* script/*: fix gpg usage wrt keyboxd. (#4189) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
* runc exec: fix setting rlimit_nofile. (#4195, #4265, #4282, #4290) | ||
* libct/cg/fs: fix setting rt_period vs rt_runtime. (#4094, #4258) | ||
* script/*: fix gpg usage wrt keyboxd. (#4189) | ||
* Fix a debug log message for user namespaces in nsexec. (#4311) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
|
||
### Changed | ||
|
||
* libcontainer: allow containers to make apps think fips is enabled/disabled for testing. (#4246) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here, it's in 1.1.13.
[1.2.0-rc.2] - 2024-06-26
Important Notes
import libcontainer/cgroups/devices. (Decouple setting cgroup device rules from cgroup manager #3452, libct: decouple libct/cg/devices #4248)
(see runc doesn't work with go1.22 #4233 for more details)
Added
Fixed
Changed