Skip to content

cluster-version-operator: Update Assisted Installer jobs #25120

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
lranjbar wants to merge 1 commit into from
Closed

cluster-version-operator: Update Assisted Installer jobs #25120

lranjbar wants to merge 1 commit into from

Conversation

@lranjbar
Copy link
Contributor

@lranjbar lranjbar commented Jan 10, 2022

Updates the jobs for Assisted Installer to the latest images and uses the correct cluster profile.

@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 Jan 10, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 10, 2022

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

@lranjbar lranjbar marked this pull request as ready for review January 10, 2022 23:45
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 10, 2022
@hongkailiu
Copy link
Member

hongkailiu commented Jan 13, 2022

/uncc

@openshift-ci openshift-ci bot removed the request for review from hongkailiu January 13, 2022 15:54
@lranjbar lranjbar force-pushed the packet-account-split-01-cvo branch 2 times, most recently from 81c6260 to 2d824a2 Compare January 17, 2022 21:39
@lranjbar
cluster-version-operator: Update Assisted Installer jobs
dc809bf 
This updates the cluster-version-operator jobs for Assisted Installer:

1. Updates the cluster_profile to "packet-assisted" from "packet".
This will increase flexibility in using the Equinix cloud offering
by splitting the workflows into an account for Assisted Installer.

2. Updates the Assisted Installer images to the latest images coming
from our master branch. We needed to update the promotion scheme for
Assisted Installer and this will pick the lastest version for testing.
@lranjbar lranjbar force-pushed the packet-account-split-01-cvo branch from 2d824a2 to dc809bf Compare January 19, 2022 21:22
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 19, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lranjbar
To complete the pull request process, please assign wking after the PR has been reviewed.
You can assign the PR to them by writing /assign @wking 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

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 19, 2022

@lranjbar: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/rehearse/openshift/cluster-version-operator/release-4.7/e2e-agnostic-upgrade 2a4bdcab0a5f5034971ae032786460ea495f56ff link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-version-operator/release-4.11/e2e-agnostic-operator 2d824a2b2df3eb8f89f6b61ba2e1fdd89d5f1e56 link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-version-operator/release-4.11/e2e-agnostic-upgrade 2d824a2b2df3eb8f89f6b61ba2e1fdd89d5f1e56 link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-version-operator/master/e2e-agnostic-upgrade dc809bf link unknown /test pj-rehearse
ci/rehearse/openshift/cluster-version-operator/master/e2e-agnostic-operator dc809bf link unknown /test pj-rehearse
ci/prow/pj-rehearse dc809bf link false /test pj-rehearse

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@wking
Copy link
Member

wking commented Jan 24, 2022

Possibly replacing with #25571.

@wking
Copy link
Member

wking commented Jan 24, 2022

#25571 has landed, and we expect future pre-merge CVO assisted-installer testing to happen via /payload ....

/close

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 24, 2022

@lranjbar: PR needs rebase.

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.

@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 24, 2022
@openshift-ci openshift-ci bot closed this Jan 24, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 24, 2022

@wking: Closed this PR.

Details

In response to this:

#25571 has landed, and we expect future pre-merge CVO assisted-installer testing to happen via /payload ....

/close

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.

@lranjbar lranjbar mentioned this pull request Jan 25, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LalatenduMohanty LalatenduMohanty Awaiting requested review from LalatenduMohanty

Assignees

No one assigned

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lranjbar @hongkailiu @wking