Thursday, 2018-03-15

*** gildub has joined #openstack-sdks00:18
*** r-daneel has quit IRC00:48
*** chenyb4 has joined #openstack-sdks00:52
*** dave-mccowan has joined #openstack-sdks01:05
*** bobh has joined #openstack-sdks01:10
*** bobh has quit IRC01:11
*** gildub has quit IRC01:32
*** gildub has joined #openstack-sdks01:46
*** harlowja has quit IRC01:59
*** salv-orl_ has joined #openstack-sdks02:04
*** salv-orlando has quit IRC02:07
*** gkadam has joined #openstack-sdks02:42
*** gkadam has quit IRC02:46
*** gildub has quit IRC03:13
*** dave-mccowan has quit IRC03:14
*** annp has joined #openstack-sdks03:26
*** gildub has joined #openstack-sdks03:56
*** harlowja has joined #openstack-sdks04:11
openstackgerritLingxian Kong proposed openstack/service-types-authority master: Add qinling project  https://review.openstack.org/55078704:12
*** gkadam has joined #openstack-sdks04:33
*** harlowja has quit IRC04:50
*** gkadam has quit IRC04:55
*** gildub has quit IRC04:59
*** gkadam has joined #openstack-sdks06:35
openstackgerritOpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements  https://review.openstack.org/55230506:44
openstackgerritOpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements  https://review.openstack.org/54867206:49
*** gkadam has quit IRC07:03
*** gkadam has joined #openstack-sdks07:07
*** halbbalda has quit IRC07:13
*** halbbalda has joined #openstack-sdks07:13
*** gkadam has quit IRC07:15
*** halbbalda has quit IRC07:20
*** halbbalda has joined #openstack-sdks07:21
*** ldl has joined #openstack-sdks07:36
openstackgerritOpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements  https://review.openstack.org/52190207:42
openstackgerritOpenStack Proposal Bot proposed openstack/os-api-ref master: Updated from global requirements  https://review.openstack.org/55324207:43
openstackgerritOpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements  https://review.openstack.org/55159507:44
openstackgerritOpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements  https://review.openstack.org/55159807:44
openstackgerritOpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements  https://review.openstack.org/55160107:46
*** halbbalda has quit IRC07:48
*** halbbalda has joined #openstack-sdks07:48
*** e0ne has joined #openstack-sdks07:57
*** gkadam has joined #openstack-sdks08:00
openstackgerritOpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements  https://review.openstack.org/55325708:02
openstackgerritOpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements  https://review.openstack.org/55238508:02
*** halbbalda has quit IRC08:05
*** ioggstream has joined #openstack-sdks08:05
*** halbbalda has joined #openstack-sdks08:05
openstackgerritMerged openstack/os-api-ref master: Updated from global requirements  https://review.openstack.org/55324208:08
*** halbbalda has quit IRC08:17
*** halbbalda has joined #openstack-sdks08:17
*** halbbalda has quit IRC08:27
*** halbbalda has joined #openstack-sdks08:27
openstackgerritAndrey Volkov proposed openstack/cliff master: Allow finding command by partial name  https://review.openstack.org/55297008:35
*** jpena|off is now known as jpena08:43
*** ralonsoh has joined #openstack-sdks08:52
*** jpich has joined #openstack-sdks09:03
*** cdent has joined #openstack-sdks09:12
*** niraj_singh has joined #openstack-sdks10:00
niraj_singhhi i am getting AttributeError: Service Descriptors cannot be set error10:00
niraj_singhi dont understand the error please help10:01
*** chenyb4 has quit IRC10:43
*** liuyulong has quit IRC11:16
*** cdent_ has joined #openstack-sdks11:19
*** Kevin_Zheng_ has joined #openstack-sdks11:20
*** pooja-jadhav has joined #openstack-sdks11:20
*** mnaser_ has joined #openstack-sdks11:20
*** GrineBloude has joined #openstack-sdks11:21
*** frickler_ has joined #openstack-sdks11:23
*** Anticime1 has joined #openstack-sdks11:23
*** kuzko_ has joined #openstack-sdks11:23
*** slaweq_ has joined #openstack-sdks11:26
*** cdent has quit IRC11:27
*** sri_ has quit IRC11:27
*** ttsiouts has quit IRC11:27
*** GreenBlood has quit IRC11:27
*** Kevin_Zheng has quit IRC11:27
*** mnaser has quit IRC11:27
*** frickler has quit IRC11:27
*** Anticimex has quit IRC11:27
*** cdent_ is now known as cdent11:27
*** mnaser_ is now known as mnaser11:27
*** sri_ has joined #openstack-sdks11:27
*** pzchen has quit IRC11:28
*** pooja_jadhav has quit IRC11:28
*** slaweq has quit IRC11:28
*** kuzko has quit IRC11:28
*** pzchen has joined #openstack-sdks11:28
*** Kevin_Zheng_ has quit IRC11:28
*** Kevin_Zheng_ has joined #openstack-sdks11:28
*** mnaser has quit IRC11:28
*** mnaser has joined #openstack-sdks11:28
*** ttsiouts has joined #openstack-sdks11:29
*** frickler_ is now known as frickler11:35
*** annp has quit IRC11:48
*** dmellado has quit IRC11:50
*** halbbalda has quit IRC11:51
*** stevebaker has quit IRC11:52
*** halbbalda has joined #openstack-sdks11:53
*** dmellado has joined #openstack-sdks11:58
*** dtantsur|afk is now known as dtantsur12:04
*** thrash|g0ne is now known as thrash12:05
*** dmellado has quit IRC12:05
openstackgerritChris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ  https://review.openstack.org/49535612:09
openstackgerritChris Dent proposed openstack/microversion-parse master: Import functions from placement  https://review.openstack.org/49621212:09
openstackgerritChris Dent proposed openstack/microversion-parse master: Improve Version.matches() method  https://review.openstack.org/55025112:09
openstackgerritChris Dent proposed openstack/microversion-parse master: Add MicroversionMiddleware  https://review.openstack.org/55025212:09
openstackgerritChris Dent proposed openstack/microversion-parse master: Allow passing a json_error_formatter to the middleware  https://review.openstack.org/55026312:09
openstackgerritChris Dent proposed openstack/microversion-parse master: Clean up package version and python version support  https://review.openstack.org/55026412:09
cdentelmiko, 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-sdks12:15
*** ioggstream has quit IRC12:17
*** jpena is now known as jpena|lunch12:21
elmikocdent: ack, i'll add it to my queue for today12:22
openstackgerritEmilien Macchi proposed openstack/python-openstackclient master: neutron: add --mtu for create/set network  https://review.openstack.org/55337412:23
cdentthanks elmiko12:23
*** dmellado has joined #openstack-sdks12:23
cdentI probably use %s a lot so you'll love it12:23
*** dmellado has quit IRC12:30
*** ldl has quit IRC12:32
*** chenyb4 has joined #openstack-sdks12:41
*** dayou has quit IRC12:50
edleafecdent: Can you refresh my memory about extract_version? https://review.openstack.org/#/c/496212/4/microversion_parse/__init__.py@16512:57
edleafeWhere is the 'versions_list' supposed to come from?12:57
cdentsee line 58 on this DNM of placement using these changes https://review.openstack.org/#/c/550265/3/nova/api/openstack/placement/deploy.py12:58
cdentin 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 somehow12:59
edleafeWhat 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-sdks13:01
*** dayou has joined #openstack-sdks13:03
cdentIt's a) a convenience, b) later the MicroversionMiddleware uses the method and there is no interaction with the caller13:04
cdentlater == two patches further in the stack13:05
cdentedleafe: 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 middleware13:06
edleafegot it13:06
edleafebut I am still not clear on what is supposed to be authoritative when it comes to "acceptable versions"13:07
edleafehow do you see this middleware knowing that for a service?13:07
cdentthat version_list, which is a thing like: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/microversion.py#L3613:08
*** ioggstream has joined #openstack-sdks13:08
cdentthe DNM think I linked before passes VERSIONS into the middleware13:08
edleafeI saw that; I'm still not clear on the benefit of passing that from the service to the middleware13:10
edleafeWith it, every service using the middleware has to implement a versions list and pass it during setup13:10
edleafeWithout it, services that care about checking specific versions can do so locally13:11
cdentthe job of the middleware is twofold:13:12
cdentextract a version and put it in the environ to be used by the app13:12
cdentto return a 4xx is the microversion is malformed or out of range13:12
cdentin order for it to "out of range" is has to know the range13:13
cdentwe want it to do that job, because we want it to fail early13:13
cdentand 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
edleafeIs there a way to say "any version is ok" for those that do not do such housekeeping?13:17
cdentIf you don't do that housekeeping you can't do matches()13:18
cdentand you can't do version_handler13:18
cdentso you may as well not use microversion-parse13:18
cdentI guess I'm not understanding how/when you wouldn't want to do the housekeeping?13:18
edleafe*I* would want to do it13:19
*** dmellado has joined #openstack-sdks13:19
edleafeJust wondering if that is the standard approach13:19
edleafeThinking about projects that have not yet adopted microversions13:19
edleafeWould this be an attractive feature, or a turn-off13:20
*** dmellado has quit IRC13:20
edleafeMaybe it will encourage good microversion behavior :)13:20
cdentI think the versions_list provides a useful bit of cognitive scaffodling, without it, you aren't really thinking like a microversioner13:21
edleafeI believe the correct term is 'microversioneer'13:22
cdentnova does a similar thing, but instead of a list it uses a min and max, but I think that's limiting13:23
cdentyou can't easily disable a microversion in the middle13:23
elmikocdent: re: %s, i read a really cool blog that actually explained when it would be more efficient to use %s vs. format13:25
cdentstring 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 party13:27
elmikoi was kinda shocked to learn about the performance differences13:27
elmikobut i agree, would be cool if they could pick one and stick to it13:28
elmiko"string interpolation choice as a carbon dating of python usage" XD13:28
*** jpena|lunch is now known as jpena13:29
edleafeF-strings FTW!13:29
edleafeThey would also force people to upgrade to Ptyhon 3.613:29
edleafe(and they are faster!)13:30
elmiko++13:31
*** ralonsoh_ has joined #openstack-sdks13:32
*** dave-mccowan has quit IRC13:34
*** ralonsoh has quit IRC13:35
*** kmy has quit IRC13:44
*** kmy has joined #openstack-sdks13:45
openstackgerritHengqing Hu proposed openstack/python-openstacksdk master: Allow cascade deletion of load balancer  https://review.openstack.org/55340413:49
openstackgerritEmilien Macchi proposed openstack/python-openstackclient master: neutron: add --mtu for create/set network  https://review.openstack.org/55337413:51
*** ldl has joined #openstack-sdks13:53
openstackgerritHengqing Hu proposed openstack/python-openstacksdk master: Allow cascade deletion of load balancer  https://review.openstack.org/55340414:01
*** ralonsoh__ has joined #openstack-sdks14:04
*** ralonsoh__ is now known as ralonsoh14:04
*** ralonsoh_ has quit IRC14:07
*** bobh has joined #openstack-sdks14: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 identified14: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
cdentelmiko: line 45 https://review.openstack.org/#/c/550251/2/microversion_parse/tests/test_extract_version.py14:19
elmikocdent: ack, i /thought/ that was it but wanted to make sure. thanks!14:24
*** bobh has quit IRC14:27
cdentthanks 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 stuff14:44
cdentreleased bugs are the gasoline of opensource14:44
*** dmellado_ has joined #openstack-sdks14:46
edleafeso... you want to burn it all down? :)14:46
cdentonly some of it14:47
openstackgerritAndrey Volkov proposed openstack/cliff master: Allow finding command by partial name  https://review.openstack.org/55297014:49
elmikoedleafe: lol14:50
openstackgerritAndrey Volkov proposed openstack/cliff master: Allow finding command by partial name  https://review.openstack.org/55297014:51
*** chenyb4 has quit IRC14:52
*** dmellado_ is now known as dmellado14:54
openstackgerritDmitry Tantsur proposed openstack/api-wg master: Add guideline on exposing microversions in SDKs  https://review.openstack.org/53281414:57
dtantsurupdates with the PTG outcome ^^^14:57
cdentdtantsur++14:57
elmikonice14:57
*** bobh has joined #openstack-sdks14:57
*** bobh has quit IRC15:03
*** ldl has quit IRC15:03
*** bobh has joined #openstack-sdks15:07
*** bobh has quit IRC15:09
*** bobh has joined #openstack-sdks15:09
*** stevebaker has joined #openstack-sdks15:14
*** thrash is now known as thrash|biab15:24
*** kmy has quit IRC15:31
*** kmy has joined #openstack-sdks15:33
*** dave-mccowan has joined #openstack-sdks15:36
*** kmalloc has joined #openstack-sdks15:53
elmikocdent: ping16:00
cdentoops16:01
*** ralonsoh_ has joined #openstack-sdks16:01
*** harlowja has joined #openstack-sdks16:03
*** thrash|biab is now known as thrash16:04
*** ralonsoh has quit IRC16:04
*** gildub has joined #openstack-sdks16:17
*** e0ne has quit IRC16:27
*** Suramya_ has joined #openstack-sdks16:28
*** ldl has joined #openstack-sdks17:02
*** gkadam has quit IRC17:03
*** ralonsoh__ has joined #openstack-sdks17:06
*** ldl has quit IRC17:06
*** ralonsoh_ has quit IRC17:09
*** harlowja has quit IRC17:09
cdentdtantsur, edleafe, elmiko https://etherpad.openstack.org/p/api-sig-newsletter I started out with some energy and then fizzled out, please proof17:11
edleafewow, I opened up that link, and I could hear the fizzing!17:12
*** dtantsur is now known as dtantsur|afk17:15
*** ralonsoh__ has quit IRC17:16
cdentyou happy with it edleafe ?17:17
edleafeMade two micro-nit corrections, and now LGTM17:19
cdentnice17:20
cdentI'm often confused on project names, because back when docs wrote docs they preferred lowercase project names17:20
*** e0ne has joined #openstack-sdks17:23
*** gkadam has joined #openstack-sdks17:25
*** jpich has quit IRC17:49
*** Matias has quit IRC17:52
*** slaweq_ has quit IRC17:55
*** slaweq has joined #openstack-sdks17:55
*** gkadam has quit IRC18:00
elmikocdent: sorry got pulled away18:01
cdentno worries18:02
elmikoooh, i had not noticed that we are cross-posting those to openstack-sigs now as well. i will update my proceedures18:08
cdenti don't always remember18:09
cdentbut I think we said at some point that it was maybe a good idea18:09
elmikoyeah, it is a good idea. i had just forgotten we said that18:13
*** jpena is now known as jpena|off18:14
*** harlowja has joined #openstack-sdks18:20
openstackgerritChris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ  https://review.openstack.org/49535618:24
openstackgerritChris Dent proposed openstack/microversion-parse master: Import functions from placement  https://review.openstack.org/49621218:24
openstackgerritChris Dent proposed openstack/microversion-parse master: Improve Version.matches() method  https://review.openstack.org/55025118:25
openstackgerritChris Dent proposed openstack/microversion-parse master: Add MicroversionMiddleware  https://review.openstack.org/55025218:25
openstackgerritChris Dent proposed openstack/microversion-parse master: Allow passing a json_error_formatter to the middleware  https://review.openstack.org/55026318:25
openstackgerritChris Dent proposed openstack/microversion-parse master: Clean up package version and python version support  https://review.openstack.org/55026418:25
cdentelmiko, edleafe ^ I think I got the things you mentioned18:25
elmikocdent: ack18:32
*** ioggstream has quit IRC18:33
openstackgerritJames Penick proposed openstack/python-openstackclient master: Stop prefetching images in server list  https://review.openstack.org/55357718:39
edleafecdent: will look again soon-ish18:49
cdentthanks edleafe, elmiko18:50
*** e0ne has quit IRC18:55
edleafecdent: Looking over the HTTP Guidelines doc, it looks like it is logically 3 documents: HTTP caching, HTTP response code guidance, and HTTP method guidance19: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 now19:26
cdentyes, breaking things up is probably warranted19:26
edleafePerhaps breaking it into 3 separate docs would make them more "discoverable"19:27
edleafedue to the more specific titles19:27
edleafeIf you like, I can take a crack at it.19:27
*** frickler has quit IRC19:29
elmikoedleafe: that sounds like a nice idea19:30
cdentedleafe: feel free to go for it if you're feeling motivated19:30
cdentdo we want to leave redirects or something?19:30
*** frickler has joined #openstack-sdks19:30
*** Suramya_ has quit IRC19:32
edleafenot sure - does sound like a good idea just in case someone has bookmarked the current page19:32
cdentit would be very bad form for the API-SIG to break the web19:33
* edleafe tries to remember who it was who wanted to break things more often...19:35
elmikocdent: ++19:43
elmikolol19:43
cdentedleafe: bah, broken client code is broken client code19:45
cdentand in any case when I say "the web" i mean public document urls. I went to library school after all.19:45
elmikocan we just have the page return 4xx, "we're not the problem, you are!"19:45
edleafeelmiko: "Figure it out yourself!"19:46
elmikoso helpful XD19:47
edleafe"We're not your babysitter!"19:47
elmikoLOL19:47
elmikobest.error.message.ever.19:47
*** dave-mccowan has quit IRC19:53
*** dave-mccowan has joined #openstack-sdks20:11
*** cdent has quit IRC20:30
openstackgerritBrian Haley proposed openstack/python-openstackclient master: Fix typo in 'floating ip associate' command and doc  https://review.openstack.org/55360320:31
*** dave-mccowan has quit IRC20:53
*** edmondsw has quit IRC21:02
EmilienMosc-functional-devstack failed on my patch, is it accurate to do recheck? I've seen some messages about failures recently21:12
EmilienMfailure is in post21:13
EmilienM(job passed all tests but fail after, I think it's infra related)21:13
*** dave-mccowan has joined #openstack-sdks21:39
*** bobh has quit IRC21:53
*** bobh has joined #openstack-sdks21:56
*** bobh has quit IRC21:59
*** dave-mccowan has quit IRC22:48
*** salv-orl_ has quit IRC22:53
*** salv-orlando has joined #openstack-sdks22:54
*** salv-orlando has quit IRC22:59
openstackgerritMerged openstack/osc-lib master: Allow to use the none auth plugin  https://review.openstack.org/51269923:34
*** salv-orlando has joined #openstack-sdks23:54
*** salv-orlando has quit IRC23:59

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