Skip to content

Use artifacts to store compiled tester image - #802

Merged
byteocean merged 1 commit into
ironcore-dev:mainfrom
byteocean:ci/upload-artifact-during-test
Jul 17, 2026
Merged

byteocean merged 1 commit into
ironcore-dev:mainfrom
byteocean:ci/upload-artifact-during-test

Conversation

@byteocean

Copy link
Copy Markdown
Contributor

Follow the suggestion from @mkalcok, use artifacts to store resulted tester image.

@byteocean
byteocean requested a review from a team as a code owner July 16, 2026 14:08
@byteocean
byteocean force-pushed the ci/upload-artifact-during-test branch 2 times, most recently from 3bce97a to 3934ba3 Compare July 16, 2026 14:47

@mkalcok mkalcok left a comment

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.

Overall LGTM, thank you. I think there's one left-over secret reference that's no longer needed (see below).

Comment thread .github/workflows/publish-docker-tester.yml
Comment thread .github/workflows/publish-docker-tester.yml
Signed-off-by: Tao Li <tao.li06@sap.com>

@mkalcok mkalcok left a comment

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.

LGTM, thanks for the updates.

@byteocean
byteocean merged commit f859820 into ironcore-dev:main Jul 17, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this to Done in Roadmap Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants