*** Qiming has quit IRC | 00:17 | |
*** stevemar has joined #openstack-sdks | 00:18 | |
*** hitalia has quit IRC | 00:36 | |
*** david-lyle has quit IRC | 00:40 | |
*** david-lyle has joined #openstack-sdks | 00:44 | |
*** stevemar has quit IRC | 00:55 | |
*** stevemar has joined #openstack-sdks | 00:55 | |
*** stevemar has quit IRC | 00:58 | |
*** trown is now known as trown|outttypeww | 01:05 | |
*** Qiming has joined #openstack-sdks | 01:21 | |
*** pratikmallya has quit IRC | 01:24 | |
*** Yanyanhu has joined #openstack-sdks | 01:32 | |
*** stevemar has joined #openstack-sdks | 01:42 | |
*** pratikmallya has joined #openstack-sdks | 02:13 | |
*** pratikmallya has quit IRC | 03:14 | |
*** gildub has quit IRC | 03:57 | |
*** gildub has joined #openstack-sdks | 04:17 | |
openstackgerrit | Steve Martinelli proposed openstack/keystoneauth: add openid connect plugins https://review.openstack.org/224993 | 04:19 |
---|---|---|
openstackgerrit | Steve Martinelli proposed openstack/keystoneauth: add openid connect plugins https://review.openstack.org/224993 | 05:14 |
*** openstackgerrit has quit IRC | 05:31 | |
*** Qiming has quit IRC | 05:31 | |
*** openstackgerrit has joined #openstack-sdks | 05:32 | |
openstackgerrit | Steve Martinelli proposed openstack/keystoneauth: add openid connect plugins https://review.openstack.org/224993 | 05:44 |
*** stevemar has quit IRC | 06:09 | |
*** aufi has joined #openstack-sdks | 06:10 | |
*** stevemar has joined #openstack-sdks | 06:11 | |
*** e0ne has joined #openstack-sdks | 06:42 | |
*** e0ne has quit IRC | 06:45 | |
*** e0ne has joined #openstack-sdks | 06:48 | |
*** e0ne has quit IRC | 06:55 | |
*** pratikmallya has joined #openstack-sdks | 06:57 | |
*** e0ne has joined #openstack-sdks | 06:59 | |
*** karimb has joined #openstack-sdks | 07:02 | |
*** woodster_ has quit IRC | 07:09 | |
*** e0ne has quit IRC | 07:10 | |
*** HenryG has quit IRC | 07:21 | |
*** karmatronic has joined #openstack-sdks | 07:34 | |
*** e0ne has joined #openstack-sdks | 07:37 | |
*** karimb has quit IRC | 07:38 | |
*** e0ne has quit IRC | 07:44 | |
*** stevemar has quit IRC | 07:49 | |
*** stevemar has joined #openstack-sdks | 07:50 | |
*** stevemar has quit IRC | 07:52 | |
*** karmatronic has quit IRC | 07:57 | |
*** gildub has quit IRC | 07:58 | |
*** karimb has joined #openstack-sdks | 07:58 | |
*** HenryG has joined #openstack-sdks | 08:12 | |
*** HenryG has quit IRC | 08:16 | |
*** HenryG has joined #openstack-sdks | 08:30 | |
*** chlong has quit IRC | 08:47 | |
*** stevemar has joined #openstack-sdks | 08:51 | |
*** stevemar has quit IRC | 09:05 | |
*** e0ne has joined #openstack-sdks | 09:27 | |
openstackgerrit | Niall Bunting proposed openstack/python-openstackclient: Add 'image create' to openstack https://review.openstack.org/203280 | 09:35 |
*** Yanyanhu has quit IRC | 09:38 | |
*** gildub has joined #openstack-sdks | 09:46 | |
*** karimb has quit IRC | 10:53 | |
*** karimb has joined #openstack-sdks | 10:54 | |
*** e0ne has quit IRC | 11:10 | |
*** thrash|g0ne is now known as thrash | 11:16 | |
*** e0ne has joined #openstack-sdks | 11:33 | |
*** stevemar has joined #openstack-sdks | 11:33 | |
*** stevemar has quit IRC | 11:37 | |
*** trown|outttypeww is now known as trown | 12:01 | |
*** chlong has joined #openstack-sdks | 12:06 | |
*** chlong has quit IRC | 12:07 | |
*** chlong has joined #openstack-sdks | 12:07 | |
*** pratikmallya has quit IRC | 12:13 | |
*** pratikmallya has joined #openstack-sdks | 12:16 | |
*** pratikmallya has quit IRC | 12:47 | |
*** fifieldt has quit IRC | 13:00 | |
*** gildub has quit IRC | 13:10 | |
*** edleafe is now known as figleaf | 13:15 | |
*** woodster_ has joined #openstack-sdks | 13:32 | |
*** stevemar has joined #openstack-sdks | 13:34 | |
*** stevemar has quit IRC | 13:37 | |
*** openstackgerrit has quit IRC | 13:46 | |
*** openstackgerrit has joined #openstack-sdks | 13:47 | |
*** jose4183 has joined #openstack-sdks | 14:02 | |
*** karimb has quit IRC | 14:11 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:14 | |
*** Qiming has joined #openstack-sdks | 14:30 | |
briancurtin | terrylhowe: fyi, I’m still working through reviews but sidetracked a bit to read through ksa so i have a good understanding of your change to have us use that. also, i’m out tues-fri next week, so will try to knock as much out today and monday as i can | 14:30 |
*** david-lyle has quit IRC | 14:31 | |
terrylhowe | awesome briancurtin I won’t get around to fixing the ksa patch until next week | 14:34 |
terrylhowe | but I think the only thing wrong with it is some broken tests | 14:34 |
*** stevemar has joined #openstack-sdks | 14:39 | |
openstackgerrit | Niall Bunting proposed openstack/python-openstackclient: Add 'image create' to openstack https://review.openstack.org/203280 | 14:54 |
*** zz_steveortiz is now known as steveortiz | 14:59 | |
openstackgerrit | Niall Bunting proposed openstack/python-openstackclient: Image fix bug with --volume https://review.openstack.org/225195 | 15:25 |
openstackgerrit | Niall Bunting proposed openstack/python-openstackclient: Image fix bug with --volume https://review.openstack.org/225195 | 15:31 |
*** aufi has quit IRC | 15:32 | |
*** jose4183 has quit IRC | 15:59 | |
*** jose4183 has joined #openstack-sdks | 15:59 | |
*** trown is now known as trown|lunch | 16:04 | |
*** e0ne has quit IRC | 16:05 | |
*** Qiming has quit IRC | 16:07 | |
*** pratikmallya has joined #openstack-sdks | 16:09 | |
openstackgerrit | Monty Taylor proposed openstack/os-client-config: Move plugin loader creation to try block https://review.openstack.org/225217 | 16:20 |
openstackgerrit | Monty Taylor proposed openstack/os-client-config: Handle the 'insecure' option that osc uses https://review.openstack.org/225218 | 16:20 |
openstackgerrit | Monty Taylor proposed openstack/os-client-config: Handle default_domain as a domain option https://review.openstack.org/225219 | 16:20 |
mordred | terrylhowe, dtroyer, GheRivero: ^^ https://review.openstack.org/225217 is the problem that was breaking osc | 16:21 |
mordred | Shrews: ^^ you too | 16:21 |
dtroyer | so I would _really_ like to not have to do things like that insecure bit, but I still don't quite understand what broke it in the first place | 16:22 |
GheRivero | got it | 16:22 |
*** hitalia has joined #openstack-sdks | 16:27 | |
*** pratikmallya has quit IRC | 16:34 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/keystoneauth: Updated from global requirements https://review.openstack.org/225224 | 16:38 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/224681 | 16:42 |
openstackgerrit | Merged openstack/os-client-config: Move plugin loader creation to try block https://review.openstack.org/225217 | 16:46 |
mordred | dtroyer: oh - the insecure bit is totally not needed | 16:47 |
mordred | dtroyer: anyting other than the first patch is just me finding things osc was doing that occ wasn't doing for other places but that one could imagine wanting to work in all of your things, not just osc | 16:48 |
mordred | (like default_domain too... I'd want ansible fokls who are using osc to have that do what they think it would do) | 16:48 |
mordred | dtroyer: but the "try block" patch is the thing that broke - it should just return None for an auth plugin if it doesn't work - I missed that the line that got moved was doing Stuff that Mattered | 16:49 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Pass validate=False to get_one_cloud https://review.openstack.org/224451 | 16:49 |
mordred | dtroyer: moving it to the try block keeps the osc interaction with the system like it was before .. or it does when combined with ... ^^ that | 16:50 |
mordred | that didn't work before because it was hitting a different error. with the occ patch and that patch above, osc has no new behavior and operates as before | 16:50 |
*** pratikmallya has joined #openstack-sdks | 16:56 | |
dtroyer | osc unit tests are still failing in a —insecure cli test that doesn't happen with occ 1.6.4 | 16:59 |
dtroyer | that's what I can' tfigure out why | 16:59 |
dtroyer | I suppose I need to let go of occ having more knowledge of things like ksa and other arg conventions and let it all live there | 16:59 |
*** pratikmallya has quit IRC | 16:59 | |
dtroyer | also, OSC | 17:00 |
openstackgerrit | Monty Taylor proposed openstack/os-client-config: Handle the 'insecure' option that osc uses https://review.openstack.org/225218 | 17:00 |
dtroyer | 's need for the osc_password auth plugin is probably near an end | 17:00 |
openstackgerrit | Monty Taylor proposed openstack/os-client-config: Handle default_domain as a domain option https://review.openstack.org/225219 | 17:00 |
dtroyer | we probably should make it not-the-default | 17:00 |
mordred | dtroyer: ++ | 17:01 |
mordred | dtroyer: I've got a crazy patch for you that I wrote on the plane | 17:01 |
dtroyer | + to plane patches! | 17:01 |
mordred | dtroyer: when I didn't have internet and nothing else to hack on :) | 17:01 |
dtroyer | btw, that's where OSC was born ;) | 17:01 |
mordred | hehe | 17:01 |
dtroyer | after talking it over with dolphm in a cab to SFO | 17:02 |
mordred | dtroyer: the osc unittests worked for me with that patch - I'll try again here in just a bit (need to find lunch for face) | 17:03 |
dtroyer | I'm about to blow away all of the venvs and try it again now too | 17:03 |
*** trown|lunch is now known as trown | 17:04 | |
*** e0ne has joined #openstack-sdks | 17:26 | |
stevemar | mordred: dtroyer i look forward to seeing crazy patches | 17:36 |
crinkle | we're seeing openstackclient default to using the glance v2 api (http://logs.openstack.org/74/224974/8/check/gate-puppet-openstack-integration-dsvm-centos7/0522764/console.html#_2015-09-18_14_33_01_115), which is weird because it looks like it should still be using v1 https://github.com/openstack/python-openstackclient/blob/master/openstackclient/image/client.py#L23 | 17:40 |
crinkle | looks like you all ran into this too since your func tests work around it http://git.openstack.org/cgit/openstack/python-openstackclient/commit/?id=817ab3ec0ea5a74aeda12850f39a56f332e9558b | 17:40 |
crinkle | why is that happening? | 17:40 |
*** e0ne has quit IRC | 17:41 | |
stevemar | crinkle: devstack sets it to v2 by default now | 17:41 |
crinkle | stevemar: we're not using devstack | 17:41 |
stevemar | crinkle: actually, now that i look into it, its not the case | 17:44 |
stevemar | which you've just determined | 17:44 |
crinkle | \o/ yay i'm helping | 17:45 |
stevemar | hehe | 17:48 |
stevemar | i have no idea why it switched | 17:48 |
*** e0ne has joined #openstack-sdks | 17:49 | |
*** boris-42 has quit IRC | 17:50 | |
*** hitalia has quit IRC | 17:54 | |
*** pratikmallya has joined #openstack-sdks | 18:22 | |
stevemar | mordred: i'm being lazy here, does the small patch you tossed up to occ fix osc's issue? | 18:29 |
*** e0ne has quit IRC | 18:39 | |
mordred | stevemar: yes | 18:39 |
mordred | at least | 18:39 |
mordred | it does on my laptop | 18:39 |
mordred | stevemar: let me test clean from scratch again | 18:39 |
stevemar | mordred: alright, so theres no giant / crazy patch? | 18:39 |
mordred | stevemar: you need the occ patch and then the osc patch | 18:39 |
mordred | stevemar: oh, there is | 18:39 |
mordred | that's not pushed up yet | 18:39 |
mordred | because it doesn't yet work | 18:39 |
stevemar | but thats not needed for this fix? | 18:39 |
stevemar | mordred: alright, so we probably need another occ release then, s'all good | 18:43 |
mordred | stevemar: yes. will need a new occ release | 18:43 |
mordred | stevemar: I just pushed a 1.7.2 tag | 18:56 |
mordred | wiht the occ fix | 18:56 |
mordred | stevemar: which should make testing the osc patch easier | 18:56 |
*** e0ne has joined #openstack-sdks | 18:58 | |
dtroyer | I'll try again, but about 30 min ago osc still wasn't fixed with osc-password as the default | 19:04 |
dtroyer | the —insecure arg test is still failing for me and I can't for the life of me figure out what changed in occ commit 093d7b085 to cause that… I think I have Friday Brain | 19:11 |
mordred | dtroyer: I only made the tag 10 minutes ago - so it's worth trying | 19:17 |
*** e0ne has quit IRC | 19:17 | |
mordred | dtroyer: I'm retrying from a clean tree now | 19:17 |
dtroyer | no change for me, still 6 failed tests, 5 of which are due to the plugin problem | 19:17 |
mordred | py27: commands succeeded | 19:17 |
mordred | dtroyer: ^^ | 19:17 |
mordred | dtroyer: did you pull the osc change? | 19:17 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Set default auth plugin back to 'password' https://review.openstack.org/225284 | 19:18 |
mordred | dtroyer: https://review.openstack.org/#/c/224451/ | 19:18 |
dtroyer | no, I don't necessarily like that one… why bother loading the plugins if we're not going to use them to validate? | 19:18 |
mordred | dtroyer: validate in this case means "use keystoneauth to validate them" | 19:18 |
mordred | dtroyer: we're not using keystoneauth in osc yet | 19:19 |
dtroyer | but occ does so we have to adapt, or cap back to 1.6.4 | 19:19 |
mordred | dtroyer: so asking occ to create and validate the plugins is a bit much | 19:19 |
mordred | dtroyer: it doesn't | 19:19 |
mordred | dtroyer: not if you pass validate=False | 19:19 |
mordred | validate=False is basically the pre-ksa behavior | 19:20 |
dtroyer | ok. I'm going to give up trying to keep the layers separate here anymore. | 19:21 |
mordred | dtroyer: if we want to go the other direction - it's a bit more involved - that's what my crazy plane patch is working on - an actual ksa use in osc | 19:22 |
mordred | it snowballs though - so I think after I get it working I need to go backtataking what I learned and break it in to smaller patches | 19:22 |
mordred | wow. backtataking | 19:22 |
mordred | that's a word | 19:22 |
*** pratikmallya has quit IRC | 19:23 | |
*** pratikmallya has joined #openstack-sdks | 19:23 | |
dtroyer | as long as the semantics and spelling of the options don't change, whatever. I hate ksa doing cli ard handling, don't think occ should know the semantics of those args, but I'm the only one and my brain isn't fully here… I think that's a clue | 19:23 |
dtroyer | just so OSC doesn't break compatibility | 19:24 |
mordred | ++ | 19:24 |
*** pratikmallya has quit IRC | 19:27 | |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Move option logging back to start if initialize_app() https://review.openstack.org/225287 | 19:29 |
openstackgerrit | Steve Martinelli proposed openstack/keystoneauth: add openid connect plugins https://review.openstack.org/224993 | 19:34 |
*** steveortiz is now known as zz_steveortiz | 19:40 | |
mordred | dtroyer: ok. https://review.openstack.org/#/c/224451/ fixes unittests - lemme go look at the functional tests that failed | 19:45 |
mordred | dtroyer: wth? | 19:46 |
mordred | 2015-09-18 19:30:01.939 | Missing parameter(s): | 19:46 |
mordred | 2015-09-18 19:30:01.939 | Set a token with --os-token, OS_TOKEN or auth.token | 19:46 |
mordred | 2015-09-18 19:30:01.939 | Set a service URL, with --os-url, OS_URL or auth.url | 19:46 |
mordred | 2015-09-18 19:30:01.124 | ++ openstack --os-url=http://127.0.0.1:5000/v3 --os-identity-api-version=3 project create service --domain=default --or-show -f value -c id | 19:46 |
mordred | I should have pasted that last line first | 19:46 |
mordred | dtroyer: uhm | 19:50 |
mordred | dtroyer: os-client-config is pinned to 1.6.3 because of upper-constraint | 19:50 |
openstackgerrit | Merged openstack/keystoneauth: Updated from global requirements https://review.openstack.org/225224 | 19:51 |
mordred | dtroyer: so latest occ is not causing that fail | 19:51 |
mordred | morgan: ^^ I has confuses | 19:57 |
morgan | mordred: yeah something odd there? | 19:58 |
mordred | morgan: WELL | 20:00 |
mordred | morgan: it's using 1.6.3, which is pre-ksa | 20:00 |
mordred | morgan: but the error it gives is one related to not having the right stuff for osc plugins | 20:00 |
*** jose4183 has quit IRC | 20:01 | |
morgan | Which should have worked with ksc? Right? | 20:02 |
*** jose4183 has joined #openstack-sdks | 20:02 | |
morgan | We didn't remove but added a new entry point | 20:02 |
*** hitalia has joined #openstack-sdks | 20:02 | |
*** jose4183 has quit IRC | 20:04 | |
*** jose4183 has joined #openstack-sdks | 20:05 | |
*** bnemec has quit IRC | 20:15 | |
*** bnemec has joined #openstack-sdks | 20:15 | |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Pass validate=False to get_one_cloud https://review.openstack.org/224451 | 20:22 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Pass validate=False to get_one_cloud https://review.openstack.org/224451 | 20:23 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: WIP - Rework everything to just use ksa/occ https://review.openstack.org/225318 | 20:25 |
mordred | the patch above is only pushed up for reference - there is no way it's going to look like that - 2 problems a) too much in one patch b) it breaks things it should not | 20:26 |
openstackgerrit | James E. Blair proposed openstack/os-client-config: Fix typo in ovh region names https://review.openstack.org/225321 | 20:29 |
dhellmann | mordred: is there no way to fix this issue in occ instead of osc? we have other bugs on master in osc, and a rather long list of unreleased changes, so I'm not sure we're actually in a good state to make a release there | 20:37 |
openstackgerrit | Steve Martinelli proposed openstack/keystoneauth: add openid connect plugins https://review.openstack.org/224993 | 20:38 |
dhellmann | mordred, terrylhowe, dtroyer, stevemar: the list of unreleased python-openstackclient changes: http://paste.openstack.org/show/468435/ | 20:38 |
stevemar | dhellmann: yep, we're trying to fix up occ and the osc gate | 20:39 |
stevemar | dhellmann: since we're dependent on occ and the rest of the clients we always get bitten | 20:39 |
mordred | dhellmann: oh. we can't cut an osc release when we fix it? | 20:39 |
dhellmann | stevemar: we need to add more tests | 20:40 |
dhellmann | mordred: well, we can, but that's a long list of changes to dump on a friday | 20:40 |
dhellmann | stevemar: either more jobs, or functional tests within the clients themselves | 20:41 |
stevemar | dhellmann: it is a long list, do you want to wait til monday? | 20:43 |
morgan | dhellmann: release and quit for the weekend... Worse than commit and quit ;) :P | 20:44 |
dhellmann | stevemar: I'd feel better about not breaking anyone else, but how badly are things jammed up right now? | 20:44 |
mordred | I believe the two things that are broken | 20:44 |
mordred | are osc | 20:44 |
stevemar | dhellmann: right now only osc is mucked | 20:44 |
mordred | and constraints | 20:44 |
stevemar | oh, and ^ | 20:44 |
mordred | we can't land new constraints that update occ | 20:44 |
morgan | Yah | 20:44 |
stevemar | gah | 20:44 |
dhellmann | mordred: I'm not at all concerned by the constraints auto-update because I'm doing our libs by hand and -1 any other lib changes | 20:44 |
mordred | and ... it's entirely possible this is breaking people with current osc installed | 20:44 |
mordred | out in the world | 20:44 |
stevemar | mordred: yes | 20:45 |
* dhellmann nods | 20:45 | |
mordred | if they also happen to install latest occ | 20:45 |
mordred | now, the set of people who would do those two things | 20:45 |
stevemar | yeah | 20:45 |
mordred | is pretty low | 20:45 |
stevemar | but it exists | 20:45 |
mordred | but it's at least a theory | 20:45 |
morgan | mordred: pip update -U | 20:45 |
dhellmann | is this an api change in occ? I don't really understand the fix | 20:45 |
stevemar | there was already a bug about it | 20:45 |
morgan | People do stranger things | 20:45 |
morgan | Erm install -U | 20:45 |
mordred | dhellmann: we added the creation of ... hang on | 20:46 |
mordred | dhellmann: thank you for asking that | 20:46 |
mordred | I do not understand why anything breaks with 1.7.2 of occ | 20:46 |
mordred | becuase that should no-op the new ksa stuff if it doesn't work | 20:46 |
dhellmann | mordred: it breaks with 1.7.0, 1.7.1, and 1.7.2 | 20:46 |
dhellmann | is it's not that recent a break | 20:47 |
mordred | yes. it should breka with 1.7.0 and 1.7.1 | 20:47 |
mordred | because bug | 20:47 |
mordred | which 1.7.2 fixed | 20:47 |
mordred | now - the thing I don't grok | 20:47 |
mordred | is why it's breaking with 1.7.2 | 20:47 |
mordred | the thing that 1.7.2 fixed should make the ksa stuff a noop if it can't happen | 20:48 |
dhellmann | ah | 20:48 |
mordred | let me dig again | 20:48 |
mordred | it's possible I missed something - I was under the impression we needed the osc patch ... but honestly, we should NOT | 20:48 |
mordred | because if we do, that means we've broken current installs, which is not what we want to do | 20:49 |
openstackgerrit | Merged openstack/os-client-config: Fix typo in ovh region names https://review.openstack.org/225321 | 20:49 |
mordred | dhellmann: I have an idea for a fix | 20:49 |
dhellmann | mordred: that's encouraging! | 20:49 |
mordred | dhellmann: ok. yes. I can produce fix only in occ | 20:51 |
mordred | dhellmann: it'll take a little bit | 20:51 |
dhellmann | mordred: cool. it's good to make both sides more robust, but if we can fix it just with occ that simplifies the question of when/whether to release osc | 20:52 |
dhellmann | stevemar, terrylhowe : since python-openstackclient is a managed project, someone from the core team just needs to submit a release request to openstack/releases | 20:53 |
mordred | btw - I'm not making this stylish - just working | 20:57 |
openstackgerrit | Monty Taylor proposed openstack/os-client-config: Fall back to keystoneclient arg processing https://review.openstack.org/225328 | 20:58 |
mordred | https://review.openstack.org/225328 | 20:58 |
mordred | stevemar, morgan: ^^ | 20:58 |
morgan | Nod | 21:00 |
morgan | Looking at it. | 21:00 |
openstackgerrit | Monty Taylor proposed openstack/os-client-config: Fall back to keystoneclient arg processing https://review.openstack.org/225328 | 21:11 |
mordred | morgan, stevemar: ^^ needed to update it | 21:11 |
mordred | found another way it can fai | 21:11 |
mordred | fail | 21:11 |
morgan | Oy! | 21:11 |
dhellmann | mordred: I'm going to start my weekend. I put up a patch to change the ACLs for occ releases, but it hasn't merged, yet, so you can still do them. After you do, go ahead and submit a patch to requirements to update the constraints file by hand because the automated thingy is trying to update all sorts of third-party libs that we're not updating right now. | 21:12 |
mordred | dhellmann: kk | 21:12 |
mordred | morgan: new logic: | 21:12 |
morgan | "Don't do the dumb"? ;) | 21:12 |
mordred | morgan: "try doing it with ksa, if that doens't work, try ksc. if it still does't work, raise the original ksa error, becuase that means it wasn't a transition issue" | 21:12 |
morgan | Ah. That is a good approach | 21:13 |
mordred | morgan: we need to try the ksc step becuse we use the ksc and ksa options not just for validation but to know how to split ot the auth parameters to pass to the constructors | 21:13 |
morgan | Yah | 21:13 |
morgan | Icky, but makes sense | 21:14 |
morgan | Silly ksc / ksa things | 21:14 |
mordred | Shrews: ^^ you may also want to look | 21:14 |
morgan | And tla soup | 21:14 |
mordred | morgan: yah man | 21:14 |
*** chlong has quit IRC | 21:14 | |
morgan | I try hard to avoid the TLAs | 21:14 |
morgan | But.. | 21:14 |
mordred | morgan: BUT - honestly I believe the only thing that this is affecting is osc | 21:14 |
morgan | Thats good | 21:15 |
mordred | morgan: so once we get it updated, we can drop this back out of occ | 21:15 |
morgan | Yah | 21:15 |
*** stevemar has quit IRC | 21:16 | |
*** stevemar has joined #openstack-sdks | 21:32 | |
*** stevemar has quit IRC | 21:32 | |
*** stevemar has joined #openstack-sdks | 21:33 | |
dtroyer | so… I'll say this just once more just because I'm in a grouchy mood… if occ didn't reach down into the app layer (ksa) this wouldn't be a problem. it doesn't validate other args, just accepts them and composes them into the dict/object. | 21:34 |
stevemar | dtroyer: you're in a grouchy mood? never... | 21:35 |
stevemar | :) | 21:35 |
*** trown is now known as trown|outttypeww | 21:36 | |
*** pratikmallya has joined #openstack-sdks | 21:41 | |
*** terrylhowe has quit IRC | 21:46 | |
*** jose4183 has quit IRC | 21:57 | |
stevemar | mordred: rechecked your 224444 patch now that occ 1.7.2 has been out, and its not looking like it fixed that last test >.< | 22:18 |
stevemar | tried it in my dev. env. too, same result as jenkins | 22:18 |
*** figleaf is now known as edleafe | 22:21 | |
mordred | steve | 22:23 |
mordred | gah | 22:23 |
mordred | stevemar: you need the new patch | 22:24 |
stevemar | mordred: one that is not https://review.openstack.org/#/c/224444/4 ? | 22:25 |
morgan | stevemar: https://review.openstack.org/225328 | 22:26 |
morgan | stevemar: the osc change is future looking and should happen eventually | 22:26 |
morgan | That one, 225328 fixes the immediate bug/issue | 22:27 |
stevemar | gotcha | 22:27 |
stevemar | then we'll need 1.7.3 :P | 22:27 |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:32 | |
morgan | Yar | 22:34 |
*** openstackgerrit has quit IRC | 22:46 | |
*** openstackgerrit has joined #openstack-sdks | 22:47 | |
stevemar | mordred: thank you sir | 22:55 |
stevemar | mordred: hmm, do you not use os/releases for occ? https://review.openstack.org/#/q/project:openstack/releases,n,z - i don't see patches there for 1.7.2 or 1.7.1 | 22:57 |
openstackgerrit | Merged openstack/os-client-config: Fall back to keystoneclient arg processing https://review.openstack.org/225328 | 22:57 |
mordred | stevemar: nope - we're just about to get that going | 23:02 |
stevemar | coolio | 23:09 |
mordred | dhellmann: https://review.openstack.org/225363 | 23:15 |
mordred | stevemar: 1.7.3 tagged and pushed | 23:15 |
mordred | stevemar: if that patch above passes gate, it means we've fixed it | 23:15 |
stevemar | mordred: rechecking now | 23:22 |
stevemar | mordred: oh hello old friend: gate-python-openstackclient-python26 SUCCESS | 23:25 |
*** stevemar has quit IRC | 23:26 | |
mordred | stevelle: woot! | 23:27 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!