
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Sep 18, 2014, at 7:43 AM, Thomas Goirand <<a href="mailto:zigo@debian.org" class="">zigo@debian.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">On 09/18/2014 04:01 PM, Flavio Percoco wrote:<br class=""><blockquote type="cite" class="">After having gone through the whole thread and read all the concerns,<br class="">problems and reasonings, I think we should stick to requests as-is for<br class="">now and deal with this particular issue.<br class=""><br class="">Regardless of the vendorized urllib3 package, I believe requests is a<br class="">good library with an easy-to-consume API and it has solved several<br class="">problems throughout OpenStack. Not to mention it's also helpped with<br class="">making OpenStack more consistent. We've put a lot of effort to get to<br class="">this point and I don't think we should revert all that because of the<br class="">vendorized `urllib3` package.<br class=""><br class="">Cheers,<br class="">Flavio<br class=""></blockquote><br class="">I, at least, haven't suggested that we stop using requests. Just that we<br class="">don't internally use any of the requests.packages.* stuff.<br class=""><br class="">The rest of the debate about the good or bad things with vendorizing,<br class="">even if it is my view that it's a really bad thing, is IMO not<br class="">interesting for the OpenStack project.</div></blockquote><br class=""></div><div>I don’t believe that’s a good idea. If you’re wanting to use urllib3 in order</div><div>to interact with requests than you *should* be using requests.packages.urllib3,</div><div>to do anything else risks having two different versions of urllib3 primitives at</div><div>play in one subsystem.</div><div><br class=""></div><div>It’s not even completely possible in the case that prompted this thread originally</div><div>since the reason requests.packages.urllib3 was being imported from was so that</div><div>there could be an is instance() check against one of the classes. If that wasn’t</div><div>imported from requests.packages.urllib3 but instead from just urllib3 than the</div><div>isinstance check would always fail.</div><div><br class=""></div><div class="">

<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">---</div><div class="">Donald Stufft</div><div class="">PGP: 7C6B 7C5D 5E2B 6356 A926 F04F 6E3C BCE9 3372 DCFA</div></div></div>

</div>

<br class=""></body></html>