| *** tjakobs has quit IRC | 00:49 | |
| *** tjakobs has joined #openstack-powervm | 01:05 | |
| *** chhagarw has joined #openstack-powervm | 03:57 | |
| *** tjakobs has quit IRC | 05:47 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 07:20 | |
| *** AlexeyAbashkin has quit IRC | 07:44 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 07:54 | |
| *** k0da has joined #openstack-powervm | 09:36 | |
| *** edmondsw has joined #openstack-powervm | 09:40 | |
| *** edmondsw has quit IRC | 09:45 | |
| *** AlexeyAbashkin has quit IRC | 09:56 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 09:57 | |
| *** AlexeyAbashkin has quit IRC | 11:27 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 11:30 | |
| *** tjakobs has joined #openstack-powervm | 12:58 | |
| *** svenkat has joined #openstack-powervm | 13:18 | |
| *** apearson has joined #openstack-powervm | 13:30 | |
| *** efried is now known as fried_rice | 13:32 | |
| *** edmondsw has joined #openstack-powervm | 13:54 | |
| *** apearson_ has joined #openstack-powervm | 14:29 | |
| *** apearson has quit IRC | 14:32 | |
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: ci check https://review.openstack.org/328315 | 14:55 |
|---|---|---|
| openstackgerrit | Eric Berglund proposed openstack/nova-powervm master: DNM: CI Check2 https://review.openstack.org/328317 | 14:55 |
| *** chhagarw has quit IRC | 15:20 | |
| *** chhagarw has joined #openstack-powervm | 15:20 | |
| *** chhagarw has quit IRC | 15:26 | |
| *** AlexeyAbashkin has quit IRC | 15:40 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 15:42 | |
| *** AlexeyAbashkin has quit IRC | 15:46 | |
| *** k0da has quit IRC | 16:15 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 17:12 | |
| *** fried_rice is now known as fried_rolls | 17:20 | |
| *** AlexeyAbashkin has quit IRC | 17:48 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 18:00 | |
| *** AlexeyAbashkin has quit IRC | 18:04 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 18:08 | |
| *** k0da has joined #openstack-powervm | 18:14 | |
| *** k0da has quit IRC | 18:41 | |
| *** AlexeyAbashkin has quit IRC | 19:21 | |
| *** k0da has joined #openstack-powervm | 19:35 | |
| *** AlexeyAbashkin has joined #openstack-powervm | 19:49 | |
| *** AlexeyAbashkin has quit IRC | 19:53 | |
| *** fried_rolls is now known as fried_rice | 20:05 | |
| tjakobs | edmondsw: if i'm ignoring a conf option, should I keep the option in conf.powervm.py for depraction documentation? | 20:09 |
| tjakobs | (volume refactor patch) https://review.openstack.org/#/c/530816/3/nova_powervm/conf/powervm.py@139 ^ | 20:09 |
| fried_rice | tjakobs: If the conf option is going to go away, there's no reason to keep it in the conf file. A deprecation means you keep observing it for a little while, and then eventually get rid of it (so the old one would be ineffective). Here you're jumping straight to ineffective. | 20:14 |
| fried_rice | (NB: I haven't formulated an opinion as to whether we should get rid of this conf option) | 20:15 |
| tjakobs | fried_rice: ok, i'll just leave it until an agreements is made about the future of these options | 20:20 |
| fried_rice | ack | 20:20 |
| fried_rice | esberglu: Since we're not under governance yet, we need to propose https://review.openstack.org/#/c/542881/ equivalents for nova- and networking-powervm, yes? | 20:21 |
| esberglu | fried_rice: Yep | 20:23 |
| fried_rice | ight. Assume that's on your TODO list. | 20:23 |
| esberglu | fried_rice: Is now | 20:24 |
| fried_rice | :) hugs | 20:24 |
| fried_rice | tjakobs: What are one's options for that conf setting? | 20:26 |
| fried_rice | Like, what could one possibly set it to that's not the default? | 20:27 |
| tjakobs | fried_rice: a custom built volume adapter? that's about the only thing i can think of | 20:27 |
| fried_rice | tjakobs: Custom built by whom? | 20:28 |
| fried_rice | Hell, anyone doing that can monkey-patch it in. | 20:28 |
| tjakobs | exactly... only using what's in nova-powervm i don't think there is anything useful you could set it to | 20:28 |
| edmondsw | fried_rice tjakobs even when you jump straight to ignored, you have to deprecate the conf option so that there's a deprecation warning in the logs telling folks it's being ignored | 20:35 |
| fried_rice | meh, okay | 20:36 |
| edmondsw | not just my opinion, I've seen this done in OpenStack | 20:36 |
| fried_rice | what do they know? | 20:36 |
| edmondsw | lol | 20:36 |
| edmondsw | but yeah, I can't think of any good reason to have that option | 20:36 |
| edmondsw | and I know that in porting things to nova we've talked about having as few conf options as possible, so removing cruft in nova-powervm will only help with that when it comes time for porting | 20:37 |
| fried_rice | ++ | 20:40 |
| *** svenkat has quit IRC | 21:12 | |
| openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm master: Refactor volume drivers https://review.openstack.org/530816 | 21:26 |
| *** k0da has quit IRC | 21:28 | |
| openstackgerrit | Taylor Jakobson proposed openstack/nova-powervm master: Refactor volume drivers https://review.openstack.org/530816 | 21:38 |
| *** apearson_ has quit IRC | 21:45 | |
| *** tjakobs has quit IRC | 21:49 | |
| edmondsw | esberglu did you see the ML post about depends-on the other day? If not, give it a read: http://lists.openstack.org/pipermail/openstack-dev/2018-January/126535.html | 22:33 |
| edmondsw | have a good weekend, folks | 22:58 |
| *** edmondsw has quit IRC | 22:59 | |
| *** AndyWojo has quit IRC | 23:04 | |
| *** AndyWojo has joined #openstack-powervm | 23:04 | |
| *** esberglu has quit IRC | 23:37 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!