Skip to content

fix: no build env var shares a name with a secret id (1.39.4, RT#1550) - #121

Merged
fatherlinux merged 1 commit into
mainfrom
fix/rhsm-distinct-files
Nov 11, 2026
Merged

fatherlinux merged 1 commit into
mainfrom
fix/rhsm-distinct-files

Conversation

@fatherlinux

Copy link
Copy Markdown
Member

Summary

  • The runner's podman 4.9 mounts --secret id=X empty when an env var named X exists, ignoring src=. The build step exported RHSM_ACTIVATION_KEY, so every id=RHSM_ACTIVATION_KEY build (ubi10 mariadb/postgres trio) failed with "Must specify an activation key".
  • Build-step variables renamed (RHSM_KEY_VALUE / RHSM_ORG_VALUE); a test holds that no build env var shares a secret id. Release 1.39.4.

Test plan

  • reproduced on quay.io/podman/stable:v4.9.3 (env var of the id's name → empty mount); renamed var → mount carries the file
  • CI green; then the chain trio's PR builds pass on v1.39.4

🤖 Generated with Claude Code

https://claude.ai/code/session_01R7Y1jh5QhdxRuFKyWVev5h

podman 4.9 on the runner mounts --secret id=X from $X when such a variable exists, ignoring src=; the build step exported RHSM_ACTIVATION_KEY, so id=RHSM_ACTIVATION_KEY mounted empty and the ubi10 mariadb/postgres builds failed with 'Must specify an activation key'. Reproduced and fix confirmed on quay.io/podman/stable:v4.9.3.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01R7Y1jh5QhdxRuFKyWVev5h

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gatehouse found 1 issues (1 high)

Full review.

with:
repository: crunchtools/constitution
ref: v1.39.3
ref: v1.39.4

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

HIGH (Constitution): The self-validation workflow now checks out the candidate v1.39.4 validator. The constitution requires this repo's required check to pin validate.yml to the last release so a PR is not judged by the validator it changes.

Suggestion: Keep the checkout pinned to the previous release while this change is under review; update it to v1.39.4 after that release, as required by the self-validation process.

Evidence:

ref: v1.39.4

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

not a bug: this repo's required self-check is .github/workflows/constitution.yml, which calls [email protected], a past release, so this PR is not judged by the validator it changes. The ref: literal inside validate.yml is the release that file belongs to: callers pinning [email protected] must check out the v1.39.4 validator, and tests hold that literal equal to the version. v1.39.2 and v1.39.3 shipped the same way.

@fatherlinux
fatherlinux merged commit 71b6b19 into main Nov 11, 2026
18 of 20 checks passed
@fatherlinux
fatherlinux deleted the fix/rhsm-distinct-files branch November 11, 2026 16:21
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.

1 participant