Skip to content

Addition of new resource and data source for mso_tenant_policies_l3out_interface_routing_policy (DCNE-281)#412

Open
shrsr wants to merge 3 commits intoCiscoDevNet:masterfrom
shrsr:l3_irp
Open

Addition of new resource and data source for mso_tenant_policies_l3out_interface_routing_policy (DCNE-281)#412
shrsr wants to merge 3 commits intoCiscoDevNet:masterfrom
shrsr:l3_irp

Conversation

@shrsr
Copy link
Collaborator

@shrsr shrsr commented Jan 15, 2026

No description provided.

@shrsr shrsr self-assigned this Jan 15, 2026
@shrsr shrsr changed the title Addition of new resource and data source for mso_tenant_policies_l3out_interface_routing_policy Addition of new resource and data source for mso_tenant_policies_l3out_interface_routing_policy (DCNE-281) Jan 15, 2026
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.

build provider fails can you have a look

akinross
akinross previously approved these changes Jan 16, 2026
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

* `name` - (Required) The name of the L3Out Interface Routing Policy.
* `description` - (Optional) The description of the L3Out Interface Routing Policy. When unset during creation, no description is applied.
* `bfd_multi_hop_settings` - (Optional) BFD multi-hop configuration block. Omitting this block will remove BFD multi-hop settings if they exist.
* `admin_state` - (Optional) Administrative state. Default: enabled when unset during creation. Valid values: enabled, disabled.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you highlight the default and valid values for all the attributes?

Comment on lines 74 to 75
* `echo_admin_state` - (Optional) Echo administrative state. Default: `enabled` when unset during creation. Valid values: `enabled`, `disabled`.
* `interface_control` - (Optional) Interface control. Default: `false` (disabled) when unset during creation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

is it possible to add explaination as to what does echo_admin_state and interface_control do?

same for datasource/docs

…terface_routing_policy related to BFD settings
@shrsr shrsr requested a review from anvitha-jain January 21, 2026 18:55
Copy link
Collaborator

@anvitha-jain anvitha-jain left a comment

Choose a reason for hiding this comment

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

LGTM

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

Copy link
Collaborator

@sajagana sajagana left a comment

Choose a reason for hiding this comment

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

LGTM!

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

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.

Create a resource and data source for "L3Out Interface Routing Policy" object under Tenant Policies (DCNE-281)

5 participants