<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 4:</p><p style="white-space: pre-wrap; word-wrap: break-word;">(5 comments)</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think in general it would be better if we could have a textual config file containing the mappings of redmine project name vs. git repository.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Would we use this text file for something else? I could create an extra text file and put the mapping there, but I'm not sure if we should add the logic to fetch and interpret that if nothing else is using it. Isn't the mapping inside the jenkins job very readable and easy to update if needed?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Note that also this is for the special cases, it seems that newer projects are created with the same gerrit project name, redmine project name and redmine repository name. For OsmoHLR for example, no mapping entry is needed: https://osmocom.org/projects/osmo-hlr/repository/osmo-hlr</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Or, obviously, any method that doesn't require manually kept configuration which will soon be outdated as somebody forgets about it...</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The jenkins job will fail if the URL downloaded at the end returns a 404. So if the mapping is wrong somewhere, or we add a new project that doesn't have the standard naming scheme, I will get a mail and can fix it. I think updating this mapping is low maintenance, and errors will get noticed quickly.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As mentioned in the commit message, I have considered other approaches too but they seemed like more effort. It is not like there's one redmine api call that lists all repositories in all projects.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ci/+/26423">View Change</a></p><p>5 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ci/+/26423/4/jobs/redmine-fetch.yml">File jobs/redmine-fetch.yml:</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/osmo-ci/+/26423/4/jobs/redmine-fetch.yml@45">Patch Set #4, Line 45:</a> <code style="font-family:monospace,monospace">libusrp</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">libusrp could be secondary repo of osmo-trx?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think I have all needed permissions to add them. For some redmine projects, I can edit the listed repositories. But they require a local path, and - following the existing naming scheme - /usr/local/git/repositories/libusrp.git does apparently not exist. When adding the repository as such, it gives a 404. Can you create those?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ci/+/26423/4/jobs/redmine-fetch.yml@45">Patch Set #4, Line 45:</a> <code style="font-family:monospace,monospace">libgtpnl</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">should be added as secondary repo to osmo-ggsn?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same, /usr/local/git/repositories/libgtpnl.git does not exist</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ci/+/26423/4/jobs/redmine-fetch.yml@51">Patch Set #4, Line 51:</a> <code style="font-family:monospace,monospace">osmo-iuh</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">this is certainly a but, the repository should be added to redmine</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I've missed that it's here, patch updated: https://osmocom.org/projects/osmohnbgw/repository/osmo-iuh/</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ci/+/26423/4/jobs/redmine-fetch.yml@54">Patch Set #4, Line 54:</a> <code style="font-family:monospace,monospace">osmocom-lcs</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">osmocom-lcs can be added as secondary repo to the osmo-bsc redmine project, in case there is no smlc […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Added successfully to osmo-smlc: https://osmocom.org/projects/osmo-smlc/repository/osmocom-lcs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ci/+/26423/4/jobs/redmine-fetch.yml@54">Patch Set #4, Line 54:</a> <code style="font-family:monospace,monospace">osmo-upf</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">a related redmine project with repository should be created</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">There's a redmine project: https://osmocom.org/projects/osmo-upf</p><p style="white-space: pre-wrap; word-wrap: break-word;">But I don't have permissions to modify the repositories.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ci/+/26423">change 26423</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/+/26423"/><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: Ia7c6c8426e5ca0de04378cedd0360c368e9d5027 </div>
<div style="display:none"> Gerrit-Change-Number: 26423 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Nov 2021 08:30:34 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>