Tuesday, 2018-01-16

*** markvoelker has joined #refstack00:11
*** tosky has quit IRC00:52
*** markvoelker has quit IRC03:13
*** markvoelker has joined #refstack05:44
*** markvoelker has quit IRC05:49
*** markvoelker has joined #refstack07:46
*** markvoelker has quit IRC08:19
*** markvoelker has joined #refstack10:16
*** tosky has joined #refstack10:27
*** arxcruz|rover is now known as arxcruz|off10:33
*** markvoelker has quit IRC10:50
*** markvoelker has joined #refstack12:47
*** markvoelker has quit IRC13:20
*** markvoelker has joined #refstack13:33
*** openstackgerrit has quit IRC17:33
*** designbybeck has joined #refstack18:07
mtreinishhogepodge, mguiney: what am I supposed to look at? (saw the meeting log)20:47
mguineyah right. I am currently fighting oslo config to try and get the api utils to recognize both refstack and subunit2sql conf flags20:50
mguineywas going to try and forumlate me question a little better before pinging you20:50
mguiney*my20:50
mtreinishmguiney: ah ok20:51
mtreinishmguiney: fwiw my biggest mistake with subunit2sql was using oslo.config and oslo.db to implement it20:51
mtreinishwhen I write consuming code I do not even bother with trying to get the oslo.config stuff working and just call the db api directly with an engine20:52
mguineyi'm honestly at this point just considering adding a section in refstack conf with the info we need, given that its only a few flags20:53
mtreinishmguiney: like: https://github.com/openstack/openstack-health/blob/master/openstack_health/api.py#L92-L101 and20:53
* mguiney nods. i'll try that out20:53
mtreinishmguiney: honestly that's what I'd probably do20:53
mtreinishI find dealing with oslo.config more of a pain than it's worth20:53
mguineycool, thank you20:54
mtreinishalthough, you're supposed to be able to just leverage CONF.import_opt() to load external opts. When you have to start changing sections and stuff it's no fun20:55
* mguiney nods20:58
mguineyi'll continue tinkering on it and see what works best20:59
mguineythank you for input, apologies for the very vague question20:59
mguineyone more thing- i may have a patch to subumit to subunit2sql that just allows for the optional usage of a flag that you currently cannot use, but allows for the handling of "impure" subunit (like that converted to v2 via command line)and doesnt otherwise affect the current implementation21:10
mguineywhat would be the best way to submit that21:10
mguiney^mtreinish21:10
mtreinishmguiney: like?: https://review.openstack.org/#/c/507192/21:11
mguineyoh look21:11
mguineythat's actually exactly the thing21:12
mtreinishmguiney: ok cool, feel free to give that a review then. I'll ping masayukig, larainema, oomichi, and andreaf about it to try and get it merged soon21:18
mguineyexcellent, can do21:18
andreafmtreinish, mguiney: the change looks ok to me, but I'll leave it to masayukig to approve since he reviewed it already before - if he's not around today I can approve it tomorrow morning if it's urgent21:24
* mguiney nods21:24
mguineythank you!21:24
hogepodgethanks mtreinish and andreaf and mguiney! #teamwork21:48

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