Skip to content

Conversation

copejon
Copy link
Contributor

@copejon copejon commented May 13, 2024

FYI - branch name is wrong, this is correctly tagged for USHIFT-2971

@openshift-ci openshift-ci bot requested review from ggiguash and pliurh May 13, 2024 19:20
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 13, 2024
@copejon copejon force-pushed the ushift-2970-rhel-94-periodics branch from f63c71f to 0e7d3d5 Compare May 13, 2024 19:26
Copy link
Member

@pmtk pmtk left a comment

Choose a reason for hiding this comment

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

Also, seems that [email protected] was missed when changing

@copejon
Copy link
Contributor Author

copejon commented May 14, 2024

/test metal-periodic-test

@copejon copejon force-pushed the ushift-2970-rhel-94-periodics branch from 2fae1d1 to 9f919d6 Compare May 15, 2024 18:05
@copejon
Copy link
Contributor Author

copejon commented May 16, 2024

/retest

@copejon
Copy link
Contributor Author

copejon commented May 22, 2024

/test metal-periodic-test

Copy link
Member

Choose a reason for hiding this comment

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

Filename -> el92-yminus2@[email protected]

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 29, 2024
@copejon copejon force-pushed the ushift-2970-rhel-94-periodics branch from 127fc6c to b23a62b Compare May 29, 2024 16:56
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 29, 2024
@copejon
Copy link
Contributor Author

copejon commented May 29, 2024

/retest

Failed during infra deployment

@copejon copejon force-pushed the ushift-2970-rhel-94-periodics branch 2 times, most recently from 4b8dd49 to 78ab95f Compare May 29, 2024 23:54
@copejon
Copy link
Contributor Author

copejon commented May 29, 2024

