| *** jamielennox is now known as jamielennox|away | 00:25 | |
| *** jamielennox|away is now known as jamielennox | 00:31 | |
| reedip | anything new happeneing? | 00:33 |
|---|---|---|
| reedip | armax : never mind, got the info from the new stadium docs :) | 00:36 |
| *** hoangcx has joined #openstack-sdks | 00:38 | |
| reedip | wrong place :P | 00:50 |
| *** yuanying has quit IRC | 00:58 | |
| *** yuanying has joined #openstack-sdks | 01:10 | |
| *** reedip has quit IRC | 01:20 | |
| *** huanxuan has joined #openstack-sdks | 01:22 | |
| *** john-davidge has joined #openstack-sdks | 01:54 | |
| *** john-davidge has quit IRC | 01:59 | |
| *** reedip has joined #openstack-sdks | 02:24 | |
| *** bobh has quit IRC | 02:31 | |
| *** bobh has joined #openstack-sdks | 03:04 | |
| *** bobh has quit IRC | 03:09 | |
| *** yuanying has quit IRC | 03:11 | |
| *** yuanying has joined #openstack-sdks | 03:14 | |
| RuiChen | hi rabel: I saw your discussion in IRC meeting about "router show interface", seems that the key is identify which port is interface and which is gateway, if i'm not wrong, so "port list --router <router> --long" might be helpful, that show "device owner", looks like: network:router_interface/network:router_gateway | 03:21 |
| *** amotoki has quit IRC | 03:22 | |
| *** reedip has quit IRC | 03:25 | |
| *** reedip has joined #openstack-sdks | 03:28 | |
| *** amotoki has joined #openstack-sdks | 03:50 | |
| *** amotoki has quit IRC | 03:56 | |
| *** amotoki has joined #openstack-sdks | 04:00 | |
| *** Qiming has quit IRC | 04:04 | |
| *** Qiming has joined #openstack-sdks | 04:06 | |
| *** gouthamr has quit IRC | 04:14 | |
| *** reedip has quit IRC | 04:17 | |
| *** john-davidge has joined #openstack-sdks | 04:21 | |
| *** amotoki has quit IRC | 04:22 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Remove log translations https://review.openstack.org/447888 | 04:22 |
| *** john-davidge has quit IRC | 04:26 | |
| *** reedip has joined #openstack-sdks | 04:26 | |
| *** frickler has quit IRC | 05:11 | |
| *** frickler has joined #openstack-sdks | 05:12 | |
| *** reedip has quit IRC | 05:14 | |
| *** e0ne has joined #openstack-sdks | 05:25 | |
| *** e0ne has quit IRC | 05:26 | |
| *** rabel has joined #openstack-sdks | 05:26 | |
| *** amotoki has joined #openstack-sdks | 06:05 | |
| *** amotoki_ has joined #openstack-sdks | 06:08 | |
| *** amotoki has quit IRC | 06:10 | |
| *** Serlex has joined #openstack-sdks | 06:29 | |
| *** d0ugal has joined #openstack-sdks | 06:52 | |
| *** d0ugal has quit IRC | 06:54 | |
| *** d0ugal has joined #openstack-sdks | 06:58 | |
| *** d0ugal has quit IRC | 06:58 | |
| *** d0ugal has joined #openstack-sdks | 06:58 | |
| *** hoangcx_ has joined #openstack-sdks | 07:03 | |
| *** hoangcx has quit IRC | 07:05 | |
| *** jpich has joined #openstack-sdks | 07:14 | |
| *** d0ugal has quit IRC | 07:22 | |
| *** reedip has joined #openstack-sdks | 07:23 | |
| *** Qiming has quit IRC | 07:26 | |
| *** d0ugal has joined #openstack-sdks | 07:26 | |
| *** d0ugal has quit IRC | 07:26 | |
| *** d0ugal has joined #openstack-sdks | 07:26 | |
| *** Qiming has joined #openstack-sdks | 07:32 | |
| *** ralonsoh has joined #openstack-sdks | 07:34 | |
| *** ralonsoh_ has joined #openstack-sdks | 07:35 | |
| *** hoangcx has joined #openstack-sdks | 07:39 | |
| *** ralonsoh has quit IRC | 07:39 | |
| *** hoangcx_ has quit IRC | 07:41 | |
| *** ralonsoh_ is now known as ralonsoh | 07:44 | |
| -openstackstatus- NOTICE: Jobs in gate are failing with POST_FAILURE. Infra roots are investigating | 07:44 | |
| *** ChanServ changes topic to "Jobs in gate are failing with POST_FAILURE. Infra roots are investigating" | 07:44 | |
| *** e0ne has joined #openstack-sdks | 08:24 | |
| -openstackstatus- NOTICE: logs.openstack.org has corrupted disks, it's being repaired. Please avoid rechecking until this is fixed | 08:25 | |
| *** ChanServ changes topic to "logs.openstack.org has corrupted disks, it's being repaired. Please avoid rechecking until this is fixed" | 08:25 | |
| *** ssbarnea has quit IRC | 08:54 | |
| *** fzdarsky has joined #openstack-sdks | 08:55 | |
| *** ssbarnea has joined #openstack-sdks | 09:01 | |
| *** annegentle has quit IRC | 09:18 | |
| *** cdent has joined #openstack-sdks | 09:20 | |
| reedip | RuiChen : need your help | 09:36 |
| reedip | RuiChen : https://review.openstack.org/#/c/356263/18 was lost somewhere , couldnt track it. I am pushing it now.... can you review it ? | 09:37 |
| *** yanyanhu has joined #openstack-sdks | 09:37 | |
| reedip | just doing the final test cases | 09:38 |
| reedip | ankur-gupta-f4 : for you as well, when you are awake :D ^^ | 09:38 |
| *** sdague has joined #openstack-sdks | 09:44 | |
| *** john-davidge has joined #openstack-sdks | 09:47 | |
| *** yanyanhu has quit IRC | 09:48 | |
| openstackgerrit | Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263 | 09:59 |
| openstackgerrit | Reedip proposed openstack/python-openstackclient master: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263 | 10:07 |
| *** cdent has quit IRC | 10:07 | |
| *** e0ne has quit IRC | 10:13 | |
| *** amotoki_ has quit IRC | 10:15 | |
| *** e0ne has joined #openstack-sdks | 10:16 | |
| reedip | sindhu : ^^ | 10:25 |
| reedip | thanks | 10:25 |
| *** ralonsoh_ has joined #openstack-sdks | 10:30 | |
| *** ralonsoh has quit IRC | 10:34 | |
| *** hoangcx has quit IRC | 10:35 | |
| *** cdent has joined #openstack-sdks | 10:38 | |
| *** fzdarsky is now known as fzdarsky|afk | 11:08 | |
| *** fzdarsky|afk has quit IRC | 11:10 | |
| *** jkilpatr has joined #openstack-sdks | 11:21 | |
| *** yanyanhu has joined #openstack-sdks | 11:21 | |
| *** yanyanhu has quit IRC | 11:26 | |
| *** john-davidge has quit IRC | 11:32 | |
| *** amotoki has joined #openstack-sdks | 11:32 | |
| *** john-davidge has joined #openstack-sdks | 11:38 | |
| *** cdent has quit IRC | 12:08 | |
| *** edleafe is now known as figleaf | 12:10 | |
| *** chlong has joined #openstack-sdks | 12:40 | |
| openstackgerrit | Cedric Brandily proposed openstack/python-openstackclient master: Enable to create legacy router https://review.openstack.org/449176 | 13:01 |
| *** gouthamr has joined #openstack-sdks | 13:01 | |
| *** cdent has joined #openstack-sdks | 13:05 | |
| *** e0ne has quit IRC | 13:06 | |
| *** cleong has joined #openstack-sdks | 13:10 | |
| *** dave-mccowan has joined #openstack-sdks | 13:18 | |
| *** ralonsoh__ has joined #openstack-sdks | 13:41 | |
| *** ralonsoh_ has quit IRC | 13:43 | |
| *** ZZelle has quit IRC | 13:49 | |
| *** ZZelle has joined #openstack-sdks | 13:50 | |
| *** ZZelle is now known as Guest11182 | 13:50 | |
| *** e0ne has joined #openstack-sdks | 13:53 | |
| *** e0ne has quit IRC | 13:53 | |
| *** Serlex has quit IRC | 13:54 | |
| *** Dinesh_Bhor has quit IRC | 13:55 | |
| *** e0ne has joined #openstack-sdks | 13:56 | |
| *** e0ne has quit IRC | 13:56 | |
| *** amotoki has quit IRC | 13:59 | |
| *** huanxuan has quit IRC | 14:07 | |
| *** Guest11182 is now known as ZZelle_ | 14:12 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: start using api.versions https://review.openstack.org/451618 | 14:16 |
| *** jamielennox is now known as jamielennox|away | 14:16 | |
| *** bobh has joined #openstack-sdks | 14:20 | |
| *** brianw has joined #openstack-sdks | 14:24 | |
| sindhu | reedip: will take a look :) | 14:49 |
| reedip | great sindhu : cheers :D | 14:50 |
| openstackgerrit | Cedric Brandily proposed openstack/python-openstackclient master: Enable to specify which vm fixed-ip to add https://review.openstack.org/452235 | 14:56 |
| *** e0ne has joined #openstack-sdks | 15:18 | |
| *** hongbin has joined #openstack-sdks | 15:23 | |
| *** e0ne has quit IRC | 15:39 | |
| *** ralonsoh__ is now known as ralonsoh | 15:54 | |
| *** jpich has quit IRC | 16:16 | |
| *** cdent has quit IRC | 16:25 | |
| *** ZZelle has joined #openstack-sdks | 16:36 | |
| ZZelle | reedip, hi | 16:37 |
| reedip | hi Cedrick / ZZelle :) | 16:38 |
| ZZelle | reedip, about https://review.openstack.org/#/c/452235/1/doc/source/command-objects/server.rst | 16:38 |
| reedip | yeah | 16:38 |
| *** cdent has joined #openstack-sdks | 16:38 | |
| ZZelle | reedip, i don't see the added value of your proposed change | 16:39 |
| reedip | ZZelle : okay | 16:39 |
| ZZelle | reedip, 'Requested fixed ip address' comes from novaclient help | 16:39 |
| reedip | it may not add too much value , thats why I said nit. But it is in line with the message format followed in OSC. | 16:39 |
| reedip | ZZelle : its not necessary to translate the help messages exactly as that from the original client. But similarity is expected. | 16:40 |
| reedip | ZZelle : I find the message , IMO, short. But this is my opinion. | 16:40 |
| reedip | again, its a nit. If others find it Ok, then I dont have a problem. | 16:41 |
| ZZelle | reedip, short doesn't mean meaningless | 16:42 |
| reedip | ZZelle : does my message change the meaning of the one proposed in the patch? | 16:43 |
| ZZelle | reedip, no, longer without added value IMO | 16:44 |
| *** ralonsoh has quit IRC | 16:44 | |
| reedip | ZZelle ; I am not in agreement with you , but yes, thats a nit. I will revert my analysis, but retain my comment | 16:45 |
| ZZelle | reedip, longer also means line-wrapped and less readable | 16:45 |
| reedip | ZZelle : if we are going beyond 80 characters, yes | 16:45 |
| openstackgerrit | Cedric Brandily proposed openstack/python-openstackclient master: Enable to specify which vm fixed-ip to add https://review.openstack.org/452235 | 17:15 |
| *** e0ne has joined #openstack-sdks | 17:50 | |
| *** reedip is now known as reedip_afk | 17:56 | |
| *** cdent has quit IRC | 18:30 | |
| *** cdent has joined #openstack-sdks | 18:32 | |
| *** john-davidge has quit IRC | 18:40 | |
| *** e0ne has quit IRC | 18:40 | |
| *** e0ne has joined #openstack-sdks | 18:41 | |
| *** e0ne has quit IRC | 18:43 | |
| *** e0ne has joined #openstack-sdks | 18:45 | |
| *** rabel has quit IRC | 18:52 | |
| *** e0ne has quit IRC | 19:02 | |
| *** cleong has quit IRC | 19:12 | |
| *** cdent has quit IRC | 19:17 | |
| *** sdague_ has joined #openstack-sdks | 19:23 | |
| *** john-davidge has joined #openstack-sdks | 19:40 | |
| *** john-davidge has quit IRC | 19:46 | |
| *** sdague_ has quit IRC | 19:51 | |
| *** sdague_ has joined #openstack-sdks | 19:56 | |
| -openstackstatus- NOTICE: lists.openstack.org will be offline from 20:00 to 23:00 UTC for planned upgrade maintenance | 19:59 | |
| *** stevelle has joined #openstack-sdks | 20:11 | |
| *** Qiming has quit IRC | 20:12 | |
| stevelle | elmiko_ or any other API-WG types paying attention? | 20:12 |
| *** Qiming has joined #openstack-sdks | 20:16 | |
| *** gouthamr has quit IRC | 20:22 | |
| *** sdague has quit IRC | 20:36 | |
| figleaf | stevelle: I'm around | 20:38 |
| rosmaita | figleaf: got a question about supporting partial downloads. we are thinking of supporting only single-range requests. so in response to a multi-range request, would it be better to (A) return 501, or (B) return 200 and the entire content | 20:40 |
| *** prg3 has quit IRC | 20:40 | |
| rosmaita | the standard isn't very clear on this point | 20:40 |
| stevelle | figleaf: reference we have been looking at https://tools.ietf.org/html/rfc7233#section-4.4 | 20:41 |
| figleaf | rosmaita: 5xx are only for server-side errors | 20:41 |
| *** sdague_ has quit IRC | 20:41 | |
| rosmaita | well, it's kind of on us because we aren't implementing something | 20:41 |
| figleaf | If you only support single-range, and they request something unsupported, it should be a 400 Bad Request | 20:41 |
| stevelle | our library doesn't support multi-range | 20:41 |
| figleaf | Is that documented? | 20:42 |
| stevelle | we are choosing not to try to work around that limitation at this time | 20:42 |
| *** prg3 has joined #openstack-sdks | 20:43 | |
| figleaf | I still think a 400 with an error message that explains how to correct the request would be best | 20:45 |
| rosmaita | figleaf: i think this is what we have doc'd now: https://developer.openstack.org/api-ref/image/v2/?expanded=download-binary-image-data-detail | 20:45 |
| *** gouthamr has joined #openstack-sdks | 20:46 | |
| figleaf | ah, forgot about 416 | 20:47 |
| figleaf | Not one of the more popular return codes :) | 20:47 |
| rosmaita | nope | 20:47 |
| stevelle | pointing at what we have now (which is for the wrong header) is kind of terrible though :) | 20:47 |
| stevelle | makes me sad | 20:47 |
| stevelle | the current text saying "For an unsatisfiable partial download request" seems to suggest it is used where it probably shouldn't | 20:49 |
| rosmaita | figleaf: our reading of the rfc7233 is that 416 is for inconsistent range requests, but in this case, you could be making an otherwise OK request syntax-wise, we just don't suport multi-range requests | 20:49 |
| rosmaita | (well, my reading, anyway) | 20:49 |
| figleaf | rosmaita: I tend to agree. | 20:50 |
| rosmaita | figleaf: that worries me, because we are both sensible fellows, so if we agree, we are probably misreading the standard | 20:52 |
| rosmaita | :) | 20:52 |
| figleaf | what makes you think I'm sensible??? | 20:52 |
| rosmaita | we tend to agree a lot, it's the only conclusion to draw | 20:53 |
| rosmaita | in any case, you seem to not like the 501 idea | 20:54 |
| figleaf | No, since the server *does* support the functionality. The user can re-formulate the request so that it will succeed | 20:56 |
| stevelle | to advocate for the other side, the server doesn't support the correctly-formatted request | 20:56 |
| stevelle | so "not supported" | 20:56 |
| stevelle | "I can't do that, Dave." is different from "Syntax Error" | 20:57 |
| figleaf | Well, if I were a user, and I got that, I would assume you don't support any downloads | 20:58 |
| * rosmaita is trying to think if there's a response code for "i can do that, and you are authorized, but i refuse" | 20:59 | |
| figleaf | the problem with that is that the server *can't* do it | 20:59 |
| stevelle | I thought 501 was that. Note that we WILL return an Accepts-Range header | 21:00 |
| stevelle | regardless of the status code | 21:00 |
| figleaf | Remember that in the RFC, "method" means GET, PUT, etc. | 21:01 |
| *** Qiming has quit IRC | 21:01 | |
| stevelle | iirc the RFC says "may" mean the method is unsupported... checking | 21:01 |
| figleaf | 4xx is client error. If multi-range is documented as not supported, and a client requests multi-range, that's client error | 21:02 |
| *** Qiming has joined #openstack-sdks | 21:02 | |
| stevelle | I have no horse here. Just trying to thoroughly consider all alternatives. The note on https://tools.ietf.org/html/rfc7233#section-4.4 is the argument for just returning the full image and 200 status | 21:04 |
| stevelle | I see multiple "right" answers, just trying to find more consensus on what is best | 21:04 |
| rosmaita | figleaf: i am worried about the openstack API implications of returing a 400 and then if we later add support ... what do we do? if it's a 501, we are allowed to make the change | 21:05 |
| stevelle | ahh, the meta discussion :) | 21:05 |
| *** prg3 has quit IRC | 21:05 | |
| figleaf | My tiebreaker is always what would be more helpful for the consumer of the API? | 21:05 |
| figleaf | rosmaita: that's a very cynical way of approaching API goodness | 21:06 |
| rosmaita | my ocata experiences have made me very cynical! | 21:06 |
| figleaf | Being "allowed" to fix a 5xx is in the spirit of 5xx responses are bugs | 21:07 |
| rosmaita | ok, so 501 might be confused as "no support at all", that makes it un-good | 21:07 |
| figleaf | and a client should never expect a bug | 21:07 |
| rosmaita | 400 would mean reformulate your request, and possibly RTFM before making another request | 21:07 |
| rosmaita | 200 plus the entire thing appears to be allowed by the standard | 21:08 |
| figleaf | rosmaita: well, the error message in the response should be so helpful that they won't have to RTFM! | 21:08 |
| rosmaita | i think we should add "and RTFM" to all 400s just on general principles | 21:09 |
| stevelle | figleaf: in that case, and error message is more important than the status code chosen :) | 21:09 |
| stevelle | I don't disagree, but still what code | 21:09 |
| figleaf | stevelle: +1 | 21:09 |
| figleaf | You guys could always come to next week's API WG meeting | 21:09 |
| figleaf | stevelle said he'd bring bagels | 21:10 |
| stevelle | lies! | 21:10 |
| rosmaita | i think they only have to be virtual bagels | 21:10 |
| rosmaita | and, you only have to bring them to your keyboard | 21:10 |
| figleaf | Just the holes | 21:10 |
| stevelle | I don't know where to find bagels in PDX at 6am that don't suck | 21:10 |
| * stevelle has homework | 21:11 | |
| rosmaita | so the argument for 400 and against 416 is that it's a properly formatted request, we just won't accept it | 21:11 |
| figleaf | stevemar: get up at 3am and make 'em yourself! | 21:11 |
| stevelle | poor steve | 21:11 |
| stevelle | he has to make me bagels now :) | 21:11 |
| rosmaita | he is going to be very surprised when he sees that irc highlight | 21:12 |
| figleaf | stevemar: sorry. Freakin' tab completion! | 21:12 |
| rosmaita | "wtf? bagels?" | 21:12 |
| figleaf | Hey, Toronto has some good bagels | 21:12 |
| figleaf | rosmaita: 416 seems to me like it fits the case where they request a range that is larger than the actual resource | 21:13 |
| figleaf | This matter is not that at all | 21:13 |
| rosmaita | right | 21:14 |
| figleaf | if they request multiple ranges that are all within the resource length, 416 isn't applicable | 21:14 |
| openstackgerrit | Cedric Brandily proposed openstack/python-openstackclient master: Enable to add/remove port to/from a server https://review.openstack.org/452328 | 21:14 |
| figleaf | But since you don't support multiple ranges, 400 is returned | 21:14 |
| rosmaita | ok, 400 plus informative error message sounds good ... what about just dumping the entire content with a 200 ? | 21:15 |
| stevelle | reviewed where we talked about 400 vs 416, and I don't see any reason to dwell on that here. 400 or 200 is where I think we are at | 21:15 |
| figleaf | rosmaita: wait - are they requesting the whole thing, just in parts? | 21:15 |
| rosmaita | figleaf: not necessarily | 21:15 |
| rosmaita | i think we don't even want to try to find out | 21:16 |
| rosmaita | we are just saying "no multiple range requests are accepted" | 21:16 |
| figleaf | So no, I wouldn't return anything other than what they requested | 21:16 |
| *** sdague has joined #openstack-sdks | 21:16 | |
| figleaf | 'zactly | 21:16 |
| figleaf | brb | 21:17 |
| rosmaita | i'm leaving, too ... have a good weekend, and thanks for the help | 21:17 |
| figleaf | no worries - enjoy your weekend! | 21:24 |
| *** sdague has quit IRC | 21:33 | |
| *** prg3 has joined #openstack-sdks | 21:47 | |
| -openstackstatus- NOTICE: The upgrade maintenance for lists.openstack.org has been completed and it is back online. | 21:51 | |
| *** bobh has quit IRC | 22:18 | |
| *** figleaf is now known as edleafe | 22:18 | |
| *** gouthamr has quit IRC | 22:34 | |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: start using api.versions https://review.openstack.org/451618 | 22:39 |
| *** john-davidge has joined #openstack-sdks | 23:05 | |
| *** john-davidge has quit IRC | 23:10 | |
| *** bobh has joined #openstack-sdks | 23:29 | |
| *** bobh has quit IRC | 23:30 | |
| *** bobh has joined #openstack-sdks | 23:31 | |
| *** bobh has quit IRC | 23:36 | |
| *** prg3 has quit IRC | 23:41 | |
| *** ZZelle has quit IRC | 23:42 | |
| *** prg3 has joined #openstack-sdks | 23:45 | |
| *** gouthamr has joined #openstack-sdks | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!