| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-11-12 | |||
| 21:18:53 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/519167 | |
| #openstack-sdks - 2017-11-13 | |||
| 00:12:44 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Replace profile with openstack.config https://review.openstack.org/501428 | |
| 00:12:45 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Strip endpoint_filter https://review.openstack.org/501429 | |
| 00:12:45 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename _Request.uri to _Request.url https://review.openstack.org/501430 | |
| 00:12:46 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Make proxy subclass of adapter https://review.openstack.org/501431 | |
| 00:12:46 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename bare_metal to baremetal https://review.openstack.org/501432 | |
| 00:12:47 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename cluster to clustering https://review.openstack.org/501433 | |
| 00:12:47 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename block_store to block_storage https://review.openstack.org/501434 | |
| 00:12:48 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rename telemetry to meter https://review.openstack.org/501435 | |
| 00:12:48 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Create generic proxies for all services in STA https://review.openstack.org/501436 | |
| 00:12:49 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: More connection cleanup https://review.openstack.org/501437 | |
| 00:12:49 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix all of the unit tests https://review.openstack.org/519024 | |
| 00:12:50 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add profile back for osc compat https://review.openstack.org/519204 | |
| 00:14:38 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Rework config and rest layers https://review.openstack.org/519029 | |
| 00:14:39 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add notes about moving forward https://review.openstack.org/501438 | |
| 01:25:35 | openstackgerrit | Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/519167 | |
| 06:00:48 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Add cluster support force delete parameter when cluster/node delete https://review.openstack.org/518874 | |
| 07:07:51 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Add block_store support single volume display image message https://review.openstack.org/519244 | |
| 08:06:28 | openstackgerrit | yangweiwei proposed openstack/python-openstackclient master: Fix credentials in create, update and list. https://review.openstack.org/519254 | |
| 09:36:50 | openstackgerrit | yangweiwei proposed openstack/python-openstackclient master: Fix credentials in create, update and list. https://review.openstack.org/519254 | |
| 12:16:09 | kuzko | can someone tell me if it's possible in shade to change the tenant after having declared my openstack_cloud() element? | |
| 12:18:20 | kuzko | I would like to list running instances on all the tenants my specified credential has access to, so I list the tenants and for each of them change the auth['project_name'] to the tenant name i'm processing... but it seems the id of the current_project doesn't change... | |
| 13:38:37 | mordred | kuzko: yah - auth['project_name'] will only get read at create time - adding a thing to do what you're talking about it on the list ... was thinking something like "with other_tenant('tenant-name') as new_tenant: new_tenant.list_servers()" or something similar | |
| 13:39:25 | mordred | kuzko: for now the best bet would just be to make another cloud object but overriding the project_name in kwargs ... there's an example in the functional tests of doing that - one sec ... | |
| 13:40:57 | kuzko | oh good day sir mordred ! thank you for the workaround, the onjly thing that came to my mind would have to create a temporary cloud.yml change the tenant it wazs supposed to point to and create a new openstack_cloud with it... if yyou tell me there is a pure python of doing it i'll teka it ^^ | |
| 13:40:59 | kuzko | *take | |
| 13:42:33 | mordred | kuzko: yah - so you can actually overlay values ... | |
| 13:42:43 | mordred | kuzko: https://github.com/openstack-infra/shade/blob/master/shade/tests/functional/test_users.py#L135 is the example I was thinking of | |
| 13:42:46 | kuzko | how so? | |
| 13:43:30 | mordred | kuzko: so you can just say openstack_cloud(cloud='cloud-name', project_name='new-project') and it'll give you a cloud basd on cloud-name but with project_name set to a different project | |
| 13:43:53 | mordred | or - at least, you should be able to | |
| 13:44:04 | kuzko | oh it's that simple? let me try that right away | |
| 13:46:46 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Add cluster support force delete parameter when cluster/node delete https://review.openstack.org/518874 | |
| 13:47:06 | kuzko | nope, doing so sends a MissingRequiredOptions auth plugin requires parameters which were not given : auth_url | |
| 13:47:45 | kuzko | mordred : maybe I should specifiy every parameter of the cloud endpoint? | |
| 13:49:50 | kuzko | nope even with that it doesn't return the proper values... | |
| 13:49:54 | mordred | hrm. | |
| 13:50:06 | mordred | gimme a sec ... lemme poke real quick | |
| 13:52:11 | mordred | kuzko: http://paste.openstack.org/show/626160/ <-- this works for me | |
| 13:52:36 | mordred | (I commented project_name out of the clouds.yaml entry for vexxhost) | |
| 13:56:06 | kuzko | I replied to your paste | |
| 13:57:51 | kuzko | it's using keystone V2.0 api endpoint if it means anythings | |
| 13:58:34 | mordred | hrm | |
| 13:58:55 | kuzko | http://paste.openstack.org/show/626161/ | |
| 13:59:33 | mordred | ok. it's not overriding the setting. one sec ... | |
| 14:00:49 | mordred | aha - duh. it's in the auth dict ... try wrapping it in an auth=dict( ... | |
| 14:00:50 | mordred | cloud = shade.openstack_cloud(cloud='vexxhost', auth=dict(project_name=project_name)) | |
| 14:02:53 | kuzko | no, it's still giving me the empty list treatment | |
| 14:04:27 | kuzko | and if I don't specify the auth_url, username and password in the dict() it fails | |
| 14:04:28 | kuzko | cloud2=openstack_cloud(cloud2="cloudwatt_fr1", auth=dict(project_name=tenant["name"])) | |
| 14:04:36 | kuzko | gives keystoneauth1.exceptions.auth_plugins.MissingRequiredOptions: Auth plugin requires parameters which were not given: auth_url | |
| 14:05:27 | mordred | oh - I see it | |
| 14:05:39 | mordred | cloud2=openstack_cloud(cloud2="cloudwatt_fr1", auth=dict(project_name=tenant["name"])) needs to be cloud2=openstack_cloud(cloud="cloudwatt_fr1", auth=dict(project_name=tenant["name"])) | |
| 14:07:10 | kuzko | y am I dumb? | |
| 14:07:34 | kuzko | it works, thank you very much | |
| 14:17:56 | mordred | \o/ | |
| 14:18:12 | mordred | I'll hopefully get a better answer for that soon - it comes up in a bunch of contexts | |
| 14:42:29 | Shrews | mordred: mmm, that tenant context manager would be nifty | |
| 14:53:57 | mordred | Shrews: I think I've almost got it written now (got nerdsniped) | |
| 14:54:38 | mordred | Shrews, briancurtin, dtroyer, SamYaple: email overview of the outstanding patches send to dev list with subject: "[python-openstacksdk][shade] Reviewing the merge-shade patches" | |
| 14:57:04 | Shrews | mordred: oh, so shade as a repo would be no more? that's news to me | |
| 14:57:16 | Shrews | but i guess it makes sense | |
| 14:59:21 | Shrews | mordred: what about the ansible tests? quick scan of that email doesn't show a reference | |
| 15:01:21 | Shrews | actually, i'll reply to that email | |
| 15:19:07 | mordred | Shrews: well - shade as a repo will stick around - but it'll become a very thin layer that wraps the shade code in openstacksdk | |
| 15:20:00 | mordred | Shrews: because we have an opportunity to fix some of the defaults with moving the code to the sdk tree - but we need to keep backwards compat for shade users ... | |
| 15:21:25 | mordred | Shrews: so, taking extra_specs as an example (turns out our default behavior of always fetching was stupid) ... we can turn that off by default in the sdk code, but have list_flavors in shade be def list_flavors(self, fetch_extra_specs=True): sdk.list_flavors(fetch_extra_specs=fetch_extra_specs) ... | |
| 15:22:21 | mordred | Shrews: I should probably write down thinking about that too shouldn't I? | |
| 15:22:58 | mordred | Shrews: btw - same thing with OCC ... opportunity to clean up the code in the sdk tree - as long as we can keep a compat shim for folks using it directly via occ itself | |
| 15:24:05 | openstackgerrit | Hongbin Lu proposed openstack/python-openstackclient master: Replace %r with %s on printing string variable https://review.openstack.org/519385 | |
| 15:36:58 | kuzko | hum, I'm back on the same thing but with a new case... how can I deal with v3 domains??? | |
| 15:38:36 | kuzko | running a cloud.list_projects() returns a 401 , so I'm looking to return the domains and search in them for servers, but I don't really see how... cloud.list_domains() doesn't seem to be a thing | |
| 15:41:24 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Add ability to work in other auth contexts https://review.openstack.org/519394 | |
| 15:41:35 | mordred | kuzko: ^^ maybe that will actually give you the tools you need | |
| 15:42:35 | mordred | kuzko: also - list_domains IS a thing, but it's in the OperatorCloud object, so instead of shade.openstack_cloud() you need to use shade.operator_cloud() ... and thanks, that points out a bug in the patch above | |
| 15:43:53 | Shrews | mordred: k, I get it now. Thx | |
| 15:43:54 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Add ability to work in other auth contexts https://review.openstack.org/519394 | |
| 15:47:31 | mordred | kuzko, Shrews: ^^ ... if that looks good to folks, I'll add some tests for it ... I tested it locally with http://paste.openstack.org/show/626175/ ... although I just added one more line to that test and found a place it's unhappy | |
| 15:54:06 | kuzko | hummmm... I didn't expect to raise a patch ... so I'm kinda stuck because the public cloud I use seems to lock access to operator_cloud() stuff (quite understandable) so ... hmmmmm | |
| 15:54:58 | mordred | kuzko: well - the patch I've got above should handle the domain stuff without needing operator cloud or list_domains ... | |
| 15:55:51 | mordred | kuzko: if the projects are in different domains, then list_projects should return domain-id in the project payload - and connect_as_project / as_project should properly pull that from the project dict returned from list_projects | |
| 15:57:10 | mordred | kuzko: you can TOTALLY copy-pasta code from those methods in to your program - but we can also likely get a new shade release cut with those methods within the day if they work for you | |
| 15:57:32 | mordred | speaking of - I should really get a cloudwatt acount ... | |
| 15:57:52 | kuzko | oh right, i'll try to copy paste my way out of this... | |
| 15:58:47 | kuzko | and I don't really suggest this, they are quite terrible... for them, an horizon console unavailable is not considered downtime if the API's endpoints on one of the two regions are available... | |
| 16:03:02 | mordred | kuzko: well - that may be - but I like to have accounts on all the public clouds so that I can do testing against them from time to time | |
| 16:03:29 | mordred | kuzko: it wasn't an issue this time, but sometimes if you have an issue on cloudwatt and I try to reproduce it on citycloud I can't ... | |
| 16:03:42 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Add ability to work in other auth contexts https://review.openstack.org/519394 | |
| 16:03:45 | mordred | this time with tests | |
| 16:04:58 | mordred | kuzko: if you copy/paste those methods into your program, just replace the self parameter with a parameter that you can pass a cloud object too and they should otherwise work - you'll need to add imports for keystoneauth1.session, copy and contextlib | |
| 16:05:03 | mordred | Shrews: ^^ added tests | |
| 16:06:14 | kuzko | well, I modified openstackcloud.py, and now I'll try to find a way to change that self stuff you mentionned | |
| 16:10:40 | GreenBlood | mordred: kuzko and I, at the office, are always amazed by your work on shade and the speed at which you program stuff | |
| 16:18:10 | kuzko | mordred by any means, do you have an account on opentelekom cloud? | |
| 16:31:16 | mordred | kuzko: I do not - I had one for a little bit but I don't think it works any more - I keep meaning to follow up on that | |
| 16:32:04 | mordred | GreenBlood: thanks - well, it's usually because there is something that's been on the TODO list for a while and someone just gives an excuse to do it - it makes it look like I'm thinking faster than I actually am :) | |
| 16:33:01 | kuzko | well, I was asking because telefonica's public cloud, t-system open public cloud and oranges cloud for business are the same backend, so verifying on one of these is pretty much validating the 3 of them... | |
| 16:33:39 | mordred | ah - neat | |
| 16:34:08 | mordred | cloudwatt moved to using the huawei openstack stuff? I didn't realize that | |
| 16:35:41 | briancurtin | mordred: i’ll look around for that mailing list thread and try to prioritize a few of them. i normally do some open source stuff in the morning before start The Real Job, so i’ll try to get a few of them looked at this week | |