Skip to content

Re-add agent-ops CI build and image mirror for odh-dashboard#80241

Draft
Gkrumbach07 wants to merge 1 commit into
openshift:mainfrom
Gkrumbach07:readd-agent-ops-mirror
Draft

Re-add agent-ops CI build and image mirror for odh-dashboard#80241
Gkrumbach07 wants to merge 1 commit into
openshift:mainfrom
Gkrumbach07:readd-agent-ops-mirror

Conversation

@Gkrumbach07

@Gkrumbach07 Gkrumbach07 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Re-adds the agent-ops Prow CI build and image mirror jobs that were reverted in #80211 due to missing quay push permissions.

Do not merge until the Prow CI robot has been granted push access to quay.io/opendatahub/odh-mod-arch-agent-ops (tracked in RHOAIENG-63302).

Original PR: #80040

/cc @lferrnan

Summary by CodeRabbit

This PR re-adds OpenShift CI build and image-mirroring configuration for the agent-ops modular-architecture module in the ODH dashboard repository. Specifically, it modifies the CI configuration to:

  1. Add a new container image build for the agent-ops component, using the Dockerfile at ./packages/agent-ops/Dockerfile.workspace to produce the odh-mod-arch-agent-ops-image artifact.

  2. Enable automated image mirroring to the quay.io registry by introducing two new mirror job entries:

    • A PR-run variant that mirrors built images for pull request testing
    • A postsubmit variant that mirrors images to quay.io/opendatahub/odh-mod-arch-agent-ops after merges to the main branch, with the RELEASE_VERSION: main label applied

This restores the agent-ops CI pipeline that was previously reverted due to missing Prow CI robot permissions on the target quay.io repository. The changes are gated—the PR should not be merged until push access has been granted to the Prow CI service account.

…0040)

* Add agent-ops mod-arch module to ODH dashboard CI builds

Add OpenShift CI build configuration for the agent-ops modular
architecture module in the ODH dashboard. This enables automated
image builds on PRs and main branch merges, pushing to
quay.io/opendatahub/odh-mod-arch-agent-ops.

RHOAIENG-62592

* Generated job files for agent-ops CI builds

Auto-generated via `make jobs` from the config change.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 8, 2026
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 5fd4f611-1a2a-45ee-bace-4da4b1823785

📥 Commits

Reviewing files that changed from the base of the PR and between dcaae87 and 4526d8f.

⛔ Files ignored due to path filters (2)
  • ci-operator/jobs/opendatahub-io/odh-dashboard/opendatahub-io-odh-dashboard-main-postsubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/opendatahub-io/odh-dashboard/opendatahub-io-odh-dashboard-main-presubmits.yaml is excluded by !ci-operator/jobs/**
📒 Files selected for processing (1)
  • ci-operator/config/opendatahub-io/odh-dashboard/opendatahub-io-odh-dashboard-main.yaml

Walkthrough

This PR adds the agent-ops component to the ODH dashboard CI pipeline by defining a new container image build from ./packages/agent-ops/Dockerfile.workspace mapped to odh-mod-arch-agent-ops-image, and extends the CI testing matrix with mirror entries for both PR and postsubmit runs, including release version configuration for postsubmit builds.

Changes

agent-ops image build and mirroring

Layer / File(s) Summary
agent-ops image build definition
ci-operator/config/opendatahub-io/odh-dashboard/opendatahub-io-odh-dashboard-main.yaml
New images entry for agent-ops component pointing to ./packages/agent-ops/Dockerfile.workspace and targeting the odh-mod-arch-agent-ops-image.
CI mirror and test entries for agent-ops image
ci-operator/config/opendatahub-io/odh-dashboard/opendatahub-io-odh-dashboard-main.yaml
Two new tests mirror entries added for odh-mod-arch-agent-ops-image: one for PR runs and one for postsubmit runs with RELEASE_VERSION: main.

Possibly related PRs

  • openshift/release#80040: Both PRs update the same ODH dashboard CI YAML to add the agent-ops build targeting odh-mod-arch-agent-ops-image and extend the image-mirroring matrix with PR and postsubmit mirror entries.
  • openshift/release#80211: Main PR adds the agent-ops Konflux image build and its image-mirroring entries in opendatahub-io-odh-dashboard-main.yaml, while retrieved PR reverts/removes those same entries.

Suggested labels

ok-to-test, rehearsals-ack

Suggested reviewers

  • christianvogt
  • lucferbux
  • andrewballantyne

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Re-add agent-ops CI build and image mirror for odh-dashboard' accurately and specifically describes the main change—re-adding CI build and image mirror configuration for the agent-ops component to the odh-dashboard project.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed This PR contains no Ginkgo test code or test definitions. It only modifies a YAML CI configuration file with stable job names, so the check for dynamic Ginkgo test titles does not apply.
Test Structure And Quality ✅ Passed This PR adds CI configuration for the agent-ops module, not Ginkgo tests. The test files added use standard Go testing.T, not Ginkgo's Describe/It blocks. Check is not applicable.
Microshift Test Compatibility ✅ Passed PR does not add any Ginkgo e2e tests; only modifies YAML CI configuration for agent-ops image building and mirroring. MicroShift compatibility check not applicable.
Single Node Openshift (Sno) Test Compatibility ✅ Passed This PR modifies only a CI configuration YAML file for agent-ops image builds and mirror jobs. No Ginkgo e2e test code is added; the check does not apply to CI configuration files.
Topology-Aware Scheduling Compatibility ✅ Passed PR modifies only CI build configuration (ci-operator YAML), not deployment manifests, operators, or controllers. No scheduling constraints introduced.
Ote Binary Stdout Contract ✅ Passed PR modifies only a YAML CI configuration file with no executable code. OTE stdout checks apply to code-level stdout in main/init/suite setup functions, which this YAML lacks.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed No Ginkgo e2e tests added. PR only modifies CI/CD configuration (YAML) for container image builds and CI jobs, not test code.
No-Weak-Crypto ✅ Passed PR contains only YAML CI/CD configuration with no code, cryptographic implementations, or weak crypto algorithm usage.
Container-Privileges ✅ Passed No privileged container settings found in PR. Adds CI/CD config for agent-ops image builds without privileged, hostPID/Network/IPC, SYS_ADMIN, allowPrivilegeEscalation settings.
No-Sensitive-Data-In-Logs ✅ Passed The PR adds agent-ops CI config with image mirror tests. Log analysis shows only public data (dates, repo names, versions). Credentials never logged. No sensitive data exposure found.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@openshift-ci openshift-ci Bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jun 8, 2026
@openshift-ci

openshift-ci Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Hi @Gkrumbach07. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci

openshift-ci Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Gkrumbach07
Once this PR has been reviewed and has the lgtm label, please assign darachcawley for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@Gkrumbach07: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-agent-ops-pr-image-mirror opendatahub-io/odh-dashboard presubmit Presubmit changed
pull-ci-opendatahub-io-odh-dashboard-main-images opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-dashboard-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-automl-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-autorag-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-eval-hub-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-gen-ai-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-maas-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-mlflow-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed
pull-ci-opendatahub-io-odh-dashboard-main-odh-mod-arch-model-registry-pr-image-mirror opendatahub-io/odh-dashboard presubmit Ci-operator config changed

Prior to this PR being merged, you will need to either run and acknowledge or opt to skip these rehearsals.

Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant