Monday, 2017-05-15

*** openstack has joined #openstack-shade06:57
*** ioggstream has joined #openstack-shade07:28
*** slaweq has joined #openstack-shade08:54
*** slaweq has quit IRC08:59
*** slaweq has joined #openstack-shade11:31
*** slaweq has quit IRC11:36
*** iogg has joined #openstack-shade11:38
*** ioggstream has quit IRC11:39
*** yfried has quit IRC11:47
*** yfried has joined #openstack-shade12:04
fbacahey mordred! you said we should be able to feed a dictionary to the flavor in os_server but I still couldn't make it work last week with the latest pip version of shade, is this something that only doesn't work for me, any idea?12:09
*** iogg is now known as ioggstream12:22
mordredfbaca: hrm. just waking up - but will poke at it in just a sec12:34
*** gkadam has quit IRC13:04
*** yfried has quit IRC13:14
Shrewsmordred: http://logs.openstack.org/79/460779/12/check/gate-shade-dsvm-functional-legacy-libs-nv/1a62e3a/console.html#_2017-05-14_13_05_42_92363813:37
*** gouthamr has joined #openstack-shade13:38
mordredShrews: yup. STILL working on that job13:39
mordredShrews: I've got a project-config change up that should _finally_ fix it13:40
mordredShrews: https://review.openstack.org/#/c/464335/ fwiw13:40
mordredShrews: tl;dr - the shade change finally gets shade to properly install the git version of the library that zuul puts on the node13:40
mordredShrews: but the job config is causing zuul to put the stable/newton git version on the node13:41
mordredwhich is not the thing we actually are trying to test here13:41
*** iogg has joined #openstack-shade15:26
*** ioggstream has quit IRC15:27
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix broken version discovery endpoints  https://review.openstack.org/46468116:16
openstackgerritMonty Taylor proposed openstack-infra/shade master: Optimize the case of versioned image endpoint in catalog  https://review.openstack.org/46468216:16
mordredShrews, rcarrillocruz: if you have a second to double-check me on those ^^ ... got a bug report https://storyboard.openstack.org/#!/story/2001027 that the first is a fix for and the second an optimization for16:20
*** slaweq has joined #openstack-shade16:27
*** slaweq has quit IRC16:28
*** slaweq has joined #openstack-shade16:28
*** slaweq has quit IRC16:33
*** pabelanger_ is now known as pabelanger16:57
*** slaweq has joined #openstack-shade17:09
Shrewsmordred: well, the test failures for those aren't promising17:22
mordredShrews: I agree17:33
mordredShrews: looks like we'll need more unit tests! yay!17:33
mordredShrews: ugh. those all seem to be related to the legacy-libs stuff17:35
mordredShrews: hah! I believe we have properly fixed the gate job to do what we'd intended, and have found a place where the test not doing so was papering over an actual problem17:38
mordredShrews: yay us!17:38
mordredShrews: I mean, now it means I've got yet-another-rathole to descend this afternoon17:38
Shrewsi'm still confused over the previous ratholes17:39
mordredShrews: rats will do that to you17:39
mordredShrews: so - the tl;dr on this one is that before we fixed the job in project-config - we were testing tip of shade with stable/newton of python-novaclient in order to talk to stable/newton ... but that's not actually what we wanted to test - we want to test tip of shade with the version of novaclient you'll get from pip install will still work against old clouds17:40
mordredShrews: it's an issue in this case because novaclient 8.0 removed support for nova-network17:40
mordredthat's not a problem for us with pike because we also don't deploy nova-network in pike17:41
*** slaweq has quit IRC17:41
mordredbut we missed the novaclient removal on the legacy jobs because we were running with the wrong version of novaclient17:41
mordredI guess that's not really a tl;dr17:41
*** slaweq has joined #openstack-shade17:42
mordredShrews: I'm going to make a fix stacked on top of the fix-tips patch for review - but then we'll need to squash toland it17:42
Shrewsnod17:43
*** slaweq has quit IRC17:46
*** calebb has joined #openstack-shade18:09
*** iogg has quit IRC18:30
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix broken version discovery endpoints  https://review.openstack.org/46468119:10
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove cinderclient from install-tips.sh  https://review.openstack.org/46432919:10
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix tips jobs to do what we think they do  https://review.openstack.org/46077919:10
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert Nova Floating IP tests to REST  https://review.openstack.org/46472919:10
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert Nova Floating IP calls to REST  https://review.openstack.org/46473019:10
mordredShrews: ^^ ok - the first two there should fail but the third should be green. if we're happy with all three we can squash them into a single commit we can land19:10
*** iogg has joined #openstack-shade19:20
*** slaweq has joined #openstack-shade19:28
*** slaweq has quit IRC19:38
openstackgerritMonty Taylor proposed openstack-infra/shade master: Optimize the case of versioned image endpoint in catalog  https://review.openstack.org/46468219:42
*** iogg is now known as ioggstream19:47
openstackgerritMonty Taylor proposed openstack-infra/shade master: Optimize the case of versioned image endpoint in catalog  https://review.openstack.org/46468220:44
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix broken version discovery endpoints  https://review.openstack.org/46468120:44
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert Nova Floating IP calls to REST  https://review.openstack.org/46473020:44
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove cinderclient from install-tips.sh  https://review.openstack.org/46432920:44
openstackgerritMonty Taylor proposed openstack-infra/shade master: Include error message from server if one exists  https://review.openstack.org/46474820:44
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove unused ServerConsoleGet task  https://review.openstack.org/46475221:10
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert list_server_security_groups to REST  https://review.openstack.org/46475321:10
*** larainema has quit IRC21:13
*** ioggstream has quit IRC21:24
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert Nova Floating IP calls to REST  https://review.openstack.org/46473021:25
openstackgerritMonty Taylor proposed openstack-infra/shade master: Optimize the case of versioned image endpoint in catalog  https://review.openstack.org/46468222:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Include error message from server if one exists  https://review.openstack.org/46474822:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix broken version discovery endpoints  https://review.openstack.org/46468122:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove cinderclient from install-tips.sh  https://review.openstack.org/46432922:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Fix tips jobs and convert Nova Floating IP calls  https://review.openstack.org/46077922:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Convert list_server_security_groups to REST  https://review.openstack.org/46475322:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove two unused nova tasks  https://review.openstack.org/46475222:18
mordredboom. patch bomb22:18
openstackgerritSlawek Kaplonski proposed openstack-infra/shade master: Remove designateclient mock from recordset tests  https://review.openstack.org/46476622:19
mordredslaweq_: excellent. that's probably going to fail tests because of broken gate - but the patch to fix that is finally green - just pushed the new version of it up22:21
mordredslaweq_: https://review.openstack.org/460779 is the magic, fwiw22:21
*** slaweq has joined #openstack-shade22:23
*** slaweq has quit IRC22:33
*** slaweq has joined #openstack-shade22:35
*** slaweq has quit IRC22:35
*** morgan_ has joined #openstack-shade22:41
openstackgerritMonty Taylor proposed openstack-infra/shade master: Remove direct uses of nova_client in functional tests  https://review.openstack.org/46477223:02
*** gouthamr has quit IRC23:56
openstackgerritMerged openstack-infra/shade master: Fix tips jobs and convert Nova Floating IP calls  https://review.openstack.org/46077923:56
openstackgerritMerged openstack-infra/shade master: Remove cinderclient from install-tips.sh  https://review.openstack.org/46432923:57

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