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

[telemetry]: change the service dependency from swss to database #3072

Merged
merged 1 commit into from
Jun 24, 2019

Conversation

jipanyang
Copy link
Collaborator

Signed-off-by: Jipan Yang jipan.yang@alibaba-inc.com

- What I did
Change Telemetry container dependency from swss to database.

Telemetry container doesn't have dependency on swss service.

- How I did it

- How to verify it
Stop swss service, telemetry container remains up

root@vlab-01:/etc/systemd/system# systemctl stop swss
root@vlab-01:/etc/systemd/system# docker ps
CONTAINER ID        IMAGE                           COMMAND                  CREATED             STATUS              PORTS               NAMES
dc801e18e9ed        docker-sonic-telemetry:latest   "/usr/bin/supervisord"   36 hours ago        Up About a minute                       telemetry
334d8a1ce0ef        docker-fpm-frr:latest           "/usr/bin/supervisord"   36 hours ago        Up 5 hours                              bgp
899e7f34b639        docker-lldp-sv2:latest          "/usr/bin/supervisord"   36 hours ago        Up 5 hours                              lldp
d8d419e050e6        docker-database:latest          "/usr/bin/supervisord"   36 hours ago        Up 5 hours                              database
root@vlab-01:/etc/systemd/system# systemctl status telemetry
● telemetry.service - Telemetry container
   Loaded: loaded (/etc/systemd/system/telemetry.service; enabled; vendor preset: enabled)
   Active: active (running) since Mon 2019-06-24 08:45:03 UTC; 1min 23s ago
  Process: 27588 ExecStartPre=/usr/bin/telemetry.sh start (code=exited, status=0/SUCCESS)
 Main PID: 28180 (telemetry.sh)
    Tasks: 9 (limit: 4915)
   Memory: 20.5M
      CPU: 2.563s
   CGroup: /system.slice/telemetry.service
           ├─28180 /bin/bash /usr/bin/telemetry.sh wait
           └─28183 docker wait telemetry

Jun 24 08:44:51 vlab-01 systemd[1]: Starting Telemetry container...
Jun 24 08:44:55 vlab-01 telemetry.sh[27588]: Starting existing telemetry container with HWSKU Force10-S6000
Jun 24 08:44:56 vlab-01 telemetry.sh[27588]: telemetry
Jun 24 08:44:59 vlab-01 telemetry.sh[27588]: Set hostname in telemetry container
Jun 24 08:45:03 vlab-01 systemd[1]: Started Telemetry container.

Restart swss service, telemetry service stays as before:

root@vlab-01:/etc/systemd/system# systemctl  restart swss
root@vlab-01:/etc/systemd/system# docker ps
CONTAINER ID        IMAGE                             COMMAND                  CREATED             STATUS              PORTS               NAMES
5f38ac6776e2        docker-syncd-vs:latest            "/usr/bin/supervisord"   36 hours ago        Up 3 minutes                            syncd
25e96b649ee1        docker-dhcp-relay:latest          "/usr/bin/docker_ini…"   36 hours ago        Up 2 minutes                            dhcp_relay
15fe114b6e9c        docker-snmp-sv2:latest            "/usr/bin/supervisord"   36 hours ago        Up 3 minutes                            snmp
21caf4cb0468        docker-router-advertiser:latest   "/usr/bin/supervisord"   36 hours ago        Up 3 minutes                            radv
1c52ab2a93f3        docker-teamd:latest               "/usr/bin/supervisord"   36 hours ago        Up 3 minutes                            teamd
dc801e18e9ed        docker-sonic-telemetry:latest     "/usr/bin/supervisord"   36 hours ago        Up 5 minutes                            telemetry
7322f76f110d        docker-orchagent:latest           "/usr/bin/supervisord"   36 hours ago        Up 3 minutes                            swss
334d8a1ce0ef        docker-fpm-frr:latest             "/usr/bin/supervisord"   36 hours ago        Up 5 hours                              bgp
899e7f34b639        docker-lldp-sv2:latest            "/usr/bin/supervisord"   36 hours ago        Up 5 hours                              lldp
d8d419e050e6        docker-database:latest            "/usr/bin/supervisord"   36 hours ago        Up 5 hours                              database
root@vlab-01:/etc/systemd/system# systemctl status telemetry
● telemetry.service - Telemetry container
   Loaded: loaded (/etc/systemd/system/telemetry.service; enabled; vendor preset: enabled)
   Active: active (running) since Mon 2019-06-24 08:45:03 UTC; 6min ago
  Process: 27588 ExecStartPre=/usr/bin/telemetry.sh start (code=exited, status=0/SUCCESS)
 Main PID: 28180 (telemetry.sh)
    Tasks: 9 (limit: 4915)
   Memory: 20.7M
      CPU: 2.586s
   CGroup: /system.slice/telemetry.service
           ├─28180 /bin/bash /usr/bin/telemetry.sh wait
           └─28183 docker wait telemetry

