*** catintheroof has quit IRC | 00:55 | |
*** jkilpatr has quit IRC | 01:32 | |
*** catintheroof has joined #openstack-mistral | 03:48 | |
*** gkadam has joined #openstack-mistral | 03:53 | |
*** catintheroof has quit IRC | 03:55 | |
*** sharatss has joined #openstack-mistral | 05:43 | |
*** shardy has joined #openstack-mistral | 06:54 | |
*** sharatss has quit IRC | 07:23 | |
*** jpich has joined #openstack-mistral | 07:33 | |
*** sharatss has joined #openstack-mistral | 08:58 | |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479804 | 09:00 |
---|---|---|
openstackgerrit | Sharat Sharma proposed openstack/mistral master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479791 | 09:02 |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479804 | 09:03 |
*** livelace has joined #openstack-mistral | 10:03 | |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Use keystoneauth plugins and session instead of keystoneclient https://review.openstack.org/455174 | 10:15 |
*** dmellado is now known as dmellado_ | 10:57 | |
*** dmellado_ is now known as dmellado | 10:58 | |
openstackgerrit | bhavani proposed openstack/mistral-specs master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/463487 | 10:59 |
sharatss | rakhmerov, d0ugal, therve apetrich mgershen pls review this never ending patch https://review.openstack.org/#/c/455174/ | 11:22 |
*** Guest47284 has quit IRC | 11:29 | |
*** rook has joined #openstack-mistral | 11:30 | |
*** rook is now known as Guest7130 | 11:31 | |
openstackgerrit | bhavani proposed openstack/mistral-specs master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/463487 | 11:31 |
*** jkilpatr has joined #openstack-mistral | 12:05 | |
openstackgerrit | bhavani proposed openstack/mistral-specs master: Remove pbr warnerrors in favor of sphinx check https://review.openstack.org/463487 | 12:28 |
d0ugal | sharatss: sure | 12:30 |
sharatss | d0ugal, got a -1 already :D | 12:30 |
d0ugal | oh | 12:30 |
d0ugal | :) | 12:30 |
sharatss | therve, can you pls help me address those comments? | 12:36 |
therve | Maybe? I don't know the answer | 12:38 |
therve | sharatss, He seems to be under the impression that sessions don't work with v2 | 12:39 |
sharatss | actually the code we have committed is working fine in my env with both the versions | 12:40 |
sharatss | therve, ^^ | 12:41 |
sharatss | can someone else verify? | 12:41 |
therve | sharatss, Even if you disable v3? | 12:41 |
sharatss | therve, i will check and paste the results | 12:42 |
therve | OK thanks | 12:42 |
apetrich | sharatss, looking at it | 12:45 |
sharatss | apetrich, thanks | 12:45 |
apetrich | sharatss, I didn't see anything wrong with that patch, but I don't have andras kovi experience with keystone | 13:09 |
*** sharatss has quit IRC | 13:14 | |
*** jtomasek has joined #openstack-mistral | 13:19 | |
*** gkadam has quit IRC | 13:19 | |
*** jtomasek has quit IRC | 13:24 | |
breton | oh wow | 13:24 |
breton | do you guys send the whole service catalog in a header? | 13:25 |
*** catintheroof has joined #openstack-mistral | 13:39 | |
*** mgershen has quit IRC | 13:48 | |
d0ugal | breton: ? | 14:49 |
d0ugal | context? | 14:49 |
therve | d0ugal, https://review.openstack.org/#/c/455174/56/mistralclient/api/httpclient.py | 14:57 |
breton | > No, HTTP does not define any limit. However most web servers do limit size of headers they accept. For example in Apache default limit is 8KB | 14:59 |
breton | you will easily excess 8kb limit by sending service catalog in a header | 14:59 |
d0ugal | :/ | 15:02 |
d0ugal | I guess this is something to learn about | 15:02 |
breton | i have just retrieved service catalog which is 31117 bytes long | 15:02 |
breton | bugreporting time! | 15:02 |
d0ugal | ouch | 15:02 |
therve | I wonder if it was ever tested | 15:08 |
d0ugal | Sounds unlikely :) | 15:11 |
d0ugal | Why would we send it all in the header anyway? | 15:12 |
therve | Well why would you send it at all | 15:14 |
therve | "there is no API to retrieve the endpoints in V2 with an arbitrary user." ? You get it the endpoints with your token | 15:16 |
* therve gently step way | 15:16 | |
breton | therve: the issue is that token and all kind of data for some "target" is sent in additional headers | 15:42 |
breton | therve: the tokens in X-Auth-Token and X-Target-Auth-Token are different | 15:42 |
breton | therve: theoretically mistral could use a service user to fetch service catalog for X-Target-Auth-Token | 15:43 |
breton | therve: i don't know if it does it now or not | 15:43 |
*** d0ugal has quit IRC | 15:46 | |
*** chlong_ has quit IRC | 15:48 | |
*** gkadam has joined #openstack-mistral | 16:11 | |
openstackgerrit | Ryan Brady proposed openstack/mistral-extra master: [WIP] Port Keystone Utilities to mistral-extra https://review.openstack.org/479851 | 16:15 |
*** jpich has quit IRC | 16:52 | |
*** gkadam has quit IRC | 17:13 | |
*** shardy has quit IRC | 17:40 | |
*** livelace has quit IRC | 18:57 | |
*** openstackgerrit has quit IRC | 20:18 | |
*** bobh has joined #openstack-mistral | 20:19 | |
*** catintheroof has quit IRC | 20:53 | |
*** bobh has quit IRC | 22:04 | |
*** bobh has joined #openstack-mistral | 23:03 | |
*** szaher has quit IRC | 23:03 | |
*** szaher has joined #openstack-mistral | 23:06 | |
*** bobh has quit IRC | 23:07 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!