Tuesday, 2016-03-08

*** ppai has joined #openstack-swauth05:06
*** ppai is now known as ppai|afk08:09
openstackgerritMerged openstack/swauth: Don't pass unicode to hmac.new()  https://review.openstack.org/28219108:36
openstackgerritMerged openstack/swauth: Document swift3 compatibility  https://review.openstack.org/28230008:45
peterlisakppai|afk, patch 285195, do you know why the setting up a salt was canceled before? Or what older version supported that? I'm just curious08:46
ppai|afkpeterlisak, I can point you to the commit that did it08:48
*** ppai|afk is now known as ppai08:49
peterlisakppai, great, thanks08:49
ppaipeterlisak, https://github.com/openstack/swauth/commit/e14a7b308:51
ppaipeterlisak, my rationale behind reintroducing it is that it's necessary for fixing swift3 compatibility and it's a tunable now (default off) for anyone paranoid about security08:54
peterlisakyes, I understand. It makes sense08:58
openstackgerritMerged openstack/swauth: Allow configuring salt manually  https://review.openstack.org/28519509:27
ppaipeterlisak, thanks for the quick merge! Please have a look at 285292 as well when you've got time.09:32
peterlisakof course, i'm working on that ;)09:35
peterlisakppai, In general it's a good and working solution but IMHO it would be more clear to have two common complementary functions for building and parsing credentials string (I mean this string "auth_type:key") ... It's just my thought, now I'm going to give +A.10:13
ppaipeterlisak, Ah, I see.10:16
ppaipeterlisak, sorry about the loop in tests, it's easy and makes sure all combinations are tested10:17
peterlisakppai: reviews done, thanks for your help!10:21
ppaipeterlisak, np, thanks for the reviews.10:22
openstackgerritMerged openstack/swauth: Fix changing of auth_type in existing deployments  https://review.openstack.org/28529210:26
*** openstackgerrit has quit IRC10:33
*** openstackgerrit has joined #openstack-swauth10:33
*** ppai has quit IRC10:36
*** nadeem has joined #openstack-swauth14:06
*** nadeem has quit IRC15:21
*** nadeem has joined #openstack-swauth16:25
*** nadeem has quit IRC17:30
*** nadeem has joined #openstack-swauth17:31
*** nadeem has quit IRC19:59
*** nadeem has joined #openstack-swauth20:01

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