-
Notifications
You must be signed in to change notification settings - Fork 59
Deploy CloudSQL instance for performance tests #3634
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
Conversation
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
| enabled: true, | ||
| maintenanceWindow: { day: 2, hour: 8 }, | ||
| protected: false, | ||
| tier: 'db-custom-2-7680', // same as devnet & testnet as of Jan 2026 |
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.
the actual configuration is available in the internal repo, not in this one...
that being said, it's likely we'll increase the DB size, not make it smaller, so this should be fine as baseline
| HELM_MAX_HISTORY_SIZE: 42, | ||
| imagePullSecretByNamespaceNameForServiceAccount: () => [], | ||
| infraAffinityAndTolerations: {}, | ||
| CloudPostgres: function CloudPostgres() { |
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 below breaks without this
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
| name: GHA_NAMESPACE_NAME, | ||
| }, | ||
| }); | ||
| const exactNs: ExactNamespace = { |
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.
One day I'll understand why we have this ExactNamespace all over the place...
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.
you mean this?
| // There is no way to read the logical name off a Namespace. Exactly |
isegall-da
left a comment
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.
Nice, thank you
This reverts commit 35543de.
This reverts commit 35543de. Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
* Revert "[static] Add missing secretKeyRef in GHA setup (#3638)" This reverts commit 080c7d9. * Revert "Deploy CloudSQL instance for performance tests (#3634)" This reverts commit 35543de. * [ci] run Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com> --------- Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Part of #3562
I expect things to break (missing permissions). After this is deployed: