Skip to content

Conversation

@lsm5
Copy link
Member

@lsm5 lsm5 commented Oct 16, 2025

No description provided.

@github-actions github-actions bot added the common Related to "common" package label Oct 16, 2025
@packit-as-a-service
Copy link

Packit jobs failed. @containers/packit-build please check.

@lsm5 lsm5 force-pushed the podman6-no-slirp branch from 57c62b9 to 92c00a9 Compare October 16, 2025 13:19
@Luap99 Luap99 added the podman 6 breaking changes that should go only into podman 6 only label Oct 16, 2025
@lsm5 lsm5 force-pushed the podman6-no-slirp branch 2 times, most recently from db2cffc to 90bb692 Compare October 21, 2025 18:07
lsm5 added 15 commits October 25, 2025 09:27
Replace with cgroups.IsCgroup2UnifiedMode.

Signed-off-by: Lokesh Mandvekar <[email protected]>
And all the cgroup v1 logic that relied on the cgroup2 bool

Signed-off-by: Lokesh Mandvekar <[email protected]>
- exclude param from getAvailableControllers
- cgroupV1GetAllSubsystems
- CgroupControl.createGroupDirectory
- readAcct
- readAcctList

Signed-off-by: Lokesh Mandvekar <[email protected]>
Also inline some struct element values with constant values

Signed-off-by: Lokesh Mandvekar <[email protected]>
I can only see it being called in libpod/oci_conmon_linux.go that too
when CgroupManager is not systemd, so it should be safe to remove.

Signed-off-by: Lokesh Mandvekar <[email protected]>
Also rename systemd.MoveRootlessNetnsSlirpProcessToUserSlice to
systemd.MoveRootlessNetnsProcessToUserSlice.

Signed-off-by: Lokesh Mandvekar <[email protected]>
Signed-off-by: Lokesh Mandvekar <[email protected]>
@lsm5 lsm5 force-pushed the podman6-no-slirp branch from 90bb692 to 49f1c83 Compare October 25, 2025 13:39
@lsm5 lsm5 changed the title [WIP] common: no slirp for podman6 common: no slirp for podman6 Oct 25, 2025
@lsm5 lsm5 marked this pull request as ready for review October 25, 2025 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

common Related to "common" package podman 6 breaking changes that should go only into podman 6 only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants