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

Expanding on Provider Edge bridge #30

Open
demx8as6 opened this issue Nov 2, 2022 · 3 comments
Open

Expanding on Provider Edge bridge #30

demx8as6 opened this issue Nov 2, 2022 · 3 comments
Assignees

Comments

@demx8as6
Copy link
Collaborator

demx8as6 commented Nov 2, 2022

Change on the model based on Equipment issue #7 : adding a parameter for "forwaredCVlanListByDefaultSVlanId" to the VLANInterface in case of "Provider Edge Bridge" and VLANInterface::sublayer equals to C-VLAN.

@openBackhaul
Copy link
Owner

openBackhaul commented Nov 2, 2022

During defining the first release of the VlanInterface, 5G-xhaul team could not agree on a way of modelling the ProvideEdgeBridge.
Mapping of C-VLAN-tagged frames into S-VLANs could either be expressed by structures of additional VlanFC and FcPorts or by an additional datatype in the VlanInterface.
During discussing openBackhaul/equipment#7, adding a datatype in the VlanInterface was the preferred option. This has been documented as a formal decision in openBackhaul/vlanFd#22.

Next step:
Modelling a forwaredCVlanIdListByDefaultSVlanIdType that holds a defaultSVlanId : Integer key attribute and associates a list of forwaredCVlanId : Integer.

@openBackhaul openBackhaul self-assigned this Nov 2, 2022
@openBackhaul
Copy link
Owner

openBackhaul commented Nov 16, 2022

Decision during the 5G-xhaul call on 11th of January 2023:
The following changes shall be made to the modelling of the VlanInterface:

  • A VlanInterfaceConfiguration::forwardedCVlanIdToSVlanIdList : forwardedCVlanIdToSVlanIdType [*] attribute to be added. It shall have "Only effective if (subLayerProtocolName==S_VLAN_COMPONENT AND interfaceKind==CUSTOMER_EDGE_PORT). List of associations between an incoming C-VLAN ID and an S-VLAN ID, which is to be tagged before forwarding the frame into the respective S-VLAN ForwardingConstruct." as a comment.
  • A forwardedCVlanIdToSVlanIdType datatype with comment "To express which C-VLAN ID gets tagged with which S-VLAN ID." shall be added.
  • A forwardedCVlanIdToSVlanIdType::CVlanId : INT16 key attribute with comment "C-VLAN ID, which shall be mapped into an S-VLAN." shall be added.
  • A forwardedCVlanIdToSVlanIdType::SVlanId : INT16 = -1 attribute with comment "S-VLAN the C-VLAN shall be mapped into." shall be added.
  • A forwardedCVlanIdToSVlanIdType::sVlanPriorityKind : SVlanPriorityKindType = NOT_YET_DEFINED attribute with comment "Defines the behavior of the S-VLAN priority setting." shall be added.
  • A forwardedCVlanIdToSVlanIdType::fixedSVlanPriority : INT16 = -1 attribute with comment "Only effective if (sVlanPriorityKind==FIXED). Value of the three priority bits of the S-VLAN header." shall be added.
  • A SVlanPriorityKindType enumeration shall be created. It shall contain the following values:
    • NONE (comment: "The device is setting some default value.")
    • COPY (comment: "The value of the S-VLAN priority is copied from the C-VLAN priority.")
    • FIXED (comment: "The value defined in the fixed-s-vlan-priority attribute is used.")
    • NOT_YET_DEFINED
  • A forwardedCVlanIdToSVlanIdType::untaggedPep : Boolean = false attribute with comment "If set on true, C-tag will be removed while the frame is leaving the CustomerEdgePort (LTP of type C_VLAN_COMPONENT) in direction towards the ProviderBridge (ForwardingDomain of type S_VLAN_COMPONENT)." shall be added.
  • A forwardedCVlanIdToSVlanIdType::untaggedCep : Boolean = false attribute with comment "If set on true, C-tag will be removed while the frame is leaving the ProviderBridge (ForwardingDomain of type S_VLAN_COMPONENT) in direction towards the CustomerEdgePort (LTP of type C_VLAN_COMPONENT)." shall be added.
  • A read-only VlanInterfaceCapability::untaggedPepIsAvail : Boolen = false attribute with comment "If untaggedPepIsAvail==false, the VlanInterfaceConfiguration::untaggedPep attribute would be redundant and frames are always keeping their C-VLAN tag on the ingress." shall be added.
  • A read-only VlanInterfaceCapability::untaggedCepIsAvail : Boolen = false attribute with comment "If untaggedCepIsAvail==false, the VlanInterfaceConfiguration::untaggedCep attribute would be redundant and frames are always keeping their C-VLAN tag on the egress." shall be added.

@openBackhaul
Copy link
Owner

UML changed as described above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants