Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-08-21
18:11:14 sean-k-mooney apparently its only there 3 times
18:11:20 sean-k-mooney i tought it was more
18:11:25 sean-k-mooney guess not
18:11:57 sean-k-mooney anyway im going to call it a day o/
21:46:06 openstackgerrit Merged openstack/nova master: Add new default roles in FIP policies https://review.opendev.org/742030
23:16:42 openstackgerrit Artom Lifshitz proposed openstack/nova master: post live migration: don't call Neutron needlessly https://review.opendev.org/747451
#openstack-nova - 2020-08-22
14:35:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Few todo fixes for API new policies https://review.opendev.org/746610
#openstack-nova - 2020-08-23
20:47:17 openstackgerrit Merged openstack/nova master: Pass the actual target in FIP policy https://review.opendev.org/742570
#openstack-nova - 2020-08-24
03:36:25 tony_su stephenfin: it would be great if you could spare some hours to review provider-config-file patches this week so that I can upgrade them for any comments.
04:08:15 openstackgerrit norman shen proposed openstack/nova stable/ussuri: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747609
04:08:26 openstackgerrit norman shen proposed openstack/nova stable/train: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747610
04:08:35 openstackgerrit norman shen proposed openstack/nova stable/stein: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747611
04:08:46 openstackgerrit norman shen proposed openstack/nova stable/rocky: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/747612
08:41:59 openstackgerrit Brin Zhang proposed openstack/nova master: Define and catch InstanceAgentNotRunning for set-password failure https://review.opendev.org/661466
09:10:53 openstackgerrit Luyao Zhong proposed openstack/nova master: Introduce a new driver interface 'cleanup_instance' https://review.opendev.org/714653
09:10:53 openstackgerrit Luyao Zhong proposed openstack/nova master: Track error migrations in resource tracker https://review.opendev.org/745281
09:10:54 openstackgerrit Luyao Zhong proposed openstack/nova master: Improve orphans tracking in resource tracker https://review.opendev.org/745282
09:15:51 openstackgerrit Mark Goddard proposed openstack/nova master: docs: fix aggregate weight multiplier property names https://review.opendev.org/746128
09:45:06 admin0 how does nova decide ( which field decides) if a hypervisor will use local storage vs ceph ?
09:48:23 lyarwood admin0: depends on the virt driver, for libvirt it's the [libvirt]/images_type configurable
09:49:02 lyarwood admin0: https://github.com/openstack/nova/blob/0d062ac7b6acc769117e77c3094b32a84411fe83/nova/conf/libvirt.py#L899-L916
09:50:54 admin0 images_type, default/null = local, rbd = ceph
09:50:57 admin0 thanks lyarwood
09:52:07 lyarwood np
11:19:14 gibi lyarwood: thanks for running the nova meetings in the last two weeks
11:20:50 lyarwood gibi: np, hopefully someone found it useful :D
11:21:10 gibi lyarwood: at least I read it ;)
11:36:52 openstackgerrit Merged openstack/nova master: libvirt: Fix indentation in test__video_model_supported https://review.opendev.org/746979
11:39:40 openstackgerrit Merged openstack/nova master: libvirt: Add libvirt version mocks in test_private_destroy_* tests https://review.opendev.org/746980
12:05:28 jsuchome lyarwood: hi, I wasn't around for a few days, I'm not sure what's the agreement about https://review.opendev.org/#/c/574301/ - do you want to rework it, or merge soon and continue in followup patches... ?
12:09:02 jsuchome or is any of the remaining requests critical?
12:11:32 lyarwood jsuchome: yeah sorry I just need to clean that up and we should be good to merge, noticed you were offline so wanted to push things along while you were out
12:24:00 nightmare_unreal hello I am trying to run functional test for openstackclient on my devstack machine . I am getting this error ModuleNotFoundError: No module named 'openstackclient.tests.compute'
12:24:09 nightmare_unreal i have not changed anything, just cloned the repo and ran tox
12:40:33 lyarwood we don't have a flavor extra spec or image property to force instances to boot from volume right?
12:51:39 gibi sean-k-mooney: hi! is there any os-vif patch that we desperatly need before the non-client library freeze that will happen next week?
12:56:22 sean-k-mooney gibi: i want to get https://review.opendev.org/#/c/744816/ merged but i need to fix it first
12:56:50 sean-k-mooney i need to make it so we dont initalies the ovsdb python binding unless you try to use them
12:57:07 gibi sean-k-mooney: OK, just ping me if review is needed
12:57:35 sean-k-mooney stephen has some mypy patches too but they can merge after teh freeze if needed
12:57:44 sean-k-mooney im not against merging them before too
12:58:17 sean-k-mooney im not a fan of the style stephen is using but im going to look into addint automated code formating after m3
12:59:33 sean-k-mooney gibi: those are stephens patches. https://review.opendev.org/#/q/status:open+project:openstack/os-vif+branch:master+topic:mypy-integration not super urgent but if there good to go im not going to block them just because i think the code look ugly
12:59:43 sean-k-mooney it works and adds type checking which is what matters
13:01:19 gibi sean-k-mooney: ack, I will handle those patches as good-to-have
13:56:53 artom lyarwood, I think we have a policy...? Lemme dig
14:01:12 artom lyarwood, https://bugzilla.redhat.com/show_bug.cgi?id=1826334
14:01:12 openstack bugzilla.redhat.com bug 1826334 in openstack-nova "[rfe] Ability to prevent creation of an instance with Ephemeral disk" [Low,Closed: upstream] - Assigned to nova-maint
14:07:00 lyarwood artom: right, policy to block but that doesn't covert requests into using bfv right?
14:07:09 lyarwood convert*
14:07:15 artom Nooo
14:07:25 lyarwood kk
14:08:01 artom You mean like, --image <funky UUID> gets converted into --block-device-mapping source=image,dest=volume,id=<funky UUID>?
14:09:00 lyarwood yeah
14:09:08 sean-k-mooney lyarwood: we have a config option
14:09:16 sean-k-mooney lyarwood: you can contole the number of local disk
14:09:34 sean-k-mooney https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.max_local_block_devices
14:10:03 sean-k-mooney if you set that to 0 then you can only do boot form volume or rbd
14:10:31 artom Still no conversion though
14:10:49 artom But I feel like automagic conversion is a bad idea, unless we significantly change the semantics
14:10:49 sean-k-mooney right
14:11:01 artom User: do X. Nova: lulz no, u get Y.
14:11:09 lyarwood right I'm not suggesting we do this
14:11:11 sean-k-mooney there conversion feacture would be a generic cinder imageges_type backend
14:11:16 lyarwood just checking that it wasn't there somewhere already
14:11:38 artom I love how "it's a stupid idea, lemme just make sure someone else didn't already implement it" is a thing
14:13:26 lyarwood we spoke about it for a while tbh
14:13:40 sean-k-mooney we did
14:13:52 sean-k-mooney im still not against a generic cinder image backend
14:14:01 sean-k-mooney but im not sure that is the same as auto bfv
14:14:12 sean-k-mooney auto bfv would be less work i think
14:14:22 sean-k-mooney but ya not a thing we support right now
14:15:06 sean-k-mooney we would have to default to delete on terminate for the volume if we did auto bfv
14:15:12 sean-k-mooney to get the correct semantics
14:15:28 sean-k-mooney but it could confuse people as to why there cinder quota is being consumed
14:15:43 sean-k-mooney unified limits i guess would change that
14:16:17 sean-k-mooney since we would only have DISK_GB for both right
14:16:44 sean-k-mooney assuming cinder is tracked in placment?
14:16:59 lyarwood I don't think cinder is in placement yet
14:19:01 sean-k-mooney ya i think your right
14:19:16 sean-k-mooney then never added support because sharing resouce provider did not work fully
14:19:21 sean-k-mooney which makes me sad
14:34:52 openstackgerrit Jiri Suchomel proposed openstack/nova master: Bring back allowed_direct_url_schemes in support of RBD image download https://review.opendev.org/728095
14:40:51 openstackgerrit Jiri Suchomel proposed openstack/nova master: rbd: Move rbd_utils out of libvirt driver under nova.storage https://review.opendev.org/746904
14:40:52 openstackgerrit Jiri Suchomel proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301
14:40:52 openstackgerrit Jiri Suchomel proposed openstack/nova master: Bring back allowed_direct_url_schemes in support of RBD image download https://review.opendev.org/728095
14:42:07 lyarwood jsuchome: thanks for that, just looking at the failing tests now
14:43:41 jsuchome you mean tests for the https://review.opendev.org/#/c/743220/ I assume?
14:45:07 lyarwood jsuchome: no https://review.opendev.org/#/c/574301/33 had some failures in the imagebackend tests that I think are being caused by the way the tests are written
14:46:15 gibi stephenfin: hi! there is an empty slot in the runways and you have the vtpm in the queue. Are you OK with me moving the vtpm to the runway slot?
14:46:31 stephenfin gibi: hey, welcome back. Sure.
14:46:40 gibi thanks. then it is moved :)
14:48:55 lyarwood jsuchome: oh and these failures were due to the changes I made btw
14:49:06 lyarwood jsuchome: so apologies for that, should have them fixed shortly
14:50:53 jsuchome ah, I see, I did not look closely at your changes
14:55:43 openstackgerrit Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/744958
14:55:43 openstackgerrit Stephen Finucane proposed openstack/nova master: manager: Move context manager up one level https://review.opendev.org/745674
14:55:44 openstackgerrit Stephen Finucane proposed openstack/nova master: compute: Add type hints for resize functions https://review.opendev.org/745341
14:55:44 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: compute: Add more type hints for resize functions https://review.opendev.org/745675
14:55:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Add generic reproducer for bug #1879878 https://review.opendev.org/747744

Earlier   Later