Posted	Nick	Remark
#openstack-nova - 2019-04-04
21:38:18	mriedem	the sighup for nova-compute is particularly busted b/c of the privsep usage and neutron event callback handler stuff,
21:38:21	dansmith	what a tangled mess of shit
21:38:35	dansmith	right, but it's also going to cause anything else to do a full restart
21:38:42	mriedem	but i wasn't sure if hup'ing the controller services was still ok because we have the cells cache for connections and disabled cells and all that
21:38:44	dansmith	not as impactful for apis I guess
21:39:01	dansmith	but for conductor it's probably not great either
21:39:13	mriedem	ok so i think you're saying huping the controller services will do what you want - reload caches, but not in the graceful way we intended
21:39:30	mriedem	b/c it's a restart
21:39:30	dansmith	I dunno about conductor, probably not great there,
21:39:43	mriedem	scheduler has the hup for the disabled cells i think
21:39:47	dansmith	as I think anything currently waiting on an rpc call, or polling for state or whatever will die
21:39:49	mriedem	and cells cache in the request context
21:40:18	mriedem	tangled mess of shit yes
21:40:24	dansmith	for serious
21:40:25	mriedem	who's being depressing NOW!
21:40:33	dansmith	o/
21:40:42	mriedem	maybe my depression rubs off on those that speak with me
21:40:46	dansmith	and it's thursday, closer to friday
21:40:57	mriedem	efried: melwitt: how are you feeling right now?
21:41:03	dansmith	you're also the arsehole that is finding this stuff
21:41:10	dansmith	we could just ignore it if it wasn't for you
21:43:20	melwitt	yeah, seriously
21:43:35	melwitt	:)
21:44:51	mriedem	it is also cold and rainy here right now (well most of this week)
21:44:54	mriedem	so it's just a gloom
21:46:27	fungi	efried: cool, yeah i was advancing/monitoring the master branch version of 650043 earlier so that one could be approved sooner, can also shuffle that one up now that it's been approved (sorry for the delay, was just out for an afternoon walk)
21:46:49	efried	thanks fungi
21:59:23	openstackgerrit	Matt Riedemann proposed openstack/nova master: Log notifications if assertion in _test_live_migration_force_complete fails https://review.openstack.org/650155
22:19:48	openstackgerrit	Matt Riedemann proposed openstack/nova-specs master: Re-propose cross-cell-resize spec for Train https://review.openstack.org/642807
22:28:36	melwitt	efried, fungi: 650043,2 just failed lower-constraints :( for what looks to be https://bugs.launchpad.net/nova/+bug/1793364
22:28:37	openstack	Launchpad bug 1793364 in Manila "mysql db opportunistic unit tests timing out intermittently in the gate (bad thread switch?)" [Undecided,In progress] - Assigned to Goutham Pacha Ravi (gouthamr)
22:30:57	fungi	melwitt: thanks for the heads up, i've manually reset the gate so it will restart jobs on that change now
22:31:09	melwitt	thanks!
22:58:48	efried	melwitt, mriedem: fungi: I gotta run. You got this?
22:59:09	melwitt	efried: yeah, I'll watch the patches tonight
22:59:13	efried	Thanks melwitt
23:00:29	fungi	i'll be around as well
23:00:34	fungi	feel free to ping me for anything
23:00:58	fungi	eager to see this resolved so it doesn't become a bigger issue after release
23:01:17	melwitt	ok, thank you
23:01:20	fungi	(since that usually means more work for all of us, and we already have more than enough of that to go around)
23:01:20	mriedem	no VMT bias there :)
23:01:27	fungi	;)
23:02:31	fungi	yeah, juggling plenty of security issues which have cropped up in releases we can't go back in time to fix, so it's nice when we catch one *before* releasing
#openstack-nova - 2019-04-05
00:26:55	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add testing guide for down cells https://review.openstack.org/650167
00:28:19	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add testing guide for down cells https://review.openstack.org/650167
01:01:04	openstackgerrit	Merged openstack/nova stable/stein: Remove CellMappingPayload database_connection and transport_url fields https://review.openstack.org/650043
01:01:14	openstackgerrit	Merged openstack/nova stable/stein: Libvirt: gracefully handle non-nic VFs https://review.openstack.org/649630
01:01:34	openstackgerrit	Merged openstack/nova master: Delete allocations even if _confirm_resize raises https://review.openstack.org/647566
01:01:41	openstackgerrit	Merged openstack/nova master: api-ref: add more details to confirmResize troubleshooting https://review.openstack.org/647741
01:06:00	ileixe	efried: Thanks for the answer. Yes I already saw the dansmith 'oschown' repository and actually we have an third API handle the request (It's very much looks like his solution). I felt hard to maintain the custom API since there's too much exceptional cases that I could not imagine.
01:07:28	ileixe	efried: So I was expecting the blueprint, but I wonder why the community decided to give up the blueprint.
01:12:47	melwitt	ileixe: some of the folks in the "transfer ownership" forum session didn't like the 'oschown' idea as much and preferred something more like API plugins. but if you are in support of 'oschown', you are encouraged to tell your opinion in the session or add your opinion on the etherpad for the session with your IRC nick
01:14:46	melwitt	it looks like there isn't a new etherpad yet though
01:14:56	ileixe	melwitt: Thanks for the procedure. It seems to not be abandonded completely yet then. I'm with plugin like API plugins, so I willing to make comments.
01:16:30	melwitt	ileixe: I don't think a solid decision has been made yet, discussion hasn't reached a way forward yet. sounds good. I'm trying to find other threads about it, one sec
01:21:23	melwitt	ileixe: this is the thread I was thinking about http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000280.html
01:34:11	openstackgerrit	melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock spec https://review.openstack.org/650171
01:45:05	ileixe	@melwitt: Many thanks. I was thirsty about the background of decision making.
01:46:48	melwitt	ileixe: np. are you going to be at the summit? if so, try to make it to the session https://www.openstack.org/summit/denver-2019/summit-schedule/events/23675/change-ownership-of-resources-followup
01:47:51	ileixe	@melwitt: Unfortunately, I do not participate.
01:48:06	ileixe	Thanks anyway :)
01:48:37	melwitt	ileixe: ok. I'll ping you when the etherpad for the session appears, so you can write your thoughts on it and it will be part of the discussion
01:49:26	ileixe	@melwitt: Cool. See you then.
01:49:36	melwitt	:)
01:53:37	openstackgerrit	melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172
01:53:56	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.openstack.org/637716
01:56:20	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
01:56:25	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
02:33:49	openstackgerrit	Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960
03:17:17	openstackgerrit	melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172
03:17:17	openstackgerrit	melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock spec https://review.openstack.org/650171
04:31:51	openstackgerrit	Seyeong Kim proposed openstack/nova stable/rocky: prevent common kwargs from glance client failure https://review.openstack.org/650064
04:31:52	openstackgerrit	Seyeong Kim proposed openstack/nova stable/rocky: Share snapshot image membership with instance owner https://review.openstack.org/643853
04:57:35	openstackgerrit	melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172
05:34:59	openstackgerrit	Tushar Patil proposed openstack/nova-specs master: Allow compute nodes to use DISK_GB from shared storage RP https://review.openstack.org/650188
06:54:19	openstackgerrit	Stephen Finucane proposed openstack/python-novaclient master: Revert "Fix crashing console-log" https://review.openstack.org/650235
07:24:13	openstackgerrit	Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960
07:57:10	kashyap	efried: Oops, I'll add them to the Etherpad.
07:58:05	kashyap	bauzas: or anyone -- Morning, do you have a URL to the review-day Etherpad?
07:59:17	tssurya	kashyap: https://etherpad.openstack.org/p/nova-spec-review-day
08:05:23	kashyap	Thanks.
08:14:10	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova master: Add requested_networks to RequestSpec https://review.openstack.org/570201
08:14:10	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova master: Add instance hard delete https://review.openstack.org/570202
08:14:11	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova master: Enable rebuild for instances in cell0 https://review.openstack.org/570203
08:14:39	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova master: Introduce the PENDING instance state https://review.openstack.org/566473
08:14:40	openstackgerrit	Theodoros Tsioutsias proposed openstack/nova master: Allow rebuild for instances in PENDING state https://review.openstack.org/637585
08:19:33	openstackgerrit	Merged openstack/os-vif stable/stein: Replace openstack.org git:// URLs with https:// https://review.openstack.org/646869
08:27:07	openstackgerrit	Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.openstack.org/612626
09:53:43	openstackgerrit	Michael Still proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.openstack.org/644793
09:53:43	openstackgerrit	Michael Still proposed openstack/nova master: Remove remaining vestiges of fake_libvirt_utils from unit tests. https://review.openstack.org/643896
09:53:43	openstackgerrit	Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from libvirt imagebackend tests. https://review.openstack.org/643895
09:53:43	openstackgerrit	Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from virt driver tests. https://review.openstack.org/643894
09:53:43	openstackgerrit	Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from snapshot tests. https://review.openstack.org/642558
09:53:43	openstackgerrit	Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from connection tests. https://review.openstack.org/642557
12:17:09	jaypipes	sean-k-mooney: see my request on https://review.openstack.org/#/c/629589/ please. If you can knock that out quick, I can quick-approve since stephenfin was +2 on it already.
12:21:38	sean-k-mooney	jaypipes: thank i will check it in a sec. currently removing utils.lock i do actully think its useful but im happy to update it too
12:22:34	jaypipes	sean-k-mooney: ack
12:23:50	sean-k-mooney	and ya the change you asked for in https://review.openstack.org/#/c/629589/21 are all pretty straigt forward so ill do those before the end of the day
13:20:56	kaisers	fried_rice: good lunch? :) Should you later on have the capacity, would you perhaps take a look at https://review.openstack.org/#/c/522245/ (missing a second +2)? I's a second long hung fix...

Earlier   Later