Skip to content

Generate Downtimes commands for pup - #829

Open
dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
datadog-api-spec/generated/6744
Open

dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
datadog-api-spec/generated/6744

Conversation

@dd-octo-sts

@dd-octo-sts dd-octo-sts Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

PLEASE, DO NOT MERGE THIS PULL REQUEST MANUALLY. This is automatically managed by Datadog Tooling

See DataDog/datadog-api-spec#6744 Test branch datadog-api-spec/test/ulysse.mavrocordatos/AAWF-1386/pup-downtimes-pilot

@dd-octo-sts
dd-octo-sts Bot requested a review from a team as a code owner September 16, 2026 08:48
@dd-octo-sts dd-octo-sts Bot added the changelog/no-changelog Changes don't appear in changelog label Sep 16, 2026
@datadog-datadog-prod-us1

This comment has been minimized.

@datadog-datadog-prod-us1 datadog-datadog-prod-us1 Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Datadog Autotest: FAIL

The new cancel command ignores the selected output format. The two list commands also mark partial pages as complete.

Open Bits AI session

🤖 Datadog Autotest · Commit 9187732 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest

api.cancel_downtime(downtime_id)
.await
.map_err(|e| anyhow::anyhow!("failed to cancel_downtime: {:?}", e))?;
println!("downtimes cancel: ok");

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Use the shared formatter for cancel output

Agents and scripts cannot parse a successful cancel result.

Assertion details
  • Input: Run pup --agent downtimes cancel <valid-id> or request JSON output.
  • Expected: The command must use the selected output format and the agent JSON envelope.
  • Actual: The command prints downtimes cancel: ok as plain text.

Was this helpful? React 👍 or 👎
🤖 Datadog Autotest · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest · Open Bits AI session

.await
.map_err(|e| anyhow::anyhow!("failed to list_downtimes: {:?}", e))?;
let count = resp.data.as_ref().map_or(0, |d| d.len());
let truncated = false;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Report partial downtime pages

Agents can stop after the first page and omit valid downtimes.

Assertion details
  • Input: The API returns fewer records than total_filtered_count, such as 30 of 31.
  • Expected: The metadata must mark the page as partial and give the next offset.
  • Actual: Both list functions set truncated to false and omit the next action.

Was this helpful? React 👍 or 👎
🤖 Datadog Autotest · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest · Open Bits AI session

@MintsInc

Copy link
Copy Markdown
Member

You can ignore this for now this is a WIP

@dd-octo-sts
dd-octo-sts Bot force-pushed the datadog-api-spec/generated/6744 branch from 2d556f9 to 9f3b48a Compare September 18, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog Changes don't appear in changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant