Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-21
20:26:00 dansmith right but with the batching we never hit that because we don't start executing the queries until the heapq
20:27:10 melwitt oh, I see
20:27:45 melwitt hmm
20:29:21 dansmith I'll just have to get a little more into the middle of that process and we just won't get the standard handlers from scatter gather
21:13:09 mriedem huh, you don't see instance.save() messaging timeouts in the gate very often http://logs.openstack.org/98/591898/3/check/nova-next/2d5e60c/logs/screen-n-cpu.txt.gz#_Aug_21_17_10_17_732263
21:14:24 mriedem guessing this isn't good http://logs.openstack.org/98/591898/3/check/nova-next/2d5e60c/logs/screen-n-cpu.txt.gz#_Aug_21_17_10_17_426420
21:16:54 mriedem seen here too http://logs.openstack.org/85/567785/7/check/nova-tox-functional-py35/d5a8036/job-output.txt#_2018-08-17_09_06_20_241427
21:36:54 mriedem melwitt: what do you think is missing from placement for shared storage providers support? as far as i know, it's the nova stuff that's lacking as we identified ~2 weeks ago
21:37:05 mriedem for sure move operations are not ready for shared storage providers
21:37:18 mriedem the placement side of shared storage is pretty simple though, and has been done for a long time
21:40:28 openstackgerrit Dmitry Sutyagin proposed openstack/nova-specs master: Allow disabling KSM / mem-merge via extra spec https://review.openstack.org/593197
21:41:23 melwitt mriedem: it's not that I think anything is missing, I'm pragmatically thinking of the integration work and if there is something unforeseen we need to fix. I expect bugs to shake out when we integrate something for the first time. I think not having bugs shake out will be the rarer case
21:41:38 dansmith mriedem: I think we were saying the same about aggregates being done in placement before we added the placement filter stuff and realized we needed tweaks
21:41:56 dansmith and surely thought what was being done in placement for NRPs was going to be usable by nova until we thought about it
21:46:01 mriedem dansmith: the member_of thing with aggs right?
21:46:17 dansmith we needed member_of with and and or
21:46:33 dansmith and I meant prefilter above
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 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:31 dansmith mriedem: by the way, I wonder if the object action rpc methods to conductor ought to be long_rpc_timeouts
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 { 'name': 'CUSTOM_INTEL_FOR_SERIOUS_WORKLOADS,
22:08:39 efried POST /traits/CUSTOM_INTEL_FOR_SERIOUS_WORKLOADS
22:08:40 efried }
22:08:40 efried ...
22:08:40 efried 'allowed_project_ids': [...],
22:08:40 efried 'allowed_user_ids': [...],
22:08:40 efried 'required': true,
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 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:16 kashyap As the relevant upstrema libvirt bug is still open: https://bugzilla.redhat.com/show_bug.cgi?id=1382165
10:51:33 kashyap And Eric Blake (from libvirt) did acknowledge:

Earlier   Later