Posted	Nick	Remark
#openstack-nova - 2018-08-22
14:12:42	dansmith	I have suddenly collected a "cells performance mafia" following
14:12:55	tssurya	dansmith: :P (ack)
14:13:31	dansmith	tssurya: I certainly don't want to insert myself in front of any of your patches, but if you are waiting on another set anyway, it might make sense to go at the end of this series
14:13:40	dansmith	as you said
14:14:48	tssurya	yea, looks like the last patches needed from gmann are not there yet and existing last one needs update, so I can go after you..
14:15:12	dansmith	okay
14:15:34	tssurya	however I will wait for your initial patches so start geting merged before rebasing, since some are WIPs
14:15:34	dr_gogeta86	sean-k-mooney, no news
14:15:34	dr_gogeta86	?
14:16:43	tssurya	dr_gogeta86: the default path is /etc/nova/nova.conf, you need not have to put it explicitly
14:16:49	dansmith	tssurya: ack, let's plan to have the cells meeting today so we can get melwitt and mriedem on board with getting all these merged quickly
14:17:00	tssurya	dansmith: cool :)
14:17:05	dr_gogeta86	but doens't
14:18:03	holser_	thanks mriedem
14:18:56	sean-k-mooney	tssurya: actully looking at the code i can find where we set the defult. for the wsgi scipt for the api and placement i found it but i dont see it anywayre for the rest of the agents or command files
14:23:55	sean-k-mooney	dr_gogeta86: i assume this is a production deployment where you need a password to connect to the db and not devstack correct
14:25:41	sean-k-mooney	dr_gogeta86: tssurya i think the reason nova-manage works for devstack deployments without --config-file is that devstack install allows access to the db from localhost without any password
14:33:04	mriedem	but i plan on talking about cross cell migration in the cells meeting...
14:33:10	mriedem	an infinitely more fun topic
14:41:48	openstackgerrit	Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698
14:41:49	openstackgerrit	Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717
14:41:50	openstackgerrit	Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131
14:41:51	openstackgerrit	Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265
14:41:52	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577
14:48:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged on resize revert https://review.openstack.org/595069
14:51:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged on resize revert https://review.openstack.org/595069
15:07:33	dansmith	mriedem: I think we should be good on this now: https://review.openstack.org/#/c/591735/5
15:10:45	melwitt	.
15:19:20	mriedem	ying wang doesn't think so
15:38:21	openstackgerrit	sahid proposed openstack/nova master: compute: reset instance events https://review.openstack.org/420026
15:55:23	openstackgerrit	Balazs Gibizer proposed openstack/nova-specs master: Add subtree filter for GET /resource_providers https://review.openstack.org/595236
16:03:27	openstackgerrit	Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236
16:03:34	openstackgerrit	Balazs Gibizer proposed openstack/nova-specs master: Placement: support mixing required traits with any traits https://review.openstack.org/565741
16:09:21	openstackgerrit	Eric Fried proposed openstack/nova master: [placement] Regex consts for placement schema https://review.openstack.org/591863
16:09:22	openstackgerrit	Eric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
16:09:23	openstackgerrit	Eric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033
16:09:24	openstackgerrit	Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider raise https://review.openstack.org/584598
16:09:25	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599
16:09:26	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648
16:09:27	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034
16:09:28	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049
16:09:29	openstackgerrit	Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236
16:10:31	efried	Does anyone have the power to remove a bogus review? viz https://review.openstack.org/#/c/584598/
16:10:48	efried	Not that we would necessarily let that stop us from merging the thing, but it's potentially distracting.
16:11:55	openstackgerrit	Balazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/595243
16:14:28	melwitt	efried: not that I know of. I think if anyone could, it might be infra people
16:15:08	efried_rollin	ight
16:17:33	dansmith	trivial change'll fix it :)
16:17:55	dansmith	that person has been -1ing with no comment on a lot of patches
16:18:06	melwitt	yeah. I'm wondering if they're scripting it
16:22:10	mriedem	they are -1ing changes that have other -1s on them
16:22:48	mriedem	a few +1s i guess https://review.openstack.org/#/q/reviewer:wangying%2540gohighsec.com
16:22:50	mriedem	so probably not scripted
16:22:55	mriedem	given the small sample size
16:23:01	mriedem	Kevin_Zheng: ^ shame them on wechat :)
16:25:11	openstackgerrit	Merged openstack/nova master: Handle binding_failed vif plug errors on compute restart https://review.openstack.org/587498
16:25:40	mriedem	lyarwood: weren't you just working on fixing something like this? http://logs.openstack.org/50/594050/6/check/nova-next/af897cc/logs/screen-n-cpu.txt.gz?level=TRACE#_Aug_22_13_55_27_694243
16:25:53	mriedem	seems this volume detach from live/persistent domain thing needs to be fixed every other week
16:27:44	mriedem	ah yes https://github.com/openstack/nova/commit/636c7461dee4002571da6e99986eb17e9a28b0f4#diff-ec64afccc111bbe81f5c85b27d73b41b
16:27:51	Sundar	efried: Please ping me when you have the time
16:29:04	Sundar	sean-k-mooney: Hope you have seen the updated os-acc spec (#link https://review.openstack.org/#/c/577438/). It should reflect the feedback you gave.
16:31:40	mriedem	tssurya needs to re-propose the handling a down cell spec for stein
16:32:24	openstackgerrit	Matthew Booth proposed openstack/nova-specs master: Add serial numbers for local disks https://review.openstack.org/595247
16:35:52	sean-k-mooney	Sundar: i have not had time to fully review this sorry. ill try to get back to it by the end of the week
16:37:27	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Wait for network-vif-plugged before starting live migration https://review.openstack.org/591735
16:37:28	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: DNM: Debug patch to test live migration waiting https://review.openstack.org/591775
16:38:24	mriedem	dansmith: are you also going to do ^ for pike?
16:38:33	mriedem	because https://review.openstack.org/#/c/591762/
16:38:51	dansmith	I had assumed we would yeah
16:43:49	openstackgerrit	Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698
16:43:50	openstackgerrit	Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717
16:43:51	openstackgerrit	Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131
16:43:52	openstackgerrit	Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265
16:43:53	openstackgerrit	Dan Smith proposed openstack/nova master: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577
16:43:54	openstackgerrit	Dan Smith proposed openstack/nova master: Clarify which context is used by do_query() https://review.openstack.org/595250
16:45:30	dansmith	mriedem: FYI the last patch removes an extra db hit we do in the listing process which was a fixme from the original work
16:45:59	dansmith	probably small and not super impactful, but..fewer is better
17:16:36	openstackgerrit	Merged openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548
17:33:22	imacdonn	Could someone help me understand (or point me to a good description of) the deprecation of nova-consoleauth? From the way it's described, it sounds like you can just not run the service, but when I do that 'nova get-vnc-console ... novnc' gets an "unknown error" (HTTP 504)
17:50:29	melwitt	imacdonn: sec, looking for docs link
17:55:00	melwitt	imacdonn: https://docs.openstack.org/nova/latest/user/cellsv2-layout.html#consoleauth-service-and-console-proxies and the release notes https://docs.openstack.org/releasenotes/nova/rocky.html#relnotes-18-0-0-0b2-stable-rocky-upgrade-notes
17:55:29	imacdonn	melwitt: OK... will review - I just found https://review.openstack.org/#/c/557489/ too
17:56:55	imacdonn	Somehow I missed that upgrade release notes.... maybe was looking at the wrong version of release notes .. thans
17:56:58	imacdonn	thanks*
17:58:30	imacdonn	melwitt: Actually, though, this still seems to suggest that I can "just not run the service" ... and that's not working for me
17:59:19	melwitt	yeah, sorry. indeed you are supposed to be able to not run the service
18:00:08	melwitt	imacdonn: do you have a single cell deployment or multiple cells?
18:00:30	imacdonn	melwitt: single cell, not counting cell0
18:02:10	melwitt	do you have a trace you can pastebin of the nova-api log and nova-novnc
18:03:22	imacdonn	melwitt: I guess I'll enable debug logging and see what I can get ... wanted to make sure I wasn't missing a required config change or something first
18:05:39	melwitt	here's examples from the gate of what the configs should look like for the cell and compute host http://logs.openstack.org/58/540258/13/check/tempest-full-py3/f59f34a/controller/logs/etc/nova/nova_cell1_conf.txt.gz http://logs.openstack.org/58/540258/13/check/tempest-full-py3/f59f34a/controller/logs/etc/nova/nova-cpu_conf.txt.gz
18:19:03	melwitt	imacdonn: I made a mistake, sorry. you do have to run the consoleauth service in Rocky still. being able to discontinue running it will be in Stein
18:19:30	imacdonn	melwitt: Oooh okay .... so ... uhm ... it's not really deprecated yet, then ....... ?
18:20:10	melwitt	imacdonn: it's deprecated but not yet removed. in rocky it's doing things side-by-side, then in stein we can remove it
18:20:16	imacdonn	melwitt: where my definition of "deprecated" is something like "you can still do this, but you really shouldn't, and you should do it the new way instead"
18:21:20	melwitt	yeah, I see what you mean. and the docs are wrong saying you can stop running it in rocky, so I need to fix that
18:21:44	imacdonn	melwitt: yes, that was going to be my next point ;)
18:22:37	melwitt	yeah, apologies for that. I'll work on some updates to the docs right now
18:23:01	melwitt	mriedem_lunch ^ think this will be another thing for rc3
18:23:46	imacdonn	no problem! Thanks for working through it with me

Earlier   Later