Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-16
01:09:45 alex_xu gmann: cleanup few things https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:cleanup_extension
01:09:58 alex_xu gmann: I rememer we have a blueprint for those cleanup, do you have the link?
01:21:49 gmann to track the work
01:21:49 gmann alex_xu: https://etherpad.openstack.org/p/api-extensions-merge-queens
01:21:49 gmann let me search
01:21:49 gmann alex_xu: i think i created etherpad also for that, may be forgot to ping you.
01:21:49 alex_xu gmann: thanks
01:21:49 gmann alex_xu: yea, you can use this https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-queens
01:36:10 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190
02:16:10 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205
02:26:17 openstackgerrit Merged openstack/nova stable/newton: Set group_members when converting to legacy request spec https://review.openstack.org/517868
02:51:09 openstackgerrit Matt Riedemann proposed openstack/nova stable/newton: Pass requested_destination in filter_properties https://review.openstack.org/516119
02:51:10 openstackgerrit Matt Riedemann proposed openstack/nova stable/newton: Validate new image via scheduler during rebuild https://review.openstack.org/519684
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

Earlier   Later