Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add vrf_global resource module #1055

Merged
merged 28 commits into from
Jun 6, 2024
Merged

Add vrf_global resource module #1055

merged 28 commits into from
Jun 6, 2024

Conversation

Ruchip16
Copy link
Contributor

@Ruchip16 Ruchip16 commented Apr 16, 2024 •
edited by roverflow
Loading

SUMMARY

This PR adds support for ios.vrf_global resource module

Here is the model design for the vrf_global module

ISSUE TYPE
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME

cisco.ios.ios_vrf_global

MILESTONES
  • Parsing entire argspec ( which covers gathered and parsed state )
  • Merged and Rendered state working
  • Replaced state working
  • Overridden state working
  • Deleted and purged state
  • Unit tests & Integration tests

@Ruchip16 Ruchip16 added the feature This issue/PR relates to a feature request. label Apr 16, 2024
@Ruchip16 Ruchip16 marked this pull request as draft April 16, 2024 12:19
@KB-perByte KB-perByte marked this pull request as ready for review April 17, 2024 09:31
@KB-perByte KB-perByte closed this Apr 17, 2024
@KB-perByte KB-perByte reopened this Apr 17, 2024
@rohitthakur2590 rohitthakur2590 self-requested a review April 17, 2024 09:54
rohitthakur2590
rohitthakur2590 requested changes Apr 18, 2024
View reviewed changes
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Show resolved Hide resolved
@Ruchip16 Ruchip16 marked this pull request as draft April 19, 2024 14:41
@Ruchip16 Ruchip16 marked this pull request as ready for review April 22, 2024 06:47
rohitthakur2590
rohitthakur2590 requested changes Apr 22, 2024
View reviewed changes
plugins/action/vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
@Ruchip16 Ruchip16 requested a review from NilashishC May 31, 2024 07:09
KB-perByte
KB-perByte requested changes May 31, 2024
View reviewed changes
plugins/modules/ios_vrf_global.py Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Outdated Show resolved Hide resolved
plugins/modules/ios_vrf_global.py Show resolved Hide resolved
@Ruchip16 Ruchip16 requested a review from KB-perByte June 3, 2024 08:22
@roverflow roverflow merged commit c4ccf7b into ansible-collections:main Jun 6, 2024
54 checks passed
@Ruchip16 Ruchip16 deleted the feature_vrf branch June 6, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rohitthakur2590 rohitthakur2590 rohitthakur2590 approved these changes

@NilashishC NilashishC NilashishC approved these changes

@roverflow roverflow roverflow approved these changes

@KB-perByte KB-perByte KB-perByte approved these changes

Assignees
No one assigned
Labels
feature This issue/PR relates to a feature request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

cisco.ios.ios_vrf fail when sending status absent on missing VRF Add parsed and rendered states to ios_vrf
5 participants
@Ruchip16 @NilashishC @KB-perByte @rohitthakur2590 @roverflow