| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-22 | |||
| 13:48:54 | kashyap | melwitt: Wauw, isn't it like 05:45 AM for you there? | |
| 13:48:59 | melwitt | tis | |
| 13:49:12 | kashyap | Jee | |
| 13:50:40 | melwitt | oh, fyi everyone I'm traveling today and tomorrow to the PTG. so I won't be around later today and tomorrow | |
| 13:51:00 | openstackgerrit | Merged openstack/nova stable/queens: Only pull associated *sharing* providers https://review.openstack.org/546740 | |
| 13:51:14 | jaypipes | efried: you around? | |
| 13:51:20 | efried | jaypipes: Yup. | |
| 13:51:30 | sean-k-mooney | melwitt: nice :) going to do some siteseeing over the weekend? | |
| 13:51:47 | jaypipes | efried: you pinged me last night about something but it's gone from my scrollback... what was it? | |
| 13:52:09 | efried | jaypipes: I desperately needed some SQL help. | |
| 13:52:22 | melwitt | sean-k-mooney: on saturday yes. on sunday we have an internal meeting day | |
| 13:52:27 | jaypipes | efried: I don't know anything about SQL | |
| 13:52:29 | efried | jaypipes: But I slogged through it, with much cursing, and put up a patch for GET /resource_providers?required=<traits> | |
| 13:52:45 | jaypipes | efried: ah, cool. | |
| 13:53:11 | efried | jaypipes: We could conceivably get this blueprint closed before the PTG :) | |
| 13:53:24 | jaypipes | efried: nothing wrong with that, IMHO. | |
| 13:53:25 | efried | jaypipes: https://review.openstack.org/#/c/546837/ | |
| 13:53:33 | efried | Yeah, I thought it would be a nice tickymark. | |
| 13:53:54 | jaypipes | efried: that multiattach failure related? | |
| 13:54:04 | jaypipes | don't see how it could be, but worth asking... | |
| 13:54:06 | efried | jaypipes: No, random timeout. | |
| 13:54:08 | efried | I rechecked. | |
| 13:54:10 | jaypipes | k | |
| 13:54:44 | jaypipes | melwitt, dansmith, mriedem, bauzas: one of the easier specs to +2/W here: https://review.openstack.org/#/c/546009/2 | |
| 13:55:36 | bauzas | mriedem: melwitt: gibi_off told me about that, I was planning to chair it | |
| 13:55:43 | bauzas | but fine by me | |
| 13:56:10 | sean-k-mooney | jaypipes: efried: if your makeing a list of blueprints that could be closed before the ptg the nic feature based scheduling blueprint has been code complete since pike and is fully rebased for rocky. spec needs to be reapproved and patches re reviwed though | |
| 13:56:36 | melwitt | bauzas: okay, cool. yeah, I realized I'd have to get up early to make sure you were able to run it anyway so I might as well do it. it was my bad. thanks though | |
| 13:56:38 | jaypipes | sean-k-mooney: link pls | |
| 13:57:28 | sean-k-mooney | jaypipes: code https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/enable-sriov-nic-features spec https://review.openstack.org/#/c/545951/ | |
| 13:57:46 | sean-k-mooney | on tox faild on the spec ill go fix that now | |
| 13:58:10 | jaypipes | sean-k-mooney: danke | |
| 13:58:35 | sean-k-mooney | did something change in the template between rocky and queens? | |
| 13:58:59 | melwitt | sean-k-mooney: there's one new section, Upgrade impact | |
| 13:59:03 | bauzas | melwitt: heh, no worries | |
| 13:59:28 | sean-k-mooney | melwitt: ah yes that is exactly the error http://logs.openstack.org/51/545951/1/check/openstack-tox-pep8/9898780/job-output.txt.gz#_2018-02-19_20_27_16_685503 ill add it and reupload | |
| 13:59:52 | mriedem | jaypipes: i'll look after said busification | |
| 14:00:01 | mriedem | and coffee injections | |
| 14:00:30 | bauzas | mriedem: alrready looking ;) | |
| 14:00:34 | jaypipes | mriedem: 10-4 | |
| 14:05:52 | openstackgerrit | Carlos Goncalves proposed openstack/os-vif master: WIP: Add internal port https://review.openstack.org/546991 | |
| 14:11:39 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | |
| 14:13:35 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | |
| 14:13:50 | bauzas | melwitt: mriedem: I'm about to set Invalid for https://bugs.launchpad.net/nova/+bug/1751036 | |
| 14:13:50 | openstack | Launchpad bug 1751036 in OpenStack Compute (nova) "ConfigFileValueError when defining enabled_apis" [Undecided,New] | |
| 14:14:02 | bauzas | because of the interop APIs | |
| 14:14:09 | mriedem | bauzas: i'm in a meeting atm | |
| 14:14:10 | bauzas | but do you agree ? | |
| 14:14:19 | bauzas | that I now, let's discuss later | |
| 14:14:23 | bauzas | know* | |
| 14:27:34 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | |
| 14:38:10 | tetsuro | jaypipes, cfriesen: now this spec https://review.openstack.org/543805 is ready for review again. | |
| 14:41:58 | jaypipes | tetsuro: cool, thanks. (currently in nova meeting) | |
| 14:42:22 | openstackgerrit | Eric Fried proposed openstack/nova master: Make _get_sharing_providers more efficient https://review.openstack.org/547008 | |
| 14:42:36 | tetsuro | Oops | |
| 14:42:42 | efried | jaypipes, cdent: ^ the optimization that prompted the feature | |
| 14:44:28 | jaypipes | tetsuro: it's no prob :) | |
| 14:53:33 | tetsuro | jaypipes: I put your proposal into alternative section. The proposal sounds really attractive to me. But it is not kindful to operators, I guess. | |
| 14:54:47 | jaypipes | tetsuro: the proposal to just determine the shared pin set by subtracting the dedicate pin set from vcpu_pin_set? | |
| 14:55:44 | tetsuro | jaypipes: right. | |
| 14:56:07 | jaypipes | tetsuro: gotcha. ok, will re-review this morning :) | |
| 14:56:19 | tetsuro | thanks in advance! | |
| 14:58:05 | jaypipes | tetsuro: no problemo :) | |
| 15:01:19 | bauzas | melwitt: mriedem: maybe worth discussing Rambo's spec at the PTG during the cinder/nova cross-session on Thurs ? | |
| 15:02:13 | bauzas | or we could discuss the design before that, but if after, we loose the possibility to ask Cinder for a rebuild thingy | |
| 15:07:35 | melwitt | bauzas: that might be a good idea if we have time. since it's possible there's something they could do on their side to facilitate it | |
| 15:08:01 | bauzas | melwitt: for the moment I added L333 on https://etherpad.openstack.org/p/nova-ptg-rocky | |
| 15:08:22 | bauzas | melwitt: because mriedem wanted to discuss about all the rebuild problems we have | |
| 15:14:13 | mriedem | there is nothing i would like to discuss more than rebuild problems | |
| 15:15:49 | bauzas | mriedem: melwitt: maybe we could start a discussion about rebuild on wed if time allows | |
| 15:17:22 | bauzas | I know there is a huuuuuge Placement discussion that would occur most on all the afternoon, but maybe I should be tired after 3 hours ;) | |
| 15:20:55 | mriedem | if we talk about this rebuild + bfv thing, | |
| 15:21:01 | mriedem | it should either be with the cinder people in the room, | |
| 15:21:03 | mriedem | or on misc friday | |
| 15:21:10 | melwitt | +1 | |
| 15:21:11 | mriedem | b/c this is a misc thing | |
| 15:22:39 | bauzas | mriedem: melwitt: FWIW, I'll need to leave the room by Friday noon | |
| 15:22:47 | bauzas | for my flight :( | |
| 15:23:14 | bauzas | but yeah, we could discuss about rebuilding volume-backed stuff with Cinder folks | |
| 15:23:16 | bauzas | anyway | |
| 15:23:40 | mriedem | we'll be sure to schedule all cheese-related discussions for friday morning | |
| 15:26:12 | dansmith | jaypipes: I approved that spec | |
| 15:26:25 | dansmith | jaypipes: you can repay me by approving this compute rpc compat drop, which I know you want anyway: https://review.openstack.org/#/c/543580/ | |
| 15:27:17 | openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813 | |
| 15:31:15 | bauzas | mriedem: on an interop concern, should we accept other APIs but the ones we have in-tree ? https://bugs.launchpad.net/nova/+bug/1751036 | |
| 15:31:15 | openstack | Launchpad bug 1751036 in OpenStack Compute (nova) "ConfigFileValueError when defining enabled_apis" [Undecided,New] | |
| 15:31:19 | bauzas | I tend to say no | |
| 15:36:10 | melwitt | mriedem: please double check me on the rc3 tag https://review.openstack.org/#/c/547031/1/deliverables/queens/nova.yaml | |
| 15:37:58 | mriedem | efried: jaypipes: dansmith: left a comment in that spec https://review.openstack.org/#/c/546009/ - fine to amend with some details | |
| 15:39:07 | mriedem | melwitt: done | |
| 15:39:32 | mriedem | melwitt: also that commit hash might end up being a merge commit | |
| 15:39:41 | mriedem | that's why we need the depends-on | |
| 15:39:59 | melwitt | mriedem: thanks. so if I keep the commit hash the same + depends-on, it will work fine? | |
| 15:40:37 | mriedem | not if we need to use a merge commit | |
| 15:40:47 | mriedem | i would just add the depends-on and then WIP | |
| 15:41:02 | mriedem | don't want to get caught by the old smcginnis trigger finger | |
| 15:41:15 | melwitt | oh okay. I thought I had seen you propose a commit hash ahead of merge before but maybe I made that up | |
| 15:41:29 | smcginnis | :) | |
| 15:41:31 | mriedem | i generally do, and the job fails until i fix it up | |
| 15:41:36 | melwitt | thanks | |