| *** gildub has joined #openstack-sdks | 00:09 | |
| openstackgerrit | Merged openstack/osc-lib master: Add cloud to warning about domain related config https://review.openstack.org/577324 | 00:18 |
|---|---|---|
| Shrews | mordred: done | 00:30 |
| *** alex_xu has quit IRC | 00:40 | |
| *** alex_xu has joined #openstack-sdks | 00:41 | |
| *** mhen has quit IRC | 00:46 | |
| *** mhen has joined #openstack-sdks | 00:51 | |
| *** mhen has quit IRC | 00:56 | |
| *** mhen has joined #openstack-sdks | 00:56 | |
| *** edmondsw has joined #openstack-sdks | 01:04 | |
| *** edmondsw has quit IRC | 01:08 | |
| *** flwang has quit IRC | 01:34 | |
| *** mhen has quit IRC | 01:56 | |
| *** mhen has joined #openstack-sdks | 01:59 | |
| *** annp has joined #openstack-sdks | 02:30 | |
| *** edmondsw has joined #openstack-sdks | 02:52 | |
| *** edmondsw has quit IRC | 02:57 | |
| mordred | Shrews: thanks! | 03:03 |
| openstackgerrit | Monty Taylor proposed openstack/osc-lib master: Fix exception in log warning when no cloud config exists https://review.openstack.org/580898 | 03:34 |
| *** bobh has joined #openstack-sdks | 03:34 | |
| mordred | dtroyer, dhellmann: ^^ that patch is needed to unbreak the new osc-tips break that https://review.openstack.org/577324 caused | 03:35 |
| mordred | once that lands, we can land the openstacksdk fix for the other osc-tips break, then release sdk, then we can land the patch to add osc-tips jobs to osc-lib so that we don't keep breaking osc's unittests with osc-lib patches :) | 03:36 |
| *** annp has quit IRC | 03:56 | |
| *** annp has joined #openstack-sdks | 03:56 | |
| *** bobh has quit IRC | 04:01 | |
| openstackgerrit | Monty Taylor proposed openstack/osc-lib master: Pass prompt_for_password to OSC_Config constructor https://review.openstack.org/580900 | 04:09 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Remove local openstack.config overrides https://review.openstack.org/580901 | 04:10 |
| mordred | dtroyer: then, as followups to all of that - there's a couple of patches that get rid of one of the sets of OSC_Config stuff | 04:11 |
| mordred | once these bits are in and unwound, I'll start piecing apart the stuff in osc-lib OSC_Config and figuring out what needs to get added to openstacksdk | 04:12 |
| *** edmondsw has joined #openstack-sdks | 04:40 | |
| *** edmondsw has quit IRC | 04:44 | |
| *** gildub has quit IRC | 05:17 | |
| *** noama has joined #openstack-sdks | 05:39 | |
| *** yolanda has joined #openstack-sdks | 05:46 | |
| *** pooja_jadhav has joined #openstack-sdks | 05:46 | |
| *** Luzi has joined #openstack-sdks | 06:27 | |
| *** edmondsw has joined #openstack-sdks | 06:28 | |
| *** annp has quit IRC | 06:30 | |
| *** annp has joined #openstack-sdks | 06:30 | |
| *** edmondsw has quit IRC | 06:33 | |
| *** jpich has joined #openstack-sdks | 06:58 | |
| *** peereb has joined #openstack-sdks | 07:05 | |
| *** gtema has joined #openstack-sdks | 07:14 | |
| *** ttsiouts has joined #openstack-sdks | 07:23 | |
| *** d0ugal_ has quit IRC | 07:33 | |
| *** d0ugal has joined #openstack-sdks | 07:33 | |
| *** d0ugal has quit IRC | 07:33 | |
| *** d0ugal has joined #openstack-sdks | 07:33 | |
| *** ttsiouts has quit IRC | 07:40 | |
| *** noama is now known as Noam | 07:43 | |
| *** Noam is now known as noama | 07:43 | |
| *** noama has quit IRC | 07:44 | |
| *** noama has joined #openstack-sdks | 07:44 | |
| *** ttsiouts has joined #openstack-sdks | 07:46 | |
| *** zigo has quit IRC | 08:03 | |
| *** zigo has joined #openstack-sdks | 08:05 | |
| *** edmondsw has joined #openstack-sdks | 08:16 | |
| *** edmondsw has quit IRC | 08:21 | |
| *** gtema has quit IRC | 09:02 | |
| *** d0ugal has quit IRC | 09:17 | |
| *** d0ugal has joined #openstack-sdks | 09:23 | |
| openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tag support for server add port https://review.openstack.org/580977 | 09:35 |
| *** finucannot is now known as stephenfin | 09:53 | |
| *** cdent has joined #openstack-sdks | 10:17 | |
| *** gtema has joined #openstack-sdks | 10:22 | |
| *** ttsiouts has quit IRC | 10:22 | |
| *** cdent has quit IRC | 10:41 | |
| *** gtema has quit IRC | 10:50 | |
| *** gtema has joined #openstack-sdks | 10:51 | |
| *** ttsiouts has joined #openstack-sdks | 11:02 | |
| *** gtema has quit IRC | 11:07 | |
| frickler | mordred: dtroyer: someone on #openstack just reported issues running with python3.7, how do our plans to support that look like? | 11:10 |
| *** gtema has joined #openstack-sdks | 11:10 | |
| mordred | frickler: I don't think we've made any yet | 11:19 |
| frickler | mordred: ah, I just noticed that it's not even two weeks old, I was thinking it had been a couple of months. but then, yeah, maybe next cycle | 11:22 |
| mordred | frickler: yeah. I think so :) | 11:23 |
| *** ttsiouts has quit IRC | 11:35 | |
| *** ttsiouts has joined #openstack-sdks | 11:54 | |
| *** abishop has joined #openstack-sdks | 11:59 | |
| *** d0ugal has quit IRC | 12:06 | |
| *** d0ugal has joined #openstack-sdks | 12:09 | |
| *** cdent has joined #openstack-sdks | 12:14 | |
| *** thrash|g0ne is now known as thrash | 12:20 | |
| *** mriedem has joined #openstack-sdks | 12:23 | |
| *** edmondsw has joined #openstack-sdks | 12:24 | |
| *** edmondsw has quit IRC | 12:29 | |
| *** edmondsw has joined #openstack-sdks | 12:31 | |
| *** edmondsw has quit IRC | 12:35 | |
| *** edmondsw has joined #openstack-sdks | 12:37 | |
| frickler | mordred: dtroyer: spotz found an interesting issue, please take a look: https://bugs.launchpad.net/devstack/+bug/1780587 | 12:39 |
| openstack | Launchpad bug 1780587 in devstack "Creds files in accr directory return Identity API errors" [Medium,Confirmed] - Assigned to Dr. Jens Harbott (j-harbott) | 12:39 |
| mordred | frickler: what are acccr files? | 12:39 |
| frickler | mordred: devstack generates these, similar in use to "openrc", but different files per account/project instead of having parameters. http://paste.openstack.org/show/725342/ | 12:41 |
| mordred | oh for the love of | 12:41 |
| frickler | didn't know they existed before, either | 12:41 |
| mordred | well, it's still likely a real bug | 12:42 |
| frickler | mordred: bug in OSC/libs? | 12:42 |
| mordred | there's an osc patch I reviewed over the weekend related to identity_api version | 12:42 |
| *** edmondsw has quit IRC | 12:42 | |
| mordred | yeah | 12:42 |
| mordred | somewhere | 12:42 |
| mordred | there is confusion as to what identity_api_version is supposed to do | 12:42 |
| mordred | there is code in osc that uses it to set some defaults for authentication - which I think we're just going to have to adopt in openstacksdk even though identity_api_versin is _supposed_ to be a setting controlling which version of the CRUD api to use - and auth_type is _supposed_ to control which version of auth one uses | 12:44 |
| mordred | the thing is - literally nobody understands that - so I think the osc attempt at inferring intent from identity_api_version is better than telling people they're wrong | 12:44 |
| mordred | all that said- I have NO IDEA why that would have broken just now | 12:44 |
| frickler | mordred: yes, even setting auth_type=v3password doesn't solve the above issue | 12:45 |
| mordred | we have not cut any releases of anything with the 'make osc-lib use sdk for session' code | 12:45 |
| *** edmondsw has joined #openstack-sdks | 12:45 | |
| frickler | mordred: I tried with various older lib versions, none of them helped. so maybe spotz was confused by other clients working, and it was always broken for osc | 12:45 |
| mordred | maybe so | 12:46 |
| mordred | if you set OS_IDENTITY_API_VERSION=3 does it work? | 12:46 |
| frickler | mordred: yes | 12:46 |
| mordred | k. that's where the bug is at least | 12:46 |
| *** crunchengine has joined #openstack-sdks | 12:46 | |
| mordred | openstack.config/os-client-config set a default identity_api_version to 2 - I'm betting that is confusing things | 12:47 |
| mordred | we're also getting very close to being able to remove that default value too | 12:47 |
| mordred | anywho - as a bugfix for devstack, adding OS_IDENTITY_API_VERSION=3 to those files will likely fix the specific bug - but we should remember this use case | 12:48 |
| *** edmondsw has quit IRC | 12:49 | |
| *** edmondsw has joined #openstack-sdks | 12:51 | |
| *** edmondsw has quit IRC | 12:53 | |
| *** edmondsw has joined #openstack-sdks | 12:53 | |
| *** gtema has quit IRC | 13:09 | |
| *** ttsiouts has quit IRC | 13:16 | |
| *** ttsiouts has joined #openstack-sdks | 13:19 | |
| *** lbragstad has joined #openstack-sdks | 13:37 | |
| *** Luzi has quit IRC | 14:05 | |
| dhellmann | mordred : +2 on https://review.openstack.org/580898 | 14:12 |
| mordred | yay! | 14:12 |
| mordred | dhellmann: tyvm | 14:12 |
| dhellmann | did you need another one right away or do we want to wait for that one? | 14:12 |
| dhellmann | and how broken are things? should I single-approve that? | 14:13 |
| mordred | nah - that's the only one - and the openstacksdk gate is broken without it, which is blocking landing the patch to unbreak the osc gate :) | 14:13 |
| mordred | dhellmann: so, if you're comfortable single-coring, that would be great, but also if you're not that's cool | 14:13 |
| mordred | if you're in the reviewing mood though, https://review.openstack.org/#/c/580900 is eventually needed and is one line - but *totally* non-urgent | 14:15 |
| dhellmann | mordred : ok, I'll give dtroyer a little while to come online. If you need me to approve it later today I will | 14:16 |
| dhellmann | and +2 on the other one | 14:16 |
| dhellmann | oh, stand by, if the gate's broken I'm OK with approving it | 14:17 |
| openstackgerrit | Monty Taylor proposed openstack/osc-lib master: Run OSC unit test tips jobs https://review.openstack.org/580875 | 14:17 |
| dhellmann | mordred : will we need a release to unbreak the gate? | 14:17 |
| mordred | of sdk - but one is already teed up | 14:17 |
| dhellmann | ok | 14:17 |
| mordred | as soon as we land the osc-lib patch, I can recheck the sdk patch (which is already approved) and then we can cut a release | 14:18 |
| dhellmann | got it | 14:18 |
| mordred | yay! | 14:18 |
| dhellmann | we don't need an osc-lib release, too? | 14:18 |
| mordred | no - it's just the osc-tips jobs that are busted | 14:18 |
| dhellmann | aha, ok | 14:18 |
| * dhellmann clearly needs more caffeine | 14:18 | |
| mordred | we actually don't need the sdk release for fixing gate - we just were going to do one anyway, so getting in the patch that will be needed before we can release osc or osc-lib seemed like a good idea :) | 14:19 |
| dhellmann | mordred : I'm going to run an errand (because driving while confused is less dangerous than coding). I'll watch for the release request to be updated and ready to go when I get back. | 14:19 |
| mordred | dhellmann: awesome. maybe have another coffee as you drive for your errand | 14:20 |
| dhellmann | good plan | 14:20 |
| dtroyer | what a great day to start working late! :) | 14:30 |
| dtroyer | a lot of this stuff is the sort of stuff I was hoping to have just "go away" with OSC 4 and some subtle breakage in how these global options worked… so much for that plan | 14:31 |
| dtroyer | the devstack accrc stuff was added by someone (I don't recall) for a specific purpose, if it's bit-rotted I wouldn't be sad to see it go away | 14:32 |
| dtroyer | this all points out how stupidly fragile OSC is with this handling and why I'm ready for that subtle breakage and cleaning out the cruft | 14:33 |
| *** tellesnobrega has joined #openstack-sdks | 14:36 | |
| *** ttsiouts has quit IRC | 14:42 | |
| mordred | dtroyer: ++ | 14:47 |
| mordred | dtroyer: that said - I think I've become convinced that the identity_api_version introspection that osc does that occ/sdk doesn't do is more user friendly and we should add it to occ/sdk | 14:48 |
| *** kimamisa has joined #openstack-sdks | 15:02 | |
| *** peereb has quit IRC | 15:03 | |
| *** ttsiouts has joined #openstack-sdks | 15:07 | |
| *** cdent has quit IRC | 15:13 | |
| openstackgerrit | Merged openstack/osc-lib master: Add release note link in README https://review.openstack.org/578459 | 15:27 |
| *** ttsiouts has quit IRC | 16:03 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Skip calls to glance and nova when got no servers https://review.openstack.org/568344 | 16:12 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --name-lookup option to server list https://review.openstack.org/568345 | 16:12 |
| *** kimamisa has quit IRC | 16:20 | |
| *** bobh has joined #openstack-sdks | 16:38 | |
| *** bobh has quit IRC | 16:39 | |
| *** jpich has quit IRC | 16:52 | |
| *** noama has quit IRC | 17:14 | |
| *** cdent has joined #openstack-sdks | 17:16 | |
| mordred | dtroyer, Shrews: I've gotta be offline for the next few hours - when https://review.openstack.org/#/c/580898/ lands, can one of y'all recheck https://review.openstack.org/#/c/580795/ ? | 17:16 |
| Shrews | mordred: sure | 17:34 |
| *** mriedem1 has joined #openstack-sdks | 18:19 | |
| *** mriedem has quit IRC | 18:21 | |
| *** mriedem1 is now known as mriedem | 18:22 | |
| openstackgerrit | Merged openstack/osc-lib master: Fix exception in log warning when no cloud config exists https://review.openstack.org/580898 | 19:25 |
| Shrews | mordred: rechecked | 19:36 |
| *** thrash is now known as thrash|g0ne | 20:12 | |
| *** crunchengine has quit IRC | 20:23 | |
| *** e0ne has joined #openstack-sdks | 20:36 | |
| *** walk has joined #openstack-sdks | 21:19 | |
| *** walk has quit IRC | 21:29 | |
| *** abishop has quit IRC | 21:29 | |
| *** e0ne has quit IRC | 21:31 | |
| openstackgerrit | Merged openstack/openstacksdk master: Add support for processing insecure https://review.openstack.org/580795 | 22:22 |
| *** cdent has quit IRC | 22:47 | |
| openstackgerrit | Feilong Wang proposed openstack/openstacksdk master: Fix Magnum cluster update https://review.openstack.org/581157 | 23:01 |
| *** yolanda_ has joined #openstack-sdks | 23:06 | |
| *** yolanda has quit IRC | 23:09 | |
| *** edmondsw has quit IRC | 23:26 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!