Skip to content

[DNM] pin-bundle-images: resolve fork tags from Quay under local registry CI - #1901

Draft
amartyasinha wants to merge 2 commits into
openstack-k8s-operators:mainfrom
amartyasinha:fix/pin-bundle-images-fork-local-registry
Draft

[DNM] pin-bundle-images: resolve fork tags from Quay under local registry CI#1901
amartyasinha wants to merge 2 commits into
openstack-k8s-operators:mainfrom
amartyasinha:fix/pin-bundle-images-fork-local-registry

Conversation

@amartyasinha

Copy link
Copy Markdown
Contributor
When LOCAL_REGISTRY=1 and the go.mod module path is not under
openstack-k8s-operators, keep bundle pullspecs on IMAGEREGISTRY but
query Quay for bundle tag digests. Upstream modules still use the
local registry v2 tags/list path when mirrored.

Do not merge: just to make content-provider CI pass on #1900

@openshift-ci

openshift-ci Bot commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci

openshift-ci Bot commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: amartyasinha
Once this PR has been reviewed and has the lgtm label, please assign rabi 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

@github-actions

github-actions Bot commented Apr 24, 2026

Copy link
Copy Markdown

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 322464 bytes (315KB)
Base branch size 337535 bytes
Change -4.47%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

    When LOCAL_REGISTRY=1 and the go.mod module path is not under
    openstack-k8s-operators, keep bundle pullspecs on IMAGEREGISTRY but
    query Quay for bundle tag digests. Upstream modules still use the
    local registry v2 tags/list path when mirrored.

Signed-off-by: Amartya Sinha <amsinha@redhat.com>
this script strips BASE from repo name, while we haven't renamed the repo yet. this hack will make CI happy and enable testing

Signed-off-by: Amartya Sinha <amsinha@redhat.com>
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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.

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/12f2e3baf52f407f83fb87fd4986a87f

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 49m 43s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 31m 20s
cifmw-crc-podified-edpm-baremetal NODE_FAILURE Node(set) request 099-0000181255 failed in 0s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 48m 07s
✔️ openstack-operator-edpm-baremetal-minor-update SUCCESS in 2h 15m 42s

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant