Change in ...osmo-sip-connector[master]: .service: require to ran after network-online.target

2019-09-16 Thread lynxis lazus
lynxis lazus has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533 )

Change subject: .service: require to ran after network-online.target
..

.service: require to ran after network-online.target

Relates: OS#4197
Change-Id: I09bf8139582ef198fdbe664be32b14d8c9c99f1a
---
M contrib/systemd/osmo-sip-connector.service
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved
  keith: Looks good to me, but someone else must approve



diff --git a/contrib/systemd/osmo-sip-connector.service 
b/contrib/systemd/osmo-sip-connector.service
index a08f167..673aa53 100644
--- a/contrib/systemd/osmo-sip-connector.service
+++ b/contrib/systemd/osmo-sip-connector.service
@@ -1,5 +1,7 @@
 [Unit]
 Description=Osmo SIP Connector
+After=network-online.target
+Wants=network-online.target

 [Service]
 Type=simple

--
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: I09bf8139582ef198fdbe664be32b14d8c9c99f1a
Gerrit-Change-Number: 15533
Gerrit-PatchSet: 2
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: keith 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: zecke 
Gerrit-MessageType: merged


Change in ...osmo-sip-connector[master]: .service: require to ran after network-online.target

2019-09-16 Thread keith
keith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533 )

Change subject: .service: require to ran after network-online.target
..


Patch Set 1: Code-Review+1

I've verified this in production.


--
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: I09bf8139582ef198fdbe664be32b14d8c9c99f1a
Gerrit-Change-Number: 15533
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: keith 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: zecke 
Gerrit-Comment-Date: Mon, 16 Sep 2019 14:54:20 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sip-connector[master]: .service: require to ran after network-online.target

2019-09-16 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533 )

Change subject: .service: require to ran after network-online.target
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: I09bf8139582ef198fdbe664be32b14d8c9c99f1a
Gerrit-Change-Number: 15533
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: keith 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: zecke 
Gerrit-Comment-Date: Mon, 16 Sep 2019 08:41:22 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sip-connector[master]: .service: require to ran after network-online.target

2019-09-15 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533 )

Change subject: .service: require to ran after network-online.target
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: I09bf8139582ef198fdbe664be32b14d8c9c99f1a
Gerrit-Change-Number: 15533
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Holger Freyther 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: keith 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Comment-Date: Sun, 15 Sep 2019 23:24:19 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sip-connector[master]: .service: require to ran after network-online.target

2019-09-15 Thread lynxis lazus
lynxis lazus has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533


Change subject: .service: require to ran after network-online.target
..

.service: require to ran after network-online.target

Relates: OS#4197
Change-Id: I09bf8139582ef198fdbe664be32b14d8c9c99f1a
---
M contrib/systemd/osmo-sip-connector.service
1 file changed, 2 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-sip-connector 
refs/changes/33/15533/1

diff --git a/contrib/systemd/osmo-sip-connector.service 
b/contrib/systemd/osmo-sip-connector.service
index a08f167..673aa53 100644
--- a/contrib/systemd/osmo-sip-connector.service
+++ b/contrib/systemd/osmo-sip-connector.service
@@ -1,5 +1,7 @@
 [Unit]
 Description=Osmo SIP Connector
+After=network-online.target
+Wants=network-online.target

 [Service]
 Type=simple

--
To view, visit https://gerrit.osmocom.org/c/osmo-sip-connector/+/15533
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Change-Id: I09bf8139582ef198fdbe664be32b14d8c9c99f1a
Gerrit-Change-Number: 15533
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus 
Gerrit-MessageType: newchange