Skip to content

Conversation

@cgwalters
Copy link
Member

I've been pushing a test release image to the coreos/ namespace,
but that's not sufficient for it to be mirrored into the build clusters.

Now, what would be a lot cleaner is to either:

  • Actually build that release image from source in api.ci on pushes
    to openshift/os, but that would require some more new novel code
  • Push to some place else like quay.io and mirror it like any other
    external image, but that makes it harder to share the ability for
    other people to push without figuring out how to manually add
    everyone to a new quay.io organization, not to mention juggling
    new quay credentials

So...let's mirror inside the CI clusters.

I've been pushing a test release image to the `coreos/` namespace,
but that's not sufficient for it to be mirrored into the build clusters.

Now, what would be a lot cleaner is to either:

- Actually build that release image from source in api.ci on pushes
  to openshift/os, but that would require some more new novel code
- Push to some place else like quay.io and mirror it like any other
  external image, but that makes it harder to share the ability for
  other people to push without figuring out how to manually add
  everyone to a new quay.io organization, not to mention juggling
  new quay credentials

So...let's mirror inside the CI clusters.
@cgwalters
Copy link
Member Author

Dependency of: #24222

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 2, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: cgwalters
To complete the pull request process, please assign bbguimaraes after the PR has been reviewed.
You can assign the PR to them by writing /assign @bbguimaraes in a comment when ready.

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

@cgwalters
Copy link
Member Author

Closing in favor of #24225

@cgwalters cgwalters closed this Dec 2, 2021
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.

1 participant