Jun 24 08:44:51 vlab-01 systemd[1]: Starting Telemetry container...
Jun 24 08:44:55 vlab-01 telemetry.sh[27588]: Starting existing telemetry container with HWSKU Force10-S6000
Jun 24 08:44:56 vlab-01 telemetry.sh[27588]: telemetry
Jun 24 08:44:59 vlab-01 telemetry.sh[27588]: Set hostname in telemetry container
Jun 24 08:45:03 vlab-01 systemd[1]: Started Telemetry container.
root@vlab-01:/etc/systemd/system# 

Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
@jleveque jleveque merged commit 9a1bebe into sonic-net:master Jun 24, 2019
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request Feb 9, 2021
…ic-net#3072)

Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>

RB=2177797
G=lnos-reviewers
R=pchaudhary,pmao,rmolina,samaity,sfardeen,zxu
A=pmao
mssonicbld added a commit that referenced this pull request Jan 12, 2024
…atically (#17752)

#### Why I did it
src/sonic-utilities
```
* 72b6c04c - (HEAD -> 202311, origin/202311) Support disable/enable syslog rate limit feature (#3072) (2 days ago) [Junchao-Mellanox]
```
#### How I did it
#### How to verify it
#### Description for the changelog
yxieca added a commit to yxieca/sonic-buildimage that referenced this pull request Jan 13, 2024
Old format: version_a_b_c
New format: version_<branch>_<nn>

sonic-utilities:
* 371d62a2 2024-01-13 | Revert "Enhanced route_check.py for multi_asic platforms (sonic-net#3077)" (HEAD -> master, github/ying-test-submodule-advance) [Ying Xie]
* 7ddc709a 2024-01-13 | Revert "route_check: Skip route checks if bgp feature is not enabled (sonic-net#3075)" [Ying Xie]
* 1489c727 2024-01-12 | [Techsupport]Adding more FRR and BGP dumps (sonic-net#3118) (github/master) [Sudharsan Dhamal Gopalarathnam]
* 359e6925 2024-01-11 | Disable Key Validation feature during sonic-installation for Cisco Platforms (sonic-net#3115) [selvipal]
* 9515c642 2024-01-04 | [chassis]: Support show ip bgp summary to display without error when no external neighbors are configured on chassis LC (sonic-net#3099) [Arvindsrinivasan Lakshmi Narasimhan]
* 9400691c 2023-12-25 | Fix database initialization for db_migrator (sonic-net#3100) [ganglv]
* 56dafb07 2023-12-25 | Support disable/enable syslog rate limit feature (sonic-net#3072) [Junchao-Mellanox]
* 529bb96b 2023-12-23 | route_check: Skip route checks if bgp feature is not enabled (sonic-net#3075) [anamehra]
* bcb10f18 2023-12-23 | Support golden config in db migrator (sonic-net#3076) [ganglv]
* 20d1495b 2023-12-21 | [db_migrator] add db migrator version space for 202305/202311 branch (sonic-net#3081) [Ying Xie]
* a68d3d3a 2023-12-20 | Collect module EEPROM data in dump (sonic-net#3009) [Junchao-Mellanox]
* e7a8def6 2023-12-19 | Enhanced route_check.py for multi_asic platforms (sonic-net#3077) [Deepak Singhal]

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
mssonicbld added a commit that referenced this pull request Jan 15, 2024
…atically (#17573)

#### Why I did it
src/sonic-utilities
```
* 942a7c9e - (HEAD -> master, origin/master, origin/HEAD) Revert "Enhanced route_check.py for multi_asic platforms" (#3122) (2 minutes ago) [Ying Xie]
* 01ee98ec - Revert "route_check: Skip route checks if bgp feature is not enabled" (#3121) (3 minutes ago) [Ying Xie]
* 1489c727 - [Techsupport]Adding more FRR and BGP dumps (#3118) (2 days ago) [Sudharsan Dhamal Gopalarathnam]
* 359e6925 - Disable Key Validation feature during sonic-installation for Cisco Platforms (#3115) (4 days ago) [selvipal]
* 9515c642 - [chassis]: Support show ip bgp summary to display without error when no external neighbors are configured on chassis LC (#3099) (10 days ago) [Arvindsrinivasan Lakshmi Narasimhan]
* 9400691c - Fix database initialization for db_migrator (#3100) (3 weeks ago) [ganglv]
* 56dafb07 - Support disable/enable syslog rate limit feature (#3072) (3 weeks ago) [Junchao-Mellanox]
* 529bb96b - route_check: Skip route checks if bgp feature is not enabled (#3075) (3 weeks ago) [anamehra]
* bcb10f18 - Support golden config in db migrator (#3076) (3 weeks ago) [ganglv]
* 20d1495b - [db_migrator] add db migrator version space for 202305/202311 branch (#3081) (4 weeks ago) [Ying Xie]
* a68d3d3a - Collect module EEPROM data in dump (#3009) (4 weeks ago) [Junchao-Mellanox]
* e7a8def6 - Enhanced route_check.py for multi_asic platforms (#3077) (4 weeks ago) [Deepak Singhal]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants