<div dir="ltr"><span style="font-size:12.8000001907349px">Interesting, I was not aware of puppetlabs-inifile.</span><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">Anyway, I think Jeremy mentioned to me that the creation of puppet-pip class was to manage the python3-python2 differences for</div><div style="font-size:12.8000001907349px">pip paths (Jeremy can you chime in?)</div><div style="font-size:12.8000001907349px">The manage pip.conf functionality was added afterwards (I think it was Yolanda who added that), but the main intention</div><div style="font-size:12.8000001907349px">for this class was to install pip (which doesn't work btw).</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">Regards</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-08-12 14:40 GMT+02:00 Paul Belanger <span dir="ltr"><<a href="mailto:pabelanger@redhat.com" target="_blank">pabelanger@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Aug 11, 2015 at 10:59:35PM +0200, Ricardo Carrillo Cruz wrote:<br>
> We have followed this topic on IRC channel, and it seems the consensus so<br>
> far<br>
> is to:<br>
><br>
> 1. Deprecate python-pip from infra modules<br>
> 2. Create another module python-pip_settings that just manages pip.conf<br>
> (this will typically used for downstream mainly, for proxies whatnot).<br>
> 3. Put a note on modules and documentation stating that in order to use<br>
> infra modules install_puppet.sh should be run or pip installed prior to<br>
> applying the puppet modules<br>
><br>
> If there are no objections with the above, I'd start doing refactoring.<br>
><br>
</span>I have to admit, I didn't really follow the discussion to much yesterday.<br>
However, what was the reason again for not using an existing puppet-python[1]<br>
module to do this?<br>
<br>
Do we really need puppet-pip_setting module or can we just not use the<br>
puppet-inifile[2] module to manage pip.conf?<br>
<br>
Be kind, I haven't had coffee yet.<br>
<br>
[1] <a href="https://github.com/stankevich/puppet-python" rel="noreferrer" target="_blank">https://github.com/stankevich/puppet-python</a><br>
[2] <a href="https://github.com/puppetlabs/puppetlabs-inifile" rel="noreferrer" target="_blank">https://github.com/puppetlabs/puppetlabs-inifile</a><br>
<div class="HOEnZb"><div class="h5"><br>
> Regards<br>
><br>
> 2015-08-11 12:36 GMT+02:00 Ricardo Carrillo Cruz <<br>
> <a href="mailto:ricardo.carrillo.cruz@gmail.com">ricardo.carrillo.cruz@gmail.com</a>>:<br>
><br>
> > I will add this to the infra meeting agenda, since I can't attend due to<br>
> > TZ differences.<br>
> > Will put some context on what options were discussed on IRC yesterday.<br>
> ><br>
> > Thanks<br>
> ><br>
> > 2015-08-10 13:56 GMT+02:00 Jeremy Stanley <<a href="mailto:fungi@yuggoth.org">fungi@yuggoth.org</a>>:<br>
> ><br>
> >> On 2015-08-10 10:54:28 +0200 (+0200), Ricardo Carrillo Cruz wrote:<br>
> >> [...]<br>
> >> > I suggest we get rid of puppet-pip from all service puppet modules<br>
> >> > (like puppet-zuul) and just use puppet-python for sanity (and also<br>
> >> > adding the manage pip.conf functionality to upstream stankevich<br>
> >> > puppet-python).<br>
> >> [...]<br>
> >><br>
> >> It would need to be made optional. On our systems, we want to have a<br>
> >> Puppet package provider for pip but we don't want Puppet managing<br>
> >> installation of pip because we do that through some fairly<br>
> >> convoluted logic to forcibly purge any package-managed pip<br>
> >> installation and run a cached copy of the get-pip.py script. See the<br>
> >> setup_pip function in the install_puppet.sh script of<br>
> >> openstack-infra/system-config for details.<br>
> >> --<br>
> >> Jeremy Stanley<br>
> >><br>
> >> _______________________________________________<br>
> >> OpenStack-Infra mailing list<br>
> >> <a href="mailto:OpenStack-Infra@lists.openstack.org">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>
<br>
> _______________________________________________<br>
> OpenStack-Infra mailing list<br>
> <a href="mailto:OpenStack-Infra@lists.openstack.org">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>
</div></div></blockquote></div><br></div>