Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-22
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
18:24:40 imacdonn part of my practice for each new release is to review logs for deprecation warnings, and eliminate as many as I can
18:25:33 melwitt that's a good practice. sorry that doc threw a wrench in it
18:26:11 imacdonn Glad to catch it before the release!
18:26:34 melwitt yes, definitely. thank you for that
18:27:27 imacdonn np

Earlier   Later