*** slaweq has joined #openstack-sdks | 00:11 | |
*** mriedem has quit IRC | 00:15 | |
*** slaweq has quit IRC | 00:15 | |
*** ricolin has joined #openstack-sdks | 01:23 | |
*** bobh has joined #openstack-sdks | 01:43 | |
*** bobh has quit IRC | 02:05 | |
*** irclogbot_0 has joined #openstack-sdks | 02:06 | |
*** bobh has joined #openstack-sdks | 02:07 | |
*** slaweq has joined #openstack-sdks | 02:11 | |
*** irclogbot_0 has quit IRC | 02:13 | |
*** openstackgerrit has joined #openstack-sdks | 02:16 | |
openstackgerrit | Bin Lu proposed openstack/python-openstackclient master: add optional argument metric to set metric for routes https://review.opendev.org/637658 | 02:16 |
---|---|---|
*** slaweq has quit IRC | 02:16 | |
*** irclogbot_0 has joined #openstack-sdks | 02:17 | |
*** irclogbot_3 has joined #openstack-sdks | 02:17 | |
*** altlogbot_1 has joined #openstack-sdks | 02:18 | |
*** irclogbot_3 has quit IRC | 02:19 | |
*** irclogbot_0 has quit IRC | 02:19 | |
*** altlogbot_1 has quit IRC | 02:34 | |
*** bobh has quit IRC | 03:00 | |
*** slaweq has joined #openstack-sdks | 03:11 | |
*** slaweq has quit IRC | 03:15 | |
*** whoami-rajat has joined #openstack-sdks | 03:20 | |
*** irclogbot_2 has joined #openstack-sdks | 03:20 | |
*** whoami-rajat has quit IRC | 03:24 | |
*** whoami-rajat has joined #openstack-sdks | 03:25 | |
*** irclogbot_2 has quit IRC | 03:25 | |
*** altlogbot_2 has joined #openstack-sdks | 03:54 | |
*** altlogbot_2 has quit IRC | 03:59 | |
*** slaweq has joined #openstack-sdks | 04:11 | |
*** slaweq has quit IRC | 04:16 | |
*** Luzi has joined #openstack-sdks | 05:43 | |
*** slaweq has joined #openstack-sdks | 06:11 | |
*** slaweq has quit IRC | 06:16 | |
*** altlogbot_0 has joined #openstack-sdks | 06:18 | |
*** altlogbot_0 has quit IRC | 06:23 | |
*** vishakha has joined #openstack-sdks | 06:24 | |
*** altlogbot_0 has joined #openstack-sdks | 06:24 | |
*** altlogbot_0 has quit IRC | 06:29 | |
*** gtema has quit IRC | 06:37 | |
*** gtema_ has joined #openstack-sdks | 06:38 | |
*** ricolin has quit IRC | 06:39 | |
*** slaweq has joined #openstack-sdks | 06:46 | |
*** ricolin has joined #openstack-sdks | 06:46 | |
*** gtema_ has quit IRC | 06:47 | |
*** gtema has joined #openstack-sdks | 06:48 | |
*** altlogbot_2 has joined #openstack-sdks | 06:56 | |
*** altlogbot_1 has joined #openstack-sdks | 06:56 | |
*** altlogbot_2 has quit IRC | 07:01 | |
*** altlogbot_1 has quit IRC | 07:01 | |
*** irclogbot_0 has joined #openstack-sdks | 07:17 | |
*** irclogbot_3 has joined #openstack-sdks | 07:17 | |
*** tosky has joined #openstack-sdks | 07:20 | |
*** irclogbot_3 has quit IRC | 07:21 | |
*** irclogbot_0 has quit IRC | 07:21 | |
*** irclogbot_3 has joined #openstack-sdks | 07:25 | |
*** irclogbot_1 has joined #openstack-sdks | 07:25 | |
*** irclogbot_1 has quit IRC | 07:27 | |
*** irclogbot_3 has quit IRC | 07:27 | |
*** jpich has joined #openstack-sdks | 07:34 | |
*** irclogbot_0 has joined #openstack-sdks | 07:53 | |
*** irclogbot_0 has quit IRC | 07:55 | |
*** ralonsoh has joined #openstack-sdks | 07:59 | |
*** ttsiouts has joined #openstack-sdks | 08:05 | |
*** cdent has joined #openstack-sdks | 08:08 | |
*** irclogbot_0 has joined #openstack-sdks | 08:09 | |
*** irclogbot_2 has joined #openstack-sdks | 08:09 | |
*** irclogbot_0 has quit IRC | 08:13 | |
*** irclogbot_2 has quit IRC | 08:13 | |
*** ttsiouts has quit IRC | 08:17 | |
*** ttsiouts has joined #openstack-sdks | 08:18 | |
*** gkadam has joined #openstack-sdks | 08:28 | |
*** dtantsur|afk is now known as dtantsur | 08:57 | |
*** gtema_ has joined #openstack-sdks | 09:07 | |
*** gtema__ has joined #openstack-sdks | 09:39 | |
*** gtema has quit IRC | 09:41 | |
*** gtema has joined #openstack-sdks | 09:42 | |
*** gtema_ has quit IRC | 09:42 | |
*** gtema has quit IRC | 09:43 | |
*** gtema__ has quit IRC | 09:43 | |
*** gtema has joined #openstack-sdks | 09:44 | |
*** irclogbot_3 has joined #openstack-sdks | 09:49 | |
*** irclogbot_2 has joined #openstack-sdks | 09:49 | |
*** irclogbot_2 has quit IRC | 09:51 | |
*** irclogbot_3 has quit IRC | 09:51 | |
*** gtema_ has joined #openstack-sdks | 09:53 | |
*** gtema has quit IRC | 09:57 | |
*** cdent has quit IRC | 10:00 | |
*** tosky has quit IRC | 10:02 | |
*** tosky has joined #openstack-sdks | 10:05 | |
*** ttsiouts has quit IRC | 10:19 | |
*** ttsiouts has joined #openstack-sdks | 10:20 | |
*** ttsiouts has quit IRC | 10:25 | |
*** altlogbot_3 has joined #openstack-sdks | 10:27 | |
*** altlogbot_2 has joined #openstack-sdks | 10:27 | |
*** altlogbot_3 has quit IRC | 10:29 | |
*** altlogbot_2 has quit IRC | 10:29 | |
*** gtema_ has quit IRC | 10:29 | |
*** altlogbot_3 has joined #openstack-sdks | 10:35 | |
*** altlogbot_3 has quit IRC | 10:39 | |
*** altlogbot_0 has joined #openstack-sdks | 10:41 | |
*** altlogbot_1 has joined #openstack-sdks | 10:41 | |
*** irclogbot_2 has joined #openstack-sdks | 10:41 | |
*** altlogbot_0 has quit IRC | 10:45 | |
*** altlogbot_1 has quit IRC | 10:45 | |
*** irclogbot_2 has quit IRC | 10:45 | |
*** altlogbot_0 has joined #openstack-sdks | 10:57 | |
*** altlogbot_2 has joined #openstack-sdks | 10:57 | |
*** altlogbot_2 has quit IRC | 11:01 | |
*** altlogbot_0 has quit IRC | 11:01 | |
*** gtema has joined #openstack-sdks | 11:07 | |
*** ttsiouts has joined #openstack-sdks | 11:38 | |
*** altlogbot_3 has joined #openstack-sdks | 11:57 | |
*** altlogbot_3 has quit IRC | 12:01 | |
*** gtema has quit IRC | 12:11 | |
*** ttsiouts has quit IRC | 12:12 | |
*** ttsiouts has joined #openstack-sdks | 12:13 | |
*** irclogbot_1 has joined #openstack-sdks | 12:33 | |
*** irclogbot_3 has joined #openstack-sdks | 12:33 | |
*** irclogbot_1 has quit IRC | 12:35 | |
*** irclogbot_3 has quit IRC | 12:35 | |
openstackgerrit | Grégoire Unbekandt proposed openstack/openstacksdk master: Can't create a metadata named key, clear, delete https://review.opendev.org/670293 | 12:35 |
*** altlogbot_1 has joined #openstack-sdks | 12:51 | |
*** altlogbot_1 has quit IRC | 12:53 | |
*** ITD27M01 has quit IRC | 12:54 | |
*** goldyfruit has joined #openstack-sdks | 13:33 | |
*** irclogbot_2 has joined #openstack-sdks | 13:41 | |
*** irclogbot_0 has joined #openstack-sdks | 13:41 | |
*** mriedem has joined #openstack-sdks | 13:42 | |
*** irclogbot_2 has quit IRC | 13:45 | |
*** irclogbot_0 has quit IRC | 13:45 | |
*** whoami-rajat has quit IRC | 13:48 | |
*** whoami-rajat has joined #openstack-sdks | 13:52 | |
openstackgerrit | Grégoire Unbekandt proposed openstack/openstacksdk master: Specify store when importing an image https://review.opendev.org/670309 | 13:58 |
*** Luzi has quit IRC | 14:32 | |
*** ttsiouts has quit IRC | 15:15 | |
*** ttsiouts has joined #openstack-sdks | 15:16 | |
*** dtantsur is now known as dtantsur|afk | 15:36 | |
*** tosky has quit IRC | 15:53 | |
openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Add a per-request global_request_id https://review.opendev.org/670341 | 15:56 |
edleafe | Let the API-SIG Office Hour begin! | 16:00 |
elmiko | \o/ | 16:00 |
edleafe | elmiko: Don't know if you got the letter from Kendall Nelson about groups planning to be in Shanghai | 16:00 |
elmiko | i did see that, is anyone from the sig going to be there? | 16:01 |
elmiko | i will not be ther | 16:01 |
edleafe | I was hoping to ask you and dtantsur|afk, but he's afk, apparently | 16:02 |
edleafe | I won't be there, either | 16:02 |
elmiko | i don't recall if dmitry had mentioned it | 16:05 |
*** altlogbot_1 has joined #openstack-sdks | 16:05 | |
*** altlogbot_1 has quit IRC | 16:07 | |
edleafe | Well, she mentioned that unless someone says "Yes, our team will be at that PTG", the assumption is that they won't | 16:08 |
edleafe | So if dtantsur|afk wants to host a session there, he'll have to a) let Kendall know and b) plan on being lonely :) | 16:09 |
elmiko | heh, good point | 16:11 |
elmiko | i can send an email to dmitry just to confirm if he will be there | 16:11 |
*** ttsiouts has quit IRC | 16:14 | |
*** jpich has quit IRC | 16:17 | |
*** jpich has joined #openstack-sdks | 16:17 | |
cmurphy | hello, i have a question to bring to the api-sig | 16:18 |
cmurphy | We have a change to remove some deprecated config options related to PKI tokens https://review.opendev.org/659434 (cc vishakha) | 16:18 |
cmurphy | for keystone | 16:18 |
openstackgerrit | Merged openstack/openstacksdk master: Can't create a metadata named key, clear, delete https://review.opendev.org/670293 | 16:18 |
openstackgerrit | Merged openstack/openstacksdk master: Repair masakari FT https://review.opendev.org/670145 | 16:18 |
openstackgerrit | Merged openstack/openstacksdk master: Add Python 3 Train unit tests https://review.opendev.org/669449 | 16:18 |
* elmiko looks | 16:19 | |
cmurphy | When those options are unconfigured, one of the APIs can't work and to date we've been raising a 500 to indicate the server is misconfigured | 16:19 |
cmurphy | With this change, there will no longer be any chance for that API to work since the server is effectively always misconfigured | 16:19 |
cmurphy | So the question is, do we keep raising the 500 consistent with how it worked before, or do we change the response to a 410 Gone to be a little more honest about the fact that this API just doesn't work any more | 16:19 |
cmurphy | keeping in mind that keystone doesn't do microversions | 16:19 |
*** factor has quit IRC | 16:20 | |
elmiko | so, off the top of my head if that api is no longer supported i would expect some 4xx error as opposed to 5xx since the client is in error here | 16:20 |
elmiko | i'm not sure if 410 is the proper status or not, i'd have to do a little more reading | 16:21 |
cmurphy | okay, yeah someone else made the point that a 4XX at least indicates that retrying isn't going to do any good | 16:22 |
elmiko | right, imo 4xx means the client needs to do something different. which in this case means don't call those apis anymore XD | 16:22 |
elmiko | are you leaving the routes in for backward compatibility? like, how come you don't just return 404 for those? | 16:23 |
edleafe | cmurphy: Since you don't provide access to the old API behavior, it would be my preference to have keystone handle this case more gracefully. IOW, if it can do what it needs to do without those config settings, just do that | 16:23 |
*** jpich has quit IRC | 16:24 | |
elmiko | ++ good question edleafe | 16:24 |
cmurphy | elmiko: yes we wanted to leave the route there to maintain some kind of backwards compatibility even though that's not really achievable in this case | 16:25 |
elmiko | ack | 16:25 |
cmurphy | edleafe: there's no way to have keystone do the same thing without those config settings, the config setting just set a location of a CA cert file that was read and returned by that GET API | 16:26 |
edleafe | cmurphy: you *do* realize that mordred will hunt you down and punish you, right? :) | 16:26 |
cmurphy | yes >.> | 16:26 |
elmiko | haha | 16:26 |
edleafe | cmurphy: So you're making what amounts to a major version change, without leaving the old version in place. Is that correct? | 16:27 |
cmurphy | I don't think it's that drastic, the API was already badly designed because it depended on the operator generating this file and configuring this parameter, we're just cementing in the behavior it had if the operator failed to do that | 16:29 |
cmurphy | and since PKI tokens are not in use 99% of clouds will already have this behavior | 16:30 |
edleafe | By a "major version change" I meant one where a request that worked before can no longer work the same way. | 16:30 |
edleafe | If it's a case of "no one uses it anyway", then that's a practical matter | 16:31 |
edleafe | In that case, I think that 410 Gone is the proper response | 16:32 |
cmurphy | okay | 16:33 |
cmurphy | I'll take this back to the team, maybe we can find a way to handle this more gracefully, otherwise we'll go with a 410 | 16:33 |
cmurphy | thanks for the advice edleafe elmiko | 16:34 |
elmiko | gladly cmurphy =) | 16:35 |
edleafe | That's what we're here for! :) | 16:36 |
elmiko | speaking of that, i have to leave a little early edleafe | 16:38 |
edleafe | Awww, I'll miss you! | 16:38 |
elmiko | <3 | 16:38 |
elmiko | catch you next week if not sonner o/ | 16:38 |
elmiko | er even sooner! | 16:39 |
edleafe | I'll be here (for a while, at least!) | 16:39 |
*** altlogbot_3 has joined #openstack-sdks | 16:45 | |
*** gkadam has quit IRC | 16:46 | |
*** altlogbot_3 has quit IRC | 16:49 | |
*** ralonsoh has quit IRC | 17:17 | |
*** factor has joined #openstack-sdks | 17:23 | |
*** altlogbot_2 has joined #openstack-sdks | 17:24 | |
*** altlogbot_2 has quit IRC | 17:25 | |
*** altlogbot_3 has joined #openstack-sdks | 17:31 | |
cmurphy | mordred: could you ack https://review.opendev.org/670136 | 17:33 |
*** altlogbot_3 has quit IRC | 17:35 | |
*** altlogbot_2 has joined #openstack-sdks | 17:37 | |
*** altlogbot_2 has quit IRC | 17:41 | |
*** altlogbot_2 has joined #openstack-sdks | 17:44 | |
*** altlogbot_2 has quit IRC | 17:45 | |
mordred | cmurphy: ack | 17:57 |
cmurphy | ty | 18:00 |
*** cdent has joined #openstack-sdks | 18:46 | |
*** tosky has joined #openstack-sdks | 19:26 | |
*** irclogbot_2 has joined #openstack-sdks | 19:29 | |
*** irclogbot_0 has joined #openstack-sdks | 19:29 | |
*** irclogbot_2 has quit IRC | 19:33 | |
*** irclogbot_0 has quit IRC | 19:33 | |
*** altlogbot_3 has joined #openstack-sdks | 19:39 | |
*** altlogbot_2 has joined #openstack-sdks | 19:39 | |
*** altlogbot_3 has quit IRC | 19:41 | |
*** altlogbot_2 has quit IRC | 19:41 | |
*** irclogbot_3 has joined #openstack-sdks | 20:13 | |
*** irclogbot_3 has quit IRC | 20:16 | |
*** whoami-rajat has quit IRC | 20:48 | |
*** altlogbot_2 has joined #openstack-sdks | 21:01 | |
*** altlogbot_2 has quit IRC | 21:05 | |
*** irclogbot_2 has joined #openstack-sdks | 21:24 | |
*** irclogbot_0 has joined #openstack-sdks | 21:24 | |
*** irclogbot_0 has quit IRC | 21:26 | |
*** irclogbot_2 has quit IRC | 21:26 | |
*** mriedem has quit IRC | 21:26 | |
*** irclogbot_1 has joined #openstack-sdks | 21:29 | |
*** irclogbot_0 has joined #openstack-sdks | 21:29 | |
openstackgerrit | Merged openstack/keystoneauth master: Add a per-request global_request_id https://review.opendev.org/670341 | 21:30 |
*** irclogbot_1 has quit IRC | 21:32 | |
*** irclogbot_0 has quit IRC | 21:32 | |
*** cdent has quit IRC | 21:34 | |
openstackgerrit | Eric Fried proposed openstack/keystoneauth master: reno: per-request global_request_id https://review.opendev.org/670399 | 21:48 |
openstackgerrit | Merged openstack/keystoneauth master: reno: per-request global_request_id https://review.opendev.org/670399 | 22:16 |
*** altlogbot_1 has joined #openstack-sdks | 22:19 | |
*** irclogbot_0 has joined #openstack-sdks | 22:19 | |
*** altlogbot_3 has joined #openstack-sdks | 22:19 | |
*** altlogbot_3 has quit IRC | 22:23 | |
*** altlogbot_1 has quit IRC | 22:23 | |
*** irclogbot_0 has quit IRC | 22:24 | |
*** slaweq has quit IRC | 22:56 | |
*** altlogbot_2 has joined #openstack-sdks | 22:57 | |
*** altlogbot_0 has joined #openstack-sdks | 22:57 | |
*** altlogbot_2 has quit IRC | 22:59 | |
*** altlogbot_0 has quit IRC | 22:59 | |
*** irclogbot_3 has joined #openstack-sdks | 23:01 | |
*** irclogbot_3 has quit IRC | 23:06 | |
*** tosky has quit IRC | 23:07 | |
*** slaweq has joined #openstack-sdks | 23:07 | |
*** irclogbot_1 has joined #openstack-sdks | 23:09 | |
*** slaweq has quit IRC | 23:12 | |
*** irclogbot_1 has quit IRC | 23:16 | |
*** goldyfruit has quit IRC | 23:16 | |
*** slaweq has joined #openstack-sdks | 23:23 | |
*** slaweq has quit IRC | 23:27 | |
*** bobh has joined #openstack-sdks | 23:32 | |
*** bobh has quit IRC | 23:37 | |
*** goldyfruit has joined #openstack-sdks | 23:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!