-
Notifications
You must be signed in to change notification settings - Fork 116
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
Update docker API version in recipes to match balena API version #3438
Conversation
a4f6ec1
to
b9c4327
Compare
849fd72
to
8de8f68
Compare
I tried building generic-amd64 against this PR and got an error:
|
We could reduce the API version to increase the compatibility with docker hosts, or enforce that the minimum version of a docker installed in the host needs to match the balenaEngine version. Was this a native build @klutchell or a builder instance? |
@alexgg this was on our GitHub runners, using the balena-build.sh helper image. I would expect anyone using the helper image to see the same error, including Jenkins? Since the helper image provides a DinD I assume we can just bump the version of Docker in there to remain aligned? I've already been considering the bump from Ubuntu 18 to 20 for that image being that 18 is EOL. As you say we can recommend that users not using the helper image should update their host Docker version. |
@resin-jenkins retest this please |
The helper images has been updated to 22.04. |
Pending on device repositories to update balena-yocto-scripts. |
37490f9
to
2a23a94
Compare
Requires balena-os/balena-up-board#612 |
654bfbe
to
3e91ac8
Compare
Website deployed to CF Pages, 👀 preview link https://65596ea5.balena-os.pages.dev |
This is used to match the API version used in the build pipeline with the balena version used in the image. Change-type: patch Signed-off-by: Alex Gonzalez <[email protected]>
Update the API version to the version set in the distro configuration, which must match the API version of the balena package. Change-type: patch Signed-off-by: Alex Gonzalez <[email protected]>
4974613
to
b129f29
Compare
Contributor checklist
Change-type
present on at least one commitSigned-off-by
is presentReviewer Guidelines