Earlier  
Posted Nick Remark
#openstack-nova - 2017-11-16
02:51:10 openstackgerrit Matt Riedemann proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
02:51:11 openstackgerrit Matt Riedemann proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755
02:51:23 mriedem tonyb: if you're around i just had to rebase those ^
02:51:28 mriedem just need a re-approve on the stack
02:57:41 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
02:57:42 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684
02:57:42 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
02:57:43 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755
02:58:30 tonyb mriedem: It seems I was off doign the same thing :(
02:59:38 mriedem :( https://review.openstack.org/#/c/516119/5..6/nova/tests/unit/objects/test_request_spec.py
02:59:43 mriedem plus you didn't drop the depends-on
02:59:47 mriedem should i just push mine again?
03:00:52 mriedem looks like i can't, no new changes
03:01:55 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
03:01:55 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
03:01:56 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Add security release note for OSSA-2017-005 https://review.openstack.org/519755
03:01:56 openstackgerrit Tony Breeds proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684
03:02:37 tonyb mriedem: Fixed my messup and I switch the Depends-On to the merged review which should be harmless
03:03:05 mriedem yeah ok
03:03:09 mriedem you going to re-approve or should i?
03:03:25 tonyb either or
03:03:35 mriedem i will do it
03:03:59 tonyb I was worried about it given I resolved a merge conflict but given yours was the same I'm happy now
03:04:03 tonyb cool.
03:05:07 mriedem done
03:05:14 tonyb \/o
03:05:24 mriedem and now i must go - laura is getting irritated because this is cutting into our 'stranger things' time
03:05:26 tonyb Oh I seem to have a broken neck
03:05:27 tonyb :(
03:05:41 tonyb hehe okay enjoy
03:07:14 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641
03:07:53 takashin mriedem: Hi
03:08:23 takashin mriedem: The condition under which py27 job is executed is wrong in nova-specs.
03:09:17 takashin mriedem: https://review.openstack.org/#/c/508164/ has been merged.
03:10:10 takashin mriedem: But it causes the following error.
03:10:19 tonyb takashin: does it pass locally?
03:11:10 takashin tonyb: does it mean py27 job for https://review.openstack.org/#/c/508164/ ?
03:11:30 openstackgerrit Huan Xie proposed openstack/nova master: [WIP] XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641
03:11:52 takashin following error is: AssertionError: False is not true : specs/queens/approved/symmetric-allocations.rst:61: Line limited to a maximum of 79 characters.
03:12:03 tonyb takashin: Yes if you run tox -repy27 on a clean nova-specs does it work?
03:13:50 tonyb takashin: interesting, I didn't see that in the gate
03:14:05 openstack Launchpad bug 1727602 in OpenStack Compute (nova) "nova-specs: tests.test_titles.TestTitles.test_template fails" [Undecided,In progress] - Assigned to Takashi NATSUME (natsume-takashi)
03:14:05 takashin tonyb: In master, there is a bug https://bugs.launchpad.net/nova/+bug/1727602
03:14:19 takashin tonyb: So it fails.
03:15:06 takashin tonyb: After applying my patch https://review.openstack.org/#/c/517223/, "Line limited to a maximum of 79 characters." occurs.
03:15:43 takashin tonyb: because py27 job was not executed in https://review.openstack.org/#/c/508164/ .
03:16:04 takashin tonyb: So the condition under which py27 job is executed is wrong in nova-specs.
03:16:54 takashin mriedem: Are you around?
03:17:32 tonyb takashin: mriedem just stepped away for the evening.
03:17:43 takashin mriedem: oh..
03:17:56 takashin tonyb: oh..
03:18:16 tonyb takashin: It looks like we'll need a zuul change to always update py27?
03:18:36 tonyb sorry 'run' not 'update'
03:19:20 takashin tonyb: py27 includes template check. So it should be executed when spec(*.rst) was added or updated.
03:20:03 takashin tonyb: So it should be executed when spec(*.rst) is added or updated.
03:20:25 tonyb takashin: That sounds correct to me
03:20:50 takashin tonyb: The condition should be amended.
03:20:58 tonyb takashin: It seems liek py27 is partially a linter
03:21:56 openstackgerrit Merged openstack/nova master: Add microversion to allow setting flavor description https://review.openstack.org/515213
03:24:05 tonyb takashin: I'm sorry I can't see anything that is causeing the py27 jobs to be skipped :(
03:24:40 takashin tonyb: Thank you. I will send an e-mail to openstack-dev mailing list.
03:24:55 tonyb takashin: okay, Thanks
03:31:47 tonyb takashin: It may be that it was a by-product of the zuul v2 -> v3 transition and it isn't skiped anymore
03:32:10 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899
03:32:10 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217
03:32:31 tonyb takashin: can you propose a fix for the line length issue (perhaps it will need to be part of your 517223 change
03:33:33 takashin tonyb: okay. I will submit a patch for the line length issue.
03:36:15 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965
03:36:15 openstackgerrit Jianghua Wang proposed openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580
03:36:16 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899
03:36:16 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217
03:39:20 vivsoni connector['multipath'] always set to default value mentioned in nova/conf/libvirt.py... it is not picking up the value from nova.conf's 'libvirt' section 'volume_use_multipath'
03:39:21 openstack Launchpad bug 1723928 in os-brick "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,New]
03:39:21 vivsoni Have raised launchpad defect https://bugs.launchpad.net/nova/+bug/1723928
03:44:06 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Fix "Line limited to a maximum of 79 characters" https://review.openstack.org/520281
04:22:09 vivsoni Any ref link for nova and os-brick interaction
04:22:35 vivsoni while attaching cinder volume to nova instance
05:13:07 openstackgerrit Merged openstack/nova stable/newton: Correct log message when removing a security group https://review.openstack.org/519586
06:24:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
06:24:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
06:39:10 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217
06:39:10 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965
06:39:11 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
06:39:11 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899
06:41:57 openstackgerrit Nguyen Van Trung proposed openstack/nova-specs master: Add linters test to run spec validation and remove py27 https://review.openstack.org/519561
06:42:04 openstackgerrit Merged openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
06:42:11 openstackgerrit Merged openstack/nova stable/newton: Fix non-parameterized service id in hypervisors sample tests https://review.openstack.org/519763
06:54:18 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
07:23:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
07:43:18 openstackgerrit Alex Xu proposed openstack/nova master: placement: avoid multiple join in _get_all_with_shared https://review.openstack.org/519081
07:48:16 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
07:57:30 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP] Add pagination and changes-since for instance-actions https://review.openstack.org/326326
08:01:25 openstackgerrit Naichuan Sun proposed openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580
08:01:26 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217
08:01:26 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965
08:01:27 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:01:27 openstackgerrit Naichuan Sun proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899
08:01:54 openstackgerrit Nguyen Van Trung proposed openstack/nova-specs master: Add validation to improve pep8 check for designate project https://review.openstack.org/519561
08:10:55 openstackgerrit Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815
08:16:27 openstackgerrit Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815

Earlier   Later