Security perf #869
Open
Security perf #869
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
charleswool
requested review from
anson627 and
sumanthreddy29
as code owners
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR creates a new scale and performance test for security features, specifically for testing disabled SSH access on AKS clusters. The test includes three different scale scenarios (small, medium, and large) with varying node counts and automated scheduling.
Key changes:
- Introduces a new security-perf scenario with terraform configurations
- Sets up a comprehensive pipeline with three scale tiers (10, 500, and 1000 nodes)
- Configures automated daily execution schedules for performance testing
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| azure.json | Test input configuration with run ID and region settings |
| azure.tfvars | Terraform variables defining AKS cluster configuration with disabled SSH and multiple node pools |
| new-pipeline-test.yml | Azure DevOps pipeline with three scheduled stages for different scale scenarios |
scenarios/perf-eval/security-perf/terraform-inputs/azure.tfvars
Outdated
Show resolved
Hide resolved
scenarios/perf-eval/security-perf/terraform-inputs/azure.tfvars
Outdated
Show resolved
Hide resolved
Updated AKS configuration for performance evaluation, changing role and names, adjusting node counts and VM sizes, and adding maxPods and nodepool-labels parameters.
Added schedules for small, medium, and large scale performance evaluations. Updated scenario variables and job parameters for Azure environments.
Updated nodeCountMax and maxPods values for performance tuning.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
charleswool
force-pushed
the
security-perf
branch
from
71c0cdc to
347609a
Compare
Updated the node_count for scalepool1 to 251.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
create scale&perf test for security feature-disable SSH
Test pipeline results: https://dev.azure.com/akstelescope/telescope/_build/results?buildId=36989&view=results