<p>osmith <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ci/+/18512">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  osmith: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">repo-install-test: use sh -ex<br><br>Make it consistent with run-inside-docker.sh by also using -e and -x.<br>This makes development easier.<br><br>Change-Id: I733164829bf076fb42df3fde725f3e330f8abecc<br>---<br>M scripts/repo-install-test.sh<br>1 file changed, 6 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/scripts/repo-install-test.sh b/scripts/repo-install-test.sh</span><br><span>index d3ce8d6..0a19b04 100755</span><br><span>--- a/scripts/repo-install-test.sh</span><br><span>+++ b/scripts/repo-install-test.sh</span><br><span>@@ -1,4 +1,4 @@</span><br><span style="color: hsl(0, 100%, 40%);">-#!/bin/sh</span><br><span style="color: hsl(120, 100%, 40%);">+#!/bin/sh -ex</span><br><span> . "$(dirname "$0")/common.sh"</span><br><span> docker_images_require "debian-repo-install-test"</span><br><span> </span><br><span>@@ -46,12 +46,14 @@</span><br><span> check_if_systemd_is_running</span><br><span> </span><br><span> # Run the test script</span><br><span style="color: hsl(0, 100%, 40%);">-docker exec "$CONTAINER" /repo-install-test/run-inside-docker.sh</span><br><span style="color: hsl(0, 100%, 40%);">-ret="$?"</span><br><span style="color: hsl(120, 100%, 40%);">+ret=0</span><br><span style="color: hsl(120, 100%, 40%);">+if ! docker exec "$CONTAINER" /repo-install-test/run-inside-docker.sh; then</span><br><span style="color: hsl(120, 100%, 40%);">+       ret=1</span><br><span style="color: hsl(120, 100%, 40%);">+fi</span><br><span> </span><br><span> # Interactive shell</span><br><span> if [ -n "$INTERACTIVE" ]; then</span><br><span style="color: hsl(0, 100%, 40%);">-        docker exec -it "$CONTAINER" bash</span><br><span style="color: hsl(120, 100%, 40%);">+   docker exec -it "$CONTAINER" bash || true</span><br><span> fi</span><br><span> </span><br><span> docker container kill "$CONTAINER"</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ci/+/18512">change 18512</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-ci/+/18512"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ci </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I733164829bf076fb42df3fde725f3e330f8abecc </div>
<div style="display:none"> Gerrit-Change-Number: 18512 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>