Thursday, 2017-07-13

*** thorst has joined #openstack-powervm00:02
*** thorst has quit IRC00:07
*** svenkat has joined #openstack-powervm01:31
*** jwcroppe has joined #openstack-powervm01:54
*** svenkat has quit IRC02:02
*** thorst has joined #openstack-powervm02:08
*** thorst has quit IRC02:13
*** apearson has joined #openstack-powervm02:37
*** esberglu has joined #openstack-powervm03:00
*** esberglu has quit IRC03:00
*** chhavi has joined #openstack-powervm03:02
*** apearson has quit IRC03:52
*** esberglu has joined #openstack-powervm03:53
*** esberglu has quit IRC03:58
*** apearson has joined #openstack-powervm04:28
*** apearson has quit IRC04:47
*** apearson has joined #openstack-powervm04:49
*** apearson has quit IRC05:13
*** thorst has joined #openstack-powervm05:57
*** thorst has quit IRC06:03
*** esberglu has joined #openstack-powervm06:37
*** esberglu has quit IRC06:38
*** esberglu has joined #openstack-powervm06:38
*** esberglu has quit IRC06:38
*** thorst has joined #openstack-powervm07:59
*** thorst has quit IRC08:03
*** edmondsw has joined #openstack-powervm09:16
*** edmondsw has quit IRC09:20
*** thorst has joined #openstack-powervm10:00
*** thorst has quit IRC10:04
*** esberglu has joined #openstack-powervm10:16
*** esberglu has quit IRC10:16
*** thorst has joined #openstack-powervm11:01
*** thorst has quit IRC11:05
*** esberglu has joined #openstack-powervm11:10
*** esberglu has quit IRC11:14
*** thorst has joined #openstack-powervm11:17
*** smatzek has joined #openstack-powervm11:24
*** svenkat has joined #openstack-powervm11:43
*** svenkat has quit IRC11:47
*** svenkat has joined #openstack-powervm11:48
*** thorst has quit IRC12:06
*** thorst has joined #openstack-powervm12:11
*** jwcroppe has quit IRC12:53
*** apearson has joined #openstack-powervm13:01
*** kylek3h has joined #openstack-powervm13:07
*** jwcroppe has joined #openstack-powervm13:26
*** esberglu has joined #openstack-powervm13:28
*** thorst has quit IRC13:33
*** smatzek has quit IRC13:44
*** smatzek has joined #openstack-powervm14:06
*** thorst has joined #openstack-powervm14:29
*** apearson has quit IRC14:58
*** efried is now known as efried_bbiab15:00
*** apearson has joined #openstack-powervm15:02
*** apearson_ has joined #openstack-powervm15:34
*** jwcroppe has quit IRC15:36
*** apearson has quit IRC15:36
*** efried_bbiab is now known as efried15:38
*** thorst has quit IRC15:45
*** thorst has joined #openstack-powervm15:46
*** apearson_ has quit IRC15:49
*** apearson_ has joined #openstack-powervm15:49
*** thorst has quit IRC15:50
*** apearson_ has quit IRC15:58
*** apearson has joined #openstack-powervm16:00
*** jwcroppe has joined #openstack-powervm16:02
*** apearson has quit IRC16:03
*** jwcroppe_ has joined #openstack-powervm16:09
*** jwcroppe has quit IRC16:13
*** jwcroppe_ has quit IRC16:16
*** apearson has joined #openstack-powervm16:24
*** jwcroppe has joined #openstack-powervm16:27
*** jwcroppe_ has joined #openstack-powervm16:34
*** jwcroppe has quit IRC16:37
*** thorst has joined #openstack-powervm16:46
esbergluefried: 5548 (generating networking api_extensions) is ready for review17:19
esbergluI tested that full commit chain just now and everything looked good17:19
*** jwcroppe has joined #openstack-powervm17:20
*** jwcroppe_ has quit IRC17:22
esbergluefried: Nvm I messed up the push. Hold on until I fix it17:29
*** miltonm has quit IRC17:35
*** jwcroppe_ has joined #openstack-powervm17:51
*** jwcroppe has quit IRC17:52
efriedesberglu Ready now?17:52
esbergluefried: Yep17:52
efriedesberglu Not loving your regex logic.17:57
efriedIt'll probably work forever, but I can break it.17:57
esbergluefried: What's your issue with it17:57
esbergluI was trying to search from the section being edited up to the next "["17:58
esbergluRather than searching the whole file for it17:58
efriedFirst thing: You haven't assured yourself that the var exists in the section you're mucking with.17:58
esbergluYeah, if it doesn't exist it gets added by the else of that block17:59
efriedNono, I mean if it exists in the file, but it does *not* exist in the section you're targeting, it won't get added.  The sed command will no-op.17:59
esbergluefried: We could only search the section in question instead of the whole file18:00
efriedThat could be stated as a limitation - the LIMITATIONS comments needs updating anyway.18:00
efriedNext, your range gizmo should be using the section_regex.  Otherwise you run the risk of spanning too much of the file.18:02
efriedAnd your substitution should be using the var regex too.18:02
efriedNext, if the section in question is the last section in the file, your range gizmo won't work (because no subsequent opening square bracket)18:04
efriedI'm thinking perhaps we need to crib some of devstacks config file libs.18:04
efriedesberglu Ugh, the devstack ini stuff also has the bug of not working if the section is the last one in the file.18:12
efriedMaybe.  I wonder if it goes to EOF if the second half of the range doesn't match....18:12
esbergluefried: I think I read that somewhere when I was messing with this yesterday18:12
esbergluLet me see if I can find it18:13
efriedesberglu Nah, I just confirmed it.18:13
efriedIt does indeed go to EOF.18:13
esbergluAh cool. So I'll just update to use the var_regex and section_regex. And add the new limitation and update the rest.18:14
esbergluSound good? Remember this is only in play during the transitionary period while getting devstack generated tempest.conf working18:15
efriedesberglu Do we have access to devstack at the point where this runs?18:18
esbergluWhat do you mean by that?18:18
efriedesberglu Have we cloned devstack?18:19
esbergluYeah. This is post stack18:19
efriedYeah, I guess we must've18:19
efriedOkay.  So why don't we just get rid of set_conf_option entirely, source devstack/inc/ini-config, and use iniset throughout?18:20
esbergluefried: Sure18:20
efriedThat'll make me a lot more confident that it works.18:21
esbergluefried: Sure. There's only one other place that uses set_conf_option in os_ci_tempest.sh anyways18:22
efriedThat's slightly depressing.18:23
esbergluefried: Actually using iniset is going to be fantastic. Because then I can set any tempest.conf settings that don't have corresponding devstack settings there18:25
esbergluWhile we wait for the fixes in devstack to get through18:25
esbergluMuch smoother transition18:25
*** miltonm has joined #openstack-powervm18:42
*** thorst has quit IRC18:44
*** chhavi has quit IRC19:38
*** apearson has quit IRC20:00
*** apearson has joined #openstack-powervm20:02
*** smatzek has quit IRC20:21
*** esberglu has quit IRC21:12
*** smatzek has joined #openstack-powervm21:15
*** smatzek has quit IRC21:20
*** kylek3h has quit IRC21:20
*** apearson has quit IRC21:21
*** svenkat has quit IRC21:21
*** esberglu has joined #openstack-powervm21:26
*** apearson has joined #openstack-powervm21:28
*** apearson has quit IRC21:41
*** thorst has joined #openstack-powervm21:42
*** thorst has quit IRC21:48
*** apearson has joined #openstack-powervm21:54
*** apearson has quit IRC22:37
*** jwcroppe_ has quit IRC23:09
*** jwcroppe has joined #openstack-powervm23:09
*** jwcroppe has quit IRC23:13
*** jwcroppe has joined #openstack-powervm23:44
*** thorst has joined #openstack-powervm23:48
*** thorst has quit IRC23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!