| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-05-16 | |||
| 13:44:23 | mordred | frickler: I think it's a thing dtroyer has already started in on ... are you going to be in vancouver next week? | |
| 13:45:26 | frickler | nope | |
| 13:46:34 | mordred | darn. well - after I finish debugging this morning's sdk patch, lemme see if I can spot an easier path to fixing yourbug | |
| 13:47:24 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049 | |
| 13:47:24 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935 | |
| 15:16:35 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Collect timing information for API calls https://review.openstack.org/568877 | |
| 15:16:36 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add oslo.config option for split-loggers https://review.openstack.org/568878 | |
| 15:30:58 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Return the endpoint_override from EndpointData https://review.openstack.org/491947 | |
| 15:56:46 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Stop using pbr autodoc https://review.openstack.org/568889 | |
| 16:20:56 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Allow registering additional plugin loaders https://review.openstack.org/333126 | |
| 17:33:13 | openstackgerrit | Colleen Murphy proposed openstack/keystoneauth master: [DNM] See if latest pbr is causing our docs problems https://review.openstack.org/568909 | |
| 20:44:43 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Collect timing information for API calls https://review.openstack.org/568877 | |
| 20:44:44 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add oslo.config option for split-loggers https://review.openstack.org/568878 | |
| 20:45:07 | openstackgerrit | Merged openstack/keystoneauth master: Fix logging of encoded headers https://review.openstack.org/568365 | |
| 21:52:28 | openstackgerrit | Merged openstack/keystoneauth master: Return the endpoint_override from EndpointData https://review.openstack.org/491947 | |
| #openstack-sdks - 2018-05-17 | |||
| 09:58:54 | openstackgerrit | Stephen Breyer-Menke proposed openstack/openstacksdk master: Support add/remove user to/from group https://review.openstack.org/569080 | |
| 10:17:01 | openstackgerrit | Stephen Breyer-Menke proposed openstack/openstacksdk master: Support add/remove user to/from group https://review.openstack.org/569082 | |
| 12:04:33 | openstackgerrit | Stephen Breyer-Menke proposed openstack/openstacksdk master: Support add/remove user to/from group https://review.openstack.org/569097 | |
| 15:56:20 | gtema | mordred: what is the default python version for shade-ansible tests? Is it the one from system? In openstacksdk tox it is set by default for python2, but not in shade. There is an issue in Ansible (https://github.com/ansible/ansible/issues/39472), which makes keypair role fail for me with python 3.6 | |
| 16:02:26 | mordred | gtema: yeah - we're not setting it explicitly in the tox config. we should definitely fix that module with the fix in that issue ... do you have any interest in making a PR to ansible and I can approve it? (otherwise I can get it up there) | |
| 16:10:30 | gtema | mordred: making it currently. Will be in couple of minutes | |
| 16:18:32 | gtema | mordred: https://github.com/ansible/ansible/pull/40341 | |
| 16:32:45 | edleafe | elmiko: too late - the session was just announced: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130617.html | |
| 16:36:03 | elmiko | edleafe: thanks for the heads up, and no worries | |
| 16:36:21 | elmiko | i'm gonna see if i can just stay till thursday | |
| 16:42:52 | elmiko | cdent edleafe dtantsur, ready for review: https://etherpad.openstack.org/p/api-sig-newsletter | |
| 16:43:54 | elmiko | how did i drop bof from there... i had it in originally | |
| 16:44:54 | elmiko | ++ good additions | |
| 16:45:23 | edleafe | elmiko: done | |
| 16:45:42 | elmiko | thanks edleafe ! | |
| 16:47:56 | cdent | not much to add beyond what edleafe has said. what's there is fine, but if you want to make the second session sound more exciting you could order it a bit differently: it is _explicitly_ about how to drop old functionality | |
| 16:48:46 | cdent | elmiko: ^ | |
| 16:48:58 | edleafe | cdent: you should add wording to that effect then | |
| 16:49:15 | cdent | i'm not sure how to say it. thus bouncing the idea here | |
| 16:49:21 | elmiko | i changed it a little, see if that works | |
| 16:50:17 | dtantsur | "clarify what it means when we bump the minimum microversion" ambitiously :) | |
| 16:50:46 | dtantsur | otherwise LGTM | |
| 16:51:06 | elmiko | i like the reference point cdent | |
| 16:51:23 | elmiko | dtantsur: haha, totally | |
| 16:51:55 | elmiko | ooh, even more ambitious edleafe | |
| 16:52:03 | edleafe | asbestos jackets are required | |
| 16:52:10 | elmiko | hahaha, indeed! | |
| 16:52:30 | edleafe | I changed my color. Blue is cdent | |
| 16:52:36 | elmiko | oh, whoops | |
| 16:52:40 | edleafe | our colors were too close | |
| 16:52:40 | elmiko | then good show cdent | |
| 16:53:05 | cdent | I just took what I added from the session description. I'm done. | |
| 16:53:09 | elmiko | can we add (abestos jackets recommended) at the end? | |
| 16:53:45 | elmiko | thanks for the help all, gonna publish | |
| 16:53:51 | cdent | rumor has it all the firefighting will happen over beer beforehand | |
| 16:53:58 | elmiko | +1 | |
| 16:53:59 | edleafe | nah, let 'em figure it out themselves! | |
| 16:54:04 | cdent | so I phoned ahead to the vancouver breweries to prepare them for mordred and mugsie | |
| 16:54:08 | elmiko | edleafe: haha | |
| 16:54:16 | cdent | dtantsur: it was mordred's idea | |
| 16:54:23 | cdent | (partly) | |
| 16:54:50 | dtantsur | that's a plan: collect all people who want to bump the minimum version in one room aaaaand <invent your even ending> | |
| 16:55:01 | dtantsur | s/even/evil/ | |
| 16:55:07 | elmiko | sent | |
| 16:55:29 | mugsie | well, there is one person who wants to `rm -rf microversions` going to be in that room :P | |
| 16:56:18 | dtantsur | oh, I"m going to miss an epic battle :) | |
| 16:57:07 | edleafe | Will we all have little flags to show our allegiances? | |
| 16:57:27 | mugsie | that is a great idea :) | |
| 17:00:04 | dtantsur | I have love-hate relationships with microversions | |
| 17:08:32 | cdent | I have no idea what flag to fly. | |
| 17:22:44 | elmiko | cdent edleafe, ok, i'm good for wednesday too now =) | |
| 17:23:20 | cdent | like I said: | |
| 17:23:50 | elmiko | hahaha | |
| 17:25:13 | edleafe | how does that even work? | |
| 17:25:26 | edleafe | I change a flight, and it's several hundred $$ | |
| 17:25:50 | elmiko | they let me change for free, maybe status on delta? | |
| 17:32:29 | cdent | didn't you buy your flight late? In which case it would be changeable anyway? | |
| 17:32:44 | elmiko | good point, i was still in some sort of grace period | |
| 18:16:50 | mordred | dtantsur|afk: I'm all for bumping min microversions - I'm just arguing that doing so already has a name and a well-understood and supported system. it's called "bumping the major version" and we support it everywhere with no additional effort :) | |
| 18:53:46 | elmiko | mordred: i'm kinda curious about the effects on maintenance of keeping around old versions, like what if a team wants to move away from keeping the old code (assuming there is some rational reasoning behind it) | |
| 18:54:36 | elmiko | or am i misunderstanding this issue? | |
| 19:01:06 | cdent | elmiko: that's one of the main points of this discussion: making it possible to drop old versions | |
| 19:01:25 | cdent | but doing it in concert | |
| 19:05:09 | elmiko | yeah | |
| 19:05:30 | elmiko | i just really appreciate mordred's take on the idea of bump max keep min, i'm curious to hear more about his thoughts | |
| 19:05:44 | elmiko | i'm also trying to confirm that i understood this XD | |
| 19:06:32 | elmiko | from a compatability standpoint i totally get it, i'm just concerned about the developer response to keeping old stuff | |
| 21:06:52 | openstackgerrit | Merged openstack/keystoneauth master: Collect timing information for API calls https://review.openstack.org/568877 | |
| 21:06:53 | openstackgerrit | Merged openstack/keystoneauth master: Add oslo.config option for split-loggers https://review.openstack.org/568878 | |
| 21:06:55 | openstackgerrit | Merged openstack/keystoneauth master: Expose version_between as a real function https://review.openstack.org/568640 | |
| 22:39:14 | dmsimard|off | mordred, Shrews: does that ring you a bell ? http://paste.openstack.org/show/721195/ | |
| 22:39:29 | dmsimard|off | when trying to use the openstack dynamic inventory | |
| 22:39:39 | dmsimard|off | fresh install, afaict shade, os_client_config are installed etc | |
| 22:41:10 | dmsimard|off | looks like https://github.com/openstack/os-client-config/commit/5c2eae8f8242332aff59d18b45a7d17f01524aaa | |
| 23:28:23 | openstackgerrit | Alexander D. proposed openstack/os-client-config master: fix saltstack breakage introduced in v1.31.0 https://review.openstack.org/569291 | |
| #openstack-sdks - 2018-05-18 | |||
| 02:06:40 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | |
| 02:12:59 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | |
| 02:31:48 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server Part 1 https://review.openstack.org/568549 | |
| 09:21:29 | dtantsur | mordred: well, I'd argue with the "no additional effort" bit.. at least for me as rust-openstack developer supporting major versions is a bit of a pain | |
| 10:12:24 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375 | |
| 11:00:35 | openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server Part 2 https://review.openstack.org/569385 | |
| 11:00:37 | openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | |
| 11:15:12 | frickler | dmsimard|off: do you use the default contrib/inventory/openstack.py from the ansible repo or do you have a modified version? I had a similar issue with the former that was resolved in https://github.com/openstack/os-client-config/commit/e65033910714d6f7d198d8eff6997fdaec102474 | |
| 11:57:49 | mordred | dtantsur: that's fair. I guess what I should have said is that since major versions already exist, clients that want to support the existing openstack installations have to have major version discovery / handling code today anyway - and we have documentation about how to handle all of the cases of it | |
| 11:58:36 | mordred | dmsimard|off, frickler: I think in this case it's an issue with executing the openstack dynamic inventory from a directory containing the inventory with the inventory script still named 'openstack.py' | |