Wednesday, 2018-01-03

openstackgerritShangXiao proposed openstack/refstack master: Update http with https for doc links in refstack  https://review.openstack.org/53070900:44
*** chkumar246_ has joined #refstack00:59
*** chkumar246 has quit IRC01:01
*** markvoelker has quit IRC01:08
*** markvoelker has joined #refstack02:05
*** xinliang has quit IRC02:57
*** xinliang has joined #refstack03:10
*** chkumar246_ has quit IRC05:01
*** markvoelker has quit IRC06:28
*** markvoelker has joined #refstack08:29
*** markvoelker has quit IRC09:03
*** markvoelker has joined #refstack10:01
*** markvoelker has quit IRC10:34
*** markvoelker has joined #refstack11:31
*** tosky has joined #refstack11:56
*** markvoelker has quit IRC12:05
*** markvoelker has joined #refstack13:02
*** markvoelker has quit IRC13:19
*** markvoelker has joined #refstack13:20
*** jaosorior has joined #refstack13:53
jaosoriortosky, chandankumar: Hey, I'm looking into continuing this patch https://review.openstack.org/#/c/528380/1/config_tempest/config_tempest.py   Seems that we're gonna need a client to store the secrets. Are you guys OK with that library depending on barbicanclient? or should I use castellan?13:55
toskyjaosorior: isn't the purpose of castellan to be the generic library so that you don't need to depend on barbican explicitly?13:57
jaosoriortosky: right, but that's for key management; and it surely provides all the facilities for that. Seems that we're using just secret storage, which I don't see much value in attempting to add a bunch of flexibility when we could just use straight barbican.14:00
toskyI see14:01
toskyuhm14:01
jaosorioruhm... I'll use barbicanclient for now, and if someone has a strong opinion about it we can discuss it further14:03
jaosoriorit should be fine :)14:03
toskyjaosorior: but wait a minute, we don't use clients at all14:04
toskycan you depend on the tempest client for barbican?14:04
jaosoriortosky: right, that was the confusing part14:04
jaosoriortosky: so, you use temptest lib14:04
jaosoriorwhich has a bunch of clients there already14:04
toskyexactly14:05
jaosoriorbuuuut tempest lib doesn't have barbican support there (nor castellan)14:05
jaosoriorso... either we use barbicanclient directly, or we add barbicanclient support in tempestlib14:05
jaosorior* tempest.lib14:06
toskyit won't go in tempest.lib14:06
toskyit should go into the barbican plugin for tempest (where it may already be)14:06
jaosorioroooh14:06
jaosoriorright14:06
jaosoriorlet me look for that14:06
toskywe may depend on that - bonus point if the dependency is not mandatory if the feature is not used, but let's see14:07
toskywe can solve it somehow14:07
jaosoriortosky: you are right https://github.com/openstack/barbican-tempest-plugin/blob/master/barbican_tempest_plugin/services/key_manager/json/secret_client.py14:08
jaosoriorshould be there already14:08
jaosorioruhm... it doesn't use barbicanclient... and instead reimplements most of the REST stuff... funky14:10
jaosoriorlet me investigate what's up with that14:10
toskyjaosorior: that's normal14:15
toskyjaosorior: the tempest clients are another implementation of the clients, with more features targeted towards testing14:15
jaosorioruhm... ok14:16
jaosoriortosky: so... should I add then a dependency for barbican_tempest_plugin and just import that client?14:17
toskyjaosorior: yes, that's the idea14:17
toskyand (unrelated to the client) please check my comment about the code structure, if possible14:18
jaosoriorack14:19
openstackgerritAde Lee proposed openstack/python-tempestconf master: Make tempest setup image verification aware  https://review.openstack.org/52838014:43
*** martinkopec has joined #refstack15:13
*** martinkopec has quit IRC15:55
*** openstackgerrit has quit IRC16:19
*** jaosorior has left #refstack17:06
*** alai has quit IRC17:14
*** alai has joined #refstack17:24
*** arxcruz is now known as arxcruz|ruck17:44
*** ChanServ sets mode: +r 18:22
*** jhesketh has quit IRC19:08
*** jhesketh has joined #refstack19:11
*** openstackgerrit has joined #refstack19:33
openstackgerritAde Lee proposed openstack/python-tempestconf master: Make tempest setup image verification aware  https://review.openstack.org/52838019:33
openstackgerritAde Lee proposed openstack/python-tempestconf master: Make tempest setup image verification aware  https://review.openstack.org/52838021:58
openstackgerritChris Hoge proposed openstack/refstack master: Disable anonymous result upload  https://review.openstack.org/50769522:25
openstackgerritChris Hoge proposed openstack/refstack master: Disable anonymous result upload  https://review.openstack.org/50769523:09
*** tosky has quit IRC23:49

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