Skip to content
New issue

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

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

Already on GitHub? # to your account

Generate interface table to have an entry designated to default VRF. #2848

Merged
merged 7 commits into from
Jun 10, 2019

Conversation

prsunny
Copy link
Contributor

@prsunny prsunny commented May 1, 2019

- What I did
Generate interface table to have an entry designated to default VRF.

Current:

    "VLAN_INTERFACE": {
        "Vlan1000|192.168.0.1/21": {}
    },

Proposed:

    "VLAN_INTERFACE": {
        "Vlan1000": {},
        "Vlan1000|192.168.0.1/21": {}
    },

- How I did it
Modify minigraph parser:

- How to verify it
Run load_minigraph and verify entries without IP prefix in config_db:

 2) "LOOPBACK_INTERFACE|Loopback0|FC00:1::32/128"
 3) "MGMT_INTERFACE|eth0|FC00:2::32/64"
 4) "PORTCHANNEL_INTERFACE|PortChannel0002|FC00::75/126"
 5) "PORTCHANNEL_INTERFACE|PortChannel0001|FC00::71/126"
 6) "PORTCHANNEL_INTERFACE|PortChannel0004|10.0.0.62/31"
 7) "PORTCHANNEL_INTERFACE|PortChannel0001"
 8) "PORTCHANNEL_INTERFACE|PortChannel0004|FC00::7D/126"
 9) "PORTCHANNEL_INTERFACE|PortChannel0003|10.0.0.60/31"
10) "PORTCHANNEL_INTERFACE|PortChannel0003"
11) "PORTCHANNEL_INTERFACE|PortChannel0004"
12) "PORTCHANNEL_INTERFACE|PortChannel0003|FC00::79/126"
13) "VLAN_INTERFACE|Vlan1000|192.168.0.1/21"
14) "LOOPBACK_INTERFACE|Loopback0|10.1.0.32/32"
15) "PORTCHANNEL_INTERFACE|PortChannel0001|10.0.0.56/31"
16) "MGMT_INTERFACE|eth0|10.3.147.47/23"
17) "VLAN_INTERFACE|Vlan1000"
18) "PORTCHANNEL_INTERFACE|PortChannel0002|10.0.0.58/31

- Description for the changelog
This is to handle backward compatibility issues and to address the dependency with sonic-net/sonic-swss#796

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

@prsunny prsunny changed the title Intf migrate Generate interface table to have an entry designated to default VRF. May 1, 2019
@lguohan
Copy link
Collaborator

lguohan commented May 2, 2019

can you check the build failure?

@prsunny
Copy link
Contributor Author

prsunny commented May 28, 2019

retest this please

@lguohan
Copy link
Collaborator

lguohan commented Jun 6, 2019

can you resolve the conflict? also does this pr requires other pr?

@prsunny prsunny merged commit 231d309 into sonic-net:master Jun 10, 2019
seiferteric pushed a commit to project-arlo/sonic-buildimage that referenced this pull request Oct 14, 2019
Minigraph migration and jinja2 template changes for INTF_PREFIX_TABLE
changes from PRs:
sonic-net#2848

Change-Id: Ia6c3bf37702b20c57f7f808299a5a861fce15e11
seiferteric pushed a commit to project-arlo/sonic-buildimage that referenced this pull request Oct 14, 2019
…gration and jinja2 template changes for INTF_PREFIX_TABLE changes from PRs: sonic-net#2848" into broadcom_sonic
@prsunny prsunny deleted the intf_migrate branch February 7, 2020 22:09
yxieca pushed a commit that referenced this pull request Jul 8, 2023
…lly (#15753)

src/sonic-swss

* 4a3af63d - (HEAD -> 202205, origin/202205) Allow NOT_IMPLEMENTED sai return status for availability monitoring API (#2848) (6 hours ago) [Tejaswini Chadaga]
mssonicbld added a commit that referenced this pull request Jul 20, 2023
…lly (#15919)

#### Why I did it
src/sonic-swss
```
* fa342305 - (HEAD -> 202305, origin/202305) Remove redundant updateFabricPortState (#2850) (18 hours ago) [kenneth-arista]
* c571d8bf - Allow NOT_IMPLEMENTED sai return status for availability monitoring API (#2848) (18 hours ago) [Tejaswini Chadaga]
```
#### How I did it
#### How to verify it
#### Description for the changelog
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants