Skip to content

[WIP][SIMI-376] Add AMS API reference to Databricks Integration Docs#22906

Draft
goncalves-guilherme wants to merge 1 commit intomasterfrom
guilherme.goncalves/semi-376-add-ams-link-to-databricks-docs
Draft

[WIP][SIMI-376] Add AMS API reference to Databricks Integration Docs#22906
goncalves-guilherme wants to merge 1 commit intomasterfrom
guilherme.goncalves/semi-376-add-ams-link-to-databricks-docs

Conversation

@goncalves-guilherme
Copy link

What does this PR do?

Motivation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@github-actions
Copy link
Contributor

⚠️ Recommendation: Add qa/skip-qa label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

Copy link
Contributor

@joepeeples joepeeples left a comment

Choose a reason for hiding this comment

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

Requesting fixing a broken docs link, thanks!

[36]: https://docs.datadoghq.com/data_observability/
[8]: https://docs.datadoghq.com/integrations/spark/#metrics
[9]: https://docs.datadoghq.com/integrations/spark/#service-checks
[37]: https://docs.datadoghq.com/api/latest/other-integrations/
Copy link
Contributor

Choose a reason for hiding this comment

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

This link returns a 404. I'm not sure what the correct URL should be.

Choose a reason for hiding this comment

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

Thanks, this is indeed broken. I forgot to leave this PR in draft, sorry about that. I am waiting for an API doc to be released so that I can publish these changes with an actual valid url

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for the context @goncalves-guilherme! I've dismissed my previous review so it doesn't block the WIP work. Feel free to ping in #documentation when ready for review!

@goncalves-guilherme goncalves-guilherme changed the title [SIMI-376] Add AMS API reference to Databricks Integration Docs [WIP][SIMI-376] Add AMS API reference to Databricks Integration Docs Mar 16, 2026
@goncalves-guilherme goncalves-guilherme marked this pull request as draft March 16, 2026 17:06
@joepeeples joepeeples dismissed their stale review March 16, 2026 18:04

Dismissing requested changes because PR is still a WIP draft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants