Skip to content

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

Merged
parkanzky merged 2 commits into
mainfrom
nvbug6360186-auditor-seed-data/pparkanzky
Jun 24, 2026
Merged

fix(docs): update IGW target to be consistent with other docs#421
parkanzky merged 2 commits 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 auditor examples, target creation/update, the inference gateway worked example, and the local audit tutorial to use the nvidia-build inference gateway provider name instead of build.
  • Chores
    • Updated platform seed service package metadata and added nemo-auditor-plugin as a new runtime dependency, including the matching workspace source entry.
    • Extended the platform seed service description to include “auth, guardrails, auditor, and plugin seed jobs.”

@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

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

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: 204988be-3b21-4c90-a66f-92b1abc8094d

📥 Commits

Reviewing files that changed from the base of the PR and between 2932e32 and 44361e0.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • packages/nemo_platform/pyproject.toml
✅ Files skipped from review due to trivial changes (1)
  • packages/nemo_platform/pyproject.toml

📝 Walkthrough

Walkthrough

Updates auditor docs to use nvidia-build for Inference Gateway examples. Adds nemo-auditor-plugin to services/platform-seed/pyproject.toml and updates the package 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
Example provider values and worked-example text now use nvidia-build instead of 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 is added to dependencies and tool.uv.sources, and the project description is reworded.

Suggested labels

docs

Suggested reviewers

  • maxdubrinsky
🚥 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 The title matches the docs-focused provider rename, though it omits the workspace and dependency updates.
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 20908/27478 76.1% 61.2%
Integration Tests N/A N/A N/A

@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

@parkanzky parkanzky force-pushed the nvbug6360186-auditor-seed-data/pparkanzky branch from b9aa86b to 7705e89 Compare June 24, 2026 19:32
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 7705e89 to 2932e32 Compare June 24, 2026 21:33
Signed-off-by: Paul A. Parkanzky <parkanzky@users.noreply.github.com>
@parkanzky parkanzky force-pushed the nvbug6360186-auditor-seed-data/pparkanzky branch from 2932e32 to 44361e0 Compare June 24, 2026 21:38
@parkanzky parkanzky enabled auto-merge June 24, 2026 21:39
@parkanzky parkanzky added this pull request to the merge queue Jun 24, 2026
Merged via the queue into main with commit db97946 Jun 24, 2026
52 of 54 checks passed
@parkanzky parkanzky deleted the nvbug6360186-auditor-seed-data/pparkanzky branch June 24, 2026 22:03
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