| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-29 | |||
| 15:02:05 | dansmith | efried: I added a few notes, I gotta jump on a call now | |
| 15:02:10 | jaypipes | cdent: btw, have you started on an os-resource-classes lib? if not, I was going to hop on that... | |
| 15:02:24 | efried | dansmith: ack | |
| 15:03:03 | cdent | jaypipes: go for it. I'd been trying to lay it out as bait for $random_person, but if you're keen go for it | |
| 15:03:17 | jaypipes | alex_xu_: see my question to Intel folks on line 13 of the etherpad... | |
| 15:06:32 | alex_xu_ | jaypipes: yes, I think we should say it is 'pre-programmed' or 'static' compare to 'fully dynamic' | |
| 15:08:29 | cdent | jaypipes, efried, others: watching that etherpad it appears that the conversation from the hangout is continuing rather than being summarized? | |
| 15:09:09 | edleafe | cdent: exactly. IOW, we can't summarize our conclusions if we didn't reach consensus | |
| 15:09:24 | efried | cdent: Sorry, got pulled away as soon as I posted that. Will look in a couple minutes. | |
| 15:10:43 | jaypipes | cdent: yes :( | |
| 15:11:35 | efried | mriedem: The fix https://review.openstack.org/#/c/557508/ works. But I think the hacking check fix isn't right just yet. I will revise. | |
| 15:12:12 | efried | mriedem: (The assertion that it works is based on internal discussions - I've asked them to update the bug accordingly to make it official.) | |
| 15:12:13 | mriedem | efried: pep8 passes | |
| 15:12:21 | efried | mriedem: But pyXX won't. | |
| 15:12:31 | mriedem | oh, i didn't run those... | |
| 15:12:51 | mriedem | if you change the hacking check, it likely should be a separate patch before that cinder fix | |
| 15:12:51 | efried | I didn't either, just assuming based on how mock.patch works... | |
| 15:13:06 | mriedem | i was just being lazy and expediant | |
| 15:13:53 | efried | mriedem: Yeah, I'll fix the hacking check totally separately. I'll tweak the fix you did on this patch cause we want it quickish. | |
| 15:14:10 | alex_xu_ | jaypipes: dansmith to avoid program a pre-programmed device, we need 'OR' logic in allocation candidates, I commented at line 30 | |
| 15:15:06 | openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa session for cinder microversion check https://review.openstack.org/557508 | |
| 15:15:07 | efried | mriedem: ^ | |
| 15:17:18 | mriedem | oh right i screwed up the order of the args to the test metohd | |
| 15:17:19 | mriedem | oops | |
| 15:18:22 | mriedem | ok i see "@efried, patch seems to be working fine with vm deploy/delete, volume attach/delete flows" in the bug | |
| 15:18:42 | efried | mriedem: Oh, yeah, they did comment on the bug. Cool. | |
| 15:19:22 | mriedem | +2 | |
| 15:19:36 | mriedem | i know this has to go back to queens but not sure if it's also needed in pike, will have to check | |
| 15:21:15 | openstackgerrit | Merged openstack/nova-specs master: Update 'share-pci-between-numa-nodes' spec to reflect reality https://review.openstack.org/555000 | |
| 15:23:36 | kashyap | mriedem: I changed the exception here to a warning (locally) & fixed all the other things you pointed out; is _another_ test still required here: https://review.openstack.org/#/c/534384/20/nova/virt/libvirt/driver.py@3875 | |
| 15:25:49 | efried | sdague: You seem like a likely candidate to +A https://review.openstack.org/#/c/557508/ -- got a minute for that? | |
| 15:28:09 | mriedem | kashyap: you asked dansmith that question yesterday | |
| 15:28:13 | mriedem | i remember the answer | |
| 15:28:31 | kashyap | mriedem: Heh, not trying to "circumvent" anything! | |
| 15:28:43 | kashyap | Wanted to double-confirm. Something about "assume good intent"? | |
| 15:29:05 | bauzas | to the audience, keep in mind that if you tell to a French folk "I like you, but you're bad", there is a very strong meaning https://review.openstack.org/#/c/547990/16/nova/scheduler/client/report.py@341 | |
| 15:29:37 | bauzas | tl;dr: the "but" litterally cancels what you just said before | |
| 15:29:58 | mriedem | that means the same thing in english | |
| 15:30:08 | mriedem | "i like you, but we're breaking up" | |
| 15:30:27 | mriedem | "you're a valuable member of the team, but...." | |
| 15:31:16 | dansmith | kashyap: keeping the patch small is good, but not generally at the expense of tests | |
| 15:31:29 | dansmith | kashyap: when in doubt, cut down the four-page release notes :) | |
| 15:31:30 | kashyap | dansmith: Okido; I'll shut up and get that going. :-) | |
| 15:31:49 | kashyap | dansmith: Actually, most of that info belongs in the config option help text | |
| 15:31:59 | kashyap | dansmith: But, yes. I trimmed it | |
| 15:32:26 | kashyap | dansmith: Wonder if you could cut some slack, English is my 3rd language, much as I love it :P | |
| 15:32:40 | bauzas | mriedem: what's interesting is that 'but' in english can mean "except that one" | |
| 15:32:50 | bauzas | mriedem: while we don't have that one in French | |
| 15:33:02 | bauzas | it's another word | |
| 15:33:03 | edleafe | alex_xu_: jaypipes: sorry, meeting time. Would love to continue our etherpad conversation, but... | |
| 15:33:11 | bauzas | hah | |
| 15:33:15 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix N332 api_version decorator hacking check https://review.openstack.org/557743 | |
| 15:33:17 | efried | mriedem: ^ | |
| 15:33:20 | bauzas | edleafe: good usage of "but" :p | |
| 15:33:24 | kashyap | bauzas: Speaking of French and English; do you know this: "How a Mistake Gave Us the Word 'Cherry'" -- https://www.merriam-webster.com/words-at-play/cherry-history-origin | |
| 15:33:25 | bauzas | right on time | |
| 15:33:38 | alex_xu_ | edleafe: np, my brain doesn't work also | |
| 15:34:21 | jaypipes | alex_xu_, edleafe: do we have something that can be standardized in os-traits that represents the things that an FPGA is *capable* of programming? For example, in vGPU land, we have the VGPU_RESOLUTION_XXX traits and VGPU_MAX_DISPLAY_HEAD traits etc. | |
| 15:34:55 | edleafe | jaypipes: dunno - that would be a good question for the FPGA vendors | |
| 15:36:05 | efried | bauzas: Would you mind casting your eye upon https://review.openstack.org/#/c/557508/ ? | |
| 15:36:17 | bauzas | if that's only one eye | |
| 15:36:20 | bauzas | I can blink | |
| 15:37:02 | bauzas | efried: CC'd | |
| 15:37:04 | efried | acceptable | |
| 15:37:12 | bauzas | efried: just focusing on dansmith's series | |
| 15:37:18 | bauzas | but then I can help | |
| 15:37:21 | efried | thanks | |
| 15:37:24 | mriedem | bauzas: ever word in english has at least 3 different meanings | |
| 15:37:45 | mriedem | *every even | |
| 15:37:47 | efried | holy shit, I just looked back at that etherpad. | |
| 15:37:48 | bauzas | efried: food for thoughts too https://review.openstack.org/#/c/557065/ | |
| 15:37:56 | bauzas | efried: since you asked me about that | |
| 15:38:02 | efried | bauzas: ack | |
| 15:38:09 | efried | brb... | |
| 15:38:11 | bauzas | I'm not a big fan of a nova-manage command just for that | |
| 15:38:36 | bauzas | if one day libvirt provides the API to set this, then we would deprecate the conf option | |
| 15:38:55 | bauzas | while a nova-manage command for a very specific libvirt hack makes me worried by the precedence | |
| 15:42:57 | bauzas | gibi: happy travels | |
| 15:43:45 | cdent | happy honeymoon gibi | |
| 15:45:14 | melwitt | o/ gibi | |
| 15:49:08 | alex_xu_ | jaypipes: FPGA_FUNCTION_X,y,z, I guess | |
| 15:49:24 | alex_xu_ | jaypipes: and I thought we should have a trait FPGA_DEVICE_PRE_PROGRAMMED | |
| 15:49:38 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Fix rebuild of baremetal instance when vm_state is ERROR https://review.openstack.org/523559 | |
| 15:53:47 | dansmith | mriedem: to use osc-placement do I have to tell osc to use a specific microversion? | |
| 15:54:12 | dansmith | getting "Operation or argument is not supported with version 1.0" | |
| 15:54:59 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Fix bad management of _TRAITS_SYNCED flag https://review.openstack.org/557722 | |
| 15:55:34 | openstackgerrit | Merged openstack/nova stable/queens: add check before adding cpus to cpuset_reserved https://review.openstack.org/557621 | |
| 15:55:55 | mriedem | dansmith: yup | |
| 15:56:02 | mriedem | osc doesn't default to latest like nova cli does | |
| 15:56:31 | dansmith | yeah I found it | |
| 15:56:35 | jaypipes | gibi: congrats again and have a great time on your honeymoon, man :) | |
| 15:57:16 | mriedem | get used to always being wrong | |
| 16:10:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Documentation for tenant isolation with placement https://review.openstack.org/557490 | |
| 16:10:42 | dansmith | mriedem: wanna glance at this ^ before I shut down my devstack node I used to generate the command outputs? | |
| 16:18:34 | mriedem | please hold | |
| 16:18:40 | efried | dansmith, jaypipes, edleafe, cdent, bauzas, alex_xu_: I'm caught up on the etherpad now. I think there's actually agreement on the salient points. The discussion of "capable of X" versus "flashed with X" is orthogonal. (Still relevant, still needing discussion, but the outcome doesn't affect the rest.) Good if I summarize and respond on the ML? | |
| 16:18:42 | mriedem | https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/hypervisor.html#hypervisor-list | |
| 16:18:45 | mriedem | dansmith: let's use osc | |
| 16:18:54 | dansmith | mriedem: it doesn't show id | |
| 16:19:05 | mriedem | --os-compute-api-version 2.53 | |