| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-23 | |||
| 18:34:06 | openstackgerrit | Merged openstack/os-service-types master: Updated from global requirements https://review.openstack.org/555579 | |
| 18:34:06 | openstackgerrit | Merged openstack/os-service-types master: Fix docs link in the README https://review.openstack.org/555324 | |
| 19:56:35 | openstackgerrit | Merged openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/555558 | |
| 19:59:59 | openstackgerrit | Merged openstack/os-client-config master: Updated from global requirements https://review.openstack.org/551595 | |
| 21:31:10 | openstackgerrit | Merged openstack/osc-lib master: Rename python-openstacksdk to openstacksdk https://review.openstack.org/555793 | |
| 21:32:39 | openstackgerrit | Merged openstack/openstacksdk master: Update python-openstacksdk references to openstacksdk https://review.openstack.org/555875 | |
| 22:00:03 | openstackgerrit | Merged openstack/keystoneauth master: Remove tox_install.sh and align with constraints consumption https://review.openstack.org/550837 | |
| #openstack-sdks - 2018-03-24 | |||
| 00:39:35 | openstackgerrit | Jacky Hu proposed openstack/openstacksdk master: Allow cascade deletion of load balancer https://review.openstack.org/553404 | |
| 03:00:33 | openstackgerrit | Emilien Macchi proposed openstack/python-openstackclient master: CI test - never merge https://review.openstack.org/555979 | |
| 14:55:58 | openstackgerrit | Doug Hellmann proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037 | |
| 14:56:24 | openstackgerrit | Doug Hellmann proposed openstack/os-api-ref master: add lower-constraints job https://review.openstack.org/556039 | |
| 14:57:05 | openstackgerrit | Doug Hellmann proposed openstack/os-client-config master: add lower-constraints job https://review.openstack.org/556042 | |
| 19:53:22 | openstackgerrit | Doug Hellmann proposed openstack/osc-lib master: add lower-constraints job https://review.openstack.org/556059 | |
| 21:08:02 | EmilienM | dtroyer: when do you think we could have a release of openstackclient? | |
| 22:38:05 | openstackgerrit | Slawek Kaplonski proposed openstack/openstacksdk master: Add support for trunk ports and subports https://review.openstack.org/556089 | |
| #openstack-sdks - 2018-03-25 | |||
| 00:14:17 | openstackgerrit | Doug Hellmann proposed openstack/cliff master: add lower-constraints job https://review.openstack.org/555405 | |
| 00:14:28 | openstackgerrit | Doug Hellmann proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037 | |
| 01:02:07 | openstackgerrit | Doug Hellmann proposed openstack/cliff master: add lower-constraints job https://review.openstack.org/555405 | |
| 01:02:16 | openstackgerrit | Doug Hellmann proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037 | |
| 08:17:46 | openstackgerrit | Slawek Kaplonski proposed openstack/openstacksdk master: Add support for trunk ports and subports https://review.openstack.org/556089 | |
| 10:35:10 | openstackgerrit | Doug Hellmann proposed openstack/python-openstackclient master: add lower-constraints job https://review.openstack.org/556172 | |
| 10:46:08 | openstackgerrit | Doug Hellmann proposed openstack/openstacksdk master: add lower-constraints job https://review.openstack.org/556173 | |
| 13:07:38 | openstackgerrit | Doug Hellmann proposed openstack/openstacksdk master: add lower-constraints job https://review.openstack.org/556173 | |
| 15:27:18 | openstackgerrit | Doug Hellmann proposed openstack/python-openstackclient master: add lower-constraints job https://review.openstack.org/556172 | |
| #openstack-sdks - 2018-03-26 | |||
| 01:49:10 | openstackgerrit | Merged openstack/cliff master: Remove the warning of getargspec removal https://review.openstack.org/552652 | |
| 01:49:11 | openstackgerrit | Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/548672 | |
| 04:54:48 | openstackgerrit | Gilles Dubreuil proposed openstack/api-wg master: Updated with high level links https://review.openstack.org/556346 | |
| 05:00:43 | openstackgerrit | Gilles Dubreuil proposed openstack/api-wg master: Adds API-Schema guideline https://review.openstack.org/524467 | |
| 08:03:05 | openstackgerrit | melissaml proposed openstack/keystoneauth master: fix a typo in session.py https://review.openstack.org/556397 | |
| 14:48:34 | tobasco | catching the following for a queens openstackclient version | |
| 14:48:36 | tobasco | 'Execution of '/usr/bin/openstack project list --quiet --format csv --long' returned 1: __init__() got an unexpected keyword argument 'user_domain_name'' | |
| 14:48:52 | tobasco | anybody know's if something has changed somewhat related to that? | |
| 16:16:15 | EmilienM | dtroyer: hey, when do you think we could have a release of openstackclient (even a minor tag)? we need a new version in tripleo for the mtu parameter in neutron client | |
| 16:22:18 | dtroyer | EmilienM: I'll try to get one out this week. Our gate has been so screwed for the last week or two a bunch of stuff is backed up, and it looks like it is dicey yet again this morning :( | |
| 16:22:40 | EmilienM | oops | |
| 22:17:40 | openstackgerrit | Merged openstack-infra/shade master: Updated from global requirements https://review.openstack.org/555551 | |
| 23:18:26 | adriant | mordred: I have an interesting one for you again | |
| 23:18:48 | adriant | actually... wait lemme try one more thing | |
| 23:41:13 | adriant | mordred: the one more thing worked, but leads to the understanding that gophercloud is somewhat limited in the auth department compared to our various python libraries :( | |
| 23:56:43 | adriant | mordred: I think we need example auth files/options for other libraries to use as reference for implementation. The issue in question is that gophercloud doesn't support token auth via clouds.yaml but os-client-config and in turn the openstackclient do just fine. | |
| 23:58:40 | adriant | I'm not sure if that work exists, but a reference spec for: "this is how auth should work across all libraries" with examples would be great. We don't even have examples of all all the options in our code bases it looks like. At least I can't find a reference example of clouds.yaml with token auth, but I did get it to work now for a coworker's odd | |
| 23:58:41 | adriant | requirements. Turns out it 'works' we just put the auth_type in the wrong place | |
| 23:59:22 | adriant | mordred: http://paste.openstack.org/show/715236/ | |
| 23:59:42 | adriant | ^ that works with the openstachcli and such just fine, but terraform (gophercloud) falls over | |
| #openstack-sdks - 2018-03-27 | |||
| 00:02:52 | adriant | not that gophercloud is your problem, but we potentially should as part of... that SDKs initiative that was started in Sydney try and have a reference spec for Auth across all libraries, and write something to help test that. | |
| 00:03:46 | adriant | What was that initiative called? Am I misremembering something? OpenLabs or something? | |
| 00:04:59 | adriant | mrhillsman: You might be the person to bug about this! | |
| 00:05:07 | adriant | mrhillsman: ^ the above | |
| 00:26:26 | mrhillsman | just getting notification, reading | |
| 00:28:05 | mrhillsman | yes adriant you are absolutely right | |
| 00:29:02 | mrhillsman | not just auth but i spoke with mordred i think or maybe someone else about reference spec(s) for a number of "this is how x should work across all libraries" thing | |
| 00:31:55 | adriant | mrhillsman: yeah, we really do. I keep running into weird inconsistencies and it drives me mad :( | |
| 00:32:50 | mrhillsman | that ^ | |
| 00:33:08 | adriant | like terraform expects OS_AUTH_TOKEN while we've been using OS_TOKEN for ages | |
| 00:33:27 | adriant | and I can't find the terraform code or gophercloud where that is even being read in... | |
| 00:34:18 | adriant | At this point I assume os-client-config is the reference implementation, and everyone 'should' be based on that when it comes to auth and source variables and such | |
| 00:37:38 | mrhillsman | https://github.com/gophercloud/gophercloud/blob/master/openstack/auth_env.go | |
| 00:38:37 | adriant | mrhillsman: no token in there though | |
| 00:39:10 | adriant | terraform supports token auth with OS_AUTH_TOKEN so it has to exist somewhere | |
| 00:40:04 | mrhillsman | ah, i think i understand you now | |
| 00:42:48 | adriant | mrhillsman: or in today's case, os-client-config supports token in clouds.yaml, but gophercloud doesn't :P | |
| 00:43:06 | mrhillsman | ^ that might be the actual case | |
| 00:44:08 | adriant | well, sort of. It's both. Ideally terraform/gopherclouds should use both OS_TOKEN and OS_AUTH_TOKEN and try and deprecate the latter, and then ALSO should add token support to clouds.yaml | |
| 00:44:50 | mrhillsman | hmm. well, it says it does (terraform provider at least documents it) | |
| 00:45:34 | adriant | mrhillsman: https://www.terraform.io/docs/providers/openstack/#token | |
| 00:45:39 | adriant | just through OS_AUTH_TOKEN | |
| 00:46:52 | mrhillsman | ++ | |
| 00:48:02 | mrhillsman | the good thing is that jtopjian and in general the gophercloud folks are very responsive | |
| 00:48:15 | mrhillsman | and i do not think if you opened an issue it would take long to resolve | |
| 00:48:46 | mrhillsman | seems like a if you get one or the other do these things | |
| 00:49:02 | mrhillsman | unless there is something else offered by one vs the other | |
| 00:55:19 | mrhillsman | i'm creating an issue adriant | |
| 00:55:29 | mrhillsman | is ^ your github username? | |
| 00:56:06 | adriant | mrhillsman: adrian-turjak | |
| 00:56:15 | mrhillsman | thx | |
| 00:56:18 | adriant | mrhillsman: coworker already also opened: https://github.com/gophercloud/utils/issues/25 | |
| 01:02:29 | mrhillsman | cool | |
| 01:02:46 | adriant | hmmmm | |
| 01:02:50 | mrhillsman | i think that repo is only for figuring out if client config is being used and if so grab it | |
| 01:03:02 | adriant | mrhillsman: I've just realised something | |
| 01:03:33 | adriant | mrhillsman: https://github.com/terraform-providers/terraform-provider-openstack/blob/master/openstack/provider.go#L72-L77 | |
| 01:03:54 | adriant | the OS_AUTH_TOKEN is in the provider code. I was silly and looking in the wrong place | |
| 01:04:14 | adriant | but now I'm wondering, does that provider use gophercloud or write it's own stuff? | |
| 01:04:26 | adriant | I assumed (probably incorrectly) that it did | |
| 01:04:27 | mrhillsman | it uses gophercloud | |
| 01:04:34 | adriant | ok, then I'm not entirely mad | |
| 01:04:36 | mrhillsman | so you see the section right below that one | |
| 01:04:58 | mrhillsman | basically, the section you referenced would turn into multi like the one below | |
| 01:05:05 | mrhillsman | and OS_TOKEN as an option | |
| 01:05:32 | mrhillsman | and then - if i understand right - gophercloud needs to do the same essentially | |
| 01:05:39 | adriant | mrhillsman: so easy fix in the provide then | |
| 01:05:45 | adriant | provider* | |
| 01:05:49 | mrhillsman | for the provider yeah | |
| 01:06:09 | mrhillsman | looks like | |
| 01:06:43 | adriant | and that shouldn't need a change in gophercloud | |
| 01:07:02 | adriant | although we do want to add better clouds.yaml support there too | |
| 01:07:28 | mrhillsman | https://github.com/gophercloud/gophercloud/blob/master/auth_options.go | |
| 01:07:46 | mrhillsman | well, gophercloud will need to change | |
| 01:11:03 | adriant | although on the auth front... I have to apologise in advance because I'm probably going to end up adding a lot of extra complexity once https://review.openstack.org/#/c/553670 is merged and implemented | |
| 01:11:10 | adriant | mordred: ^ don't hate me too much :P | |