| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-15 | |||
| 13:47:33 | mriedem | gibi: +2 | |
| 13:51:33 | gibi | mriedem: thanks | |
| 13:53:00 | openstackgerrit | Merged openstack/nova master: [placement] Use util.extract_json in allocations handler https://review.openstack.org/471067 | |
| 13:53:54 | frickler | mriedem: did you see my comment about the novaclient bug earlier? would be good if someone could at least triage it, if not fix it. sadly no time to look deeper myself | |
| 13:54:12 | mriedem | frickler: i didn't, wasn't around | |
| 13:54:19 | mriedem | link? | |
| 13:54:50 | frickler | mriedem: https://launchpad.net/bugs/1697452 | |
| 13:54:50 | openstack | Launchpad bug 1697452 in python-novaclient "openstack client dumps debug info on console even without specifying --debug option" [Undecided,Confirmed] | |
| 13:55:43 | mriedem | frickler: when you say volume commands in there, you mean osc volume commands? | |
| 13:55:47 | mriedem | b/c those would be cinderclient | |
| 13:55:54 | mriedem | which would make me think it's a shared common library issue | |
| 13:56:03 | mriedem | nova meeting in 4 minutes | |
| 13:56:34 | frickler | mriedem: no, "volume show" does a "server show" internally in order to display attachments. so its really only novaclient=9.0.0 being broken | |
| 13:57:24 | mriedem | huh, well that seems totally wasteful and redundant since cinder's volume api shows attachments within it's own response | |
| 13:57:25 | mriedem | but ok | |
| 13:57:55 | mriedem | https://developer.openstack.org/api-ref/block-storage/v2/?expanded=show-volume-details-detail#show-volume-details | |
| 13:58:26 | bauzas | cdent: quick question, why https://review.openstack.org/#/c/473479/1/nova/tests/functional/api/openstack/placement/gabbits/inventory.yaml@654 is having generation by 8 ? | |
| 13:58:41 | bauzas | cdent: because we delete ? | |
| 13:58:44 | bauzas | oh, nvm | |
| 13:59:36 | frickler | mriedem: anyway, main issue is getting debug output on any nova related command like "server show" or "flavor list" | |
| 13:59:52 | mriedem | yeah, will have to do a diff between 8.0.0 and 9.0.0 | |
| 13:59:56 | mriedem | about to start a meeting though | |
| 14:04:21 | cdent | bauzas: you figured it out? | |
| 14:04:45 | bauzas | cdent: yup, because since we delete, we need to increment the generation | |
| 14:16:24 | openstackgerrit | Baodong (Robert) Li proposed openstack/nova-specs master: Expose vlan trunking in metadata/configdrive https://review.openstack.org/471815 | |
| 14:22:52 | frickler | mriedem: sdague: reverting https://review.openstack.org/463508 fixes 1697452 for me, but there might be a better solution than that | |
| 14:25:05 | sdague | frickler: right, so https://review.openstack.org/#/c/463508/5/novaclient/v2/client.py is the bit that's wrong? | |
| 14:25:29 | sdague | frickler: can you propose a revert of just that part of the patch? | |
| 14:26:04 | sdague | frickler: or would you like me to do it? | |
| 14:26:08 | sfinucan | mriedem: Went for tea when you were discussing https://review.openstack.org/#/c/472275/ | |
| 14:27:03 | frickler | sdague: I'm not sure whether the rest of the patch will break then. maybe rather do a complete revert and then work on a better fix. anyway, please do as you like, I'm off for today now | |
| 14:27:16 | sdague | frickler: ok, I'll play with it | |
| 14:27:41 | sfinucan | I've been talking to asettle about that. Happy to "own" it if there's work to be done moving stuff into tree | |
| 14:27:47 | mriedem | sdague: +1 to just https://review.openstack.org/#/c/463508/5/novaclient/v2/migrations.py | |
| 14:27:56 | mriedem | sdague: warnings.warn is what we've always done in the python api binding code | |
| 14:28:01 | sdague | mriedem: yeh | |
| 14:28:07 | sdague | it was just too aggressive | |
| 14:28:08 | mriedem | sfinucan: i'd love someone to be a point person on that docs transition | |
| 14:28:13 | sdague | I'm going to do a partial revert | |
| 14:28:24 | openstack | Launchpad bug 1697452 in python-novaclient "openstack client dumps debug info on console even without specifying --debug option" [Undecided,Confirmed] | |
| 14:28:24 | mriedem | sdague: ack, this is the bug https://bugs.launchpad.net/python-openstackclient/+bug/1697452 | |
| 14:29:10 | mriedem | frickler: thanks for bringing it up | |
| 14:29:33 | sfinucan | mriedem: Cool. I'm your guy, in that case. Assuming nothing is happening til Queens anyway | |
| 14:29:37 | mriedem | sfinucan: nicolasbock is also docs CPL so you can double team | |
| 14:29:46 | sfinucan | seeing as the spec need to be approved | |
| 14:29:53 | sfinucan | Sounds good | |
| 14:30:04 | mriedem | btw, does double-team translate in irish? | |
| 14:30:14 | mriedem | https://en.wikipedia.org/wiki/Professional_wrestling_double-team_maneuvers | |
| 14:30:22 | sfinucan | tag-team, I guess? | |
| 14:30:35 | sfinucan | or just "team up" | |
| 14:31:10 | openstackgerrit | Sean Dague proposed openstack/python-novaclient master: Revert "client.logger.warning wrongly used in migrations" https://review.openstack.org/474600 | |
| 14:31:34 | sdague | mriedem: ^^^ | |
| 14:31:46 | nicolasbock | Hi sfinucan | |
| 14:31:53 | sdague | oh, crap, I did the wrong revert direction | |
| 14:31:55 | sdague | doh | |
| 14:31:57 | sdague | one sec | |
| 14:32:06 | nicolasbock | let me know how you want to 'tag-team' :) | |
| 14:33:59 | openstackgerrit | Sean Dague proposed openstack/python-novaclient master: Revert "client.logger.warning wrongly used in migrations" https://review.openstack.org/474600 | |
| 14:37:23 | openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Add traits to the ResourceProviders filters https://review.openstack.org/474602 | |
| 14:37:46 | edleafe | jaypipes: bauzas: ^^ | |
| 14:48:38 | ftersin | mdbooth: hi. have you had a chance to look at the reviews which i told you at Tuesday? | |
| 14:49:08 | mdbooth | ftersin: Still no, I'm afraid. Caught up with downstream work. | |
| 14:49:40 | sean-k-mooney | moshele: sfinucan question inline in https://review.openstack.org/#/c/460278/3 | |
| 14:50:15 | sean-k-mooney | moshele: sfinucan: moshele can you confim if that was the last commit required for the hw offload? | |
| 14:51:14 | moshele | sean-k-mooney: yes all the patches are merged | |
| 14:51:45 | sean-k-mooney | moshele: ok then assuming the recheck passes py3 im happy to +2 | |
| 14:52:12 | moshele | sean-k-mooney: cool , sfinucan: can you remove you -2 | |
| 14:52:58 | sean-k-mooney | moshele: i have the recheck running. the six error should now be fixed in the gate so im expecting it to pass | |
| 14:53:25 | moshele | sean-k-mooney: thanks | |
| 14:54:46 | ftersin | mdbooth: could you estimate when you become able to work on the reviews? | |
| 14:55:33 | mdbooth | ftersin: I'd hoped this week, but now I'd guess maybe early next week. Tuesday-ish? | |
| 15:00:48 | ftersin | mdbooth: i think that not so much time is left in Pike for such big work. Because that it'd be perfect if we start in the closest future | |
| 15:01:40 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: hardware offload support for openvswitch https://review.openstack.org/460278 | |
| 15:04:11 | sfinucan | moshele: Will do | |
| 15:14:43 | cfriesen | dansmith: can "nova-manage db online_data_migrations" run with nova-conductor not running? | |
| 15:14:57 | dansmith | of course | |
| 15:16:48 | cfriesen | thanks | |
| 15:19:12 | sfinucan | nicolasbock: Hey - sorry, was in a meeting | |
| 15:19:31 | sfinucan | I don't think there's too much to be done right now. asettle is gone for the week so the spec is stuck | |
| 15:19:48 | sfinucan | Maybe just review that spec and see if you agree with it. I'll do the same myself tomorrow | |
| 15:28:48 | tasker | morning! I'm tracking down a problem with evacuation in Mitaka and came across https://review.openstack.org/#/c/385086/, which mentions commit c15ed42. I cannot find that commit anywhere. Similarly, Artom is mentioning (in more than one bug / review) that "since event dispatching is fixed" this is not a problem anymore, but saying so without reference. Can someone put their finger on these patches? | |
| 15:29:00 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | |
| 15:30:28 | tasker | is Artom referring to this one? https://review.openstack.org/#/c/371048/ | |
| 15:35:55 | mriedem | tasker: probably | |
| 15:47:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Provide original fault message when BFV fails https://review.openstack.org/467715 | |
| 15:48:36 | nicolasbock | sfinucan, no problem | |
| 15:48:42 | nicolasbock | sfinucan, I'll have a look at it | |
| 15:52:29 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1766 | |
| 15:52:29 | mriedem | dansmith: i just found that if we try to create a server with tagged devices and it lands on a host that doesn't support that, it's going to abort the build rather than reschedule | |
| 15:52:34 | mriedem | doesn't that seem...wrong? | |
| 15:52:54 | dansmith | um, why? | |
| 15:52:58 | mriedem | in thinking about the "95% of reschedules are due to claim contention" thing | |
| 15:53:14 | dansmith | it's an abort not a reschedule right? | |
| 15:53:15 | mriedem | why wouldn't we reschedule to try another host | |
| 15:53:45 | dansmith | because you likely picked a flavor that has you on a specific hypervisor type anyway, so reschedule would just waste resources until you run out of retries | |
| 15:54:38 | mriedem | seems like a big assumption | |
| 15:54:51 | mriedem | i don't need a special flavor to create a server with nic/bdm tags | |
| 15:55:19 | mriedem | granted, the chances of hitting a xen host and failing and then hitting a libvirt host are probably slim anyway | |
| 15:55:27 | mriedem | or chances of having a mixed compute env to start with anyway | |
| 15:55:36 | mriedem | *mixed hypervisor | |