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 the feature to get port configuration #68

Open
zhouhao3 opened this issue Jun 23, 2021 · 3 comments
Open

Add the feature to get port configuration #68

zhouhao3 opened this issue Jun 23, 2021 · 3 comments

Comments

@zhouhao3
Copy link
Contributor

Hi! We tried to use network-runner as the backend to configure the switch, but we found that network-runner does not have the function of obtaining port configuration. So I would like to ask if it is possible to add a feature in network-runner to get the port configuration?

@radez
Copy link
Contributor

radez commented Jun 23, 2021

It is possible, we recently added list_vlans as an api call.
What would your vision be for the naming? would you like to submit a patch that would add to the api?

@zhouhao3
Copy link
Contributor Author

What would your vision be for the naming?

Get Port Config(get_port_conf)

would you like to submit a patch that would add to the api?

Yes, If the community accepts this idea, I will post a PR in the near future to add this feature.

@radez
Copy link
Contributor

radez commented Jun 24, 2021

Sounds good.
I think get_port_conf or show_port or something like that will work great. Thanks!

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

No branches or pull requests

2 participants