Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-08
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
22:01:06 mriedem no work works for me
22:09:41 openstackgerrit Eric Fried proposed openstack/os-traits master: Introduce traits for NUMA and NIC subtree affinity https://review.opendev.org/657898

Earlier   Later