Earlier  
Posted Nick Remark
#openstack-nova - 2017-10-26
04:37:32 Dinesh_Bhor cfriesen: Hi, you there?
05:39:42 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
06:10:25 openstackgerrit Naichuan Sun proposed openstack/nova master: VGPU_support: add enabled white list https://review.openstack.org/512580
07:03:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update document related to host aggregate https://review.openstack.org/514499
07:10:18 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: properly decode error message from qemu guest agent https://review.openstack.org/511459
07:34:55 openstackgerrit jichenjc proposed openstack/nova master: Downgrade log for keystone verify client fail https://review.openstack.org/513243
07:36:16 openstackgerrit jichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement https://review.openstack.org/512215
07:47:29 openstackgerrit jichenjc proposed openstack/nova master: add regression test for 1546396 https://review.openstack.org/293709
07:47:54 openstackgerrit jichenjc proposed openstack/nova master: fix race condition of instance host https://review.openstack.org/494458
07:52:15 openstackgerrit jichenjc proposed openstack/nova master: Fix warning on {'cell_id': 1} is an invalid UUID https://review.openstack.org/505471
07:56:44 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Remove incorrect comment about instance.locked https://review.openstack.org/515306
08:02:17 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Remove incorrect comment about instance.locked https://review.openstack.org/515306
08:16:09 openstackgerrit jichenjc proposed openstack/nova master: Add description for resource class creation https://review.openstack.org/508083
08:20:45 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
08:22:18 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
08:24:13 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
08:24:59 bauzas naichuans: jianghuaw: could you please upload new revisions in a series by rebasing on the existing changes?
08:25:55 bauzas naichuans: jianghuaw: https://review.openstack.org/#/c/512580/ is no longer dependent on https://review.openstack.org/#/c/514942/
08:26:28 bauzas naichuans: jianghuaw: I'll rebase those
08:28:31 bauzas jianghuaw: thanks for your comment https://review.openstack.org/#/c/514763/ you're right
08:29:40 jianghuaw bauzas, u r welcome. Have you rebased the patch set. Otherwise Naichuan can do it now.
08:33:41 kashyap bauzas: stephenfin: Hi there, what is the way to restore a specification?
08:35:26 stephenfin kashyap: In Gerrit? If it's not your spec, you need a core to do it I think? bauzas can
08:35:37 kashyap stephenfin: Yeah, it's not my specc.
08:35:59 bauzas stephenfin: kashyap: which spec?
08:36:04 kashyap Context is, I was talking to KVM / QEMU developers at KVM Forum here. Incidentally Markus Zoeller was also here
08:36:12 kashyap And I'd like to work on this spec -- https://review.openstack.org/#/c/230968/
08:36:15 kashyap bauzas: ^
08:36:23 kashyap "libvirt: iothreads for disk devices"
08:36:57 bauzas kashyap: you know it'll be a rocky spec, right?
08:37:03 kashyap It's a future thing: since it's a performance feature, need to backup any claims in the spec with a couple of real benchmarks.
08:37:08 kashyap bauzas: Yep, sure.
08:37:19 bauzas kashyap: okay, so restoring
08:37:23 kashyap bauzas: No urgency right now. Just want to chug along at it, as I get time
08:37:26 kashyap Thank you
08:37:53 bauzas restored
08:38:35 kashyap bauzas: Merci beaucoup. (/me heads out for a break)
08:49:45 openstackgerrit Naichuan Sun proposed openstack/nova master: VGPU_support: add enabled white list https://review.openstack.org/512580
08:50:30 jianghuaw bauzas, ^
08:53:40 bauzas jianghuaw: ta
08:55:30 jianghuaw bauzas, how to get the allocation for vgpu in driver?
08:58:35 bauzas jianghuaw: I'm working on your comment
08:58:52 bauzas basically, I should just add availableInstances + mdevs
08:59:14 jianghuaw yeah, it should work.
09:00:12 jianghuaw bauzas, I'm wondering how to get the allocation in driver at spawn.
09:01:01 bauzas jianghuaw: the allocations are not passed by the parameter in spawn
09:01:04 bauzas now*
09:01:09 bauzas s/not/now
09:02:11 jianghuaw bauzas, where? I didn't find it.
09:03:11 jianghuaw ah, yes. I see it.
09:03:42 jianghuaw that's great. I was in a old branch.
09:14:52 openstackgerrit jichenjc proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219
09:16:57 openstackgerrit jichenjc proposed openstack/nova stable/pike: propagate OSError to MigrationPreCheckError https://review.openstack.org/515330
09:17:45 openstackgerrit Zhenyu Zheng proposed openstack/nova master: block_device_mapping_v2.bus_type is missing from api-ref https://review.openstack.org/499488
09:18:26 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190
09:20:54 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138
09:20:55 openstackgerrit Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
09:20:56 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
09:20:56 openstackgerrit Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
09:20:58 jaypipes gibi: ^
09:24:56 sahid bauzas: you don't want to deal with availableInstances
09:25:48 sahid that is informining of the number of a specific mdev type can be allocated by operators
09:26:24 sahid in this implmentation you are not going to allocate the mdev devices
09:26:31 sahid the mdev devices are going to be pre-allocated
09:26:44 sahid what you want is to list the allocated mdev devices
09:27:07 sahid calling libvirt with list_devices('mdevs')
09:28:38 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff https://review.openstack.org/515149
09:28:39 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
09:32:42 stephenfin jaypipes, bauzas: Trivial "move nova-network stuff around" patch here, if you want it? https://review.openstack.org/#/c/499168/
09:32:57 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Move additional nova-net opts to 'network' https://review.openstack.org/499168
09:33:06 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Move additional nova-net opts to 'network' https://review.openstack.org/499168
09:35:34 openstackgerrit jichenjc proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219
09:38:11 gibi jaypipes: thanks, the server side patches look good to me. Now reading the report client patch
09:40:43 jaypipes gibi: cheers
09:41:20 gibi jaypipes: Do I understand correctly that you will change the versioning still in the report client, based on your answer in https://review.openstack.org/#/c/385693/54/nova/scheduler/client/report.py ?
09:41:27 bauzas jaypipes: I have a design question for you
09:42:05 bauzas jaypipes: do you think it's acceptable to have an inventory of some resource class that is based on the sum of "left available resources" + "existing resources" ?
09:42:16 jaypipes gibi: see latest patch on that. yeah, I set it to always pass 1.11 and bomb if server doesn't support that.
09:42:20 bauzas jaypipes: speaking of the total value of course
09:42:45 jaypipes bauzas: no
09:42:59 gibi jaypipes: so the always doesn't mean that it is set in the __init__ of the client
09:43:00 bauzas jaypipes: what makes me a sad panda is that I misread some documentation in https://libvirt.org/drvnodedev.html#MDEVCap
09:43:30 jaypipes gibi: no, just that that method always passes 1.11
09:43:42 jaypipes gibi: because it expects to operate on the n-r-p attrs
09:43:52 bauzas jaypipes: I thought "availableInstances" was the total capacity of a specific device for creating vGPUs, but that's actually just an integer which decrements every time you create a mediated device
09:44:16 jaypipes bauzas: that is available i.e. total - used.
09:44:28 bauzas jaypipes: yeah, my point
09:44:37 bauzas jaypipes: so I can't use it for reporting the inventory
09:44:42 jaypipes bauzas: it's a pointless attribute if you already have total and used counts.
09:44:59 bauzas libvirt doesn't give me the API for the total number
09:45:10 jaypipes bauzas: oh well, that's a flaw in libvirt.
09:45:38 bauzas the "still" word in "how many instances of the given type can still be successfully created on the physical device" is just pissing me off
09:46:07 openstackgerrit Michael Still proposed openstack/nova master: Move infiniband vif plugging to privsep. https://review.openstack.org/515197
09:46:07 openstackgerrit Michael Still proposed openstack/nova master: Convert IVS VIF plugging / unplugging to privsep. https://review.openstack.org/515196
09:46:08 openstackgerrit Michael Still proposed openstack/nova master: Move plumgrid vif plugging to privsep. https://review.openstack.org/515336
09:46:08 openstackgerrit Michael Still proposed openstack/nova master: Move midonet vif plugging to privsep. https://review.openstack.org/515198
09:46:45 bauzas jaypipes: so seriously you would nack some reporting that would sum up that value with the existing already created mediated devices?
09:47:19 bauzas I know, it's something racy racy
09:47:22 jaypipes bauzas: no, I would be fine if you wanted to calculate total from used and available.
09:47:34 bauzas ah ok, you make my day then

Earlier   Later