| *** jkilpatr has quit IRC | 01:08 | |
| *** chlong_ has quit IRC | 03:08 | |
| *** chlong_ has joined #openstack-mistral | 03:09 | |
| *** chlong_ has quit IRC | 03:17 | |
| *** chlong_ has joined #openstack-mistral | 03:18 | |
| *** gkadam has joined #openstack-mistral | 03:32 | |
| khandhedia | kong: I added auth_type=password in [keystone_authtoken]; same error still :( | 04:32 |
|---|---|---|
| khandhedia | rakhmerov: kong: Launchpad bug raised: https://bugs.launchpad.net/mistral/+bug/1702818 | 04:45 |
| openstack | Launchpad bug 1702818 in Mistral "Authorization failed: Cannot authenticate without an auth_url" [Undecided,New] | 04:45 |
| rakhmerov | khandhedia: thanks | 04:45 |
| *** sharatss has joined #openstack-mistral | 05:08 | |
| khandhedia | https://bugs.launchpad.net/mistral/+bug/1702818 updated with additional findings | 06:05 |
| openstack | Launchpad bug 1702818 in Mistral "Authorization failed: Cannot authenticate without an auth_url" [Undecided,New] | 06:05 |
| *** jtomasek has joined #openstack-mistral | 07:04 | |
| *** shardy has joined #openstack-mistral | 07:06 | |
| *** jtomasek has quit IRC | 07:18 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral master: Turn on warning-is-error for sphinx_build https://review.openstack.org/480504 | 07:27 |
| *** jpich has joined #openstack-mistral | 07:41 | |
| rakhmerov | d0ugal: hi, I left a comment in https://review.openstack.org/#/c/476110/ where I try to convince you that this change is ok :) Although fully understand your concern | 07:43 |
| rakhmerov | pls have a look | 07:43 |
| d0ugal | rakhmerov: sure, I'll look now | 07:46 |
| d0ugal | rakhmerov: I am okay with the change, but it needs a release note IMO | 07:47 |
| rakhmerov | sure, no question here | 07:47 |
| rakhmerov | can you explicitly reflect it there pls? | 07:48 |
| d0ugal | okay | 07:48 |
| d0ugal | That was my intention on the previous review, but I can see I didn't do a very good job :) | 07:48 |
| sharatss | rakhmerov, d0ugal releasenote is enough or way to delimit also be mentioned? | 07:51 |
| d0ugal | sharatss: just a release note, we can maybe add a way to remove the limit later | 07:52 |
| d0ugal | but I guess for now I could do --limit=1000000000000000000 | 07:52 |
| d0ugal | :) | 07:52 |
| sharatss | d0ugal, ok.. can i add it now? :) | 07:52 |
| d0ugal | sharatss: please do, then I'll +2 | 07:52 |
| sharatss | d0ugal, namaste | 07:53 |
| *** jtomasek has joined #openstack-mistral | 07:55 | |
| rakhmerov | sharatss, d0ugal: I'd use a negative for unlimited sets | 07:55 |
| rakhmerov | e.g. -1 | 07:56 |
| rakhmerov | it seems to be a normal practice | 07:56 |
| sharatss | rakhmerov, should that be explicitly added in the warning msg? | 07:56 |
| rakhmerov | sure | 07:56 |
| sharatss | rakhmerov, like "use -1 to remove the default limit" | 07:57 |
| rakhmerov | yes, or something like "use -1 to fetch the full result set" | 07:59 |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Set the default value of --limit parameter https://review.openstack.org/476110 | 08:19 |
| *** jtomasek has quit IRC | 08:25 | |
| sharatss | rakhmerov, should action-execution-list also be handled that way? | 08:27 |
| rakhmerov | yes, please | 08:27 |
| rakhmerov | sharatss: it's potentially even more dangerous than task-list and execution-list | 08:28 |
| rakhmerov | more objects | 08:28 |
| sharatss | rakhmerov, ok.. adding that as well in next patch | 08:28 |
| rakhmerov | yes, ok | 08:28 |
| rakhmerov | thanks a lot! | 08:29 |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Set the default value of --limit parameter https://review.openstack.org/476110 | 08:55 |
| sharatss | rakhmerov, d0ugal this must be it ^^ | 08:56 |
| rakhmerov | sharatss: one last thing from my side: please mention "-1" meaning in the help message for all 3 commands | 08:59 |
| rakhmerov | the rest looks fine | 08:59 |
| sharatss | rakhmerov, like? | 09:02 |
| rakhmerov | sorry, I wasn't clear enough | 09:06 |
| rakhmerov | now you write "Use -1 for the full set", but it's printed only in warning messages | 09:06 |
| rakhmerov | please also add this info into help messages for these commands | 09:07 |
| rakhmerov | about the default limit and -1 | 09:07 |
| apetrich | sharatss, looks fine but I miss the 100 coming from one single place. How do rakhmerov and d0ugal feel about adding a constants to python-mistralclient ? | 09:09 |
| rakhmerov | apetrich: usually I'm for declaring such things at one place, yes. Missed it this time. sharatss: can you change that too? | 09:10 |
| rakhmerov | apetrich: good catch | 09:10 |
| sharatss | rakhmerov, yes. both in the next one | 09:11 |
| apetrich | cheers | 09:11 |
| d0ugal | +1 | 09:11 |
| openstackgerrit | Alfredo Moralejo proposed openstack/mistral master: Revert "Use recommended function to setup auth middleware in devstack" https://review.openstack.org/481535 | 09:18 |
| *** gkadam_ has joined #openstack-mistral | 09:25 | |
| *** gkadam is now known as Guest24494 | 09:25 | |
| *** gkadam_ is now known as gkadam | 09:25 | |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** gkadam_ has joined #openstack-mistral | 10:03 | |
| *** gkadam has quit IRC | 10:05 | |
| *** gkadam__ has joined #openstack-mistral | 10:10 | |
| *** gkadam_ has quit IRC | 10:12 | |
| *** thrash|g0ne is now known as thrash | 10:59 | |
| csatari | sharat> You mean to continue https://review.openstack.org/480372 for example? | 11:19 |
| sharatss | csatari, yes.. whatever related to doc-migration in mistral | 11:37 |
| *** openstackgerrit has joined #openstack-mistral | 11:40 | |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Set the default value of --limit parameter https://review.openstack.org/476110 | 11:40 |
| *** sharatss has quit IRC | 12:09 | |
| *** shardy has quit IRC | 12:18 | |
| *** catintheroof has joined #openstack-mistral | 12:40 | |
| *** dprince has joined #openstack-mistral | 12:42 | |
| *** chlong_ has quit IRC | 12:43 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: Fixing deleting cron-trigger trusts https://review.openstack.org/481622 | 12:57 |
| *** gkadam__ has quit IRC | 12:58 | |
| *** rbrady has quit IRC | 13:01 | |
| d0ugal | rakhmerov: Hey | 13:03 |
| rakhmerov | yes sir | 13:03 |
| d0ugal | rakhmerov: so the reverted change breaks the Ironic inspector actions. | 13:04 |
| d0ugal | Which are not tested in the tripleo gate we have - only in the periodic CI tests | 13:05 |
| rakhmerov | ok, the it breaks our own actions | 13:05 |
| d0ugal | You asked for more details - I am not sure where you want them or what other details you would like. | 13:05 |
| rakhmerov | it breaks.. | 13:05 |
| d0ugal | yeah | 13:05 |
| rakhmerov | I see now, it's enough | 13:05 |
| d0ugal | They fail in the db populate | 13:05 |
| rakhmerov | let's revert it | 13:05 |
| d0ugal | thanks | 13:05 |
| rakhmerov | I'll ask Mike to test it better when applying again | 13:06 |
| *** rbrady has joined #openstack-mistral | 13:06 | |
| *** rbrady has joined #openstack-mistral | 13:06 | |
| d0ugal | Is he in here? I am happy to help him test it | 13:06 |
| rakhmerov | d0ugal: I asked him to join | 13:14 |
| *** mfedosin has joined #openstack-mistral | 13:14 | |
| d0ugal | Thanks | 13:14 |
| rakhmerov | mfedosin: hey | 13:14 |
| mfedosin | d0ugal: hi! | 13:14 |
| rakhmerov | d0ugal can help you | 13:15 |
| d0ugal | mfedosin: Hey | 13:15 |
| mfedosin | just wanted to find the place where mistal fails... | 13:15 |
| *** bobh has joined #openstack-mistral | 13:15 | |
| mfedosin | can you show it in the logs? | 13:15 |
| d0ugal | mfedosin: Trying to find it | 13:17 |
| d0ugal | mfedosin: the log files I seen it in before have expired | 13:17 |
| d0ugal | mfedosin: http://logs.openstack.org/96/473796/18/check/gate-tripleo-ci-centos-7-scenario003-multinode-oooq-puppet/2e7f93a/logs/undercloud/var/log/mistral/mistral-db-manage.log.txt.gz#_2017-07-05_12_50_17_675 | 13:19 |
| d0ugal | mfedosin: that is the new error | 13:19 |
| mfedosin | probably mistral keystone middleware had been configured incorrectly and keystoneauth1.loading couldn't parse it | 13:20 |
| d0ugal | mfedosin: you can see in a patch that passed/merged before yours registered the actions correctly: http://logs.openstack.org/83/461683/3/check/gate-tripleo-ci-centos-7-scenario003-multinode-oooq-puppet/c8fd2ac/logs/undercloud/var/log/mistral/mistral-db-manage.log.txt.gz#_2017-07-05_12_07_16_170 | 13:20 |
| d0ugal | There are lots of errors in that log file you can ignore, because tripleo doesn't have all the openstack clients, but the errors specific to introspection are the "real" errors. | 13:20 |
| mfedosin | okay, I'm looking | 13:21 |
| mfedosin | thank you! | 13:21 |
| d0ugal | np | 13:21 |
| d0ugal | mfedosin: if you post your patch again and leave the comment "check experimental" then tripleo CI will do a more extensive test that will use the introspection actions. | 13:22 |
| d0ugal | so that might be useful :) | 13:22 |
| mfedosin | good to know :) | 13:22 |
| mfedosin | okay, I'll try to fix the cause of the issue | 13:23 |
| d0ugal | mfedosin: I wont be easy to contact next week but you can ask apetrich (europe times) or rbrady (us times) in here for help running that check or understanding the logs | 13:23 |
| d0ugal | mfedosin: thanks | 13:23 |
| mfedosin | got it | 13:23 |
| *** zhurong has joined #openstack-mistral | 13:24 | |
| mfedosin | my plan is to compare both configurations of keystone_authtoken | 13:24 |
| apetrich | cool | 13:25 |
| mfedosin | d0ugal: apetrich: folks, I found the cause | 13:28 |
| d0ugal | great | 13:28 |
| mfedosin | I looked at keystone_auth configuration section in mistral and found | 13:28 |
| *** bobh has quit IRC | 13:28 | |
| mfedosin | identity_uri=http://192.168.24.1:35357 | 13:28 |
| mfedosin | admin_user=mistral | 13:28 |
| mfedosin | admin_password=320aedfde927b587189fc8273908ca44013a061b | 13:28 |
| mfedosin | admin_tenant_name=service | 13:28 |
| mfedosin | http://logs.openstack.org/96/473796/18/check/gate-tripleo-ci-centos-7-scenario003-multinode-oooq-puppet/2e7f93a/logs/undercloud/etc/mistral/mistral.conf.txt.gz | 13:29 |
| mfedosin | This is exactly what I was trying to get rid of | 13:29 |
| mfedosin | all these parameters were deprecated in keystone and not recommended for usage | 13:30 |
| mfedosin | https://github.com/openstack/keystonemiddleware/blob/712438ebf9ee30d553f6b515e38b1c91a9ae498e/keystonemiddleware/auth_token/_auth.py#L29-L35 | 13:30 |
| mfedosin | keystoneauth1.loading can't parse them correctly, as far as I know | 13:31 |
| *** Dinesh_Bhor has quit IRC | 13:32 | |
| *** apetrich_ has joined #openstack-mistral | 13:32 | |
| rakhmerov | hah! | 13:32 |
| *** apetrich_ has quit IRC | 13:33 | |
| openstackgerrit | Gergely Csatari proposed openstack/mistral master: Applying Pike document structure https://review.openstack.org/480643 | 13:43 |
| *** bobh has joined #openstack-mistral | 14:04 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: Fixing deleting cron-trigger trusts https://review.openstack.org/481622 | 14:06 |
| *** chlong_ has joined #openstack-mistral | 14:08 | |
| openstackgerrit | Merged openstack/mistral master: Revert "Use recommended function to setup auth middleware in devstack" https://review.openstack.org/481535 | 14:17 |
| *** zhurong has quit IRC | 14:26 | |
| *** bobh has quit IRC | 14:35 | |
| *** bobh has joined #openstack-mistral | 14:44 | |
| *** catintheroof has quit IRC | 14:52 | |
| *** catintheroof has joined #openstack-mistral | 14:53 | |
| *** bobh has quit IRC | 14:53 | |
| *** bobh has joined #openstack-mistral | 14:55 | |
| *** catinthe_ has joined #openstack-mistral | 14:56 | |
| *** catintheroof has quit IRC | 14:59 | |
| *** bobh has quit IRC | 15:02 | |
| *** catintheroof has joined #openstack-mistral | 15:06 | |
| *** bobh has joined #openstack-mistral | 15:08 | |
| *** catinthe_ has quit IRC | 15:09 | |
| *** bobh has quit IRC | 15:19 | |
| *** bobh has joined #openstack-mistral | 15:21 | |
| *** catinthe_ has joined #openstack-mistral | 15:35 | |
| *** catintheroof has quit IRC | 15:36 | |
| *** thrash is now known as thrash|biab | 15:40 | |
| *** bobh has quit IRC | 15:41 | |
| *** bobh has joined #openstack-mistral | 15:43 | |
| *** thrash|biab is now known as thrash | 16:07 | |
| *** catinthe_ has quit IRC | 16:19 | |
| *** catintheroof has joined #openstack-mistral | 16:19 | |
| *** jpich has quit IRC | 16:37 | |
| *** dtantsur|afk has quit IRC | 18:10 | |
| *** dtantsur has joined #openstack-mistral | 18:13 | |
| *** chlong_ has quit IRC | 18:40 | |
| openstackgerrit | Winson Chan proposed openstack/mistral master: Allow async action execution to be paused and resumed https://review.openstack.org/481738 | 19:20 |
| openstackgerrit | Winson Chan proposed openstack/mistral master: Allow async action execution to be paused and resumed https://review.openstack.org/481738 | 19:24 |
| *** dtantsur has quit IRC | 19:45 | |
| *** bobh has quit IRC | 19:47 | |
| *** bobh has joined #openstack-mistral | 19:49 | |
| *** dtantsur has joined #openstack-mistral | 19:50 | |
| *** catinthe_ has joined #openstack-mistral | 19:56 | |
| *** dtantsur has quit IRC | 19:57 | |
| *** catintheroof has quit IRC | 19:59 | |
| *** dtantsur has joined #openstack-mistral | 20:02 | |
| *** mattybrennan has quit IRC | 20:11 | |
| *** bobh has quit IRC | 20:29 | |
| *** bobh has joined #openstack-mistral | 20:32 | |
| *** bobh has quit IRC | 20:37 | |
| *** catinthe_ has quit IRC | 20:39 | |
| *** bobh has joined #openstack-mistral | 20:44 | |
| *** dprince has quit IRC | 20:49 | |
| *** catintheroof has joined #openstack-mistral | 21:33 | |
| *** openstackgerrit has quit IRC | 21:47 | |
| *** openstack has joined #openstack-mistral | 21:58 | |
| *** apetrich has quit IRC | 22:05 | |
| *** apetrich has joined #openstack-mistral | 22:05 | |
| *** bobh has quit IRC | 22:26 | |
| *** catintheroof has quit IRC | 22:42 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!