Monday, 2018-04-16

*** bobh has joined #openstack-mistral00:02
*** bobh has quit IRC00:07
*** d0ugal_ has quit IRC00:10
*** bobh has joined #openstack-mistral00:11
*** bobh has quit IRC00:16
*** bobh has joined #openstack-mistral00:23
*** d0ugal_ has joined #openstack-mistral00:25
*** bobh has quit IRC00:28
*** bobh has joined #openstack-mistral00:30
*** bobh has quit IRC00:35
*** bobh has joined #openstack-mistral00:46
*** bobh has quit IRC00:50
*** bobh has joined #openstack-mistral00:52
*** bobh has quit IRC00:55
*** bobh has joined #openstack-mistral00:55
*** d0ugal__ has joined #openstack-mistral01:09
*** d0ugal_ has quit IRC01:11
*** d0ugal has joined #openstack-mistral01:15
*** d0ugal__ has quit IRC01:16
openstackgerritVitalii Solodilov proposed openstack/mistral master: Minor `update_on_match` improvement:  https://review.openstack.org/56149001:25
*** bobh has quit IRC03:10
*** bobh has joined #openstack-mistral03:11
*** bobh has quit IRC03:15
*** bobh has joined #openstack-mistral03:22
*** bobh has quit IRC03:27
*** bobh has joined #openstack-mistral03:30
*** bobh has quit IRC03:34
*** bobh has joined #openstack-mistral03:42
*** bobh has quit IRC03:46
*** bobh has joined #openstack-mistral03:52
*** bobh has quit IRC03:57
*** bobh has joined #openstack-mistral04:03
*** bobh has quit IRC04:08
*** bobh has joined #openstack-mistral04:18
*** bobh has quit IRC04:23
*** bobh has joined #openstack-mistral04:30
*** bobh has quit IRC04:35
*** bobh has joined #openstack-mistral04:44
*** hardikjasani has joined #openstack-mistral04:48
*** bobh has quit IRC04:49
*** bobh has joined #openstack-mistral04:55
*** bobh has quit IRC04:59
*** bobh has joined #openstack-mistral05:02
*** bobh has quit IRC05:07
*** bobh has joined #openstack-mistral05:14
*** bobh has quit IRC05:19
*** bobh has joined #openstack-mistral05:24
*** d0ugal has quit IRC05:27
*** bobh has quit IRC05:29
*** d0ugal has joined #openstack-mistral05:30
*** bobh has joined #openstack-mistral05:35
*** bobh has quit IRC05:38
*** bobh has joined #openstack-mistral05:44
*** bobh has quit IRC05:49
*** bobh has joined #openstack-mistral06:11
*** bobh has quit IRC06:16
*** bobh has joined #openstack-mistral06:21
*** bobh has quit IRC06:25
*** bobh has joined #openstack-mistral06:30
*** bobh has quit IRC06:35
*** pgaxatte has joined #openstack-mistral06:45
*** jtomasek has joined #openstack-mistral06:46
*** bobh has joined #openstack-mistral06:52
*** bobh has quit IRC06:57
*** jpich has joined #openstack-mistral06:59
*** bobh has joined #openstack-mistral07:02
*** bobh has quit IRC07:07
*** livelace-link has quit IRC07:09
*** livelace-link has joined #openstack-mistral07:10
*** livelace-link has quit IRC07:11
openstackgerritHardik Jasani proposed openstack/mistral master: Fix DB connection url in config guide  https://review.openstack.org/56001507:12
*** bobh has joined #openstack-mistral07:13
*** bobh has quit IRC07:18
*** threestrands has quit IRC07:47
*** itlinux has joined #openstack-mistral07:47
*** bobh has joined #openstack-mistral07:54
*** shardy has joined #openstack-mistral07:56
*** AlexeyAbashkin has joined #openstack-mistral07:58
*** itlinux_ has joined #openstack-mistral08:02
*** itlinux has quit IRC08:04
*** itlinux_ has quit IRC08:06
*** jaosorior has joined #openstack-mistral08:10
openstackgerritMerged openstack/mistral master: Fix DB connection url in config guide  https://review.openstack.org/56001508:32
openstackgerritRenat Akhmerov proposed openstack/mistral master: Adding a test for std.test_dict action  https://review.openstack.org/56117408:38
openstackgerritRenat Akhmerov proposed openstack/mistral master: WIP: Optimizing big 'on-XXX' clauses  https://review.openstack.org/56117508:38
*** itlinux has joined #openstack-mistral08:53
*** itlinux has quit IRC08:54
*** itlinux has joined #openstack-mistral08:57
*** gkadam has joined #openstack-mistral09:05
*** gkadam has quit IRC09:06
openstackgerritRenat Akhmerov proposed openstack/mistral master: WIP: Optimizing big 'on-XXX' clauses  https://review.openstack.org/56117509:06
*** gkadam has joined #openstack-mistral09:11
*** gkadam has quit IRC09:13
*** itlinux has quit IRC09:28
*** itlinux has joined #openstack-mistral11:16
openstackgerritRenat Akhmerov proposed openstack/mistral master: Optimizing big 'on-XXX' clauses  https://review.openstack.org/56117511:46
*** thrash|g0ne is now known as thrash11:59
*** bobh has quit IRC12:09
*** d0ugal_ has joined #openstack-mistral12:14
*** d0ugal has quit IRC12:15
*** itlinux has quit IRC12:46
openstackgerritMerged openstack/mistral master: Adding a test for std.test_dict action  https://review.openstack.org/56117413:23
*** d0ugal_ has quit IRC13:40
*** d0ugal has joined #openstack-mistral13:41
*** d0ugal has quit IRC13:41
*** d0ugal has joined #openstack-mistral13:41
*** hardikjasani has quit IRC13:48
*** mcdoker181818 has joined #openstack-mistral13:56
mcdoker181818Hi all. Why run_existing function doesn't execute the "self._before_task_start()" function?14:09
mcdoker181818https://github.com/openstack/mistral/blob/master/mistral/engine/tasks.py#L40814:09
mcdoker181818https://thepasteb.in/p/r0hwz0LWmwrCK For example this workflow was frozen in RUNNING state. Is it bug or by design?14:10
*** bobh has joined #openstack-mistral14:19
d0ugalHey Mistralites15:00
d0ugal#startmeeting mistral15:01
openstackMeeting started Mon Apr 16 15:01:05 2018 UTC and is due to finish in 60 minutes.  The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
*** openstack changes topic to " (Meeting topic: mistral)"15:01
openstackThe meeting name has been set to 'mistral'15:01
d0ugald0ugal,rakmerov,apetrich: Office hour time!15:01
d0ugalhttps://etherpad.openstack.org/p/mistral-office-hours15:01
d0ugalI am going to start pinging people on line 17 of that etherpad15:01
d0ugalAdd your IRC nick if you want a ping when the office hour starts.15:01
d0ugalmcdoker181818: We can talk about your question, if you are still around.15:03
mcdoker181818Yea, i'm here15:03
d0ugalbobh: Thanks for the typo fix!15:05
d0ugalrakhmerov: ^15:05
bobhd0ugal: :-)15:06
d0ugalmcdoker181818: so I am not very familiar with that code15:07
d0ugalJust reading it now.15:07
mcdoker181818How to change the nickname color? I don't want to be pink :D15:09
d0ugalmcdoker181818: click on the icon on the top right15:09
d0ugalmcdoker181818: I think you have found a bug, I am not sure about the code but I don't think that workflow should be stuck in RUNNING.15:09
d0ugalbecause it isn't :)15:10
d0ugalIt should, IMO, be in an error state.15:10
therved0ugal, I tested the token expiration bug a little bit15:11
therveI could reproduce it with a heat action as well15:11
d0ugaltherve: Interesting.15:12
d0ugalso that means it is likely our fault.15:12
d0ugalDarn15:12
therved0ugal, I tried a few quick fixes, but I didn't make it work15:12
therveSession stuff is confusing15:12
d0ugalyeah, it really is.15:12
therveUsing token_endpoint.Token seems like a big no-no though15:13
d0ugalIs that something we do?15:13
thervehttps://github.com/openstack/keystoneauth/blob/master/keystoneauth1/token_endpoint.py#L1915:13
d0ugalI have not looked at that code for some time.15:13
d0ugalta15:13
therveYeah that's what get_session_and_auth creates15:13
d0ugaloh yeah15:13
therveAt any rate, it's really easy to reproduce with std.sleep and any action15:14
therveIdeally after reducing token expiration time15:14
therve(Though not too little because I don't think it's supported)15:14
d0ugaltherve: yeah, I never reduced the expiration which made testing tedious.15:15
therveYeah15:15
d0ugalhttps://github.com/d0ugal/tripleo-util/blob/master/scripts/tests/general.yaml#L8-L3115:15
d0ugalbut I did a similar test15:15
mcdoker181818d0ugal: Should i create a ticket and just fix it?15:15
d0ugalmcdoker181818: yes please15:16
d0ugalmcdoker181818: I think adding what you suggested makes sense - but it would be good to get more eyes on that idea. A review is probably the best place for it.15:16
d0ugalmcdoker181818: if you send me a bug I can triage it.15:16
d0ugaltherve: do you have a link to the bug handy? I can't find it.15:17
d0ugalGot it.15:17
d0ugalhttps://bugs.launchpad.net/tripleo/+bug/176105015:17
openstackLaunchpad bug 1761050 in tripleo "Mistral or Zaqar fail to maintain a keystone session while deploying an overcloud" [High,Triaged]15:17
d0ugalbobh: did you have anything you want to talk about?15:19
*** rbrady has quit IRC15:22
openstackgerritBrad P. Crochet proposed openstack/mistral stable/pike: Remove any old client actions that no longer exist  https://review.openstack.org/56164915:23
d0ugalthrash: Nice! I have been wanting to do that.15:23
thrashd0ugal: I'm not sure how much of that is relevant for pike though. We'll need to keep a close eye on the logs to make sure it doesn't introduce new issues.15:25
d0ugalthrash: oh, I didn't notice that was a backport. I missed the original patch.15:25
thrash:D15:25
d0ugalthrash: yeah, I have no idea how to verify that patch.15:25
d0ugalWe do not have good coverage of all those actions15:26
d0ugalso I am not sure it is safe to backport?15:26
thrashd0ugal: Not really, no. Maybe we need a job that tests action creation against devstack?15:26
thrashd0ugal: and reports on actions that can't be created?15:27
d0ugalthrash: That would be good.15:27
thrashd0ugal: All I can think of right now is specifically looking at the tripleo job when it is done to see what the logs say.15:27
d0ugalthrash: does it have every client?15:27
thrashd0ugal: https://bugzilla.redhat.com/show_bug.cgi?id=151605615:27
openstackbugzilla.redhat.com bug 1516056 in openstack-mistral "mapping error in mistral-db-manage.log" [High,Post] - Assigned to brad15:27
thrashrelated bz15:27
thrashd0ugal: tripleo does not, no.15:28
thrashd0ugal: which is why we would need a devstack job that would have it.15:28
d0ugaloh, January - that is why I missed the original patch.15:28
d0ugalI have no memory of January.15:28
thrashbahaha15:29
*** thrash is now known as thrash|f00dz15:29
mcdoker181818d0ugal: https://bugs.launchpad.net/mistral/+bug/176444115:30
openstackLaunchpad bug 1764441 in Mistral "Timeout policy doesn't work with retry policy properly" [Undecided,New]15:30
d0ugalthrash|f00dz: Looks like the devstack jobs currently log it. Just need to dig around. http://logs.openstack.org/75/561175/4/check/mistral-devstack-kombu/5f97c38/job-output.txt.gz#_2018-04-16_12_12_20_48290115:30
d0ugalQuite a few failures in there.15:30
d0ugalmcdoker181818: Thanks!15:30
d0ugalmcdoker181818: What is your launchpad username? or you can just assign it to yourself if you want to work on it.15:31
d0ugalI put it as Rocky-2 (Rocky-1 ends on Friday)15:31
mcdoker181818Done.15:32
*** jistr is now known as jistr|mtg15:33
d0ugalThanks15:33
d0ugalOkay, I am going to do some bug triage for 25 mins15:34
d0ugalbut I'll be around and happy to talk about whatever if people have topics.15:34
d0ugalWe have 13 "NEW" bugs - I hope to triage all of these by the end of the week.15:34
d0ugalhttps://bugs.launchpad.net/mistral/+bugs?search=Search&field.status=New&orderby=id&start=015:34
d0ugalStarting with https://bugs.launchpad.net/mistral/+bug/150472515:37
openstackd0ugal: Error: Could not gather data from Launchpad for bug #1504725 (https://launchpad.net/bugs/1504725). The error has been logged15:37
d0ugalIt looks like Andras was able to confirm this in Mistral recently. So I guess it is legit. The two other projects closed it however which is interesting.15:37
d0ugalIt is a very old bug.15:37
d0ugalConfirmed it for now, assume it is fairly low priority as it has been around for such a long time.15:40
d0ugalNext. https://bugs.launchpad.net/mistral/+bug/165527315:40
openstackLaunchpad bug 1655273 in Mistral "Human-readable failed action error" [Undecided,New]15:40
d0ugalI think this is actually quite an important one. The error formatting for actions isn't great. It is hard to use programatically15:41
d0ugalI would say it is human readable now, just - but isn't really machine readable.15:41
d0ugalAll the information is good to have, but it is hard to extract just the error message (if you want to show it to users for example)15:42
d0ugalTriaged and commented.15:45
d0ugalhttps://bugs.launchpad.net/mistral/+bug/166059015:46
openstackLaunchpad bug 1660590 in Mistral "Mistralclient does not pass cacert to other Openstack clients" [Undecided,New]15:46
d0ugal I am not sure about this. I don't really know anything about cacerts in OpenStack clients.15:46
d0ugalMarked as Incomplete and I asked a question.15:47
d0ugalhttps://bugs.launchpad.net/mistral/+bug/166085315:48
openstackLaunchpad bug 1660853 in Mistral "Lexical error when registering std.wait_ssh action" [Undecided,New]15:48
mcdoker181818I faced with this problem. And I fixed it somehow...15:48
d0ugalmcdoker181818: the lexical error?15:49
mcdoker181818with tls15:49
*** hardikjasani has joined #openstack-mistral15:50
d0ugaloh, interesting15:51
mcdoker181818But i rewrite openstack actions, and because I can't merge fix :)15:51
d0ugalYou can't? Why not?15:51
mcdoker181818My openstack action differs from vanilla version15:52
d0ugalah, I see15:52
*** rbrady has joined #openstack-mistral16:00
*** rbrady has quit IRC16:00
*** rbrady has joined #openstack-mistral16:00
d0ugalOkay, that is the end of the slot!16:01
d0ugalThanks everyone. Next slot is probably Friday morning - I can never make the Wednesday one16:01
d0ugalI am still thinking we should cancel or move it.16:01
d0ugalUntil next time!16:02
d0ugal#endmeeting16:02
*** openstack changes topic to " (Meeting topic: test)"16:02
openstackMeeting ended Mon Apr 16 16:02:13 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:02
openstackMinutes:        http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-04-16-15.01.html16:02
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-04-16-15.01.txt16:02
openstackLog:            http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-04-16-15.01.log.html16:02
*** jistr|mtg is now known as jistr16:03
*** mcdoker181818 has quit IRC16:07
openstackgerritVitalii Solodilov proposed openstack/mistral master: WIP: Remove spec validation from mistral-engine  https://review.openstack.org/55560616:22
*** jpich has quit IRC16:30
*** jrist has quit IRC16:39
*** bobh has quit IRC17:00
*** jrist has joined #openstack-mistral17:01
*** bobh has joined #openstack-mistral17:13
*** AlexeyAbashkin has quit IRC17:17
*** jrist has quit IRC17:22
*** jrist has joined #openstack-mistral17:32
*** jrist has quit IRC17:32
*** jrist has joined #openstack-mistral17:32
*** shardy has quit IRC17:45
*** thrash|f00dz is now known as thrash17:46
*** bobh has quit IRC18:37
*** bobh has joined #openstack-mistral18:37
*** harlowja has joined #openstack-mistral18:40
*** bobh has quit IRC18:42
*** AlexeyAbashkin has joined #openstack-mistral18:44
*** AlexeyAbashkin has quit IRC18:49
*** AlexeyAbashkin has joined #openstack-mistral18:49
*** AlexeyAbashkin has quit IRC18:59
*** openstackgerrit has quit IRC19:05
*** hardikjasani has quit IRC19:06
*** bobh has joined #openstack-mistral19:13
*** AlexeyAbashkin has joined #openstack-mistral20:41
*** AlexeyAbashkin has quit IRC20:45
*** harlowja has quit IRC20:55
*** openstackgerrit has joined #openstack-mistral21:35
openstackgerritAdriano Petrich proposed openstack/python-mistralclient master: Do not let keystoneauth mask the errors  https://review.openstack.org/56173221:35
*** kong is now known as lxkong21:45
openstackgerritAdriano Petrich proposed openstack/python-mistralclient master: Add the force parameter to delete executions  https://review.openstack.org/56115923:09
*** bobh has quit IRC23:14
*** jaosorior has quit IRC23:26
*** jaosorior has joined #openstack-mistral23:27
*** jaosorior has quit IRC23:34
*** jaosorior has joined #openstack-mistral23:34
*** d0ugal has quit IRC23:36
*** d0ugal has joined #openstack-mistral23:53

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