public icon indicating copy to clipboard operation
public copied to clipboard

Private VLAN configuration changes to VLAN Yang model

Open madhankumar76 opened this issue 5 years ago • 7 comments

Submitting this on behalf of Cisco as per request from Google team. Private VLAN (a.k.a PVLAN) configuration is introduced as part of VLAN yang file. Kindly review the same.

Snippets from Cisco configuration guide explaining PVLAN,

Using private VLANs provides scalability and IP address management benefits for service providers and Layer 2 security for customers. Private VLANs partition a regular VLAN domain into subdomains. A subdomain is represented by a pair of VLANs: a primary VLAN and a secondary VLAN. A private VLAN can have multiple VLAN pairs, one pair for each subdomain. All VLAN pairs in a private VLAN share the same primary VLAN. The secondary VLAN ID differentiates one subdomain from another.

madhankumar76 avatar Nov 05 '20 02:11 madhankumar76

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

google-cla[bot] avatar Nov 05 '20 02:11 google-cla[bot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

googlebot avatar Nov 05 '20 02:11 googlebot

Compatibility Report for commit 9e49e9df4879c7542402dd844aab5819781766e2: ⛔ yanglint@SO 1.7.11

OpenConfigBot avatar Nov 05 '20 02:11 OpenConfigBot

@googlebot I signed it

From: "google-cla[bot]" [email protected] Reply-To: openconfig/public [email protected] Date: Wednesday, November 4, 2020 at 6:11 PM To: openconfig/public [email protected] Cc: Madhan Kumar [email protected], Author [email protected] Subject: Re: [openconfig/public] Private VLAN configuration changes to VLAN Yang model (#387)

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA Individual signers

Corporate signers

ℹ️ Googlers: Go herehttps://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fopenconfig%2Fpublic%2Fpull%2F387 for more info.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/openconfig/public/pull/387#issuecomment-722080267, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARUJMRZD5PFLCLUETFOZFMTSOICVHANCNFSM4TKYWWHQ.

madhankumar76 avatar Dec 14 '20 17:12 madhankumar76

@googlebot I signed it

From: googlebot [email protected] Reply-To: openconfig/public [email protected] Date: Wednesday, November 4, 2020 at 6:12 PM To: openconfig/public [email protected] Cc: Madhan Kumar [email protected], Author [email protected] Subject: Re: [openconfig/public] Private VLAN configuration changes to VLAN Yang model (#387)

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA Individual signers

Corporate signers

ℹ️ Googlers: Go herehttps://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fopenconfig%2Fpublic%2Fpull%2F387 for more info.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/openconfig/public/pull/387#issuecomment-722080270, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARUJMR6IBCMAN7Z4KXNYJLLSOICVJANCNFSM4TKYWWHQ.

madhankumar76 avatar Dec 14 '20 17:12 madhankumar76

Hi @madhankumar76 - Can you please make sure that the linter checks pass here as well? Our linter is publicly available so you should be able to validate this before submitting. Let me know if you need help or directions with that.

jkoolwal avatar Mar 17 '21 07:03 jkoolwal

Hi @madhankumar76 - Can you please make sure that the linter checks pass here as well? Our linter is publicly available so you should be able to validate this before submitting. Let me know if you need help or directions with that.

I ran this in my repository and it did not produce any error, pyang --strict --lint -p /workspace/openconfig/public/release/models/ openconfig-vlan*

Let me know if this is sufficient or do i need to use any other tool for linter checks?

madhankumar76 avatar Mar 17 '21 18:03 madhankumar76