Skip to content

USHIFT-1201: Transition to AWS#38877

Merged
openshift-merge-robot merged 9 commits intoopenshift:masterfrom
copejon:ushift-1201
May 4, 2023
Merged

USHIFT-1201: Transition to AWS#38877
openshift-merge-robot merged 9 commits intoopenshift:masterfrom
copejon:ushift-1201

Conversation

@copejon
Copy link
Contributor

@copejon copejon commented Apr 28, 2023

No description provided.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 28, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Apr 28, 2023 •
edited by openshift-ci bot
Loading

@copejon: This pull request references USHIFT-1201 which is a valid jira issue.

Details

In response to this:

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 requested review from pacevedom and pmtk April 28, 2023 20:15
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 28, 2023
@copejon copejon force-pushed the ushift-1201 branch 2 times, most recently from bd261f8 to 56cd33e Compare April 28, 2023 21:15
@copejon
Copy link
Contributor Author

copejon commented Apr 28, 2023

/pj-rehearse

@dhellmann dhellmann mentioned this pull request Apr 28, 2023
Merged
@copejon
Copy link
Contributor Author

copejon commented Apr 28, 2023

/pj-rehearase

@copejon
Copy link
Contributor Author

copejon commented Apr 28, 2023

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 28, 2023
pacevedom
pacevedom reviewed Apr 28, 2023
View reviewed changes
Copy link
Contributor

@pacevedom pacevedom left a comment

Choose a reason for hiding this comment

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

Looking good overall. Maybe we could merge the -aws with non-aws workflows by doing a platform discovery in order to get the right commands to retrieve the IP, but that can be done later.

@copejon
Copy link
Contributor Author

copejon commented Apr 28, 2023

/pj-rehearse

@copejon
Copy link
Contributor Author

copejon commented Apr 30, 2023

/retest

@copejon
Copy link
Contributor Author

copejon commented Apr 30, 2023

/pj-rehearse

1 similar comment
@copejon
Copy link
Contributor Author

copejon commented Apr 30, 2023

/pj-rehearse

@copejon
Copy link
Contributor Author

copejon commented May 1, 2023

/pj-rehearse max

@copejon
Copy link
Contributor Author

copejon commented May 1, 2023

/pj-rehearse max

This was referenced May 1, 2023
Closed
Merged
@copejon
Copy link
Contributor Author

copejon commented May 2, 2023

/retest

@eggfoobar
Copy link
Contributor

eggfoobar commented May 2, 2023

/pj-rehearse

@copejon
Copy link
Contributor Author

copejon commented May 2, 2023

These are expected to fail until openshift/microshift#1742 is merged

@copejon
Copy link
Contributor Author

copejon commented May 3, 2023

/pj-rehearse max

pacevedom
pacevedom reviewed May 3, 2023
View reviewed changes
ci-operator/config/openshift/microshift/openshift-microshift-release-4.13.yaml Outdated
Copy link
Contributor

@pacevedom pacevedom May 3, 2023

Choose a reason for hiding this comment

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

Keep in mind that new static analysis only runs on 4.14+. This target does not exist in this branch.

ci-operator/config/openshift/microshift/openshift-microshift-release-4.13.yaml Outdated
Copy link
Contributor

@pacevedom pacevedom May 3, 2023

Choose a reason for hiding this comment

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

This test only exists from 4.14. You need to keep the reboot, greenboot, load-balancer and router smoke tests in this branch.

ci-operator/config/openshift/microshift/openshift-microshift-release-4.13.yaml Outdated
Copy link
Contributor

@pacevedom pacevedom May 3, 2023

Choose a reason for hiding this comment

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

Need to keep these nightlies for reporting to testgrid.

@copejon
Copy link
Contributor Author

copejon commented May 3, 2023

/pj-rehearse

1 similar comment
@copejon
Copy link
Contributor Author

copejon commented May 3, 2023

/pj-rehearse

@copejon
Copy link
Contributor Author

copejon commented May 3, 2023

These tests are running the openshift/conformance/[parallel/serial] test suites, which depend on CRDs from the openshift cluster-config-operator. So these tests will never pass.

