From 187212835588a9a3d8d52db3fddaff2162579d0d Mon Sep 17 00:00:00 2001
From: Joe LeVeque <jleveque@users.noreply.github.com>
Date: Fri, 8 Mar 2019 10:59:41 -0800
Subject: [PATCH] [services] Services which start containers now use 'docker
 wait' instead of 'docker attach' (#2661)

---
 files/build_templates/bgp.service.j2        | 2 +-
 files/build_templates/database.service.j2   | 2 +-
 files/build_templates/dhcp_relay.service.j2 | 2 +-
 files/build_templates/docker_image_ctl.j2   | 8 ++++----
 files/build_templates/lldp.service.j2       | 2 +-
 files/build_templates/pmon.service.j2       | 2 +-
 files/build_templates/radv.service.j2       | 2 +-
 files/build_templates/snmp.service.j2       | 2 +-
 files/build_templates/swss.service.j2       | 2 +-
 files/build_templates/syncd.service.j2      | 2 +-
 files/build_templates/teamd.service.j2      | 2 +-
 files/build_templates/telemetry.service.j2  | 2 +-
 files/scripts/swss.sh                       | 8 ++++----
 files/scripts/syncd.sh                      | 8 ++++----
 14 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/files/build_templates/bgp.service.j2 b/files/build_templates/bgp.service.j2
index bc023b7d714d..7200a0e3ecf2 100644
--- a/files/build_templates/bgp.service.j2
+++ b/files/build_templates/bgp.service.j2
@@ -7,7 +7,7 @@ Before=ntp-config.service
 [Service]
 User={{ sonicadmin_user }}
 ExecStartPre=/usr/bin/{{docker_container_name}}.sh start
-ExecStart=/usr/bin/{{docker_container_name}}.sh attach
+ExecStart=/usr/bin/{{docker_container_name}}.sh wait
 ExecStop=/usr/bin/{{docker_container_name}}.sh stop
 
 [Install]
diff --git a/files/build_templates/database.service.j2 b/files/build_templates/database.service.j2
index b248dfc7fecd..efa98b168beb 100644
--- a/files/build_templates/database.service.j2
+++ b/files/build_templates/database.service.j2
@@ -6,7 +6,7 @@ After=docker.service
 [Service]
 User=root
 ExecStartPre=/usr/bin/{{docker_container_name}}.sh start
-ExecStart=/usr/bin/{{docker_container_name}}.sh attach
+ExecStart=/usr/bin/{{docker_container_name}}.sh wait
 ExecStop=/usr/bin/{{docker_container_name}}.sh stop
 
 [Install]
diff --git a/files/build_templates/dhcp_relay.service.j2 b/files/build_templates/dhcp_relay.service.j2
index b4d9ceb38f0c..5a462e53973d 100644
--- a/files/build_templates/dhcp_relay.service.j2
+++ b/files/build_templates/dhcp_relay.service.j2
@@ -7,7 +7,7 @@ Before=ntp-config.service
 [Service]
 User={{ sonicadmin_user }}
 ExecStartPre=/usr/bin/{{ docker_container_name }}.sh start
-ExecStart=/usr/bin/{{ docker_container_name }}.sh attach
+ExecStart=/usr/bin/{{ docker_container_name }}.sh wait
 ExecStop=/usr/bin/{{ docker_container_name }}.sh stop
 
 [Install]
diff --git a/files/build_templates/docker_image_ctl.j2 b/files/build_templates/docker_image_ctl.j2
index a3cbf5adbae7..ca25ac2b43e4 100644
--- a/files/build_templates/docker_image_ctl.j2
+++ b/files/build_templates/docker_image_ctl.j2
@@ -150,8 +150,8 @@ start() {
     postStartAction
 }
 
-attach() {
-    docker attach --no-stdin {{docker_container_name}}
+wait() {
+    docker wait {{docker_container_name}}
 }
 
 stop() {
@@ -159,11 +159,11 @@ stop() {
 }
 
 case "$1" in
-    start|stop|attach)
+    start|wait|stop)
         $1
         ;;
     *)
-        echo "Usage: $0 {start|stop|attach}"
+        echo "Usage: $0 {start|wait|stop}"
         exit 1
         ;;
 esac
diff --git a/files/build_templates/lldp.service.j2 b/files/build_templates/lldp.service.j2
index d294a08900f4..c317e18efc5c 100644
--- a/files/build_templates/lldp.service.j2
+++ b/files/build_templates/lldp.service.j2
@@ -7,7 +7,7 @@ Before=ntp-config.service
 [Service]
 User={{ sonicadmin_user }}
 ExecStartPre=/usr/bin/{{docker_container_name}}.sh start
-ExecStart=/usr/bin/{{docker_container_name}}.sh attach
+ExecStart=/usr/bin/{{docker_container_name}}.sh wait
 ExecStop=/usr/bin/{{docker_container_name}}.sh stop
 
 [Install]
diff --git a/files/build_templates/pmon.service.j2 b/files/build_templates/pmon.service.j2
index de4e62df5e10..33f3173b4887 100644
--- a/files/build_templates/pmon.service.j2
+++ b/files/build_templates/pmon.service.j2
@@ -7,7 +7,7 @@ Before=ntp-config.service
 [Service]
 User={{ sonicadmin_user }}
 ExecStartPre=/usr/bin/{{docker_container_name}}.sh start
-ExecStart=/usr/bin/{{docker_container_name}}.sh attach
+ExecStart=/usr/bin/{{docker_container_name}}.sh wait
 ExecStop=/usr/bin/{{docker_container_name}}.sh stop
 
 [Install]
