<div dir="ltr">The git puppet module requires some additional work, not looks stable for me. It have RedHat support only, and it won't work for askbot, because those instances based on Ubuntu 14.04LTS actually. I guess we need a stable solution within a reasonable timeframe. Git puppet should evolve, but I see no new commits in the last 6months.<div><br></div><div>Brgds,</div><div>  Marton</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 15, 2015 at 9:05 PM Paul Belanger <<a href="mailto:pabelanger@redhat.com">pabelanger@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Sep 15, 2015 at 06:09:45PM +0000, Marton Kiss wrote:<br>
> Dear all,<br>
><br>
> Actually openstack-infra/system-config is using the<br>
> openstack-infra/puppet-vcsrepo module for handling git repositories.<br>
><br>
> With Jeremy and Clark we found an issue related this vcsrepo module which<br>
> is a fork of original puppetlabs-vcsrepo, but not follow the upstream. I<br>
> made a little poc to demonstrate the issue, which is related to the refresh<br>
> event invocation of github repositories.<br>
><br>
> <a href="https://github.com/mkissam/puppet-vcsrepo-refresh-poc/blob/master/vcsrepo-refresh-poc.md" rel="noreferrer" target="_blank">https://github.com/mkissam/puppet-vcsrepo-refresh-poc/blob/master/vcsrepo-refresh-poc.md</a><br>
><br>
> The story here, if we checking out a specific commit ref, every puppet run<br>
> will trigger a refresh event, even the repository not changed when we use<br>
> the ensure => latest class parameter.<br>
><br>
> The upstream have a patch that resolve this specific issue:<br>
> (MODULES-660) Correct detached HEAD on latest<br>
> <a href="https://github.com/puppetlabs/puppetlabs-vcsrepo/commit/6624f40651f44e184878a9fbb862bda886d899e8" rel="noreferrer" target="_blank">https://github.com/puppetlabs/puppetlabs-vcsrepo/commit/6624f40651f44e184878a9fbb862bda886d899e8</a><br>
><br>
> I see to options here: backport the patch from upstream or as an<br>
> alternative, drop openstack-infra/puppet-vcsrepo and use upstream one.<br>
><br>
> The question here, what was the exact reason of forking vcsepo? Can we go<br>
> with backport, or freely upgrade the system-config to consume upstream<br>
> vcsrepo?<br>
><br>
I think a 3rd option is to remove the dependency on vcsrepo all together.  IIRC,<br>
nibalizer has started the ball rolling on the migration.  A new puppet module,<br>
puppet-git, has been added into our modules for the purpose of replacing<br>
vcsrepo.<br>
<br>
I am unsure of the progress, but maybe this issue can help move it along.<br>
<br>
[1] <a href="https://github.com/nanliu/puppet-git" rel="noreferrer" target="_blank">https://github.com/nanliu/puppet-git</a><br>
> Brgds,<br>
>   Marton Kiss<br>
<br>
> _______________________________________________<br>
> OpenStack-Infra mailing list<br>
> <a href="mailto:OpenStack-Infra@lists.openstack.org" target="_blank">OpenStack-Infra@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra</a><br>
<br>
<br>
_______________________________________________<br>
OpenStack-Infra mailing list<br>
<a href="mailto:OpenStack-Infra@lists.openstack.org" target="_blank">OpenStack-Infra@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra</a><br>
</blockquote></div>