Skip to content

GCP quota alerts via pulumi#4296

Merged
stephencompall-DA merged 12 commits intomainfrom
s11/cni-3651-quota-alerts
Mar 23, 2026
Merged

GCP quota alerts via pulumi#4296
stephencompall-DA merged 12 commits intomainfrom
s11/cni-3651-quota-alerts

Conversation

@stephencompall-DA
Copy link
Contributor

@stephencompall-DA stephencompall-DA commented Mar 6, 2026

Fixes DACH-NY/canton-network-internal#3651.

Pull Request Checklist

Cluster Testing

  • If a cluster test is required, comment /cluster_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.
  • If a hard-migration test is required (from the latest release), comment /hdm_test on this PR to request it, and ping someone with access to the DA-internal system to approve it.

PR Guidelines

  • Include any change that might be observable by our partners or affect their deployment in the release notes.
  • Specify fixed issues with Fixes #n, and mention issues worked on using #n
  • Include a screenshot for frontend-related PRs - see README or use your favorite screenshot tool

Merge Guidelines

  • Make the git commit message look sensible when squash-merging on GitHub (most likely: just copy your PR description).

@stephencompall-DA
refactor alertStrategy [skip ci]
dede0d7 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
add quotas alert policy [skip ci]
90f0eb6 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
off by default [skip ci]
ff9868d 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA stephencompall-DA self-assigned this Mar 6, 2026
@stephencompall-DA
remove 0 quotas [skip ci]
0ce9dc1 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
work around google API limitation [skip ci]
0ad892c 
googleapi: Error 400: Alert policies with "prometheus_query_language" condition type can only have a single condition.: provider=google-beta@9.10.0
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
fix aligner [skip ci]
6c8d302 
Field aggregation.perSeriesAligner had an invalid value of "ALIGN_SUM":
The aligner cannot be applied to metrics with kind GAUGE and value type
BOOL.

Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
threshold variable; add cluster label [ci]
4a2e957 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
type-check baseArgs [ci]
48042ed 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
Merge commit '2fa5b3376d9df6e0e67fa7d6e7440efe53c3b69a' into s11/cni-…
0ea7b27 
…3651-quota-alerts

Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>

[skip ci]
@stephencompall-DA
moved setting to base.yaml [skip ci]
b01c246 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA
update resolved [ci]
359c073 
Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA stephencompall-DA marked this pull request as ready for review March 6, 2026 20:39
ray-roestenburg-da
ray-roestenburg-da approved these changes Mar 6, 2026
View reviewed changes
Copy link
Contributor

@ray-roestenburg-da ray-roestenburg-da left a comment

Choose a reason for hiding this comment

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

Thanks!

martinflorian-da
martinflorian-da approved these changes Mar 9, 2026
View reviewed changes
Copy link
Contributor

@martinflorian-da martinflorian-da left a comment

Choose a reason for hiding this comment

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

Thanks! <3

@stephencompall-DA
Merge commit '8d79788eec2dd1fb92f7d217c0feb9a86479bda9' into s11/cni-…
dbf6899 
…3651-quota-alerts

[ci]

Signed-off-by: Stephen Compall <stephen.compall@digitalasset.com>
@stephencompall-DA stephencompall-DA enabled auto-merge (squash) March 23, 2026 13:55
@stephencompall-DA stephencompall-DA merged commit ff6e099 into main Mar 23, 2026
67 checks passed
@stephencompall-DA stephencompall-DA deleted the s11/cni-3651-quota-alerts branch March 23, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ray-roestenburg-da ray-roestenburg-da ray-roestenburg-da approved these changes

@martinflorian-da martinflorian-da martinflorian-da approved these changes

@rautenrieth-da rautenrieth-da Awaiting requested review from rautenrieth-da

Assignees

@stephencompall-DA stephencompall-DA

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stephencompall-DA @ray-roestenburg-da @martinflorian-da