Posted	Nick	Remark
#openstack-nova - 2018-06-29
09:27:59	openstack	Launchpad bug 1779287 in OpenStack Compute (nova) "placement can throw exception if webob<1.8 in use" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)
09:27:59	PapaOurs	cdent: https://bugs.launchpad.net/nova/+bug/1779287
09:28:42	PapaOurs	cdent: I'm firing a patch against nova
09:30:09	cdent	you nick today is really throwing me off
09:32:04	PapaOurs	cdent: it's my new friday nick
09:32:17	cdent	what does it mean?
09:32:29	PapaOurs	BearDad
09:33:25	cdent	is this were slack I'd have to respond to that with some kind of emoji
09:33:39	cdent	Anyway, I spent the whole first 3/4 of our conversation having no idea who you were
09:33:54	cdent	thinking "oh cool, new downstream person coming upstream"
09:34:29	cdent	so I am half sad and half happy
09:38:08	PapaOurs	cdent: you can still whois me ;)
09:38:42	PapaOurs	but sorry for the emotional lift :)
09:45:38	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
09:45:39	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
09:45:40	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
09:45:41	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
09:45:42	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
09:45:43	openstackgerrit	Andrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
09:46:44	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Fix placement incompatible with webob 1.7 https://review.openstack.org/579110
09:50:22	PapaOurs	cdent: finucannot: efried: https://review.openstack.org/579110
09:55:48	finucannot	PapaOurs: Can we not just bump the lower-constraints on that?
09:58:43	cdent	finucannot: see the backlog, there's some discussion of doing both
09:58:58	cdent	s/doing/trying/ and seeing which one floats
10:02:42	kaisers	Hey Novas, anyone familiar with Nova+QEMU please have a look: It seems to me the change https://review.openstack.org/#/c/553072/ introduced a dependency for QEMU v2.9+ while https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix shows QEMU v2.5 as minimum supported version. The patch only checks for a libvirt version but not for QEMU 2.9+ . Or maybe i'm overlooking something? (QEMU v2.9 adds the pass mtu to guest fe
10:03:57	kaisers	I'm not absolutely sure my 2.9 ref is the required feature but it's the best lead i found so far. Our CIs are stumbling on setting the MTU as they're running QEMU 2.5 (because Xenial release)
10:11:57	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Refactor the policies to policy https://review.openstack.org/579113
10:11:57	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object https://review.openstack.org/573628
10:39:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: Define common variables for irrelevant-files https://review.openstack.org/578882
10:50:08	openstackgerrit	Stephen Finucane proposed openstack/nova master: manage: Remove dead code https://review.openstack.org/579118
10:50:45	finucannot	PapaOurs: Wanna send this through? https://review.openstack.org/#/c/530923/
10:59:00	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Update admin/flavors document https://review.openstack.org/573063
11:18:57	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Handle LM rollback error when detaching volumes from transient domain https://review.openstack.org/579125
11:18:59	lyaaaaaaaarwood	kashyap: ^ as discussed earlier, would you mind taking a look?
11:19:17	kashyap	lyaaaaaaaarwood: _Just_ about to grab a quick bite
11:19:27	lyaaaaaaaarwood	kashyap: ack no rush
11:20:01	kashyap	Have it open in a tab. Will look.
11:20:10	lyaaaaaaaarwood	thanks, enjoy lunch :)
12:19:48	tssurya	mriedem, melwitt: good morning! I have replied on spec with comments on the confusions regarding delete stuff (sorry for the delay, I missed the notifications somehow).
12:20:17	tssurya	if its not clear, whenever you are free later on, we could do an IRC chat
12:22:38	openstackgerrit	Surya Seetharaman proposed openstack/nova stable/queens: Make nova service-list use scatter-gather routine https://review.openstack.org/579135
12:31:10	mriedem	tssurya: ok, i'm not adequately caffeinated yet
12:32:04	tssurya	mriedem: np, we can do it later in your evening as well
12:33:06	openstackgerrit	Surya Seetharaman proposed openstack/nova stable/pike: Make nova service-list use scatter-gather routine https://review.openstack.org/579136
12:41:24	PapaOurs	finucannot: re: https://review.openstack.org/#/c/579110/ we can't just bump the lower version easily
12:42:05	PapaOurs	finucannot: because a lot of projects use webob, and in between 1.7 and 1.8, there are some not backwards compatibilty
12:44:00	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
12:44:01	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
12:44:02	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
12:44:03	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
12:44:04	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
12:44:05	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
12:45:24	mriedem	we do have per-project lower constraints now for a reason
12:45:32	mriedem	this doesn't need to be backported
12:45:48	mriedem	i'm not sure if everyone assumes that all services on the same host are deployed in containers or venvs now though
12:46:10	mriedem	we certainly don't in devstack..
12:50:34	PapaOurs	mriedem: so, what's your opinion ?
12:50:52	mriedem	meh
12:51:24	mriedem	would probably need to talk to the requirements or release team about how they think per-project lower-constraints is meant to be handled in this case, since we don't have a common shared global-requirements anymore
12:51:48	mriedem	does that mean we, as the openstack community, assert that deployments must use containers/venvs if deploying multiple different services on the same host?
12:51:56	mriedem	maybe that's a question for the ML...
12:54:36	kashyap	Yeah, definitely sounds like an ML topic, that can end up being KM-long thread
12:58:07	mriedem	wwad
12:58:12	mriedem	what would airship do
12:58:57	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
12:58:59	openstackgerrit	Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
13:01:06	fried_rice	PapaOurs: I agree with finucannot; is there some reason we can't just bump requirements and l-c?
13:01:21	giblet	finucannot: quick question in https://review.openstack.org/#/c/578825/1/tox.ini@3 before I approve
13:01:39	PapaOurs	fried_rice: wait, lower-constraints is for all the projects
13:01:51	finucannot	PapaOurs: It's per-project now
13:02:19	finucannot	See the lower-constraints.txt file in root (and lower-constraints tox target)
13:03:06	finucannot	giblet: Nope. That's a hangover from a previous rebase, I imagine. It's harmless but I can edit and switch it back if you'd like?
13:03:32	giblet	finucannot: no need to change it back, I was just curious
13:03:35	giblet	finucannot: approving
13:03:37	PapaOurs	finucannot: so, we should bump it for nova ?
13:03:56	fried_rice	giblet: I answered at least why it was reversed to begin with.
13:04:00	fried_rice	PapaOurs: Yes
13:04:21	finucannot	fried_rice: Ah, correct
13:04:24	fried_rice	PapaOurs: You'll need to do it in both requirements.txt and lower-constraints.txt
13:04:31	giblet	fried_rice: thanks
13:05:40	finucannot	PapaOurs: Yeah, that was the idea behind moving those things in-tree, I thought
13:06:34	giblet	fried_rice: I opened https://review.openstack.org/#/c/560317/17/nova/tests/functional/libvirt/test_report_cpu_traits.py but at a first glance I don't know the answer of the failure.
13:06:50	giblet	fried_rice: I can try to dig on Monday
13:07:11	fried_rice	giblet: What was the patch you got into recently where it was something to do with creating or deleting a file on the file system, and that raced among several tests?
13:08:14	fried_rice	If you can just point me at that, I can look into it. Cause I suspect the failure is the same or similar. Or at least worth eliminating as a suspect.
13:08:33	giblet	fried_rice: that wasn't my patch but I try to find it as now I remember what you are referring to
13:08:51	fried_rice	giblet: Right, I remember you swooped in and saved the day
13:09:40	cdent	he has a habit of doing that
13:10:05	fried_rice	Heh, I misread, at first I thought you were saying *you* have a habit of doing that.
13:10:26	fried_rice	i.e. a habit of picturing giblet in a cape.
13:10:44	giblet	:)
13:10:47	cdent	don't share my secrets in public, please
13:16:00	giblet	fried_rice: is the one? https://review.openstack.org/#/c/560459/16
13:16:12	openstackgerrit	Merged openstack/nova master: [placement] Extract create_allocation_list https://review.openstack.org/577200
13:16:21	openstackgerrit	Merged openstack/nova master: [placement] Fix capacity tracking in POST /allocations https://review.openstack.org/578133
13:16:22	giblet	fried_rice: here the CONF.instance_path was the root cause
13:17:06	openstackgerrit	Merged openstack/nova master: libvirt: Add missing encryption_secret_uuid tests for pre_live_migration https://review.openstack.org/540679
13:17:24	giblet	http://logs.openstack.org/59/560459/16/gate/nova-tox-functional/c04cbeb/testr_results.html.gz
13:19:40	fried_rice	giblet: Yeah, yeah, doesn't that look pretty much identical?
13:19:50	giblet	fried_rice: yeah it does

Earlier   Later