Skip to content

fix(docs): update IGW target to be consistent with other docs#421

Open
parkanzky wants to merge 1 commit into
mainfrom
nvbug6360186-auditor-seed-data/pparkanzky
Open

fix(docs): update IGW target to be consistent with other docs#421
parkanzky wants to merge 1 commit into
mainfrom
nvbug6360186-auditor-seed-data/pparkanzky

Conversation

@parkanzky

@parkanzky parkanzky commented Jun 23, 2026

Copy link
Copy Markdown
Contributor
  • Update docs to point at IGW provider that is created by setup scripts.
  • Add auditor seed job to necessary workspaces so that it works according to docs.

Note that running the platform in local mode will not automatically run the seed jobs. To test, I manually ran the full platform seed task as nemo-platform run task --task nmp.platform_seed , which now correctly discovers AuditorSeedJob. This isolates that particular problem. If nmp.platform_seed is not executed on startup where expected, that would be a separate bug.

Summary by CodeRabbit

  • Documentation
    • Updated inference gateway configuration examples across auditor documentation to use the nvidia-build provider instead of the previous build provider
    • Refreshed auditor target “persist/update” and local run tutorial examples to match the new provider name
  • Chores
    • Added a new runtime dependency for the auditor plugin to the platform seed service and updated related project metadata

@parkanzky parkanzky requested review from a team as code owners June 23, 2026 19:51
@github-actions github-actions Bot added the fix label Jun 23, 2026
@github-actions

Copy link
Copy Markdown
Contributor

@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 8398cb4b-881b-466e-8d3a-a2d7ec84e37e

📥 Commits

Reviewing files that changed from the base of the PR and between c2cad91 and b9aa86b.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (5)
  • docs/auditor/sdk-resources.mdx
  • docs/auditor/targets/index.mdx
  • docs/auditor/targets/inference-gateway.mdx
  • docs/auditor/tutorials/run-audit-locally.mdx
  • services/platform-seed/pyproject.toml
✅ Files skipped from review due to trivial changes (4)
  • docs/auditor/sdk-resources.mdx
  • docs/auditor/targets/inference-gateway.mdx
  • docs/auditor/targets/index.mdx
  • docs/auditor/tutorials/run-audit-locally.mdx
🚧 Files skipped from review as they are similar to previous changes (1)
  • services/platform-seed/pyproject.toml

📝 Walkthrough

Walkthrough

Renames the inference_gateway provider value from "build" to "nvidia-build" across four auditor documentation files. Separately, adds nemo-auditor-plugin as a runtime dependency and workspace source in services/platform-seed/pyproject.toml, with an updated project description.

Changes

Docs: inference_gateway provider rename

Layer / File(s) Summary
Provider name update across auditor docs
docs/auditor/sdk-resources.mdx, docs/auditor/targets/index.mdx, docs/auditor/targets/inference-gateway.mdx, docs/auditor/tutorials/run-audit-locally.mdx
All inference_gateway.provider example values changed from "build" to "nvidia-build".

platform-seed: nemo-auditor-plugin dependency

Layer / File(s) Summary
Add nemo-auditor-plugin to platform-seed
services/platform-seed/pyproject.toml
nemo-auditor-plugin added to dependencies and tool.uv.sources; description reworded to include auditor and plugin seeding.

Suggested labels

docs

Suggested reviewers

  • mckornfield
  • tylersbray
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title accurately describes the primary change: updating IGW provider references across documentation for consistency.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch nvbug6360186-auditor-seed-data/pparkanzky

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor
Suite Lines Covered Line Rate Branch Rate
Unit Tests 20898/27461 76.1% 61.1%
Integration Tests 12133/26230 46.3% 19.8%

Also, fix auditor seed job so that all docs work out of the box.

Signed-off-by: Paul A. Parkanzky <parkanzky@users.noreply.github.com>
@parkanzky parkanzky force-pushed the nvbug6360186-auditor-seed-data/pparkanzky branch from c2cad91 to b9aa86b Compare June 23, 2026 20:16
@parkanzky

Copy link
Copy Markdown
Contributor Author

bah working on the lint issues

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants