Skip to content

Conversation

@uparrows
Copy link

@uparrows uparrows commented Dec 1, 2025

📦 Package Details

Maintainer: @gerard Ryan
(You can find this by checking the history of the package Makefile.)

Description:

Update all docker packages to 29.1.1

🧪 Run Testing Details

  • **OpenWrt Version:OpenWrt 24.10.4 r28959-29397011cc / LuCI openwrt-24.10 branch 25.292.66247~75e41cb
  • **OpenWrt Target/Subtarget:x86/64
  • **OpenWrt Device:CncTion J4125-4L

1

2

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

If your PR contains a patch:

  • It can be applied using git am
  • It has been refreshed to avoid offsets, fuzzes, etc., using
    make package/<your-package>/refresh V=s
  • It is structured in a way that it is potentially upstreamable
    (e.g., subject line, commit description, etc.)
    We must try to upstream patches to reduce maintenance burden.

@uparrows
Copy link
Author

uparrows commented Dec 2, 2025

before compile , edit /home/user/openwrt/feeds/packages/utils/dockerd/files/etc/config/dockerd option iptables '0' ,docker will use nftables instead of iptables.
4

@hnyman
Copy link
Contributor

hnyman commented Dec 6, 2025

Cc @G-M0N3Y-2503

@G-M0N3Y-2503
Copy link
Contributor

Just a couple of things, and the CI is complaining too. But thanks for the work!

PKG_NAME:=containerd
PKG_VERSION:=1.7.22
PKG_VERSION:=2.2.0
PKG_RELEASE:=2
Copy link
Contributor

Choose a reason for hiding this comment

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

Oops, we missed resetting this to 1

PKG_NAME:=dockerd
PKG_VERSION:=27.3.1
PKG_VERSION:=29.1.1
PKG_RELEASE:=4
Copy link
Contributor

Choose a reason for hiding this comment

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

We should reset this to 1 also

PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz
PKG_GIT_URL:=github.com/moby/moby
PKG_GIT_REF:=v$(PKG_VERSION)
PKG_GIT_REF:=docker-v$(PKG_VERSION)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did we add "docker-" here?

Copy link
Author

Choose a reason for hiding this comment

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

1
2
0

because the version number 29.x has an extra "docker-" at the beginning.

Copy link
Contributor

Choose a reason for hiding this comment

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

oh, I missed that detail, thanks!

Copy link
Member

Choose a reason for hiding this comment

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

Should be in the commit description.

Copy link
Author

Choose a reason for hiding this comment

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

Should be in the commit description.
I have added it to the commit description.


define Package/containerd/install
$(INSTALL_DIR) $(1)/usr/bin/
$(INSTALL_BIN) $(PKG_INSTALL_DIR)/bin/{ctr,containerd,containerd-stress,containerd-shim,containerd-shim-runc-v1,containerd-shim-runc-v2} $(1)/usr/bin/
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these not available anymore or something?

Copy link
Author

Choose a reason for hiding this comment

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

Docker 29 has removed old containerd shim processes (containerd-shim and containerd-shim-runc-v1).

5

Copy link
Author

Choose a reason for hiding this comment

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

Docker version 29 is an upcoming major update that will use containerd-shim-runc-v2 by default and deprecate the older containerd-shim-runc-v1. Therefore, removing containerd-shim-runc-v1 is not a "manual" operation, but rather the default state after the Docker upgrade. It is an inevitable result of the transition of container runtime from the old standard to the new standard (from runc v1 to runc v2). The main purpose is to improve performance, security, and support for the OCI standard. You do not need to actively "remove" it; simply upgrading Docker will achieve the switch.

Copy link
Contributor

Choose a reason for hiding this comment

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

thanks again for the info!

Copy link
Member

Choose a reason for hiding this comment

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

Well, please add this to your commit.

Copy link
Author

Choose a reason for hiding this comment

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

commit
I have added it to the commit.

PKG_NAME:=runc
PKG_VERSION:=1.1.14
PKG_VERSION:=1.3.3
PKG_RELEASE:=2
Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably reset this to 1 too.

PKG_GIT_SHORT_COMMIT:=41ca978 # SHA1 used within the docker executables
PKG_HASH:=65221f1c70feb1bd1562bb1017b586e4528be877656dc16f5be5659fc9b7e522
PKG_GIT_SHORT_COMMIT:=9a84135 # SHA1 used within the docker executables

Copy link
Contributor

@Yang-Wei-Ting Yang-Wei-Ting Dec 8, 2025

Choose a reason for hiding this comment

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

Hi, please remove this extra newline

Copy link
Author

Choose a reason for hiding this comment

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

removed

@hnyman
Copy link
Contributor

hnyman commented Dec 8, 2025

You need to squash the extra commits. One per package should be enough.

And please make sure that each commit start with the "packagename: " prefix (like "docker: ".
Now they do not.

update to v2.2.0, remove containerd-shim and containerd-shim-runc-v1, docker 29.x has removed old containerd shim processes (containerd-shim and containerd-shim-runc-v1).
update docker cli to v29.1.1.
docker version 29.x has an extra "docker-" prefix. add "docker-" prefix to "PKG_GIT_REF".
docker v29.1.1 requires runc v1.3.3.
@uparrows
Copy link
Author

uparrows commented Dec 8, 2025

You need to squash the extra commits. One per package should be enough.

And please make sure that each commit start with the "packagename: " prefix (like "docker: ". Now they do not.

okay ,revised as requested

@hnyman
Copy link
Contributor

hnyman commented Dec 9, 2025

It still fails the new strict CI formality test, as you have no space after the prefix in some of commits.
(and as it fails formality test, no compilation test is even performed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants