Update assisted operator index image #1364

carbonin · 2022-03-11T16:04:39Z

The ocpmetal image is no longer being maintained or updated.
All assisted images should now be pulled from edge-infrastructure.

cc @dtantsur

openshift-ci · 2022-03-11T16:04:49Z

Hi @carbonin. Thanks for your PR.

I'm waiting for a openshift-metal3 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/test-infra repository.

dtantsur · 2022-03-11T16:06:03Z

/ok-to-test
/approve

Thanks!

openshift-ci · 2022-03-11T16:06:15Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dtantsur

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~~ [dtantsur]

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

flaper87

/lgtm

carbonin · 2022-03-14T15:25:30Z

/retest

mkowalski · 2022-03-16T15:41:21Z

I tested manually and works. Without this PR make assisted does not deploy correctly, with this PR it works 💪🏻

mkowalski · 2022-03-16T15:50:22Z

/retest-required

mkowalski · 2022-03-16T18:01:04Z

/retest-required

flaper87 · 2022-03-23T08:54:38Z

/retest
/lgtm

carbonin · 2022-03-23T12:41:11Z

/retest

eranco74 · 2022-04-06T07:31:34Z

/retest-required

mkowalski · 2022-04-06T10:37:09Z

/test e2e-metal-ipi-ovn-ipv6

carbonin · 2022-04-21T17:44:49Z

/retest-required

The ocpmetal image is no longer being maintained or updated. All assisted images should now be pulled from edge-infrastructure.

carbonin · 2022-04-22T15:13:11Z

I don't understand why tests keep failing here so I rebased. Maybe that will help?

mkowalski · 2022-04-22T17:00:34Z

/lgtm

elfosardo · 2022-04-22T22:44:34Z

/retest

mkowalski · 2022-05-11T16:35:33Z

/retest-required

carbonin · 2022-05-12T12:13:22Z

/retest-required

carbonin · 2022-05-12T12:15:30Z

These tests haven't passed on any PR as far as I can see. Can someone override them please?

carbonin · 2022-05-12T14:08:13Z

@hardys I noticed you overrode these jobs on #1350 (comment)

Any chance we can not run these on PRs until they're fixed? They seem to be blocking quite a few patches in this repo.

hardys · 2022-06-15T09:22:34Z

/override ci/prow/e2e-metal-ipi-proxy-ipv6
/override ci/prow/e2e-metal-ipi-proxy-ipv4

These are still failing, we need to either fix them or remove the blocking status - sorry @carbonin I missed your earlier comment!

openshift-ci · 2022-06-15T09:22:39Z

@hardys: Overrode contexts on behalf of hardys: ci/prow/e2e-metal-ipi-proxy-ipv4, ci/prow/e2e-metal-ipi-proxy-ipv6

Details

In response to this:

/override ci/prow/e2e-metal-ipi-proxy-ipv6
/override ci/prow/e2e-metal-ipi-proxy-ipv4

These are still failing, we need to either fix them or remove the blocking status - sorry @carbonin I missed your earlier comment!

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/test-infra repository.

hardys · 2022-06-15T09:51:02Z

/retest-required

javipolo · 2022-06-16T10:21:32Z

/retest-required

elfosardo · 2022-06-16T15:44:17Z

/retest

elfosardo · 2022-06-17T08:10:38Z

I proposed openshift/origin#27263 that should fix the issue we're seeing in the ipv6 jobs with the watch channels for the etcd-operator

mkowalski · 2022-06-17T08:16:30Z

Can we override to unblock and remove the need for manual cherry-picks in all dev environments? The ipi-ovn-ipv6 job does not use at all the assisted_deployment.sh file that we have modified here, so looks like we are solving an issue that belongs somewhere else

dtantsur · 2022-06-21T14:10:35Z

/override e2e-metal-ipi-ovn-ipv6

openshift-ci · 2022-06-21T14:10:39Z

@dtantsur: /override requires a failed status context or a job name to operate on.
The following unknown contexts were given:

e2e-metal-ipi-ovn-ipv6

Only the following contexts were expected:

ci/prow/e2e-metal-ipi
ci/prow/e2e-metal-ipi-ovn-dualstack
ci/prow/e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-metal-ipi-proxy-ipv4
ci/prow/e2e-metal-ipi-proxy-ipv6
ci/prow/e2e-metal-ipi-serial-ipv4
ci/prow/e2e-metal-ipi-serial-ovn-ipv6
ci/prow/e2e-metal-ipi-virtualmedia
ci/prow/images
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-ovn-dualstack
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-ovn-ipv6
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-proxy-ipv4
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-proxy-ipv6
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-serial-ipv4
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-serial-ovn-ipv6
pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-virtualmedia
pull-ci-openshift-metal3-dev-scripts-master-images
tide

