Posted	Nick	Remark
#openstack-nova - 2018-08-21
21:46:37	dansmith	or request filter
21:46:40	dansmith	or whatever that guy called it
21:47:29	dansmith	the granular request stuff is another similar example, where we try to use what we think is just clean resource requests for actual nova stuff and realize we need this giant complex syntax instead
21:47:54	dansmith	all that could be developed in two separate rooms for sure, just like multiattach or multiple host bindings
21:48:05	dansmith	and hot damn, in a few years we'll be golden
21:49:31	dansmith	mriedem: by the way, I wonder if the object action rpc methods to conductor ought to be long_rpc_timeouts
21:49:31	melwitt	yeah. in case it wasn't clear, the thing I care about is delivering stuff that operators and users need, that we know they need, and I don't see how becoming two separate groups helps that
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/

Earlier   Later