-
Notifications
You must be signed in to change notification settings - Fork 259
Long running test pipeline for AKS swiftv2 clusters. #4098
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.
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
Long running test pipeline for AKS swiftv2 clusters. #4098
Conversation
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: sivakami-projects <126191544+sivakami-projects@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: sivakami-projects <126191544+sivakami-projects@users.noreply.github.com>
…ween vnet 1 subnet 1 and vnet 1 subnet2.
|
Link to the initial PR #4092 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces automated infrastructure provisioning for long-running tests on AKS Swift v2 clusters. The implementation creates a complete testing environment including AKS clusters, virtual networks, storage accounts, and security controls through Azure DevOps pipelines.
Key Changes:
- Creates a parameterized Azure DevOps pipeline template that orchestrates infrastructure deployment across three jobs
- Implements bash scripts for provisioning Azure resources including AKS clusters, VNets, peerings, storage accounts, NSGs, and private endpoints
- Configures parallel AKS cluster creation to reduce deployment time
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
.pipelines/swiftv2-long-running/template/long-running-pipeline-template.yaml |
Defines the pipeline template with three jobs: resource group creation, parallel AKS cluster creation, and networking/storage setup |
.pipelines/swiftv2-long-running/scripts/create_aks.sh |
Creates two AKS clusters in parallel with different node pool configurations |
.pipelines/swiftv2-long-running/scripts/create_vnets.sh |
Provisions four virtual networks with multiple subnets for test isolation |
.pipelines/swiftv2-long-running/scripts/create_peerings.sh |
Establishes bidirectional VNet peerings between customer networks |
.pipelines/swiftv2-long-running/scripts/create_storage.sh |
Creates storage accounts with security configurations and exports account names as pipeline variables |
.pipelines/swiftv2-long-running/scripts/create_nsg.sh |
Configures network security groups with subnet isolation rules |
.pipelines/swiftv2-long-running/scripts/create_pe.sh |
Sets up private endpoints and DNS zones for secure storage access |
.pipelines/swiftv2-long-running/pipeline.yaml |
Main pipeline definition with parameterized defaults for subscription, location, and VM SKUs |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| @@ -0,0 +1,42 @@ | |||
| trigger: none | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you'll want to change the branch this PR is on
Right now, it's on sivakami-project:long-running-pipeline-test-branch, which is a fork of this repo
I don't think the PR checks and all that let the users run pipeline tests from forks anymore, it has to be a branch from this repo itself
Kind of like you had in #4092, that branch was just long-running-pipeline in this repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test:
This PR introduces a new Azure DevOps pipeline for running long-term tests on AKS Swift v2 clusters.
The pipeline automates the creation of AKS clusters and the necessary networking infrastructure.
Initial PR - #4092