Skip to content

CI: Run Image Acceptance Tests for PRs - #141

Closed
anokfireball wants to merge 6 commits into
mainfrom
feat/test-runs-for-prs
Closed

anokfireball wants to merge 6 commits into
mainfrom
feat/test-runs-for-prs

Conversation

@anokfireball

Copy link
Copy Markdown
Member

No description provided.

@anokfireball
anokfireball force-pushed the feat/test-runs-for-prs branch 3 times, most recently from 0898bf8 to cdacabd Compare November 7, 2025 12:22
@anokfireball
anokfireball force-pushed the feat/test-runs-for-prs branch from cdacabd to f59b7ba Compare November 7, 2025 12:47
Comment thread .github/workflows/dev.yml
@anokfireball
anokfireball marked this pull request as ready for review November 7, 2025 13:12
Comment thread .github/workflows/upload_oci.yml Outdated
@anokfireball
anokfireball force-pushed the feat/test-runs-for-prs branch 7 times, most recently from 0d26629 to 6fa7008 Compare November 11, 2025 09:24
@anokfireball
anokfireball force-pushed the feat/test-runs-for-prs branch 2 times, most recently from db8d412 to 4897f9f Compare November 11, 2025 09:58

# setup OCI_TAG, UKI_SHA and fetch UKI
OCI_TAG="$GARDENLINUX_VERSION-$VARIANT_ID-$GARDENLINUX_COMMIT_ID-$OCI_ARCH"
OCI_TAG=${OCI_TAG:-"$GARDENLINUX_VERSION-$VARIANT_ID-$GARDENLINUX_COMMIT_ID-$OCI_ARCH"}

@anokfireball anokfireball Nov 11, 2025 •

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.

Note while I consider this a security risk (silently bootstrapping a different tag than the currently running one), the same thing was possible before this change by overwriting /etc/os-release via ignition.

@anokfireball
anokfireball force-pushed the feat/test-runs-for-prs branch from 4897f9f to 5aafae8 Compare November 11, 2025 10:17
@anokfireball
anokfireball force-pushed the feat/test-runs-for-prs branch from 1b4c3fd to 6133fe3 Compare November 11, 2025 11:20
@anokfireball
anokfireball deleted the feat/test-runs-for-prs branch November 11, 2025 14:07
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.

2 participants