@pmtk The current release version has shifted to 4.17 but there are no ec's/rc's available yet. This causes the edge commits for crel builds to be skipped, resulting in the scenarios which expect these commits to fail during the vm creation. Instead of deleting the scenarios (which we'll need in the near future) I've added the extension .pending_crel to the filenames. This should exempt them from the ci boot phase. If this works, I'll create a ticket to remind us to reenable them once 4.17 is available.

The logic I'm abusing here is this line:

cp "${SCENARIO_SOURCES}"/*.sh "${SCENARIOS_TO_RUN}"/

*Obviously we need to make the scripts a little smarter about handling this situation.

@copejon
Copy link
Contributor Author

copejon commented May 30, 2024

The images built and booted, but for some reason the cos9 scenarios never installed microshift.

@copejon
Copy link
Contributor Author

copejon commented May 30, 2024

cos9-src@rpm-standard{1,2}.sh scenarios are also affected by the 4.17 rpms not being available yet, so these will also need to be disabled temporarily

@copejon
Copy link
Contributor Author

copejon commented May 30, 2024

Infra failures prevented the metal host from starting

/retest

@copejon
Copy link
Contributor Author

copejon commented May 30, 2024

  • microshift-metal-periodic test failed due to aws infra issue.
  • microshift-metal-test failed due to curl timeout. The error should not have been fatal but the RF logic only checks for the string error in the rpm install output. Install actually succeeded
  • microshift-metal-tests-arm failed during OS install when it timed out while trying to download a dependency from the local repo:
[  190.440070] anaconda[2595]: Failed to pull from repository: g-io-error-quark: While fetching http://192.168.
[  190.440746] anaconda[2595]: 122.1:8080/repo/objects/59/1a8d6991f9804a1674cf7cb8d9a8be39dcbc65281bc00f8c527e7
[  190.441431] anaconda[2595]: b43a296cf.filez: [28] Timeout was reached (0)

@copejon
Copy link
Contributor Author

copejon commented May 30, 2024

/retest

@ggiguash
Copy link
Contributor

/test metal-periodic-test

@ggiguash
Copy link
Contributor

We need the #3415 PR to merge before we can get a successful run.
These amount of VMs seems to be too heavy for the default nginx config.

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD d9ed425 and 2 for PR HEAD 3d82dafbba8060b2bb4929791a2898ab914294a9 in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD e368f30 and 1 for PR HEAD 3d82dafbba8060b2bb4929791a2898ab914294a9 in total

@copejon
Copy link
Contributor Author

copejon commented May 31, 2024

el93 OS installation has consistently failed with the following:

[  132.686196] anaconda[2622]: Failed to pull from repository: g-io-error-quark: While fetching http://192.168.
[  132.686289] anaconda[2622]: 122.1:8080/repo/objects/1e/a4dd1d5fe17f4b061e40837e14c58f923e8750fabeba6d9955f9b
[  132.686374] anaconda[2622]: 6b06f8b51.filez: [28] Timeout was reached (0)

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 0f9c968 and 0 for PR HEAD 3d82dafbba8060b2bb4929791a2898ab914294a9 in total

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 31, 2024
@copejon copejon force-pushed the ushift-2970-rhel-94-periodics branch from 3d82daf to dc9f61e Compare May 31, 2024 19:14
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label May 31, 2024
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 31, 2024
… rpms not being available yet, so these will also need to be disabled temporarily

crel is now 4.17 with does not exist yet. this break scenarios expecting a crel image, which is not built during build phase. we will need these tests when a ec/rc 4.17 is available, so disabling the tests with a bogus extension is preferable to deleting them

moved el92-yminus@[email protected] to periodics (where it was intended to be). this should fix the boot failure caused by missing image, since the ostree commit isn't built in presubmits

removed rhel93-prel-with-optionals.toml as prel will be 4.15 which does not support multus

removed rhel92-prel-with-optionals as multus is not supported for 4.15

start [email protected] and [email protected] to install with 92 as a base, the immediately upgrade to 94 via ostree

Update test/scenarios/el92-yminus@[email protected]

Updated periodics to include tests from yminus2 and previous minor releases to el94 current release and source combinations

Signed-off-by: Jon Cope <[email protected]>

Update test/scenarios-periodics/el94-crel@[email protected]

Update test/image-blueprints/layer1-base/group2/rhel93-microshift-yminus2.toml

Update test/scenarios-periodics/el94-crel@[email protected]

add new periodic images for optionals

Co-authored-by: Patryk Matuszak <[email protected]>
@copejon copejon force-pushed the ushift-2970-rhel-94-periodics branch from dc9f61e to 93d6090 Compare May 31, 2024 19:17
@ggiguash
Copy link
Contributor

ggiguash commented Jun 4, 2024

/retest-required

@ggiguash
Copy link
Contributor

ggiguash commented Jun 4, 2024

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 4, 2024
Copy link
Contributor

openshift-ci bot commented Jun 4, 2024

[APPROVALNOTIFIER] This PR is APPROVED

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

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

The pull request process is described here

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-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 1363db4 and 2 for PR HEAD 93d6090 in total

@ggiguash
Copy link
Contributor

ggiguash commented Jun 4, 2024

/test metal-periodic-test-arm

1 similar comment
@ggiguash
Copy link
Contributor

ggiguash commented Jun 4, 2024

/test metal-periodic-test-arm

@ggiguash
Copy link
Contributor

ggiguash commented Jun 4, 2024

/retest-required

Copy link
Contributor

openshift-ci bot commented Jun 4, 2024

@copejon: The following test 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/prow/metal-bootc-test-arm fd0464c09b94ab78df334859bddb6ceab73b7091 link true /test metal-bootc-test-arm

Full PR test history. Your PR dashboard.

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

@ggiguash
Copy link
Contributor

ggiguash commented Jun 4, 2024

/test metal-bootc-test metal-periodic-test-arm

@ggiguash
Copy link
Contributor

ggiguash commented Jun 4, 2024

/override ci/prow/metal-bootc-test

Copy link
Contributor

openshift-ci bot commented Jun 4, 2024

@ggiguash: Overrode contexts on behalf of ggiguash: ci/prow/metal-bootc-test

In response to this:

/override ci/prow/metal-bootc-test

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-sigs/prow repository.

@openshift-merge-bot openshift-merge-bot bot merged commit 915fb85 into openshift:main Jun 4, 2024
@copejon
Copy link
Contributor Author

copejon commented Jun 4, 2024

/cherry-pick release-4.16

@openshift-cherrypick-robot

@copejon: new pull request created: #3434

In response to this:

/cherry-pick release-4.16

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-sigs/prow repository.

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.

6 participants