Skip to content

Merging base repo changes #3

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

Merged
merged 2 commits into from
Dec 8, 2020
Merged

Conversation

macikgozwa
Copy link
Owner

No description provided.

macikgozwa and others added 2 commits December 3, 2020 10:09
* New data set to query Sonic build version.

* Adding error message in case version file is not read with success.

Co-authored-by: Murat Acikgoz <[email protected]>
Motivation of this PR:
This PR aims to enable the streaming telemetry container to stream out the system uptime of SONiC Host. This dataset is added into non-database client since it is a kind of value which will be refreshed periodically.

How can I do that?
I follow the example of dataset in non-database client such as meminfo to do the implementation. The data source of system uptime is from the file /proc/uptime on the SONiC host.

We can use the command ./gnmi_cli -client_types=gnmi -a <DuT_IP>:8080 -t OTHERS -logtostderr -insecure -qt p -pi 10s -q proc/uptime to query the system uptime every 10 seconds.

admin@str-a7050-acs-3:~$ ./gnmi_cli -client_types=gnmi -a localhost:8080 -t OTHERS -logtostderr -insecure -qt p -pi 10s -q proc/uptime
{
"OTHERS": {
"proc": {
"uptime": "{"total":314.74,"idle":981.27}"
}
}
}
{
"OTHERS": {
"proc": {
"uptime": "{"total":324.74,"idle":1016.85}"
}
}
}
@macikgozwa macikgozwa merged commit 2911afe into macikgozwa:master Dec 8, 2020
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