| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-22 | |||
| 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 | |
| 15:44:03 | openstackgerrit | Merged openstack/nova-specs master: Filter Resource Provider List for Traits https://review.openstack.org/546009 | |
| 15:44:08 | jaypipes | mriedem: k | |
| 15:45:30 | mriedem | bauzas: i'm sort of surprised we don't allow ec2 in enabled_apis? i'm not sure how the ec2api code works w/o that | |
| 15:46:53 | bauzas | mriedem: ec2api is somehow stale now AFAIK | |
| 15:47:05 | bauzas | ansiwen: around ? | |
| 15:47:23 | bauzas | ansiwen: see mriedem's point about ec2api not being in the enabled apis | |
| 15:47:31 | bauzas | if so, is it run separately? | |
| 15:47:56 | mriedem | it's still maintained | |
| 15:47:56 | bauzas | mriedem: back to my original concern, I don't see why Nova should support other out-of-tree APIs | |
| 15:48:26 | bauzas | if people want to run their own APIs, why not running a specific WSGI server et voilà ? | |
| 15:48:32 | bauzas | a new endpoint | |
| 15:49:31 | edleafe | bauzas: I've wanted to simplify the pathing to reflect that there is only one Nova API | |
| 15:49:48 | edleafe | bauzas: e.g., change nova/api/openstack/compute to just nova/api/compute | |
| 15:50:13 | edleafe | But alex_xu has pointed out there are compatibilty issues | |
| 15:50:24 | edleafe | with api-paste.ini | |
| 15:50:35 | bauzas | edleafe: so you pointed me that enabled_apis shouldn't accept more than just nova and metadata ? | |
| 15:50:52 | mriedem | what does the GCE API do? | |
| 15:50:53 | bauzas | s/pointed/agreed with ? | |
| 15:52:32 | bauzas | mriedem: mmm, WAT ? | |
| 15:52:34 | dansmith | mriedem: I guess I see that distinction as a fundamental difference between allocation_candidates and resource_providers | |
| 15:52:36 | mriedem | https://github.com/openstack/gce-api/blob/master/gceapi/cmd/api.py#L34 | |