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

Revise metadata semantics yang module for INT v2 #91

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mickeyspiegel
Copy link
Contributor

  • Rename p4-dtel-metadata-semantics.yang to p4-int-metadata-semantics.yang
    • Align with strategy to call all aspects and variants part of INT
  • Replace per metadata field groupings with generic grouping that
    can be used for metadata semantics details of all fields
    • Add identity statements for each baseline metadata field
    • Add identity statements for three common domain specific fields
      • flow_id
      • sequence_number
      • mac_address
    • Move leafs that apply to specific metadata fields to the generic
      grouping, conditioning the leafs using "when" statements
    • Move node-id to generic grouping, conditioning this leaf
      when the metadata field is node_id
    • Remove grouping for identifiers since this is moved to generic per
      metadata field groupings
    • Add grouping for domain specific metadata details including
      • size in 4-byte words
      • INT-MX properties as defined in INT 2.1
  • Add nested groupings for domain specific ids, along with bit
    position and identity reference for each domain specific metadata
    field associated with each domain

Align with strategy to call all aspects and variants part of INT.
- Replace per metadata field groupings with generic grouping that
  can be used for metadata semantics details of all fields
  - Add identity statements for each baseline metadata field
  - Add identity statements for three common domain specific fields
    - flow_id
    - sequence_number
    - mac_address
  - Move leafs that apply to specific metadata fields to the generic
    grouping, conditioning the leafs using "when" statements
  - Move node-id to generic grouping, conditioning this leaf
    when the metadata field is node_id
  - Remove grouping for identifiers since this is moved to generic per
    metadata field groupings
  - Add grouping for domain specific metadata details including
    - size in 4-byte words
    - INT-MX properties as defined in INT 2.1
- Add nested groupings for domain specific ids, along with bit
  position and identity reference for each domain specific metadata
  field associated with each domain
@mickeyspiegel mickeyspiegel requested review from jklr and a team April 3, 2021 02:07
@onf-cla-manager
Copy link

onf-cla-manager bot commented Apr 3, 2021

Hi @mickeyspiegel, this is the ONF bot 🤖 I'm glad you want to contribute to our projects! However, before accepting your contribution, we need to ask you to sign a Contributor License Agreement (CLA). You can do it online, it will take only a few minutes:

✒️ 👉 https://cla.opennetworking.org

After signing, make sure to add your Github user ID mickeyspiegel to the agreement.

For more information or help:"
https://wiki.opennetworking.org/x/BgCUI

Also add support for bit field and boolean.
Map between drop reason codepoints and identityrefs to drop reason
name and semantics. The identity statements for the many drop
reasons are to be added in future commits.
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.

1 participant