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

Add first version of ConfigDB schema #9

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

Conversation

taoyl-ms
Copy link
Contributor

No description provided.

### Definitions of common tokens
name = 1*DIGIT/1*ALPHA
ref_hash_key_reference = "[" hash_key "]" ;The token is a refernce to another valid DB key.
hash_key = name ; a valid key name (i.e. exists in DB)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

align the ;

; Key
portchannel_key = 1*64VCHAR
; Attributes
members = LIST(1*64VCHAR)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how different members are split?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by comma, see line 7.

@sonic-net sonic-net deleted a comment from msftclas Sep 27, 2017
id = 1*5DIGIT
members = LIST(1*64VCHAR)

### VLAN_INTERFACE table
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we have different interface table for VLAN? why not inside the INTERFACE table?

; Attributes
members = LIST(1*64VCHAR)

### PORTCHANNEL_INTERFACE table
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we have different interface table for port channel? why not inside the INTERFACE table?

; Attributes
; No attributes

### PORTCHANNEL table
Copy link
Contributor

@stcheng stcheng Oct 4, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's better to use LAG. port channel is mostly a NXOS term.

Copy link
Contributor

@stcheng stcheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check comments

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

Successfully merging this pull request may close these issues.

5 participants