diff --git a/files/build_templates/radv.service.j2 b/files/build_templates/radv.service.j2
index 065f01906076..8cda2fdd0afb 100644
--- a/files/build_templates/radv.service.j2
+++ b/files/build_templates/radv.service.j2
@@ -7,7 +7,7 @@ Before=ntp-config.service
 [Service]
 User={{ sonicadmin_user }}
 ExecStartPre=/usr/bin/{{ docker_container_name }}.sh start
-ExecStart=/usr/bin/{{ docker_container_name }}.sh attach
+ExecStart=/usr/bin/{{ docker_container_name }}.sh wait
 ExecStop=/usr/bin/{{ docker_container_name }}.sh stop
 
 [Install]
diff --git a/files/build_templates/snmp.service.j2 b/files/build_templates/snmp.service.j2
index f344f2e805ea..b00c2107702c 100644
--- a/files/build_templates/snmp.service.j2
+++ b/files/build_templates/snmp.service.j2
@@ -6,5 +6,5 @@ Before=ntp-config.service
 
 [Service]
 ExecStartPre=/usr/bin/{{docker_container_name}}.sh start
-ExecStart=/usr/bin/{{docker_container_name}}.sh attach
+ExecStart=/usr/bin/{{docker_container_name}}.sh wait
 ExecStop=/usr/bin/{{docker_container_name}}.sh stop
diff --git a/files/build_templates/swss.service.j2 b/files/build_templates/swss.service.j2
index 36858b30b60a..abf57b7bc89f 100644
--- a/files/build_templates/swss.service.j2
+++ b/files/build_templates/swss.service.j2
@@ -14,7 +14,7 @@ Before=ntp-config.service
 User=root
 Environment=sonic_asic_platform={{ sonic_asic_platform }}
 ExecStartPre=/usr/local/bin/swss.sh start
-ExecStart=/usr/local/bin/swss.sh attach
+ExecStart=/usr/local/bin/swss.sh wait
 ExecStop=/usr/local/bin/swss.sh stop
 
 [Install]
diff --git a/files/build_templates/syncd.service.j2 b/files/build_templates/syncd.service.j2
index ddb51232426b..0c61b3193877 100644
--- a/files/build_templates/syncd.service.j2
+++ b/files/build_templates/syncd.service.j2
@@ -20,7 +20,7 @@ Before=ntp-config.service
 User=root
 Environment=sonic_asic_platform={{ sonic_asic_platform }}
 ExecStartPre=/usr/local/bin/syncd.sh start
-ExecStart=/usr/local/bin/syncd.sh attach
+ExecStart=/usr/local/bin/syncd.sh wait
 ExecStop=/usr/local/bin/syncd.sh stop
 
 [Install]
diff --git a/files/build_templates/teamd.service.j2 b/files/build_templates/teamd.service.j2
index 5cd36c6748b9..792b8247119f 100644
--- a/files/build_templates/teamd.service.j2
+++ b/files/build_templates/teamd.service.j2
@@ -7,7 +7,7 @@ Before=ntp-config.service
 [Service]
 User={{ sonicadmin_user }}
 ExecStartPre=/usr/bin/{{docker_container_name}}.sh start
-ExecStart=/usr/bin/{{docker_container_name}}.sh attach
+ExecStart=/usr/bin/{{docker_container_name}}.sh wait
 ExecStop=/usr/bin/{{docker_container_name}}.sh stop
 
 [Install]
diff --git a/files/build_templates/telemetry.service.j2 b/files/build_templates/telemetry.service.j2
index 132924b04193..d6f70a13206e 100644
--- a/files/build_templates/telemetry.service.j2
+++ b/files/build_templates/telemetry.service.j2
@@ -7,7 +7,7 @@ Before=ntp-config.service
 [Service]
 User={{ sonicadmin_user }}
 ExecStartPre=/usr/bin/{{docker_container_name}}.sh start
-ExecStart=/usr/bin/{{docker_container_name}}.sh attach
+ExecStart=/usr/bin/{{docker_container_name}}.sh wait
 ExecStop=/usr/bin/{{docker_container_name}}.sh stop
 
 [Install]
diff --git a/files/scripts/swss.sh b/files/scripts/swss.sh
index 107a2e81b4b2..780051def9e6 100755
--- a/files/scripts/swss.sh
+++ b/files/scripts/swss.sh
@@ -115,9 +115,9 @@ start() {
     unlock_service_state_change
 }
 
-attach() {
+wait() {
     startPeerService
-    /usr/bin/${SERVICE}.sh attach
+    /usr/bin/${SERVICE}.sh wait
 }
 
 stop() {
@@ -142,11 +142,11 @@ stop() {
 }
 
 case "$1" in
-    start|attach|stop)
+    start|wait|stop)
         $1
         ;;
     *)
-        echo "Usage: $0 {start|attach|stop}"
+        echo "Usage: $0 {start|wait|stop}"
         exit 1
         ;;
 esac
diff --git a/files/scripts/syncd.sh b/files/scripts/syncd.sh
index 7e5ce780e072..0a2b9aefd0df 100755
--- a/files/scripts/syncd.sh
+++ b/files/scripts/syncd.sh
@@ -119,8 +119,8 @@ start() {
     unlock_service_state_change
 }
 
-attach() {
-    /usr/bin/${SERVICE}.sh attach
+wait() {
+    /usr/bin/${SERVICE}.sh wait
 }
 
 stop() {
@@ -168,11 +168,11 @@ stop() {
 }
 
 case "$1" in
-    start|attach|stop)
+    start|wait|stop)
         $1
         ;;
     *)
-        echo "Usage: $0 {start|attach|stop}"
+        echo "Usage: $0 {start|wait|stop}"
         exit 1
         ;;
 esac