Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-18
21:41:28 mriedem they could be done in parallel actually
21:41:30 mriedem but dan's builds on yours
21:41:55 edleafe well, maybe it only throws errors with the init_host updating inventory
21:42:05 edleafe didn't get to test that scenario
21:42:40 melwitt dansmith: I'm looking at the fleetify patch to figure out how to build upon it for per cell console proxies. it looks like it sets up only one cell? https://review.openstack.org/#/c/477556
21:43:13 openstackgerrit Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288
21:43:38 mriedem melwitt: you have to look at the multinode job
21:43:48 mriedem http://logs.openstack.org/56/477556/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/aed9486/
21:44:03 melwitt mriedem: I did, I only see super conductor and one conductor process and NOVA_NUM_CELLS is 1?
21:44:29 mriedem hmm http://logs.openstack.org/56/477556/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/aed9486/logs/devstack-gate-discover-hosts.txt.gz
21:44:48 mriedem yeah i see 2 computes, 2 cell mappings, but really only 1 cell
21:45:00 dansmith melwitt: it treats cell0 as the main cell by pointing everything to it, so cell1 is the different cell,
21:45:04 dansmith which is where the computes are
21:45:17 melwitt oh, okay
21:45:25 melwitt thanks
21:45:42 dansmith the plan was to make the second node create a second cell in some configurations to validate that cross-cell migrations don't happen and stuff,
21:45:54 dansmith but there is more devstack and d-g work to do there,
21:46:05 dansmith but have had a lot of time waiting for this to get merged yaknow
21:46:19 mriedem it did get merged once remember? :)
21:46:31 melwitt heh
21:46:52 dansmith even when it merged a couple weeks ago I figured we were a little late in the cycle to be splitting that out and dealing with big config changes
21:46:53 dansmith heh yes for a hot minute
21:47:04 dansmith when we do the two computes in two cells thing we also have to make sure we end up with a two-computes-one-cell job to keep testing migrations and stuff
21:47:07 dansmith it's a whole thing
21:47:25 dansmith but, for your console thing I think the current config is going to test what you want
21:47:34 dansmith because if you don't switch cells, you'll hit cell0 and find nothing
21:48:19 dansmith mriedem: I don't think there's any strict ordering required between ed's patch and mine
21:48:28 mriedem dansmith: there isn't a strict ordering no
21:48:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Do not mention that tags are case sensitive in docs https://review.openstack.org/484957
21:48:32 mriedem could be totally parallel
21:48:41 dansmith mine is fine in isolation,
21:48:42 dansmith mriedem: mine without his would work for all new instances
21:50:04 edleafe dansmith: when I tested the first version (with the allocation stuff in init_host) it would throw errors in the periodic update until I rebased on yours.
21:50:21 edleafe dansmith: now that I removed the allocations, it probably doesn't matter
21:50:41 dansmith "when done properly, ordering doesn't matter"
21:50:42 dansmith I should have sai
21:50:43 dansmith *said
21:51:08 edleafe "properly"
21:51:29 edleafe heh
22:06:00 melwitt lost power, pasting what I wrote:
22:06:02 melwitt yeah, I'll have to think about it more. currently confused
22:06:04 melwitt console proxies currently run at the top, now we'll run them per cell. the initial request for a console will generate the auth in the cell based on the instance, then hand out the proxy url from the cell to the user
22:06:07 melwitt then the user should connect directly to the proxy in the cell
22:06:35 melwitt I was trying to think what could possibly go wrong there or how to test it with the multicell devstack
22:10:46 openstackgerrit Dan Smith proposed openstack/nova master: Consider instance flavor resource overrides in allocations https://review.openstack.org/484935
22:11:43 dansmith melwitt: I'm not sure what you mean, but the proxy should start with the nova-cell1 config, which will point it at the real cell, where the compute is
22:12:03 dansmith melwitt: which will mean that the api will find it on that rpc bus, and not the api-level one
22:14:06 melwitt dansmith: I'm not sure what I mean either. I guess just trying to think what needs to be different. so currently the proxies are started with the api cell conf and have settings in the regular nova conf
22:14:41 dansmith um
22:14:42 dansmith wat
22:14:46 dansmith you mean they use the api db part of the used-to-be-unified config?
22:15:33 melwitt like this? https://review.openstack.org/#/c/477556/5/lib/nova@885
22:16:02 melwitt $api_cell_conf which is just $NOVA_CONF
22:16:42 dansmith oh I see, that's the api conf if cellsv1 is in place right?
22:16:59 dansmith but normally that's the same config as everything else, for non-cellsv1
22:17:05 melwitt yeah, think that's why it's named that
22:17:11 melwitt (as I keep looking at this)
22:17:22 melwitt yeah
23:00:23 openstackgerrit Merged openstack/nova master: Use plain routes list for os-security-groups instead of stevedore https://review.openstack.org/478901
23:04:24 openstackgerrit Merged openstack/nova master: Use plain routes list for server-security-groups instead of stevedore https://review.openstack.org/478902
23:42:58 mriedem uh oh http://logs.openstack.org/99/471899/25/check/gate-grenade-dsvm-neutron-ubuntu-xenial/3f9a9e3/logs/new/screen-n-cond.txt.gz?level=TRACE
23:44:04 mriedem someone better change the sheets
23:46:25 mriedem https://bugs.launchpad.net/nova/+bug/1705141
23:46:26 openstack Launchpad bug 1705141 in OpenStack Compute (nova) "select_destinations reults in "TypeError: 'NoneType' object is not iterable" randomly" [High,Confirmed]
23:49:33 openstackgerrit Octave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643
#openstack-nova - 2017-07-19
00:00:35 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148
00:10:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988
00:16:37 mriedem bauzas: ^
00:16:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988
00:17:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988
00:17:58 mriedem huanxie: is the citrix xenserver CI broken?
00:18:03 mriedem it's automatically failed all day
00:22:25 tbachman mriedem: consistency is perfection
00:48:18 gmann mriedem, ohh, thanks a lot for fixing quota set microversion
01:09:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Correct the description of 'disable-log-reason' api-ref https://review.openstack.org/480209
01:11:05 openstackgerrit Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997
01:28:11 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218
01:38:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
01:38:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
01:59:05 mriedem1 melwitt: i'm about to +2 the counting instances change, just one question about test coverage in https://review.openstack.org/#/c/416521/63/nova/conductor/manager.py@989
01:59:15 mriedem1 i don't really get it though since i think you have the test for it, unless that test is missing something
02:07:06 mriedem nvm, i poked at it, i think the coverage report is busted, so +2
02:07:20 mriedem dansmith: ^ +2 on counting instances https://review.openstack.org/#/c/416521/
02:18:46 openstackgerrit Tuan Luong-Anh proposed openstack/python-novaclient master: Replace uuid.uuid4() with uuidutils.generate_uuid() https://review.openstack.org/484244
02:21:35 mriedem ok i'm +2 up through https://review.openstack.org/#/c/446243/ now
02:45:32 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for versions instead of stevedore https://review.openstack.org/485011
02:58:15 mriedem alex_xu: i finally got comments addressed and rebased the os-services/os-hypervisors to use uuids series https://review.openstack.org/#/c/464280/ - it's probably easiest at quick glance to just look at the diff from the last patch set
02:58:18 mriedem to see the major changes
02:58:47 alex_xu mriedem: yea, got it
02:58:52 mriedem cool, thanks
02:58:58 alex_xu np
03:05:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
03:05:28 openstackgerrit Chason Chan proposed openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751
03:07:02 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
03:07:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
03:38:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
03:38:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
04:01:26 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218
04:39:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
04:53:19 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Correct the description of 'disable-log-reason' api-ref https://review.openstack.org/480209

Earlier   Later