| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-24 | |||
| 15:58:58 | mriedem | ildikov: yeah i already commented on it | |
| 15:59:00 | johnthetubaguy | mriedem: I don't get a vote in that, its some other sub team of the TC looking at that, I believe | |
| 15:59:36 | ildikov | mriedem: thanks | |
| 16:00:00 | johnthetubaguy | but yeah, better to split to start, and combine later if needed, I guess | |
| 16:00:28 | leakypipes | mriedem: agreed. | |
| 16:00:55 | mriedem | ok done http://forumtopics.openstack.org/ | |
| 16:02:06 | mriedem | huh, apparently if you rename it's a new topic | |
| 16:02:10 | mriedem | and you can't delete a topic | |
| 16:02:12 | mriedem | because now i have 3 | |
| 16:02:29 | mriedem | EmilienM: ttx: do you know how to delete http://forumtopics.openstack.org/cfp/details/28 ? | |
| 16:02:54 | EmilienM | mriedem: Edit and Delete | |
| 16:03:08 | mriedem | ah nice | |
| 16:03:08 | mriedem | thanks | |
| 16:03:48 | mriedem | leakypipes: ah i see you created http://forumtopics.openstack.org/cfp/details/26 so that probably covers what i was going to submit | |
| 16:03:51 | leakypipes | cdent: I'm sorry, I still don't understand what you're asking for. | |
| 16:04:08 | leakypipes | mriedem: ya | |
| 16:04:30 | mriedem | we probably need a separate topic for claims in the scheduler though, i'm not sure yet | |
| 16:04:40 | mriedem | but we didn't talk about it at the ptg | |
| 16:04:49 | leakypipes | mriedem: ++ | |
| 16:05:07 | cdent | leakypipes: maybe sean-k-mooney can take a stab at describing it? otherwise, if you're still curious I'll try a very quick spike at it after I rebase some stuff to fix merge conflicts after that float fix | |
| 16:05:33 | leakypipes | cdent: well, lemme try one more thing and push it. | |
| 16:06:02 | cdent | k | |
| 16:07:50 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | |
| 16:09:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | |
| 16:09:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Set 'size' during image create with FakeImageService https://review.openstack.org/449702 | |
| 16:09:43 | mriedem | sdague: ^ you'd probably appreciate that bottom change | |
| 16:11:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | |
| 16:11:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Set 'size' during image create with FakeImageService https://review.openstack.org/449702 | |
| 16:13:05 | mriedem | superdan: pep8 https://review.openstack.org/#/c/449640/ | |
| 16:13:31 | superdan | mriedem: yeah I have it fixed, as waiting for the run to finish before I pushed it up again | |
| 16:13:34 | superdan | and was on a call | |
| 16:13:38 | mriedem | ok | |
| 16:13:39 | sdague | johnthetubaguy: while I'm doing that, you want to hit this easy thing - https://review.openstack.org/#/c/448019/ | |
| 16:13:40 | superdan | here | |
| 16:13:47 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove legacy regeneration of RequestSpec in MigrationTask https://review.openstack.org/449640 | |
| 16:15:05 | johnthetubaguy | sdague: got my head in the policy spec still, adding that to the queue | |
| 16:17:58 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625 | |
| 16:17:59 | openstackgerrit | Chris Dent proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201 | |
| 16:17:59 | openstackgerrit | Chris Dent proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | |
| 16:18:01 | openstackgerrit | Chris Dent proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199 | |
| 16:19:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Remove dead code https://review.openstack.org/449708 | |
| 16:30:22 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in tenant_networks.py https://review.openstack.org/449714 | |
| 16:34:08 | cdent | leakypipes: while waiting for some tests to run I made a hack to do demo what I was thinking of. it can probably cleaned up monstrously, but shows the idea: https://github.com/cdent/pony | |
| 16:35:13 | leakypipes | cdent: lol, very similar to what I put together locally. | |
| 16:35:29 | cdent | leakypipes: ah, good, so we are understanding one another then? | |
| 16:36:06 | leakypipes | cdent: would seem so.\ | |
| 16:36:15 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/446674 | |
| 16:36:24 | cdent | praise be or something | |
| 16:40:25 | openstackgerrit | Sean Dague proposed openstack/nova-specs master: New intro https://review.openstack.org/449722 | |
| 16:40:33 | sdague | johnthetubaguy: what about something like that? | |
| 16:42:14 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Idempotent PUT /resource_classes/{name} https://review.openstack.org/448791 | |
| 16:46:07 | johnthetubaguy | sdague: looking | |
| 16:48:41 | johnthetubaguy | sdague: thanks I will roll that into the patch I have cooking at the moment | |
| 17:02:35 | tojuvone | johnthetubaguy, Hi I had other things going like floorball. Looks that somehow I did not receive your comments in the patch. | |
| 17:04:27 | johnthetubaguy | tojuvone: added a follow on patch, but I don't really like my ideas in there now, after catch up with sdague on the direction he was suggesting before | |
| 17:05:31 | tojuvone | johnthetubaguy, ok, thanks | |
| 17:06:58 | openstackgerrit | Anthony PERARD proposed openstack/nova master: Add exclusion list for tempest for a libvirt+xen job https://review.openstack.org/449687 | |
| 17:22:22 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in simple_tenant_usage.py https://review.openstack.org/449742 | |
| 17:27:57 | EmilienM | can we nova ephemeral disk using rbd backend be encrypted with key manager like Barbican? I know it's doable with LVM but not sure about rbd | |
| 17:29:22 | cdent | BobBall_: is this still a thing: https://review.openstack.org/#/c/394857/ | |
| 17:39:36 | tojuvone | johnthetubaguy, sdague, bauwser Thanks and sorry if wasted your time around the host maintenance. | |
| 17:41:15 | cfriesen | when running "openstack server migrate" will that do a cold migration? If so, is there any way to do a live migration without specifying the destination compute node? | |
| 17:41:26 | leakypipes | cdent, figleaf, alex_xu, sean-k-mooney: OK, try this on for size: https://review.openstack.org/448282 and https://review.openstack.org/448283 | |
| 17:41:35 | sdague | tojuvone: I mostly don't want to waste your time iterating here if we're not going to do it | |
| 17:41:51 | sdague | and I feel like we said we weren't going to do this a while ago | |
| 17:43:12 | tojuvone | sdague, Yes, agree | |
| 17:44:39 | johnthetubaguy | cfriesen: do you try passing host=None or null? it might be an openstack client limitation (https://developer.openstack.org/api-ref/compute/?expanded=live-migrate-server-os-migratelive-action-detail#live-migrate-server-os-migratelive-action) | |
| 17:46:29 | johnthetubaguy | tojuvone: sdague: I agree it probably makes sense for OpenStack to support that use case, its just the preferred implementation/architecture has 99% or 100% of the code changes outside of Nova. | |
| 17:46:35 | tojuvone | sdague, good you stop it there again and can focus doing just what talked already back in Austin without wasting anybody time on something more | |
| 17:47:07 | johnthetubaguy | what we said in austin was? | |
| 17:48:07 | tojuvone | Well, if I recall was around to have something as simple as linking | |
| 17:48:48 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to nova/policies/shelve.py https://review.openstack.org/449763 | |
| 17:49:14 | sean-k-mooney | leakypipes: that looks really good to me. | |
| 17:49:23 | tojuvone | Everytim I just touched the review it went bigger and bigger | |
| 17:49:43 | leakypipes | sean-k-mooney: that's more in-line to what you were envisioning? | |
| 17:49:56 | sean-k-mooney | leakypipes: ya it is | |
| 17:50:41 | johnthetubaguy | tojuvone: right, I am not sure the link is useful enough by its-self, disabled host probably is enough for the state in Nova. | |
| 17:51:34 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Split api-ref topics per file https://review.openstack.org/449764 | |
| 17:51:35 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [WIP] Add inventories https://review.openstack.org/449765 | |
| 17:51:46 | sean-k-mooney | leakypipes: i still need to fully read trough the register and symbolize function implementations but while its a little more complicated then just a single files with hand coded constances i think is a nicer approch | |
| 17:52:26 | sean-k-mooney | leakypipes: hum they are even simpler then i taught | |
| 17:52:42 | leakypipes | sean-k-mooney: yeah, definitely not rocket science :) | |
| 17:52:44 | tojuvone | johnthetubaguy, if link is set, you know you should read it, if not system is up-to-date | |
| 17:53:22 | cfriesen | johnthetubaguy: looks like openstackclient is fubar'd. there's no way to do a live migration without specifying the dest, and the help text says the default is shared but it's actually defaulting to block. | |
| 17:53:30 | cdent | leakypipes: I've left comments on both. acceptable as is, but still a bit noisy ;) | |
| 17:53:31 | tojuvone | but yes if it is always set, it doesn't help much, but know where to look more detailed info | |
| 17:53:40 | leakypipes | cdent: responded. | |
| 17:53:41 | figleaf | leakypipes: heading out soon, so only a quick perusal so far. I think this is a step closer to what I envisioned, but it still requires more hardcoding than I imagined | |
| 17:54:08 | figleaf | IOW, auto-discovery of the modules and packages, vs. spelling them out | |
| 17:54:55 | leakypipes | figleaf: k. well, I'm happy to check out alternative approaches if you want to hack together something this weekend? | |
| 17:54:56 | cdent | leakypipes: I think a list (without the register) woud make things a lot easier to read: the focus would be on the info (and the associated comments) rather than the code | |
| 17:55:24 | leakypipes | cdent: meh, maybe? :) | |
| 17:55:39 | cdent | it would be more pythonic :) | |
| 17:56:41 | figleaf | leakypipes: ok, don't know if I'll have time, but by Monday fo sho | |
| 17:58:26 | sdague | tojuvone: so... I'm not sure if this helps you at all. But I did a quick exercise this morning of how I was imaging doing these flows outside of nova - https://etherpad.openstack.org/p/host-maintenance-not-in-nova | |
| 17:59:21 | sdague | it really feels like there is a need for a maintenance/status system which users can poll / subscribe to, but I think it lives on it's own | |
| 18:01:43 | sean-k-mooney | leakypipes: im just heading home but ill take a look at it again and play with it over the weekend. | |
| 18:02:01 | leakypipes | sean-k-mooney: cool, thanks! | |
| 18:02:03 | johnthetubaguy | cfriesen: we need to get osc onto the latest microversion, where we default to auto detecting if we need to do block migration or not | |
| 18:02:09 | johnthetubaguy | cfriesen: among other things! | |