Skip to content

WIP: STOR-2534: UPSTREAM: 135069: Update group snapshot tests to v1beta2 #2508

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
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

WIP: STOR-2534: UPSTREAM: 135069: Update group snapshot tests to v1beta2 #2508

wants to merge 2 commits into from

Conversation

@jsafrane
Copy link

@jsafrane jsafrane commented Nov 6, 2025

Update e2e tests to use v1beta2 volume group snapshot API.

@jsafrane
UPSTREAM: 135069: Update group snapshot tests to v1beta2
4747d7d 
Copy VolumeGroupSnapshot v1beta2 CRDs from the external-snapshotter and
disable v1beta1 API in them - we can't run the conversion webhook in e2e
tests easily.

Update e2e tests to use v1beta2.
@openshift-ci-robot openshift-ci-robot added backports/validated-commits Indicates that all commits come to merged upstream PRs. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. labels Nov 6, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 6, 2025 •
edited by openshift-ci bot
Loading

@jsafrane: This pull request references STOR-2534 which is a valid jira issue.

In response to this:

Update e2e tests to use v1beta2 volume group snapshot API.

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 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 6, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 6, 2025

@jsafrane: the contents of this pull request could be automatically validated.

The following commits are valid:

Comment /validate-backports to re-evaluate validity of the upstream PRs, for example when they are merged upstream.

@openshift-ci openshift-ci bot requested review from p0lyn0mial and tkashem November 6, 2025 13:28
@openshift-ci
Copy link

openshift-ci bot commented Nov 6, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jsafrane
Once this PR has been reviewed and has the lgtm label, please assign tkashem for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@jsafrane
UPSTREAM: 135135: Update CSI manifests from csi-driver-hostpath master
9ccc0a6 
This is only a part of the cherry-pick to update the external snapshotter
to get v1beta2 group snapshot support.
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 6, 2025

@jsafrane: the contents of this pull request could be automatically validated.

The following commits are valid:

Comment /validate-backports to re-evaluate validity of the upstream PRs, for example when they are merged upstream.

@openshift-ci
Copy link

openshift-ci bot commented Nov 6, 2025

@jsafrane: 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-ovn-techpreview-serial-1of2 9ccc0a6 link false /test e2e-aws-ovn-techpreview-serial-1of2
ci/prow/e2e-aws-ovn-serial-1of2 9ccc0a6 link true /test e2e-aws-ovn-serial-1of2
ci/prow/e2e-aws-crun-wasm 9ccc0a6 link true /test e2e-aws-crun-wasm
ci/prow/e2e-aws-ovn-cgroupsv2 9ccc0a6 link true /test e2e-aws-ovn-cgroupsv2
ci/prow/k8s-e2e-gcp-serial 9ccc0a6 link true /test k8s-e2e-gcp-serial
ci/prow/e2e-gcp 9ccc0a6 link true /test e2e-gcp
ci/prow/k8s-e2e-gcp-ovn 9ccc0a6 link true /test k8s-e2e-gcp-ovn
ci/prow/e2e-aws-ovn-runc 9ccc0a6 link true /test e2e-aws-ovn-runc
ci/prow/e2e-metal-ipi-ovn-ipv6 9ccc0a6 link true /test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-aws-ovn-serial-2of2 9ccc0a6 link true /test e2e-aws-ovn-serial-2of2
ci/prow/e2e-aws-ovn-techpreview-serial-2of2 9ccc0a6 link false /test e2e-aws-ovn-techpreview-serial-2of2
ci/prow/e2e-aws-ovn-crun 9ccc0a6 link true /test e2e-aws-ovn-crun
ci/prow/e2e-aws-ovn-fips 9ccc0a6 link true /test e2e-aws-ovn-fips
ci/prow/e2e-aws-ovn-hypershift 9ccc0a6 link true /test e2e-aws-ovn-hypershift
ci/prow/e2e-aws-ovn-techpreview 9ccc0a6 link false /test e2e-aws-ovn-techpreview

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.

@jsafrane jsafrane mentioned this pull request Nov 7, 2025
Open
@openshift-ci
Copy link

openshift-ci bot commented Nov 7, 2025

@jsafrane: This PR was included in a payload test run from openshift/cluster-csi-snapshot-controller-operator#252
trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-ci-4.21-e2e-aws-ovn-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/97c92ec0-bbe3-11f0-860c-bb4a5945e9fe-0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@p0lyn0mial p0lyn0mial Awaiting requested review from p0lyn0mial

@tkashem tkashem Awaiting requested review from tkashem

Assignees

No one assigned

Labels

backports/validated-commits Indicates that all commits come to merged upstream PRs. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jsafrane @openshift-ci-robot