Skip to content

Add RedisClient to pyext#176

Merged
lguohan merged 1 commit intosonic-net:masterfrom
kcudnik:ro
Jan 15, 2018
Merged

Add RedisClient to pyext#176
lguohan merged 1 commit intosonic-net:masterfrom
kcudnik:ro

Conversation

@kcudnik
Copy link
Contributor

@kcudnik kcudnik commented Jan 15, 2018

Required for pyext to get access to VIDTORID table

@kcudnik kcudnik requested a review from lguohan January 15, 2018 23:02
std::vector<std::string> keys(std::string key);

std::vector<std::string> hkeys(std::string key);
//std::vector<std::string> hkeys(std::string key);
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 comment them?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

swig is complaining that there is no implementation for those methods since i just left signature for those mthods, but there is no implementation yet

@lguohan lguohan merged commit d630d57 into sonic-net:master Jan 15, 2018
lguohan added a commit that referenced this pull request Jan 16, 2018
kcudnik pushed a commit that referenced this pull request Jan 16, 2018
prgeor pushed a commit to prgeor/sonic-swss-common that referenced this pull request Feb 27, 2025
sonic-net#176)

This PR adds functions definition with details for enabling/disabling Y cable auto-switch feature

Description
Added functions for enabling/disabling auto-switch on Y cable

def set_switching_mode(physical_port, mode)
def get_switching_mode(physical_port)
Motivation and Context
We need to have this feature within host cli so that a user can enable/disable the autoswitch on Y cable

How Has This Been Tested?
opened a python shell and executed API's for correctness

Signed-off-by: vaibhav-dahiya <[email protected]>
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