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

[Sonic-Telemetry] update telemetry submodule #6810

Merged
merged 1 commit into from
Feb 25, 2021

Conversation

pra-moh
Copy link
Contributor

@pra-moh pra-moh commented Feb 18, 2021

Why I did it

Updating telemetry submodule including following commits:

Adding instructions to build telemetry server/go client inside docker (#68)
Add RBAC, gNOI, API Versioning and Bulk Set operations. (#61) …
Support NonTLS option for testing purpose only. (#67) …
Oc gnmi examples (#62) …

How I did it

How to verify it

Docker starting in noTLS without redis keys as expected

root@str-s6000-acs-10:/# ps aux
USER PID %CPU %MEM VSZ RSS TTY STAT START TIME COMMAND
root 1 0.0 0.2 58964 21304 pts/0 Ss+ Feb05 23:57 /usr/bin/python /usr/bin/supervisord
root 9 0.0 0.1 51908 16088 pts/0 S Feb05 0:00 python /usr/bin/supervisor-proc-exit-listener --container-name telemetry
root 13 0.0 0.0 262984 3588 pts/0 Sl Feb05 0:00 /usr/sbin/rsyslogd -n -iNONE
root 53 1.2 0.5 1164628 41572 pts/0 Sl Feb05 328:03 /usr/sbin/dialout_client_cli -insecure -logtostderr -v 2
root 388 0.0 0.0 18188 3388 pts/1 Ss+ Feb06 0:00 bash
root 609 0.0 0.0 18192 3304 pts/2 Ss+ Feb06 0:00 bash
root 662 0.0 0.0 18188 3352 pts/3 Ss+ Feb06 0:00 bash
root 701 0.0 0.0 18240 3460 pts/4 Ss+ Feb06 0:00 bash
root 822 0.0 0.0 18256 3388 pts/5 Ss+ Feb06 0:00 bash
root 858 0.0 0.0 18188 3200 pts/6 Ss+ Feb17 0:00 bash
root 864 0.0 0.0 18192 3320 pts/7 Ss+ 18:31 0:00 bash
root 898 0.0 0.0 18192 3392 pts/8 Ss 19:15 0:00 bash
root 907 1.7 0.6 1038604 50868 pts/8 Sl+ 19:16 0:07 /usr/sbin/telemetry -logtostderr --noTLS --port 8080 --allow_no_client_auth -v=2
root 941 2.0 0.0 18188 3196 pts/9 Ss 19:23 0:00 bash
root 946 0.0 0.0 36632 2836 pts/9 R+ 19:23 0:00 ps aux
root@str-s6000-acs-10:/#

Performed sudo config load minigraph
copied certs on device and restarted docker

After above steps docker starts with secure mode as expecetd:

admin@str-s6000-acs-10:~$ docker exec -it telemetry bash
root@str-s6000-acs-10:/# ps aux
USER PID %CPU %MEM VSZ RSS TTY STAT START TIME COMMAND
root 1 5.4 0.2 58940 21348 pts/0 Ss+ 22:02 0:02 /usr/bin/python /usr/bin/supervisord
root 9 1.0 0.1 51908 16300 pts/0 S 22:02 0:00 python /usr/bin/supervisor-proc-exit-listener --container-name telemetry
root 13 0.1 0.0 262984 3628 pts/0 Sl 22:02 0:00 /usr/sbin/rsyslogd -n -iNONE
root 21 5.0 0.4 1104140 39756 pts/0 Rl 22:02 0:02 /usr/sbin/telemetry -logtostderr --server_crt /etc/sonic/telemetry/streamingtelemetryserver.cer --server_key /etc/sonic/telemetry/streamingtelemetryserver.key --ca_crt
root 49 4.1 0.3 959824 29100 pts/0 Sl 22:02 0:01 /usr/sbin/dialout_client_cli -insecure -logtostderr -v 2
root 71 2.5 0.0 18192 3192 pts/1 Ss 22:02 0:00 bash
root 76 0.0 0.0 36632 2848 pts/1 R+ 22:03 0:00 ps aux
root@str-s6000-acs-10:/#

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

@pra-moh pra-moh changed the title [Telemetry] update telemetry submodule [Sonic-Telemetry] update telemetry submodule Feb 18, 2021
@abdosi
Copy link
Contributor

abdosi commented Feb 19, 2021

is this needed for 201911 ?

@pra-moh
Copy link
Contributor Author

pra-moh commented Feb 19, 2021

is this needed for 201911 ?

No need to backport

@lguohan lguohan merged commit 2a339fa into sonic-net:master Feb 25, 2021
lguohan pushed a commit that referenced this pull request Mar 4, 2021
- Adding instructions to build telemetry server/go client inside docker (#68)
- Add RBAC, gNOI, API Versioning and Bulk Set operations. (#61) …
- Support NonTLS option for testing purpose only. (#67) …
- Oc gnmi examples (#62) …
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
- Adding instructions to build telemetry server/go client inside docker (sonic-net#68)
- Add RBAC, gNOI, API Versioning and Bulk Set operations. (sonic-net#61) …
- Support NonTLS option for testing purpose only. (sonic-net#67) …
- Oc gnmi examples (sonic-net#62) …
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.

4 participants