[ROB-2864] updated docs for gcp

[Avi-Robusta]

No description provided.

[coderabbitai]

Walkthrough

Documentation updated for Google Cloud Managed Prometheus: title and quick-start phrasing reworded, prerequisites simplified to require Managed Prometheus and a Prometheus Frontend endpoint, exporter and alertmanager_url references removed, and example YAML comments and formatting adjusted.

Changes

Cohort / File(s)

Summary

Documentation docs/configuration/metric-providers-google.rst

Reworded title/intro and Quick Start phrasing; simplified prerequisites to require Google Managed Prometheus and a Prometheus Frontend endpoint (link guidance added); removed legacy exporter listings and the alertmanager_url example; updated YAML snippet to show a concrete Prometheus frontend URL, added inline comment about frontend URL location/namespace, retained check_prometheus_flags: false, and made minor wording/formatting tweaks.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks and finishing touches

❌ 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 relevance to the changeset.	Add a description explaining the purpose of the documentation updates and any context for the changes made.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly indicates the main change: updating documentation for Google Cloud Platform (GCP) configuration.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch update-gcp-docs

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eb67fb7 and 8e2cec0.

📒 Files selected for processing (1)

• docs/configuration/metric-providers-google.rst

🚧 Files skipped from review as they are similar to previous changes (1)

• docs/configuration/metric-providers-google.rst

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: run_tests
• GitHub Check: run_tests
• GitHub Check: Deploy docs

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

docs/configuration/metric-providers-google.rst (1)

42-45: Clarify whether AlertManager configuration is required.

The Service Locations section documents AlertManager in the gmp-system namespace (lines 42–45), but the Quick Start YAML example doesn't include an alertmanager_url configuration. Add a brief note explaining whether users need to configure AlertManager separately or if it's automatically discovered/optional.

Also applies to: 22-28

🧹 Nitpick comments (1)

docs/configuration/metric-providers-google.rst (1)

60-66: Configuration Notes are helpful but could be more specific.

The warning about the Prometheus flags API is clear and necessary. However, the bullet points about exporters and architecture differences are somewhat vague. Consider either expanding these with concrete examples or removing them if they don't directly affect the configuration steps.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7b10d13 and 0d0c21d.

📒 Files selected for processing (1)

• docs/configuration/metric-providers-google.rst

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: run_tests
• GitHub Check: run_tests
• GitHub Check: Deploy docs

🔇 Additional comments (3)

docs/configuration/metric-providers-google.rst (3)

18-28: Excellent simplification of the Quick Start configuration.

The YAML example is now more user-focused with clear placeholder guidance and a practical in-cluster example. The removal of extraneous fields and the addition of explicit comments about required settings (especially check_prometheus_flags: false) greatly improve usability.

---

35-56: Verify service location examples match actual deployments.

The Service Locations section assumes that:

• The Prometheus Frontend service is in the default namespace
• The AlertManager service is in the gmp-system namespace

Confirm these are the standard/typical locations for Google Managed Prometheus deployments, or add a note that users should adjust based on their cluster configuration.

---

68-93: Verification section is well-structured and practical.

The three verification steps (connectivity check, demo alert, log inspection) provide a good user experience and help with troubleshooting. The commands are concrete and immediately actionable.