Skip to content

ci: add reusable workflow to build dynamic plugins container images#3636

Open
christoph-jerolimov wants to merge 1 commit into
redhat-developer:mainfrom
christoph-jerolimov:ci/build-container-images
Open

ci: add reusable workflow to build dynamic plugins container images#3636
christoph-jerolimov wants to merge 1 commit into
redhat-developer:mainfrom
christoph-jerolimov:ci/build-container-images

Conversation

@christoph-jerolimov

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

Introduce build-container-image.yml that uses rhdh-cli to package and push workspace images to quay.io.

Wire it into CI for a per-PR build, and the release workflow for next and latest tags.

The per-PR images has a 2w expiry, next and images a 4w expiry.

The images are build per workspace, not separated per package. The workspace is also added as a prefix to the tag, so that we don't need a quay repo per workspace. Feedback is very welcome for this. Both is different to the overlay build process but simplifies this, because rhdh-cli just builds all packages.

⚠️ Untested, and it includes a high risk that this could break the CI or release pipelines.

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@christoph-jerolimov christoph-jerolimov self-assigned this Jun 30, 2026
@christoph-jerolimov christoph-jerolimov requested review from a team as code owners June 30, 2026 09:17
@rhdh-qodo-merge

Copy link
Copy Markdown

Code Review by Qodo

Grey Divider

Sorry, something went wrong

We weren't able to complete the code review on our side. Please try again

Grey Divider

Qodo Logo

Introduce build-container-image.yml that uses rhdh-cli to package and
push workspace images to quay.io. Wire it into CI (per-PR images with
2w expiry) and into the release workflow (next + latest tags).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Christoph Jerolimov <jerolimov+git@redhat.com>
@christoph-jerolimov christoph-jerolimov force-pushed the ci/build-container-images branch from eafbe12 to 158888e Compare June 30, 2026 09:22
@sonarqubecloud

Copy link
Copy Markdown

- name: Push image to registry
env:
REGISTRY: ${{ inputs.registry }}
QUAY_USERNAME: ${{ secrets.QUAY_USERNAME }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think that this will not work.
This workflow is triggered from ci.yaml which is triggered by on: pull_request, this means that secrets won't be injected into this workflow.

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