Earlier  
Posted Nick Remark
#openstack-nova - 2017-07-18
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 Chason Chan proposed openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751
03:05:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
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
05:22:53 openstackgerrit Jianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming https://review.openstack.org/485039
05:44:53 openstackgerrit Merged openstack/nova master: Consider instance flavor resource overrides in allocations https://review.openstack.org/484935
05:55:09 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218
06:01:57 openstackgerrit Merged openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore https://review.openstack.org/478903
06:11:10 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218
06:41:12 openstackgerrit Alex Xu proposed openstack/nova master: Remove the unittest for plugin framework https://review.openstack.org/485060
06:41:13 openstackgerrit Alex Xu proposed openstack/nova master: [DNM] Using plain routes for the microversions test https://review.openstack.org/485061
07:33:26 gibi jaypipes: I tested your patch https://review.openstack.org/#/c/484900 I commented the results in the review
07:35:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix an expand button in os-quota-sets https://review.openstack.org/485075
07:47:30 openstackgerrit Moshe Levi proposed openstack/nova master: WIP: remove physnet https://review.openstack.org/485078
07:49:46 openstackgerrit Moshe Levi proposed openstack/nova master: WIP: remove physnet https://review.openstack.org/485078
08:15:53 openstackgerrit Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250
08:23:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: [placement] test shared custom resource provider https://review.openstack.org/485088
08:30:59 stephenfin gmann: Could you take a look at https://review.openstack.org/#/c/478484/ again?
08:40:34 openstackgerrit Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122
08:45:27 openstackgerrit Merged openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore https://review.openstack.org/478904
08:47:28 openstackgerrit Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757
08:47:37 stephenfin bauzas: Some easy +2/+W's here https://review.openstack.org/#/q/topic:doc-migration+project:openstack/os-traits+status:open
08:47:46 bauzas stephenfin: yup, I saw them
08:48:11 bauzas stephenfin: I wanted to discuss with matt about if he wanted to pass a new version for os-traits
08:48:22 bauzas we're very close to the library deadline, right?
08:48:52 bauzas yeah https://releases.openstack.org/pike/schedule.html#p-final-clientlib
08:49:13 stephenfin bauzas: We are, yeah
08:49:25 stephenfin Do docs affect that deadline?
08:49:38 bauzas in theory nope
08:49:54 bauzas that's not a "feature freeze"
08:50:08 bauzas here we call of the last time to version the library
08:50:13 bauzas for pike
08:50:19 bauzas in order to have stable dependencies
08:50:53 bauzas so, we can merge your docs changes, but the big question is whether we need to tag for it

Earlier   Later