Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-24
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
07:38:45 Kevin_Zheng ping gibi
07:39:08 gmann alex_xu: so either is ok for me revert and revert in queens also or backport the fix till queens
07:39:33 gmann alex_xu: not sure revert is possible for queens as it is bug since queens not just master
07:39:52 alex_xu gmann: got it
07:44:56 openstackgerrit Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596
07:52:26 gibi Kevin_Zheng: pong
07:52:49 Kevin_Zheng hi, I saw your message today
07:53:31 Kevin_Zheng I'm willing to do it, I mean the traceback bp
07:54:05 Kevin_Zheng while I'm thinking about it, I found that it might be hard to test
07:54:12 gibi Kevin_Zheng: cool, thanks! Let me assigne the bp to you.
07:54:38 Kevin_Zheng you know, currently we compare samples with ture notify
07:54:52 Kevin_Zheng but for traceback, if the code line changes
07:55:03 Kevin_Zheng the traceback will also change
07:55:34 gibi Kevin_Zheng: true, I think we have to accept that we will only test parts of the traceback field
07:56:15 Kevin_Zheng I think that means changing the current test algorithm right?
07:56:58 gibi the current test framework allows overwriting what is in the sample for example in case of uuid we always overwrite what is in the samepl
07:57:02 gibi sample
07:57:30 gibi so we just need to set the replacement for traceback to ANY
07:58:00 gibi and then manually look into the traceback field in a single test and verify that some parts are present
08:00:33 gibi this code https://github.com/openstack/nova/blob/8a6665bc58d53f6ba0346760ba86d97d3437bfc5/nova/tests/functional/notification_sample_tests/test_instance.py#L379 does a lot of replacements
08:01:16 kashyap sean-k-mooney: Thanks for the review: https://review.openstack.org/#/c/563926/. Will address
08:01:23 kashyap Good you didn't catch anything more :-)

Earlier   Later