| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-01-29 | |||
| 17:22:33 | dtroyer | mordred: do we have any idea if anything that isn't OSC does that Session thing? | |
| 17:27:06 | mordred | dtroyer: looking in heat right now to check | |
| 17:27:42 | mordred | dtroyer: it doesn't look like heat does | |
| 17:28:01 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Provide compatibility for people passing raw sessions https://review.openstack.org/538968 | |
| 17:29:43 | mordred | dtroyer: so we could just call the osc patch a good enough fix | |
| 17:29:59 | mordred | dtroyer: I think I'm mostly just shotgunning all of the possible options I can think of right now :) | |
| 17:30:33 | dtroyer | sure, I'd like to avoid more special cases that will be around forever in the SDK if we can | |
| 17:31:10 | mordred | dtroyer: totally agree | |
| 18:07:38 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Use find_ip from openstacksdk https://review.openstack.org/538958 | |
| 19:17:44 | Bar_ | dtroyer, hey, When is the next osc-lib package is expected to be released? | |
| 19:22:21 | dtroyer | Bar_: We just did what becomes the queens/stable release a week or so ago, nothing is queued up that I'm aware of | |
| 19:24:42 | Bar_ | dtroyer, I have proposed https://review.openstack.org/#/c/531025/ earlier this month, but it didn't make the cut. I have another patch that is dependent on that, though it is not urgent, I wanted to know what time table could i expect. | |
| 19:30:44 | dtroyer | That merged, but yes, not before 1.9.0 was released, gotcha | |
| 19:33:12 | dtroyer | Bar_: is next week soon enough? I'd like to hold off adding tot he release teams work (as small as this is) until the gate issues/backlog is cleared | |
| 19:35:58 | Bar_ | dtroyer, totally fine. Not urgent at all. I suspected it might take months, that's why I asked. | |
| 19:36:11 | Bar_ | am I correct to expect it to be v1.9.1? | |
| 19:36:27 | dtroyer | no, we also merged some global requirements, which force a minor rev | |
| 19:36:33 | dtroyer | it'll be 1.10.0 | |
| 19:38:12 | Bar_ | dtroyer, I see. thanks. By the way, it is besides the point, but maybe we should reconsider the reliance on testtools, it hasn't seen meaninful updates for almost a year now. | |
| 19:38:54 | Bar_ | My patch only suppresses exceptions of functionality that is natively supported by unittest | |
| 19:39:55 | Bar_ | testtools breaks that functionality, and there's not much hope there. | |
| 19:57:42 | mordred | Bar_: subTest does not exist in python2 - it's only in python3.4 and above - have you tried testscenarios instead? | |
| 19:59:04 | mordred | http://git.openstack.org/cgit/openstack/python-openstacksdk/tree/openstack/tests/functional/cloud/test_devstack.py#n25 | |
| 19:59:07 | Bar_ | mordred, sorry, I'm unfamiliar with testscenarios . We don't use it in the relevant repo | |
| 19:59:08 | mordred | is an example | |
| 20:00:13 | mordred | line 25 is key, then the scenarios attribute on line 32 lists a list of scenarios with a name for the scenario and a dict of attributes to add to the test for each scenario | |
| 20:00:33 | mordred | load_tests then explodes that out during unittest discovery listing to a test named for each scenario | |
| 20:00:50 | mordred | which means they can be scheduled across test runner backends for multi-process/parallel testing | |
| 20:02:57 | mordred | Bar_: if you look at http://codesearch.openstack.org/?q=testscenarios&i=nope&files=&repos= you can see it's used fairly extensively across openstack -so before we were to go trying to solve the same problem using a different tech, I'd personally like to figure out if there is a benefit to using subTest over testscenarios | |
| 20:03:02 | Bar_ | mordred, subtest fails with PY35 when inheriting from testtools. | |
| 20:04:28 | mordred | Bar_: right. but what I'm saying is that rather than trying to convince openstack to move off of testtools, you might have better luck just using testscenarios which works with both python2.7 and python3.5 | |
| 20:04:49 | Bar_ | mordred, cool | |
| 22:25:26 | mordred | dtroyer: https://review.openstack.org/#/c/538963/ failed its functional tests - but the error is seems weird | |
| 22:25:28 | mordred | dtroyer: http://logs.openstack.org/63/538963/2/check/osc-functional-devstack/c475bfb/job-output.txt.gz#_2018-01-29_22_22_13_041554 | |
| 22:25:37 | mordred | dtroyer: does that make the senses to you? | |
| 22:35:19 | dtroyer | mordred: I've never seen that before… | |
| 22:37:41 | mordred | dtroyer: I just pushed up https://review.openstack.org/539032 Enable cinder services explicitly | |
| 22:37:48 | mordred | dtroyer: just to see if that helps | |
| 22:38:04 | mordred | dtroyer: as we seem to be getting http://logs.openstack.org/63/538963/2/check/osc-functional-devstack/c475bfb/job-output.txt.gz#_2018-01-29_22_20_41_586456 | |
| 22:38:39 | mordred | dtroyer: it does not make any sense to me why that would have stopped working though | |
| 22:42:21 | mordred | dtroyer: looking at git logs it doesn't seem like anything relevant would have changed since the last time that job worked : | |
| 22:42:23 | mordred | :( | |
| 23:07:55 | dtroyer | mordred: sorry, endless-phone-syndrome | |
| 23:08:17 | dtroyer | I don't recall ever seeing that level of Cinder issue, usually it's races with resources. this is different | |
| 23:10:21 | dtroyer | FWIW, I just +W 538958 | |
| 23:11:02 | mordred | woot | |
| #openstack-sdks - 2018-01-30 | |||
| 00:09:31 | adriant | mordred: as part of the sdk stuff you were going to potentially make some nice constructor functions for people using it without an underlying clouds.yaml | |
| 00:10:57 | adriant | there is something that sort of lets you do that, but it... still looks for underlying cloud.yaml unless you explicitly tell it not to or something. | |
| 00:11:07 | adriant | am trying to find where that doc was | |
| 00:16:40 | adriant | here: https://github.com/openstack/python-openstacksdk/blob/master/doc/source/user/connection.rst#getting-a-connection-from-python-arguments-avoiding-cloudsyaml | |
| 00:17:39 | adriant | you were talking about making that into a nicer callable constructor that does that, without all the pointless boiler plate | |
| 00:20:56 | mordred | adriant: yes indeed. I'm just now AFKing - but thanks for the reminder I can hit that first thign in the morning | |
| 00:21:14 | adriant | mordred: sweet, thanks :) | |
| 00:21:38 | adriant | and are they any docks on doing that type of thing and saying what versions of stuff you want? | |
| 00:22:09 | adriant | since sometimes you do want to use a specific version of a given services API when you setup something like that in code | |
| 00:22:13 | adriant | ... | |
| 00:22:18 | adriant | docs* not docks | |
| 00:23:50 | adriant | I'll be around tomorrow and can test/review as needed! | |
| 00:30:08 | mordred | adriant: the docs need to be cleaned up - the tl;dr is that you can pass args like service_type_api_version to get_one_cloud (or to Connection itself) ... or via env vars .. | |
| 00:30:53 | mordred | adriant: so - for instance, OS_COMPUTE_API_VERSION='2' or compute_api_version='2' | |
| 00:31:01 | adriant | ideally, I would love a constructor function that is basically: explicitly ignore clouds.yaml, envvars, etc, and ONLY use what I'm giving it | |
| 00:31:23 | mordred | adriant: nod - can do | |
| 00:31:54 | adriant | a consumer of that is 100% certain of exactly where the config data it gets is coming from | |
| 00:32:13 | adriant | which is what I'd want for using it in services type stuff | |
| 00:32:40 | adriant | and cool, if you can easily enough supply what versions of a given service you need that makes it ideal | |
| 00:39:44 | mordred | adriant: woot. for now it's BASICALLY "config = openstack.config.OpenStackConfig(load_yaml_config=False, envvar_prefix='GARBAGE').get_one(region_name='my-region', auth=dict(auth_url='http://foo.example.com', username='', password=''), compute_api_version='2', compute_interface='public') ; conn = openstack.connect(config=config)" | |
| 00:40:05 | mordred | adriant: but that's obviously ugly - so when I awaken I'll get you a better option :) | |
| 00:40:12 | openstackgerrit | Merged openstack/python-openstackclient master: Use find_ip from openstacksdk https://review.openstack.org/538958 | |
| 00:40:15 | adriant | mordred: tyvm good dir! | |
| 00:40:18 | adriant | ... sir* | |
| 00:40:28 | mordred | I'm more like a dir :) | |
| 00:41:10 | mordred | adriant: I also have another version-selection related patch I've been hacking on today that should be ready for you to check out by tomorrow as well | |
| 00:41:51 | adriant | very cool. I'm going to play with the SDK as is today and see what it's like to use and mostly test out the old pain points I had with it. | |
| 00:42:03 | adriant | and tomorrow I'll follow up and see where you're at with that stuff | |
| 01:41:56 | openstackgerrit | chenyb4 proposed openstack/python-openstacksdk master: Rename unit test cluster to clustering https://review.openstack.org/539071 | |
| 02:26:57 | openstackgerrit | OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902 | |
| 03:33:33 | openstackgerrit | Merged openstack/python-openstackclient master: Fix tox -e venv -- reno new |
|
| 05:01:50 | Qiming | modred | |
| 05:02:18 | Qiming | there? | |
| 05:19:12 | openstackgerrit | Merged openstack/python-openstacksdk master: Rename unit test cluster to clustering https://review.openstack.org/539071 | |
| 05:26:00 | adriant | mordred: the envvar_prefix doesn't work as intended. I'm hitting this when adding SDK support to my interpreter tool since I have a constructor function that takes elements from the cloud config as defaults, and lets you build a different connection object with a specified region and api versions. | |
| 05:26:24 | adriant | the error I'm getting is: OpenStackConfigException: Region RegionTwo is not a valid region name for cloud envvars. Valid choices are RegionOne. Please note that region names are case sensitive. | |
| 05:27:20 | adriant | when I do it in a clean env, no OS_* stuff set, it does work, but... when there are some set, I can actually build an entirely clean config/connection object entirely with values I supply. | |
| 05:28:30 | adriant | I can't* actually build | |
| 06:12:49 | adriant | mordred: code I was trying to get working that complains if OS_REGION_NAME is set: https://github.com/Adrian-Turjak/openstack-interpreter/pull/3/files#diff-f02c38fdd6ae5f99d7f313ce3277d81d | |
| 06:15:46 | adriant | also, if you know of how I can do something similar for the shade cloud object, that would be cool, because I would like to be able to build a shade cloud object with a constructor function that do pretty much the same thing I'm doing for my get_connection function with the SDK. | |
| 07:08:02 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: set paginated to False for list operations https://review.openstack.org/539115 | |
| 07:10:18 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: set paginated to False for list operations https://review.openstack.org/539115 | |
| 07:11:38 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: set paginated to False for list operations https://review.openstack.org/539115 | |
| 07:17:28 | openstackgerrit | Hunt Xu proposed openstack/python-openstacksdk master: orchestration: fix typo in doc https://review.openstack.org/539116 | |
| 09:53:48 | openstackgerrit | Deepak Mourya proposed openstack/keystoneauth master: Override support message in AuthorizationFailure class https://review.openstack.org/539166 | |
| 10:54:14 | openstackgerrit | Kairat Kushaev proposed openstack/keystoneauth master: use defusedxml for XML parsing https://review.openstack.org/536761 | |
| 13:57:11 | openstackgerrit | Merged openstack/service-types-authority master: Add .zuul.yaml file with tox job https://review.openstack.org/538305 | |
| 14:19:01 | openstackgerrit | Merged openstack/service-types-authority master: Add masakari and service-type instance-ha https://review.openstack.org/534875 | |
| 15:03:04 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add a descriptor object for each service proxy https://review.openstack.org/539232 | |
| 15:03:04 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Ensure Connection can be made from keyword arguments https://review.openstack.org/539233 | |
| 15:03:05 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Add OpenStackCloud object to Connection https://review.openstack.org/539234 | |
| 15:03:22 | mordred | adriant: ok. that ^^ should take care of all of your questions and use cases (I hope) | |
| 15:36:16 | openstackgerrit | Monty Taylor proposed openstack/service-types-authority master: Update build to put content into sphinx location https://review.openstack.org/539239 | |
| 15:37:26 | mordred | dtroyer: jeez. it seems like devstack on pike is just flat broken for us there does't it :( | |