Skip to content

Commit 181290c

Browse files
nccurrytongyangyeyuehalcyon-radityasingh0510allenz92
authored
DCGM-Exporter 4.6.0-4.8.3 (#711)
Filtered, squashed public import of the internal release snapshot. Source tag: 4.6.0-4.8.3 Source commit: 6e9668062deb36d7315f69731ad22eccd4019401 Source branch: origin/rel_exp_4_8_3 GitHub base: d5e5f51 Excluded paths: - .gitlab-ci.yml - .gitlab/** - .coderabbit.yaml - .greptile/** - internal/pkg/counters/llms_contract_test.go Attribution audit: carried work from #489, #510, #654, #658, #660, #663, #664, #665, #667, #671, #678, #687, #697, #699, and #704. Issues #641 and #703 are acknowledged through their corresponding implementers. Daphne's #655 remains directly attributed on GitHub main. Signed-off-by: Nick Curry <ncurry@nvidia.com> Co-authored-by: Yang Tong <25810794+tongyangyeyue@users.noreply.github.com> Co-authored-by: Hairui Yang <hairuiyang@deeproute.ai> Co-authored-by: Aditya Singh <aditya.chouhan@neevcloud.com> Co-authored-by: Allen Zhou <loveinjavac@gmail.com> Co-authored-by: Yufeng Huang <91405051+Raid57@users.noreply.github.com> Co-authored-by: Rocker Zhang <zhang.rocker.liyuan@gmail.com> Co-authored-by: Swati Gupta <swatig@nvidia.com> Co-authored-by: Asutosh Panda <asutosh.pda@gmail.com> Co-authored-by: Shravya Vorugallu <shravyavorugallu@gmail.com> Co-authored-by: randomizedcoder <64496590+randomizedcoder@users.noreply.github.com> Co-authored-by: skartikey <1942366+skartikey@users.noreply.github.com> Co-authored-by: Dave Young <dave.andrew.young@gmail.com> Co-authored-by: rvatkar <263779040+rvatkar@users.noreply.github.com> Co-authored-by: dshaiknvidia <152336361+dshaiknvidia@users.noreply.github.com>
1 parent d5e5f51 commit 181290c

323 files changed

Lines changed: 52787 additions & 4188 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.codex/README.md

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
# Codex Project Notes
2+
3+
This directory contains Codex-facing guidance for DCGM Exporter. The
4+
repository-level `AGENTS.md` remains authoritative; these files are supporting
5+
notes for implementation, validation, and review sessions.
6+
7+
## Safety Boundary
8+
9+
Do not add Codex permission overrides, auto-approval config, automatic mutating
10+
hooks, or local environment secrets to this repository. Follow the active
11+
session sandbox and approval policy.
12+
13+
## Files
14+
15+
- `project-instructions.md` - compact implementation rules.
16+
- `validation.md` - explicit validation recipes.
17+
- `agent-profiles.md` - role guidance for focused review and verification.
18+
- `review-guardrails.md` - MR review checks.
19+
- `coderabbit-prevention.md` - recurring review issue prevention checklist.

.codex/agent-profiles.md

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
# Codex Role Profiles
2+
3+
Use these as role guidance, not as permission grants.
4+
5+
## Debugger
6+
7+
- Capture command, input, environment, exact error, and reproduction path.
8+
- Read the owning package before proposing a fix.
9+
- Verify with the narrow failing test first, then the broader relevant gate.
10+
11+
## Verifier
12+
13+
- Check that claimed files and behavior actually changed.
14+
- Run the relevant commands from `.codex/validation.md`.
15+
- Separate current-change failures from pre-existing or prerequisite failures.
16+
17+
## Test Runner
18+
19+
- Use package tests for pure logic.
20+
- Use GPU integration, Docker image, or Kubernetes e2e tests only when the
21+
behavior needs that layer.
22+
- Do not weaken assertions to pass environmental tests.
23+
24+
## Security Reviewer
25+
26+
- Check secrets, auth material, token logging, web config handling, container
27+
privilege, RBAC, and fail-open behavior.
28+
29+
## Performance Reviewer
30+
31+
- Measure before optimizing.
32+
- Check scrape latency, allocation-heavy rendering, lock contention, duplicate
33+
parsing, and polling/reload behavior.
34+
35+
## Documentation Specialist
36+
37+
- Keep examples current and command outputs realistic.
38+
- Prefer direct links to repository files and official docs over memory.

.codex/coderabbit-prevention.md

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# CodeRabbit Prevention Checklist
2+
3+
Apply before handing off a change.
4+
5+
## Contracts
6+
7+
- Do not invent metric labels, CLI flags, env vars, Helm values, or DCGM fields.
8+
- Keep README, Helm values/templates, raw YAML, and tests aligned with code.
9+
- Broad terminology changes need `rg` sweeps across Go, YAML, CSV, docs, tests,
10+
metrics, and CI.
11+
12+
## Go
13+
14+
- Prefer explicit errors with context.
15+
- Avoid silent fallback for failed parsing, file I/O, metric registration,
16+
serialization, or scrape rendering.
17+
- Do not add global mutable state without reset/restore tests.
18+
- Keep mock seams in wrapper packages and test helpers.
19+
20+
## Tests
21+
22+
- Assert the behavior named by the test.
23+
- Prefer typed field assertions over string/debug output matching.
24+
- Use package tests for logic and reserve GPU/e2e tests for hardware or
25+
deployment contracts.
26+
27+
## Docs
28+
29+
- Comments and docs describe current behavior, not planned future cleanup.
30+
- Examples use current commands and real paths.
31+
- Security-sensitive examples avoid real-looking credentials.

.codex/project-instructions.md

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
# DCGM Exporter Codex Instructions
2+
3+
Use `AGENTS.md` and scoped `AGENTS.md` files as the source of truth.
4+
5+
## Implementation Rules
6+
7+
- Start feature work with the root `AGENTS.md` Feature Planning checklist and
8+
Repository Surface Map.
9+
- Read the owning package, tests, Makefile target, and CI job before editing.
10+
- Keep docs and examples tied to real flags, env vars, Helm values, metrics,
11+
and file paths from the repo.
12+
- Prefer small package-local helpers. Do not add broad agent-only abstractions.
13+
- Do not mutate generated artifacts unless the change requires regeneration and
14+
the matching command is run.
15+
- Do not include unrelated worktree files, local settings, or coverage output in
16+
commits.
17+
18+
## Security
19+
20+
- Never log or commit secrets, auth headers, kubeconfig material, web config
21+
credentials, registry tokens, or secret-manager values.
22+
23+
## Metrics
24+
25+
- Metric names, labels, HELP text, and TYPE values are user-visible contracts.
26+
- Exporter-owned counters must stay aligned across code, CSV examples, docs,
27+
and `llms.txt`.
28+
- Hardware-backed behavior should be tested with fakes where possible and with
29+
GPU integration/e2e only when necessary and available.

.codex/review-guardrails.md

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
# Review Guardrails
2+
3+
Use these before MR handoff.
4+
5+
## Planning
6+
7+
- The change is mapped to an owning surface from root `AGENTS.md`.
8+
- User-visible contracts, docs, and prerequisite-dependent gates are accounted
9+
for without duplicating root guidance.
10+
11+
## Metrics And CLI
12+
13+
- Metric CSV rows load through `internal/pkg/counters`.
14+
- Exporter-owned counters are documented in `llms.txt`.
15+
- CLI flags, env vars, defaults, Helm values, raw YAML, and README examples are
16+
aligned.
17+
- Remote hostengine URI examples match `pkg/cmd/app.go`.
18+
19+
## Tests
20+
21+
- New behavior has the lowest useful test layer.
22+
- GPU/Docker/Kubernetes gates are run when prerequisites exist, or the skip is
23+
documented.
24+
- Tests avoid tautological assertions and fixed sleeps when readiness can be
25+
observed.
26+
27+
## CI, Docker, And Scripts
28+
29+
- Verification commands are not masked with `|| true` outside cleanup.
30+
- Shell scripts keep strict mode and quote path variables.
31+
- Version pins go through `hack/versions.env`, `make sync-versions`, and
32+
`make validate-versions`.

.codex/validation.md

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,54 @@
1+
# Validation Recipes
2+
3+
Run commands from the repository root.
4+
5+
## Common Docs And Go Gates
6+
7+
```bash
8+
go test ./internal/pkg/counters
9+
make test-main
10+
make lint
11+
make check-fmt
12+
make validate
13+
```
14+
15+
## Conditional Gates
16+
17+
Generated mocks or go:generate inputs:
18+
19+
```bash
20+
make generate
21+
git diff --exit-code
22+
```
23+
24+
Version-derived files:
25+
26+
```bash
27+
make sync-versions
28+
make validate-versions
29+
```
30+
31+
Docker image behavior:
32+
33+
```bash
34+
make local
35+
make test-images
36+
```
37+
38+
GPU/DCGM integration behavior:
39+
40+
```bash
41+
make test-integration
42+
```
43+
44+
GPU Kubernetes behavior:
45+
46+
```bash
47+
make e2e-local-check
48+
make e2e-local-up
49+
make e2e-local-deploy
50+
make test-e2e
51+
```
52+
53+
If GPU, Docker, Kubernetes, DCGM, or network prerequisites are unavailable,
54+
record the skip and reason in the MR.

.cursor/README.md

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
# Cursor Agent Configuration
2+
3+
This directory contains project guidance for Cursor. It mirrors the intent of
4+
`AGENTS.md` without granting permissions or installing hooks.
5+
6+
## Structure
7+
8+
```text
9+
.cursor/
10+
README.md
11+
commands/
12+
rules/
13+
skills/
14+
```
15+
16+
No `hooks.json` is provided. Formatting, validation, and security checks are
17+
explicit commands that agents or developers run intentionally.
18+
19+
## Quick Reference
20+
21+
| Area | Primary guidance |
22+
|---|---|
23+
| Go implementation | `rules/go-standards.mdc` |
24+
| Metrics and CSV contracts | `rules/metrics-contracts.mdc` |
25+
| Helm/Kubernetes | `rules/helm-kubernetes.mdc` |
26+
| Shell, CI, security | `rules/shell-ci-security.mdc` |
27+
| Tests | `rules/testing.mdc` |
28+
29+
Use `commands/plan-feature.mdc` before broad feature work and
30+
`commands/validate-after-change.mdc` before handoff.

.cursor/commands/README.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# Cursor Commands
2+
3+
These command files describe explicit workflows. They do not install hooks or
4+
change permissions.
5+
6+
- `run-quality-checks`
7+
- `plan-feature`
8+
- `validate-after-change`
9+
- `code-review`
10+
- `show-dev-guide`

.cursor/commands/code-review.mdc

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
description: Review a DCGM Exporter change before MR handoff
3+
globs: ["**/*"]
4+
---
5+
6+
# Code Review
7+
8+
Check:
9+
10+
- Metric, CLI, Helm, Docker, and package contracts stayed aligned.
11+
- Tests prove the behavior named by each test.
12+
- Shell and CI verification commands are not masked.
13+
- No secrets, local settings, coverage outputs, or unrelated files are staged.

.cursor/commands/plan-feature.mdc

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
---
2+
description: Plan a DCGM Exporter feature before implementation
3+
globs: ["**/*"]
4+
---
5+
6+
# Plan Feature
7+
8+
Use the root `AGENTS.md` Feature Planning checklist and Repository Surface Map.
9+
10+
Produce a concise plan that names:
11+
12+
- the owning surface and scoped `AGENTS.md` files to read;
13+
- source-of-truth files, tests, and generated artifacts to verify;
14+
- user-visible contracts affected;
15+
- implementation approach and smallest useful abstraction;
16+
- validation commands and prerequisite-dependent gates.

0 commit comments

Comments
 (0)