Skip to content

fix template with stale plugin name#61

Open
zsblevins wants to merge 1 commit into
mainfrom
stale-plugin-name
Open

fix template with stale plugin name#61
zsblevins wants to merge 1 commit into
mainfrom
stale-plugin-name

Conversation

@zsblevins

@zsblevins zsblevins commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Description

Validation

  • Standard CI passes.
  • Kind integration passes, or this PR explains why it was not run.

The kind integration test is manual due to taking ~30 min to complete. When the PR is ready for review,
run Actions -> Kind Integration -> Run workflow against the copy-pr-bot generated
pull-request/<PR_NUMBER> branch. Use the default test_path for the full suite,
or narrow it only while debugging.

Passing Kind Integration run:

Checklist

  • I am familiar with the contributing guidelines in CONTRIBUTING.md.
  • Commits are signed off for DCO compliance.
  • New or existing tests cover these changes, or the PR explains why tests are not needed.
  • Documentation is updated for user-facing behavior changes.
  • Generated artifacts are updated when applicable, such as OpenAPI specs,
    docs screenshots, or Helm/rendered outputs.

Summary by CodeRabbit

  • New Features
    • Updated the set of enabled plugins to include new configuration and data model capabilities.
    • Removed two plugins no longer included, while keeping existing core plugins enabled.

Signed-off-by: Zoe Blevins <zblevins@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@coderabbitai

coderabbitai Bot commented Jun 24, 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: 03024c94-de25-468e-8d96-c90da0aec978

📥 Commits

Reviewing files that changed from the base of the PR and between ca6470c and 50555d7.

📒 Files selected for processing (1)
  • deploy/helm/templates/_nautobot-configmap-data.tpl

📝 Walkthrough

Walkthrough

The Helm Nautobot configmap template updates NAUTOBOT_PLUGINS to remove two plugins and add three others.

Changes

Nautobot plugin list update

Layer / File(s) Summary
Update NAUTOBOT_PLUGINS
deploy/helm/templates/_nautobot-configmap-data.tpl
The NAUTOBOT_PLUGINS value changes to remove nautobot_kiwi and nautobot_broker_nats, and add nv_config_manager, nautobot_nvdatamodels, and nautobot_app_overlays.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I hopped through the chart with a cheerful wiggle,
Swapped plugins in line with a tiny little jiggle.
New names in the list, old names out of sight,
A configmap tweak makes the bunnies delight.

🚥 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 accurately reflects the main change: updating the template to replace stale plugin names in the Nautobot config map.
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 stale-plugin-name

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants