scripts: update drtprod create drt-large
#123464
Merged
Conversation
This file contains 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
|
This change is |
itsbilal
reviewed
May 2, 2024
scripts/drtprod
Outdated
| @@ -128,17 +128,21 @@ EOF" | |||
| fi | |||
| case "${2}" in | |||
| "drt-main") | |||
There was a problem hiding this comment.
nit: Change this to drt-large too?
There was a problem hiding this comment.
yep, was about to double check this before moving it out of draft, thanks for pointing it out!
herkolategan
force-pushed
the
hbl/drt-large-script
branch
from
9ccb9e8
to
4af8674
Compare
herkolategan
force-pushed
the
hbl/drt-large-script
branch
from
4af8674
to
f5d7bdb
Compare
itsbilal
approved these changes
May 9, 2024
scripts/drtprod
Outdated
| --clouds gce \ | ||
| --gce-managed \ | ||
| --gce-zones "us-east1-b,us-west1-b,europe-west2-b" \ | ||
| --gce-enable-multiple-stores \ | ||
| --gce-zones "northamerica-northeast2-a:2,northamerica-northeast2-b:2,northamerica-northeast2-c:1,europe-central2-a:2,europe-central2-b:2,europe-central2-c:1,northamerica-northeast1-a:2,northamerica-northeast1-b:2,northamerica-northeast1-c:1" \ |
There was a problem hiding this comment.
should we swap Warsaw for Columbus?
There was a problem hiding this comment.
Yes, I will update it before merging.
herkolategan
force-pushed
the
hbl/drt-large-script
branch
from
f5d7bdb
to
92642b2
Compare
Bring the creation in line with what is currently running on cockroach-drt. Uses zone expansion to balance the nodes evenly across regions. Epic: None Release Note: None
In an effort to save cost on regional network transfers the Warsaw Europe Zone will now be switched to Columbus. Epic: None Release Note: None
herkolategan
force-pushed
the
hbl/drt-large-script
branch
from
0a6d131
to
8d4f6a9
Compare
|
TFTRs! bors r=itsbilal,ajstorm |
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.
Bring the creation in line with what is currently running on cockroach-drt. Uses zone expansion to balance the nodes evenly across regions.
Epic: None
Release Note: None