Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-24
19:06:54 efried edmondsw, esberglu, mriedem: https://bugs.launchpad.net/pypowervm/+bug/1766692
19:06:54 openstack Launchpad bug 1766692 in pypowervm "instance.uuid no longer being a str breaks powervm scsi disconnect" [Undecided,New]
19:07:12 efried mriedem: What are the odds of backporting a pypowervm requirements bump to queens & pike?
19:08:33 mriedem not good
19:08:47 mriedem i don't understand why this is an issue on stable though
19:11:56 openstackgerrit Merged openstack/os-traits master: Add compute capabilities traits https://review.openstack.org/546713
19:14:13 edmondsw mriedem because https://review.openstack.org/#/q/Ic6b6308fb1960ec40407e6efde30137b64543e72 was backported to pike and queens
19:15:10 edmondsw I suspect we'll have to do something like https://review.openstack.org/#/c/563314/1/nova_powervm/virt/powervm/vm.py@808 for pike and queens, as much as I hate that, to avoid a pypowervm requirements bump
19:15:26 edmondsw but if we could get a bump, we could avoid that
19:18:25 mriedem edmondsw: those aren't released in pike or queens
19:18:33 mriedem upper-constraints depends on released packages on pypi
19:19:22 mriedem are you guys sure that your CI running against stable isn't using upper-constraints from master?
19:19:55 edmondsw mriedem yes, we're sure
19:19:58 edmondsw but I'm not following you
19:20:15 mriedem your CI should not be pulling in those changes from stable
19:20:36 edmondsw I don't know that we've seen this in CI for stable... we just saw that it merged in stable
19:20:48 edmondsw we've seen this in CI for master
19:20:51 mriedem then you revert those changes before they are released
19:21:13 mriedem or, backport whatever the fix ends up being from master
19:21:52 edmondsw for master, we should just fix pypowervm and bump requirements
19:22:05 edmondsw hence the question of whether we can backport a requirements bump
19:22:23 edmondsw we could try to get the oslo change reverted
19:22:50 efried mriedem: We have seen this in queens, live, this morning.
19:22:59 mriedem efried: in CI?
19:23:03 efried no, live.
19:23:04 mriedem if so, link me to the failed patch
19:23:21 mriedem so someone isn't using upper-constraints in production
19:23:35 efried esberglu: Can you spin up a queens CI run without the pypowervm patch so we can confirm whether this fails there?
19:23:44 efried please and thank you
19:23:54 esberglu efried: Sure
19:24:13 mriedem also, i don't know what "live" means here, in your internal CI/CD pipeline? in a customer site, in some ibm public cloud service, other?
19:32:33 efried mriedem: An internal customer, I think.
19:33:24 efried mriedem: They claim to have installed canonical's nova-compute/Queens and then nova-powervm stable/queens via git clone.
19:33:31 efried and python setup.py install
19:40:02 mriedem alright, well, they aren't using upper-constraints,
19:40:10 mriedem or testing what they are doing apparently if they are pushing that to production
19:50:07 openstackgerrit Merged openstack/nova master: Update os_compute_api:os-flavor-extra-specs:index docs for 2.61 https://review.openstack.org/561404
19:50:13 openstackgerrit Merged openstack/nova master: docs: link to volume multi-attach demo recording https://review.openstack.org/563734
20:00:57 cdent melwitt: I dumped a small amount of state on that bug. 1.8.x is way different and my brain has stalled for the day.
20:01:56 melwitt cdent: cool, thanks for looking into it. much appreciated
20:14:09 openstackgerrit melanie witt proposed openstack/nova master: Use ConsoleAuthToken object to generate authorizations https://review.openstack.org/325414
20:14:10 openstackgerrit melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
20:14:11 openstackgerrit melanie witt proposed openstack/nova master: Update layout docs for running console proxies https://review.openstack.org/557489
20:39:15 openstackgerrit Eric Fried proposed openstack/nova master: Proxy is_volume through DriverBlockDevice https://review.openstack.org/564017
20:39:26 efried mriedem, melwitt, TheJulia: With UT ^
20:39:46 TheJulia efried: awesome!
20:54:55 esberglu efried: Queens CI is fine without the pypowervm patch
20:55:11 esberglu Must have been something wrong on the customers end
20:55:23 efried esberglu: Yeah, what's weird is, I'm talking to Vlad and he's got 1.31.2 installed - which should be fine.
20:55:38 efried esberglu: But he may have had some pip weirdness hanging around before.
20:56:32 efried esberglu: so - that's a relief. We only need to fix this in master. But also, https://review.openstack.org/#/c/560639/ (edmondsw FYI)
20:57:41 edmondsw efried well not quite... they haven't cut oslo.versionedobjects releases for stable pike/queens *yet* but that doesn't mean they aren't going to
20:57:56 efried edmondsw: But they *aren't* going to bump upper-constraints for same.
20:57:57 edmondsw I fully expect they will, or they wouldn't have backported
20:58:14 efried edmondsw: But also ---^
20:58:15 edmondsw efried they won't need to
20:58:23 efried edmondsw: What do you mean?
20:58:56 efried edmondsw: If they don't bump upper-constraints in pike/queens, then it's illegal to use the bad versions of the packages.
20:59:09 efried illegal => unsupported
20:59:16 edmondsw oh, well you're right, they would... I thought u-c was setup differently than it is
20:59:23 edmondsw but why do you think they won't update u-c?
20:59:33 edmondsw I fully expect they will
20:59:37 edmondsw as they have in the past
20:59:52 efried Well, they recognize this one is broken, so hopefully they'll wait until the backports of ----^ merge.
21:00:39 edmondsw efried ok that I agree with
21:15:14 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067
21:15:45 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067
21:51:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged before starting live migration https://review.openstack.org/558001
21:51:18 openstackgerrit Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548
21:51:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604
21:51:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170
21:51:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947
21:51:22 openstackgerrit Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333
21:51:23 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334
21:51:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423
21:51:25 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370
21:51:26 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: compute: use port binding extended API during live migration https://review.openstack.org/551371
21:51:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870
21:51:28 openstackgerrit Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537
22:35:48 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
22:35:49 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
22:35:50 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
22:54:28 openstackgerrit Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596
22:56:22 Kevin_Zheng mriedem I can take the traceback BP, that one seems easy
22:58:41 Kevin_Zheng I already start working on migration one, had some poc in my pc, and setting up multi-node env for further testing last night, since we moved to our public cloud, I have to set up a new one:)
#openstack-nova - 2018-04-25
00:35:50 openstackgerrit Merged openstack/nova master: Remove mox in unit/api/openstack/compute/test_hosts.py https://review.openstack.org/559266
01:40:28 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Address nits in Iac23a574a05e62a99aab20ffce4265f4fb5a44d9 https://review.openstack.org/563802
02:05:47 openstackgerrit Leopard Ma proposed openstack/nova master: fix paused instance not support shelve https://review.openstack.org/541660
02:17:36 openstackgerrit Merged openstack/nova master: Use an independent transaction for _trait_sync https://review.openstack.org/558068
02:20:56 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
03:20:53 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/564092
05:26:29 openstackgerrit Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596
05:32:01 openstackgerrit Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596
05:45:34 TheJulia jaypipes: w/r/t https://review.openstack.org/#/c/563722/ is your feeling still the same given the discussion yesterday. Curious because it looks like the discussion was winding down around the same time.
07:31:17 openstackgerrit Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596
07:34:58 alex_xu gmann: for https://review.openstack.org/#/c/563728/1, I did a test, if we don't provide boot index, the boot from volume will fail, that sounds like a bug for API
07:36:00 gmann alex_xu: yea, i also tried to test this scenario in tempest test - https://review.openstack.org/#/c/563801/
07:36:17 gmann alex_xu: it pass in ocata in pike and fail in queens and mastyer
07:37:30 alex_xu gmann: cool, sounds like we can revert that patch
07:38:31 gmann alex_xu: yea that is one option but matt is trying to fix it without revert -https://review.openstack.org/#/c/563732/2

Earlier   Later