| openstackgerrit | Jamie Lennox proposed openstack-infra/shade master: Don't use project-id in catalog tests https://review.openstack.org/451123 | 00:16 |
|---|---|---|
| *** jamielennox is now known as jamielennox|away | 01:02 | |
| *** jamielennox|away is now known as jamielennox | 01:16 | |
| *** gouthamr has quit IRC | 02:37 | |
| *** gouthamr has joined #openstack-shade | 02:43 | |
| *** gouthamr has quit IRC | 03:19 | |
| *** jeblair has quit IRC | 03:24 | |
| *** jeblair has joined #openstack-shade | 03:25 | |
| openstackgerrit | Jamie Lennox proposed openstack-infra/shade master: Add server security groups to shade https://review.openstack.org/451183 | 05:10 |
| openstackgerrit | Jamie Lennox proposed openstack-infra/shade master: Add server security groups to shade https://review.openstack.org/451183 | 05:15 |
| * jamielennox learnt many things.... ^ | 05:15 | |
| *** jamielennox is now known as jamielennox|away | 06:10 | |
| *** jamielennox|away is now known as jamielennox | 06:25 | |
| *** jamielennox is now known as jamielennox|away | 06:50 | |
| *** jamielennox|away is now known as jamielennox | 06:57 | |
| *** yfried has joined #openstack-shade | 07:07 | |
| *** Matias has quit IRC | 07:27 | |
| *** Matias has joined #openstack-shade | 07:39 | |
| *** jamielennox is now known as jamielennox|away | 08:17 | |
| *** ioggstream has joined #openstack-shade | 08:22 | |
| mordred | jamielennox|away: woot! | 08:37 |
| *** rcarrillocruz has joined #openstack-shade | 08:52 | |
| mordred | jamielennox|away: and yes - to your note in the code - the pattern elsewhere in shade is to just return False if a removal doesn't remove anything | 09:01 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Add bare parameter to get/list/search server https://review.openstack.org/451264 | 09:09 |
| openstackgerrit | Merged openstack-infra/shade master: Don't use project-id in catalog tests https://review.openstack.org/451123 | 09:25 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Fix a few minor annoyances that snuck in https://review.openstack.org/451276 | 09:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Enable H238 - classes should be subclasses of object https://review.openstack.org/451277 | 09:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Enable H201 - don't throw bare exceptions https://review.openstack.org/451278 | 09:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Reenable hacking tests that already pass https://review.openstack.org/451279 | 09:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Take care of multiple imports and update explanation https://review.openstack.org/451280 | 09:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Update tox build settings https://review.openstack.org/451312 | 10:14 |
| *** yfried has quit IRC | 12:02 | |
| *** gouthamr has joined #openstack-shade | 12:47 | |
| *** Matias has quit IRC | 13:25 | |
| *** Matias has joined #openstack-shade | 13:37 | |
| *** rcarrill1 has joined #openstack-shade | 16:35 | |
| *** rcarrillocruz has quit IRC | 16:36 | |
| *** ioggstream has quit IRC | 17:30 | |
| *** openstackstatus has joined #openstack-shade | 18:45 | |
| *** ChanServ sets mode: +v openstackstatus | 18:45 | |
| *** openstackgerrit has quit IRC | 19:33 | |
| *** jamielennox|away is now known as jamielennox | 19:33 | |
| *** ioggstream has joined #openstack-shade | 19:33 | |
| *** rcarrill1 is now known as rcarrillocruz | 20:49 | |
| *** openstackgerrit has joined #openstack-shade | 20:55 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/os-client-config master: Docs: add a note about rackspace API keys https://review.openstack.org/451563 | 20:55 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Add bare parameter to get/list/search server https://review.openstack.org/451264 | 21:09 |
| *** gouthamr has quit IRC | 21:21 | |
| *** ioggstream has quit IRC | 21:38 | |
| TheJulia | So I'm starting to hack on a decent addition of code that crosses a number of ironic microversions. Is there actively a plan for handling microversions? I guess I could just refuse to change a value that is not returned in the original request to compare for patch generation, and just continue to use the python-ironicclient for now.... which actually | 22:02 |
| TheJulia | makes some sense.... | 22:02 |
| TheJulia | except I'll have to update the pin :( | 22:05 |
| *** jamielennox is now known as jamielennox|away | 22:09 | |
| *** jamielennox|away is now known as jamielennox | 22:12 | |
| TheJulia | that stupid pin and the client library having a pin of it's own .... :( | 22:34 |
| jamielennox | mordred: why does https://review.openstack.org/#/c/451264/2 have a dependency on something in designate? | 22:36 |
| mordred | jamielennox: because designate is falling down and that was an attempt to reparent on top of the fix - however, the new plan is removing designate from shade's gate - so we can remove that depends-on | 22:43 |
| mordred | TheJulia: good timing! so - we're about to start working on replacing ironicclient with direct rest calls in shade | 22:44 |
| TheJulia | \o/ | 22:44 |
| mordred | TheJulia: so, as part of that, adding the ability to pass the microversion to a particular call is important | 22:44 |
| mordred | TheJulia: as is doing the version detection step and finding the available microversion range on the server | 22:44 |
| mordred | my general thought on it is that for any single call that has multiple microversions, we should treat it like we treat features across clouds - have a logical if in shade for each different microversion that affects the call | 22:46 |
| TheJulia | mordred: So I guess for data structures, the client version could be available, we could then evaluate if a requested change or data is actually invalid for the available api rev and give a sane error message | 22:46 |
| mordred | then when we go to do things, try the highest version logic we have for the highest the server supports | 22:46 |
| mordred | TheJulia: yes! | 22:46 |
| *** gouthamr has joined #openstack-shade | 22:46 | |
| mordred | I think it's totally reasonable to say "your version of ironic cannot do this thing you want to do - maybe consider (insert remediation) - or if possible uprade ironic" | 22:46 |
| mordred | but then for other things, we might be able to hide over it in some way or something | 22:47 |
| TheJulia | Yeah, the only stupidly impacting change has been state machine changes that are breaking that are guarded by microversion. Everything else has been data, but both can't be had, and the pin in the client lib is just causing my brain to go "10 print "ouch"\n20 goto 10" | 22:48 |
| mordred | ++ | 22:48 |
| mordred | I know that brain program | 22:48 |
| clarkb | http://logs.openstack.org/92/451492/4/check/gate-devstack-dsvm-py35-updown-ubuntu-xenial-nv/a537722/logs/screen-n-cpu.txt.gz#_2017-03-29_22_44_08_090 I just discovered that one and its making me run that program | 22:49 |
| mordred | TheJulia: so, pabelanger said he was going to try to start working on the ironic change this week - but given you have some needs you want to solve, I may escalate getting started on that | 22:49 |
| mordred | since I'd LOVE to have your eyes/attention on it as we do that | 22:49 |
| mordred | clarkb: you're so happy | 22:50 |
| clarkb | mordred: its amazing, so old libvirt doesnt' work, next step is try newer libvirt (actually asked nova libvirt people and what they really said was use centos) but it turns out that nova doesn't work with newer libvirt? | 22:51 |
| TheJulia | mordred: tl;dr I want to add real networking support for ironic stuffs since we have different network interface concepts now. Oh, ouch, that is another microversion guarded thing. | 22:51 |
| clarkb | oh you know what, this could be a bug in python3 specifically | 22:51 |
| clarkb | that would explain a lot | 22:51 |
| mordred | clarkb: many many things | 22:53 |
| mordred | TheJulia: :) | 22:53 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Add bare parameter to get/list/search server https://review.openstack.org/451264 | 22:54 |
| mordred | jamielennox: ok - there it is withouth the designate depends-on. the project-config change landed a while back, so this time it should run without designate enabled and we should be good to go | 22:55 |
| clarkb | mordred: actually this could be a wheel build issue :/ | 22:55 |
| clarkb | RIP everything | 22:55 |
| mordred | clarkb: oh goodie | 22:55 |
| jamielennox | mordred: thanks - i was just wanting to rebase my security groups patch on that and i thought it was already +Aed | 22:56 |
| mordred | clarkb: are we building a wheel against a different verison fo libvirt? | 22:56 |
| mordred | jamielennox: yah - it was- then we ran in to the gate | 22:56 |
| clarkb | mordred: ya that code is using cloud archive but normally we buil dagainst base xenial libvirt | 22:56 |
| mordred | yup. that's not going to work so much | 22:57 |
| mordred | I think that code if it'susing cloud archive should also disable the wheel mirror | 22:57 |
| clarkb | this actually poses a somewhat majorish problem with wheels | 22:58 |
| clarkb | especially if we want to every do something like rolling releases | 22:58 |
| clarkb | or genttoo | 22:58 |
| clarkb | anyways this isn't the forum /me goes away | 22:59 |
| openstackgerrit | Jamie Lennox proposed openstack-infra/shade master: Add server security groups to shade https://review.openstack.org/451183 | 23:03 |
| openstackgerrit | Jamie Lennox proposed openstack-infra/shade master: Add server security groups to shade https://review.openstack.org/451183 | 23:12 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!