Skip to content

Conversation

@dd-dominic
Copy link
Contributor

What does this PR do? What is the motivation?

Add Developer Platform instructions for Agent-based integrations

@dd-dominic dd-dominic requested review from a team as code owners October 29, 2025 20:31
@github-actions
Copy link
Contributor

github-actions bot commented Oct 29, 2025

@OliviaShoup OliviaShoup added the editorial review Waiting on a more in-depth review label Oct 29, 2025
@dd-dominic dd-dominic added the WORK IN PROGRESS No review needed, it's a wip ;) label Oct 30, 2025
@dd-dominic dd-dominic removed the WORK IN PROGRESS No review needed, it's a wip ;) label Oct 31, 2025
@dd-dominic dd-dominic requested a review from eho1307 October 31, 2025 17:56
Copy link
Contributor

@evazorro evazorro left a comment

Choose a reason for hiding this comment

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

A few fixes and a note/question about formatting on some of the procedures. I'd love to take another look before merge.

dd-dominic and others added 3 commits November 6, 2025 11:54
Co-authored-by: Erica Ho <109224841+eho1307@users.noreply.github.com>
Co-authored-by: Erica Ho <109224841+eho1307@users.noreply.github.com>
@dd-dominic dd-dominic requested a review from a team as a code owner November 7, 2025 12:12
Ensure following tools are installed:

## Prerequisites
- Python v3.12
Copy link
Contributor

Choose a reason for hiding this comment

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

We're using Python 3.13 now - perhaps we could remove this step here since it's part of the ddev steps anyway?

The following example shows logic for an integration named `Awesome`.

For Awesome, the Agent Check is composed of a [service check][25] named `awesome.search` that searches for a string on a web page. It results in `OK` if the string is present, `WARNING` if the page is accessible but the string was not found, and `CRITICAL` if the page is inaccessible.
This check defines a [service check][4] called `awesome.search`, which searches a webpage for a specific string:
Copy link
Contributor

@nubtron nubtron Nov 7, 2025

Choose a reason for hiding this comment

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

(non-blocker) I'll make an issue to remove service checks from examples in general, we want to move away from them because metrics are better for the same purpose.


<div class="alert alert-info">If you are creating an integration for the Datadog Marketplace, ensure that your directory follows the pattern of {partner name}_{integration name}.</div>
```shell
ddev create -t check_only <YOUR_INTEGRATION_NAME> --skip-manifest
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the reason behind using check_only?

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

Labels

editorial review Waiting on a more in-depth review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants