Posted	Nick	Remark
#openstack-nova - 2018-10-29
21:31:51	mriedem	and then there is the "Operators may unset the configuration option when:" section which says once you're upgraded (all hosts are Rocky), and stuff is expired, you can drop the workaround,
21:32:01	mriedem	if you need to add "and you're running console proxy per cell" to that list, then just add that
21:32:38	melwitt	yeah, ok. thanks. I've felt so mired in it that I was having a hard time thinking about it
21:32:49	mriedem	i left comments,
21:33:17	mriedem	but i think it's just (1) describing what rolling upgrade means and (2) add the part about console proxy being deployer per cell before unsetting the workaround
21:34:02	mriedem	we have checks for the former in the nova-status upgrade check, but not the latter, but i'm not sure if it's possible to check the latter via nova-status is it?
21:39:46	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Destination.cross_cell_move field https://review.openstack.org/614035
21:39:46	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
21:39:47	openstackgerrit	Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037
21:39:47	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
21:39:57	melwitt	mriedem: I don't think so. the only possible way to check is if the nova.conf on the console proxy host has [database]connection configured (implying access to a cell database)
21:42:08	mnaser	mriedem: sorry checking things in and out, let me push up backport
21:42:28	mnaser	I wonder if the UI adds the cherry picked from if it hasn’t merged yet
21:42:32	mriedem	it does not
21:42:46	mnaser	Boo. Alright
21:42:52	mnaser	This needs to go back to rocky or further?
21:43:05	mriedem_away	rocky
21:43:09	mnaser	ok cool
21:43:36	mriedem_away	_update_vif_xml was added in rocky,
21:44:10	mriedem_away	setting the mtu was backported to queens. if that could fail for different reasons, like live migrating a guest from source host which set mtu to something and then to a dest host that is not new enough to support that mtu config, well that's a different issue
21:44:45	mnaser	mriedem_away: most likely not, this feature was added a while back ago
21:44:51	mnaser	let me verify that
21:45:01	mriedem_away	libvirt 3.3
21:45:27	mnaser	oh yes only that got support for it
21:45:39	mriedem_away	but our minimum libvirt support is 1.3.1
21:45:56	mnaser	so unless someone is running 1.3.1 alongside 3.3.0
21:46:08	mriedem_away	i think this might catch it anyway https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L235
21:46:37	mriedem_away	alright really leaving now
21:46:42	openstackgerrit	Mohammed Naser proposed openstack/nova stable/rocky: libvirt: Avoid setting MTU during live migration if unset https://review.openstack.org/614041
21:46:42	openstack	bug 1800511 in OpenStack Compute (nova) "VMs with vif_type bridge/tap started before Rocky upgrade cannot be live migrated" [High,In progress] https://launchpad.net/bugs/1800511 - Assigned to Mohammed Naser (mnaser)
21:46:42	openstackgerrit	Mohammed Naser proposed openstack/nova stable/rocky: Add tests for bug #1800511 https://review.openstack.org/614040
21:46:43	mnaser	^ thats for later
22:08:07	openstackgerrit	Eric Fried proposed openstack/nova master: Kill @safe_connect in _get_provider_traits https://review.openstack.org/613613
22:10:00	openstackgerrit	Eric Fried proposed openstack/nova master: Kill @safe_connect in _get_provider_traits https://review.openstack.org/613613
23:11:21	openstackgerrit	melanie witt proposed openstack/nova stable/rocky: Add rolling upgrade info to enable_consoleauth workaround reno https://review.openstack.org/613174
23:12:25	melwitt	mnaser: updated to address your comments ^ let me know if there's anything else unclear in the message
23:14:36	openstackgerrit	melanie witt proposed openstack/nova stable/rocky: Add rolling upgrade info to enable_consoleauth workaround reno https://review.openstack.org/613174
#openstack-nova - 2018-10-30
00:15:31	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Destination.cross_cell_move field https://review.openstack.org/614035
00:15:31	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
00:15:32	openstackgerrit	Matt Riedemann proposed openstack/nova master: Change HostManager to allow scheduling to other cells https://review.openstack.org/614037
00:15:32	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
00:15:33	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
01:29:27	openstackgerrit	Chason Chan proposed openstack/nova stable/rocky: Remove deprecated nova-consoleauth reference from doc https://review.openstack.org/614055
01:57:59	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Use links to placement docs in nova docs https://review.openstack.org/614056
05:12:27	openstackgerrit	Merged openstack/nova master: Drop legacy live migrate allocation compat code https://review.openstack.org/611975
07:31:09	openstackgerrit	Merged openstack/nova master: Fix ironic client ironic_url deprecation warning https://review.openstack.org/611872
08:16:50	bauzas	good morning Nova
08:45:38	kashyap	mnaser: Hey, I saw the ping here. But I guess your live migration issue is nailed down now, thanks to mdbooth_390 & co
08:55:08	mnaser	kashyap: indeed, your comment on that bz helped point me towards the why :)
08:55:14	mnaser	So thanks! :)
08:55:45	kashyap	mnaser: Which bugzilla?
08:55:57	kashyap	Sorry, just catching up after 10 days of travel and conferences
08:56:33	mnaser	kashyap: https://bugzilla.redhat.com/show_bug.cgi?id=1449346
08:56:34	openstack	bugzilla.redhat.com bug 1449346 in libvirt "Addition of host_mtu=XXXX changes PCI ioport size & addresses for virtio-net device" [Unspecified,Closed: errata] - Assigned to laine
08:56:45	mnaser	Which referenced in https://review.openstack.org/#/c/614008/
08:56:51	kashyap	Ah, that one.
08:58:30	openstack	bug 1800511 in OpenStack Compute (nova) rocky "VMs with vif_type bridge/tap started before Rocky upgrade cannot be live migrated" [High,In progress] https://launchpad.net/bugs/1800511 - Assigned to Mohammed Naser (mnaser)
08:58:30	openstackgerrit	Merged openstack/nova master: Add tests for bug #1800511 https://review.openstack.org/614004
10:10:21	openstackgerrit	Maciej Jozefczyk proposed openstack/nova master: WIP: Add online_data_migration script https://review.openstack.org/614167
10:11:06	openstackgerrit	Maciej Jozefczyk proposed openstack/nova master: WIP: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167
10:35:01	openstackgerrit	wangqiang-bj proposed openstack/nova-specs master: fix wrong spelling of "unnecessary" https://review.openstack.org/614173
10:35:05	pvc	hi
10:35:06	pvc	anyone successfully setup cyborg on openstack and attach an accelerator on an instance?
10:37:38	openstackgerrit	wangqiang-bj proposed openstack/nova-specs master: fix wrong spelling of "unnecessary" https://review.openstack.org/614173
10:39:35	openstackgerrit	Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607
10:39:36	openstackgerrit	Maciej Jozefczyk proposed openstack/nova master: WIP: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167
10:53:38	openstackgerrit	John Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec https://review.openstack.org/602201
11:02:53	openstackgerrit	wanghui proposed openstack/nova-specs master: Fix a typo of 'keystoneauth1' https://review.openstack.org/614187
11:10:07	openstackgerrit	Gorka Eguileor proposed openstack/nova master: Fix unnecessary locking when connecting volumes https://review.openstack.org/614190
11:12:24	openstackgerrit	wangqiang-bj proposed openstack/nova master: fix wrong spelling of "explicit" https://review.openstack.org/614191
13:16:42	openstackgerrit	Merged openstack/nova master: Remove restart_scheduler_service() method https://review.openstack.org/613348
13:33:08	openstackgerrit	Gorka Eguileor proposed openstack/nova master: Fix unnecessary locking when connecting volumes https://review.openstack.org/614190
13:34:04	efried	bauzas: Did you see http://lists.openstack.org/pipermail/openstack-dev/2018-October/136123.html ?
13:42:07	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125
13:42:32	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
13:42:32	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840
13:42:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
13:42:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847
13:42:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
13:42:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459
14:06:12	jaypipes	lbragstad: never got around to quota stuff yesterday, sorry. day ended up completely consumed by chasing down Chef bugs.
14:06:19	jaypipes	lbragstad: will try again this afternoon.
14:09:52	mriedem	gibi: you rebased while i was reviewing https://review.openstack.org/#/c/568840/24
14:09:56	mriedem	but you can remove those todos now
14:10:44	efried	melwitt: Would you mind giving https://review.openstack.org/#/c/610174/ another pass today please? (<== edmondsw)
14:11:36	efried	jaypipes: Since https://review.openstack.org/#/c/468560/ was +A before and the rebase was trivial (uuidsentinel moved to oslo), any chance of a fast approve?
14:12:11	jaypipes	efried: done
14:12:15	efried	jaypipes: Thanks!
14:21:34	lbragstad	jaypipes sounds good
14:21:43	lbragstad	i'll be around
14:32:04	fghaas	Hi kashyap, assuming you're back and recovered from Edinburgh, could I bother you to take another peek at https://review.openstack.org/#/c/609788/ ?
14:32:19	kashyap	fghaas: Hi, I'm also recovering from "other things", will look. :-)
14:32:34	gibi	mriedem: thanks for the review.
14:32:47	fghaas	Perfect. No rush, just wanted to make sure it didn't fall off the table. :)
14:33:39	kashyap	fghaas: Ah, you've answered the questions. And have added live migration sections
14:35:04	johnthetubaguy	jaypipes: are you planning work on placement quotas, just saw your tidy up patches
14:35:50	johnthetubaguy	so I should have read the scrollback
14:36:10	johnthetubaguy	I was planning on looking at this stuff ASAP: https://review.openstack.org/#/c/602201
14:39:12	johnthetubaguy	jaypipes: I see your thread now, quota classes I think come from some out of tree rackspace idea that never made it downstream.
14:40:01	johnthetubaguy	jaypipes: the idea was a project was assigned a class, if not assigned you fallback to the default. i.e. you could upgrade someone to the I_HAVE_BIG_CREDIT_LIMIT quota class, etc. A bit like flavors.

Earlier   Later