| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-22 | |||
| 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 | openstack | Launchpad bug 1751036 in OpenStack Compute (nova) "ConfigFileValueError when defining enabled_apis" [Undecided,New] | |
| 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: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 | bauzas | mriedem: back to my original concern, I don't see why Nova should support other out-of-tree APIs | |
| 15:47:56 | mriedem | it's still maintained | |
| 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) | |
| 15:55:03 | bauzas | mriedem: back to the ec2api, ansiwen told us that it was barely maintained | |
| 15:55:19 | mriedem | bauzas: done | |
| 15:55:24 | bauzas | so maybe it's some misunderstanding from me | |
| 15:55:38 | mriedem | well the occi nova ooi or whatever code is in github | |
| 15:55:47 | mriedem | so it's available, which is fine | |
| 16:02:14 | efried | mriedem: What dansmith said. It doesn't make sense for me to say "return me a list of resource providers where the union of all their traits is [this list]". Because I would either get all of them or none of them. An allocation candidate is a list of *groups* of resource providers, where each *group* must satisfy the list of traits. s/traits/resources/ and the answer is the same. | |
| 16:02:47 | efried | I.e. the difference between rp and ac for traits is the same as the difference between rp and ac for resources, if that makes any sense :) | |
| 16:03:14 | dansmith | efried: right, that's how I think of it exactly | |
| 16:06:47 | mriedem | i get it fellas | |
| 16:06:55 | mriedem | and it was discussed in comments in the spec, | |
| 16:07:00 | mriedem | but for posterity sake, | |
| 16:07:02 | mriedem | let's doc that in the spec | |
| 16:07:06 | mriedem | so a year from now we can say why | |
| 16:07:13 | efried | mriedem: Roger that, will do. | |
| 16:07:39 | dansmith | mriedem: if you mean saying "because a_c is $description, we behave this way on this..." then sure | |
| 16:08:13 | dansmith | I don't really want to have to fully explain a_c every time we do something like this, but... | |
| 16:08:30 | mriedem | you can explain it now in a doc so we can point people to it, | |