Posted	Nick	Remark
#openstack-nova - 2018-10-29
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: Add tests for bug #1800511 https://review.openstack.org/614040
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: libvirt: Avoid setting MTU during live migration if unset https://review.openstack.org/614041
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 Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
00:15:31	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add Destination.cross_cell_move field https://review.openstack.org/614035
00:15:32	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
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: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	openstackgerrit	Merged openstack/nova master: Add tests for bug #1800511 https://review.openstack.org/614004
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)
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 request_spec.RequestGroup versioned object https://review.openstack.org/568840
13:42:32	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
13:42:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847
13:42:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
13:42:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459
13:42:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
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.
14:40:25	gaudenz	Hi
14:40:51	mriedem	gibi: a few nits and questions in https://review.openstack.org/#/c/567267/
14:41:37	sean-k-mooney	johnthetubaguy: is that related to the idea of qoutas for flavors and generic placement resouce class quotas
14:41:55	johnthetubaguy	sadly no, that's different
14:42:17	johnthetubaguy	this is more a groups of limits, rather than limits on flavors
14:42:18	sean-k-mooney	oh ok i like that usecase but never herard of quota classes until now
14:42:33	johnthetubaguy	its half lurking in the code, and was never finished
14:42:39	johnthetubaguy	best ignored
14:42:47	openstackgerrit	Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989
14:43:00	sean-k-mooney	ok ill continue to do so
14:43:00	jaypipes	johnthetubaguy: or you could just have quota templates internally and assign a new project the values of that quota template... quota classes never needed to be in the API.
14:43:33	johnthetubaguy	jaypipes: yeah, totally, that is the current plan for unified limits. its crazy pants stuff
14:43:40	jaypipes	in a similar way that "registered quotas" in the new keystone limits API doesn't need to be exposed to users at all.
14:43:54	jaypipes	johnthetubaguy: ++
14:46:06	johnthetubaguy	jaypipes: did you get chance to look at the unified limits spec I am looking at working on with alex_xu?
14:48:01	mriedem	bauzas: do you remember this? https://review.openstack.org/#/c/582417/1/nova/conductor/tasks/migrate.py@a287
14:48:31	mriedem	because i'm having a hell of a time figuring out why i can't pass a RequestSpec object to the compute prep_resize RPC API but we can pass request spec to other things like select_destinations and build_and_run_instance
14:49:28	jaypipes	johnthetubaguy: in my queue.
14:49:57	johnthetubaguy	jaypipes: cool, just making sure to avoid duplicate work
14:55:30	kashyap	fghaas: One latest remark in the change; regardless, I think it can be merged now. And can fine-tune in further patches.

Earlier   Later