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.

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

[yang] BGP support autort in l2vpn address family #21575

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bradh352
Copy link
Contributor

@bradh352 bradh352 commented Jan 30, 2025

Why I did it

autort is a supported option in the frr-mgmt-framework as can be seen in

('autort', '{no:no-prefix}autort {}', hdl_enum_conversion),
and
{% if 'autort' in af_val %}
autort {{af_val['autort']}}

However, the yang models do not allow this so when adding this value to config_db.json then running config replace a validation issue is encountered.

Work item tracking

How I did it

This adds yang support and validates the only known valid value of rfc8365-compatible is set.

How to verify it

Test cases have been updated to validate the behavior.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305
  • 202411

Just backport to current stable branch, doesn't appear any one is really using this with yang validation.

Tested branch (Please provide the tested image version)

Master as of 20250130

Description for the changelog

[yang] BGP support autort in l2vpn address family

Link to config_db schema for YANG module changes

N/A pre-existing, just adding yang validation

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Brad House (@bradh352)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

bradh352 added a commit to bradh352/sonic-buildimage that referenced this pull request Jan 30, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
@bradh352 bradh352 changed the title [yang] BGP support autort in address family [yang] BGP support autort in l2vpn address family Jan 30, 2025
bradh352 added a commit to bradh352/sonic-buildimage that referenced this pull request Jan 30, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 31, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Jan 31, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 1, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 1, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 2, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 2, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 3, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 4, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 4, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 5, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 5, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 6, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 6, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 7, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 7, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 8, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 8, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 9, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 9, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 11, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 12, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 12, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 13, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 13, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 14, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 14, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 15, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 15, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 18, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 19, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 19, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Pull request contains merge conflicts.

@bradh352 bradh352 force-pushed the bradh352/yang-evpn-autort branch from 4548029 to 1b756e3 Compare February 19, 2025 19:37
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 20, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 20, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
@bradh352 bradh352 force-pushed the bradh352/yang-evpn-autort branch from 1b756e3 to 437c033 Compare February 21, 2025 01:59
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 21, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 21, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 22, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
github-actions bot pushed a commit to bradh352/sonic-buildimage that referenced this pull request Feb 22, 2025
`autort` is a supported option in the frr-mgmt-framework as can be seen in
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/frrcfgd/frrcfgd.py#L1830
and
https://github.com/sonic-net/sonic-buildimage/blob/3d8c940997c3855b650fc0eea7f9db46546b17ac/src/sonic-frr-mgmt-framework/templates/bgpd/bgpd.conf.db.addr_family.evpn.j2#L4-L5

However, the yang models do not allow this so when adding this value
to `config_db.json` then running `config replace` a validation issue
is encountered.

This adds yang support and validates the only known valid value of
`rfc8365-compatible` is set.

Test cases have been updated to validate the behavior.

Signed-off-by: Brad House (@bradh352)
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