Thursday, 2019-08-08

*** markvoelker has joined #openstack-keystone00:09
*** markvoelker has quit IRC00:42
*** markvoelker has joined #openstack-keystone00:43
*** gyee has quit IRC00:49
*** jamesmcarthur has joined #openstack-keystone01:05
*** jamesmcarthur has quit IRC01:11
*** jamesmcarthur has joined #openstack-keystone01:14
*** wxy-xiyuan has joined #openstack-keystone01:15
*** jamesmcarthur has quit IRC01:23
*** jamesmcarthur has joined #openstack-keystone01:26
*** markvoelker has quit IRC01:37
*** jamesmcarthur has quit IRC01:40
*** markvoelker has joined #openstack-keystone01:44
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504901:44
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504901:46
openstackgerritMerged openstack/keystoneauth master: Add remove_service to token fixtures  https://review.opendev.org/67517401:47
*** jhesketh has quit IRC01:53
*** jamesmcarthur has joined #openstack-keystone02:04
*** joshualyle has joined #openstack-keystone02:14
*** jamesmcarthur has quit IRC02:17
*** joshualyle has quit IRC02:18
*** jhesketh has joined #openstack-keystone02:22
*** jamesmcarthur has joined #openstack-keystone02:28
*** adriant has quit IRC02:54
*** whoami-rajat has joined #openstack-keystone03:12
*** jhesketh has quit IRC03:29
*** jhesketh has joined #openstack-keystone03:36
*** jamesmcarthur has quit IRC04:01
*** jamesmcarthur has joined #openstack-keystone04:05
*** jamesmcarthur has quit IRC04:09
*** jdwidari has quit IRC04:10
*** jamesmcarthur has joined #openstack-keystone04:10
*** jdwidari has joined #openstack-keystone04:10
*** jdwidari has quit IRC04:15
*** aning_ has joined #openstack-keystone04:18
*** jamesmcarthur has quit IRC04:19
*** aning has quit IRC04:22
*** jamesmcarthur has joined #openstack-keystone04:23
*** aning has joined #openstack-keystone04:29
*** aning_ has quit IRC04:31
*** jamesmcarthur has quit IRC04:33
*** jamesmcarthur has joined #openstack-keystone04:34
*** jamesmcarthur has quit IRC04:42
*** jamesmcarthur has joined #openstack-keystone04:42
*** adriant has joined #openstack-keystone04:45
*** jamesmcarthur has quit IRC04:48
*** jdennis has quit IRC04:53
*** shyamb has joined #openstack-keystone05:01
*** spsurya has joined #openstack-keystone05:02
*** dave-mccowan has quit IRC05:03
*** jdennis has joined #openstack-keystone05:04
*** shyamb has quit IRC05:04
*** shyamb has joined #openstack-keystone05:05
*** shyam89 has joined #openstack-keystone05:07
*** shyamb has quit IRC05:11
*** shyamb has joined #openstack-keystone05:43
*** shyam89 has quit IRC05:43
*** shyamb has quit IRC05:54
*** shyam89 has joined #openstack-keystone05:54
*** jaosorior has quit IRC05:56
*** dancn has joined #openstack-keystone06:09
*** jaosorior has joined #openstack-keystone06:09
*** rcernin has quit IRC06:18
*** jaosorior has quit IRC06:29
*** jaosorior has joined #openstack-keystone07:16
*** xek has joined #openstack-keystone07:17
*** shyam89 has quit IRC07:25
*** shyam89 has joined #openstack-keystone07:25
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504907:27
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504907:30
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504907:31
*** tesseract has joined #openstack-keystone07:31
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504907:33
adriantkmalloc, mordred: some of that code makes me feel dirty.... if anyone has suggestions or alternatives, you are more than welcome to recommend them!07:46
*** takamatsu has quit IRC08:06
*** ivve has joined #openstack-keystone08:19
*** shyamb has joined #openstack-keystone08:25
*** shyam89 has quit IRC08:27
*** shyamb has quit IRC08:33
openstackgerritVishakha Agarwal proposed openstack/keystone master: Implement scope_type checking for endpoint_groups  https://review.opendev.org/67527208:46
openstackgerritVishakha Agarwal proposed openstack/keystone master: Implement scope_type checking for endpoint_groups  https://review.opendev.org/67527208:52
*** shyamb has joined #openstack-keystone09:43
*** dmellado has quit IRC09:53
*** dmellado has joined #openstack-keystone09:55
*** shyamb has quit IRC10:19
*** shyamb has joined #openstack-keystone10:19
*** jaosorior has quit IRC10:26
brtknrhmm magnum on devstack seems broken10:52
brtknrit is only running script-004 which installs heat container agent, i dont see any other scripts in the cloud-init repo10:52
*** shyamb has quit IRC11:00
*** jaosorior has joined #openstack-keystone11:10
openstackgerritVishakha Agarwal proposed openstack/keystone master: Update api-ref for revocation list OS-PKI  https://review.opendev.org/67529611:18
*** kplant has joined #openstack-keystone11:18
*** raildo has joined #openstack-keystone11:18
*** shyamb has joined #openstack-keystone11:22
openstackgerritJose Castro Leon proposed openstack/keystone master: Allows LDAP extra attributes to be exposed to the end user  https://review.opendev.org/67530311:48
*** jamesmcarthur has joined #openstack-keystone11:51
*** jamesmcarthur has quit IRC11:51
*** jamesmcarthur has joined #openstack-keystone11:51
*** jamesmcarthur has quit IRC12:02
*** jamesmcarthur has joined #openstack-keystone12:04
*** dancn has quit IRC12:08
*** dancn has joined #openstack-keystone12:13
*** shyamb has quit IRC12:13
*** shyamb has joined #openstack-keystone12:14
*** shyamb has quit IRC12:16
*** shyam89 has joined #openstack-keystone12:16
*** jamesmcarthur has quit IRC12:21
*** shyam89 has quit IRC12:21
*** jaosorior has quit IRC12:22
*** jmlowe has quit IRC12:43
*** jamesmcarthur has joined #openstack-keystone12:54
*** mchlumsky has joined #openstack-keystone13:41
*** dave-mccowan has joined #openstack-keystone13:55
*** dave-mccowan has quit IRC14:30
*** dancn has quit IRC15:00
openstackgerritMerged openstack/oslo.policy master: Add unit tests on the sphinxext indent function  https://review.opendev.org/67124115:27
*** dave-mccowan has joined #openstack-keystone15:31
*** spsurya has quit IRC15:34
kmallocadriant: we have a LOT of that.15:41
*** gyee has joined #openstack-keystone15:43
*** beekneemech has joined #openstack-keystone16:16
*** beekneemech has quit IRC16:16
mordredkmalloc, cmurphy: can haz ksa release with the remove_service calls (and also dtantsur's fixed retry delay)16:19
cmurphymordred: can you propose and i'll +116:19
mordredwoot16:21
mordredcmurphy: https://review.opendev.org/675402 Release 3.16.0 of keystoneauth16:22
kmalloccmurphy/mordred ++17:10
*** markvoelker has quit IRC17:12
cmurphyknikolla: i reviewed https://review.opendev.org/580041 it looks really good, it just needs a rebase and lance's question answered17:18
cmurphyknikolla: kmalloc i brainstormed some things the edge hackers could do tomorrow https://etherpad.openstack.org/p/ece7c7d4dc although they would probably spend the whole day just setting up devstack with the keystone plugin17:19
*** jamesmcarthur has quit IRC17:22
kmallocyeah sadly it's one of those really in-depth things and tough to just work on17:25
kmalloc:(17:25
*** markvoelker has joined #openstack-keystone17:26
*** tesseract has quit IRC17:27
*** jamesmcarthur has joined #openstack-keystone17:33
knikollacmurphy: i guess even just documentation about the devstack plugin would be really helpful17:47
knikollai got an email the other day asking about that.17:47
rm_workkey, i'm trying to debug my keystone auth plugin to see EXACTLY what it's sending to the token endpoint -- where is the place in the client where it's ACTUALLY preparing the final request and sending it? like, where is the real requests.post() call?17:48
cmurphyknikolla: good idea17:50
cmurphyrm_work: in keystoneauth1.session.Session.request()17:50
rm_workok, looking -- thanks17:51
*** jamesmcarthur has quit IRC18:30
*** mchlumsky has quit IRC18:39
*** jamesmcarthur has joined #openstack-keystone18:42
*** ivve has quit IRC19:29
*** kplant has quit IRC19:46
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504920:27
adriantkmalloc: fixed my old broken test... the worst part is, that while the plugin code is VERY inelegant, the usage code (in the fixed test) looks ok.20:30
adrianthttps://review.opendev.org/#/c/675049/8/keystoneauth1/tests/unit/identity/test_identity_v3.py@76320:32
adriantoh wait, wrong patch set link20:32
adriantugh20:32
*** ivve has joined #openstack-keystone20:32
adrianthttps://review.opendev.org/#/c/675049/9/keystoneauth1/tests/unit/identity/test_identity_v3.py@76320:32
adriant^ that20:32
adriantSo my thinking for that plugin, and the loader, is that in the CLI we can specify OS_AUTH_METHODS=v3password,v3totp (or in clouds.yaml) and then all the needed values get pulled in for those, with a prompt for password and passcode20:34
adriantit does mean we now have a meta wrapper, and maybe we need to make doing that easier20:34
adriantmeta loader*20:34
*** xek has quit IRC20:41
openstackgerritAdrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth  https://review.opendev.org/67504920:46
kmallocadriant: wfm.21:01
*** whoami-rajat has quit IRC21:01
*** rm_work has quit IRC21:05
*** rm_work has joined #openstack-keystone21:05
adrianta mapping between method name > method class would make things easier21:06
adriantbecause: 'loading.get_plugin_loader(method).plugin_class._auth_method_class' feels dirty21:07
*** jamesmcarthur has quit IRC21:19
*** vishwanathj has joined #openstack-keystone21:31
*** raildo has quit IRC21:34
*** markvoelker has quit IRC21:36
*** markvoelker has joined #openstack-keystone21:53
*** markvoelker has quit IRC21:57
openstackgerritMerged openstack/keystone master: Fix list_mappings deprecation warning message  https://review.opendev.org/67493422:14
openstackgerritMerged openstack/keystone master: Clean up irrelevant comment  https://review.opendev.org/67493522:14
*** whoami-rajat has joined #openstack-keystone22:24
adriantkmalloc: so yeah... zuul now passes22:30
adriantI still think the code is awful, but I can't think of a better way to do it right now22:30
*** rcernin has joined #openstack-keystone22:53
kmallocadriant: if you don't vomit a little in your mouth...it's not keystoneauth22:55
adriantmy worry is then having to maintain it22:55
kmalloclots of testing22:55
kmallocover test the code22:55
kmallocmakes maintenance easier because we have more indicators when something goes sideways22:55
kmallocstevedore loading is always somewhat nightmare-ish22:56
adriantYeah.... I was tempted to use that for Adjutant, then just stuck with Django's app system22:56
adriantseemed safer22:56
kmallocwe could remove a chunk of the stevedore loading bits from KSA (still support for out-of-tree modules/plugins) but it would lighten the code/ick as well22:56
kmallocthere is almost no reason for us to have the loading code in ksa used for in-tree modules.22:57
adriantwe could replace it all with our own in code mappings22:58
adriantwell, nah, for this awful new MFA stuff I'd still need the plugin loaders in case of extra auth methods people want23:01
* adriant sighs23:01
adriantmordred: You'll ultimately be one of the consumers of this, so I'll definitely want your opinion when you've got the time!23:02
adriantOnce there is some feedback I can hopefully make this better23:02

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!