| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-25 | |||
| 15:53:17 | dansmith | sean-k-mooney: yeah, I'd rather focus on the right fix | |
| 15:53:35 | mriedem | i haven't thought yet about how we replace the affinity filters with placement | |
| 15:54:03 | mriedem | i guess sum the reqs for all members of an affinity group and find a provider for them | |
| 15:54:23 | sean-k-mooney | the affinity filters in general or the server group affinity filters | |
| 15:54:27 | dansmith | mriedem: needs more placement support I think.. jaypipes has waxed poetic about it in the past | |
| 15:54:29 | mriedem | server group | |
| 15:54:44 | sean-k-mooney | mriedem: traits maybe? | |
| 15:55:00 | mriedem | anti-affinity, you could get RPs where len(RPs) > anti-affinity server group members, and then have to fit each member into a unique provider | |
| 15:55:04 | mriedem | but idk | |
| 15:55:12 | mriedem | sean-k-mooney: traits aren't the answer to everything :) | |
| 15:55:26 | dansmith | lol | |
| 15:55:36 | mriedem | when i give the project roadmap thing for R, | |
| 15:55:53 | mriedem | i think i'm going to say, "fix traits which has become the new bdm/quotas/local delete quagmire" | |
| 15:56:27 | dansmith | lol | |
| 15:56:31 | mriedem | roadmap for R: turn traits into CIMaaS | |
| 15:56:36 | jaypipes | mriedem: server groups are on my list of "figure out how to port this over to placement" things, yeah.... | |
| 15:56:39 | sean-k-mooney | true i was thinking more of taging a resouce prvider in some way and saying only condider providers with this tag | |
| 15:56:55 | jaypipes | mriedem: for the immediate future, they will need to stay in the scheduler filters/weighers. | |
| 15:57:00 | mriedem | sean-k-mooney: that's more like an aggregate to me | |
| 15:57:06 | jaypipes | mriedem: right. | |
| 15:57:08 | mriedem | jaypipes: yeah that's fine | |
| 15:57:26 | jaypipes | mriedem: the problem with server groups has *always* been how to handle changes in the group over time. | |
| 15:57:43 | sean-k-mooney | mriedem: yes more of dynamcally created one though | |
| 15:58:07 | mriedem | sean-k-mooney: now you're just talking sci fi | |
| 15:58:13 | mriedem | :) | |
| 15:58:19 | sean-k-mooney | :) | |
| 15:58:30 | mriedem | don't forget we have to add support for migrating affinity groups as a unit | |
| 15:59:28 | sean-k-mooney | mriedem: that would be an entirely new feature right? currently you cant migrate a group just the instance of a group | |
| 16:00:04 | sean-k-mooney | that is going to create a lot of headaches if one of the member fails to migrate | |
| 16:00:14 | mriedem | sean-k-mooney: yes entirely new | |
| 16:00:29 | mriedem | sean-k-mooney: let me find some light reading for your evening | |
| 16:01:02 | mriedem | sean-k-mooney: https://review.openstack.org/#/c/440580/ | |
| 16:01:35 | mriedem | basically we can spew out scheduler hints from the API so external systems can attempt to coordinate things, or we build something into nova | |
| 16:02:41 | sean-k-mooney | i see are the sceduler hints corrent stored in the db and this is just exposing them or are we also going start storeing them | |
| 16:02:50 | sean-k-mooney | just reading over it now | |
| 16:03:00 | mriedem | they are stored | |
| 16:03:03 | mriedem | we don't expose them | |
| 16:03:27 | openstackgerrit | Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753 | |
| 16:03:39 | sean-k-mooney | ah first line in promblem discription well that make upgrades simplar as we still have the info for exisint server groups | |
| 16:05:09 | sdague | dansmith / jaypipes - easy +2 | |
| 16:05:12 | sdague | https://review.openstack.org/#/c/459735/ | |
| 16:06:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate Multinic, floatingip action and os-virtual-interface API https://review.openstack.org/457181 | |
| 16:06:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: move createBackup to server-actions https://review.openstack.org/452971 | |
| 16:06:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: 2.43: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443 | |
| 16:07:29 | mriedem | sdague: https://review.openstack.org/#/c/457181/ | |
| 16:07:30 | mriedem | for 2.44 | |
| 16:07:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | |
| 16:07:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | |
| 16:07:51 | sdague | mriedem: loking | |
| 16:08:32 | openstackgerrit | Matt Riedemann proposed openstack/nova master: 2.45: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443 | |
| 16:09:42 | jaypipes | sdague: done | |
| 16:11:21 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801 | |
| 16:14:01 | sdague | mriedem: done | |
| 16:14:09 | sdague | mriedem: one issue on your 2.45 patch I think | |
| 16:14:16 | sdague | if you want to go look, just left review comments | |
| 16:17:29 | mriedem | ok | |
| 16:19:02 | mriedem | replied, i can make a change | |
| 16:20:00 | sdague | mriedem: cool | |
| 16:25:04 | efried | mriedem These, whenever you get a moment: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/powervm-nova-compute-driver+is:mergeable | |
| 16:25:32 | efried | please and thanks | |
| 16:25:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | |
| 16:25:52 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | |
| 16:28:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: 2.45: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443 | |
| 16:28:10 | mriedem | sdague: ^ | |
| 16:34:46 | sdague | mriedem: you want to also assert the body is empty? We probably should have done that before | |
| 16:35:04 | sdague | which I was thinking, but apparently forgot to write | |
| 16:35:59 | mriedem | i suppose i can do that | |
| 16:36:54 | sdague | just so that the 2.45 code doesn't slip back into there | |
| 16:37:15 | sdague | then, I'm +2 for reals | |
| 16:44:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: 2.45: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443 | |
| 16:44:24 | mriedem | sdague: done ^ | |
| 16:54:37 | openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | |
| 16:55:12 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP/PoC: nova.utils.get_service_url(group) https://review.openstack.org/458257 | |
| 16:55:52 | efried | mordred jaypipes johnthetubaguy If y'all have a few minutes, I'd like to chat about the direction of this ^^ | |
| 16:56:04 | jaypipes | efried: yuppers. it's on my list for today. | |
| 16:56:34 | efried | jaypipes Cool; just updated with some fixes, did some live testing, and I think it's working the way I want, as far as it goes. | |
| 16:57:01 | efried | Before I get any further, I've got some outstanding questions. | |
| 16:57:04 | johnthetubaguy | efried: yeah, my take was lets make all the bits of Nova consistent in our usage, where possible, but it looks like mordred has some alternative ideas | |
| 16:57:29 | efried | johnthetubaguy I think I've isolated mordred's alternatives down to the guts of one private method. | |
| 16:57:29 | johnthetubaguy | efried: I am about to go and make my dinner in a few mins though, could answer some quick questions | |
| 16:57:47 | efried | What I gathered was that those suggestions involved pieces that weren't quiiite ready for primetime usage. | |
| 16:57:58 | johnthetubaguy | efried: I got a similar vibe | |
| 16:57:59 | efried | So I tried to make it easy for us to swap those in as/if/when they become ready. | |
| 16:58:10 | johnthetubaguy | sounds like a good way forward, on the face of it | |
| 16:58:35 | johnthetubaguy | if we could look like we will use the same config for both approaches, that would be good | |
| 16:58:53 | efried | johnthetubaguy Oh, yeah, pretty sure that's the deal. | |
| 16:58:59 | efried | But mordred may refute :( | |
| 16:59:16 | johnthetubaguy | I think the config being consistent was my worry | |
| 16:59:21 | johnthetubaguy | right now we are all over the place | |
| 16:59:22 | efried | Please note (my) review comments on older patch sets, which enumerate some of my questions/concerns. Also some in the code comments & commit message. | |
| 16:59:24 | openstackgerrit | Kaitlin Farr proposed openstack/nova master: Parse algorithm from cipher for ephemeral disk encryption https://review.openstack.org/446072 | |
| 17:00:11 | efried | johnthetubaguy Course, the "all over the place" conf opts will need to be addressed individually, but I think I have a neat approach for handling that. | |
| 17:00:27 | efried | might be a bit too "out there"... | |
| 17:02:43 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | |
| 17:02:44 | openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | |
| 17:02:44 | openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | |
| 17:02:46 | openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | |
| 17:02:47 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | |
| 17:03:37 | johnthetubaguy | efried: I added some comments, not sure if they help | |
| 17:03:43 | efried | johnthetubaguy Thanks, looking... | |