Skip to content

Addition of host-resource subagents to return supported OIDs in HOST-…#332

Closed
rsh2prasad wants to merge 0 commit into
sonic-net:masterfrom
rsh2prasad:master
Closed

Addition of host-resource subagents to return supported OIDs in HOST-…#332
rsh2prasad wants to merge 0 commit into
sonic-net:masterfrom
rsh2prasad:master

Conversation

@rsh2prasad
Copy link
Copy Markdown

@rsh2prasad rsh2prasad commented Oct 3, 2024

What I did ...
Added host resource subagents in src/sonic-snmpagent to return the data for the supported OIDs in HOST-RESOURCES-V2-MIB
The OIDs can be found here: https://mibbrowser.online/mibdb_search.php?mib=HOST-RESOURCES-V2-MIB
In addition, the REDIS entry "MOUNT_POINTS | LastUpdateTime" shows up in the SNMP walk, when it should not, because there is no associated OID entry for this value. This entry was added for logging and consistency purposes.

How I did it ...
Added classes hrFSTable & hrStorageTable to handle the filesystem and physical memory related attributes for SNMP GET request

How I verified it ...
Added unit tests to verify the newly added OID implementation. The steps to run the unit tests can be found in azure-pipelines.yml in the sonic-snmpagent repo

Description ...
Add SNMP GET support and related unit tests for memory stats, filesystem and total disk usage table in ietf rfc2790 MIB

Please note, this PR is dependent on the following 2 PRs:
sonic-net/sonic-host-services#235
sonic-net/sonic-buildimage#20428

Signed-off-by: Rakshintha Prasad [rakprasa@cisco.com]

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Oct 3, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@rsh2prasad rsh2prasad marked this pull request as ready for review March 14, 2025 18:01
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@rsh2prasad rsh2prasad marked this pull request as draft March 14, 2025 18:35
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@rsh2prasad rsh2prasad marked this pull request as ready for review April 29, 2025 19:32
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

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