Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-11-08
20:52:53 dansmith so you could also just say len(cache)==2, but.. checking that there is only one non-cell0 cell seems like a better assertion
20:53:34 mriedem wfm
20:53:41 dansmith other things would be wrong if those weren't the same, for sure, it just seems more of a positive thing
20:53:42 dansmith but that's just a detail, either way that optimization is easy
20:53:55 melwitt yeah. we could have a "is_single_cell" method. though I wonder how that works before the cache has been populated. I guess we know it has been before quota checks happen?
20:54:21 dansmith if the cache isn't populated then we have to populate it before we look at the cache
20:54:25 dansmith that's how all that code works
20:54:37 melwitt ok
20:54:43 dansmith so is_single_cell() makes a _load_cells() call first
21:05:24 artom This is weird, why is conductor failing with 'CantStartEngineError: No sql_connection parameter is established'
21:06:34 mriedem artom: it's trying to hit the API DB
21:06:36 mriedem and not configured for it
21:06:39 mriedem if you're using devstack
21:06:45 mriedem the cell conductor that is
21:06:54 artom mriedem, that's exactly it, it's almost creey ;)
21:07:12 mriedem e.g. resize fails on compute1, casts back to cell conductor which looks up the instance mapping and kablamo
21:07:14 mriedem known issue
21:07:19 artom It's a reschedule, the (presumably cell) conductor is trying to get the AZ
21:07:33 mriedem yup, i know the bug
21:07:35 mriedem sec
21:08:02 mriedem artom: linked from https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls
21:08:14 mriedem i have a fix for one of those
21:08:17 mriedem discussed at the ptg
21:08:17 artom Thanks :)
21:08:25 openstackgerrit Jack Ding proposed openstack/nova master: Add cache=none option for qemu-img convert https://review.openstack.org/616692
21:08:31 mriedem the other, the one you're hitting, is a bigger change b/c we have to pass the azs down to the cell
21:08:32 artom Actually also affects plain boot
21:08:34 mriedem on the Selection object
21:09:06 artom Hah, so I kinda need this fixed so that I can reproduce *another* bug correctly
21:09:27 mriedem artom: b/c of this right? https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L678
21:09:42 artom mriedem, yep
21:09:44 mriedem artom: well as a workaround just configure the [api_database]/connection in your nova-cell1.conf
21:09:47 mriedem and restart the cell conductor
21:09:57 artom Ah, true
21:10:22 mriedem i think devstack is the only thing out there that really does this split MQ thing
21:10:37 mriedem everyone else just runs with api db configured everywhere
21:10:59 artom Doesn't that kind of defeat the scalability point of the API db?
21:11:11 mriedem it's not really a scalability thing,
21:11:15 mriedem it's an isolation thing
21:11:39 artom OK - I just thought, if you're going to have all conductors talk to the api db, then why have a separate API db in the first place?
21:11:49 mriedem https://docs.openstack.org/nova/latest/user/cellsv2-layout.html#operations-requiring-upcalls
21:11:51 mriedem b/c dude
21:12:23 artom (Not being flippant on purpose or anything, genuinely trying to grok this)
21:13:06 mriedem b/c some things are in the api db schema like aggregates and cells, and some things are cell-specific, and we don't need to replicate that api db information across all cells
21:14:15 artom Ah, I see.
21:14:32 mriedem like cells v1 did
21:14:35 mriedem it had to sync everything
21:14:48 artom The API DB wasn't born out of a need to reduce load, it was born out of a need to keep a single source of truth for things that are common across all cells
21:14:59 artom It's the *cells DBs* that are the scalability stuff
21:15:07 mriedem yeah
21:15:36 artom Got it. Thanks :)
21:39:43 openstackgerrit Doug Hellmann proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724
21:39:43 openstackgerrit Doug Hellmann proposed openstack/nova master: Cap oslo.service to 1.32.0 (temporarily) https://review.openstack.org/616591
21:39:44 openstackgerrit Doug Hellmann proposed openstack/nova master: remove mocks of oslo.service private members https://review.openstack.org/616697
21:51:02 openstackgerrit Doug Hellmann proposed openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724
21:51:02 openstackgerrit Doug Hellmann proposed openstack/nova master: remove mocks of oslo.service private members https://review.openstack.org/616697
21:51:14 efried melwitt, mriedem: Y'all still around to approve the new version of the oslo.service mess?
21:52:59 melwitt efried: yeah
21:53:20 efried melwitt: First https://review.openstack.org/616697
21:53:20 melwitt the top change says merge conflict tho
21:53:38 efried yeah, we're still working on the requirements bit in the middle.
21:54:08 melwitt ok. I didn't think that depends-on could cause the red "merge conflict" message
21:54:57 efried yes
21:55:10 efried which is a good thing, else we be like, "why is my damn patch stuck?"
21:56:20 melwitt TIL
21:56:55 efried except that might not be the actual problem here, sigh. Doug and I are still unwinding. But this is gonna work, so feel free to sprinkle +2s around like Halloween candy.
22:02:29 mriedem efried: on https://review.openstack.org/#/c/616697/ the only downside is the slower tests right?
22:02:46 efried melwitt: correct, immediately resolved by your patch.
22:02:53 mriedem i'm mriedem
22:03:24 efried My bad, you're both 7c starting with m and a very similar shade of green and melwitt was the last one to respond.
22:03:25 mriedem +W
22:03:36 efried thanks
22:09:32 efried oh, and I do have a fractured orbital, though my vision is supposedly fine.
22:22:00 mriedem gdi eric, you need a helmet
22:22:07 mriedem while looking up helmets, i found this https://pbs.twimg.com/media/DiAdSIWUwAEXkAg.jpg
22:22:19 mriedem looks like jerry rice in there
22:26:24 openstackgerrit Jack Ding proposed openstack/nova master: Change align_size to 4096 to support 4Kn drives https://review.openstack.org/616701
22:44:16 openstackgerrit Merged openstack/nova master: remove mocks of oslo.service private members https://review.openstack.org/616697
23:32:33 openstackgerrit Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646
23:32:34 openstackgerrit Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695
23:32:34 openstackgerrit Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677
23:32:35 openstackgerrit Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033
23:32:35 openstackgerrit Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705
23:33:28 openstackgerrit Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646
23:33:29 openstackgerrit Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695
23:33:29 openstackgerrit Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677
23:33:30 openstackgerrit Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033
23:33:30 openstackgerrit Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705
#openstack-nova - 2018-11-09
01:32:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add description of custom resource classes https://review.openstack.org/616721
01:50:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix a help string in nova-manage https://review.openstack.org/616723
01:53:52 openstackgerrit Merged openstack/nova master: Harden placement init under wsgi https://review.openstack.org/610034
03:09:04 openstackgerrit Merged openstack/nova master: Use SleepFixture instead of mocking _ThreadingEvent.wait https://review.openstack.org/615724
03:18:41 openstackgerrit 98k proposed openstack/os-traits master: Add python 3.6 unit test job https://review.openstack.org/616749
07:13:26 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix server query examples https://review.openstack.org/616834
07:20:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
07:20:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
07:20:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
07:52:57 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support admin to specify project to create snapshot https://review.openstack.org/616843
07:57:37 gibi melwitt: I think there is only one patch left from use-nested-allocation-candidates that handles allocations that only use nested RPs and not the root RP https://review.openstack.org/#/c/608298/
07:58:41 gibi melwitt: that happen to be lower prio to me as that does not needed for GPU and Bandwidth work. It would be needed for the NUMA work though

Earlier   Later