Earlier  
Posted Nick Remark
#openstack-nova - 2017-11-28
22:09:42 melwitt penick: hm, no. what I was saying would only work to have 1 set of quotas for a flavor if every tenant gets the same quotas for that flavor
22:09:52 penick oh, crud
22:10:42 melwitt basically, you'd need a quota class per unique set of quota limits
22:11:14 melwitt I would think to a large extent many tenants would be able to use the same class?
22:12:23 penick i'm checking the DB on one of our prod clusters right now..
22:12:41 mriedem if we ever get limits information from keystone, doesn't that kind of throw a wrench into all of this?
22:12:50 melwitt what I mean is, you would only need one class per tenant/flavor if they were all literally different
22:12:51 mriedem because that's the eventual replacement for quota classes in nova
22:13:53 melwitt it would throw a wrench if we won't have quota classes in keystone
22:16:10 penick Ok, so i'll need to assume one class per tenant and flavor
22:16:25 melwitt hang on, let me look at this again real quick.
22:16:34 penick mriedem are limits definitely moving to keystone? Because i've heard talk about that for a year or two..and nothing's happened.
22:16:57 dansmith well, people keep getting laid off
22:17:00 dansmith kinda gets in the way of progress
22:17:37 mriedem someone from huawei was supposedly working on picking up that spec in keystone, but last i heard in sydney it was dropped in lbragstad's lap, and he's trying to move the rbac stuff forward
22:18:34 lbragstad we do have a new owner for that work
22:18:59 lbragstad https://review.openstack.org/#/c/455709/
22:19:14 lbragstad which is still under sdague's name but wxy is picking it up
22:20:02 lbragstad so if there is anything nova needs out of that interface from keystone, let us know
22:21:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548
22:22:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548
22:22:23 penick if Keystone is going to be able to land the limits api in queens, will Nova move to use it for Rocky?
22:22:37 lbragstad yeah - that's the idea
22:23:05 lbragstad but there are other things that playing into it too, which sean was really driving in Boston (like what enforcement models to use)
22:23:21 lbragstad we're not quite sure how that's going to get applied as the service yet
22:30:05 melwitt penick: I looked through the code again and indeed quota classes are not per tenant. so it won't be as simple as what I thought earlier
22:30:33 penick melwitt: dang
22:40:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548
22:43:11 openstackgerrit Thomas Goirand proposed openstack/nova master: qemu-img do not use cache=none if no O_DIRECT support https://review.openstack.org/523554
22:45:15 zigo melwitt: ^
22:46:05 melwitt cool, looking
22:51:41 openstackgerrit Mathieu Gagné proposed openstack/nova master: Fix rebuild of baremetal instance when vm_state is ERROR https://review.openstack.org/523559
22:53:08 mriedem efried: can i make simple rest calls like post() from a ksa adapter object?
22:53:37 mriedem mgagne: broken already :)
22:54:13 mgagne mriedem: what's broken?
22:54:18 mriedem mgagne: oh nvm, was thinking the rebuild + config drive thing
22:54:45 mgagne mriedem: hehe, we have a theme this month: rebuilding stuff =)
22:55:40 dansmith and re-rebuilding
22:55:48 mgagne that one had an interesting side effect: rebuild is actually happening on Ironic side but Nova stays in ERROR while rebuild continues
22:57:45 mriedem efried: nvm, answer is yes, that's exactly how it's used for the placement client in SchedulerReportClient
22:59:31 melwitt zigo: reviewed, couple of minor things and need unit tests. looks clean tho
23:00:21 efried mriedem Yuh
23:07:47 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/523562
23:10:42 zigo mtreinish: Thanks, though I'm not sure what kind of unit tests you think I should write.
23:11:40 openstackgerrit Thomas Goirand proposed openstack/nova master: qemu-img do not use cache=none if no O_DIRECT support https://review.openstack.org/523554
23:12:32 zigo mtreinish: sorry wrong person.
23:13:09 zigo melwitt: I'm not sure how the unit tests you're recommending should look like.
23:16:08 melwitt zigo: I attempted to explain in my comment. basically you'd mock supports_direct_io and utils.execute via decorators, and in one test make supports_direct_io return True and then verify that utils.execute was called with '-t none' and vice versa for it returning False
23:19:09 melwitt zigo: if you don't have time to give it a try or otherwise don't want to, I can write the tests and add them to your patch as a co-author if that's okay with you
23:19:43 zigo melwitt: It'd be great if you did so, yes.
23:19:45 zigo Thanks.
23:20:48 melwitt k, np
23:29:53 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add code to bind a port against a dest host during live migration https://review.openstack.org/523604
23:29:55 mriedem mlavalle: ^
23:29:58 mriedem it's a start
23:30:12 mlavalle mriedem: thanks :-)
23:33:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement reuest ID https://review.openstack.org/523606
23:33:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement reuest ID https://review.openstack.org/523606
23:59:41 openstackgerrit Mathieu Gagné proposed openstack/nova master: Fix rebuild of baremetal instance when vm_state is ERROR https://review.openstack.org/523559
#openstack-nova - 2017-11-29
00:12:38 openstackgerrit melanie witt proposed openstack/nova master: qemu-img do not use cache=none if no O_DIRECT support https://review.openstack.org/523554
00:57:41 openstackgerrit Merged openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404
00:57:55 openstackgerrit Merged openstack/nova master: Save updated libvirt domain XML after swapping volume https://review.openstack.org/498983
01:17:11 openstackgerrit Wangpan proposed openstack/nova master: Handle ImageInUse exception during instance backup https://review.openstack.org/523395
01:25:23 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi:(WIP:need ut) Get vgpu info from `allocations` https://review.openstack.org/521717
01:26:35 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
01:38:40 openstackgerrit Rong Han proposed openstack/nova stable/pike: Fix incorrect known vcpuset when CPUPinningUnknown raised https://review.openstack.org/523644
02:07:48 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances https://review.openstack.org/523653
02:22:38 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/523562
02:44:59 Delair Hi All .. Is it normal in openstack newton that if you delete project, your VMs will still reside in the compute? Is there a workaround where you delete project and it will delete VM automatically
03:04:08 openstackgerrit Kevin Zhao proposed openstack/os-traits master: Add AArch64 CPU traits https://review.openstack.org/523665
03:13:50 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326
03:38:12 larsks Delair: you might want to look at https://github.com/openstack/ospurge
03:40:37 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: [WIP] Add cross cells support for InstanceActionAPI https://review.openstack.org/523674
03:41:07 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
03:42:48 alex_xu efried: I will check that, to find out which code fix that bug, that really surprise me...
04:10:55 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: provide VGPU_DISPLAY_HEAD inventory in compute node https://review.openstack.org/523342
04:33:56 openstackgerrit zhongshengping proposed openstack/nova master: Fix project_domain_name and user_domain_name in doc https://review.openstack.org/523686
05:15:50 openstackgerrit Alex Xu proposed openstack/nova master: Remove the objects which related to the old v2 API implementation https://review.openstack.org/519997
05:15:51 openstackgerrit Alex Xu proposed openstack/nova master: Remove the objects for describing the extension for v2.1 API https://review.openstack.org/519998
05:16:01 alex_xu oomichi_afk: gmann ^ just updated
05:22:12 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
05:40:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
05:46:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
05:58:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
06:05:15 gmann alex_xu: thanks , ll check
06:18:15 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606
06:26:03 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: [WIP] Add cross cells support for InstanceActionAPI https://review.openstack.org/523674
06:41:59 openstackgerrit Merged openstack/nova master: PowerVM support matrix update https://review.openstack.org/523140
07:08:50 openstackgerrit Chen Hanxiao proposed openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655
07:20:38 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
08:11:12 openstackgerrit jichenjc proposed openstack/nova master: [placement]Enhance doc for placement allocation list https://review.openstack.org/511715
08:15:47 openstackgerrit jichenjc proposed openstack/nova master: check query param for server groups function https://review.openstack.org/500347
08:25:39 openstackgerrit licanwei proposed openstack/nova master: Make nova-manage syncing all cells databases https://review.openstack.org/519275
08:30:40 kalyan Hi All, is openstack/libvirt usb passthrough has a limitation USB protocol <=2.0 ??
08:31:02 kalyan could someone please correct me on this and provide some information to achieve the same if possible
08:31:10 kalyan thanks in advance! :)
08:38:11 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
08:57:44 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
09:08:05 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/523765
09:14:00 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/520478

Earlier   Later