Skip to content

Conversation

@Avi-Robusta
Copy link
Contributor

No description provided.

Add documentation for setting up a webhook notification channel in GCP Cloud
Monitoring to forward alerts to Robusta. This covers creating a managed webhook
channel with basic authentication using the Robusta account ID and API key.
Alerts must include a cluster or cluster_name label for Robusta to properly
route them. Added instructions for setting this via custom labels in the
alerting policy's Documentation section.
@github-actions
Copy link

github-actions bot commented Jan 22, 2026

Docker build failed for f8ae754 (after 4s)

View build logs

@coderabbitai
Copy link

coderabbitai bot commented Jan 22, 2026

Walkthrough

Adds a new GCP Cloud Monitoring webhook integration doc and updates documentation navigation to include the new page.

Changes

Cohort / File(s) Summary
GCP Cloud Monitoring Integration Documentation
docs/configuration/alertmanager-integration/gcp-monitoring.rst
New guide describing prerequisites, obtaining Robusta account_id/API key, creating a GCP webhook notification channel (Basic Auth), configuring the webhook endpoint, applying the webhook in alerting policies (requires cluster_name label), and validation steps.
Documentation Navigation Updates
docs/configuration/index.rst, docs/index.rst
Adds a GCP Cloud Monitoring card to the configuration grid and adds the GCP entry to the Send Alerts toctree for navigation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • arikalon1
  • RoiGlinik
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether it relates to the changeset. Add a description explaining the purpose of the GCP Cloud Monitoring webhook documentation, even if brief.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding documentation for GCP Cloud Monitoring webhook alerts integration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/add-gcp-webhook-docs-1K2de

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sheeproid
Sheeproid previously approved these changes Jan 22, 2026
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@docs/configuration/alertmanager-integration/gcp-monitoring.rst`:
- Around line 45-56: Update "Step 3: Use the Webhook in Alerting Policies" to
instruct users to add the cluster_name as an Alert Policy User Label (structured
key/value) instead of placing it in the policy's Documentation section;
specifically, replace the bullet that tells them to add the label in
Documentation with instructions to add a User Label key: cluster_name and value:
your cluster name (or external), and mention that this label will be delivered
in the webhook JSON under incident.policy_user_labels so Robusta can read it.
🧹 Nitpick comments (1)
docs/index.rst (1)

43-43: Consider alphabetical ordering of entries.

The new "GCP Cloud Monitoring" entry is placed after "Google Managed Prometheus" (line 42), but alphabetically "GCP" should come before "Google" (G-C-P vs G-o-o-g-l-e).

📝 Suggested alphabetical ordering
   Embedded Prometheus Stack <configuration/alertmanager-integration/embedded-prometheus>
-  Google Managed Prometheus <configuration/alertmanager-integration/google-managed-prometheus>
   GCP Cloud Monitoring <configuration/alertmanager-integration/gcp-monitoring>
+  Google Managed Prometheus <configuration/alertmanager-integration/google-managed-prometheus>
   Grafana - Self-Hosted <configuration/alertmanager-integration/grafana-self-hosted>

@Avi-Robusta Avi-Robusta merged commit 4f7b821 into master Jan 25, 2026
6 of 7 checks passed
@Avi-Robusta Avi-Robusta deleted the claude/add-gcp-webhook-docs-1K2de branch January 25, 2026 07:29
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.

4 participants