<p style="white-space: pre-wrap; word-wrap: break-word;">I see this is moving from make-style dependency tracking to naming each required image explicitly. It is also adding image rebuilding to the jenkins.sh directly. As you will notice in the comments, it's not so easy for me to let go of my current use pattern of 'make ttcn3-bsc-test' once, then invoke jenkins.sh as many times I like, but yeah. I think I can deal with it if any alternative would mean a lot of effort for the 'latest' tests ... ?</p><p><a href="https://gerrit.osmocom.org/11366">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11366/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11366/1//COMMIT_MSG@7">Patch Set #1, Line 7:</a> <code style="font-family:monospace,monospace">jenkins.sh: new IMAGE_SUFFIX environment variable</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">actually, the most profound change here is that you add calls to docker_images_require().</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11366/1/m3ua-test/jenkins.sh">File m3ua-test/jenkins.sh:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11366/1/m3ua-test/jenkins.sh@4">Patch Set #1, Line 4:</a> <code style="font-family:monospace,monospace">[ -z "$IMAGE_SUFFIX" ] && IMAGE_SUFFIX="master"</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I still prefer</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  if [ -z xyz ]; then abc; fi</pre><p style="white-space: pre-wrap; word-wrap: break-word;">we are C hackers after all. Or if you want to be scripty, rather go for the if-unset shell pattern:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  FOO="${FOO:-bar}"</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11366/1/m3ua-test/jenkins.sh@6">Patch Set #1, Line 6:</a> <code style="font-family:monospace,monospace">        "debian-jessie-build" \</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">so it is not enough to issue the osmo-stp-foo image below, to also rebuild the images that it depends on? That was the idea of the top-level makefile: build that image *after* all of its dependencies, implicitly.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11366/1/m3ua-test/jenkins.sh@7">Patch Set #1, Line 7:</a> <code style="font-family:monospace,monospace">     "osmo-stp-$IMAGE_SUFFIX" \</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">with IMAGE_SUFFIX, I think of ".img". How about FEED_NAME</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11366">change 11366</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/11366"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: docker-playground </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Idbb708ab16cb71bab5069127945b63388222369e </div>
<div style="display:none"> Gerrit-Change-Number: 11366 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 16 Oct 2018 11:36:35 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>