| *** edmondsw has joined #openstack-powervm | 01:18 | |
| *** edmondsw has quit IRC | 01:22 | |
| *** tjakobs has joined #openstack-powervm | 02:31 | |
| *** tjakobs has quit IRC | 02:35 | |
| *** edmondsw has joined #openstack-powervm | 04:32 | |
| *** chhagarw has joined #openstack-powervm | 05:08 | |
| *** edmondsw has quit IRC | 06:49 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 07:42 | |
| *** edmondsw has joined #openstack-powervm | 08:20 | |
| *** edmondsw has quit IRC | 08:25 | |
| *** edmondsw has joined #openstack-powervm | 10:08 | |
| *** edmondsw has quit IRC | 10:13 | |
| *** chhagarw has quit IRC | 10:41 | |
| *** edmondsw has joined #openstack-powervm | 11:57 | |
| *** edmondsw has quit IRC | 12:01 | |
| *** edmondsw has joined #openstack-powervm | 12:07 | |
| *** edmondsw has quit IRC | 12:07 | |
| *** edmondsw has joined #openstack-powervm | 12:36 | |
| *** esberglu has joined #openstack-powervm | 13:26 | |
| *** chhagarw has joined #openstack-powervm | 13:40 | |
| *** prashkre has joined #openstack-powervm | 13:45 | |
| *** chhagarw has quit IRC | 13:50 | |
| esberglu | efried: edmondsw: 6562 when you get a minute. It allows us to get active RMC in the tempest vms | 13:58 |
|---|---|---|
| *** tjakobs has joined #openstack-powervm | 14:08 | |
| edmondsw | esberglu ack | 14:11 |
| edmondsw | esberglu couple questions in the review | 14:29 |
| esberglu | edmondsw: responded | 14:33 |
| *** prashkre has quit IRC | 14:34 | |
| *** Alexey_Abashkin has joined #openstack-powervm | 14:44 | |
| *** AlexeyAbashkin has quit IRC | 14:46 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 14:46 | |
| edmondsw | esberglu +2 | 14:52 |
| esberglu | edmondsw: tx | 14:53 |
| *** prashkre has joined #openstack-powervm | 15:19 | |
| *** prashkre has quit IRC | 15:24 | |
| efried | esberglu: You gonna get back to DiskAdapter today? | 16:02 |
| *** efried is now known as efried_rollin | 16:10 | |
| *** edmondsw has quit IRC | 16:15 | |
| *** edmondsw_ has joined #openstack-powervm | 16:18 | |
| *** tjakobs_ has joined #openstack-powervm | 16:22 | |
| *** edmondsw_ has quit IRC | 16:23 | |
| *** tjakobs has quit IRC | 16:24 | |
| esberglu | efried_rollin: Just put up a new one | 16:24 |
| esberglu | efried_rollin: mriedem is +2 now | 16:39 |
| *** AlexeyAbashkin has quit IRC | 17:16 | |
| *** edmondsw has joined #openstack-powervm | 18:57 | |
| edmondsw | esberglu you saw that mriedem moved back to -1 on diskadapter? | 19:01 |
| esberglu | edmondsw: Yeah, UT is failing | 19:01 |
| edmondsw | yep | 19:02 |
| edmondsw | esberglu I like his 2nd suggestion better | 19:04 |
| esberglu | edmondsw: I don't understand what he means | 19:13 |
| esberglu | Well, more accurately, how to implement it | 19:14 |
| edmondsw | esberglu you should be able to find some "fake" class implementation in nova/tests/unit as an example | 19:16 |
| edmondsw | well, maybe not... I'm not seeing something quickly | 19:21 |
| *** prashkre has joined #openstack-powervm | 19:22 | |
| edmondsw | esberglu but couldn't you just create a fake class that stubs those abstract methods/props and instantiate it in the tests instead of DiskAdapter? | 19:22 |
| edmondsw | the fake class would be a subclass of DiskAdapter, of course | 19:23 |
| edmondsw | probably call it FakeDiskAdapter | 19:23 |
| *** prashkre has quit IRC | 19:33 | |
| esberglu | edmondsw: New patch is up | 19:40 |
| *** efried_rollin is now known as efried | 19:42 | |
| edmondsw | esberglu add a comment explaining why? | 19:44 |
| edmondsw | then I'm +1 | 19:44 |
| efried | esberglu: You're having problems running tox locally? Symptoms? | 19:46 |
| esberglu | efried: Can't find matching version of osprofiler | 19:47 |
| esberglu | I've got py27 working tho | 19:47 |
| efried | So ^ is in which venv? | 19:47 |
| esberglu | pep8, py35 | 19:48 |
| efried | my py35 venv has | 19:48 |
| efried | Name: osprofiler | 19:48 |
| efried | Version: 1.15.2 | 19:48 |
| efried | and my pep8 has 2.0.0 | 19:48 |
| efried | Pretty sure I've rebuilt my pep8 much more recently, so the latter is probably right. | 19:49 |
| efried | esberglu: Try this: | 19:49 |
| efried | source .tox/pep8/bin/activate | 19:49 |
| efried | pip install osprofiler===2.0.0 | 19:49 |
| esberglu | efried: Already did. It worked for py27. py35 and pep8 couldn't find it, some problem confirming the ssl cert | 19:50 |
| esberglu | Could not fetch URL https://pypi.python.org/simple/osprofiler/: There was a problem confirming the ssl certificate: [SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:719) - skipping | 19:50 |
| efried | esberglu: Have you tried rebuilding the env from scratch? tox -re py35 ? | 19:50 |
| esberglu | efried: yes | 19:50 |
| efried | okay. SSL and TLS stuff I punt to edmondsw. May have something to do with a local ssl lib you need to upgrade. | 19:51 |
| edmondsw | odd that it would work for py27 and not for something else, though | 19:53 |
| edmondsw | I'd have said recreate, but if you already tried that... | 19:53 |
| efried | edmondsw: py2 and 3 use different versions of the libs, nah? | 19:53 |
| edmondsw | ah, true | 19:54 |
| efried | i.e. this is something that may require apt-get install python3-dev or python3-ssl[-dev] or whatever | 19:54 |
| edmondsw | but this SSL issue doesn't seem like it would be related to versions | 19:54 |
| esberglu | efried: edmondsw: No biggie, the last patch I uploaded passed zuul's tox and this next is just comments | 19:55 |
| esberglu | I'll figure it out later | 19:55 |
| efried | ight | 19:55 |
| edmondsw | esberglu I'm gonna recreate my tox and see if I hit the same issue | 19:56 |
| efried | edmondsw taking one for the team. | 19:56 |
| edmondsw | go team! | 19:57 |
| esberglu | edmondsw: New patch is up | 19:58 |
| edmondsw | +1 | 19:59 |
| edmondsw | esberglu pep8 and py35 working fine for me with a recreate | 20:27 |
| edmondsw | esberglu maybe try updating openssl | 20:30 |
| edmondsw | what openssl do you currently have? | 20:31 |
| edmondsw | you can check with this: | 20:32 |
| edmondsw | python -c "import ssl; print(ssl.OPENSSL_VERSION)" | 20:32 |
| edmondsw | but I would really hope that's something very current (1.0.2 something) and not something old enough that I would expect that error (0.9.8) | 20:34 |
| esberglu | edmondsw: Sorry been working on other stuff | 21:07 |
| esberglu | python -c "import ssl; print(ssl.OPENSSL_VERSION)" | 21:07 |
| esberglu | OpenSSL 0.9.8zh 14 Jan 2016 | 21:07 |
| edmondsw | there's your problem | 21:07 |
| edmondsw | that doesn't support TLS 1.2 | 21:07 |
| edmondsw | you need newer | 21:07 |
| edmondsw | badly... 0.9.8 isn't safe | 21:07 |
| edmondsw | efried can you take over the 2 dhellmann patches we haven't merged yet from/per http://lists.openstack.org/pipermail/openstack-dev/2018-May/130445.html ? | 21:10 |
| edmondsw | https://review.openstack.org/#/c/555964/ and https://review.openstack.org/#/c/555358/ | 21:10 |
| efried | edmondsw: Has the root problem been addressed? I.e. are ceilometer and neutron available on pypi? | 21:11 |
| edmondsw | dunno, but we already merged the patch for networking-powervm a while ago: https://review.openstack.org/#/c/555936/ | 21:11 |
| edmondsw | and that shouldn't be an issue for nova-powervm | 21:11 |
| efried | sorry, ceilometer and nova. | 21:12 |
| edmondsw | ah, ic what you mean | 21:12 |
| edmondsw | ceilometer is on pypi: https://pypi.org/project/ceilometer/ | 21:13 |
| efried | I'm seeing those projects in pypi, but they're really old, which must mean they're not what we're looking for. Or something. | 21:13 |
| edmondsw | oh nm, that looks empty | 21:13 |
| efried | yeah, and 6years old | 21:13 |
| edmondsw | well, not empty but yeah... old | 21:14 |
| edmondsw | so we probably need to reply to the ML then and ask if anyone is working that, or... ? | 21:14 |
| efried | asking dhellmann in -infro | 21:15 |
| efried | infra | 21:15 |
| efried | heh, /me pictures a gerrit server with a '70s 'fro. | 21:15 |
| edmondsw | ok, I'll stop typing my ML reply | 21:18 |
| esberglu | efried: edmondsw: Got test_minimum_basic working in CI which is the scenario test that will be required for vSCSI | 21:44 |
| efried | nice | 21:44 |
| esberglu | Had to do some hacky stuff for now (time.sleep to wait for active RMC) but at least we know it can work | 21:45 |
| edmondsw | esberglu cool | 21:47 |
| *** esberglu has quit IRC | 21:48 | |
| *** esberglu has joined #openstack-powervm | 21:48 | |
| *** tjakobs_ has quit IRC | 21:49 | |
| *** esberglu has quit IRC | 21:53 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job https://review.openstack.org/555964 | 22:25 |
| edmondsw | efried I think this is what they were saying in -infra | 22:25 |
| edmondsw | well, zuul doesn't like that... | 22:26 |
| edmondsw | ah, I see why | 22:27 |
| efried | edmondsw: remove the hyphen in front of... | 22:27 |
| edmondsw | nah, add a : | 22:27 |
| efried | o | 22:27 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job https://review.openstack.org/555964 | 22:28 |
| efried | Well, I'll just keep an eye out. If it passes, bully. Thanks for following up. | 22:28 |
| edmondsw | no insta-failure this time | 22:28 |
| edmondsw | we'll see if it actually works | 22:28 |
| *** esberglu has joined #openstack-powervm | 22:38 | |
| *** esberglu has quit IRC | 22:38 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job https://review.openstack.org/555964 | 22:40 |
| edmondsw | ^ had to bump pypowervm in lower-constraints | 22:40 |
| *** edmondsw has quit IRC | 22:50 | |
| *** edmondsw has joined #openstack-powervm | 23:11 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job https://review.openstack.org/555964 | 23:46 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!