Skip to content

Gate infiniband behind GPU Monitoring - #25277

Merged
steveny91 merged 3 commits into
masterfrom
sy/infiniband-gpu-gate
Sep 23, 2026
Merged

steveny91 merged 3 commits into
masterfrom
sy/infiniband-gpu-gate

Conversation

@steveny91

Copy link
Copy Markdown
Collaborator

What does this PR do?

Infiniband integration has now been moved under the GPU Monitoring product. This gates the check behind enabling the GPU Monitoring.

@dd-octo-sts

dd-octo-sts Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

✅ Dispatcher tests: passed

Dispatcher beta: informational only
Existing CI remains the merge signal.

  2/2 jobs

✅ 2 passed · nothing failed

Batches · ✅ batch-01 2/2

Dispatcher finished on e5b2b9c — GitHub Run.

@cit-pr-commenter-54b7da

cit-pr-commenter-54b7da Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

evalya-impact-summary

evalya impact analysis
Impact analysis: RUN-ALL — every test task will run
Trigger:         empty diff (default branch, scheduled run, or shallow-clone fallback)
Test tasks:      0 (all selected)
Publish tasks:   2 (always emitted)
Diff:            empty (no diff information)

Learn more about CI impact filtering

@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Tests  Code Coverage

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 89.69% (-0.19%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 97ff5a8 | Docs | View more details | Give us feedback!

@dd-octo-sts

dd-octo-sts Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Disk usage change

Commit 97ff5a8 compared against e5b2b9c.

Uncompressed

linux-aarch64 linux-x86_64 macos-aarch64 macos-x86_64 windows-x86_64
🔺 +593 B 🔺 +593 B 🔺 +593 B 🔺 +593 B 🔺 +593 B
Details
Name Version Type linux-aarch64 linux-x86_64 macos-aarch64 macos-x86_64 windows-x86_64
infiniband 1.7.0 Integration 🔺 +593 B 🔺 +593 B 🔺 +593 B 🔺 +593 B 🔺 +593 B

Compressed

linux-aarch64 linux-x86_64 macos-aarch64 macos-x86_64 windows-x86_64
🔺 +217 B 🔺 +217 B 🔺 +217 B 🔺 +217 B 🔺 +217 B
Details
Name Version Type linux-aarch64 linux-x86_64 macos-aarch64 macos-x86_64 windows-x86_64
infiniband 1.7.0 Integration 🔺 +217 B 🔺 +217 B 🔺 +217 B 🔺 +217 B 🔺 +217 B

@steveny91 steveny91 added the qa/skip-qa Automatically skip this PR for the next QA label Sep 18, 2026
@steveny91
steveny91 marked this pull request as ready for review September 18, 2026 15:31
@steveny91
steveny91 requested review from a team as code owners September 18, 2026 15:31
@steveny91 steveny91 changed the title gate infiniband behind GPU Monitoring Gate infiniband behind GPU Monitoring Sep 18, 2026
brett0000FF
brett0000FF previously approved these changes Sep 18, 2026
Kyle-Neale
Kyle-Neale previously approved these changes Sep 18, 2026
steveny91 and others added 2 commits September 22, 2026 11:13
Resolves a conflict in infiniband/tests/test_unit.py, where this branch and
PR #25278 each appended test functions at the end of the file. Both sets are
kept; the additions do not overlap.

Renames the changelog fragment to its PR number now that it is known.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@temporal-github-worker-1
temporal-github-worker-1 Bot dismissed stale reviews from brett0000FF and Kyle-Neale September 22, 2026 15:14

Review from brett0000FF is dismissed. Related teams and files:

  • documentation
    • infiniband/README.md

Review from Kyle-Neale is dismissed. Related teams and files:

  • agent-integrations
    • infiniband/README.md
    • infiniband/changelog.d/25277.changed
    • infiniband/datadog_checks/infiniband/check.py
    • infiniband/tests/test_unit.py
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Major version bump
The changelog type changed or removed was used in this Pull Request, so the next release will bump major version. Please make sure this is a breaking change, or use the fixed or added type instead.

@dd-octo-sts

dd-octo-sts Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Validation Report

All 21 validations passed.

Show details
Validation Description Status
agent-reqs Verify check versions match the Agent requirements file ✅
ci Validate CI configuration and code coverage settings ✅
codeowners Validate every integration has a CODEOWNERS entry ✅
config Validate default configuration files against spec.yaml ✅
dep Verify dependency pins are consistent and Agent-compatible ✅
http Validate integrations use the HTTP wrapper correctly ✅
imports Validate check imports do not use deprecated modules ✅
integration-style Validate check code style conventions ✅
jmx-metrics Validate JMX metrics definition files and config ✅
labeler Validate PR labeler config matches integration directories ✅
legacy-signature Validate no integration uses the legacy Agent check signature ✅
license-headers Validate Python files have proper license headers ✅
licenses Validate third-party license attribution list ✅
metadata Validate metadata.csv metric definitions ✅
models Validate configuration data models match spec.yaml ✅
openmetrics Validate OpenMetrics integrations disable the metric limit ✅
package Validate Python package metadata and naming ✅
qa-label Validate the pull request declares whether it needs QA for the next Agent release ✅
readmes Validate README files have required sections ✅
saved-views Validate saved view JSON file structure and fields ✅
version Validate version consistency between package and changelog ✅

View full run

@steveny91
steveny91 added this pull request to the merge queue Sep 23, 2026
Merged via the queue into master with commit 1081bee Sep 23, 2026
65 checks passed
@steveny91
steveny91 deleted the sy/infiniband-gpu-gate branch September 23, 2026 14:09
@dd-octo-sts dd-octo-sts Bot added this to the 7.85.0 milestone Sep 23, 2026
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.

3 participants