Thursday, 2018-02-15

« Wednesday, 2018-02-14 Index Friday, 2018-02-16 »
*** sdague has quit IRC00:09
*** bobh has joined #openstack-sdks00:28
*** dave-mccowan has joined #openstack-sdks00:54
*** lbragstad has quit IRC01:45
*** fzdarsky_ has joined #openstack-sdks02:12
*** slaweq has joined #openstack-sdks02:13
*** fzdarsky has quit IRC02:16
*** slaweq has quit IRC02:18
*** slaweq has joined #openstack-sdks02:36
*** slaweq has quit IRC02:39
*** dasp has joined #openstack-sdks02:40
*** gkadam_ has quit IRC02:43
*** salv-orlando has joined #openstack-sdks02:44
*** dasp has quit IRC02:44
*** dasp has joined #openstack-sdks02:44
*** dasp has quit IRC02:46
*** dasp has joined #openstack-sdks02:46
*** salv-orl_ has quit IRC02:47
*** bobh has quit IRC02:59
*** bobh has joined #openstack-sdks03:02
*** harlowja has quit IRC03:08
*** gildub_ has quit IRC03:49
*** bobh has quit IRC04:00
*** gildub_ has joined #openstack-sdks04:10
*** gildub_ has quit IRC04:16
*** thrash is now known as thrash|g0ne04:16
*** dave-mccowan has quit IRC04:25
*** gkadam_ has joined #openstack-sdks04:26
*** gkadam_ has quit IRC04:36
*** gkadam has joined #openstack-sdks04:38
*** harlowja has joined #openstack-sdks05:15
*** gildub has joined #openstack-sdks05:46
*** harlowja has quit IRC06:14
*** slaweq has joined #openstack-sdks06:15
*** slaweq has quit IRC06:19
*** e0ne has joined #openstack-sdks06:20
*** e0ne has quit IRC06:30
*** e0ne has joined #openstack-sdks06:44
*** e0ne_ has joined #openstack-sdks06:51
*** e0ne_ has quit IRC06:55
*** e0ne has quit IRC06:55
*** e0ne has joined #openstack-sdks06:56
*** edmondsw has joined #openstack-sdks07:41
*** e0ne has quit IRC07:43
*** e0ne has joined #openstack-sdks07:43
*** slaweq has joined #openstack-sdks07:45
*** edmondsw has quit IRC07:45
*** e0ne has quit IRC07:49
*** halbbalda has joined #openstack-sdks07:50
*** slaweq_ has joined #openstack-sdks08:15
*** slaweq_ has quit IRC08:20
*** jpena|off is now known as jpena08:26
*** gkadam_ has joined #openstack-sdks08:37
*** gkadam is now known as Guest820708:37
*** gkadam_ is now known as gkadam08:37
*** jpich has joined #openstack-sdks08:59
*** gildub has quit IRC09:17
*** dtantsur|afk is now known as dtantsur09:21
*** ralonsoh has joined #openstack-sdks09:23
*** edmondsw has joined #openstack-sdks09:29
*** edmondsw has quit IRC09:34
openstackgerritMerged openstack/osc-lib master: Zuul: Remove project name  https://review.openstack.org/54379009:34
*** e0ne has joined #openstack-sdks09:37
*** cdent has joined #openstack-sdks09:45
*** slaweq_ has joined #openstack-sdks10:16
*** slaweq_ has quit IRC10:21
*** d0ugal has quit IRC10:30
*** d0ugal has joined #openstack-sdks10:33
*** gildub has joined #openstack-sdks10:37
*** sdague has joined #openstack-sdks10:38
*** kmy has quit IRC10:47
*** kmy has joined #openstack-sdks10:48
*** e0ne has quit IRC10:51
*** ldl has joined #openstack-sdks10:57
*** e0ne has joined #openstack-sdks10:59
*** d0ugal has quit IRC11:20
*** e0ne has quit IRC11:24
*** e0ne has joined #openstack-sdks11:31
*** ioggstream has joined #openstack-sdks11:32
*** d0ugal has joined #openstack-sdks11:40
*** gildub has quit IRC11:43
*** jpena is now known as jpena|lunch11:52
*** dave-mccowan has joined #openstack-sdks12:10
cdentelmiko, edleafe, dtantsur: depending on the flow of several different events this afternoon I may miss or be pulled away from the api-sig meeting to deal with local stuff12:14
dtantsurack. I plan on being there.12:14
*** slaweq_ has joined #openstack-sdks12:17
*** ldl has quit IRC12:21
*** slaweq_ has quit IRC12:22
*** ioggstream has quit IRC12:26
*** e0ne has quit IRC12:29
*** gkadam has quit IRC12:48
*** bobh has joined #openstack-sdks12:50
*** thrash|g0ne is now known as thrash13:03
*** bobh has quit IRC13:04
*** edmondsw has joined #openstack-sdks13:06
*** edmondsw has quit IRC13:10
*** pooja-jadhav is now known as pooja_jadhav13:10
*** edmondsw has joined #openstack-sdks13:24
cdentmordred: is this dead, lingering, superceded, something else? https://review.openstack.org/#/c/484341/13:25
*** jpena|lunch is now known as jpena13:37
elmikocdent: ack13:48
*** e0ne has joined #openstack-sdks13:54
edleafecdent: ack14:00
*** dtantsur is now known as dtantsur|brb14:02
*** dave-mccowan has quit IRC14:23
*** lbragstad has joined #openstack-sdks14:29
*** dtantsur|brb is now known as dtantsur14:49
*** mrhillsman has joined #openstack-sdks14:50
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Fix functional test about port  https://review.openstack.org/54499514:50
mordredcdent: lingering and still much needed - I need to get that finished up soon14:57
cdentmordred: roger that14:57
mordredcdent: if you have a sec, https://review.openstack.org/#/c/543987 and its parent https://review.openstack.org/#/c/544365 are ready for some love14:58
* cdent looks14:58
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions  https://review.openstack.org/53939615:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy  https://review.openstack.org/53927115:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Generate proxy methods from resource objects  https://review.openstack.org/54016915:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Calculate name in CloudRegion  https://review.openstack.org/54105915:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Use get_session_client in Connection  https://review.openstack.org/54106015:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: WIP Honor service type aliases from the catalog  https://review.openstack.org/54435615:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Collect tox and testr output in functional tests  https://review.openstack.org/54500115:02
openstackgerritMerged openstack/os-service-types master: Sort json data  https://review.openstack.org/54436515:08
openstackgerritMerged openstack/os-service-types master: Updated from OpenStack Service Type Authority  https://review.openstack.org/54398715:12
*** bobh has joined #openstack-sdks15:13
*** dave-mccowan has joined #openstack-sdks15:14
fricklermordred: in case you are getting bored, I re-discovered another shade issue affecting ansible ;) https://storyboard.openstack.org/#!/story/200152715:28
fricklermordred: also doing a new shade release would be nice15:28
*** ralonsoh_ has joined #openstack-sdks15:32
*** ralonsoh has quit IRC15:36
mordredfrickler: yay! SOOOOO ... that one will be fun to fix, since wait=False means we return before there is a server to attach a FIP to.15:37
mordredfrickler: we could spawn off a worker thread to do the fip attaching, but with os_server being its own process it would have the same effect as auto_ip just implying wait=True15:38
fricklermordred: I was fearing something like that, a valid alternative might be changing the default to some working variant. and also adding a big warning to os_server in ansible15:38
mordredfrickler: the *safest* way would be to have auto_ip imply wait=True ... should ponder that for a minute ...15:40
fricklermordred: another approach might be getting a port from neutron, attaching the FIP and then creating the server with that port. I never tested that, though15:40
mordredfrickler: it won't work on all clouds, unfortunately15:41
mordred(although I like the idea)15:41
mordredfrickler: hrm. now that I say that ... the clouds that I know it won't work on are clouds that don't need floating ips anyway15:42
mordredfrickler: so it might be worth testing that as an approach on the fip-requiring clouds15:42
mordredalthough then we run in to fun with clouds that have multiple private interfaces15:43
*** dave-mccowan has quit IRC15:43
cdentelmiko, dtantsur, edleafe not going to make it all15:44
*** gkadam has joined #openstack-sdks15:47
elmikocdent: thanks for the heads up, you will be missed /me sniffles15:48
elmikoalso, you posts about the placement thingie in nova make me really curious15:48
cdentelmiko: feel free to find me and ask me about whatever it is you are being curious about. When I'm not running errands (which will start soon)15:51
elmikocdent: i plan to hit you up in dublin, hopefully over brews ;)15:54
cdentcool, you can meet my lovely wife too, she'll be there15:54
edleafeoh, I'm *sure* she'll love to be chatting about placement15:57
elmikovery cool!15:58
elmikohaha15:58
cdentI was trying to make excuses for two different instances of brews15:58
*** ralonsoh_ has quit IRC16:05
*** ralonsoh_ has joined #openstack-sdks16:06
*** slaweq_ has joined #openstack-sdks16:19
*** halbbalda has quit IRC16:21
*** slaweq has quit IRC16:21
*** slaweq has joined #openstack-sdks16:22
*** bobh has quit IRC16:24
*** bobh has joined #openstack-sdks16:24
*** slaweq_ has quit IRC16:24
mordredrods_, slaweq: stack is green at https://review.openstack.org/#/c/539271 finally - slaweq, I squashed the doc patch into the BaseProxy patch based on your review16:24
rods_mordred looking16:25
*** slaweq has quit IRC16:26
elmikoedleafe, dtantsur, ready for review https://etherpad.openstack.org/p/api-sig-newsletter16:32
*** e0ne has quit IRC16:37
SamYaple/win/win 4916:39
mordredSamYaple: we see you16:39
mordredSamYaple: (wanna review 539271 ?)16:39
SamYaplealready done16:40
SamYaplebut also, damn16:40
SamYapleim about to switch to openstacksdk from shade internally. the testing is pretty painless so far16:41
mordredSamYaple: woot!16:48
edleafeelmiko: if you like my changes, ship it!16:49
*** slaweq has joined #openstack-sdks16:52
elmikoedleafe: love it! thanks =)16:53
elmikoi wasn't sure about the -1 part, but i think you are spot on16:53
*** slaweq has quit IRC16:57
edleafeelmiko: I'm not one of those "only say nice things" people16:58
elmikoedleafe: yeah, i need to become better about that17:01
*** Matias_ has joined #openstack-sdks17:06
*** Matias_ is now known as Matias17:07
*** harlowja has joined #openstack-sdks17:19
*** thrash is now known as thrash|biab17:20
openstackgerritMerged openstack/python-openstacksdk master: Fix functional test about port  https://review.openstack.org/54499517:28
openstackgerritMerged openstack/python-openstacksdk master: Collect tox and testr output in functional tests  https://review.openstack.org/54500117:29
*** cdent has quit IRC17:30
openstackgerritMerged openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions  https://review.openstack.org/53939617:31
*** jpich has quit IRC17:32
*** thrash|biab is now known as thrash17:43
openstackgerritMerged openstack/python-openstacksdk master: Rename BaseProxy to Proxy  https://review.openstack.org/53927117:48
openstackgerritMerged openstack/python-openstacksdk master: Generate proxy methods from resource objects  https://review.openstack.org/54016917:52
*** thrash is now known as thrash|vet17:55
*** gkadam has quit IRC17:56
*** dtantsur is now known as dtantsur|afk18:04
*** jpena is now known as jpena|off18:14
*** ralonsoh_ has quit IRC18:17
*** slaweq has joined #openstack-sdks18:20
*** slaweq has quit IRC18:25
*** bobh has quit IRC18:31
*** bobh has joined #openstack-sdks18:41
*** harlowja has quit IRC19:00
*** thrash|vet is now known as thrash19:00
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Update base test case to use base from oslotest  https://review.openstack.org/54037219:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Update all test base classes to use base.TestCase  https://review.openstack.org/54037319:02
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Honor service type aliases from the catalog  https://review.openstack.org/54435619:02
mordredQiming: ^^ ok. I think that should be working now. I apologize, it's a nasty patch to review - which is partially because the logic here SHOULD be in keystoneauth but I didn't make the cutoff for that and I'd like to backport this fix to queens19:04
openstackgerritColleen Murphy proposed openstack/keystoneauth master: Add pep8 import order validation  https://review.openstack.org/54509419:09
*** slaweq has joined #openstack-sdks19:33
*** harlowja has joined #openstack-sdks19:35
*** harlowja_ has joined #openstack-sdks19:37
*** harlowja has quit IRC19:39
rm_workheya19:46
rm_workso, newest(?) release of openstacksdk seems to force some version discovery of endpoints that breaks on older (liberty) clouds :(19:47
rm_workseems all of the 0.11.x versions19:49
rm_worklatest working that i have so far is a 0.9.x but i'm trying to figure out what i can test in 0.10.x19:50
rm_workwas 0.10.x basically skipped?19:50
rm_work0.9.19 seems to be the latest that works for me19:51
openstackgerritMerged openstack/keystoneauth master: Fix a spelling error  https://review.openstack.org/54194919:53
rm_workmordred: it was mentioned that you possibly were working on bits of this?19:55
*** bobh has quit IRC19:58
mordredrm_work: I am, in fact, working in a related area right now ... which service are you seeing issues with?20:10
rm_workneutron specifically20:10
rm_worki didn't test everything20:10
rm_work`port list` is the worst20:10
rm_workit just fails entirely20:10
rm_work`floating ip list` prints some errors about version discovery but somehow still seems to return results20:11
mordredrm_work: are you in a position where it would be easy for you to test a not-landed version of sdk?20:11
rm_workprolly sure20:11
rm_workI can just git-review and pip install -e20:12
mordredrm_work: cool - if you git review -d 544356 ... I was just testing that octavia worked (it also doesn't have a version discovery doc)20:12
mordredrm_work: and octavia is now working ... so if that state works, awesome - if not, we'll have to dig further20:13
rm_workwell20:13
rm_worki run octavia from master20:13
rm_workon my liberty cloud :/20:13
rm_workso it won't test the same issue, lol20:14
mordredit will not - but the port list test should be good20:14
mordredrm_work: I've been testing octavia listing load balancer against the released version running in vexxhost public cloud20:14
rm_workon your branch, port list fails20:14
rm_workoctavia lb list works tho20:15
mordredrm_work: awesome! at least it's reproducible :)20:15
rm_workbut again, i'm on like20:15
rm_workthe absolute newest possible20:15
mordredrm_work: so on liberty does neutron just not have a version discovery doc?20:15
rm_worki think that's correct20:15
rm_workwhich somehow was fine still in 0.9.1920:15
rm_workbut everything after that it explodes20:15
mordredyah. major changes between 0.9.19 and 0.1120:16
rm_work:(20:16
mordredlemme make a local test case and figure out why that's exploding20:16
mordredrm_work: if you add 'network' to the tuple on line 327 in openstack/config/cloud_region.py - does that fix port list?20:17
rm_workit does not20:20
mordredrm_work: oh - I have reproduced the issue locally20:20
rm_work;)20:20
*** e0ne has joined #openstack-sdks20:22
mordredrm_work: one more thing - can you try adding "network_api_version": "2", to openstack/config/defaults.json ?20:24
mordredrm_work: I believe that will still not work foryou on liberty - but I'd like to double-check20:24
rm_workin addition to the other change?20:24
mordredrm_work: other change not needed20:25
rm_workk, yeah, still no worky20:26
mordredkk. thanks. I grok the problem and will now commence to cry for a litle while then make a fix for you20:26
*** bobh has joined #openstack-sdks20:28
rm_workthanks :)20:32
*** bobh has quit IRC20:33
*** e0ne has quit IRC20:38
*** e0ne has joined #openstack-sdks20:38
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Handle liberty-era neutron  https://review.openstack.org/54511720:59
*** bobh has joined #openstack-sdks21:05
mordredrm_work: ^^ if you have a sec, can you try that ^^ ?21:05
openstackgerritBrian Haley proposed openstack/python-openstackclient master: Add --dns-domain to 'network create' and 'network set'  https://review.openstack.org/54512421:07
*** bobh has quit IRC21:10
openstackgerritMerged openstack/keystoneauth master: Split request logging into four different loggers  https://review.openstack.org/50576421:14
openstackgerritMerged openstack/keystoneauth master: Add some comments explaining split_loggers flag logic  https://review.openstack.org/54106621:14
rm_workmordred: seems to work! :)21:16
rm_work\o/21:16
mordredrm_work: woot!21:16
*** bobh has joined #openstack-sdks21:18
openstackgerritMerged openstack/keystoneauth master: Remove PYTHONHASHSEED setting  https://review.openstack.org/53379821:23
*** bobh has quit IRC21:24
*** bobh has joined #openstack-sdks21:27
daspmordred: I wanted to follow up on https://review.openstack.org/#/c/543035/. I get your point but I'm not sure that it would be sufficient - some proxies expose APIs at a given path (e.g. vip:443/glance/v2/images) and then when the API service (glance in this case) is cooperating (some allow you to define base path), the links to pages would not start with /vX. Ideally, I think keystoneauth1  is better place to handle that instead of working21:31
dasparound in openstacksdk. I can propose a solution as you suggested but I wasn't sure how deep we want to go there.21:31
*** e0ne has quit IRC21:31
*** bobh has quit IRC21:32
*** bobh has joined #openstack-sdks21:34
*** openstack has joined #openstack-sdks21:47
*** ChanServ sets mode: +o openstack21:47
mordreddasp: so... hrm. this is a tricky interaction between how relative paths work for http and how 'mounted' adapters deal with them clashing :(21:48
mordreddasp: there isn't really anything we can do at the keystoneauth layer, as the Adapter is mounted at the versioned endpoint. if the service is returning an absolute path but not a full url in the next links, then it's running in to requests Adapter treating it as rooted on the adapter mount point rather than rooted on the base of the http target21:50
mordreddasp: so we may need to do our own joining at the point where your patch is ...21:51
mordreddasp: basically - if the uri returned from the pagination startswith '/' - then we need to do a urljoin on it with the service's url ... in fact, we might be able to do that unconditionally21:52
mordreddasp: SO - actually, I think your patch is almost right there - it's just that it needs to use urljoin instead of +/split - and it needs to use session.get_endpoint() instead of cls.base_path21:54
daspmordred: that I can do soon. Still it's not perfect, because it's common to expose glance at https://vip:443/images and the to list images, it's https://vip:443/images/v2/images and when we split it, we will erronously  catch the first "/images". But I don't think there's any way around that here :/22:03
mordreddasp: I don't think we need to split it actually ...22:04
mordreddasp: I *think* we can just take the url we get in uri and do urlparse.urljoin(session.get_endpoint(), uri)22:05
daspmordred: I'll test a few things and let you know, thanks for input22:09
mordreddasp: http://paste.openstack.org/show/673168/22:10
mordreddasp: thanks for finding/fixing the issue!22:10
mordreddasp: there's a place in keystoneauth (an also I think in shade somewhere) where we do extra logic to join version discovery links onto the url in the catalog because somethings (glance being a notable one) return broken links (like the internal ip address)22:11
mordreddasp: it's possible we should extract that into a special join method in keystoneauth that we can use here - but also that keystoneauth can use when processing the version discovery links22:12
openstackgerritMerged openstack/keystoneauth master: Add pep8 import order validation  https://review.openstack.org/54509422:29
*** slaweq has joined #openstack-sdks22:42
*** slaweq has quit IRC22:44
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Update base test case to use base from oslotest  https://review.openstack.org/54037222:45
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Update all test base classes to use base.TestCase  https://review.openstack.org/54037322:45
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Honor service type aliases from the catalog  https://review.openstack.org/54435622:45
openstackgerritMonty Taylor proposed openstack/python-openstacksdk master: Handle liberty-era neutron  https://review.openstack.org/54511722:45
*** bobh has quit IRC23:19
*** bobh has joined #openstack-sdks23:20
*** bobh has quit IRC23:25
*** bobh has joined #openstack-sdks23:26
*** bobh has quit IRC23:39
*** sdague has quit IRC23:41
*** cdent has quit IRC23:45
« Wednesday, 2018-02-14 Index Friday, 2018-02-16 »

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