Monday, 2018-07-09

*** gildub has joined #openstack-sdks00:09
openstackgerritMerged openstack/osc-lib master: Add cloud to warning about domain related config
Shrewsmordred: done00:30
*** alex_xu has quit IRC00:40
*** alex_xu has joined #openstack-sdks00:41
*** mhen has quit IRC00:46
*** mhen has joined #openstack-sdks00:51
*** mhen has quit IRC00:56
*** mhen has joined #openstack-sdks00:56
*** edmondsw has joined #openstack-sdks01:04
*** edmondsw has quit IRC01:08
*** flwang has quit IRC01:34
*** mhen has quit IRC01:56
*** mhen has joined #openstack-sdks01:59
*** annp has joined #openstack-sdks02:30
*** edmondsw has joined #openstack-sdks02:52
*** edmondsw has quit IRC02:57
mordredShrews: thanks!03:03
openstackgerritMonty Taylor proposed openstack/osc-lib master: Fix exception in log warning when no cloud config exists
*** bobh has joined #openstack-sdks03:34
mordreddtroyer, dhellmann: ^^ that patch is needed to unbreak the new osc-tips break that caused03:35
mordredonce 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 IRC03:56
*** annp has joined #openstack-sdks03:56
*** bobh has quit IRC04:01
openstackgerritMonty Taylor proposed openstack/osc-lib master: Pass prompt_for_password to OSC_Config constructor
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Remove local openstack.config overrides
mordreddtroyer: then, as followups to all of that - there's a couple of patches that get rid of one of the sets of OSC_Config stuff04:11
mordredonce 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 openstacksdk04:12
*** edmondsw has joined #openstack-sdks04:40
*** edmondsw has quit IRC04:44
*** gildub has quit IRC05:17
*** noama has joined #openstack-sdks05:39
*** yolanda has joined #openstack-sdks05:46
*** pooja_jadhav has joined #openstack-sdks05:46
*** Luzi has joined #openstack-sdks06:27
*** edmondsw has joined #openstack-sdks06:28
*** annp has quit IRC06:30
*** annp has joined #openstack-sdks06:30
*** edmondsw has quit IRC06:33
*** jpich has joined #openstack-sdks06:58
*** peereb has joined #openstack-sdks07:05
*** gtema has joined #openstack-sdks07:14
*** ttsiouts has joined #openstack-sdks07:23
*** d0ugal_ has quit IRC07:33
*** d0ugal has joined #openstack-sdks07:33
*** d0ugal has quit IRC07:33
*** d0ugal has joined #openstack-sdks07:33
*** ttsiouts has quit IRC07:40
*** noama is now known as Noam07:43
*** Noam is now known as noama07:43
*** noama has quit IRC07:44
*** noama has joined #openstack-sdks07:44
*** ttsiouts has joined #openstack-sdks07:46
*** zigo has quit IRC08:03
*** zigo has joined #openstack-sdks08:05
*** edmondsw has joined #openstack-sdks08:16
*** edmondsw has quit IRC08:21
*** gtema has quit IRC09:02
*** d0ugal has quit IRC09:17
*** d0ugal has joined #openstack-sdks09:23
openstackgerrittianhui proposed openstack/python-openstackclient master: Compute: Add tag support for server add port
*** finucannot is now known as stephenfin09:53
*** cdent has joined #openstack-sdks10:17
*** gtema has joined #openstack-sdks10:22
*** ttsiouts has quit IRC10:22
*** cdent has quit IRC10:41
*** gtema has quit IRC10:50
*** gtema has joined #openstack-sdks10:51
*** ttsiouts has joined #openstack-sdks11:02
*** gtema has quit IRC11:07
fricklermordred: 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-sdks11:10
mordredfrickler: I don't think we've made any yet11:19
fricklermordred: 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 cycle11:22
mordredfrickler: yeah. I think so :)11:23
*** ttsiouts has quit IRC11:35
*** ttsiouts has joined #openstack-sdks11:54
*** abishop has joined #openstack-sdks11:59
*** d0ugal has quit IRC12:06
*** d0ugal has joined #openstack-sdks12:09
*** cdent has joined #openstack-sdks12:14
*** thrash|g0ne is now known as thrash12:20
*** mriedem has joined #openstack-sdks12:23
*** edmondsw has joined #openstack-sdks12:24
*** edmondsw has quit IRC12:29
*** edmondsw has joined #openstack-sdks12:31
*** edmondsw has quit IRC12:35
*** edmondsw has joined #openstack-sdks12:37
fricklermordred: dtroyer: spotz found an interesting issue, please take a look:
openstackLaunchpad bug 1780587 in devstack "Creds files in accr directory return Identity API errors" [Medium,Confirmed] - Assigned to Dr. Jens Harbott (j-harbott)12:39
mordredfrickler: what are acccr files?12:39
fricklermordred: devstack generates these, similar in use to "openrc", but different files per account/project instead of having parameters.
mordredoh for the love of12:41
fricklerdidn't know they existed before, either12:41
mordredwell, it's still likely a real bug12:42
fricklermordred: bug in OSC/libs?12:42
mordredthere's an osc patch I reviewed over the weekend related to identity_api version12:42
*** edmondsw has quit IRC12:42
mordredthere is confusion as to what identity_api_version is supposed to do12:42
mordredthere 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 uses12:44
mordredthe 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 wrong12:44
mordredall that said- I have NO IDEA why that would have broken just now12:44
fricklermordred: yes, even setting auth_type=v3password doesn't solve the above issue12:45
mordredwe have not cut any releases of anything with the 'make osc-lib use sdk for session' code12:45
*** edmondsw has joined #openstack-sdks12:45
fricklermordred: 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 osc12:45
mordredmaybe so12:46
mordredif you set OS_IDENTITY_API_VERSION=3 does it work?12:46
fricklermordred: yes12:46
mordredk. that's where the bug is at least12:46
*** crunchengine has joined #openstack-sdks12:46
mordredopenstack.config/os-client-config set a default identity_api_version to 2 - I'm betting that is confusing things12:47
mordredwe're also getting very close to being able to remove that default value too12:47
mordredanywho - 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 case12:48
*** edmondsw has quit IRC12:49
*** edmondsw has joined #openstack-sdks12:51
*** edmondsw has quit IRC12:53
*** edmondsw has joined #openstack-sdks12:53
*** gtema has quit IRC13:09
*** ttsiouts has quit IRC13:16
*** ttsiouts has joined #openstack-sdks13:19
*** lbragstad has joined #openstack-sdks13:37
*** Luzi has quit IRC14:05
dhellmannmordred : +2 on
mordreddhellmann: tyvm14:12
dhellmanndid you need another one right away or do we want to wait for that one?14:12
dhellmannand how broken are things? should I single-approve that?14:13
mordrednah - 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
mordreddhellmann: so, if you're comfortable single-coring, that would be great, but also if you're not that's cool14:13
mordredif you're in the reviewing mood though, is eventually needed and is one line - but *totally* non-urgent14:15
dhellmannmordred : ok, I'll give dtroyer a little while to come online. If you need me to approve it later today I will14:16
dhellmannand +2 on the other one14:16
dhellmannoh, stand by, if the gate's broken I'm OK with approving it14:17
openstackgerritMonty Taylor proposed openstack/osc-lib master: Run OSC unit test tips jobs
dhellmannmordred : will we need a release to unbreak the gate?14:17
mordredof sdk - but one is already teed up14:17
mordredas soon as we land the osc-lib patch, I can recheck the sdk patch (which is already approved) and then we can cut a release14:18
dhellmanngot it14:18
dhellmannwe don't need an osc-lib release, too?14:18
mordredno - it's just the osc-tips jobs that are busted14:18
dhellmannaha, ok14:18
* dhellmann clearly needs more caffeine14:18
mordredwe 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
dhellmannmordred : 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
mordreddhellmann: awesome. maybe have another coffee as you drive for your errand14:20
dhellmanngood plan14:20
dtroyerwhat a great day to start working late!  :)14:30
dtroyera 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 plan14:31
dtroyerthe 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 away14:32
dtroyerthis all points out how stupidly fragile OSC is with this handling and why I'm ready for that subtle breakage and cleaning out the cruft14:33
*** tellesnobrega has joined #openstack-sdks14:36
*** ttsiouts has quit IRC14:42
mordreddtroyer: ++14:47
mordreddtroyer: 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/sdk14:48
*** kimamisa has joined #openstack-sdks15:02
*** peereb has quit IRC15:03
*** ttsiouts has joined #openstack-sdks15:07
*** cdent has quit IRC15:13
openstackgerritMerged openstack/osc-lib master: Add release note link in README
*** ttsiouts has quit IRC16:03
openstackgerritPavlo Shchelokovskyy proposed openstack/python-openstackclient master: Skip calls to glance and nova when got no servers
openstackgerritPavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --name-lookup option to server list
*** kimamisa has quit IRC16:20
*** bobh has joined #openstack-sdks16:38
*** bobh has quit IRC16:39
*** jpich has quit IRC16:52
*** noama has quit IRC17:14
*** cdent has joined #openstack-sdks17:16
mordreddtroyer, Shrews: I've gotta be offline for the next few hours - when lands, can one of y'all recheck ?17:16
Shrewsmordred: sure17:34
*** mriedem1 has joined #openstack-sdks18:19
*** mriedem has quit IRC18:21
*** mriedem1 is now known as mriedem18:22
openstackgerritMerged openstack/osc-lib master: Fix exception in log warning when no cloud config exists
Shrewsmordred: rechecked19:36
*** thrash is now known as thrash|g0ne20:12
*** crunchengine has quit IRC20:23
*** e0ne has joined #openstack-sdks20:36
*** walk has joined #openstack-sdks21:19
*** walk has quit IRC21:29
*** abishop has quit IRC21:29
*** e0ne has quit IRC21:31
openstackgerritMerged openstack/openstacksdk master: Add support for processing insecure
*** cdent has quit IRC22:47
openstackgerritFeilong Wang proposed openstack/openstacksdk master: Fix Magnum cluster update
*** yolanda_ has joined #openstack-sdks23:06
*** yolanda has quit IRC23:09
*** edmondsw has quit IRC23:26

Generated by 2.15.3 by Marius Gedminas - find it at!