Skip to content

[schema] Add VNET Route tables in config_db#279

Merged
prsunny merged 3 commits intosonic-net:masterfrom
weixchen1215:weixi_changes_route_perisistence
May 24, 2019
Merged

[schema] Add VNET Route tables in config_db#279
prsunny merged 3 commits intosonic-net:masterfrom
weixchen1215:weixi_changes_route_perisistence

Conversation

@weixchen1215
Copy link
Copy Markdown
Contributor

  • VNET_ROUTE
  • VNET_ROUTE_TUNNEL

common/schema.h Outdated
#define CFG_NEIGH_TABLE_NAME "NEIGH"

#define CFG_VNET_RT_TABLE_NAME "VNET_ROUTE"
#define CFG_VNET_RT_TUNNEL_TABLE_NAME "VNET_ROUTE_TUNNEL"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

align with the spaces?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok. Will fix it

@prsunny
Copy link
Copy Markdown
Contributor

prsunny commented May 23, 2019

@weixi, can you please use space instead of tab to align. Its still not aligned.

@weixchen1215 weixchen1215 changed the title Add VNET Route tables in config_db [schema] Add VNET Route tables in config_db May 23, 2019
@prsunny
Copy link
Copy Markdown
Contributor

prsunny commented May 23, 2019

retest this please

3 similar comments
@weixchen1215
Copy link
Copy Markdown
Contributor Author

retest this please

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented May 24, 2019

retest this please

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented May 24, 2019

retest this please

Copy link
Copy Markdown
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.

thx

@prsunny prsunny merged commit 9e5e47e into sonic-net:master May 24, 2019
prgeor pushed a commit to prgeor/sonic-swss-common that referenced this pull request Feb 27, 2025
…percent (sonic-net#279)

Description
The command is not showing the correct value for ssd health.

admin@sonic:~$ show platform ssdhealth
Device Model : M.2 (S42) 3IE4
Health       : N/A
Temperature  : 25C
Motivation and Context
SSD health percentage not displayed on Nokia-7215 platform.

How Has This Been Tested?
"show platform ssdhealth" cli command
Output after fix:

admin@sonic:~$ show platform ssdhealth 
Device Model : M.2 (S42) 3IE4
Health       : 100%
Temperature  : 25C
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.

4 participants