Skip to content

STOR-2543: Add volumeattributesclass.yaml support for AWS EBS CSI VolumeAttributesClass tests #72189

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

Merged
openshift-merge-bot merged 1 commit into from
Dec 8, 2025
Merged

STOR-2543: Add volumeattributesclass.yaml support for AWS EBS CSI VolumeAttributesClass tests #72189

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions ci-operator/step-registry/storage/conf/csi-aws-ebs/storage-conf-csi-aws-ebs-commands.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ if [ -d /go/src/github.com/openshift/csi-operator/ ]; then
echo "Using OCP specific manifest ${SHARED_DIR}/${TEST_OCP_CSI_DRIVER_MANIFEST}:"
cat ${SHARED_DIR}/${TEST_OCP_CSI_DRIVER_MANIFEST}
fi
echo "Copying ${TEST_VOLUME_ATTRIBUTES_CLASS_MANIFEST} to ${SHARED_DIR}/${TEST_VOLUME_ATTRIBUTES_CLASS_MANIFEST}"
cp test/e2e/aws-ebs/${TEST_VOLUME_ATTRIBUTES_CLASS_MANIFEST} ${SHARED_DIR}/${TEST_VOLUME_ATTRIBUTES_CLASS_MANIFEST}
Copy link
Member

@gnufied gnufied Dec 8, 2025

Choose a reason for hiding this comment

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

So is TEST_VOLUME_ATTRIBUTES_CLASS_MANIFEST specified a relative path in test/e2e/aws-ebs or is absolute path?

Copy link
Member

@gnufied gnufied Dec 8, 2025

Choose a reason for hiding this comment

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

It is probably fine though.

Copy link
Contributor Author

@radeore radeore Dec 8, 2025

Choose a reason for hiding this comment

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

It is a relative path in csi-operator/test/e2e/aws-ebs

cat ${SHARED_DIR}/${TEST_VOLUME_ATTRIBUTES_CLASS_MANIFEST}
else
echo "Using aws-ebs-csi-driver-operator repo"
cd /go/src/github.com/openshift/aws-ebs-csi-driver-operator
Expand Down
5 changes: 5 additions & 0 deletions ci-operator/step-registry/storage/conf/csi-aws-ebs/storage-conf-csi-aws-ebs-ref.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,11 @@ ref:
documentation: |-
Enable CSI certification test for SCSI LUN overflow that creates 260 volumes + pods.
default: "false"
- name: TEST_VOLUME_ATTRIBUTES_CLASS_MANIFEST
default: "volumeattributesclass.yaml"
documentation: |-
The VolumeAttributesClass manifest to create. Used by the `openshift-tests` as a
manifest for openshift/csi tests. The file must be in ${SHARED_DIR}.
documentation: |-
The csi-aws-ebs step creates ${SHARED_DIR}/${TEST_CSI_DRIVER_MANIFEST} and
${SHARED_DIR}/${TEST_OCP_CSI_DRIVER_MANIFEST} for AWS EBS CSI driver, which is consumed by openshift-e2e-test step to
Expand Down