Skip to content

[WIP] test: add 1k QAD csi attach/detach test to existing pipeline - #1348

Draft
hccheng72 wants to merge 11 commits into
mainfrom
csi-attach-detach-1k-qad
Draft

[WIP] test: add 1k QAD csi attach/detach test to existing pipeline#1348
hccheng72 wants to merge 11 commits into
mainfrom
csi-attach-detach-1k-qad

Conversation

@hccheng72

@hccheng72 hccheng72 commented Oct 19, 2026

Copy link
Copy Markdown

This PR:

  1. Adding 1k QAD csi attach/detach test alongside the FAD and AWS tests.
  2. Fixing the existing FAD test with an available k8s version and VM sku.

Some temporary settings:

  • it runs 200 disks for debugging.
  • Assign Contributor role instead of custom role to the QAD workload ID due to the RoleDefinitionLimitExceeded issue.

@github-actions

github-actions Bot commented Oct 19, 2026

Copy link
Copy Markdown

For reviewers only: reply /run-tf-integration to trigger the terraform integration pipeline before approving the PR.

@hccheng72
hccheng72 force-pushed the csi-attach-detach-1k-qad branch from e05c665 to 15f2a8c Compare October 22, 2026 18:40
@hccheng72
Revert "use Contributor role temporarily"
ef7f9df 
This reverts commit 10dc1e4. Preserve the QAD principal and scope exports required by the current cleanup template.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@hccheng72
hccheng72 force-pushed the csi-attach-detach-1k-qad branch from 15f2a8c to ef7f9df Compare October 22, 2026 19:03

This branch has not been deployed

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

Reviewers

@alyssa1303 Alyssa Vu (alyssa1303) Awaiting requested review from alyssa1303 alyssa1303 will be requested when the pull request is marked ready for review alyssa1303 is a code owner
@sumanthreddy29 Sumanth Reddy CH (sumanthreddy29) Awaiting requested review from sumanthreddy29 sumanthreddy29 will be requested when the pull request is marked ready for review sumanthreddy29 is a code owner
@xinWeiWei24 xinWeiWei24 Awaiting requested review from xinWeiWei24 xinWeiWei24 will be requested when the pull request is marked ready for review xinWeiWei24 is a code owner
@vittoriasalim Vittoria (vittoriasalim) Awaiting requested review from vittoriasalim vittoriasalim will be requested when the pull request is marked ready for review vittoriasalim is a code owner
@wonderyl Lei Yao (wonderyl) Awaiting requested review from wonderyl wonderyl will be requested when the pull request is marked ready for review wonderyl is a code owner
@liyu-ma Liyu Ma (liyu-ma) Awaiting requested review from liyu-ma liyu-ma will be requested when the pull request is marked ready for review liyu-ma is a code owner
@LeonardCareer Leonard Du (LeonardCareer) Awaiting requested review from LeonardCareer LeonardCareer will be requested when the pull request is marked ready for review LeonardCareer is a code owner
Copilot code review Copilot
Copilot will automatically review once the pull request is marked ready for review

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hccheng72