Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-28
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
09:14:46 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
09:32:07 openstackgerrit Erik Berg proposed openstack/nova master: Cell_v2: unique constraint on database_connection mapping https://review.openstack.org/523817
09:50:55 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: throw NotImplementedError if qga is not responsive when setting password https://review.openstack.org/523828
10:10:31 openstackgerrit Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
10:42:22 openstackgerrit Surya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556
11:40:05 openstackgerrit Erik Berg proposed openstack/nova master: Cool, wasn't sure about wether the transport_url needed to be unique aswell. The more you know :) https://review.openstack.org/523849
13:15:56 jaypipes *yawn*
13:15:59 jaypipes morning supernovas
13:28:43 bauzas jaypipes: morning
13:54:27 mriedem lucasagomes: you should take a look at this change https://review.openstack.org/#/c/523559/
13:59:12 efried jaypipes Looks like some microversion digits were missed in the rebase.
13:59:25 jaypipes efried: lemme look...
13:59:39 efried jaypipes I can take care of it - I have some work to do on the series anyway.
13:59:48 lucasagomes mriedem, hi there! I will take a look, although, I haven't been involved with ironic for awhile. It might be better to get some of the #openstack-ironic folks to take a look as well
14:00:00 efried jaypipes As long as you don't have local changes?
14:00:13 jaypipes efried: go for it.
14:00:18 efried rgr
14:11:42 lucasagomes mriedem, posted a question re using task_state + vm_state to determine if the rebuild failed or not
14:12:54 mriedem lucasagomes: heh, did you see my comment in the code?
14:12:59 mriedem basically the same idea
14:13:16 lucasagomes mriedem, oh, I looked at the wrong column ahah oh right
14:13:20 lucasagomes yeah i think that would be more flexible
14:13:24 lucasagomes if it works
14:15:47 lucasagomes replied to your comment basically agreeing with the task_state + vm_state idea :-)
14:25:01 stephenfin jaypipes: I've a question on https://github.com/openstack/nova/commit/15ac5b688bf6d91ac42ca33860d187d80289d82d
14:25:20 jaypipes stephenfin: yessir?
14:25:58 stephenfin jaypipes: That contains an online data migration. Would I need to run the `nova-manage db online_data_migrations` command for that to take place, or should it happen any time?
14:26:43 stephenfin I'd thought the UUID field would be populated any time I loaded an object from the DB, but then I saw that command
14:32:21 jaypipes stephenfin: there's no nova-manage db stuff in that patch.
14:32:47 jaypipes stephenfin: all data migrations are done upon load/save of the PciDevice object
14:34:21 jaypipes stephenfin: one might make an argument that there *should* have been a corresponding patch that added a nova-manage online data migration (that read in PciDevice objects in batches)
14:39:52 mriedem that's what we did when we added the uuid column to compute_nodes, services and migrations tables
14:40:05 mriedem you could still add one...
14:42:07 jaypipes mriedem: ack
14:42:34 jaypipes mriedem: would need to be done before the PCI device manager is integrated with placement.
14:42:53 stephenfin jaypipes, mriedem: I can do that today
14:42:57 openstackgerrit Sylvain Bauza proposed openstack/nova master: libvirt: Provide VGPU inventory for a single GPU type https://review.openstack.org/514763
14:43:12 stephenfin jaypipes: In the interim, I think there's a bug with that patch. See here
14:43:13 stephenfin https://bugzilla.redhat.com/show_bug.cgi?id=1516952
14:46:54 openstack bugzilla.redhat.com bug 1516952 in openstack-nova "Cannot boot vm with sriov port after upgrade OSP11 to OSP12" [Urgent,New] - Assigned to eglynn

Earlier   Later