Posted	Nick	Remark
#openstack-nova - 2019-07-10
14:43:45	sean-k-mooney	the clouds.yaml has profiles for clouds that encode there knonw behavior for non discoverable things
14:43:56	sean-k-mooney	so that is always an option for limits
14:44:41	sean-k-mooney	at least i think nodepool gets that stuff from clouds.yaml for the sdk
14:52:06	openstackgerrit	Dakshina Ilangovan proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.opendev.org/612497
14:59:56	spatel	sean-k-mooney: did you ever see this error http://paste.openstack.org/show/754267/
15:07:05	stephenfin	johnthetubaguy: Seeing as you've been reviewing docs, could you have a look at https://review.opendev.org/#/c/667164/ ?
15:07:29	johnthetubaguy	stephenfin: sure
15:15:37	openstackgerrit	John Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec https://review.opendev.org/602201
15:15:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: ec2: Remove unused functions from 'ec2utils' https://review.opendev.org/662501
15:15:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove unused ec2 objects https://review.opendev.org/662502
15:15:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: ec2: Remove ec2.CloudController https://review.opendev.org/662503
15:15:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: ec2: Pre-move cleanup of utils https://review.opendev.org/662504
15:15:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: ec2: Move ec2utils functions to their callers https://review.opendev.org/662505
15:15:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Remove 'Debug' middleware https://review.opendev.org/662506
15:19:07	openstackgerrit	Stephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups https://review.opendev.org/666630
15:19:07	openstackgerrit	Stephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another https://review.opendev.org/666631
15:19:08	openstackgerrit	Stephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks https://review.opendev.org/666814
15:20:35	Nick_A	We are trying to offload the _base nova image cache using cephfs. our idea was to do a read-only mirror of glance's image pool and mount that to _base on compute. Would nova have problems with that setup?
15:21:04	openstackgerrit	Merged openstack/nova master: Perf: Use dicts for ProviderTree roots https://review.opendev.org/637225
15:21:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: Handle libvirt reporting incorrect 4k page quantities https://review.opendev.org/631038
15:21:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Split '_get_host_numa_topology' function https://review.opendev.org/631237
15:21:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: Make overcommit check for pinned instance pagesize aware https://review.opendev.org/631053
15:22:30	openstackgerrit	Shilpa Devharakar proposed openstack/nova master: Support filtering of hosts by forbidden aggregates https://review.opendev.org/667952
15:25:54	openstackgerrit	Stephen Finucane proposed openstack/nova stable/stein: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670096
15:26:03	openstackgerrit	Stephen Finucane proposed openstack/nova stable/rocky: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670097
15:28:05	openstackgerrit	Stephen Finucane proposed openstack/nova stable/stein: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670096
15:28:37	openstackgerrit	Stephen Finucane proposed openstack/nova stable/rocky: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670097
15:29:21	openstackgerrit	Stephen Finucane proposed openstack/nova stable/queens: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/670100
15:35:33	mriedem	stephenfin: i see https://review.opendev.org/#/c/667164/2 is already approved and you've started backports, but some issues inline
15:35:45	stephenfin	looking
15:36:43	openstackgerrit	sean mooney proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912
15:36:59	stephenfin	mriedem: Those all look fair. Want me to pull it out of the gate and address those?
15:37:15	mriedem	that would be nice yes
15:37:20	artom	There's a thing to re-run just the tests that failed in the last run... right?
15:37:28	mriedem	artom: not in the gate
15:37:32	mriedem	locally,
15:37:33	artom	mriedem, locally
15:37:36	mriedem	tox -e <whatever> -- --failing
15:37:41	artom	mriedem++
15:38:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
15:40:07	stephenfin	mriedem: 'openstack quota list --compute' won't show the _default_ quotas though, will it?
15:40:13	stephenfin	(RE: https://review.opendev.org/#/c/667164/2/doc/source/admin/quotas.rst@68)
15:41:13	mriedem	stephenfin: honestly i don't know how that command works under the covers,
15:41:25	mriedem	"default" doesn't really mean anything for the compute API if you're not looking at the default class
15:41:31	mriedem	neutron has an api to see 'defaults'
15:41:39	mriedem	but i don't think cinder does and nova doesn't besides the class
15:43:49	stephenfin	looks like 'quota list' is no good to us
15:43:51	stephenfin	"List quotas for all projects with non-default quota values or list detailed
15:43:51	stephenfin	quota informations for requested project"
15:44:10	stephenfin	I'll add a note that there are non-compute quotas mixed in there
15:48:11	mriedem	yeah this feels like a wild goose chase
15:48:19	mriedem	sorting out the clis and docs is a mess
15:50:05	openstackgerrit	sean mooney proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912
15:59:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
15:59:55	stephenfin	mriedem: ^ addressed what I could of them
16:00:01	mriedem	ack
16:00:42	spatel	sean-k-mooney: nevermind after reload rabbitmq cluster it fixed issue.
16:02:55	sean-k-mooney	spatel: sorry i was on a call earlier but yes we have seen that before
16:03:43	sean-k-mooney	restarting rabbit or the compute agged typicaly fixes it
16:03:51	spatel	same here
16:04:32	sean-k-mooney	its related to how oslo.messaging handels retrasmits and connection issue but i think they are trying to fix it
16:04:53	sean-k-mooney	its not really a nova issue as we dont actuly handel creating the queue/exchanges
16:05:08	sean-k-mooney	that is all done use by oslo.messaging
16:14:15	openstackgerrit	Sylvain Bauza proposed openstack/nova master: WIP: Add a placement audit command https://review.opendev.org/670112
16:31:23	sean-k-mooney	alex_xu: efried thanks for your input on https://review.opendev.org/#/c/662264/. I have abandoned it for train so we can focus on similar uses of placements new feature in train and come back to cache in U
16:32:22	sean-k-mooney	if im working on that again in U ill propose a placmeent native version that may pull some of the provider yaml stuff it that makes progress
16:40:34	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
16:40:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
16:40:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Scrub available quotas https://review.opendev.org/670125
16:42:18	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
16:42:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Scrub available quotas https://review.opendev.org/670125
16:42:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
16:48:08	openstackgerrit	Merged openstack/nova master: db: Add vpmems to instance_extra https://review.opendev.org/662697
16:59:01	Sundar	Hello gibi, Could you look at https://review.opendev.org/657464 ? I understand it merited a -1 earlier, but I think we have resolved the naming issues in the Cyborg-Nova spec. Thanks.
16:59:45	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/667177
16:59:46	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Pass migration to finish_revert_migration() https://review.opendev.org/668631
16:59:46	openstackgerrit	Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442
17:06:40	stephenfin	mriedem, melwitt: If we removed the injected_files, injected_file_content_bytes and injected_file_path_bytes quotas in API v2.57, does that mean the conf options in '[quotas]' should have a note saying the quotas aren't used anymore or something?
17:06:41	openstackgerrit	Merged openstack/nova master: Remove 'nova.virt.libvirt.compat' https://review.opendev.org/669120
17:07:15	stephenfin	I'm not grasping whether those quotas actually do anything any more and, if so, what's enforcing it
17:16:29	sean-k-mooney	stephenfin: i think we still need them since we still support it for older microversions
17:17:05	sean-k-mooney	file injection itself is deprecated but you can still use it if you drop back too older microverions
17:32:50	openstackgerrit	Merged openstack/nova master: libvirt: Remove unreachable native QEMU iSCSI initiator config code https://review.opendev.org/668750
17:50:34	openstackgerrit	Dakshina Ilangovan proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.opendev.org/612497
18:00:11	openstackgerrit	Dakshina Ilangovan proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.opendev.org/612497
18:11:00	mriedem	stephenfin: what sean-k-mooney said
18:12:57	openstackgerrit	Artom Lifshitz proposed openstack/nova master: [WIP-until-series-is-ready] Introduce live_migration_claim() https://review.opendev.org/635669
18:12:58	openstackgerrit	Artom Lifshitz proposed openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827
18:12:58	openstackgerrit	Artom Lifshitz proposed openstack/nova master: LM: add support for sending NUMAMigrateData to the source https://review.opendev.org/634828
18:12:59	openstackgerrit	Artom Lifshitz proposed openstack/nova master: LM: add support for updating NUMA-related XML on the source https://review.opendev.org/635229
18:12:59	openstackgerrit	Artom Lifshitz proposed openstack/nova master: RPC changes to prepare for NUMA live migration https://review.opendev.org/634605
18:13:00	openstackgerrit	Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606
18:13:00	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021
18:26:11	sean-k-mooney	someday we will have v4 api after the go rewite and then we can drop 2.1 api support
18:46:26	mriedem	efried: i'm guessing you want to backport this right? https://review.opendev.org/#/c/637225/
18:46:28	mriedem	to at least rocky
18:48:51	mriedem	lyarwood: fyi in case you never saw this https://bugs.launchpad.net/devstack-plugin-ceph/+bug/1835627
18:48:51	openstack	Launchpad bug 1835627 in devstack-plugin-ceph "test_shelve_unshelve_server failing on stable/pike (and probably ocata)" [Undecided,Confirmed]
19:26:18	dansmith	efried: normally I'd like to see the top of a series be nearly ready to go before we merge a db migration for it: https://review.opendev.org/#/c/662697/
19:26:26	dansmith	I know you didn't +W, but, just my opinion

Earlier   Later