| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-25 | |||
| 15:29:05 | mriedem | dansmith: i'm not intentionally holding it up until after the forum. i wanted to get agreement on the plan prior to the forum, start POC code, and present the plan at the forum to get feedback from ops. | |
| 15:29:05 | dansmith | sean-k-mooney: well, since we're not claiming the numa type resources in that claim, we'll still be able to race with filters choosing the same thing for multiple concurrent builds | |
| 15:29:12 | mriedem | dansmith: i just haven't read the latest revision yet | |
| 15:29:17 | dansmith | mriedem: yeah, I'd really like to see code before forum | |
| 15:29:38 | sean-k-mooney | dansmith: yes that is true | |
| 15:29:39 | dansmith | mriedem: okay I should take a pass through it.. maybe we can merge the spec this week and get some code going before forum if you're cool with it | |
| 15:30:00 | mriedem | i am cool with progress yes | |
| 15:30:11 | mriedem | remember, i like rush | |
| 15:30:38 | sean-k-mooney | dansmith: you should be able to vailidate that before calling to the host though. ie. if you can claim the numa page allocation abourt and retry | |
| 15:31:10 | sean-k-mooney | mriedem: is that late abort and retry the issue you were commenting on? | |
| 15:31:39 | mriedem | sean-k-mooney: we don't have allocations for numa yet | |
| 15:31:41 | mriedem | in placement | |
| 15:31:58 | mriedem | i believe that's nested resource providers territory | |
| 15:32:13 | sean-k-mooney | oh ok i understand the gap now. ya that was covered by nested providers | |
| 15:32:43 | mriedem | so if you're an NFV shop and we don't have nested resource providers yet, then you'll need to enable retries yes | |
| 15:32:56 | mriedem | which means you're not using the whole cells v2 superconductor deployment model | |
| 15:33:04 | bauzas | mriedem: dansmith: I can start pushing code, fersür | |
| 15:33:48 | bauzas | the spec, IHMO, is in a good shape with good work items, so that's a cool start | |
| 15:34:08 | bauzas | now that said, I need to disappear for now | |
| 15:34:18 | bauzas | ttyl folks | |
| 15:39:08 | openstackgerrit | Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753 | |
| 15:40:15 | dansmith | sean-k-mooney: if we ran filters twice or something maybe | |
| 15:41:01 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 15:42:19 | openstackgerrit | Chris Dent proposed openstack/nova master: Register osapi_compute when nova-api is wsgi https://review.openstack.org/457283 | |
| 15:42:20 | openstackgerrit | Chris Dent proposed openstack/nova master: Add pbr-installed wsgi application for metadata api https://review.openstack.org/459413 | |
| 15:46:22 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801 | |
| 15:52:24 | sean-k-mooney | dansmith: perhaps though honestly if the only main gap is the nfv constartisn of numa,cpu pinning,hugepages and pci devices it sounds like that will be addressed in time by nested resouce providers. | |
| 15:52:41 | dansmith | sean-k-mooney: indeed, that's the plan | |
| 15:52:58 | sean-k-mooney | dansmith: a validate call to the filters that accpeted a claim would also work but its a lot of work that may not be worth it | |
| 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: api-ref: move createBackup to server-actions https://review.openstack.org/452971 | |
| 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: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: explain payload inheritance in notification devref https://review.openstack.org/453667 | |
| 16:07:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | |
| 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 ^ | |