Skip to content

Conversation

RaviPatra
Copy link
Contributor

@RaviPatra RaviPatra commented Sep 8, 2025

Related Issue(s)

Proposed Changes

Cisco IOS-XR Version

24.4.2

Checklist

Test, validate and ensure pre-commit is run before submitting PR

  • Latest commit is rebased from main/master with merge conflicts resolved
  • All pre-commit hooks passed

Test results:
image

@RaviPatra RaviPatra changed the title Snmp server views Implemented Snmp server views feature Sep 8, 2025
@RaviPatra RaviPatra changed the title Implemented Snmp server views feature Implemented iosxr_snmp_server_views feature Sep 8, 2025
@rwcrowe rwcrowe self-requested a review September 8, 2025 15:38
Copy link
Contributor

@rwcrowe rwcrowe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See changes below

@RaviPatra RaviPatra requested a review from rwcrowe September 10, 2025 16:39
@rwcrowe rwcrowe merged commit 3dcd5e9 into netascode:main Sep 10, 2025
1 check passed
Rajeshwaribu pushed a commit to Rajeshwaribu/terraform-iosxr-nac-iosxr that referenced this pull request Sep 17, 2025
* Updated with banner feature.

* updated the variable file to empty list

* implemented iosxr_cdp feature.

* Removed unused files.
Updated files with default data.

* Delete examples/system/system.nac.yaml

* Updated pre-commit file.

* Updated Readme file using pre-commit command

* Address Git test failed issues by adding system.nac.yaml file.

* Replaced system.nac.yaml with master copy.

* implemented iosxr_snmp_server_view feature.

* implemented iosxr_snmp_server_view feature.

* Handled with try and default value.

* Addressed code review comments

* Updated readme file

---------

Co-authored-by: rpatraga <>
Co-authored-by: Robert Crowe <rwcrowe@users.noreply.github.com>
@RaviPatra RaviPatra deleted the snmp_server_views branch September 18, 2025 04:11
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