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

Update providers during perf measure #1576

Merged
merged 2 commits into from
Feb 6, 2024

Conversation

t0yv0
Copy link
Member

@t0yv0 t0yv0 commented Feb 6, 2024

For Python and TypeScript, provider references are floating, but for Go they are always pinned. As a result TestAccAwsGoS3Folder was always testing the performance of aws v6.0.2 not the later versions that shipped some performance improvements. Compensate for this in the benchmark by updating to latest.

Copy link

@VenelinMartinov VenelinMartinov left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!

Copy link

pulumi-staging bot commented Feb 6, 2024

🍹 The Update for pulumi/k8s-ci-cluster/0d986fc976039db30aba603701e612c8c2c73ea9-1677 was successful.

Resource Changes

    Name                                                          Type                                        Operation
+   primary-node-pool                                             gcp:container/nodePool:NodePool             create
+   gke                                                           pulumi:providers:kubernetes                 create
+   k8s-ci-cluster-0d986fc976039db30aba603701e612c8c2c73ea9-1677  pulumi:pulumi:Stack                         create
+   multicloud                                                    pulumi-kubernetes:ci:GkeCluster             create
+   password                                                      random:index/randomPassword:RandomPassword  create
+   ephemeral-ci-cluster                                          gcp:container/cluster:Cluster               create

Copy link

pulumi-staging bot commented Feb 6, 2024

🍹 The Destroy for pulumi/k8s-ci-cluster/0d986fc976039db30aba603701e612c8c2c73ea9-1677 was successful.

Resource Changes

    Name                                                          Type                                        Operation
-   gke                                                           pulumi:providers:kubernetes                 delete
-   primary-node-pool                                             gcp:container/nodePool:NodePool             delete
-   ephemeral-ci-cluster                                          gcp:container/cluster:Cluster               delete
-   multicloud                                                    pulumi-kubernetes:ci:GkeCluster             delete
-   password                                                      random:index/randomPassword:RandomPassword  delete
-   k8s-ci-cluster-0d986fc976039db30aba603701e612c8c2c73ea9-1677  pulumi:pulumi:Stack                         delete

@t0yv0
Copy link
Member Author

t0yv0 commented Feb 6, 2024

Test failure appears unrelated, I think I can merge. I'll revert if something breaks in the metrics pipeline per se.

@t0yv0 t0yv0 merged commit 2bdfd3c into master Feb 6, 2024
50 of 51 checks passed
@t0yv0 t0yv0 deleted the t0yv0/update-providers-during-perf-measure branch February 6, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants