Skip to content

chore(deps): upgrade datadog-api-client to master HEAD 2026-05-29#539

Merged
platinummonkey merged 21 commits into
mainfrom
chore/upgrade-dd-sdk-to-master
May 30, 2026
Merged

chore(deps): upgrade datadog-api-client to master HEAD 2026-05-29#539
platinummonkey merged 21 commits into
mainfrom
chore/upgrade-dd-sdk-to-master

Conversation

@platinummonkey
Copy link
Copy Markdown
Collaborator

Summary

Bumps the datadog-api-client Rust SDK pin from v0.30.0 (aa0c8416) to the latest master commit (d4954b11, 2026-05-29), picking up all SDK changes through that date including v0.31.0 and ~25 unreleased commits on top.

Changes

  • Cargo.toml:130-132 — updated comment and rev to d4954b117c5451c0a8932dd0ad0db450fcb0989c
  • Cargo.lock — updated datadog-api-client entry from 0.30.0 / old rev to 0.31.0 / new rev
  • src/commands/incidents.rs — removed api_incident_teams import and teams_list/get/create/update/delete functions + test (SDK PR #1572 removed the api_incident_teams module)
  • src/main.rs — removed IncidentTeamActions enum, Teams variant in IncidentActions, and the dispatch arm
  • src/client.rs — removed 5 incident-team entries from UNSTABLE_OPS; updated test_unstable_ops_count assertion from 167 → 162

Breaking changes handled

SDK PR Change pup impact
#1572 api_incident_teams module removed Removes pup incidents teams subcommand and 5 unstable ops

Testing

  • All changes are compile-mandatory removals; no new logic added
  • test_unstable_ops_count updated to assert 162 (was 167)
  • Removed test_incident_teams_list (tested deleted code)
  • CI will fetch the new SDK rev from GitHub and verify the full build

🤖 Generated with Claude Code


Generated by Claude Code

Bumps the SDK pin from v0.30.0 (aa0c8416) to the latest master commit
(d4954b11) to pick up all changes through 2026-05-29.

Breaking changes handled:
- Remove deprecated incident teams endpoints (SDK #1572): removes
  pup incidents teams command and 5 unstable ops (167 → 162)

Co-Authored-By: Claude <noreply@anthropic.com>
@platinummonkey platinummonkey requested a review from a team as a code owner May 29, 2026 11:37
Wrap three new SDK endpoints introduced in datadog-api-client-rust PR #1655
(available at rev d4954b11 picked up by chore/upgrade-dd-sdk-to-master):

- `pup llm-obs datasets batch-update` — batch insert/update/delete records
  via LLMObsDatasetBatchUpdateRequest / batch_update_llm_obs_dataset
- `pup llm-obs datasets clone` — clone a dataset into a new dataset
  via LLMObsDatasetCloneRequest / clone_llm_obs_dataset
- `pup llm-obs datasets restore` — restore a dataset to a previous version
  via LLMObsDatasetRestoreVersionRequest / restore_llm_obs_dataset_version

Changes:
- src/commands/llm_obs.rs: add datasets_batch_update, datasets_clone,
  datasets_restore functions; add 6 tests (happy-path + error for each)
- src/main.rs: add BatchUpdate/Clone/Restore variants to LlmObsDatasetsActions
  and dispatch arms
- src/client.rs: add 3 new unstable op IDs; update count assertion 162 → 165
- docs/COMMANDS.md: update llm-obs datasets command listing

Co-Authored-By: Claude <noreply@anthropic.com>

https://claude.ai/code/session_01MaHfvEkY66q99gwcPZi4Xc
claude added 4 commits May 29, 2026 11:55
…es search

Exposes new SDK filter params from PRs #1568 and #1480:
- --state: OPEN, ACKNOWLEDGED, RESOLVED, IGNORED, EXCLUDED
- --team <UUID>: filter by team UUID assignee
- --assignee <UUID>: filter by user UUID assignee

Co-Authored-By: Claude <noreply@anthropic.com>
restore_llm_obs_dataset_version returns no body; binding its unit
result tripped clippy::let_unit_value (-D warnings) and broke the
Check/Test and Windows cross-compile jobs. Follow the delete pattern:
call, then print a confirmation.

Co-Authored-By: Claude <noreply@anthropic.com>
The batch-update/clone/restore tests built request JSON with stale
field names (type "dataset_*", missing required id, wrong attribute
keys), so util::read_json_file failed before the API call and the
happy-path asserts panicked. Align bodies with the real SDK models:

- data.id required; data.type is "datasets"
- batch-update attrs use insert_records/delete_records
- restore attr is dataset_version (not version)
- batch-update response data is an array (LLMObsDatasetRecordsMutationResponse)

Co-Authored-By: Claude <noreply@anthropic.com>
- pup cost oci-configs list: wraps ListCostOCIConfigs (SDK #1540)
- pup cost anomalies list: wraps ListCostAnomalies (SDK #1588)

Registers v2.list_cost_anomalies as an unstable op and adds the two
API-key-only endpoints to the OAuth exclusion table.

Co-Authored-By: Claude <noreply@anthropic.com>
- pup security findings mute: wraps MuteSecurityFindings (stable, SDK #1519/#1660)
- pup security rules bulk-convert: wraps BulkConvertExistingSecurityMonitoringRules (#1675)

Co-Authored-By: Claude <noreply@anthropic.com>
Wraps the Synthetics downtime endpoints (SDK #1518):
- pup synthetics downtime list [--filters]
- pup synthetics downtime create --file
- pup synthetics downtime delete <id>

Co-Authored-By: Claude <noreply@anthropic.com>
claude added 3 commits May 30, 2026 08:37
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
New pup annotations domain wrapping the Annotations API v2 (SDK #1631):
list, get-page, create, update, delete. Registers the 5 annotation
operations as unstable ops (162 -> 167).

Co-Authored-By: Claude <noreply@anthropic.com>
- pup metrics tags list --window-seconds: lookback window param (SDK #1593)
- pup metrics query-timeseries --file: v2 timeseries formula query, supporting
  cross_org_uuids per-query for cross-organisation reads (SDK #1564)

Co-Authored-By: Claude <noreply@anthropic.com>
claude added 9 commits May 30, 2026 09:26
Annotations attach to a page (dashboard:<id> or notebook:<id>), so expose
the actions under both `pup dashboards annotations` and
`pup notebooks annotations` via a shared run_annotations dispatch, instead
of a top-level `pup annotations` domain. Removing the top-level variant
also fixes the alphabetical-order check in test_commands.rs.

Co-Authored-By: Claude <noreply@anthropic.com>
The new `pup security findings mute` command calls mute_security_findings,
which the Datadog API gates behind the security_monitoring_findings_write
authorization scope. default_scopes() only requested
security_monitoring_findings_read, so OAuth2 users would get a 403 on mute
while API/app-key users (with full app-key permissions) succeeded.

Add security_monitoring_findings_write to default_scopes() so `pup auth
login` requests it. Left out of read_only_scopes() since it is a write
scope.

Co-Authored-By: Claude <noreply@anthropic.com>
@platinummonkey platinummonkey merged commit 30a8bd7 into main May 30, 2026
6 checks passed
@platinummonkey platinummonkey deleted the chore/upgrade-dd-sdk-to-master branch May 30, 2026 10:52
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.

2 participants