-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Comparing changes
Open a pull request
base repository: opencontainers/runc
base: v1.2.1
head repository: opencontainers/runc
compare: v1.2.2
- 18 commits
- 11 files changed
- 6 contributors
Commits on Nov 1, 2024
-
Signed-off-by: Rodrigo Campos <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 5fc3558 - Browse repository at this point
Copy the full SHA 5fc3558View commit details -
merge #4500 into opencontainers/runc:release-1.2
Rodrigo Campos (2): VERSION: back to development VERSION: release v1.2.1 LGTMs: AkihiroSuda kolyshkin cyphar
Configuration menu - View commit details
-
Copy full SHA for 7a8738e - Browse repository at this point
Copy the full SHA 7a8738eView commit details
Commits on Nov 2, 2024
-
docs: remove prompt symbols from shell snippets
Remove prompt symbols (`$`, `%`) for ease of copy-pasting Signed-off-by: Akihiro Suda <[email protected]> (cherry picked from commit c8f5d03) Signed-off-by: Austin Vazquez <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 6575ab1 - Browse repository at this point
Copy the full SHA 6575ab1View commit details
Commits on Nov 4, 2024
-
merge #4507 into opencontainers/runc:release-1.2
Akihiro Suda (1): docs: remove prompt symbols from shell snippets LGTMs: AkihiroSuda cyphar
Configuration menu - View commit details
-
Copy full SHA for 2327ec2 - Browse repository at this point
Copy the full SHA 2327ec2View commit details
Commits on Nov 13, 2024
-
Vagrantfile.fedora: bump Fedora to 41
Signed-off-by: Akihiro Suda <[email protected]> (cherry picked from commit 9ce7392) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 2d6fe54 - Browse repository at this point
Copy the full SHA 2d6fe54View commit details -
Vagrantfile.fedora: stop using dnf shell
In Fedora 41, dnf5 is used and it does not have dnf shell. Let's use old dnf update; dnf install instead. It is two transactions instead of one, but dnf5 is faster. While at it: - add `--setopt=tsflags=nodocs` as we don't need docs in CI; - change golang-go to golang as this is a new rpm name; - remove gcc as it is now required by golang-bin; - remove container-selinux, criu, fuse-sshfs, iptables from rpms as they are already installed. Signed-off-by: Kir Kolyshkin <[email protected]> (cherry picked from commit 609e9a5) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 213e121 - Browse repository at this point
Copy the full SHA 213e121View commit details -
runc delete: fix for rootless cgroup + ro cgroupfs
An issue with runc 1.2.0 was reported to buildkit, in which runc delete returns with an error, with the log saying: > unable to destroy container: unable to remove container's cgroup: open /sys/fs/cgroup/snschvixiy3s74w74fjantrdg: no such file or directory Apparently, what happens is runc is running with no cgroup access (because /sys/fs/cgroup is mounted read-only). In this case error to create a cgroup path (in runc create/run) is ignored, but cgroup removal (in runc delete) is not. This is caused by commit d3d7f7d, which changes the cgroup removal logic in RemovePath. In the current code, if the initial rmdir has failed (in this case with EROFS), but the subsequent os.ReadDir returns ENOENT, it is returned (instead of being ignored -- as the path does not exist and so there is nothing to remove). Here is the minimal fix for the issue. Signed-off-by: Kir Kolyshkin <[email protected]> (cherry picked from commit db59489) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for e01db00 - Browse repository at this point
Copy the full SHA e01db00View commit details -
libct/cg: RemovePath: simplify logic
If the sub-cgroup RemovePath has failed for any reason, return the error right away. This way, we don't have to check for err != nil before retrying rmdir. This is a cosmetic change and should not change any functionality. Signed-off-by: Kir Kolyshkin <[email protected]> (cherry picked from commit 12e06a7) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 79cdf11 - Browse repository at this point
Copy the full SHA 79cdf11View commit details -
libct/cg: RemovePath: improve comments
Let's explain in greater details what's happening here and why. Signed-off-by: Kir Kolyshkin <[email protected]> (cherry picked from commit ba3d026) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for e41cc27 - Browse repository at this point
Copy the full SHA e41cc27View commit details
Commits on Nov 14, 2024
-
Merge pull request #4528 from lifubang/backport-4505
[1.2] CI: bump Fedora 40 -> 41
Configuration menu - View commit details
-
Copy full SHA for 8bfc3b5 - Browse repository at this point
Copy the full SHA 8bfc3b5View commit details
Commits on Nov 15, 2024
-
libct/cg: add test for remove a non-existent dir in a ro mount point
Signed-off-by: lfbzhm <[email protected]> (cherry picked from commit 119111a) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 832faf0 - Browse repository at this point
Copy the full SHA 832faf0View commit details -
Merge pull request #4531 from lifubang/backport-4523
[1.2] runc delete: fix for rootless cgroup + ro cgroupfs
Configuration menu - View commit details
-
Copy full SHA for 1b42ebc - Browse repository at this point
Copy the full SHA 1b42ebcView commit details -
dmz: overlay: set xino=off to disable dmesg spam
If /run/runc and /usr/bin are on different filesystems, overlayfs may enable the xino feature which results in the following log message: kernel: overlayfs: "xino" feature enabled using 3 upper inode bits. Each time we have to protect /proc/self/exe. So disable xino to remove the log message (we don't care about the inode numbers of the files anyway). Signed-off-by: Aleksa Sarai <[email protected]> (cherry picked from commit 9bc42d6) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for b1f733b - Browse repository at this point
Copy the full SHA b1f733bView commit details -
memfd-bind: mention that overlayfs obviates the need for it
Signed-off-by: Aleksa Sarai <[email protected]> (cherry picked from commit aa505bf) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 2421b59 - Browse repository at this point
Copy the full SHA 2421b59View commit details -
readme: drop unused memfd-bind reference
Fixes: 871057d ("drop runc-dmz solution according to overlay solution") Signed-off-by: Aleksa Sarai <[email protected]> (cherry picked from commit b9dfb22) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 82f3af8 - Browse repository at this point
Copy the full SHA 82f3af8View commit details -
memfd-bind: elaborate kernel requirements for overlayfs protection
Arguably these docs should live elsewhere (especially if we plan to remove memfd-bind in the future), but for now this is the only place that fully explains this issue. Suggested-by: Rodrigo Campos <[email protected]> Signed-off-by: Aleksa Sarai <[email protected]> (cherry picked from commit ac43589) Signed-off-by: lfbzhm <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for eb676de - Browse repository at this point
Copy the full SHA eb676deView commit details -
Merge pull request #4530 from lifubang/backport-4509-4525
[1.2] dmz: overlay: minor fixups
Configuration menu - View commit details
-
Copy full SHA for 24591c0 - Browse repository at this point
Copy the full SHA 24591c0View commit details -
Signed-off-by: Kir Kolyshkin <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 7cb3632 - Browse repository at this point
Copy the full SHA 7cb3632View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v1.2.1...v1.2.2