Skip to content

ENT-14361: Added --label to build by Jenkins build label - #2425

Merged
larsewi merged 1 commit into
cfengine:masterfrom
larsewi:build-in-container-label
Aug 12, 2026
Merged

ENT-14361: Added --label to build by Jenkins build label#2425
larsewi merged 1 commit into
cfengine:masterfrom
larsewi:build-in-container-label

Conversation

@larsewi

@larsewi larsewi commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

The build-in-container.py script already knows how a Jenkins label maps to a platform. Hence, an option to build based on a label makes it so that a Jenkins job does not also have to know this mapping.

With this change, adding a platform will only require editing platforms.json and labels.txt.

Signed-off-by: Lars Erik Wik lars.erik.wik@northern.tech

The build-in-container.py script already knows how a Jenkins label maps
to a platform. Hence, an option to build based on a label makes it so
that a Jenkins job does not also have to know this mapping.

With this change, adding a platform will only require editing
platforms.json and labels.txt.

Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Comment thread build-in-container.md
./build-in-container.py --platform debian-12 --project nova --role hub --build-type RELEASE

# Alternatively, use Jenkins labels (see build-scripts/labels.txt)
./build-in-container.py --label PACKAGES_HUB_x86_64_linux_debian_12 --project nova --build-type RELEASE

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For this case of HUB you don't need project nova, that is required.

I guess strangely, we don't have a distinction between community and enterprise platform labels. ok.

Might be interesting to consider changing this to --edition (community|enterprise)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@larsewi
larsewi merged commit a89b201 into cfengine:master Aug 12, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants