Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-05-04
14:17:01 dmsimard mordred: are you sitting on something ?
14:17:15 dmsimard mordred: I need to add all the IPs from a subnet to another subnet's ports allowed-address-pairs
14:17:30 dmsimard basically make the ports of a subnet able to bind the ips from another subnet
14:17:39 mordred wow
14:17:42 mordred that's awesome
14:17:53 dmsimard It's not hard, just need to figure that one thing
14:18:54 mordred dmsimard: so - you have two choices ...
14:19:06 dmsimard https://github.com/openstack-infra/shade/blob/16e7290ff45ca6fa5d7ba7360575c38e0b0fd043/shade/openstackcloud.py#L6237-L6247 :/
14:19:37 Shrews shade only supports filtering dictionaries of dictionaries. lists of dictionaries are not supported
14:19:42 Shrews yeah, that
14:19:42 mordred dmsimard: you can pass a jmsepath string to filters - or you can do a list ports and filter yourself
14:20:10 mordred listing ports and filtering yourself is likely the easier thing
14:20:22 Shrews i forgot we added jmespath support
14:20:35 mordred but if you wanna play with the jmsepath supporT: http://jmespath.org/
14:20:52 dmsimard mordred: I can filter myself I suppose
14:22:00 mordred Shrews: how about this ...
14:22:31 Shrews mordred: if the fix is clear in your head, feel free to add a PS to my review
14:22:53 Shrews i apparently need more coffee to see it
14:23:13 mordred Shrews: oh - I already pushed up https://review.openstack.org/566322 (should have done it as a PS to yours)
14:24:13 mordred Shrews: that said - I'm thinking maybe we just revert and release that - then add some more tests/test jobs and try the ectomy again
14:24:26 mordred Shrews: since we're clearly missing some coverage on ansible impact of occ changes
14:24:42 dmsimard mordred: 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:58 Shrews mordred: possibly a good idea
14:25:15 mordred dmsimard: nope. I recommend brute force
14:25:38 dmsimard mordred: I wonder if I could blindly fire create_ports and basically pass on exception
14:25:44 dmsimard but that'd be kind of dirty
14:25:49 Shrews mordred: i'll abandon mine assuming the revert
14:26:15 dmsimard mordred: hmmm.. how does http://docs.ansible.com/ansible/latest/modules/os_port_module.html handle idempotency ?
14:27:18 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Revert "Replace guts with openstack.config" https://review.openstack.org/566328
14:27:29 mordred Shrews: ^^
14:27:58 Shrews mordred: is that going to again break nodepool now that https://review.openstack.org/566138 *just* merged?
14:29:31 dmsimard mordred: looks like os_port basically relies on the name of the port.. which is not stupid
14:29:42 mordred Shrews: ugh. probably. we should get the followup patch fixed
14:29:57 pooja_jadhav mordred: Hi
14:31:02 mordred Shrews: maybe we should try the fix patch instead of the revert
14:32:45 Shrews mordred: your patch can't work i think. there is no self.get_cache_expiration_time()
14:33:09 Shrews mordred: that's only in CloudRegion, which is why i asked how to get to that
14:34:42 Shrews as are the _cache_* attrs
14:35:09 Shrews oh, no
14:35:13 Shrews wrong about the attrs
14:35:24 mordred the patch adds get_cache_expiration_time
14:35:48 Shrews oh duh
14:36:10 Shrews *sigh
14:36:36 Shrews mordred: ok, let's just try your patch first
14:37:06 mordred Shrews: 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 work
14:37:25 mordred we run occ tests on sdk patches, so that should prevent sdk patches from changing those private members
14:37:37 mordred and yes, that's terrible software engineering practice in general
14:40:57 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Revert "Replace guts with openstack.config" https://review.openstack.org/566328
14:41:12 mordred Shrews: well, there's that (without merge conflict this time) in case we decide we want to go that route
14:46:08 Shrews mordred: 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:23 Shrews mordred: accidental pluralization there?
14:47:45 Shrews http://logs.openstack.org/58/566158/1/check/nodepool-functional-py35/45d52ed/controller/logs/screen-nodepool-builder.txt.gz#_May_04_05_56_28_537111
14:48:13 mordred Shrews: crap
14:48:22 mordred Shrews: the plural/singular is intentional
14:48:24 mordred but
14:48:44 mordred not particularly awesome for this transition
14:51:32 Shrews so i think pushing your occ patch, given that ^ which would require a new osdk release, is probably the thing to do now?
14:51:46 mordred yah
14:51:59 mordred and I'll also work on a shade patch to use osdk for config directly
15:11:21 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Use openstack.config directly for config https://review.openstack.org/566340
15:11:24 mordred Shrews: ^^
15:55:14 dmsimard mordred: I got my thing to work. It's kinda dirty but I'll figure a better way later.
15:56:06 mordred dmsimard: ossum
15:56:28 mordred dmsimard: maybe at some point we should add a method to shade to be able to do the search you wanted to do
15:56:42 mordred dmsimard: since we also do it inside of shade - and there isn't a *good* way to do it otherwise
15:56:51 dmsimard mordred: in case you're curious: https://gist.github.com/dmsimard/6b21c9818878f8aacaf3ddaf63c669b3
15:56:57 dmsimard brb
16:05:02 mordred dmsimard: 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:45 mordred dmsimard: other than that - neat!
16:06:07 openstackgerrit Matt Smith (_alastor_) proposed openstack/openstacksdk master: Adding block_storage v3 API support https://review.openstack.org/566355
16:07:31 dmsimard mordred: it's copy pasta from a boilerplate thing
16:08:15 openstackgerrit Matt Smith (_alastor_) proposed openstack/openstacksdk master: Adding block_storage v3 API support https://review.openstack.org/566355
16:09:23 mordred dmsimard: ++
16:18:08 dmsimard mordred: oh god, you can specify CIDRs in allowed_address_pairs and the mac address is optional T_T
16:18:14 dmsimard I was complicating my life for nothing
16:19:48 mordred dmsimard: \o/
16:20:00 openstackgerrit David Moreau Simard proposed openstack-infra/shade master: Add a description field for port_create, port_update https://review.openstack.org/566359
16:20:01 dmsimard I did find ^ though
16:20:04 mordred dmsimard: well - just think aboutall th ethings you learned
16:20:16 dmsimard mordred: I learn new things every day and it's awesome :D
16:30:03 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Use openstack.config directly for config https://review.openstack.org/566340
16:30:12 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Use openstack.config directly for config https://review.openstack.org/566340
16:59:08 openstackgerrit Merged openstack/os-client-config master: Add cache methods back to OpenSackConfig https://review.openstack.org/566322
16:59:58 Shrews dmsimard: regarding my comment about a test for 566359, modifying existing tests should be ok too
17:00:26 dmsimard Shrews: ack
17:02:37 openstackgerrit Nobuto Murata proposed openstack/python-openstackclient master: Support --community in openstack image list https://review.openstack.org/565152
17:17:14 mordred Shrews: remote: https://review.openstack.org/566368 Release 0.13.1 of os-client-config
17:31:45 Shrews mordred: what about https://review.openstack.org/566139 ?
17:32:56 Shrews i guess not as many people affected by that one
17:45:15 mordred Shrews: yah - and it's causing the tests to flap for a reason I can't figure out yet
17:45:54 mordred Shrews: I think it's interactions with globals across test cases - but I can't find which one
18:13:51 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Clean up floating ip tests https://review.openstack.org/566320
18:14:13 mordred slaweq: ^^ that contains a fix related to the question I just asked in #openstack-neutron (the one where I flooded the channel)
18:14:40 mordred Shrews: ^^ that patch should hopefully make the sdk tests not quite so flappy/racey - and also fixes a bug
18:39:04 mordred Shrews: also, https://review.openstack.org/#/c/566340/ is ready for your scorn and derision
18:53:06 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Add nodepool functional tests https://review.openstack.org/566388
19:03:36 openstackgerrit Merged openstack/cliff master: exclude cmd2 0.8.3 and update to 0.8.4 https://review.openstack.org/560121
19:41:36 slaweq mordred: yep, I saw it now :)
19:41:57 slaweq mordred: I will take a look in few minutes
20:19:31 _alastor_ hey folks, how do you set the service version from the ones available under the service's 'valid_versions' in the OSDK?

Earlier   Later