<div dir="ltr"><div class="gmail_extra"><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font face="arial, helvetica, sans-serif">Hi,</font></pre></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap">></span><i style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap"> But once in the docker build, PBR complains about not getting access to<br></i><span style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap">></span><i style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap"> upstream GIT. And it is true: the git tree (with dependencies) has been<br></i><span style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap">></span><i style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap"> checkedout by gitlab and the docker build (running in a gitlab runner) does<br></i><span style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap">></span><i style="color:rgb(83,83,83);font-family:"andale mono","lucida console",monospace;font-size:12.0012px;white-space:pre-wrap"> not give access to the repository, because it is not required.</i><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:12.0012px;white-space:pre-wrap"><br></span></font>Is the issue here that submodules somehow cause PBR to need access to upstream git repositories or is the installation process not including the .git directories for repositories and only including their checked out state? PBR should work with local git repository checkouts without needing to talk to any remotes/upstreams.</blockquote><div class="gmail_extra"><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font face="arial, helvetica, sans-serif">I am pretty sure there is the .git in the submodules, and indeed it only occurs on the submodules, not on the main module that also uses pbr. Maybe a side effect of using git submodules with pbr. But, if you tell me this is not normal for PBR to try to connect to upstream git if all the .git directory (of the submodules) are here, i'll try to debug it and see what's wrong.</font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font face="andale mono, lucida console, monospace">
</font><blockquote class="gmail_quote" style="font-family:"andale mono","lucida console",monospace;margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">><i> </i><i>The other solution might be to allow pbr to NOT have git upstream access
</i>><i> in _get_version_from_git.
</i>
At least without submodules this shouldn't be required. </blockquote><div style="font-family:"andale mono","lucida console",monospace"><br></div><div><font face="arial, helvetica, sans-serif">Yes, on the main module, PBR is happy. Only on the submodules</font></div><div style="font-family:"andale mono","lucida console",monospace"> </div><blockquote class="gmail_quote" style="font-family:"andale mono","lucida console",monospace;margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Without submodules you only need access to the local git repository and not any remotes. I think this behavior is desireable and if we can have that behavior with submodules as well that is probably a good improvement to make.</blockquote><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font face="arial, helvetica, sans-serif">Roger, I'll debug it.</font></pre><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span style="color:rgb(83,83,83);font-size:12.0012px;white-space:pre-wrap;font-family:"andale mono","lucida console",monospace">> </span><i style="color:rgb(83,83,83);font-size:12.0012px;white-space:pre-wrap;font-family:"andale mono","lucida console",monospace">I will open another thread on this ML if<br></i><font face="andale mono, lucida console, monospace" style="color:rgb(83,83,83);font-size:12.0012px;white-space:pre-wrap">></font><i style="color:rgb(83,83,83);font-size:12.0012px;white-space:pre-wrap;font-family:"andale mono","lucida console",monospace"> you agree, when my unit tests will pass, but i will not be able to do it<br></i><font face="andale mono, lucida console, monospace" style="color:rgb(83,83,83);font-size:12.0012px;white-space:pre-wrap">></font><i style="color:rgb(83,83,83);font-size:12.0012px;white-space:pre-wrap;font-family:"andale mono","lucida console",monospace"> until net year.</i><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:12.0012px;white-space:pre-wrap"><br></span></font><font face="andale mono, lucida console, monospace">Adding pipfile support seems like an excellent thing to add considering people are trying to use pipfile and pbr together now.</font></blockquote></pre></div><div class="gmail_extra"><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;vertical-align:baseline;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5"><font face="arial, helvetica, sans-serif"><font style="color:rgb(83,83,83);font-size:12.0012px;white-space:pre-wrap">Great, I am happy to ear it. Can you give an eye to my first draft? </font><font color="#535353"><span style="font-size:12.0012px;white-space:pre-wrap"><a href="https://review.openstack.org/#/c/524436/">https://review.openstack.org/#/c/524436/</a></span></font></font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font face="arial, helvetica, sans-serif">The main issue i can see it that I had to copy toml parser and pipfile parser from external projects, I don't know if there will be any policy against it in PBR (or licence compatibility issue, ...).</font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font face="andale mono, lucida console, monospace">
</font><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font face="andale mono, lucida console, monospace">></font><i style="font-family:"andale mono","lucida console",monospace"> Also, I maintain a fork a pbr "pbrlgs" (
</i><font face="andale mono, lucida console, monospace">></font><i style="font-family:"andale mono","lucida console",monospace"> <a href="https://github.com/Stibbons/pbr/tree/pbrlgs" style="margin:0px;padding:0px;border:0px;font-weight:inherit;font-style:inherit;font-size:12.0012px;font-family:inherit;vertical-align:baseline;color:rgb(188,21,24);text-decoration-line:none">https://github.com/Stibbons/pbr/tree/pbrlgs</a>) that makes PBR happy with git
</i><font face="andale mono, lucida console, monospace">></font><i style="font-family:"andale mono","lucida console",monospace"> submodules.
</i><font face="andale mono, lucida console, monospace">
</font><font face="andale mono, lucida console, monospace">Are these improvements something that should be upstreamed as well?</font></blockquote><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font style="" face="arial, helvetica, sans-serif">It is basically HEAD of PBR + my pipfile patch + my git submodules environment variables hack. Of course it is not intended to be kept alive if PBR merges my contributions. But we use it in production now:</font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font style="" face="arial, helvetica, sans-serif">- pipfile patch allows PBR to install the dependencies of a dependency that uses a Pipfile</font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font style="" face="arial, helvetica, sans-serif">- env var patch makes pbr happy with submodules in our docker environment.</font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font style="" face="arial, helvetica, sans-serif">Thanks</font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font style="" face="arial, helvetica, sans-serif">Best Regards,</font></pre><pre style="margin-top:1.5em;margin-bottom:1.5em;padding:0px;border:0px;font-size:12.0012px;vertical-align:baseline;white-space:pre-wrap;font-variant-numeric:normal;font-variant-east-asian:normal;font-stretch:normal;line-height:1.5;color:rgb(83,83,83)"><font style="" face="arial, helvetica, sans-serif">Gaetan</font></pre></pre></div></div>