ci/rehearse/periodic-ci-openshift-microshift-release-4.13-nightly-conformance-parallel
ci/rehearse/periodic-ci-openshift-microshift-release-4.13-nightly-conformance-serial

@copejon
Copy link
Contributor Author

copejon commented May 3, 2023

/pj-rehearse pull-ci-openshift-microshift-release-4.13-e2e-openshift-conformance-reduced-arm

@copejon
Copy link
Contributor Author

copejon commented May 3, 2023

/pj-rehearse

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented May 3, 2023

@copejon, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from base revision of release repo: could not checkout worktree: '[git checkout 16130797b3787fac4f400645b3e1015f4c816b62]' failed with out: fatal: reference is not a tree: 16130797b3787fac4f400645b3e1015f4c816b62
and error exit status 128
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 10 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 20 rehearsals
Comment: /pj-rehearse max to run up to 35 rehearsals
Comment: /pj-rehearse auto-ack to run up to 10 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse abort to abort all active rehearsals

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@pacevedom
Copy link
Contributor

pacevedom commented May 4, 2023

/pj-rehearse pull-ci-openshift-microshift-release-4.13-e2e-loadbalancer-smoke-test-arm

@pacevedom
Copy link
Contributor

pacevedom commented May 4, 2023

/pj-rehearse periodic-ci-openshift-microshift-release-4.13-nightly-conformance-serial

@copejon
Copy link
Contributor Author

copejon commented May 4, 2023

/unhold
/pj-rehearse ack

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 4, 2023
@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label May 4, 2023
pacevedom
pacevedom reviewed May 4, 2023
View reviewed changes
Copy link
Contributor

@pacevedom pacevedom 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 openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 4, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 4, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: copejon, pacevedom

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:

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 May 4, 2023

@copejon: 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/periodic-ci-openshift-microshift-release-4.13-nightly-conformance-serial 89898d6 link unknown /pj-rehearse periodic-ci-openshift-microshift-release-4.13-nightly-conformance-serial
ci/rehearse/openshift/microshift/release-4.13/e2e-openshift-conformance-reduced-arm 89898d6 link unknown /pj-rehearse pull-ci-openshift-microshift-release-4.13-e2e-openshift-conformance-reduced-arm
ci/rehearse/periodic-ci-openshift-microshift-release-4.13-nightly-conformance-parallel 89898d6 link unknown /pj-rehearse periodic-ci-openshift-microshift-release-4.13-nightly-conformance-parallel

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.

@openshift-merge-robot openshift-merge-robot merged commit 8dac2ee into openshift:master May 4, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 4, 2023

@copejon: Updated the following 2 configmaps:

  • ci-operator-4.13-configs configmap in namespace ci at cluster app.ci using the following files:
    • key openshift-microshift-release-4.13.yaml using file ci-operator/config/openshift/microshift/openshift-microshift-release-4.13.yaml
  • job-config-4.13 configmap in namespace ci at cluster app.ci using the following files:
    • key openshift-microshift-release-4.13-periodics.yaml using file ci-operator/jobs/openshift/microshift/openshift-microshift-release-4.13-periodics.yaml
    • key openshift-microshift-release-4.13-presubmits.yaml using file ci-operator/jobs/openshift/microshift/openshift-microshift-release-4.13-presubmits.yaml
Details

In response to this:

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.

@copejon copejon deleted the ushift-1201 branch May 4, 2023 16:29
ascerra pushed a commit to ascerra/release that referenced this pull request May 8, 2023
@copejon @ascerra
USHIFT-1201: Transition to AWS (openshift#38877)
960a1dd 
* add loadbalancer-smoketest aws workflow

* add e2e-reboot aws workflow

* add e2e-greenboot-aws workflow

make update e2e-greenboot

* add router-smoketest aws workflow

make update router-smoke-tests

* add e2e-test aws workflow

* apply transition to aws

* make update
fix config indentation
fix ssh command

* change CPU to c6g to workaround resource shortage

* minimizing the diff to be certain i'm not adding a test that wasn't there previously
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pmtk pmtk Awaiting requested review from pmtk

1 more reviewer

@pacevedom pacevedom pacevedom left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@pacevedom pacevedom

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@copejon @openshift-ci-robot @eggfoobar @pacevedom @openshift-merge-robot