<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">No merge yet since I think some stuff is still being prepared. Better mark is as WIP meanwhile.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It's pretty much finished already, I'll just switch `docker run` parameters, so we don't need `--entrypoint`.</p><p><a href="https://gerrit.osmocom.org/10862">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/10862/6/debian-repo-install-test/jenkins.sh">File debian-repo-install-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/10862/6/debian-repo-install-test/jenkins.sh@14">Patch Set #6, Line 14:</a> <code style="font-family:monospace,monospace">                --entrypoint "/testdata/repo-install-test.sh" \</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">interesting, --entrypoint; so far in the other jenkins. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I haven't really used docker much (besides in gitlab-ci configs, where you don't specify a `docker run` command line), so I wasn't aware that you can put the command as last argument. I figured `--entrypoint` would be the way to do it. </p><p style="white-space: pre-wrap; word-wrap: break-word;">But now that you say it, placing it at the end is shorter and it even allows specifying arguments to the commands. This is the better way then, I'll push another patch that changes this.</p><p style="white-space: pre-wrap; word-wrap: break-word;">(I guess the difference is, that with the entrypoint you could override the default shell.)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/10862/6/debian-repo-install-test/testdata/repo-install-test.sh">File debian-repo-install-test/testdata/repo-install-test.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/10862/6/debian-repo-install-test/testdata/repo-install-test.sh@49">Patch Set #6, Line 49:</a> <code style="font-family:monospace,monospace">     osmo-bsc --version</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">it's not really the task at hand, but in the past we also had the problem that some packages would g […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">created https://osmocom.org/issues/3555 for that</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10862">change 10862</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/10862"/><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: I0f0a8b4f0ccd706cd1af98fe25accd6fd410e007 </div>
<div style="display:none"> Gerrit-Change-Number: 10862 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 19 Sep 2018 14:03:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>