| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-05-03 | |||
| 17:46:37 | _alastor_ | Example: http://paste.openstack.org/show/720311/ | |
| 17:47:38 | _alastor_ | The resulting object does not have the 'extra_specs' body attribute populated and the '_body' attribute is empty | |
| 17:48:24 | _alastor_ | But from the debug curl call, you can see that the data was received | |
| 17:51:41 | edleafe | jroll: hope you can join our discussion in Vancouver! | |
| 17:52:27 | elmiko | jroll: i think your comments on this email thread would be most welcome, http://lists.openstack.org/pipermail/openstack-dev/2018-April/129987.html | |
| 17:53:08 | jroll | edleafe: I cannot :( | |
| 17:53:38 | jroll | elmiko: I've been following along, was waiting to see where it went for a bit | |
| 17:56:24 | edleafe | heh, jaypipes already poo-pooed the idea. <shock!> | |
| 18:39:24 | elmiko | jroll: ++ | |
| 18:48:06 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Allow monkeypatching CONFIG_FILES https://review.openstack.org/566139 | |
| 18:53:39 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Allow monkeypatching CONFIG_FILES https://review.openstack.org/566139 | |
| 19:22:53 | _alastor_ | nevermind, looks like I needed to define a custom list function similar to this one for Server_IPs: https://github.com/openstack/openstacksdk/blob/master/openstack/compute/v2/server_ip.py#L37 | |
| 20:19:29 | _alastor_ | I can't find any examples for simple operations with the SDK. Creation/Listing examples exist, but no example for how to modify an SDK resource | |
| 20:19:41 | _alastor_ | For example, how do I go about changing the name of a volume using the SDK? | |
| 20:20:54 | _alastor_ | vol = conn.block_storage.get_volume('some-vol-id') | |
| 20:21:02 | _alastor_ | vol.name = "my-new-name" | |
| 20:21:07 | _alastor_ | then what? | |
| 20:22:30 | _alastor_ | Or maybe vol.update(conn.session, name="my-new-name")? | |
| 20:25:47 | _alastor_ | Maybe objects are look-but-don't-touch once created? | |
| 20:28:59 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Implement service_type alias lookups https://review.openstack.org/462218 | |
| 20:29:00 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add methods to get all of the version data https://review.openstack.org/559154 | |
| 20:29:01 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Make VersionData class https://review.openstack.org/564469 | |
| 20:29:02 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Turn normalize_status into a class https://review.openstack.org/564110 | |
| 20:29:03 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Infer version from old versioned service type aliases https://review.openstack.org/564299 | |
| 20:29:54 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Use Status variables in tests https://review.openstack.org/564258 | |
| 20:30:02 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Reference class variable in Status https://review.openstack.org/564262 | |
| #openstack-sdks - 2018-05-04 | |||
| 03:09:30 | pooja_jadhav | mordred: Hi | |
| 10:03:57 | frickler | seems there is a regression in os-client-config==1.31.0 breaking shade amongst other things https://bugs.launchpad.net/os-client-config/+bug/1768813 | |
| 10:03:57 | openstack | Launchpad bug 1768813 in os-client-config "config.get_cache_expiration_time() function missing in 1.31.0 release" [Undecided,New] | |
| 10:03:59 | frickler | mordred: ^^ | |
| 13:03:49 | openstackgerrit | David Shrewsbury proposed openstack/os-client-config master: Add these calls back to OCC https://review.openstack.org/566308 | |
| 13:04:15 | openstackgerrit | David Shrewsbury proposed openstack/os-client-config master: Add cache API methods back to OCC https://review.openstack.org/566308 | |
| 13:04:53 | Shrews | mordred: 566308 should add the cache API calls back, but it seems our tests are now testing openstacksdk and not os-client-config APIs | |
| 13:05:18 | Shrews | mordred: since we make those calls in occ tests and they apparently work | |
| 13:05:24 | Shrews | not sure how to correct that | |
| 13:05:34 | Shrews | frickler: ^^ | |
| 13:27:31 | pooja_jadhav | cmurphy : Hi | |
| 13:43:22 | openstackgerrit | Artem Goncharov proposed openstack/os-client-config master: fix link to openstacksdk in deprecation block https://review.openstack.org/566318 | |
| 13:50:55 | mordred | Shrews: looking | |
| 13:51:21 | mordred | Shrews: also the sdk functional tests keep hitting flapping errors related to floating ips - so I'm trying to clean that up | |
| 13:51:36 | Shrews | yay | |
| 13:52:33 | openstackgerrit | David Shrewsbury proposed openstack/os-client-config master: Add cache API methods back to OCC https://review.openstack.org/566308 | |
| 13:52:35 | frickler | Shrews: hmm, I think you are patching the wrong location, the issue for me is with config.OpenStackConfig() and not cloud_config.Config() | |
| 13:53:03 | mordred | AH | |
| 13:53:04 | Shrews | pooja_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:07 | mordred | that makes more sense to me | |
| 13:53:14 | frickler | Shrews: also yes, would be good to have tests for that | |
| 13:55:06 | pooja_jadhav | Shrews: Hi, I am not getting what you are trying to say | |
| 13:55:21 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Clean up floating ip tests https://review.openstack.org/566320 | |
| 13:55:58 | Shrews | pooja_jadhav: ooh, sorry. that was meant for someone else | |
| 13:56:38 | pooja_jadhav | shrews: ohk | |
| 13:57:29 | openstackgerrit | Merged openstack/os-client-config master: fix link to openstacksdk in deprecation block https://review.openstack.org/566318 | |
| 13:58:07 | Shrews | ok, i'm confused since those calls originally existed in CloudConfig | |
| 13:58:07 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: close files after open in unit/base https://review.openstack.org/566321 | |
| 13:59:45 | frickler | Shrews: 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-L204 | |
| 14:00:03 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Allow monkeypatching CONFIG_FILES https://review.openstack.org/566139 | |
| 14:00:04 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Add cache methods back to OpenSackConfig https://review.openstack.org/566322 | |
| 14:00:12 | mordred | Shrews: there were two sets of them | |
| 14:00:28 | Shrews | mordred: oh. well that's not confusing | |
| 14:00:34 | mordred | yah - it was terrible | |
| 14:00:37 | mordred | the CloudConfig ones originally called the OpenStackConfig ones | |
| 14:01:32 | mordred | BUT - 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 silly | |
| 14:01:53 | mordred | and I totally missed keeping them for backwards compat in the occ patch | |
| 14:02:27 | Shrews | i'm not clear on the fix then | |
| 14:02:47 | mordred | adding them back to OpenStackConfig so that occ users who were using them don't break | |
| 14:10:25 | Shrews | mordred: how do you get from occ.OpenStackConfig to the osdk.CloudRegion where those calls are defined? | |
| 14:10:49 | Shrews | those calls seem like they should be region independent | |
| 14:11:19 | Shrews | or is it allowed to have different cache characteristics per region? | |
| 14:12:00 | mordred | no, you're right- it's region independent | |
| 14:12:14 | mordred | thing is - the CloudRegion is the primary thing people work with | |
| 14:14:06 | mordred | so - basically, the original change was to have the cache settings be actual constructor arguments to CloudRegion rather than having to pass an OpenStackConfig to CloudRegion | |
| 14:14:30 | mordred | get_one_cloud then started passing the values to CloudRegion when it created one | |
| 14:14:53 | dmsimard | mordred: how would you do a search_ports for a fixed_ip ? | |
| 14:15:22 | mordred | dmsimard: conn.search_ports(filters={'fixed_ip': blah}) I believe | |
| 14:15:29 | dmsimard | http://paste.openstack.org/show/720377/ doesn't seem to work | |
| 14:15:35 | dmsimard | mordred: it's nested under fixed_ips | |
| 14:15:53 | dmsimard | mordred: i.e, http://paste.openstack.org/show/720378/ | |
| 14:16:15 | mordred | of course it is | |
| 14:16:31 | mordred | dmsimard: let's step back one - what are you trying to accomplish? | |
| 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 | |