Wednesday, 2018-05-16

*** edmondsw has joined #openstack-powervm01:18
*** edmondsw has quit IRC01:22
*** tjakobs has joined #openstack-powervm02:31
*** tjakobs has quit IRC02:35
*** edmondsw has joined #openstack-powervm04:32
*** chhagarw has joined #openstack-powervm05:08
*** edmondsw has quit IRC06:49
*** AlexeyAbashkin has joined #openstack-powervm07:42
*** edmondsw has joined #openstack-powervm08:20
*** edmondsw has quit IRC08:25
*** edmondsw has joined #openstack-powervm10:08
*** edmondsw has quit IRC10:13
*** chhagarw has quit IRC10:41
*** edmondsw has joined #openstack-powervm11:57
*** edmondsw has quit IRC12:01
*** edmondsw has joined #openstack-powervm12:07
*** edmondsw has quit IRC12:07
*** edmondsw has joined #openstack-powervm12:36
*** esberglu has joined #openstack-powervm13:26
*** chhagarw has joined #openstack-powervm13:40
*** prashkre has joined #openstack-powervm13:45
*** chhagarw has quit IRC13:50
esbergluefried: edmondsw: 6562 when you get a minute. It allows us to get active RMC in the tempest vms13:58
*** tjakobs has joined #openstack-powervm14:08
edmondswesberglu ack14:11
edmondswesberglu couple questions in the review14:29
esbergluedmondsw: responded14:33
*** prashkre has quit IRC14:34
*** Alexey_Abashkin has joined #openstack-powervm14:44
*** AlexeyAbashkin has quit IRC14:46
*** Alexey_Abashkin is now known as AlexeyAbashkin14:46
edmondswesberglu +214:52
esbergluedmondsw: tx14:53
*** prashkre has joined #openstack-powervm15:19
*** prashkre has quit IRC15:24
efriedesberglu: You gonna get back to DiskAdapter today?16:02
*** efried is now known as efried_rollin16:10
*** edmondsw has quit IRC16:15
*** edmondsw_ has joined #openstack-powervm16:18
*** tjakobs_ has joined #openstack-powervm16:22
*** edmondsw_ has quit IRC16:23
*** tjakobs has quit IRC16:24
esbergluefried_rollin: Just put up a new one16:24
esbergluefried_rollin: mriedem is +2 now16:39
*** AlexeyAbashkin has quit IRC17:16
*** edmondsw has joined #openstack-powervm18:57
edmondswesberglu you saw that mriedem moved back to -1 on diskadapter?19:01
esbergluedmondsw: Yeah, UT is failing19:01
edmondswesberglu I like his 2nd suggestion better19:04
esbergluedmondsw: I don't understand what he means19:13
esbergluWell, more accurately, how to implement it19:14
edmondswesberglu you should be able to find some "fake" class implementation in nova/tests/unit as an example19:16
edmondswwell, maybe not... I'm not seeing something quickly19:21
*** prashkre has joined #openstack-powervm19:22
edmondswesberglu 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
edmondswthe fake class would be a subclass of DiskAdapter, of course19:23
edmondswprobably call it FakeDiskAdapter19:23
*** prashkre has quit IRC19:33
esbergluedmondsw: New patch is up19:40
*** efried_rollin is now known as efried19:42
edmondswesberglu add a comment explaining why?19:44
edmondswthen I'm +119:44
efriedesberglu: You're having problems running tox locally?  Symptoms?19:46
esbergluefried: Can't find matching version of osprofiler19:47
esbergluI've got py27 working tho19:47
efriedSo ^ is in which venv?19:47
esberglupep8, py3519:48
efriedmy py35 venv has19:48
efriedName: osprofiler19:48
efriedVersion: 1.15.219:48
efriedand my pep8 has 2.0.019:48
efriedPretty sure I've rebuilt my pep8 much more recently, so the latter is probably right.19:49
efriedesberglu: Try this:19:49
efriedsource .tox/pep8/bin/activate19:49
efriedpip install osprofiler===2.0.019:49
esbergluefried: Already did. It worked for py27. py35 and pep8 couldn't find it, some problem confirming the ssl cert19:50
esbergluCould not fetch URL There was a problem confirming the ssl certificate: [SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:719) - skipping19:50
efriedesberglu: Have you tried rebuilding the env from scratch?  tox -re py35 ?19:50
esbergluefried: yes19:50
efriedokay.  SSL and TLS stuff I punt to edmondsw.  May have something to do with a local ssl lib you need to upgrade.19:51
edmondswodd that it would work for py27 and not for something else, though19:53
edmondswI'd have said recreate, but if you already tried that...19:53
efriededmondsw: py2 and 3 use different versions of the libs, nah?19:53
edmondswah, true19:54
efriedi.e. this is something that may require apt-get install python3-dev or python3-ssl[-dev] or whatever19:54
edmondswbut this SSL issue doesn't seem like it would be related to versions19:54
esbergluefried: edmondsw: No biggie, the last patch I uploaded passed zuul's tox and this next is just comments19:55
esbergluI'll figure it out later19:55
edmondswesberglu I'm gonna recreate my tox and see if I hit the same issue19:56
efriededmondsw taking one for the team.19:56
edmondswgo team!19:57
esbergluedmondsw: New patch is up19:58
edmondswesberglu pep8 and py35 working fine for me with a recreate20:27
edmondswesberglu maybe try updating openssl20:30
edmondswwhat openssl do you currently have?20:31
edmondswyou can check with this:20:32
edmondswpython -c "import ssl; print(ssl.OPENSSL_VERSION)"20:32
edmondswbut 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
esbergluedmondsw: Sorry been working on other stuff21:07
esberglupython -c "import ssl; print(ssl.OPENSSL_VERSION)"21:07
esbergluOpenSSL 0.9.8zh 14 Jan 201621:07
edmondswthere's your problem21:07
edmondswthat doesn't support TLS 1.221:07
edmondswyou need newer21:07
edmondswbadly... 0.9.8 isn't safe21:07
edmondswefried can you take over the 2 dhellmann patches we haven't merged yet from/per ?21:10
edmondsw and
efriededmondsw: Has the root problem been addressed?  I.e. are ceilometer and neutron available on pypi?21:11
edmondswdunno, but we already merged the patch for networking-powervm a while ago:
edmondswand that shouldn't be an issue for nova-powervm21:11
efriedsorry, ceilometer and nova.21:12
edmondswah, ic what you mean21:12
edmondswceilometer is on pypi:
efriedI'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
edmondswoh nm, that looks empty21:13
efriedyeah, and 6years old21:13
edmondswwell, not empty but yeah... old21:14
edmondswso we probably need to reply to the ML then and ask if anyone is working that, or... ?21:14
efriedasking dhellmann in -infro21:15
efriedheh, /me pictures a gerrit server with a '70s 'fro.21:15
edmondswok, I'll stop typing my ML reply21:18
esbergluefried: edmondsw: Got test_minimum_basic working in CI which is the scenario test that will be required for vSCSI21:44
esbergluHad to do some hacky stuff for now (time.sleep to wait for active RMC) but at least we know it can work21:45
edmondswesberglu cool21:47
*** esberglu has quit IRC21:48
*** esberglu has joined #openstack-powervm21:48
*** tjakobs_ has quit IRC21:49
*** esberglu has quit IRC21:53
openstackgerritMatthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job
edmondswefried I think this is what they were saying in -infra22:25
edmondswwell, zuul doesn't like that...22:26
edmondswah, I see why22:27
efriededmondsw: remove the hyphen in front of...22:27
edmondswnah, add a :22:27
openstackgerritMatthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job
efriedWell, I'll just keep an eye out.  If it passes, bully.  Thanks for following up.22:28
edmondswno insta-failure this time22:28
edmondswwe'll see if it actually works22:28
*** esberglu has joined #openstack-powervm22:38
*** esberglu has quit IRC22:38
openstackgerritMatthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job
edmondsw^ had to bump pypowervm in lower-constraints22:40
*** edmondsw has quit IRC22:50
*** edmondsw has joined #openstack-powervm23:11
openstackgerritMatthew Edmonds proposed openstack/nova-powervm master: add lower-constraints job

Generated by 2.15.3 by Marius Gedminas - find it at!