| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-08 | |||
| 12:00:36 | kashyap | (Particularly sparse on API usage) | |
| 12:03:59 | openstackgerrit | Merged openstack/nova master: [Docs] Change the server query parameter display into a list. https://review.opendev.org/657624 | |
| 13:05:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none' https://review.opendev.org/657796 | |
| 13:15:20 | efried | aspiers: discussing os-resource-classes build failure with stephenfin in -placement | |
| 13:17:15 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Use py3 for docs build https://review.opendev.org/657800 | |
| 13:17:26 | efried | aspiers: ^ might fix | |
| 13:17:30 | efried | will rebase your patch on that. | |
| 13:17:54 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Add MEM_ENCRYPTION_CONTEXT resource class https://review.opendev.org/655665 | |
| 13:18:33 | efried | aspiers: like that ^ | |
| 13:30:34 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Propose standard ACCELERATOR_FPGA resource class https://review.opendev.org/657464 | |
| 13:31:51 | openstackgerrit | Merged openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock of Future https://review.opendev.org/650171 | |
| 13:39:00 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Use py3 for all* tox targets https://review.opendev.org/657800 | |
| 14:13:15 | openstackgerrit | Adam Spiers proposed openstack/os-resource-classes master: Add missing upper-constraints.txt to deps in docs testenv https://review.opendev.org/657815 | |
| 14:26:40 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Add MEM_ENCRYPTION_CONTEXT resource class https://review.opendev.org/655665 | |
| 14:32:12 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: add scope check, see tests catch the change https://review.opendev.org/657823 | |
| 14:37:29 | openstackgerrit | Merged openstack/os-resource-classes master: Add missing upper-constraints.txt to deps in docs testenv https://review.opendev.org/657815 | |
| 14:41:34 | openstackgerrit | Merged openstack/os-resource-classes master: Use py3 for all* tox targets https://review.opendev.org/657800 | |
| 14:42:47 | mriedem | crap lee is out for 2 weeks huh | |
| 14:44:32 | mriedem | stable is bizzusted | |
| 14:47:37 | openstackgerrit | Merged openstack/os-resource-classes master: Add MEM_ENCRYPTION_CONTEXT resource class https://review.opendev.org/655665 | |
| 14:52:21 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Propose standard ACCELERATOR_FPGA resource class https://review.opendev.org/657464 | |
| 15:08:54 | kashyap | Wonder if the Etherpad content from -train-2 and -train-3 is replicated back to to -train | |
| 15:09:06 | kashyap | Okay ... none of the Etherpads load for me. | |
| 15:13:12 | mriedem | fyi we can't land changes on stable until https://review.opendev.org/#/q/topic:bug/1828244+(status:open+OR+status:merged) | |
| 15:34:02 | openstackgerrit | Merged openstack/nova master: Revert "Fix target_cell usage for scatter_gather_cells" https://review.opendev.org/653894 | |
| 16:28:37 | mriedem | dansmith: can you send this to its maker https://review.opendev.org/#/c/655800/ | |
| 16:35:26 | dansmith | yar | |
| 17:31:30 | melwitt | dansmith: I need some help/ideas on the cell scatter gather native thread patch https://review.opendev.org/650172 if you could take another look please | |
| 17:32:40 | dansmith | melwitt: related to the native threads sticking around that bogdan commented on? | |
| 17:33:48 | melwitt | dansmith: yeah. and gibi's comment. afaict, if the method being run in the thread is going to hang forever, we can't do anything about that. if there is a way, I don't know it | |
| 17:34:12 | melwitt | and figured if anyone would know, it would be you :) | |
| 17:34:44 | dansmith | well, hanging forever and leaving them around isn't an option | |
| 17:35:01 | dansmith | I saw bogdan's comment via email but haven't really looked in detail | |
| 17:35:03 | dansmith | but will in a bit | |
| 17:35:24 | melwitt | dansmith: I'm also wondering about gibi's question about not using the ThreadPoolExecutor in the unit and func tests. I got a deadlock in the unit tests but not the func tests. and I didn't know how to get around it | |
| 17:37:35 | melwitt | dansmith: yeah, I guess I'm confused by what can be done about it. if the call inside the thread is never going to return, how is that different than a non-thread call not ever returning | |
| 17:37:53 | dansmith | you mean an eventlet thread never returning? | |
| 17:38:04 | dansmith | because eventlet threads aren't really threads and you can cancel them at any io sync point | |
| 17:38:12 | melwitt | I mean the method inside, like InstanceList.get_by_filters. if that were to hang forever | |
| 17:38:46 | melwitt | because that's the only way the thread will stay around indefinitely, right? | |
| 17:40:02 | dansmith | well, whatever the thread is blocked on, yeah | |
| 18:09:07 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/648662 | |
| 18:42:31 | openstackgerrit | Surya Seetharaman proposed openstack/nova-specs master: Support server power state update through external event https://review.opendev.org/636132 | |
| 19:04:54 | openstackgerrit | Rodrigo Barbieri proposed openstack/nova master: Add functional confirm_migration_error test https://review.opendev.org/657870 | |
| 19:09:07 | dansmith | tssurya: couple things on that spec | |
| 19:09:14 | dansmith | tssurya: but I think it's going the right direction | |
| 19:19:32 | artom | aspiers, still around? You got volunteered for compute capabilities as traits during PTG, right? | |
| 19:19:47 | artom | http://lists.openstack.org/pipermail/openstack-discuss/2019-May/005906.html doesn't make much sense to me since my knowledge of placement is pretty slim | |
| 19:20:43 | artom | I want to help out with that because it helps my baby device tagging :) | |
| 19:23:47 | mriedem | kinda sucks we lost our entire ptg etherpad(s) | |
| 19:27:15 | artom | The main ones, at any rate :( | |
| 19:27:37 | artom | The governance one is still going strong :D | |
| 19:31:09 | sean-k-mooney | mriedem: i think i have a copy of the main one on my laptop | |
| 19:31:18 | sean-k-mooney | i backed up the -3 etherpad | |
| 19:41:15 | artom | sean-k-mooney, I'm specifically not asking you any of the questions that I have because I know you're on PTO and you shouldn't be here :D | |
| 19:43:54 | efried | artom, mriedem, sean-k-mooney: I copied -3 to https://etherpad.openstack.org/p/nova-ptg-train-4 | |
| 19:45:08 | efried | artom: tell me more about device tagging | |
| 19:45:10 | sean-k-mooney | artom: im technically playing a video game while listening to an audio book so im more or less on pto | |
| 19:45:28 | artom | "playing a video game while listening to an audio book" | |
| 19:45:31 | artom | Kids these days | |
| 19:45:58 | artom | efried, basically https://bugs.launchpad.net/nova/+bug/1817927 | |
| 19:46:00 | openstack | Launchpad bug 1817927 in OpenStack Compute (nova) "device tagging support is not checked during move operations" [Undecided,New] | |
| 19:46:09 | artom | And since I wrote the original device tagging stuff, I figure it's mine to fix | |
| 19:46:14 | sean-k-mooney | what can i say i get board easily. that our im optimising my time by multi tasking | |
| 19:46:55 | artom | So if the fix is using compute capabilities traits with whatever placement pre-filter black magic (excuse the ignorance) is needed, I'm on board with that, and will help | |
| 19:47:39 | efried | artom: Cool. Yes, the effort should help. In fact, I would expect we should get all of that for free once the framework is fixed up. | |
| 19:48:26 | artom | efried, right, and so that was basically my question to aspiers. IIUC he's been volunteered to fix up said framework, so wondering if/how he needed help | |
| 19:49:28 | efried | artom: He and I spent a couple of hours Saturday afternoon going over some of the background. A careful reading of that thread - especially gibi's response - should be pretty enlightening. | |
| 19:49:53 | artom | efried, ah, I was flying home then. Err, which specific thread :) | |
| 19:50:15 | artom | Summary: Implicit trait-based filters? | |
| 19:50:26 | efried | artom: Yes, the one you linked above :P | |
| 19:51:02 | artom | *snerk* *sigh* I amaze myself sometimes :( | |
| 19:53:23 | efried | The main thrust is: RequestSpec.requested_resources is currently only used for the numbered request groups coming from port bandwidth pieces. It should be accumulating *all* of the request groups - from flavor, image, request filters, and anything else that needs to inform the GET /a_c call. But apparently putting the unnumbered group in there is hard. | |
| 19:55:07 | artom | Full disclosure: I don't know the difference between a numbered request group and an unnumbered one. Or even what a request group is. I'll read the placement api-ref. | |
| 20:13:05 | tssurya | dansmith: thanks for the review :) I'll update it | |
| 20:24:06 | eandersson | The upgrade checker is confusing when it comes to Ironic Flavor Migration. The Upgrade suggests this is something that need to be done. | |
| 20:24:49 | eandersson | But the nova-manage documentation tell you do not do it https://docs.openstack.org/nova/pike/cli/nova-manage.html | |
| 20:26:22 | eandersson | Guessing that the key is this? | |
| 20:26:23 | eandersson | > This routine is provided only for advanced users that may be skipping the 16.0.0 Pike release, never able to run services normally at the Pike level | |
| 20:26:41 | eandersson | (e.g. for fast-forward?) | |
| 20:30:38 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec to pre-filter disabled computes with placement https://review.opendev.org/657884 | |
| 20:31:04 | mriedem | tssurya: dansmith: efried: ^ tis written but with lots of todo/question/alternates inline | |
| 20:31:36 | mriedem | eandersson: "never able to run services normally at the Pike level" is for FFU yes | |
| 20:32:15 | mriedem | eandersson: when you restart the ironic compute service it should migrate the embedded instance.flavor for each node it's managing automagically | |
| 20:32:28 | aspiers | artom: I learnt the difference on Sat, which is *almost* long enough ago for me to have forgotten it ;-) | |
| 20:33:06 | mriedem | https://opendev.org/openstack/nova/src/tag/18.0.0/nova/virt/ironic/driver.py#L545 | |
| 20:33:22 | artom | aspiers, hah :) So yeah, do you think you want/need help (from admittedly another placement newbie)? | |
| 20:33:48 | aspiers | artom: probably will, but have to catch up with a bunch of other stuff before I get to this | |
| 20:34:25 | mriedem | eandersson: if you go down that route, note there are some open issues https://review.opendev.org/#/c/637217/ | |
| 20:34:25 | aspiers | including filing expenses and sending a report about the self-healing SIG :-o | |
| 20:35:02 | artom | Reason I'm pestering is we have internal tracking requirements, so if I'm committing to helping with this thing, I'll need to file trackers for it sooner rather than later | |
| 20:36:34 | aspiers | artom: Got it. TBH I probably won't know how much help I need (if any) until I understand it better | |
| 20:37:24 | artom | aspiers, totally fair | |
| 20:37:46 | artom | In any case, nothing has to be decided tonight, and I have my own share of reading to do :) | |
| 20:37:52 | artom | And kids to fetch from daycare. | |
| 20:37:52 | aspiers | +1 | |
| 20:37:55 | openstackgerrit | Surya Seetharaman proposed openstack/nova-specs master: Support server power state update through external event https://review.opendev.org/636132 | |
| 20:45:40 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec to pre-filter disabled computes with placement https://review.opendev.org/657884 | |
| 21:59:16 | sorrison | mriedem: Yeah didn't think of that but that seems ok to me and desired. Controlling access to list other projects instances seems like it should be separate to the other filters I think | |
| 22:01:00 | mriedem | alright, sounds like no change necessary then | |