Skip to content
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

Various changes to load testing machinery to support v17 #48399

Merged
merged 4 commits into from
Nov 19, 2024

Conversation

rosstimothy
Copy link
Contributor

No description provided.

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-48399.d3pp5qlev8mo18.amplifyapp.com

@rosstimothy rosstimothy marked this pull request as ready for review November 5, 2024 14:19
@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label Nov 13, 2024
@rosstimothy
Copy link
Contributor Author

Friendly ping @r0mant @GavinFrazar

Copy link
Contributor

@GavinFrazar GavinFrazar left a comment

Choose a reason for hiding this comment

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

Not sure if these were skipped intentionally or not since you changed the assets uri in several other places, FWIW:

$ rg "public.ecr.aws/gravitational/"                                                                                                                                                      (dev) 
loadtest/helm/node-agent/values.yaml
9:  repository: public.ecr.aws/gravitational/teleport-ent-distroless-debug

loadtest/helm/soaktest/values.yaml
2:  repository: public.ecr.aws/gravitational/teleport-ent-distroless-debug

$ rg "charts.releases.teleport"                                                                                                                                                           (dev) 
loadtest/control-plane/init.sh
7:helm repo add teleport https://charts.releases.teleport.dev

assets/loadtest/azure/main.tf Outdated Show resolved Hide resolved
assets/loadtest/azure/variables.tf Show resolved Hide resolved
assets/loadtest/cluster/aws/cluster.yaml Show resolved Hide resolved
assets/loadtest/cluster/aws/cluster.yaml Show resolved Hide resolved
@rosstimothy
Copy link
Contributor Author

Not sure if these were skipped intentionally or not since you changed the assets uri in several other places, FWIW:

$ rg "public.ecr.aws/gravitational/"                                                                                                                                                      (dev) 
loadtest/helm/node-agent/values.yaml
9:  repository: public.ecr.aws/gravitational/teleport-ent-distroless-debug

loadtest/helm/soaktest/values.yaml
2:  repository: public.ecr.aws/gravitational/teleport-ent-distroless-debug

$ rg "charts.releases.teleport"                                                                                                                                                           (dev) 
loadtest/control-plane/init.sh
7:helm repo add teleport https://charts.releases.teleport.dev

This was only updated to reflect the changes that I had to make for my portion of the test plan.

@rosstimothy rosstimothy added this pull request to the merge queue Nov 19, 2024
Merged via the queue into master with commit e5c8a5a Nov 19, 2024
42 checks passed
@rosstimothy rosstimothy deleted the tross/loadtest/v17 branch November 19, 2024 14:19
@public-teleport-github-review-bot

@rosstimothy See the table below for backport results.

Branch Result
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants