-
Notifications
You must be signed in to change notification settings - Fork 84
Add dashboards to be managed by Terraform. #494
Conversation
This looks like it includes some changes from your other PR - is that intended? |
702740f
to
c7fb8c0
Compare
c7fb8c0
to
2555791
Compare
That was intended. I was trying to chain the changes and base this change on my other unmerged changes. It's ready to review now. |
NOTE: the chart on metric custom.googleapis.com/opencensus/en-verification-server/ratelimit/limitware/rate_limited_count was removed as it's no longer exported by the code.
53f8f77
to
282a864
Compare
Gentle ping? |
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 nit
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sethvargo, yegle The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This was incorrectly removed in google#494.
This was incorrectly removed in google#494.
This was incorrectly removed in #494.
NOTE: the chart on metric
custom.googleapis.com/opencensus/en-verification-server/ratelimit/limitware/rate_limited_count
was removed as it's no longer exported by the code.
The API requires JSON config but writing multi-line strings in JSON is PITA. I've decided to write it in YAML which is more compact and easier to write multi-line queries.
Related issue: #460
Release Note