openstackgerrit | ShangXiao proposed openstack/refstack master: Update http with https for doc links in refstack https://review.openstack.org/530709 | 00:44 |
---|---|---|
*** chkumar246_ has joined #refstack | 00:59 | |
*** chkumar246 has quit IRC | 01:01 | |
*** markvoelker has quit IRC | 01:08 | |
*** markvoelker has joined #refstack | 02:05 | |
*** xinliang has quit IRC | 02:57 | |
*** xinliang has joined #refstack | 03:10 | |
*** chkumar246_ has quit IRC | 05:01 | |
*** markvoelker has quit IRC | 06:28 | |
*** markvoelker has joined #refstack | 08:29 | |
*** markvoelker has quit IRC | 09:03 | |
*** markvoelker has joined #refstack | 10:01 | |
*** markvoelker has quit IRC | 10:34 | |
*** markvoelker has joined #refstack | 11:31 | |
*** tosky has joined #refstack | 11:56 | |
*** markvoelker has quit IRC | 12:05 | |
*** markvoelker has joined #refstack | 13:02 | |
*** markvoelker has quit IRC | 13:19 | |
*** markvoelker has joined #refstack | 13:20 | |
*** jaosorior has joined #refstack | 13:53 | |
jaosorior | tosky, 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 |
tosky | jaosorior: isn't the purpose of castellan to be the generic library so that you don't need to depend on barbican explicitly? | 13:57 |
jaosorior | tosky: 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 |
tosky | I see | 14:01 |
tosky | uhm | 14:01 |
jaosorior | uhm... I'll use barbicanclient for now, and if someone has a strong opinion about it we can discuss it further | 14:03 |
jaosorior | it should be fine :) | 14:03 |
tosky | jaosorior: but wait a minute, we don't use clients at all | 14:04 |
tosky | can you depend on the tempest client for barbican? | 14:04 |
jaosorior | tosky: right, that was the confusing part | 14:04 |
jaosorior | tosky: so, you use temptest lib | 14:04 |
jaosorior | which has a bunch of clients there already | 14:04 |
tosky | exactly | 14:05 |
jaosorior | buuuut tempest lib doesn't have barbican support there (nor castellan) | 14:05 |
jaosorior | so... either we use barbicanclient directly, or we add barbicanclient support in tempestlib | 14:05 |
jaosorior | * tempest.lib | 14:06 |
tosky | it won't go in tempest.lib | 14:06 |
tosky | it should go into the barbican plugin for tempest (where it may already be) | 14:06 |
jaosorior | oooh | 14:06 |
jaosorior | right | 14:06 |
jaosorior | let me look for that | 14:06 |
tosky | we may depend on that - bonus point if the dependency is not mandatory if the feature is not used, but let's see | 14:07 |
tosky | we can solve it somehow | 14:07 |
jaosorior | tosky: you are right https://github.com/openstack/barbican-tempest-plugin/blob/master/barbican_tempest_plugin/services/key_manager/json/secret_client.py | 14:08 |
jaosorior | should be there already | 14:08 |
jaosorior | uhm... it doesn't use barbicanclient... and instead reimplements most of the REST stuff... funky | 14:10 |
jaosorior | let me investigate what's up with that | 14:10 |
tosky | jaosorior: that's normal | 14:15 |
tosky | jaosorior: the tempest clients are another implementation of the clients, with more features targeted towards testing | 14:15 |
jaosorior | uhm... ok | 14:16 |
jaosorior | tosky: so... should I add then a dependency for barbican_tempest_plugin and just import that client? | 14:17 |
tosky | jaosorior: yes, that's the idea | 14:17 |
tosky | and (unrelated to the client) please check my comment about the code structure, if possible | 14:18 |
jaosorior | ack | 14:19 |
openstackgerrit | Ade Lee proposed openstack/python-tempestconf master: Make tempest setup image verification aware https://review.openstack.org/528380 | 14:43 |
*** martinkopec has joined #refstack | 15:13 | |
*** martinkopec has quit IRC | 15:55 | |
*** openstackgerrit has quit IRC | 16:19 | |
*** jaosorior has left #refstack | 17:06 | |
*** alai has quit IRC | 17:14 | |
*** alai has joined #refstack | 17:24 | |
*** arxcruz is now known as arxcruz|ruck | 17:44 | |
*** ChanServ sets mode: +r | 18:22 | |
*** jhesketh has quit IRC | 19:08 | |
*** jhesketh has joined #refstack | 19:11 | |
*** openstackgerrit has joined #refstack | 19:33 | |
openstackgerrit | Ade Lee proposed openstack/python-tempestconf master: Make tempest setup image verification aware https://review.openstack.org/528380 | 19:33 |
openstackgerrit | Ade Lee proposed openstack/python-tempestconf master: Make tempest setup image verification aware https://review.openstack.org/528380 | 21:58 |
openstackgerrit | Chris Hoge proposed openstack/refstack master: Disable anonymous result upload https://review.openstack.org/507695 | 22:25 |
openstackgerrit | Chris Hoge proposed openstack/refstack master: Disable anonymous result upload https://review.openstack.org/507695 | 23:09 |
*** tosky has quit IRC | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!