Skip to content

Conversation

@MartinBasti
Copy link
Contributor

@MartinBasti MartinBasti commented Oct 30, 2025

New config optaion added into reactor config map, allowed_build_targets.

When it;s empty or undefined, all build targets are allowed.

When buildtargets are defined, only those listed there will be allowed to be used for builds (glob patterns are allowed).

Assisted-by: Claude Code AI

Maintainers will complete the following section

  • Commit messages are descriptive enough
  • Code coverage from testing does not decrease and new code is covered
  • [n/a] Python type annotations added to new code
  • JSON/YAML configuration changes are updated in the relevant schema
  • [n/a] Changes to metadata also update the documentation for the metadata
  • [n/a] Pull request has a link to an osbs-docs PR for user documentation updates
  • New feature can be disabled from a configuration file

New config optaion added into reactor config map, allowed_build_targets.

When it;s empty or undefined, all build targets are allowed.

When buildtargets are defined, only those listed there will be allowed
to be used for builds (glob patterns are allowed).

Signed-off-by: Martin Basti <mbasti@redhat.com>
Assisted-by: Claude Code AI
@MartinBasti MartinBasti force-pushed the buildtarget_allowlist branch from 5fa0c7a to a92b1cd Compare October 30, 2025 14:41
Copy link
Member

@ben-alkov ben-alkov left a comment

Choose a reason for hiding this comment

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

LGTM; merge after actions are green

@MartinBasti MartinBasti merged commit 2097277 into containerbuildsystem:master Oct 30, 2025
12 of 14 checks passed
@MartinBasti MartinBasti deleted the buildtarget_allowlist branch October 30, 2025 14:47
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