Skip to content

hcp/openstack: periodic jobs for Cinder & Manila CSI#59089

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
shiftstack:periodic-csi-hcp
Dec 6, 2024
Merged

hcp/openstack: periodic jobs for Cinder & Manila CSI#59089
openshift-merge-bot[bot] merged 1 commit intoopenshift:masterfrom
shiftstack:periodic-csi-hcp

Conversation

@EmilienM
Copy link
Copy Markdown
Member

@EmilienM EmilienM commented Nov 21, 2024

Add periodic jobs for Hypershift on OpenStack with Cinder and Manila
CSI.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Nov 21, 2024

/cc stephenfin MaysaMacedo mandre

FYI about this PR.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Nov 21, 2024

/hold
I want to make sure things are stabilized, we still have a couple of patches to land and I want to see how it works here before anything.

@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 21, 2024
@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Nov 21, 2024

/pj-rehearse periodic-ci-openshift-hypershift-release-4.18-periodics-e2e-openstack-csi-cinder

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Nov 21, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@MaysaMacedo
Copy link
Copy Markdown
Contributor

MaysaMacedo commented Nov 27, 2024

/lgtm
/hold

depends on the backport of openshift/hypershift#5178

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Nov 27, 2024
@openshift-ci openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 28, 2024
@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Nov 28, 2024

/pj-rehearse periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-cinder periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-manila

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Nov 28, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Nov 28, 2024

/pj-rehearse periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-cinder periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-manila

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Nov 28, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

1 similar comment
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Dec 2, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 2, 2024

/pj-rehearse periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-manila periodic-ci-openshift-hypershift-release-4.18-periodics-e2e-openstack-csi-manila periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-cinder periodic-ci-openshift-hypershift-release-4.18-periodics-e2e-openstack-csi-cinder

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Dec 2, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 2, 2024

We need openshift/hypershift#5198 to be landed first.

@EmilienM
hcp/openstack: periodic jobs for Cinder & Manila CSI
86d3291 
Add periodic jobs for Hypershift on OpenStack with Cinder and Manila
CSI.
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Dec 4, 2024

[REHEARSALNOTIFIER]
@EmilienM: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-manila N/A periodic Periodic changed
periodic-ci-openshift-hypershift-release-4.18-periodics-e2e-openstack-csi-manila N/A periodic Periodic changed
periodic-ci-openshift-hypershift-release-4.18-periodics-e2e-openstack-csi-cinder N/A periodic Periodic changed
periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-cinder N/A periodic Periodic changed
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 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 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

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.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 4, 2024

/hold cancel

@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 Dec 4, 2024
@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 4, 2024

/pj-rehearse periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-manila periodic-ci-openshift-hypershift-release-4.19-periodics-e2e-openstack-csi-cinder
let's start with these two first.

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Dec 4, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 4, 2024

I'm waiting on a new release payload on 4.18 including the CIRO change in HCP before rehearsing the 4.18 jobs here.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 4, 2024

/pj-rehearse periodic-ci-openshift-hypershift-release-4.18-periodics-e2e-openstack-csi-cinder periodic-ci-openshift-hypershift-release-4.18-periodics-e2e-openstack-csi-manila

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Dec 4, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 4, 2024

@EmilienM: all tests passed!

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.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 4, 2024

/pj-rehearse ack

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

openshift-ci-robot commented Dec 4, 2024

@EmilienM: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Dec 4, 2024
@bryan-cox
Copy link
Copy Markdown
Member

bryan-cox commented Dec 5, 2024

/approve

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 5, 2024
mandre
mandre reviewed Dec 5, 2024
View reviewed changes
Comment thread ci-operator/config/openshift/hypershift/openshift-hypershift-release-4.18__periodics.yaml
steps:
cluster_profile: hypershift
env:
CLUSTER_TYPE_OVERRIDE: openstack-vh-mecha-central
Copy link
Copy Markdown
Member

@mandre mandre Dec 5, 2024

Choose a reason for hiding this comment

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

do they need to run on mecha-central? Other environments might be less loaded.

Copy link
Copy Markdown
Member Author

@EmilienM EmilienM Dec 5, 2024

Choose a reason for hiding this comment

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

yes, CSI tests don't run fine on VEXXHOST yet. We'll take care of that.

@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 6, 2024

/hold
We'll probably need to rebase it on top of #59598

@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 Dec 6, 2024
@EmilienM
Copy link
Copy Markdown
Member Author

EmilienM commented Dec 6, 2024

/hold cancel
the rename of the job won't land today so I'm trying to get things done, let's merge this one first. I'll update the rename PR.

@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 Dec 6, 2024
@MaysaMacedo
Copy link
Copy Markdown
Contributor

MaysaMacedo commented Dec 6, 2024

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Dec 6, 2024
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 6, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bryan-cox, EmilienM, MaysaMacedo

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-merge-bot openshift-merge-bot Bot merged commit 7231450 into openshift:master Dec 6, 2024
@openshift-merge-bot openshift-merge-bot Bot deleted the periodic-csi-hcp branch December 6, 2024 15:44
yingzhanredhat pushed a commit to yingzhanredhat/release that referenced this pull request Dec 18, 2024
@EmilienM @yingzhanredhat
hcp/openstack: periodic jobs for Cinder & Manila CSI (openshift#59089)
f0e0324 
Add periodic jobs for Hypershift on OpenStack with Cinder and Manila
CSI.
yingzhanredhat pushed a commit to yingzhanredhat/release that referenced this pull request Dec 24, 2024
@EmilienM @yingzhanredhat
hcp/openstack: periodic jobs for Cinder & Manila CSI (openshift#59089)
26982f3 
Add periodic jobs for Hypershift on OpenStack with Cinder and Manila
CSI.
krishvoor pushed a commit to krishvoor/release that referenced this pull request Jan 29, 2025
@EmilienM @krishvoor
hcp/openstack: periodic jobs for Cinder & Manila CSI (openshift#59089)
38435bc 
Add periodic jobs for Hypershift on OpenStack with Cinder and Manila
CSI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MaysaMacedo MaysaMacedo Awaiting requested review from MaysaMacedo

@stephenfin stephenfin Awaiting requested review from stephenfin

@csrwng csrwng Awaiting requested review from csrwng

@enxebre enxebre Awaiting requested review from enxebre

1 more reviewer

@mandre mandre mandre left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@MaysaMacedo MaysaMacedo

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. 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

@EmilienM @openshift-ci-robot @MaysaMacedo @bryan-cox @mandre