Posted	Nick	Remark
#openstack-nova - 2018-03-16
02:28:29	Spazmotic	Annnnnd morning from the Office.
02:30:49	jianghuaw_	Good morning.
02:31:47	jianghuaw_	Spazmotic, I saw your message on live-migration.
02:32:38	jianghuaw_	that's expected behavior to get the exception when block_migration is False but it's not shared storage.
02:32:59	jianghuaw_	Look this: https://developer.openstack.org/api-ref/compute/
02:33:07	Spazmotic	Will do, will take a look soon
02:33:09	jianghuaw_	Set to False means the request will fail when the source and destination hosts are not on the shared storage.
02:33:44	jianghuaw_	Searching "Live-Migrate Server" there:-)
02:34:23	jianghuaw_	the above statement is included in the table of the detail for "Live-Migrate Server (os-migrateLive Action)"
02:37:20	Spazmotic	I see... Interesting
02:37:47	Spazmotic	I'll keep reading through it more completely so I can understand what it's doing differently then sir
02:37:53	Spazmotic	thanks
02:38:34	jianghuaw_	np. you're always welcome:-)
02:52:27	Spazmotic	Co workers had me doubting the migrate_data field again and how it gets passed around
02:52:31	Spazmotic	Live migrate you too confusing for your own good
04:00:13	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288
06:00:05	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
06:15:58	openstackgerrit	Merged openstack/nova master: placement: generation in provider aggregate APIs https://review.openstack.org/548249
06:19:49	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
06:54:48	eandersson	Maybe a silly idea... but would it be possible to hook into metadata changes (at least user initiated)?
06:56:51	eandersson	We use the metadata from vm create to perform specific actions using notifications (e.g. compute.instance.create.end), but would be nice to be able to retroactively do the same on existing VMs using metadata.
07:13:30	openstackgerrit	Julia Kreger proposed openstack/nova master: WIP: Update Ironic UTs to enable REST migration https://review.openstack.org/553699
07:33:13	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
07:34:11	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/553211
07:42:50	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
07:45:15	Spazmotic	I wish I understand the entirity of that patch XD
08:14:16	openstackgerrit	Merged openstack/nova master: Scheduler multiple workers support https://review.openstack.org/159382
08:18:27	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791
08:19:33	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925
08:23:17	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288
09:06:34	openstackgerrit	Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
09:06:34	openstackgerrit	Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862
09:06:35	openstackgerrit	Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435
09:15:55	openstackgerrit	Chris Dent proposed openstack/nova master: DNM: Demo code for microversion parse extraction https://review.openstack.org/550265
09:23:36	openstackgerrit	Chris Dent proposed openstack/nova master: Provide framework for setting placement error codes https://review.openstack.org/546177
09:26:21	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791
09:40:20	openstackgerrit	Surya Seetharaman proposed openstack/nova stable/queens: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553496
09:40:51	openstackgerrit	Lee Yarwood proposed openstack/nova master: docs: Disable smartquotes https://review.openstack.org/553730
09:41:07	lyarwood	stephenfin: ^ FWIW Doug suggested this in https://bugs.launchpad.net/nova/+bug/1755783
09:41:09	openstack	Launchpad bug 1755783 in OpenStack Compute (nova) "PCI configuration documentation renders with smart quotes" [Undecided,In progress] - Assigned to Lee Yarwood (lyarwood)
09:41:28	lyarwood	stephenfin: I think that's correct, ./me wait for the docs to build
09:42:05	stephenfin	lyarwood: That's kind of working around a oslo.config bug though
09:43:08	stephenfin	That being said, we're stuck with older versions of oslo.config for older versions of nova, so that could be the only backportable solution
09:43:50	stephenfin	I've got no machine to work on NUMA-aware vSwitches, so I can work on that oslo.config fix today
09:44:13	lyarwood	stephenfin: indeed, so the bug is that even with back-ticks we still end up with smartquotes right?
09:48:48	openstackgerrit	Surya Seetharaman proposed openstack/nova stable/queens: Update the nova-manage db archive_deleted_rows description https://review.openstack.org/553733
09:49:18	openstackgerrit	jichenjc proposed openstack/nova master: Move test_allocation_candidates.py https://review.openstack.org/553149
09:49:19	openstackgerrit	jichenjc proposed openstack/nova master: Move test_resource_class_cache.py https://review.openstack.org/553153
09:49:19	openstackgerrit	jichenjc proposed openstack/nova master: Move test_resource_provider.py https://review.openstack.org/553150
09:50:43	stephenfin	lyarwood: Yeah, we're not parsing the config option help text properly so only some of the formatting is being picked up
09:51:42	stephenfin	Biggest issue I see to resolving this is all the broken config option descriptions that likely exist at this point. Guess I'll have to try handle any warnings that might be raised
10:01:01	lyarwood	stephenfin: urgh, so that doesn't work, still seeing smartquotes
10:10:06	openstackgerrit	Surya Seetharaman proposed openstack/nova stable/queens: Make archive_deleted_rows handle a missing CONF.api_database.connection https://review.openstack.org/553742
10:12:24	stephenfin	lyarwood: You need to do that in 'docutils.conf'
10:12:31	stephenfin	lyarwood: See https://github.com/rtfd/readthedocs.org/issues/2940
10:17:55	lyarwood	stephenfin: huh
10:18:18	lyarwood	stephenfin: FWIW I was using http://www.sphinx-doc.org/en/stable/config.html#confval-smartquotes
10:18:54	stephenfin	lyarwood: "New in version 1.6.6"
10:19:34	stephenfin	https://github.com/openstack/requirements/blob/master/global-requirements.txt#L421
10:19:52	lyarwood	*facepalm*
10:19:54	stephenfin	I wasn't aware of that option but it seems we can't use it anyway
10:19:58	stephenfin	:/
10:20:14	lyarwood	I might just leave it in the change anyway for when we do get above 1.6.6
10:20:57	stephenfin	lyarwood: That...could be a while. Some projects are seeing issues with it and I can't figure out why my "fix" won't pass CI https://review.openstack.org/#/c/552100/
10:21:33	stephenfin	lyarwood: If you're backporting, maybe use the older 'html' prefixed option, assuming we're using <1.6 on those branches?
10:23:00	lyarwood	stephenfin: yup good point
10:24:39	lyarwood	stephenfin: `while setting up extension ['?', 'sphinx.ext.todo']: node class 'todolist' is already registered, its visitors will be overridden`
10:24:47	lyarwood	stephenfin: ^ tox -e docs - known issue?
10:25:09	stephenfin	lyarwood: I've just spotted that too. That was me (commit 936404b90a954609f0c94582df0e2fd700948cea)
10:25:21	stephenfin	Seems that feature (which I wrote) is broken in Sphinx
10:25:29	stephenfin	my turn to *facepalm*
10:25:43	stephenfin	I'm working on a fix as we speak. Gimme 5
10:26:35	stephenfin	lyarwood: To fix that, just revert said commit
10:27:28	lyarwood	stephenfin: it's fine, it had already built the config.html page I care about
10:31:53	openstackgerrit	Lee Yarwood proposed openstack/nova master: docs: Disable smartquotes https://review.openstack.org/553730
10:33:08	stephenfin	So oslo.config is doing what I'd expect it to do. Very odd...
10:33:32	stephenfin	Namely, emitting the raw lines as its sees them and letting the docutils rST parser do its work
10:57:28	openstackgerrit	Stephen Finucane proposed openstack/nova master: Follow the new PTI for document build https://review.openstack.org/553751
10:57:28	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Correct documentation for '[pci] passthrough_whitelist' https://review.openstack.org/552874
10:57:35	stephenfin	lyarwood: ^
11:02:40	lyarwood	stephenfin: still end up with smartquotes and unformatted text in config.html
11:02:50	lyarwood	stephenfin: but the build passes now ;)
11:31:19	stephenfin	lyarwood: Yeah, that's just to fix the broken local build. Did your patch work?
11:32:01	stephenfin	My reading of that was that the old option was removed in 1.6.0 and then that alias was added in 1.6.6. For 1.6.0 and 1.6.5, the docutils config file was the only way
11:32:11	stephenfin	But that's a guess
11:59:22	lyarwood	stephenfin: it did locally, just waiting on check job running, still queued at the moment.
12:13:18	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925
12:24:23	lyarwood	stephenfin: http://logs.openstack.org/30/553730/2/check/build-openstack-sphinx-docs/28fd09c/html/configuration/config.html#pci - appears to be working
12:39:15	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: WIP: complex policy https://review.openstack.org/553776
12:45:31	figleaf	ugh - tempest failed again. Is it worth rechecking yet again?
12:51:29	tssurya	figleaf: guess not :(
12:51:48	Spazmotic	heh
13:02:15	Roamer`	a couple of years ago I proposed https://review.openstack.org/#/c/137830/ and then Things Happened(tm) and we stopped working on the StorPool Nova image backend. Looks like we'll write it after all (even if we have to maintain it out-of-tree); if I un-abandon that spec and propose it for inclusion, will somebody be very much against it?
13:10:30	fried_rice	artom_: Thank you very much sir.
13:10:46	fried_rice	alex_xu_: Thanks!
13:11:29	fried_rice	figleaf: I win!
13:12:17	fried_rice	figleaf: To save some heartache, consider rebasing on top of https://review.openstack.org/#/c/548934/ (which is already in the gate) instead of master.
13:12:17	figleaf	fried_rice: You probably sabotaged tempest so that you could win :)
13:12:37	fried_rice	figleaf: Oh, I totally did. I didn't know it was gonna *work* though.
13:13:11	figleaf	fried_rice: that patch has a tempest failure, too

Earlier   Later