Skip to content

Conversation

setchy
Copy link
Member

@setchy setchy commented Jun 25, 2025

Configure Renovate to use customManagers:dockerfileVersions to Update _VERSION variables in Dockerfiles.

Start with NODE_VERSION initially. We can apply to other _VERSION ARGS separately

Signed-off-by: Adam Setch <adam.setch@outlook.com>
@setchy setchy requested a review from prabhu as a code owner June 25, 2025 13:14
@@ -4,6 +4,7 @@ FROM registry.suse.com/bci/python:3.11 AS base
ARG JAVA_VERSION=24.0.1-tem
ARG MAVEN_VERSION=3.9.10
ARG GCC_VERSION=14
# renovate: datasource=docker depName=node
ARG NODE_VERSION=20.19.3
Copy link
Member Author

Choose a reason for hiding this comment

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

I'm not sure why this NODE_VERSION is 20 instead of 24

Copy link
Collaborator

Choose a reason for hiding this comment

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

Some old python versions do not work with latest node.

@@ -83,6 +84,7 @@ LABEL maintainer="CycloneDX" \
org.opencontainers.image.description="Rolling image with cdxgen SBOM generator for Python 3.11 apps" \
org.opencontainers.docker.cmd="docker run --rm -v /tmp:/tmp -p 9090:9090 -v $(pwd):/app:rw -t ghcr.io/cyclonedx/cdxgen-python311:v11 -r /app --server"

# renovate: datasource=docker depName=node
ARG NODE_VERSION=20.19.3
Copy link
Member Author

Choose a reason for hiding this comment

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

Similarly, I'm not sure why this NODE_VERSION is 20 instead of 24

@@ -4,6 +4,7 @@ FROM registry.suse.com/bci/python:3.6 AS base
ARG JAVA_VERSION=21.0.7-tem
ARG MAVEN_VERSION=3.9.10
ARG GCC_VERSION=14
# renovate: datasource=docker depName=node
ARG NODE_VERSION=20.19.3
Copy link
Member Author

Choose a reason for hiding this comment

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

Another one

@@ -76,6 +77,7 @@ LABEL maintainer="CycloneDX" \
org.opencontainers.image.description="Rolling image with cdxgen SBOM generator for Python 3.6 apps" \
org.opencontainers.docker.cmd="docker run --rm -v /tmp:/tmp -p 9090:9090 -v $(pwd):/app:rw -t ghcr.io/cyclonedx/cdxgen-python36:v11 -r /app --server"

# renovate: datasource=docker depName=node
ARG NODE_VERSION=20.19.3
Copy link
Member Author

Choose a reason for hiding this comment

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

Another one

@@ -1,6 +1,7 @@
# Base-image
FROM mcr.microsoft.com/dotnet/sdk:6.0-bookworm-slim AS base

# renovate: datasource=docker depName=node
ARG NODE_VERSION=22.17.0
Copy link
Member Author

Choose a reason for hiding this comment

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

Another one

Copy link
Collaborator

Choose a reason for hiding this comment

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

Might work with node 24. To test this we need a dotnet framework project with some node.js based build steps.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@@ -6,6 +6,7 @@ ARG SWIFT_PLATFORM=debian12
ARG SWIFT_BRANCH=swift-6.1.2-release
ARG SWIFT_VERSION=swift-6.1.2-RELEASE
ARG SWIFT_WEBROOT=https://download.swift.org
# renovate: datasource=docker depName=node
ARG NODE_VERSION=22.17.0
Copy link
Member Author

Choose a reason for hiding this comment

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

Another one

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this could be bumped up to 24. Although, we need to test with a bunch react-native and expo apps. @malice00 any ideas if we have any repo test that use swift 6 with node.js?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not from the top of my head. I'll pick this up when I try to split up our tests.

@setchy
Copy link
Member Author

setchy commented Jun 25, 2025

Thank you @prabhu for the feedback.

We may require a few unique package rules to fix the older node versions where applicable. I can better assess this in a follow up PR

@setchy setchy merged commit 213f1a8 into master Jun 25, 2025
4 checks passed
@setchy setchy deleted the config/renovate-dockerfile-versions branch June 25, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants