Skip to content

Port '[7.78] Release SNMP #23200' to master#23203

Closed
dkirov-dd wants to merge 2 commits intomasterfrom
port-932655163c-to-master
Closed

Port '[7.78] Release SNMP #23200' to master#23203
dkirov-dd wants to merge 2 commits intomasterfrom
port-932655163c-to-master

Conversation

@dkirov-dd
Copy link
Copy Markdown
Contributor

@dkirov-dd dkirov-dd commented Apr 7, 2026

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

@dkirov-dd dkirov-dd requested review from a team as code owners April 7, 2026 13:42
@dkirov-dd dkirov-dd requested a review from FlorianVeaux April 7, 2026 13:42
@dkirov-dd dkirov-dd changed the title [7.78] Release SNMP #23200 Port '[7.78] Release SNMP #23200' to master Apr 7, 2026
chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Apr 7, 2026
View reviewed changes
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7f1b93f885

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread requirements-agent-release.txt
@datadog-datadog-prod-us1

This comment has been minimized.

Sign in to view
@dkirov-dd dkirov-dd force-pushed the port-932655163c-to-master branch from 7f1b93f to d217358 Compare April 7, 2026 14:04
@dkirov-dd dkirov-dd force-pushed the port-932655163c-to-master branch from d217358 to 397b7ac Compare April 7, 2026 14:07
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.05%. Comparing base (5292c23) to head (397b7ac).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

nubtron
nubtron approved these changes Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@nubtron nubtron left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Approved, however heads up, is the SNMP E2E test expected to fail?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 13, 2026

⚠️ 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.

@dd-octo-sts
Copy link
Copy Markdown
Contributor

dd-octo-sts Bot commented Apr 13, 2026

Validation Report

Validation Status
agent-reqs ✅
ci ✅
codeowners ✅
config ✅
dep ✅
http ✅
imports ✅
integration-style ✅
jmx-metrics ✅
labeler ✅
legacy-signature ✅
license-headers ✅
licenses ✅
metadata ✅
models ✅
openmetrics ✅
package ✅
readmes ✅
saved-views ✅
version ✅

View full run

@dkirov-dd dkirov-dd enabled auto-merge April 13, 2026 13:54
@dkirov-dd
Copy link
Copy Markdown
Contributor Author

dkirov-dd commented Apr 13, 2026

already ported

@dkirov-dd dkirov-dd closed this Apr 13, 2026
auto-merge was automatically disabled April 13, 2026 14:35

Pull request was closed

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

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

@nubtron nubtron nubtron approved these changes

@domalessi domalessi domalessi approved these changes

@FlorianVeaux FlorianVeaux Awaiting requested review from FlorianVeaux FlorianVeaux was automatically assigned from DataDog/network-device-monitoring-core

Assignees

No one assigned

Labels

changelog/no-changelog documentation integration/snmp release team/agent-integrations team/documentation team/network-device-monitoring-core

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dkirov-dd @nubtron @domalessi