Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-12
15:29:23 hansmoleman or the migration status
15:29:32 hansmoleman which probably isn't in osc
15:32:35 sean-k-mooney im going to call it a day early(for me). is there anything i should review before next week?
15:33:00 sean-k-mooney if not i proably wont be on irc until thrudsay
15:39:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Add regression test for bug 1797580 https://review.openstack.org/610088
15:39:17 openstack bug 1797580 in OpenStack Compute (nova) "NoValidHost during live migration after cold migrating to a specified host" [High,Triaged] https://launchpad.net/bugs/1797580 - Assigned to Matt Riedemann (mriedem)
15:39:18 hansmoleman SteelyDan: to fix ^ i'm thinking we should just not persist the RequestSpec.requested_destination
15:39:25 hansmoleman similar to how we've fixed a few related things
15:39:32 hansmoleman requested_destination should be per operation and not saved
15:39:47 SteelyDan aye
15:41:11 leakypipes fried_rice: yup.
15:41:34 leakypipes fried_rice: but that describes the entire provider descriptor file.
15:42:30 fried_rice leakypipes: That schema describes the entire file as originally designed. I'll mod it to be more generic.
15:43:11 fried_rice and include versioning and whatnot
15:54:25 hansmoleman cfriesen: i asked jackding about this earlier, and just left a comment in https://review.openstack.org/#/c/603844/, but it seems we shouldn't need to list the ports again in that new method to check if there are failed port bindings,
15:54:32 hansmoleman b/c we just refreshed the instance info cache before calling that,
15:54:35 hansmoleman which lists the ports for that server
15:54:45 hansmoleman so if we can just rely on the cache, it's a much less heavy cahnge
15:54:46 hansmoleman *change
16:03:48 openstackgerrit Ivaylo Mitev proposed openstack/nova master: VMware: OVA and StrOpt images as VM templates https://review.openstack.org/609736
16:06:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Don't persist RequestSpec.requested_destination https://review.openstack.org/610098
16:07:03 melwitt tssurya: is it cool if I assign this bug to myself for the quota thing? I see you are currently assigned https://bugs.launchpad.net/nova/+bug/1771810
16:07:03 openstack Launchpad bug 1771810 in OpenStack Compute (nova) "Quota calculation connects to all available cells" [Undecided,New] - Assigned to Surya Seetharaman (tssurya)
16:07:27 hansmoleman SteelyDan: easy peasy https://review.openstack.org/#/c/610098/
16:07:38 melwitt tssurya: (if you're not working on it, that is)
16:09:27 SteelyDan I'll be the judge of easy peasy
16:09:34 SteelyDan hansmoleman: did you ever look at artom's patch from yesterday?
16:10:55 hansmoleman i got scared
16:10:59 hansmoleman it's starred, but on
16:11:00 hansmoleman *no
16:12:52 hansmoleman i'll scan it quick
16:18:10 hansmoleman SteelyDan: artom: comments inline
16:18:13 hansmoleman seems fine,
16:18:18 hansmoleman i haven't looked at tests yet
16:18:26 hansmoleman going to lunch and then a date with the mrs
16:19:52 jungleboyj Has anyone seen issues with Cloudinit not working after changing the instances_path ? Have a packstack where I changed that and now neither Bionic or Xenial set up their networking on boot.
16:21:42 melwitt I didn't think instances_path could affect how cloudinit works. does anyone else know if it would be expected to affect cloudinit?
16:23:21 SteelyDan potentially if using configdrive and the proper image isn't loaded?
16:23:55 SteelyDan I'd think more likely that something else got broken along the way
16:25:26 cfriesen hansmoleman: I'm not self.network_api.get_instance_nw_info() will do the right thing, but it's worth taking a closer look.
16:25:59 cfriesen mriedem_away: ^
16:27:36 jungleboyj SteelyDan: Ok. I was trying to look to see if there were errors in the libvirt logs but didn't see anything.
16:28:20 SteelyDan jungleboyj: it's a reach.. you'd need to have pointed it somewhere so it found an image, but just the wrong one.. like I say, I'd look elsewhere than just the instances_path change
16:28:59 jungleboyj SteelyDan: Ok. Will do. Thanks.
16:42:44 openstackgerrit Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989
17:17:02 openstackgerrit Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517
17:17:46 openstackgerrit Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517
18:34:30 openstackgerrit Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517
18:34:54 openstackgerrit melanie witt proposed openstack/nova master: Fixed concurrent access to direct io test file https://review.openstack.org/515091
18:54:23 openstackgerrit Andreas Jaeger proposed openstack/nova master: Zuul: Update barbican experimental job https://review.openstack.org/610141
19:10:05 melwitt fried_rolls, edmondsw: could one of y'all please help triage this powervm bug? https://bugs.launchpad.net/nova/+bug/1771538
19:10:05 openstack Launchpad bug 1771538 in OpenStack Compute (nova) "PowerVM config drive path is not secure" [Undecided,New]
19:10:23 fried_rice melwitt: looking.
19:10:36 melwitt it was found via a bandit scan
19:11:10 fried_rice curse those bandits
19:11:10 melwitt just need someone who knows about powervm to ack it or nak it accordingly
19:11:50 edmondsw looking
19:12:04 melwitt heh
19:12:16 edmondsw seems legitimate
19:12:18 fried_rice seems legit
19:12:23 edmondsw jinx
19:12:57 fried_rice edmondsw: You doing the needful?
19:13:42 edmondsw fried_rice I'll try to take a quick stab at it. Can't spend much time today
19:14:00 fried_rice edmondsw: I just meant marking the bug report
19:14:22 edmondsw yeah, I just want to check the code real quick and then will mark it confirmed
20:54:43 melwitt mriedem: this bug looks in your wheelhouse https://bugs.launchpad.net/nova/+bug/1788619
20:54:44 openstack Launchpad bug 1788619 in Cinder "disk cachemodes should be restricted with multiattached volumes" [High,New]
20:55:42 edleafe \
20:56:14 mriedem melwitt: sounds like it hasn't been tested
20:56:56 mriedem # Shareable disks like for a multi-attach volume need to have the
20:56:56 mriedem # driver cache disabled.
20:56:56 mriedem if getattr(conf, 'shareable', False):
20:56:56 mriedem conf.driver_cache = 'none'
20:58:26 mriedem invalidated the bug
20:58:29 melwitt mriedem: ah, thanks
20:59:18 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Handle IndexError in _populate_neutron_binding_profile https://review.openstack.org/610163
21:29:25 openstackgerrit Matthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive https://review.openstack.org/610174
21:31:41 edmondsw fried_rice ^ quick attempt at a fix, untested as yet
21:39:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix NoneType error in _notify_volume_usage_detach https://review.openstack.org/609518
21:47:03 openstackgerrit Merged openstack/nova master: Skip _remove_deleted_instances_allocations if compute is new https://review.openstack.org/609552
#openstack-nova - 2018-10-13
00:36:20 openstackgerrit melanie witt proposed openstack/nova master: Verify image even if it is cached locally on the compute host https://review.openstack.org/610189
03:14:34 openstackgerrit Matthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive https://review.openstack.org/610174
16:17:01 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Document Stein review priorities https://review.openstack.org/609807
16:24:32 openstackgerrit Andreas Jaeger proposed openstack/nova master: Convert legacy-tempest-dsvm-neutron-src-oslo.versionedobjects job https://review.openstack.org/610271
16:25:44 openstackgerrit Andreas Jaeger proposed openstack/nova master: Convert legacy-tempest-dsvm-neutron-src-oslo.versionedobjects job https://review.openstack.org/610271
16:26:06 openstackgerrit Andreas Jaeger proposed openstack/nova master: Convert legacy-tempest-dsvm-neutron-src-oslo.versionedobjects job https://review.openstack.org/610271
16:31:18 openstackgerrit Andreas Jaeger proposed openstack/nova master: Convert legacy-tempest-dsvm-neutron-src-oslo.versionedobjects job https://review.openstack.org/610271
18:46:10 openstackgerrit Merged openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398
#openstack-nova - 2018-10-14
09:27:26 openstackgerrit Brin Zhang proposed openstack/nova master: Add microversion 2.67 to rest api version history https://review.openstack.org/610349
09:36:05 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Add support for microversion 2.67: BDMv2 volume_type https://review.openstack.org/609743
14:05:09 openstackgerrit zhulingjie proposed openstack/nova master: Python 3 compatibility: fix has_key https://review.openstack.org/610378
20:43:55 openstackgerrit Matthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive https://review.openstack.org/610174
22:06:04 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Replace MB with MiB https://review.openstack.org/608807
22:07:00 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Remove unnecessary redirect https://review.openstack.org/607400
22:07:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
22:07:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
#openstack-nova - 2018-10-15
00:40:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix deprecated base64.decodestring warning https://review.openstack.org/610401
00:41:48 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix deprecated base64.decodestring warning https://review.openstack.org/610401
00:45:13 openstackgerrit Merged openstack/python-novaclient master: Enable split logging for cinder-novaclient interaction https://review.openstack.org/581218
02:23:36 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Remove unnecessary redirect https://review.openstack.org/607402

Earlier   Later