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

Added ndo_l3out_node_group_policy module to manage L3Out Node Group Policy (DCNE-181) #555

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Added ndo_l3out_node_group_policy module to manage L3Out Node Group Policy (DCNE-181) #555

wants to merge 11 commits into from
+1,276 −9

Conversation

sajagana
Copy link
Collaborator

No description provided.

@sajagana sajagana linked an issue Oct 22, 2024 that may be closed by this pull request
Enhancement: New module for NDO L3Out Node Group Policy (DCNE-181) #534
Open
@github-actions github-actions bot changed the title Added ndo_l3out_node_group_policy module to manage L3Out Node/Interface Group Policy Added ndo_l3out_node_group_policy module to manage L3Out Node/Interface Group Policy (DCNE-181) Oct 22, 2024
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from e5e4e0e to 3aacd20 Compare October 22, 2024 16:03
akinross
akinross requested changes Oct 23, 2024
View reviewed changes
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from 5e83034 to 68b3b6e Compare October 23, 2024 18:53
@sajagana sajagana requested a review from akinross October 24, 2024 02:39
samiib
samiib requested changes Oct 24, 2024
View reviewed changes
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
@sajagana sajagana requested a review from samiib October 24, 2024 05:02
akinross
akinross requested changes Oct 24, 2024
View reviewed changes
plugins/module_utils/template.py Outdated Show resolved Hide resolved
plugins/module_utils/template.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
@sajagana sajagana requested a review from akinross October 24, 2024 13:11
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from 985a395 to ce3f91f Compare October 24, 2024 13:33
@sajagana sajagana changed the title Added ndo_l3out_node_group_policy module to manage L3Out Node/Interface Group Policy (DCNE-181) Added ndo_l3out_node_group_policy module to manage L3Out Node Group Policy (DCNE-181) Oct 25, 2024
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from ef98298 to b0321c7 Compare October 30, 2024 09:28
@sajagana sajagana requested a review from akinross October 30, 2024 09:32
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from b0321c7 to 9f8028f Compare October 31, 2024 00:53
akinross
akinross requested changes Nov 6, 2024
View reviewed changes
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
plugins/modules/ndo_l3out_node_group_policy.py Outdated Show resolved Hide resolved
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from 9e01cdf to cb02685 Compare November 6, 2024 12:29
@sajagana sajagana requested a review from akinross November 6, 2024 12:31
@samiib samiib self-requested a review November 12, 2024 00:20
samiib
samiib previously approved these changes Nov 12, 2024
View reviewed changes
Copy link
Collaborator

@samiib samiib left a comment

Choose a reason for hiding this comment

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

LGTM

shrsr
shrsr previously approved these changes Nov 12, 2024
View reviewed changes
Copy link
Collaborator

@shrsr shrsr left a comment

Choose a reason for hiding this comment

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

LGTM

gmicol
gmicol previously approved these changes Nov 13, 2024
View reviewed changes
Copy link
Collaborator

@gmicol gmicol left a comment

Choose a reason for hiding this comment

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

LGTM

@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from 2a9fcd6 to 85baa5d Compare December 5, 2024 17:01
gmicol
gmicol previously approved these changes Dec 9, 2024
View reviewed changes
Copy link
Collaborator

@gmicol gmicol left a comment

Choose a reason for hiding this comment

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

LGTM

samiib
samiib previously approved these changes Dec 10, 2024
View reviewed changes
Copy link
Collaborator

@samiib samiib left a comment

Choose a reason for hiding this comment

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

LGTM

@sajagana sajagana dismissed stale reviews from samiib and gmicol via f19ca79 December 10, 2024 12:26
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from f19ca79 to edda2b5 Compare December 10, 2024 15:18
@sajagana sajagana requested review from gmicol and samiib December 10, 2024 15:24
akinross
akinross requested changes Dec 11, 2024
View reviewed changes
plugins/modules/ndo_l3out_node_group_policy.py Show resolved Hide resolved
plugins/module_utils/template.py Outdated Show resolved Hide resolved
@sajagana sajagana requested a review from akinross December 11, 2024 10:03
gmicol
gmicol previously approved these changes Dec 11, 2024
View reviewed changes
Copy link
Collaborator

@gmicol gmicol left a comment

Choose a reason for hiding this comment

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

LGTM

samiib
samiib previously approved these changes Dec 13, 2024
View reviewed changes
Copy link
Collaborator

@samiib samiib left a comment

Choose a reason for hiding this comment

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

LGTM

@sajagana sajagana dismissed stale reviews from samiib and gmicol via 3ef826a December 13, 2024 12:21
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from 3ef826a to d0b628d Compare December 13, 2024 13:39
@sajagana sajagana requested a review from akinross December 13, 2024 13:45
@sajagana sajagana force-pushed the 534_l3out_node_group_policy branch from d0b628d to 37bab55 Compare December 13, 2024 15:10
@sajagana sajagana requested review from gmicol and samiib December 13, 2024 15:13
shrsr
shrsr approved these changes Dec 13, 2024
View reviewed changes
Copy link
Collaborator

@shrsr shrsr left a comment

Choose a reason for hiding this comment

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

LGTM

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

@shrsr shrsr shrsr approved these changes

@lhercot lhercot Awaiting requested review from lhercot lhercot is a code owner

@anvitha-jain anvitha-jain Awaiting requested review from anvitha-jain anvitha-jain is a code owner

@akinross akinross Awaiting requested review from akinross akinross is a code owner

@gmicol gmicol Awaiting requested review from gmicol gmicol is a code owner

@samiib samiib Awaiting requested review from samiib samiib is a code owner

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Enhancement: New module for NDO L3Out Node Group Policy (DCNE-181)
5 participants
@sajagana @shrsr @akinross @samiib @gmicol