| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-22 | |||
| 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 | |
| 15:52:38 | bauzas | we have a GCE proxy ? | |
| 15:52:48 | mriedem | bauzas: the gce api proxy is way old | |
| 15:52:49 | dansmith | mriedem: which is kinda why we ended up adding a_c I think | |
| 15:53:15 | edleafe | bauzas: yeah, we only support the openstack api, so the extra directory level in the project is not needed | |
| 15:53:30 | bauzas | anyhow | |
| 15:53:34 | bauzas | let's triage the bug as Invalid | |
| 15:53:45 | bauzas | and ask them to run their own WSGI service | |
| 15:54:24 | bauzas | (FWIW, I know those OCCI folks since 2013, I'm not surprised they never went upstream) | |