Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-28
21:59:29 mriedem anything involving AZs is going to be hairy as hell
21:59:35 mriedem or quotas
21:59:44 mriedem is there some way we can work boot from volume into this?
21:59:51 penick You bet your ass we can
21:59:55 melwitt being able to tag a flavor with a quota class would get us quota by flavor at least, and the same use case has come up in the preemptable instances discussions
22:01:08 melwitt that is, for preemptable instances, you could have a quota class 'preemtable' which is unlimited quota and tag preemptable instance flavors with that class. so that users booting preemptable instances don't use up their quota when they boot them
22:02:35 mriedem mlavalle: ok so i'm looking for the "binding-extended" extension per https://review.openstack.org/#/c/484389/8/neutron_lib/api/definitions/portbindings_extended.py
22:03:12 mriedem mlavalle: i'm looking at starting to build up the nova patches for this thing, and part of that is just asking neutron if the api extension is even available to do the deed
22:05:13 mlavalle mriedem: that is the extension definition. as far as the implementation, I am working on it
22:06:02 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: Remove unnecessary spaces on JSON samples https://review.openstack.org/523543
22:06:24 penick So If I want to limit tenant $foo to 10 instances of flavor $bar, and I want to limit tenant $baz to 20 instances of flavor bar, i'd create 2 quota classes, one for tenant $foo, one for tenant $baz, then tag flavor $bar with both quota classes?
22:09:39 penick is that how the flow would go? Assuming I could tag a flavor with the quota class
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

Earlier   Later