Friday, 2018-05-04

« Thursday, 2018-05-03 Index Saturday, 2018-05-05 »
*** stevebaker has joined #openstack-sdks00:01
*** markvoelker has quit IRC00:05
*** tosky has quit IRC00:05
*** thrash is now known as thrash|g0ne00:25
*** dmellado has quit IRC00:31
*** stevebaker has quit IRC00:33
*** d0ugal has quit IRC00:36
*** dmellado has joined #openstack-sdks00:38
*** stevebaker has joined #openstack-sdks00:39
*** edmondsw has joined #openstack-sdks00:53
*** stevebaker has quit IRC00:59
*** dmellado has quit IRC00:59
*** d0ugal has joined #openstack-sdks01:03
*** edmondsw has quit IRC01:05
*** markvoelker has joined #openstack-sdks01:05
*** chenyb4 has joined #openstack-sdks01:06
*** annp has joined #openstack-sdks01:16
*** edmondsw has joined #openstack-sdks01:18
*** stevebaker has joined #openstack-sdks01:24
*** dmellado has joined #openstack-sdks01:26
*** dmellado has quit IRC01:32
*** stevebaker has quit IRC01:33
*** markvoelker has quit IRC01:40
*** dmellado has joined #openstack-sdks01:56
*** stevebaker has joined #openstack-sdks01:59
*** dayou_ has joined #openstack-sdks02:02
*** gildub has joined #openstack-sdks02:03
*** dmellado has quit IRC02:09
*** stevebaker has quit IRC02:09
*** dmellado has joined #openstack-sdks02:13
*** stevebaker has joined #openstack-sdks02:13
*** edmondsw has quit IRC02:18
*** markvoelker has joined #openstack-sdks02:37
*** stevebaker has quit IRC02:37
*** dmellado has quit IRC02:38
pooja_jadhavmordred: Hi03:09
*** markvoelker has quit IRC03:10
*** dave-mccowan has quit IRC03:14
*** stevebaker has joined #openstack-sdks03:37
*** dmellado has joined #openstack-sdks03:39
*** edmondsw has joined #openstack-sdks04:00
*** stevebaker has quit IRC04:03
*** dmellado has quit IRC04:04
*** edmondsw has quit IRC04:05
*** markvoelker has joined #openstack-sdks04:08
*** slaweq has joined #openstack-sdks04:11
*** slaweq has quit IRC04:15
*** dmellado has joined #openstack-sdks04:39
*** markvoelker has quit IRC04:41
*** stevebaker has joined #openstack-sdks04:43
*** ianw has quit IRC05:06
*** NobodyCam has quit IRC05:15
*** TheJulia has quit IRC05:15
*** vdrok has quit IRC05:15
*** charz has quit IRC05:15
*** charz has joined #openstack-sdks05:15
*** TheJulia has joined #openstack-sdks05:15
*** vdrok has joined #openstack-sdks05:16
*** NobodyCam has joined #openstack-sdks05:16
*** markvoelker has joined #openstack-sdks05:38
*** slaweq has joined #openstack-sdks05:47
*** slaweq has quit IRC05:52
*** ianw has joined #openstack-sdks05:55
*** markvoelker has quit IRC06:11
*** slaweq has joined #openstack-sdks06:11
*** slaweq has quit IRC06:16
*** dayou_ has quit IRC06:20
*** e0ne has joined #openstack-sdks06:21
*** yolanda_ has joined #openstack-sdks06:42
*** yolanda has quit IRC06:45
*** e0ne has quit IRC06:59
*** stevebaker has quit IRC07:01
*** dmellado has quit IRC07:01
*** dayou_ has joined #openstack-sdks07:04
*** slaweq has joined #openstack-sdks07:05
*** slaweq has quit IRC07:06
*** slaweq has joined #openstack-sdks07:06
*** markvoelker has joined #openstack-sdks07:08
*** dmellado has joined #openstack-sdks07:12
*** gildub has quit IRC07:22
*** dmellado_ has joined #openstack-sdks07:24
*** dmellado_ has quit IRC07:34
*** edmondsw has joined #openstack-sdks07:36
*** gkadam has joined #openstack-sdks07:41
*** edmondsw has quit IRC07:41
*** markvoelker has quit IRC07:42
*** tosky has joined #openstack-sdks07:48
*** jpena|off is now known as jpena07:48
*** jpich has joined #openstack-sdks07:51
*** gkadam has quit IRC07:52
*** gkadam has joined #openstack-sdks07:52
*** gtema has joined #openstack-sdks07:59
*** dmellado_ has joined #openstack-sdks08:01
*** ttsiouts has quit IRC08:04
*** ttsiouts has joined #openstack-sdks08:04
*** dmellado_ has quit IRC08:11
*** salv-orlando has joined #openstack-sdks08:15
*** salv-orlando has quit IRC08:15
*** salv-orlando has joined #openstack-sdks08:16
*** gildub has joined #openstack-sdks08:17
*** stephenfin is now known as finucannot08:29
*** markvoelker has joined #openstack-sdks08:39
*** e0ne has joined #openstack-sdks08:40
*** gkadam has quit IRC08:42
*** lbragstad has quit IRC08:45
*** lbragstad has joined #openstack-sdks08:45
*** dmellado_ has joined #openstack-sdks08:50
*** stevebaker has joined #openstack-sdks08:53
*** salv-orlando has quit IRC08:53
*** markvoelker has quit IRC09:12
*** dtantsur|afk is now known as dtantsur09:33
*** salv-orlando has joined #openstack-sdks09:57
fricklerseems there is a regression in os-client-config==1.31.0 breaking shade amongst other things https://bugs.launchpad.net/os-client-config/+bug/176881310:03
openstackLaunchpad bug 1768813 in os-client-config "config.get_cache_expiration_time() function missing in 1.31.0 release" [Undecided,New]10:03
fricklermordred: ^^10:03
*** chenyb4 has quit IRC10:06
*** markvoelker has joined #openstack-sdks10:08
*** salv-orlando has quit IRC10:31
*** markvoelker has quit IRC10:42
*** salv-orlando has joined #openstack-sdks11:07
*** dave-mccowan has joined #openstack-sdks11:13
*** salv-orlando has quit IRC11:31
*** markvoelker has joined #openstack-sdks11:39
*** edmondsw has joined #openstack-sdks11:53
*** annp has quit IRC11:57
*** jpena is now known as jpena|lunch12:03
*** gkadam has joined #openstack-sdks12:09
*** markvoelker has quit IRC12:12
*** thrash|g0ne is now known as thrash12:14
*** dtantsur is now known as dtantsur|brb12:17
*** gkadam has quit IRC12:24
*** markvoelker has joined #openstack-sdks12:24
*** jpena|lunch is now known as jpena12:54
*** gildub has quit IRC12:56
*** edleafe is now known as figleaf12:57
openstackgerritDavid Shrewsbury proposed openstack/os-client-config master: Add these calls back to OCC  https://review.openstack.org/56630813:03
openstackgerritDavid Shrewsbury proposed openstack/os-client-config master: Add cache API methods back to OCC  https://review.openstack.org/56630813:04
Shrewsmordred: 566308 should add the cache API calls back, but it seems our tests are now testing openstacksdk and not os-client-config APIs13:04
Shrewsmordred: since we make those calls in occ tests and they apparently work13:05
Shrewsnot sure how to correct that13:05
Shrewsfrickler: ^^13:05
*** dtantsur|brb is now known as dtantsur13:09
*** bobh has joined #openstack-sdks13:14
pooja_jadhavcmurphy : Hi13:27
*** e0ne has quit IRC13:39
openstackgerritArtem Goncharov proposed openstack/os-client-config master: fix link to openstacksdk in deprecation block  https://review.openstack.org/56631813:43
*** e0ne has joined #openstack-sdks13:43
mordredShrews: looking13:50
mordredShrews: also the sdk functional tests keep hitting flapping errors related to floating ips - so I'm trying to clean that up13:51
Shrewsyay13:51
openstackgerritDavid Shrewsbury proposed openstack/os-client-config master: Add cache API methods back to OCC  https://review.openstack.org/56630813:52
fricklerShrews: hmm, I think you are patching the wrong location, the issue for me is with config.OpenStackConfig() and not cloud_config.Config()13:52
mordredAH13:53
Shrewspooja_jadhav: ah, Closes-Bug. couldn't remember it and couldn't find it any docs (it's all shuffled since last i looked)13:53
mordredthat makes more sense to me13:53
fricklerShrews: also yes, would be good to have tests for that13:53
* mordred was about to say that CloudRegion has get_cache_expiration_time13:53
pooja_jadhavShrews: Hi, I am not getting what you are trying to say13:55
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Clean up floating ip tests  https://review.openstack.org/56632013:55
Shrewspooja_jadhav: ooh, sorry. that was meant for someone else13:55
pooja_jadhavshrews: ohk13:56
openstackgerritMerged openstack/os-client-config master: fix link to openstacksdk in deprecation block  https://review.openstack.org/56631813:57
Shrewsok, i'm confused since those calls originally existed in CloudConfig13:58
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: close files after open in unit/base  https://review.openstack.org/56632113:58
fricklerShrews: this is the code sample I posted on the bug report which is broken with the new release https://github.com/ansible/ansible/blob/devel/contrib/inventory/openstack.py#L193-L20413:59
openstackgerritMonty Taylor proposed openstack/os-client-config master: Allow monkeypatching CONFIG_FILES  https://review.openstack.org/56613914:00
openstackgerritMonty Taylor proposed openstack/os-client-config master: Add cache methods back to OpenSackConfig  https://review.openstack.org/56632214:00
mordredShrews: there were two sets of them14:00
Shrewsmordred: oh. well that's not confusing14:00
mordredyah - it was terrible14:00
mordredthe CloudConfig ones originally called the OpenStackConfig ones14:00
mordredBUT - if you're just working with CloudConfig objects that's awkward - so CloudConfig got fixed to not need the OpenStackConfig methods anymore - and in sdk OpenStackConfig doens't have the methods because they're silly14:01
mordredand I totally missed keeping them for backwards compat in the occ patch14:01
Shrewsi'm not clear on the fix then14:02
mordredadding them back to OpenStackConfig so that occ users who were using them don't break14:02
Shrewsmordred: how do you get from occ.OpenStackConfig to the osdk.CloudRegion where those calls are defined?14:10
Shrewsthose calls seem like they should be region independent14:10
Shrewsor is it allowed to have different cache characteristics per region?14:11
*** bobh has quit IRC14:11
mordredno, you're right- it's region independent14:12
mordredthing is - the CloudRegion is the primary thing people work with14:12
*** bobh has joined #openstack-sdks14:12
mordredso - basically, the original change was to have the cache settings be actual constructor arguments to CloudRegion rather than having to pass an OpenStackConfig to CloudRegion14:14
mordredget_one_cloud then started passing the values to CloudRegion when it created one14:14
*** dmsimard has joined #openstack-sdks14:14
dmsimardmordred: how would you do a search_ports for a fixed_ip ?14:14
mordreddmsimard: conn.search_ports(filters={'fixed_ip': blah}) I believe14:15
dmsimardhttp://paste.openstack.org/show/720377/ doesn't seem to work14:15
dmsimardmordred: it's nested under fixed_ips14:15
*** melwitt is now known as jgwentworth14:15
dmsimardmordred: i.e, http://paste.openstack.org/show/720378/14:15
mordredof course it is14:16
mordreddmsimard: let's step back one - what are you trying to accomplish?14:16
dmsimardmordred: are you sitting on something ?14:17
dmsimardmordred: I need to add all the IPs from a subnet to another subnet's ports allowed-address-pairs14:17
dmsimardbasically make the ports of a subnet able to bind the ips from another subnet14:17
mordredwow14:17
mordredthat's awesome14:17
dmsimardIt's not hard, just need to figure that one thing14:17
*** salv-orlando has joined #openstack-sdks14:18
*** salv-orlando has quit IRC14:18
mordreddmsimard: so - you have two choices ...14:18
dmsimardhttps://github.com/openstack-infra/shade/blob/16e7290ff45ca6fa5d7ba7360575c38e0b0fd043/shade/openstackcloud.py#L6237-L6247 :/14:19
Shrewsshade only supports filtering dictionaries of dictionaries. lists of dictionaries are not supported14:19
*** chenyb4 has joined #openstack-sdks14:19
Shrewsyeah, that14:19
mordreddmsimard: you can pass a jmsepath string to filters - or you can do a list ports and filter yourself14:19
mordredlisting ports and filtering yourself is likely the easier thing14:20
Shrewsi forgot we added jmespath support14:20
mordredbut if you wanna play with the jmsepath supporT: http://jmespath.org/14:20
dmsimardmordred: I can filter myself I suppose14:20
mordredShrews: how about this ...14:22
Shrewsmordred: if the fix is clear in your head, feel free to add a PS to my review14:22
Shrewsi apparently need more coffee to see it14:22
mordredShrews: oh - I already pushed up https://review.openstack.org/566322 (should have done it as a PS to yours)14:23
mordredShrews: that said - I'm thinking maybe we just revert and release that - then add some more tests/test jobs and try the ectomy again14:24
mordredShrews: since we're clearly missing some coverage on ansible impact of occ changes14:24
dmsimardmordred: the thing is I need to create a port if there's no port for that ip address. Do you have an idea other than iterating through the ports and fixed_ips and creating them if they don't exist ?14:24
Shrewsmordred: possibly a good idea14:24
mordreddmsimard: nope. I recommend brute force14:25
dmsimardmordred: I wonder if I could blindly fire create_ports and basically pass on exception14:25
dmsimardbut that'd be kind of dirty14:25
Shrewsmordred: i'll abandon mine assuming the revert14:25
dmsimardmordred: hmmm.. how does http://docs.ansible.com/ansible/latest/modules/os_port_module.html handle idempotency ?14:26
* dmsimard looks14:26
openstackgerritMonty Taylor proposed openstack/os-client-config master: Revert "Replace guts with openstack.config"  https://review.openstack.org/56632814:27
mordredShrews: ^^14:27
Shrewsmordred: is that going to again break nodepool now that https://review.openstack.org/566138 *just* merged?14:27
dmsimardmordred: looks like os_port basically relies on the name of the port.. which is not stupid14:29
mordredShrews: ugh. probably. we should get the followup patch fixed14:29
* dmsimard proceeds to give ip address names to his ports14:29
pooja_jadhavmordred: Hi14:29
mordredShrews: maybe we should try the fix patch instead of the revert14:31
Shrewsmordred: your patch can't work i think. there is no self.get_cache_expiration_time()14:32
*** e0ne has quit IRC14:32
Shrewsmordred: that's only in CloudRegion, which is why i asked how to get to that14:33
Shrewsas are the _cache_* attrs14:34
Shrewsoh, no14:35
Shrewswrong about the attrs14:35
mordredthe patch adds get_cache_expiration_time14:35
Shrewsoh duh14:35
Shrews*sigh14:36
Shrewsmordred: ok, let's just try your patch first14:36
mordredShrews: yah - this *is* accessing private members of the osdk base class ... so we'll need to add some tests to occ (which we were obviously missing anyway) to make sure those methods continue to work14:37
mordredwe run occ tests on sdk patches, so that should prevent sdk patches from changing those private members14:37
mordredand yes, that's terrible software engineering practice in general14:37
openstackgerritMonty Taylor proposed openstack/os-client-config master: Revert "Replace guts with openstack.config"  https://review.openstack.org/56632814:40
mordredShrews: well, there's that (without merge conflict this time) in case we decide we want to go that route14:41
*** chenyb4 has quit IRC14:44
Shrewsmordred: i think the nodepool followup to use osdk is failing because osdk defines a get_cache_expirations() but shade is calling get_cache_expiration()14:46
Shrewsmordred: accidental pluralization there?14:46
*** e0ne has joined #openstack-sdks14:46
Shrewshttp://logs.openstack.org/58/566158/1/check/nodepool-functional-py35/45d52ed/controller/logs/screen-nodepool-builder.txt.gz#_May_04_05_56_28_53711114:47
mordredShrews: crap14:48
mordredShrews: the plural/singular is intentional14:48
mordredbut14:48
mordrednot particularly awesome for this transition14:48
Shrewsso i think pushing your occ patch, given that ^ which would require a new osdk release, is probably the thing to do now?14:51
mordredyah14:51
mordredand I'll also work on a shade patch to use osdk for config directly14:51
*** fabian_ has joined #openstack-sdks14:56
*** dmellado has quit IRC14:57
*** dmellado_ is now known as dmellado14:57
*** fabian_ is now known as chenyb414:58
openstackgerritMonty Taylor proposed openstack-infra/shade master: Use openstack.config directly for config  https://review.openstack.org/56634015:11
mordredShrews: ^^15:11
*** thrash is now known as thrash|biab15:15
*** dave-mccowan has quit IRC15:20
*** pooja_jadhav has quit IRC15:24
*** dave-mccowan has joined #openstack-sdks15:28
*** dayou_ has quit IRC15:29
*** e0ne has quit IRC15:31
dmsimardmordred: I got my thing to work. It's kinda dirty but I'll figure a better way later.15:55
mordreddmsimard: ossum15:56
mordreddmsimard: maybe at some point we should add a method to shade to be able to do the search you wanted to do15:56
mordreddmsimard: since we also do it inside of shade - and there isn't a *good* way to do it otherwise15:56
dmsimardmordred: in case you're curious: https://gist.github.com/dmsimard/6b21c9818878f8aacaf3ddaf63c669b315:56
dmsimardbrb15:56
mordreddmsimard: fwiw - in your setup logging function - you don't need to go through yaml - you can just build that dict directly in python and pass it to dictConfig (what you're doing obviously works - and I kind of like it from a readability perspective, but thought I'd mention just in case)16:05
mordreddmsimard: other than that - neat!16:05
openstackgerritMatt Smith (_alastor_) proposed openstack/openstacksdk master: Adding block_storage v3 API support  https://review.openstack.org/56635516:06
dmsimardmordred: it's copy pasta from a boilerplate thing16:07
openstackgerritMatt Smith (_alastor_) proposed openstack/openstacksdk master: Adding block_storage v3 API support  https://review.openstack.org/56635516:08
mordreddmsimard: ++16:09
*** EmilienM is now known as EvilienM16:09
dmsimardmordred: oh god, you can specify CIDRs in allowed_address_pairs and the mac address is optional T_T16:18
dmsimardI was complicating my life for nothing16:18
mordreddmsimard: \o/16:19
openstackgerritDavid Moreau Simard proposed openstack-infra/shade master: Add a description field for port_create, port_update  https://review.openstack.org/56635916:20
dmsimardI did find ^ though16:20
mordreddmsimard: well - just think aboutall th ethings you learned16:20
dmsimardmordred: I learn new things every day and it's awesome :D16:20
*** thrash|biab is now known as thrash16:29
openstackgerritMonty Taylor proposed openstack-infra/shade master: Use openstack.config directly for config  https://review.openstack.org/56634016:30
openstackgerritMonty Taylor proposed openstack-infra/shade master: Use openstack.config directly for config  https://review.openstack.org/56634016:30
*** jpich has quit IRC16:31
*** dtantsur is now known as dtantsur|afk16:32
*** chenyb4 has quit IRC16:45
*** jpena is now known as jpena|off16:55
openstackgerritMerged openstack/os-client-config master: Add cache methods back to OpenSackConfig  https://review.openstack.org/56632216:59
Shrewsdmsimard: regarding my comment about a test for 566359, modifying existing tests should be ok too16:59
dmsimardShrews: ack17:00
openstackgerritNobuto Murata proposed openstack/python-openstackclient master: Support --community in openstack image list  https://review.openstack.org/56515217:02
mordredShrews: remote:   https://review.openstack.org/566368 Release 0.13.1 of os-client-config17:17
*** tosky has quit IRC17:27
*** tosky has joined #openstack-sdks17:30
Shrewsmordred: what about https://review.openstack.org/566139 ?17:31
Shrewsi guess not as many people affected by that one17:32
mordredShrews: yah - and it's causing the tests to flap for a reason I can't figure out yet17:45
mordredShrews: I think it's interactions with globals across test cases - but I can't find which one17:45
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Clean up floating ip tests  https://review.openstack.org/56632018:13
mordredslaweq: ^^ that contains a fix related to the question I just asked in #openstack-neutron (the one where I flooded the channel)18:14
mordredShrews: ^^ that patch should hopefully make the sdk tests not quite so flappy/racey - and also fixes a bug18:14
mordredShrews: also, https://review.openstack.org/#/c/566340/ is ready for your scorn and derision18:39
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add nodepool functional tests  https://review.openstack.org/56638818:53
openstackgerritMerged openstack/cliff master: exclude cmd2 0.8.3 and update to 0.8.4  https://review.openstack.org/56012119:03
*** gtema has quit IRC19:06
slaweqmordred: yep, I saw it now :)19:41
slaweqmordred: I will take a look in few minutes19:41
*** e0ne has joined #openstack-sdks19:59
_alastor_hey folks, how do you set the service version from the ones available under the service's 'valid_versions' in the OSDK?20:19
_alastor_https://github.com/openstack/openstacksdk/blob/master/openstack/identity/identity_service.py#L1920:20
_alastor_How does the SDK decide to use v3 vs v2?20:21
_alastor_It seems like it picks the first one in the list to use, but how can I force it to use the other one?20:21
_alastor_Is there a better place for these questions?20:30
*** bobh has quit IRC20:57
*** bobh has joined #openstack-sdks20:58
*** e0ne has quit IRC21:11
*** dave-mccowan has quit IRC21:19
*** bobh has quit IRC21:23
_alastor_Is there a mailing list?  Maybe I could ask my questions there?21:41
*** edmondsw has quit IRC21:47
*** edmondsw has joined #openstack-sdks21:47
openstackgerritMerged openstack-infra/shade master: Use openstack.config directly for config  https://review.openstack.org/56634021:51
*** edmondsw has quit IRC21:52
*** figleaf is now known as edleafe21:54
*** slaweq has quit IRC22:03
*** markvoelker has quit IRC22:29
*** gildub has joined #openstack-sdks22:43
*** _alastor_ has quit IRC23:05
*** bobh has joined #openstack-sdks23:06
*** slaweq has joined #openstack-sdks23:11
*** slaweq has quit IRC23:15
*** bobh has quit IRC23:17
*** gildub has quit IRC23:44
« Thursday, 2018-05-03 Index Saturday, 2018-05-05 »

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