Thursday, 2017-04-20

*** rods has quit IRC01:00
*** gouthamr has joined #openstack-shade02:45
*** gkadam has joined #openstack-shade02:53
*** Matias has quit IRC03:30
*** Matias has joined #openstack-shade03:32
*** gouthamr has quit IRC05:54
*** jamielennox has quit IRC07:28
*** gkadam has quit IRC07:29
*** gkadam has joined #openstack-shade07:29
*** jamielennox has joined #openstack-shade07:42
*** ioggstream has joined #openstack-shade07:55
*** gkadam is now known as gkadam-afk09:26
*** cdent has joined #openstack-shade09:30
*** rods has joined #openstack-shade09:44
*** cdent_ has joined #openstack-shade09:57
*** cdent has quit IRC09:58
*** cdent_ is now known as cdent09:58
*** cdent_ has joined #openstack-shade10:38
*** cdent has quit IRC10:40
*** cdent_ is now known as cdent10:40
*** cdent has quit IRC11:31
*** openstackgerrit has quit IRC11:32
*** yfried has joined #openstack-shade11:44
yfriedmordred: ping, shade deps just killed us again :(11:44
yfrieddhellmann: ^11:51
*** rods has quit IRC12:05
*** rods has joined #openstack-shade12:05
*** rods has quit IRC12:10
mordredyfried: we haven't cut a release yet12:28
mordredyfried: what's the issue this time?12:28
mordredyfried: (I was actually just about to get a release made)12:28
yfriedmordred: https://paste.fedoraproject.org/paste/LuAMTicTcCpJlGrZOPgE3F5M1UNdIGYhyRLivL9gydE=12:29
mordredyfried: k. https://review.openstack.org/458487 submitted for release12:36
yfriedmordred: were did you fix Bable?12:39
mordredI don't depend on babel - it's in a transitive depend somewhere12:39
yfriedmordred: so it will break, won't it?12:40
mordredit might break, yes - the pbr version is the one I can fix - we'll need to investigate more to see which libraries have incompatible babel versions12:41
yfriedmordred: why is it so hard to depend on shade?12:43
yfriedmordred: does Neutron hit the same bugs we do daily?12:43
mordredit's because we are currently depending on the python client libs. everything about this will go away as soon as we're done not depending on them12:44
mordredwhich is why we're spending basically all day every day doing nothing but removing out dependency on the shit-tastic python client libs12:45
mordredin any case, this issue was caused because osc-lib released and it has a different exclusion on babel than novaclient does, but novaclient got depended on first so was the one to cause babel to be installed12:46
mordredthis is also hard specifically because of the use of pkg_resources which is _extra_ "helpful" in complaining about when you don't have the 'right' version of something installed at runtime12:47
*** gkadam-afk has quit IRC12:51
yfriedmordred: please add pip check to some gate12:51
ShrewsThat wouldn't be useful. We can't control the versions of libraries used the client libs. The only solution is to get rid of the client libs.12:57
mordredyup13:02
mordrednext up - ironicclient13:03
mordredunfortunately, TheJulia did a REALLY good job with it and there are many test13:03
mordredthanks TheJulia13:03
mordred:)13:03
ShrewsTheJulia iz teh awwsum13:04
*** rods has joined #openstack-shade13:04
*** yfried has quit IRC13:17
*** yfried has joined #openstack-shade13:17
*** ioggstream has quit IRC13:24
*** ioggstream has joined #openstack-shade13:34
*** yfried has quit IRC13:34
*** rods has quit IRC13:58
*** yfried has joined #openstack-shade14:13
*** rods has joined #openstack-shade14:14
*** gouthamr has joined #openstack-shade14:18
*** rods has quit IRC14:19
*** rods has joined #openstack-shade14:21
dtroyer_zzmordred: /me is trying to sort out why osc-lib even has Babel in requirements.txt14:30
mordreddtroyer_zz: transitive depends maybe?14:45
dtroyer_zzIt is, from osli_i18n, so osc-lib shouldn't be include it, eh?14:46
mordreddtroyer_zz: if I make a fresh venv and pip install shade into it, then try to load an entrypoint with pkg_resources that touches shade, I get this:14:47
mordredpkg_resources.ContextualVersionConflict: (Babel 2.4.0 (/home/mordred/foo/lib/python2.7/site-packages), Requirement.parse('Babel!=2.4.0,>=2.3.4'), set(['osc-lib']))14:47
mordreddtroyer_zz: although, to be fair, there are two other conflicts that have to be fixed before that surfaces14:47
dtroyer_zzmordred: osc-lib matches osli.i18n for babel in g-r14:48
dtroyer_zzin master14:48
mordreddtroyer_zz: http://paste.openstack.org/show/607356/ is a reproduction walkthrough, should you be interested14:51
mordreddhellmann: ^^14:51
rodsmordred hi, I left a comment on this pr https://review.openstack.org/#/c/458100/4, can you give a look when you have a second?15:10
*** openstackgerrit has joined #openstack-shade15:54
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add documentation of algorithm for version discovery  https://review.openstack.org/45856515:54
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add text about microversions  https://review.openstack.org/45856615:54
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove future document  https://review.openstack.org/45856715:54
mordredrods: responded - great question15:54
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove stray line  https://review.openstack.org/45856915:56
rodsmordred I like the idea of the assert_no_calls method, I'm gonna go with self.assertEqual(0, len(self.adapter.request_history)) for now and implement assert_no_calls in a different pull request15:57
mordredsounds great15:58
mordredrods: also, the strip trailing slash patch is winding its way through the gate now, so should be in soon15:58
rodscool15:59
*** yfried has quit IRC16:00
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove future document  https://review.openstack.org/45856716:11
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add text about microversions  https://review.openstack.org/45856616:11
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add documentation of algorithm for version discovery  https://review.openstack.org/45856516:11
mordredsamueldmq: ^^ I took your PS-3 docstring and expanded it (greatly) into a document16:12
samueldmqmordred: yay! I will take a look at it today for sure!16:59
openstackgerritMerged openstack-infra/shade master: Remove extra unneeded API calls  https://review.openstack.org/45777517:12
openstackgerritMerged openstack-infra/shade master: Clarify some variable names in glance discovery  https://review.openstack.org/45805517:13
*** ioggstream has quit IRC17:20
*** cdent has joined #openstack-shade17:30
*** yfried has joined #openstack-shade18:04
*** cdent has quit IRC18:21
*** yfried has quit IRC18:42
*** yfried has joined #openstack-shade18:46
*** yfried has quit IRC18:51
openstackgerritMerged openstack-infra/shade master: Strip trailing slashes in test helper method  https://review.openstack.org/45813919:05
mordredrods: ^^ there ya go19:05
rodsmordred great, I'll fix my commit in a minute19:06
openstackgerritRosario Di Somma proposed openstack-infra/shade master: Use requests-mock for all the test_attach_volume_* tests  https://review.openstack.org/45810019:12
*** yfried has joined #openstack-shade19:27
rodsmordred ^^ should be ready to go :)20:01
*** rods has quit IRC20:03
*** rods has joined #openstack-shade20:05
*** yfried has quit IRC20:20
*** yolanda has quit IRC20:35
*** cdent has joined #openstack-shade21:03
*** cdent has quit IRC21:50
*** cdent has joined #openstack-shade21:54
*** gouthamr has quit IRC21:57
*** jamielennox is now known as jamielennox|away22:25
*** jamielennox|away is now known as jamielennox22:29
*** rods has quit IRC22:51
*** cdent has quit IRC22:59
*** rods has joined #openstack-shade23:02
*** gouthamr has joined #openstack-shade23:33

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!