Skip to content

Warm reboot: Add CONFIG DB and STATE DB warm restart table names#211

Merged
lguohan merged 2 commits intosonic-net:masterfrom
jipanyang:warm_reboot_collab
Aug 2, 2018
Merged

Warm reboot: Add CONFIG DB and STATE DB warm restart table names#211
lguohan merged 2 commits intosonic-net:masterfrom
jipanyang:warm_reboot_collab

Conversation

@jipanyang
Copy link
Copy Markdown
Contributor

@jipanyang jipanyang commented Jul 25, 2018

Signed-off-by: Jipan Yang [email protected]

"WARM_RESTART" table in CONFIG DB for warm restart related configuration.

"WARM_RESTART_TABLE" table in STATE DB for warm restart related state in swss.

@jipanyang jipanyang changed the title Add configDB and APPDB warm restart table names Warm reboot: Add configDB and APPDB warm restart table names Aug 1, 2018
@lguohan lguohan changed the title Warm reboot: Add configDB and APPDB warm restart table names Warm reboot: Add CONFIG DB and STATE DB warm restart table names Aug 2, 2018
@lguohan lguohan merged commit 531c4f1 into sonic-net:master Aug 2, 2018
@jipanyang jipanyang deleted the warm_reboot_collab branch February 3, 2019 01:42
prgeor pushed a commit to prgeor/sonic-swss-common that referenced this pull request Feb 27, 2025
….credo as a package (sonic-net#211)

Signed-off-by: vaibhav-dahiya <[email protected]>
Curreently the way python wheel includes packages is all the subdirectories inside a package need to be also included
for them to be a package. Since in the refactor for multiple vendor Y-Cable xcvrd calls these vendor specific modules by including their name which it obtains by reading the eeprom it is required to include them inside setup.py for the wheel to know it has a package.

Signed-off-by: vaibhav-dahiya [email protected]

Description
add the definition inside setup.py to include sonic_y_cabe.credo as a package

Motivation and Context
This is required by all calling API's for vendor agnostic Y-Cable API's to be called for credo specific cables.

How Has This Been Tested?
Built an image and validated the presence of the pacakge.
Ran the API's on a Arista7050cx3 switch.
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.

2 participants