Details

In response to this:

/override e2e-metal-ipi-ovn-ipv6

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/test-infra repository.

dtantsur · 2022-06-21T14:10:56Z

/override ci/prow/e2e-metal-ipi-ovn-ipv6

openshift-ci · 2022-06-21T14:11:00Z

@dtantsur: Overrode contexts on behalf of dtantsur: ci/prow/e2e-metal-ipi-ovn-ipv6

Details

In response to this:

/override ci/prow/e2e-metal-ipi-ovn-ipv6

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/test-infra repository.

carbonin · 2022-06-21T15:38:01Z

Thank you @hardys @dtantsur @elfosardo

openshift-ci bot requested review from andfasano and russellb March 11, 2022 16:04

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 Mar 11, 2022

carbonin force-pushed the update_assisted_index_image branch from be66c83 to 6709f3b Compare March 11, 2022 16:04

openshift-ci bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Mar 11, 2022

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 11, 2022

flaper87 approved these changes Mar 11, 2022

View reviewed changes

flaper87 reviewed Mar 11, 2022

View reviewed changes

openshift-ci bot assigned flaper87 Mar 11, 2022

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 11, 2022

mkowalski mentioned this pull request Mar 16, 2022

assisted-service: Add imageStorage to the service configuration #1365

Merged

mkowalski approved these changes Mar 16, 2022

View reviewed changes

Update assisted operator index image

8037f78

The ocpmetal image is no longer being maintained or updated. All assisted images should now be pulled from edge-infrastructure.

carbonin force-pushed the update_assisted_index_image branch from 6709f3b to 8037f78 Compare April 22, 2022 15:12

openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 22, 2022

openshift-ci bot assigned mkowalski Apr 22, 2022

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 22, 2022

mkowalski mentioned this pull request May 11, 2022

Update ASSISTED_OPERATOR_INDEX to point to an updated index #1394

Closed

openshift-ci bot merged commit 05bed3d into openshift-metal3:master Jun 21, 2022

carbonin deleted the update_assisted_index_image branch June 21, 2022 15:38

Update assisted operator index image #1364

Update assisted operator index image #1364

Uh oh!

Conversation

carbonin commented Mar 11, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openshift-ci bot commented Mar 11, 2022

Uh oh!

dtantsur commented Mar 11, 2022

Uh oh!

openshift-ci bot commented Mar 11, 2022

Uh oh!

flaper87 left a comment

Choose a reason for hiding this comment

Uh oh!

carbonin commented Mar 14, 2022

Uh oh!

mkowalski commented Mar 16, 2022

Uh oh!

mkowalski commented Mar 16, 2022

Uh oh!

mkowalski commented Mar 16, 2022

Uh oh!

flaper87 commented Mar 23, 2022

Uh oh!

carbonin commented Mar 23, 2022

Uh oh!

eranco74 commented Apr 6, 2022

Uh oh!

mkowalski commented Apr 6, 2022

Uh oh!

carbonin commented Apr 21, 2022

Uh oh!

carbonin commented Apr 22, 2022

Uh oh!

mkowalski commented Apr 22, 2022

Uh oh!

elfosardo commented Apr 22, 2022

Uh oh!

mkowalski commented May 11, 2022

Uh oh!

carbonin commented May 12, 2022

Uh oh!

carbonin commented May 12, 2022

Uh oh!

carbonin commented May 12, 2022

Uh oh!

hardys commented Jun 15, 2022

Uh oh!

openshift-ci bot commented Jun 15, 2022

Uh oh!

hardys commented Jun 15, 2022

Uh oh!

javipolo commented Jun 16, 2022

Uh oh!

elfosardo commented Jun 16, 2022

Uh oh!

elfosardo commented Jun 17, 2022

Uh oh!

mkowalski commented Jun 17, 2022

Uh oh!

dtantsur commented Jun 21, 2022

Uh oh!

openshift-ci bot commented Jun 21, 2022

Uh oh!

dtantsur commented Jun 21, 2022

Uh oh!

openshift-ci bot commented Jun 21, 2022

Uh oh!

carbonin commented Jun 21, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

carbonin commented Mar 11, 2022 •

edited

Loading