Posted	Nick	Remark
#openstack-nova - 2019-08-20
15:15:01	sean-k-mooney	ya i saw i can just add a depens on to make sure it passes
15:15:06	efried	prometheanfire: Any particular temest test I should look at for the websockify thing?
15:15:10	efried	tempest*
15:15:19	prometheanfire	I'm not sure for that one
15:15:27	efried	so I can pare down the jobs and conserve resources
15:15:32	efried	okay, I'll just run 'em all then.
15:15:40	openstackgerrit	Eric Fried proposed openstack/nova master: DNM: Test websockify 0.9.0 https://review.opendev.org/677483
15:15:50	prometheanfire	kk, if you want to wait for https://review.opendev.org/677479 you can
15:16:09	openstackgerrit	Eric Fried proposed openstack/nova master: DNM: Test websockify 0.9.0 https://review.opendev.org/677483
15:16:29	efried	oh, is that going to run something tempestuous?
15:16:43	efried	I made --^ depend on it
15:17:08	sean-k-mooney	efried: i am going to loudly ignore that ....
15:17:36	sean-k-mooney	thats such a dad joke
15:17:37	prometheanfire	it does :D
15:17:45	efried	sean-k-mooney: non sequitur? Assume we're talking about aspiers's python-rather-than-bash thing now?
15:18:00	openstackgerrit	melanie witt proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.opendev.org/507486
15:18:01	openstackgerrit	melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
15:19:25	sean-k-mooney	i assume the horrible pun between tempestuous and tempest was intentionally because if it wasnt that even worse some how
15:19:25	openstackgerrit	Merged openstack/nova master: Update api-ref for 2.75 to add config_drive in server update response https://review.opendev.org/677324
15:19:42	prometheanfire	we do include a general tempest test in our test suite
15:20:43	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1823370 https://review.opendev.org/650424
15:20:44	openstack	bug 1823370 in OpenStack Compute (nova) "Evacuations are not restricted to the source cell during scheduling" [Medium,In progress] https://launchpad.net/bugs/1823370 - Assigned to Matt Riedemann (mriedem)
15:20:44	openstackgerrit	Matt Riedemann proposed openstack/nova master: Restrict RequestSpec to cell when evacuating https://review.opendev.org/650429
15:20:44	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Use RequestSpec.requested_destination when rebuilding with new image https://review.opendev.org/650376
15:21:17	sean-k-mooney	prometheanfire: did you have a patch that put all the constratis fixes into one by the way or were they just listed in the email
15:23:37	prometheanfire	no, generally the global-reqs nightly update will include it
15:23:45	prometheanfire	that's where I see it generally
15:25:40	sean-k-mooney	ok i might have been looking at an automated patch
15:31:12	efried	stephenfin: Where was https://review.opendev.org/#/c/677444/1/doc/source/cli/nova-manage.rst rendering before? I don't see it anywhere in https://docs.openstack.org/nova/latest/cli/nova-manage.html
15:31:41	stephenfin	efried: https://review.opendev.org/#/c/677443/
15:32:04	efried	o
15:32:19	stephenfin	I didn't want that patch to block the docs patch
15:32:35	stephenfin	plus the docs one might be worth backporting
15:33:08	openstackgerrit	Matt Riedemann proposed openstack/python-novaclient master: API microversion 2.76: Add 'power-update' external event https://review.opendev.org/666792
15:33:26	efried	stephenfin: So the CLI remote_debug options... no longer work in that top patch?
15:34:07	stephenfin	efried: For nova-manage, nova-policy and nova-status, but they never actually worked there
15:34:14	stephenfin	Because there's no "service" to debug
15:35:09	efried	okay, that makes sense. So why bother documenting them in that middle patch?
15:35:20	efried	Or should we at least say "These don't work, don't try to use them"?
15:35:37	stephenfin	I wanted to be complete, but perhaps I should just drop them
15:35:49	stephenfin	Dropping them is probably the wiser option, actually
15:36:01	stephenfin	(never documenting them, that is)
15:37:02	efried	++
15:42:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443
15:42:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444
15:43:03	stephenfin	donezo
15:43:28	efried	stephenfin: we may have crossed in the mail, there was another comment on the middle one
15:44:22	stephenfin	efried: Hmm, there shouldn't be another '--version' parameter any more. I killed that
15:44:29	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-services API policies https://review.opendev.org/648480
15:45:07	stephenfin	looks like there's more cleanup to do \o/
15:45:13	stephenfin	and red pen
15:47:07	mriedem	mnaser: are you working around this in stein right now https://review.opendev.org/#/q/topic:bug/1839560+branch:stable/stein or is getting that in pretty critical for you?
15:48:18	stephenfin	sean-k-mooney: One easy follow-up on https://review.opendev.org/#/c/671338/ but it's +W'd
15:51:05	sean-k-mooney	ya i get a littel confused where that works and where it does not. e.g. does not in release notes but does in docs but not in someting else i tried
15:52:32	stephenfin	sean-k-mooney: Inside 'doc/source', it works. Everywhere else, it does not
15:53:07	stephenfin	That's because 'doc', 'releasenotes', 'api-ref' and 'api-guide' are all separate doc builds with their own indexes
15:54:11	sean-k-mooney	it was the api ref i think i tried to use it in
15:54:14	sean-k-mooney	ok so just docs
15:54:20	stephenfin	yeah, no go there
16:26:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443
16:26:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444
16:26:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Correct 'nova-manage db sync' documentation https://review.opendev.org/677508
16:26:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP: docs: Rewrite nova-manage docs to use proper directives https://review.opendev.org/677509
16:30:30	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Correct 'nova-manage db sync' documentation https://review.opendev.org/677508
16:30:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443
16:30:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444
16:30:32	openstackgerrit	Stephen Finucane proposed openstack/nova master: WIP: docs: Rewrite nova-manage docs to use proper directives https://review.opendev.org/677509
16:41:34	openstackgerrit	Stephen Finucane proposed openstack/nova master: fakelibvirt: Stop distinguishing between NUMA, non-NUMA https://review.opendev.org/675570
16:41:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Rename 'nova.objects.instance_numa_topology' https://review.opendev.org/671789
16:41:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Remove unnecessary try-catch around 'getCPUMap' https://review.opendev.org/671790
16:41:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Remove unnecessary argument https://review.opendev.org/674396
16:41:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_from_dict' functions https://review.opendev.org/537414
16:41:37	openstackgerrit	Stephen Finucane proposed openstack/nova master: claims: Remove useless caching https://review.opendev.org/671791
16:41:37	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add '[compute] cpu_dedicated_set' option https://review.opendev.org/671792
16:41:38	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793
16:41:38	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Rename exception argument https://review.opendev.org/671795
16:41:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'hardware.get_host_numa_usage_from_instance' https://review.opendev.org/671797
16:41:40	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'hardware.host_topology_and_format_from_host' https://review.opendev.org/671798
16:41:40	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'hardware.instance_topology_from_instance' https://review.opendev.org/671799
16:41:40	openstackgerrit	Stephen Finucane proposed openstack/nova master: Rework 'hardware.numa_usage_from_instances' https://review.opendev.org/672565
16:41:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Split NUMA object tests https://review.opendev.org/672336
16:41:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: '_get_(v|p)cpu_total' to '_get_(v|p)cpu_available' https://review.opendev.org/672693
16:41:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Rewrap definitions of 'NUMACell' https://review.opendev.org/674395
16:41:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800
16:41:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Rename 'fields' import to 'obj_fields' https://review.opendev.org/674103
16:42:02	sean-k-mooney	just on or two patches incoming is see
16:42:16	stephenfin	I put my own series into merge conflict
16:42:30	sean-k-mooney	nice
16:42:47	stephenfin	Must start annoying people to get the trivial patches that I've frontloaded merged
16:42:52	stephenfin	That's tomorrow's work though
16:43:30	sean-k-mooney	stephenfin: thats the pcpu series
16:43:36	stephenfin	sean-k-mooney: Want to hit https://review.opendev.org/#/c/675570/ again (it was a trivial rebase due to commit 85ea4f703)
16:43:38	stephenfin	Yarp
16:43:49	openstackgerrit	sean mooney proposed openstack/nova master: Libvirt: report storage bus traits https://review.opendev.org/666914
16:43:50	openstackgerrit	sean mooney proposed openstack/nova master: libvirt: use domain capabilities to get supported device models https://review.opendev.org/666915
16:43:50	openstackgerrit	sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775
16:43:53	stephenfin	decision on upgrades still TBD
16:44:09	sean-k-mooney	sure ill take a look now
16:44:33	sean-k-mooney	ah you rebased it so too i see
16:45:01	stephenfin	Had to. I need that later in the series

Earlier   Later