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_interface_routing_policy to manage L3Out Interface Routing Policy (DCNE-132) #563

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

Added ndo_l3out_interface_routing_policy to manage L3Out Interface Routing Policy (DCNE-132) #563

wants to merge 10 commits into from

Conversation

sajagana
Copy link
Collaborator

@sajagana sajagana commented Nov 8, 2024

No description provided.

@sajagana sajagana linked an issue Nov 8, 2024 that may be closed by this pull request
Enhancement: New module for L3Out Interface Routing Policy in Tenant Tenant Policies Template (DCNE-132) #458
Open
@github-actions github-actions bot changed the title Added ndo_l3out_interface_routing_policy to manage L3Out Interface Routing Policy Added ndo_l3out_interface_routing_policy to manage L3Out Interface Routing Policy (DCNE-132) Nov 8, 2024
@sajagana sajagana force-pushed the 458_l3out_interface_routing_policy branch from 6e1cc12 to 8535b6c Compare November 8, 2024 15:30
samiib
samiib requested changes Nov 12, 2024
View reviewed changes
@sajagana sajagana requested a review from samiib November 14, 2024 17:07
akinross
akinross requested changes Nov 15, 2024
View reviewed changes
@sajagana sajagana mentioned this pull request Nov 15, 2024
Merged
@sajagana sajagana requested a review from akinross November 15, 2024 16:12
@samiib samiib mentioned this pull request Nov 20, 2024
Merged
@sajagana sajagana force-pushed the 458_l3out_interface_routing_policy branch from df95ade to a3d118b Compare December 5, 2024 05:30
@sajagana sajagana requested a review from gmicol December 5, 2024 05:34
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

shrsr
shrsr previously approved these changes Dec 9, 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

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

akinross
akinross requested changes Dec 10, 2024
View reviewed changes
@sajagana sajagana dismissed stale reviews from samiib, shrsr, and gmicol via b7f7bcc December 10, 2024 12:07
akinross
akinross previously approved these changes Dec 11, 2024
View reviewed changes
Copy link
Collaborator

@akinross akinross 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 458_l3out_interface_routing_policy branch from d74e155 to d6aef2d Compare January 23, 2025 07:09
akinross
akinross previously approved these changes Jan 23, 2025
View reviewed changes
Copy link
Collaborator

@akinross akinross left a comment

Choose a reason for hiding this comment

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

LGTM

anvitha-jain
anvitha-jain previously approved these changes Jan 23, 2025
View reviewed changes
@sajagana sajagana dismissed stale reviews from anvitha-jain and akinross via f3a10e4 January 31, 2025 13:29
sajagana added 10 commits January 31, 2025 19:05
@sajagana
[minor_change] Added ndo_l3out_interface_routing_policy to manage L3O…
f46db4d 
…ut Interface Routing Policy
@sajagana
[ignore] Updated ndo_l3out_interface_routing_policy module doc and ad…
4d9cda8 
…ded new function to handle the update payload creation logic
@sajagana
[ignore] Updated the append_update_ops_data function to support remov…
6919e25 
…e operation
@sajagana
[ignore] Updated the append_update_ops_data function logic
5ef3041
@sajagana
[ignore] Rebased with latest master to include append_update_ops_data
a69a107
@sajagana
[ignore] Removed ENABLED_DISABLED_BOOLEAN_MAP from constants.py file
d9e80f4
@sajagana
[ignore] Changed the variable names in ndo_l3out_interface_routing_po…
2f22c14 
…licy module
@sajagana
[ignore] Moved bfd_multi_hop_settings argument spec to the module uti…
4ef71ed 
…ls mso.py
@sajagana
[ignore] Added seealso section to the ndo_l3out_interface_routing_pol…
667c006 
…icy and ndo_l3out_node_routing_policy modules
@sajagana
[ignore] Removed get_ospf_network_type function from ndo_l3out_interf…
23fbcc1 
…ace_routing_policy module
@sajagana sajagana force-pushed the 458_l3out_interface_routing_policy branch from f3a10e4 to 23fbcc1 Compare January 31, 2025 13:38
samiib
samiib approved these changes Feb 11, 2025
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

gmicol
gmicol approved these changes Feb 11, 2025
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

akinross
akinross approved these changes Feb 17, 2025
View reviewed changes
Copy link
Collaborator

@akinross akinross 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 approved these changes Feb 23, 2025
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

@akinross akinross akinross approved these changes

@anvitha-jain anvitha-jain anvitha-jain approved these changes

@samiib samiib samiib approved these changes

@gmicol gmicol gmicol approved these changes

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

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 L3Out Interface Routing Policy in Tenant Tenant Policies Template (DCNE-132)
6 participants
@sajagana @shrsr @akinross @anvitha-jain @samiib @gmicol