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

[201911][multi-asic][RFC1213]: Oper status of mgmt interface for multi-asic platform #210

Open
wants to merge 1 commit into
base: 201911
Choose a base branch
from

Conversation

SuvarnaMeenakshi
Copy link
Contributor

[201911][multi-asic][RFC1213]: Retrieve oper status of management
interface from kernel for multi-asic platform.
In multi-asic platform there is no daemon in the host
namespace to update the oper status of management interface.
Instead of retrieving this information from state_db,
retrieve this from /sys/class/net/../operstate file.

Signed-off-by: Suvarna Meenakshi sumeenak@microsoft.com

  • What I did
    This PR is same as : [multi-asic][RFC1213]: Oper status of mgmt interface for multi-asic platform #209; for 201911
    Currently oper status of all management interface is retrieved from STATE DB.
    Multi-asic platform does not have any daemon running on the host namespace to update oper status of management interface in STATE DB. This PR is to add a function to retrieve oper status of management interface from kernel file: "/sys/class/net/if_name/operstate".
  • How I did it

Add a new function to retrieve oper status from /sys/class/net/if_name/operstate.
Invoke this function periodically to retrieve oper status of all management interfaces.
This is done periodically, instead of retrieving only on SNMP query, so that the time take during the SNMP query does not increase due to the file read operation.
Add unit-tests for mgmt interface.

  • How to verify it
    Run snmpwalk to get interface status, ensure that the interface status of management interface is correct, on single asic and multi-asic VS testbed.
    With the changes - no impact of single asic platform.
    - Description for the changelog

interface from kernel for multi-asic platform.
In multi-asic platform there is no daemon in the host
namespace to update the oper status of management interface.
Instead of retrieving this information from state_db,
retrieve this from /sys/class/net/../operstate file.

Signed-off-by: Suvarna Meenakshi <sumeenak@microsoft.com>
@qiluo-msft
Copy link
Contributor

Pending on #209

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