Skip to content

Conversation

@wking
Copy link
Member

@wking wking commented Nov 16, 2020

The outgoing docker.io image was subject to docker.io rate limiting:

error: build error: failed to pull image: After retrying 2 times, Pull image still failed due to error: while pulling "docker://docker.io/openshift/origin-release:golang-1.13" as "docker.io/openshift/origin-release:golang-1.13"...

The new image matches openshift/cluster-version-operator@7cb2662924 (openshift/cluster-version-operator#468). Matching builders with the CVO reduces our exposure by consolidating around a smaller set of Go builder images.

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 16, 2020
@wking
Copy link
Member Author

wking commented Nov 16, 2020

I expect that after this lands, openshift/registry will pick it up via a change like openshift/release#12532.

The outgoing docker.io image was subject to docker.io rate limiting
[1]:

  error: build error: failed to pull image: After retrying 2 times, Pull image still failed due to error: while pulling "docker://docker.io/openshift/origin-release:golang-1.13" as "docker.io/openshift/origin-release:golang-1.13"...

The new image matches openshift/cluster-version-operator@7cb2662924
(Updating cluster-version-operator builder & base images to be
consistent with ART, 2020-10-07,
openshift/cluster-version-operator#468).  Matching builders with the
CVO reduces our exposure by consolidating around a smaller set of Go
builder images.

I expect that after this lands, openshift/registry will pick it up via
a change like openshift/release@d0fa1c98b9 (Registry-replacer
autocommit, 2020-10-07, openshift/release#12532).

[1]: openshift#82 (comment)
@wking
Copy link
Member Author

wking commented Nov 16, 2020

/assign @jottofar

Copy link
Member

@LalatenduMohanty LalatenduMohanty left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LalatenduMohanty, wking

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [LalatenduMohanty,wking]

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

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 17, 2020
@openshift-merge-robot openshift-merge-robot merged commit a8e9d94 into openshift:master Nov 17, 2020
@wking wking deleted the go-1.15 branch November 17, 2020 15:16
@wking
Copy link
Member Author

wking commented Nov 17, 2020

Hmm. openshift/release#13681 did not pick this up in the release repo. I'll go re-read how it figures out which Dockerfiles to look at...

PratikMahajan pushed a commit to PratikMahajan/cincinnati-operator that referenced this pull request Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants