<div dir="ltr">Dear all,<div><br></div><div>Actually openstack-infra/system-config is using the openstack-infra/puppet-vcsrepo module for handling git repositories.</div><div><br></div><div>With Jeremy and Clark we found an issue related this vcsrepo module which is a fork of original puppetlabs-vcsrepo, but not follow the upstream. I made a little poc to demonstrate the issue, which is related to the refresh event invocation of github repositories.</div><div><br></div><div><a href="https://github.com/mkissam/puppet-vcsrepo-refresh-poc/blob/master/vcsrepo-refresh-poc.md">https://github.com/mkissam/puppet-vcsrepo-refresh-poc/blob/master/vcsrepo-refresh-poc.md</a><br></div><div><br></div><div>The story here, if we checking out a specific commit ref, every puppet run will trigger a refresh event, even the repository not changed when we use the ensure => latest class parameter.</div><div><br></div><div>The upstream have a patch that resolve this specific issue:</div><div><div>(MODULES-660) Correct detached HEAD on latest</div><div><a href="https://github.com/puppetlabs/puppetlabs-vcsrepo/commit/6624f40651f44e184878a9fbb862bda886d899e8">https://github.com/puppetlabs/puppetlabs-vcsrepo/commit/6624f40651f44e184878a9fbb862bda886d899e8</a></div></div><div><br></div><div>I see to options here: backport the patch from upstream or as an alternative, drop openstack-infra/puppet-vcsrepo and use upstream one.</div><div><br></div><div>The question here, what was the exact reason of forking vcsepo? Can we go with backport, or freely upgrade the system-config to consume upstream vcsrepo?</div><div><br></div><div>Brgds,</div><div>  Marton Kiss</div></div>