| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-01-09 | |||
| 17:00:17 | fabian_ | mordred, ? | |
| 17:10:22 | mordred | fabian_: just got off - where are you seeing this? in a job somewhere? | |
| 17:10:36 | mordred | oh - I see now - master branch of osc | |
| 17:10:50 | fabian_ | mordred, yes. | |
| 17:11:16 | fabian_ | mordred, https://github.com/openstack/python-openstacksdk/blob/master/openstack/connection.py#L206 https://github.com/openstack/python-openstacksdk/blob/master/openstack/config/loader.py#L468 | |
| 17:12:29 | mordred | fabian_: sweet, thanks. lemme look at that flow real quick | |
| 17:12:59 | fabian_ | mordred, if i don't config cloud, the name is defaults, but self.cloud_config['clouds'] is envvars, so appear error rais "Cloud {name} was not found." error | |
| 17:15:39 | mordred | yah - the intent here is to not load config a second time since osc will have already loaded it, but in the envvars case that's not working ... | |
| 17:16:21 | mordred | fabian_: I've actually got a patch up which changes a constructor and I think I should update the _from_profile method to use it - we can cut out a bunch of complexity | |
| 17:16:29 | mordred | fabian_: lemme make you a patch real quick | |
| 17:20:23 | fabian_ | mordred, ok, thanks. | |
| 17:33:17 | openstackgerrit | Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514 | |
| 18:01:25 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename CloudConfig to CloudRegion https://review.openstack.org/531611 | |
| 18:01:26 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add function to make CloudRegion from session https://review.openstack.org/531612 | |
| 18:01:26 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Port wait_for_ methods to use iterate_timeout https://review.openstack.org/531268 | |
| 18:01:27 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Prefer links dicts for pagination https://review.openstack.org/530769 | |
| 19:35:34 | openstackgerrit | Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/531089 | |
| 20:25:48 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Prefer links dicts for pagination https://review.openstack.org/530769 | |
| 21:25:05 | openstackgerrit | Hongbin Lu proposed openstack/python-openstackclient master: Support filtering port with IP address substring https://review.openstack.org/532324 | |
| 21:45:06 | mordred | Shrews: if you're in the mood for some really fun reviewing ... https://review.openstack.org/#/c/530769/ and the three in front of it are ready now I believe | |
| 21:55:28 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Fix masked variable name https://review.openstack.org/484340 | |
| 22:14:21 | openstackgerrit | Merged openstack/python-openstacksdk master: Adds support to retrieve cinder backend pools information https://review.openstack.org/506242 | |
| 22:41:55 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove legacy client factory functions https://review.openstack.org/532347 | |
| 23:07:43 | Shrews | mordred: will look tomorrow | |
| 23:43:04 | mordred | Shrews: your face will look tomorrow | |
| #openstack-sdks - 2018-01-10 | |||
| 00:03:34 | openstackgerrit | Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514 | |
| 00:12:34 | openstackgerrit | Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514 | |
| 01:30:16 | chenyb4 | mordred, hi, i was miss a litter time. can you fix about cloud name patch ? | |
| 02:21:43 | openstackgerrit | Bar Elharar proposed openstack/osc-lib master: Suppress subTest() expected errors https://review.openstack.org/531025 | |
| 02:52:48 | openstackgerrit | Merged openstack/keystoneauth master: Fix masked variable name https://review.openstack.org/484340 | |
| 07:34:32 | openstackgerrit | Xi Yang proposed openstack/python-openstackclient master: Change 'Volume' to 'Block Storage' https://review.openstack.org/532439 | |
| 12:01:10 | amotoki | slaweq: could you review https://review.openstack.org/#/c/529951/ ? osc-lib depends on it and we would like to land it in queens | |
| 12:03:04 | Mr_Broken | Hey Guys | |
| 12:03:04 | Mr_Broken | Any chance that shade will be supporting SubnetPool functionality ? ( create, delete , manage ? ) | |
| 12:58:23 | slaweq | amotoki: hi, I will check it soon | |
| 12:58:36 | amotoki | slaweq: thanks | |
| 13:27:05 | slaweq | amotoki: done | |
| 13:27:50 | amotoki | slaweq: thanks. do you know who releases SDKs recently? | |
| 13:29:50 | slaweq | I don't know who did it recently but AFAIR some time ago I was asking briancurtin for that | |
| 13:30:59 | amotoki | I see. I think openstacksdk is now an official project "OpenStackSDK" and mordred is PTL, so it is clear to me | |
| 13:31:46 | briancurtin | amotoki: yeah i still release, but i would check with mordred on where things are at because there’s a lot of stuff going on right now | |
| 13:32:27 | amotoki | briancurtin: thanks. the reason I am checking this is the next week is non-client lib release, so I think openstacksdk and osc-lib are both affected | |
| 13:33:40 | amotoki | wait... I thought openstacksdk and osc-lib are not catagorized into client lib... i might be wrong | |
| 13:33:41 | briancurtin | yeah that’s something i don’t know anything about, as sdk has never been “official”, so traditionally if we needed to release something we release it | |
| 13:34:36 | amotoki | yeah. it is good time to clarify :) | |
| 14:04:01 | mordred | amotoki, slaweq: the process now will involve submitting a patch to the openstack/releases repo requesting a release be cut ... | |
| 14:04:17 | mordred | I'll check and see whether we fall in to client lib or non-client lib .. | |
| 14:06:31 | slaweq | thx mordred | |
| 14:19:21 | mordred | slaweq: if you happen to have some free time ... https://review.openstack.org/#/c/532347 and the ones before it are ready for review | |
| 14:19:49 | slaweq | mordred: sure, I will look today | |
| 14:23:01 | mordred | slaweq: thanks! | |
| 14:25:32 | Shrews | mordred: https://review.openstack.org/#/c/531611/4/doc/source/user/config/using.rst changes get_one_cloud() to get_one(). Same with get_all_clouds(). I don't see those defined in OCC | |
| 14:26:10 | Shrews | oh, duh. | |
| 14:27:32 | Shrews | silly code merging. ignore me | |
| 14:51:50 | mordred | Shrews: yah. I'm trying to make that as confusing for you as possible | |
| 14:52:57 | mordred | Shrews: that said - I think we're really close on the OCC import being in good shape, so when we cut the next sdk release we should be able to get the code ripped out from occ and have it depend on the sdk version of the code and be pretty much done with that side of the merge | |
| 15:53:20 | openstackgerrit | Merged openstack/python-openstacksdk master: Rename CloudConfig to CloudRegion https://review.openstack.org/531611 | |
| 17:10:34 | jpena | hi, I have a question about cliff 2.10.0. I think it's breaking the magnum gate, but I need some guidance on what the best fix would be | |
| 17:21:09 | jpich | jpena: Is there a bug to see more about the issue? | |
| 17:22:23 | jpena | jpich: I saw the issue when the package failed to build for RDO Trunk ()https://review.rdoproject.org/r/11275 , then started following the thread | |
| 17:23:07 | jpich | Oh, the legacy command name translation | |
| 17:23:21 | jpena | the TL;DR is that magnum calls the CommandManager constructor using None as a namespace, which causes commands not to be loaded in https://github.com/openstack/cliff/blob/2.10.0/cliff/commandmanager.py#L55-L56 | |
| 17:23:46 | jpena | I'm not familiar with cliff, so I wonder if this should be a bug in Magnum or cliff | |
| 17:26:33 | jpena | brb | |
| 17:27:40 | jpich | jpena: I'm not very familiar yet with cliff either, but since one of the patches I was involved with got included in 2.10 I thought I would ask in case I could help... I'm afraid I don't know the answer to your question, though perhaps dhellmann would have an idea? | |
| 17:28:16 | jpena | ok, thanks :) | |
| 17:28:18 | jpich | jpena: I think in that case I'd be tempted to file the bug with both magnum and cliff as components for now, so the experts on both sides can find the best solution | |
| 17:28:30 | jpena | btw, this seems to be the same as https://bugs.launchpad.net/python-cliff/+bug/1741911 | |
| 17:28:31 | openstack | Launchpad bug 1741911 in cliff "Regression with cliff 2.10.0" [Undecided,New] | |
| 17:28:38 | jpich | I mean, know which is more likely to break things or not | |
| 17:28:39 | jpich | ah! | |
| 17:28:40 | jpena | I'll add info to that bug | |
| 17:29:50 | jpich | Thanks! Good luck | |
| 17:44:23 | mordred | jpena, jpena: hah. my nick highlighting for both of you wound up being the same color and so for a minute I thought one of you just had a very long conversation with yourself | |
| 17:45:20 | openstackgerrit | Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514 | |
| 19:46:35 | openstackgerrit | Samuel Pilla proposed openstack/python-openstackclient master: Add project tags functionality https://review.openstack.org/481284 | |
| 20:51:54 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Merge user and users sections of the docs https://review.openstack.org/532633 | |
| 20:51:54 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rationalize logging helpers and docs https://review.openstack.org/532634 | |
| 21:54:09 | openstackgerrit | Merged openstack/python-openstacksdk master: Add function to make CloudRegion from session https://review.openstack.org/531612 | |
| 21:56:56 | openstackgerrit | Merged openstack/python-openstacksdk master: Port wait_for_ methods to use iterate_timeout https://review.openstack.org/531268 | |
| 22:58:39 | openstackgerrit | Michael Johnson proposed openstack/python-openstacksdk master: Add an octavia functional test gate https://review.openstack.org/531514 | |
| 23:58:38 | openstackgerrit | Merged openstack/python-openstacksdk master: Allow to pass filters like domain to find_project/user https://review.openstack.org/529951 | |
| 23:58:39 | openstackgerrit | Merged openstack/python-openstacksdk master: Add _query_mapping to identity resources https://review.openstack.org/529952 | |
| 23:58:40 | openstackgerrit | Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/531974 | |
| 23:59:20 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Remove metric service https://review.openstack.org/532697 | |
| #openstack-sdks - 2018-01-11 | |||
| 00:12:04 | mordred | ignore that ^^ | |
| 00:14:50 | mordred | Qiming: if you have a sec, feel like reviewing https://review.openstack.org/#/c/530769/ ? I think it should be ok from your earlier comments | |
| 00:16:45 | mordred | Qiming: also, I pushed up a patch to heat to shift it from using senlinclient to using SDK https://review.openstack.org/#/c/532680/ and a senlin patch https://review.openstack.org/#/c/532641/ that I'd love your feedback on | |
| 02:58:50 | adriant | is it just me, or query params/filters are busted for the openstackSDK? | |
| 02:59:52 | adriant | I'm trying to list stuff via the SDK and the list commands appear to just take key/values as kwargs, but those aren't actually being used since QueryParameters._transpose() appears to gobble them up... | |
| 03:06:43 | adriant | from what I can tell, this is because 'somewhere' on the resource we need to define what they acceptable query params are... or something? but this isn't actually happening anywhere. | |
| 03:07:52 | adriant | no wait... they are actually being used | |
| 03:07:59 | adriant | just not at all for any of the identity | |
| 03:08:26 | adriant | but then the terrifying thing is that after mangling the query, i get returned EVERYTHING | |
| 03:09:26 | adriant | so if I do: sdk.identity.projects(name="some project") | |
| 03:09:43 | adriant | a list with projects of that name, OR nothing | |
| 03:09:47 | adriant | yet I get everything... | |
| 03:10:13 | adriant | because my query is being mangled and the API then does a call for everything | |
| 03:10:34 | adriant | I think it would be much much safer to no mangle the query, just throw a damn error | |
| 03:10:42 | adriant | not* | |