*** gildub has joined #openstack-sdks | 00:18 | |
*** r-daneel has quit IRC | 00:48 | |
*** chenyb4 has joined #openstack-sdks | 00:52 | |
*** dave-mccowan has joined #openstack-sdks | 01:05 | |
*** bobh has joined #openstack-sdks | 01:10 | |
*** bobh has quit IRC | 01:11 | |
*** gildub has quit IRC | 01:32 | |
*** gildub has joined #openstack-sdks | 01:46 | |
*** harlowja has quit IRC | 01:59 | |
*** salv-orl_ has joined #openstack-sdks | 02:04 | |
*** salv-orlando has quit IRC | 02:07 | |
*** gkadam has joined #openstack-sdks | 02:42 | |
*** gkadam has quit IRC | 02:46 | |
*** gildub has quit IRC | 03:13 | |
*** dave-mccowan has quit IRC | 03:14 | |
*** annp has joined #openstack-sdks | 03:26 | |
*** gildub has joined #openstack-sdks | 03:56 | |
*** harlowja has joined #openstack-sdks | 04:11 | |
openstackgerrit | Lingxian Kong proposed openstack/service-types-authority master: Add qinling project https://review.openstack.org/550787 | 04:12 |
---|---|---|
*** gkadam has joined #openstack-sdks | 04:33 | |
*** harlowja has quit IRC | 04:50 | |
*** gkadam has quit IRC | 04:55 | |
*** gildub has quit IRC | 04:59 | |
*** gkadam has joined #openstack-sdks | 06:35 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/552305 | 06:44 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/548672 | 06:49 |
*** gkadam has quit IRC | 07:03 | |
*** gkadam has joined #openstack-sdks | 07:07 | |
*** halbbalda has quit IRC | 07:13 | |
*** halbbalda has joined #openstack-sdks | 07:13 | |
*** gkadam has quit IRC | 07:15 | |
*** halbbalda has quit IRC | 07:20 | |
*** halbbalda has joined #openstack-sdks | 07:21 | |
*** ldl has joined #openstack-sdks | 07:36 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902 | 07:42 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-api-ref master: Updated from global requirements https://review.openstack.org/553242 | 07:43 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/551595 | 07:44 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/551598 | 07:44 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/551601 | 07:46 |
*** halbbalda has quit IRC | 07:48 | |
*** halbbalda has joined #openstack-sdks | 07:48 | |
*** e0ne has joined #openstack-sdks | 07:57 | |
*** gkadam has joined #openstack-sdks | 08:00 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/553257 | 08:02 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/552385 | 08:02 |
*** halbbalda has quit IRC | 08:05 | |
*** ioggstream has joined #openstack-sdks | 08:05 | |
*** halbbalda has joined #openstack-sdks | 08:05 | |
openstackgerrit | Merged openstack/os-api-ref master: Updated from global requirements https://review.openstack.org/553242 | 08:08 |
*** halbbalda has quit IRC | 08:17 | |
*** halbbalda has joined #openstack-sdks | 08:17 | |
*** halbbalda has quit IRC | 08:27 | |
*** halbbalda has joined #openstack-sdks | 08:27 | |
openstackgerrit | Andrey Volkov proposed openstack/cliff master: Allow finding command by partial name https://review.openstack.org/552970 | 08:35 |
*** jpena|off is now known as jpena | 08:43 | |
*** ralonsoh has joined #openstack-sdks | 08:52 | |
*** jpich has joined #openstack-sdks | 09:03 | |
*** cdent has joined #openstack-sdks | 09:12 | |
*** niraj_singh has joined #openstack-sdks | 10:00 | |
niraj_singh | hi i am getting AttributeError: Service Descriptors cannot be set error | 10:00 |
niraj_singh | i dont understand the error please help | 10:01 |
*** chenyb4 has quit IRC | 10:43 | |
*** liuyulong has quit IRC | 11:16 | |
*** cdent_ has joined #openstack-sdks | 11:19 | |
*** Kevin_Zheng_ has joined #openstack-sdks | 11:20 | |
*** pooja-jadhav has joined #openstack-sdks | 11:20 | |
*** mnaser_ has joined #openstack-sdks | 11:20 | |
*** GrineBloude has joined #openstack-sdks | 11:21 | |
*** frickler_ has joined #openstack-sdks | 11:23 | |
*** Anticime1 has joined #openstack-sdks | 11:23 | |
*** kuzko_ has joined #openstack-sdks | 11:23 | |
*** slaweq_ has joined #openstack-sdks | 11:26 | |
*** cdent has quit IRC | 11:27 | |
*** sri_ has quit IRC | 11:27 | |
*** ttsiouts has quit IRC | 11:27 | |
*** GreenBlood has quit IRC | 11:27 | |
*** Kevin_Zheng has quit IRC | 11:27 | |
*** mnaser has quit IRC | 11:27 | |
*** frickler has quit IRC | 11:27 | |
*** Anticimex has quit IRC | 11:27 | |
*** cdent_ is now known as cdent | 11:27 | |
*** mnaser_ is now known as mnaser | 11:27 | |
*** sri_ has joined #openstack-sdks | 11:27 | |
*** pzchen has quit IRC | 11:28 | |
*** pooja_jadhav has quit IRC | 11:28 | |
*** slaweq has quit IRC | 11:28 | |
*** kuzko has quit IRC | 11:28 | |
*** pzchen has joined #openstack-sdks | 11:28 | |
*** Kevin_Zheng_ has quit IRC | 11:28 | |
*** Kevin_Zheng_ has joined #openstack-sdks | 11:28 | |
*** mnaser has quit IRC | 11:28 | |
*** mnaser has joined #openstack-sdks | 11:28 | |
*** ttsiouts has joined #openstack-sdks | 11:29 | |
*** frickler_ is now known as frickler | 11:35 | |
*** annp has quit IRC | 11:48 | |
*** dmellado has quit IRC | 11:50 | |
*** halbbalda has quit IRC | 11:51 | |
*** stevebaker has quit IRC | 11:52 | |
*** halbbalda has joined #openstack-sdks | 11:53 | |
*** dmellado has joined #openstack-sdks | 11:58 | |
*** dtantsur|afk is now known as dtantsur | 12:04 | |
*** thrash|g0ne is now known as thrash | 12:05 | |
*** dmellado has quit IRC | 12:05 | |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Improve Version.matches() method https://review.openstack.org/550251 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Add MicroversionMiddleware https://review.openstack.org/550252 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Allow passing a json_error_formatter to the middleware https://review.openstack.org/550263 | 12:09 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Clean up package version and python version support https://review.openstack.org/550264 | 12:09 |
cdent | elmiko, edleafe: I'd like to merge that ^ stuff to make a release of microversion parse that has it, but it needs some review and probably a bit of tuneup to dust the cdent-ness off it. | 12:10 |
*** edmondsw has joined #openstack-sdks | 12:15 | |
*** ioggstream has quit IRC | 12:17 | |
*** jpena is now known as jpena|lunch | 12:21 | |
elmiko | cdent: ack, i'll add it to my queue for today | 12:22 |
openstackgerrit | Emilien Macchi proposed openstack/python-openstackclient master: neutron: add --mtu for create/set network https://review.openstack.org/553374 | 12:23 |
cdent | thanks elmiko | 12:23 |
*** dmellado has joined #openstack-sdks | 12:23 | |
cdent | I probably use %s a lot so you'll love it | 12:23 |
*** dmellado has quit IRC | 12:30 | |
*** ldl has quit IRC | 12:32 | |
*** chenyb4 has joined #openstack-sdks | 12:41 | |
*** dayou has quit IRC | 12:50 | |
edleafe | cdent: Can you refresh my memory about extract_version? https://review.openstack.org/#/c/496212/4/microversion_parse/__init__.py@165 | 12:57 |
edleafe | Where is the 'versions_list' supposed to come from? | 12:57 |
cdent | see line 58 on this DNM of placement using these changes https://review.openstack.org/#/c/550265/3/nova/api/openstack/placement/deploy.py | 12:58 |
cdent | in placement as it stands right now the VERSIONS are used directly, but since as a library we wouldn't have access to that global, we have to pass it in somehow | 12:59 |
edleafe | What is the need to have this check in the library? IOW, why can't the lib return the version, and have the caller do whatever checking it might need? | 13:01 |
*** dave-mccowan has joined #openstack-sdks | 13:01 | |
*** dayou has joined #openstack-sdks | 13:03 | |
cdent | It's a) a convenience, b) later the MicroversionMiddleware uses the method and there is no interaction with the caller | 13:04 |
cdent | later == two patches further in the stack | 13:05 |
cdent | edleafe: the patches you're looking at so far are what amount to a "middleware construction kit". Most people will not use any of these features if they use the middleware | 13:06 |
edleafe | got it | 13:06 |
edleafe | but I am still not clear on what is supposed to be authoritative when it comes to "acceptable versions" | 13:07 |
edleafe | how do you see this middleware knowing that for a service? | 13:07 |
cdent | that version_list, which is a thing like: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/microversion.py#L36 | 13:08 |
*** ioggstream has joined #openstack-sdks | 13:08 | |
cdent | the DNM think I linked before passes VERSIONS into the middleware | 13:08 |
edleafe | I saw that; I'm still not clear on the benefit of passing that from the service to the middleware | 13:10 |
edleafe | With it, every service using the middleware has to implement a versions list and pass it during setup | 13:10 |
edleafe | Without it, services that care about checking specific versions can do so locally | 13:11 |
cdent | the job of the middleware is twofold: | 13:12 |
cdent | extract a version and put it in the environ to be used by the app | 13:12 |
cdent | to return a 4xx is the microversion is malformed or out of range | 13:12 |
cdent | in order for it to "out of range" is has to know the range | 13:13 |
cdent | we want it to do that job, because we want it to fail early | 13:13 |
cdent | and yes, it does require everyone have a versions list but that strikes me as a good bit of housekeeping (and you also need it for Version.matches() | 13:15 |
edleafe | Is there a way to say "any version is ok" for those that do not do such housekeeping? | 13:17 |
cdent | If you don't do that housekeeping you can't do matches() | 13:18 |
cdent | and you can't do version_handler | 13:18 |
cdent | so you may as well not use microversion-parse | 13:18 |
cdent | I guess I'm not understanding how/when you wouldn't want to do the housekeeping? | 13:18 |
edleafe | *I* would want to do it | 13:19 |
*** dmellado has joined #openstack-sdks | 13:19 | |
edleafe | Just wondering if that is the standard approach | 13:19 |
edleafe | Thinking about projects that have not yet adopted microversions | 13:19 |
edleafe | Would this be an attractive feature, or a turn-off | 13:20 |
*** dmellado has quit IRC | 13:20 | |
edleafe | Maybe it will encourage good microversion behavior :) | 13:20 |
cdent | I think the versions_list provides a useful bit of cognitive scaffodling, without it, you aren't really thinking like a microversioner | 13:21 |
edleafe | I believe the correct term is 'microversioneer' | 13:22 |
cdent | nova does a similar thing, but instead of a list it uses a min and max, but I think that's limiting | 13:23 |
cdent | you can't easily disable a microversion in the middle | 13:23 |
elmiko | cdent: re: %s, i read a really cool blog that actually explained when it would be more efficient to use %s vs. format | 13:25 |
cdent | string interpolation is a mess. I'm of the "pick one and stick to it" school and my choice appears to be because of when I showed up to the party | 13:27 |
elmiko | i was kinda shocked to learn about the performance differences | 13:27 |
elmiko | but i agree, would be cool if they could pick one and stick to it | 13:28 |
elmiko | "string interpolation choice as a carbon dating of python usage" XD | 13:28 |
*** jpena|lunch is now known as jpena | 13:29 | |
edleafe | F-strings FTW! | 13:29 |
edleafe | They would also force people to upgrade to Ptyhon 3.6 | 13:29 |
edleafe | (and they are faster!) | 13:30 |
elmiko | ++ | 13:31 |
*** ralonsoh_ has joined #openstack-sdks | 13:32 | |
*** dave-mccowan has quit IRC | 13:34 | |
*** ralonsoh has quit IRC | 13:35 | |
*** kmy has quit IRC | 13:44 | |
*** kmy has joined #openstack-sdks | 13:45 | |
openstackgerrit | Hengqing Hu proposed openstack/python-openstacksdk master: Allow cascade deletion of load balancer https://review.openstack.org/553404 | 13:49 |
openstackgerrit | Emilien Macchi proposed openstack/python-openstackclient master: neutron: add --mtu for create/set network https://review.openstack.org/553374 | 13:51 |
*** ldl has joined #openstack-sdks | 13:53 | |
openstackgerrit | Hengqing Hu proposed openstack/python-openstacksdk master: Allow cascade deletion of load balancer https://review.openstack.org/553404 | 14:01 |
*** ralonsoh__ has joined #openstack-sdks | 14:04 | |
*** ralonsoh__ is now known as ralonsoh | 14:04 | |
*** ralonsoh_ has quit IRC | 14:07 | |
*** bobh has joined #openstack-sdks | 14:10 | |
-openstackstatus- NOTICE: POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified | 14:10 | |
*** ChanServ changes topic to "POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified" | 14:11 | |
cdent | elmiko: line 45 https://review.openstack.org/#/c/550251/2/microversion_parse/tests/test_extract_version.py | 14:19 |
elmiko | cdent: ack, i /thought/ that was it but wanted to make sure. thanks! | 14:24 |
*** bobh has quit IRC | 14:27 | |
cdent | thanks elmiko end edleafe, I'll do some of the suggested tidy ups and push a new stack. I'm perfectly happy to have things be less than perfect, we can always fix stuff | 14:44 |
cdent | released bugs are the gasoline of opensource | 14:44 |
*** dmellado_ has joined #openstack-sdks | 14:46 | |
edleafe | so... you want to burn it all down? :) | 14:46 |
cdent | only some of it | 14:47 |
openstackgerrit | Andrey Volkov proposed openstack/cliff master: Allow finding command by partial name https://review.openstack.org/552970 | 14:49 |
elmiko | edleafe: lol | 14:50 |
openstackgerrit | Andrey Volkov proposed openstack/cliff master: Allow finding command by partial name https://review.openstack.org/552970 | 14:51 |
*** chenyb4 has quit IRC | 14:52 | |
*** dmellado_ is now known as dmellado | 14:54 | |
openstackgerrit | Dmitry Tantsur proposed openstack/api-wg master: Add guideline on exposing microversions in SDKs https://review.openstack.org/532814 | 14:57 |
dtantsur | updates with the PTG outcome ^^^ | 14:57 |
cdent | dtantsur++ | 14:57 |
elmiko | nice | 14:57 |
*** bobh has joined #openstack-sdks | 14:57 | |
*** bobh has quit IRC | 15:03 | |
*** ldl has quit IRC | 15:03 | |
*** bobh has joined #openstack-sdks | 15:07 | |
*** bobh has quit IRC | 15:09 | |
*** bobh has joined #openstack-sdks | 15:09 | |
*** stevebaker has joined #openstack-sdks | 15:14 | |
*** thrash is now known as thrash|biab | 15:24 | |
*** kmy has quit IRC | 15:31 | |
*** kmy has joined #openstack-sdks | 15:33 | |
*** dave-mccowan has joined #openstack-sdks | 15:36 | |
*** kmalloc has joined #openstack-sdks | 15:53 | |
elmiko | cdent: ping | 16:00 |
cdent | oops | 16:01 |
*** ralonsoh_ has joined #openstack-sdks | 16:01 | |
*** harlowja has joined #openstack-sdks | 16:03 | |
*** thrash|biab is now known as thrash | 16:04 | |
*** ralonsoh has quit IRC | 16:04 | |
*** gildub has joined #openstack-sdks | 16:17 | |
*** e0ne has quit IRC | 16:27 | |
*** Suramya_ has joined #openstack-sdks | 16:28 | |
*** ldl has joined #openstack-sdks | 17:02 | |
*** gkadam has quit IRC | 17:03 | |
*** ralonsoh__ has joined #openstack-sdks | 17:06 | |
*** ldl has quit IRC | 17:06 | |
*** ralonsoh_ has quit IRC | 17:09 | |
*** harlowja has quit IRC | 17:09 | |
cdent | dtantsur, edleafe, elmiko https://etherpad.openstack.org/p/api-sig-newsletter I started out with some energy and then fizzled out, please proof | 17:11 |
edleafe | wow, I opened up that link, and I could hear the fizzing! | 17:12 |
*** dtantsur is now known as dtantsur|afk | 17:15 | |
*** ralonsoh__ has quit IRC | 17:16 | |
cdent | you happy with it edleafe ? | 17:17 |
edleafe | Made two micro-nit corrections, and now LGTM | 17:19 |
cdent | nice | 17:20 |
cdent | I'm often confused on project names, because back when docs wrote docs they preferred lowercase project names | 17:20 |
*** e0ne has joined #openstack-sdks | 17:23 | |
*** gkadam has joined #openstack-sdks | 17:25 | |
*** jpich has quit IRC | 17:49 | |
*** Matias has quit IRC | 17:52 | |
*** slaweq_ has quit IRC | 17:55 | |
*** slaweq has joined #openstack-sdks | 17:55 | |
*** gkadam has quit IRC | 18:00 | |
elmiko | cdent: sorry got pulled away | 18:01 |
cdent | no worries | 18:02 |
elmiko | ooh, i had not noticed that we are cross-posting those to openstack-sigs now as well. i will update my proceedures | 18:08 |
cdent | i don't always remember | 18:09 |
cdent | but I think we said at some point that it was maybe a good idea | 18:09 |
elmiko | yeah, it is a good idea. i had just forgotten we said that | 18:13 |
*** jpena is now known as jpena|off | 18:14 | |
*** harlowja has joined #openstack-sdks | 18:20 | |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356 | 18:24 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212 | 18:24 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Improve Version.matches() method https://review.openstack.org/550251 | 18:25 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Add MicroversionMiddleware https://review.openstack.org/550252 | 18:25 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Allow passing a json_error_formatter to the middleware https://review.openstack.org/550263 | 18:25 |
openstackgerrit | Chris Dent proposed openstack/microversion-parse master: Clean up package version and python version support https://review.openstack.org/550264 | 18:25 |
cdent | elmiko, edleafe ^ I think I got the things you mentioned | 18:25 |
elmiko | cdent: ack | 18:32 |
*** ioggstream has quit IRC | 18:33 | |
openstackgerrit | James Penick proposed openstack/python-openstackclient master: Stop prefetching images in server list https://review.openstack.org/553577 | 18:39 |
edleafe | cdent: will look again soon-ish | 18:49 |
cdent | thanks edleafe, elmiko | 18:50 |
*** e0ne has quit IRC | 18:55 | |
edleafe | cdent: Looking over the HTTP Guidelines doc, it looks like it is logically 3 documents: HTTP caching, HTTP response code guidance, and HTTP method guidance | 19:26 |
*** ChanServ changes topic to "#openstack-sdks" | 19:26 | |
-openstackstatus- NOTICE: The regression stemming from one of yesterday's Zuul security fixes has been rectified, and Devstack/Tempest jobs which encountered POST_FAILURE results over the past 24 hours can safely be rechecked now | 19:26 | |
cdent | yes, breaking things up is probably warranted | 19:26 |
edleafe | Perhaps breaking it into 3 separate docs would make them more "discoverable" | 19:27 |
edleafe | due to the more specific titles | 19:27 |
edleafe | If you like, I can take a crack at it. | 19:27 |
*** frickler has quit IRC | 19:29 | |
elmiko | edleafe: that sounds like a nice idea | 19:30 |
cdent | edleafe: feel free to go for it if you're feeling motivated | 19:30 |
cdent | do we want to leave redirects or something? | 19:30 |
*** frickler has joined #openstack-sdks | 19:30 | |
*** Suramya_ has quit IRC | 19:32 | |
edleafe | not sure - does sound like a good idea just in case someone has bookmarked the current page | 19:32 |
cdent | it would be very bad form for the API-SIG to break the web | 19:33 |
* edleafe tries to remember who it was who wanted to break things more often... | 19:35 | |
elmiko | cdent: ++ | 19:43 |
elmiko | lol | 19:43 |
cdent | edleafe: bah, broken client code is broken client code | 19:45 |
cdent | and in any case when I say "the web" i mean public document urls. I went to library school after all. | 19:45 |
elmiko | can we just have the page return 4xx, "we're not the problem, you are!" | 19:45 |
edleafe | elmiko: "Figure it out yourself!" | 19:46 |
elmiko | so helpful XD | 19:47 |
edleafe | "We're not your babysitter!" | 19:47 |
elmiko | LOL | 19:47 |
elmiko | best.error.message.ever. | 19:47 |
*** dave-mccowan has quit IRC | 19:53 | |
*** dave-mccowan has joined #openstack-sdks | 20:11 | |
*** cdent has quit IRC | 20:30 | |
openstackgerrit | Brian Haley proposed openstack/python-openstackclient master: Fix typo in 'floating ip associate' command and doc https://review.openstack.org/553603 | 20:31 |
*** dave-mccowan has quit IRC | 20:53 | |
*** edmondsw has quit IRC | 21:02 | |
EmilienM | osc-functional-devstack failed on my patch, is it accurate to do recheck? I've seen some messages about failures recently | 21:12 |
EmilienM | failure is in post | 21:13 |
EmilienM | (job passed all tests but fail after, I think it's infra related) | 21:13 |
*** dave-mccowan has joined #openstack-sdks | 21:39 | |
*** bobh has quit IRC | 21:53 | |
*** bobh has joined #openstack-sdks | 21:56 | |
*** bobh has quit IRC | 21:59 | |
*** dave-mccowan has quit IRC | 22:48 | |
*** salv-orl_ has quit IRC | 22:53 | |
*** salv-orlando has joined #openstack-sdks | 22:54 | |
*** salv-orlando has quit IRC | 22:59 | |
openstackgerrit | Merged openstack/osc-lib master: Allow to use the none auth plugin https://review.openstack.org/512699 | 23:34 |
*** salv-orlando has joined #openstack-sdks | 23:54 | |
*** salv-orlando has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!