Posted	Nick	Remark
#openstack-nova - 2018-08-21
21:49:56	dansmith	mriedem: they really should never hang for a long time, but just piling up more because we time out, run the periodic again and generate more traffic is probably worse
21:50:18	dansmith	mriedem: re: your save timeout
21:52:00	mriedem	dansmith: it seems something weird happened with the servicegroup in that failure
21:52:23	dansmith	yeah, not related to your actual thing, but just thinking of what that reminds me of, which is conductor is overwhelmed
21:58:56	mriedem	dansmith: also, i was talking with efried about kevin's exclusive trait thing, and guess what https://review.openstack.org/#/c/593475/
21:58:59	mriedem	it's already been proposed :)
22:00:21	dansmith	well, having not read it and skimmed the -1, I'm assuming it's for the same reason I think it's a non-starter
22:03:41	mriedem	encoding metadata in a trait name
22:05:56	dansmith	and it's only one special prefix,
22:05:57	dansmith	which means one class
22:06:46	mriedem	CUSTOM_INTEL_FOR_SERIOUS_WORKLOADS
22:06:52	mriedem	i can see it now
22:08:39	efried	POST /traits/CUSTOM_INTEL_FOR_SERIOUS_WORKLOADS
22:08:39	efried	{ 'name': 'CUSTOM_INTEL_FOR_SERIOUS_WORKLOADS,
22:08:40	efried	'required': true,
22:08:40	efried	'allowed_user_ids': [...],
22:08:40	efried	'allowed_project_ids': [...],
22:08:40	efried	...
22:08:40	efried	}
22:08:57	mriedem	queue jay vomit
22:09:01	mriedem	*cue
22:09:07	efried	we could do the same thing with aggregates
22:09:27	mriedem	dansmith: re granular, we could still do POST queries....
22:09:29	mriedem	just saying
22:10:49	mriedem	granular request group syntax is likely something that could benefit from some kind of flavor extra specs validate api
22:13:26	efried	no argument there
23:05:25	openstackgerrit	Eric Fried proposed openstack/nova master: [placement] Regex consts for placement schema https://review.openstack.org/591863
23:05:26	openstackgerrit	Eric Fried proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
23:05:27	openstackgerrit	Eric Fried proposed openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033
23:05:28	openstackgerrit	Eric Fried proposed openstack/nova master: Make get_allocations_for_resource_provider raise https://review.openstack.org/584598
23:05:29	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: Real get_allocs_for_consumer https://review.openstack.org/584599
23:05:30	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: get_allocations_for_provider_tree https://review.openstack.org/584648
23:05:31	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: _reshape helper, placement min bump https://review.openstack.org/585034
23:05:32	openstackgerrit	Eric Fried proposed openstack/nova master: Report client: update_from_provider_tree w/reshape https://review.openstack.org/585049
23:05:33	openstackgerrit	Eric Fried proposed openstack/nova master: Compute: Handle reshaped provider trees https://review.openstack.org/576236
23:42:17	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976
23:48:21	openstackgerrit	Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698
23:48:22	openstackgerrit	Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717
23:48:23	openstackgerrit	Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131
23:48:24	openstackgerrit	Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265
23:48:25	openstackgerrit	Dan Smith proposed openstack/nova master: Make CELL_TIMEOUT a constant https://review.openstack.org/594570
23:48:26	openstackgerrit	Dan Smith proposed openstack/nova master: Stash the cell uuid on the context when targeting https://review.openstack.org/594571
23:48:27	openstackgerrit	Dan Smith proposed openstack/nova master: Make RecordWrapper record RequestContext and expose cell_uuid https://review.openstack.org/594572
#openstack-nova - 2018-08-22
00:15:15	openstackgerrit	Dan Smith proposed openstack/nova master: Batch results per cell when doing cross-cell listing https://review.openstack.org/592698
00:15:16	openstackgerrit	Dan Smith proposed openstack/nova master: List instances from all cells explicitly https://review.openstack.org/593717
00:15:17	openstackgerrit	Dan Smith proposed openstack/nova master: Make instance_list perform per-cell batching https://review.openstack.org/593131
00:15:18	openstackgerrit	Dan Smith proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265
00:15:19	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577
04:03:02	openstackgerrit	Merged openstack/nova master: Making consistent used of GiB and MiB in API ref https://review.openstack.org/559985
04:04:57	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285
04:06:28	openstackgerrit	Sam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list. https://review.openstack.org/526558
04:27:12	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Merge extended server attributes extension response https://review.openstack.org/584590
05:37:09	openstackgerrit	Merged openstack/nova master: Merge config drive extension response into server controller https://review.openstack.org/584223
06:21:08	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Make monkey patch work in uWSGI mode https://review.openstack.org/592285
06:47:15	maciejjozefczyk	mriedem_away: I'll review that, thanks
06:51:57	openstackgerrit	Merged openstack/nova master: Use CONF.long_rpc_timeout in post_live_migration_at_destination https://review.openstack.org/588668
09:26:03	openstackgerrit	Sergii Golovatiuk proposed openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548
09:55:00	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572
09:55:01	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573
09:55:03	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Add three new '[libvirt] live_migration_*' options https://review.openstack.org/456571
10:04:58	openstackgerrit	Chen proposed openstack/nova master: Normalize dashless 'resource provider create' uuid https://review.openstack.org/567191
10:09:54	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Remove unnecessary default parameter https://review.openstack.org/564451
10:21:03	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_from_dict' functions https://review.openstack.org/537414
10:27:22	openstackgerrit	Surya Seetharaman proposed openstack/nova master: WIP: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265
10:27:23	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Add get_by_cell_and_project() method to InstanceMappingList https://review.openstack.org/591656
10:27:24	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down https://review.openstack.org/567785
10:34:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Use new-style choice values https://review.openstack.org/530924
10:37:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: Modify PciDevice.uuid generation code https://review.openstack.org/530487
10:37:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid https://review.openstack.org/530905
10:45:34	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Record cell success/failure/timeout in CrossCellLister https://review.openstack.org/594265
10:45:35	openstackgerrit	Surya Seetharaman proposed openstack/nova master: WIP: Optimize global marker re-lookup in multi_cell_list https://review.openstack.org/594577
10:50:52	kashyap	mdbooth: Hi, I saw the scrollback discussion here this morning, and you pointed to your "essay code comment" about Nova consuming block device job events
10:51:01	kashyap	mdbooth: That is still indeed a thing, as you guessed
10:51:16	kashyap	As the relevant upstrema libvirt bug is still open: https://bugzilla.redhat.com/show_bug.cgi?id=1382165
10:51:16	openstack	bugzilla.redhat.com bug 1382165 in libvirt "virDomainGetBlockJobInfo: Adjust job reporting based on QEMU stats & the "ready" field of `query-block-jobs`" [Unspecified,New] - Assigned to libvirt-maint
10:51:33	kashyap	And Eric Blake (from libvirt) did acknowledge:
10:51:35	kashyap	[quote]
10:51:35	kashyap	Yes, changing Nova to use event handlers is a good idea. But I'm ALSO
10:51:35	kashyap	in favor of fixing libvirt to work around the qemu bug, by intentionally
10:51:35	kashyap	munging the output to state cur<end (even if qemu reported cur==end) if
10:51:37	kashyap	qemu reports ready:false
10:51:40	kashyap	[/quote]
10:54:55	mdbooth	kashyap: Ah, ok. I wondered if the same applied to live migration, events, that is the preferred api being the events.
10:55:05	mdbooth	danpb seemed to be saying not.
10:56:23	kashyap	mdbooth: I see. Dan knows the migration internals much better, obviously.
11:14:10	sean-k-mooney	kashyap: the qemu bug might actully be a libvirt bug in a way. dan asked me to open a bug for the fact libvirt did not fail the compatilblity check when we removed setting the host_mtu on the interface as part of a migration
11:14:53	kashyap	sean-k-mooney: Yeah, saw that discussion scroll-by. Do file the upstream bug before you forget :-)
11:15:16	kashyap	sean-k-mooney: Here's a convenient URL: https://bugzilla.redhat.com/enter_bug.cgi?product=Virtualization%20Tools&component=libvirt
11:16:28	sean-k-mooney	i will i have it written on my whiteboard. i need to check a few things in nova also to confim if there is something we can do in this case. such as setting the mtu for vhost-user interface in the xml.
11:17:44	sean-k-mooney	i left out vhost-uesr because i was told setting the mtu in the vswich was all that was required, and non of the example show setting it in qemu but i will test it and see what happens.
11:17:49	kashyap	Yep, noted. Nice, you seem to have a physical whiteboard. /me was thinking of getting such a thing for a while.
11:18:57	sean-k-mooney	ya im terible at justing any kind of software tooling to track tings but makeing a note on a phyicla whiteboard works for me
11:19:26	sean-k-mooney	/justing/using/
11:19:44	kashyap	Quite happy with it
11:20:28	sean-k-mooney	hehe but then i would have to learn emacs
11:40:41	sean-k-mooney	o/
12:08:52	openstackgerrit	Sergii Golovatiuk proposed openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548
12:14:36	mriedem	sean-k-mooney: in case you haven't seen, https://review.openstack.org/#/c/594508/ was the tactical fix for the live migration bug you were hitting where we incorrectly activate the dest host port binding even on migration failure - for rocky rc3 i'm proposing that we just ignore that event since we're not properly checking the job status;
12:14:46	mriedem	sean-k-mooney: then the patch on top of that *does* check the job status for that event
12:15:14	mriedem	but i have less confidence in the 2nd patch where we check job status

Earlier   Later