Skip to content

USHIFT-6168: Adding scenario to enable all configs#5579

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
kasturinarra:USHIFT-6168
Nov 20, 2025
Merged

USHIFT-6168: Adding scenario to enable all configs#5579
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
kasturinarra:USHIFT-6168

Conversation

@kasturinarra
Copy link
Contributor

@kasturinarra kasturinarra commented Oct 8, 2025 •
edited by openshift-ci bot
Loading

Which issue(s) this PR addresses: This PR helps to add a scenario that enables all configs.

Closes #USHIFT-6168

@openshift-ci openshift-ci bot requested review from jerpeter1 and pmtk October 8, 2025 10:26
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 8, 2025
@kasturinarra kasturinarra changed the title Adding scenario to enable all configs USHIFT-6168: Adding scenario to enable all configs Oct 8, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 8, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 8, 2025 •
edited by openshift-ci bot
Loading

@kasturinarra: This pull request references USHIFT-6168 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.21.0" version, but no target version was set.

Details

In response to this:

Which issue(s) this PR addresses: This PR helps to add a scenario that enables all configs.

Closes #USHIFT-6168

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 openshift-eng/jira-lifecycle-plugin repository.

ggiguash
ggiguash suggested changes Oct 12, 2025
View reviewed changes
test/scenarios-bootc/releases/el96-lrel@multi-config.sh
EOF"

# Restart MicroShift to apply TLS configuration
run_command_on_vm host1 "sudo systemctl restart microshift"
Copy link
Contributor

@ggiguash ggiguash Oct 12, 2025

Choose a reason for hiding this comment

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

Use healthcheck to verify that MicroShift is fully initialized after restart.
Otherwise, we're risking to run into race conditions in the subsequent scripts.

test/scenarios/releases/el96-lrel@ginkgo-multi-config.sh
EOF"

# Restart MicroShift to apply TLS configuration
run_command_on_vm host1 "sudo systemctl restart microshift"
Copy link
Contributor

@ggiguash ggiguash Oct 12, 2025

Choose a reason for hiding this comment

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

Use healthcheck to verify that MicroShift is fully initialized after restart.
Otherwise, we're risking to run into race conditions in the subsequent scripts.

@kasturinarra kasturinarra force-pushed the USHIFT-6168 branch 2 times, most recently from 0ae05f9 to 16cac1b Compare October 21, 2025 06:34
@kasturinarra
Copy link
Contributor Author

kasturinarra commented Oct 22, 2025

/retest

@kasturinarra kasturinarra force-pushed the USHIFT-6168 branch 4 times, most recently from 24aabf9 to dd1724b Compare October 22, 2025 16:51
@kasturinarra
Copy link
Contributor Author

kasturinarra commented Oct 23, 2025

/test e2e-aws-tests-release

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 3, 2025

/test test-unit

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 3, 2025

/test e2e-aws-tests-bootc-periodic

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 3, 2025

/test test-unit

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 5, 2025

/retest

@kasturinarra kasturinarra marked this pull request as draft November 5, 2025 10:40
@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 Nov 5, 2025
@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 5, 2025

/test e2e-aws-tests-release

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 5, 2025

/test e2e-aws-tests-release

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 17, 2025

@agullon all tests passed, could you please help take a look ? thanks !!

@agullon
Copy link
Contributor

agullon commented Nov 17, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 17, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 17, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: agullon, kasturinarra

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 [agullon,kasturinarra]

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

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 17, 2025

/verified by CI

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Nov 17, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 17, 2025

@kasturinarra: This PR has been marked as verified by CI.

Details

In response to this:

/verified by CI

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 17, 2025

/retest-required

Remaining retests: 0 against base HEAD f84ceaf and 2 for PR HEAD fdf66e5 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 17, 2025

/retest-required

Remaining retests: 0 against base HEAD d70b24b and 1 for PR HEAD fdf66e5 in total

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 18, 2025

/retest

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 18, 2025

/test e2e-aws-tests-bootc /test e2e-aws-tests-bootc-arm

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 18, 2025

/retest-required

Remaining retests: 0 against base HEAD 39bba2e and 0 for PR HEAD fdf66e5 in total

@kasturinarra kasturinarra marked this pull request as draft November 18, 2025 13:16
@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 Nov 18, 2025
@kasturinarra kasturinarra marked this pull request as ready for review November 20, 2025 03:05
@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 Nov 20, 2025
@openshift-ci openshift-ci bot requested a review from copejon November 20, 2025 03:06
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 20, 2025 •
edited
Loading

@kasturinarra: 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/prow/e2e-aws-tests-periodic-arm 38ac776 link true /test e2e-aws-tests-periodic-arm
ci/prow/e2e-aws-tests-bootc-periodic-arm 38ac776 link true /test e2e-aws-tests-bootc-periodic-arm

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

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 20, 2025

/hold

Revision fdf66e5 was retested 3 times: holding

@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 Nov 20, 2025
@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 20, 2025

/retest

@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 20, 2025

/unhold

@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 Nov 20, 2025
@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 20, 2025

/verified by CI

@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 20, 2025

@kasturinarra: This PR has been marked as verified by CI.

Details

In response to this:

/verified by CI

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-merge-bot openshift-merge-bot bot merged commit 64c5d56 into openshift:main Nov 20, 2025
13 checks passed
@kasturinarra
Copy link
Contributor Author

kasturinarra commented Nov 20, 2025

/cherry-pick release-4.20

@openshift-cherrypick-robot
Copy link

openshift-cherrypick-robot commented Nov 20, 2025

@kasturinarra: new pull request created: #5772

Details

In response to this:

/cherry-pick release-4.20

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-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Nov 20, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jerpeter1 jerpeter1 Awaiting requested review from jerpeter1

@pmtk pmtk Awaiting requested review from pmtk

@ggiguash ggiguash Awaiting requested review from ggiguash

@vanhalenar vanhalenar Awaiting requested review from vanhalenar

@jogeo jogeo Awaiting requested review from jogeo

@copejon copejon Awaiting requested review from copejon

1 more reviewer

@agullon agullon agullon left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@agullon agullon

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. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kasturinarra @openshift-ci-robot @agullon @openshift-cherrypick-robot @ggiguash