Thursday, 2018-02-22

*** d0ugal has quit IRC00:47
*** d0ugal has joined #openstack-mistral00:58
*** daidv has joined #openstack-mistral00:58
*** yangyapeng has quit IRC01:33
*** yangyapeng has joined #openstack-mistral01:33
*** yangyapeng has quit IRC01:38
*** ykarel|afk has joined #openstack-mistral01:52
*** itlinux has joined #openstack-mistral02:18
*** ykarel|afk has quit IRC02:31
*** zhenguo has joined #openstack-mistral02:32
*** itlinux has quit IRC03:10
*** thrash is now known as thrash|g0ne03:28
*** itlinux has joined #openstack-mistral03:37
*** d0ugal has quit IRC04:00
*** gkadam has quit IRC04:01
*** gkadam has joined #openstack-mistral04:02
*** ykarel|afk has joined #openstack-mistral04:04
*** kushalAgrawal has joined #openstack-mistral04:23
*** d0ugal has joined #openstack-mistral04:27
*** hardikjasani has joined #openstack-mistral04:31
*** gkadam has quit IRC04:41
*** zhenguo has quit IRC04:42
*** gkadam has joined #openstack-mistral04:51
*** ykarel|afk is now known as ykarel04:54
*** oidgar has joined #openstack-mistral05:32
*** oidgar has quit IRC05:51
*** jtomasek has joined #openstack-mistral06:47
*** jtomasek has quit IRC06:47
*** jtomasek has joined #openstack-mistral06:56
*** itlinux has quit IRC06:58
*** ykarel is now known as ykarel|lunch07:34
*** AlexeyAbashkin has joined #openstack-mistral08:07
*** threestrands_ has joined #openstack-mistral08:15
*** threestrands_ has quit IRC08:15
*** threestrands_ has joined #openstack-mistral08:15
*** threestrands has quit IRC08:18
*** ykarel|lunch is now known as ykarel08:28
*** jpich has joined #openstack-mistral09:02
*** oidgar has joined #openstack-mistral09:04
*** daidv has quit IRC09:14
*** daidv has joined #openstack-mistral09:14
*** threestrands_ has quit IRC09:37
*** jaosorior has joined #openstack-mistral09:39
*** gkadam_ has joined #openstack-mistral09:43
*** gkadam has quit IRC09:44
*** gkadam_ has quit IRC09:44
*** gkadam_ has joined #openstack-mistral09:44
*** jpich_ has joined #openstack-mistral10:03
*** jpich has quit IRC10:07
*** jpich_ is now known as jpich10:15
d0ugalrakhmerov: Hey10:16
d0ugalrakhmerov: just fyi, I am going to start writing up the schedule now: https://etherpad.openstack.org/p/mistral-ptg-rocky10:16
rakhmerovd0ugal: hi, great!10:30
rakhmerovlet me know if you need anything10:30
d0ugalOnly just starting :)10:30
rakhmerov:)10:30
d0ugalrakhmerov: will do. It would be good it you can check it in an hour or two when I have more and give me feedback.10:30
rakhmerovI'm at meetings all the time mostly but can do something if you need10:31
rakhmerovabsolutely, I will10:31
rakhmerovyes10:31
rakhmerovd0ugal: also we have 1 backporte dpatch in stable/queens :)10:31
rakhmerovso we can have RC2 )10:31
d0ugalk10:31
d0ugalI can do that now10:31
rakhmerovyep10:32
rakhmerovI'm not expecting anything else from me this week I think10:32
rakhmerovtoo busy with other things.10:32
rakhmerovtrying to work on https://bugs.launchpad.net/mistral/+bug/1714054 and investigated why it's broken but the fix is not ready yet10:32
openstackLaunchpad bug 1714054 in Mistral "tasks listed after pause do not run when workflow is resumed" [High,Confirmed] - Assigned to Renat Akhmerov (rakhmerov)10:32
rakhmerovseems like it was broken almost 2 years ago :)10:33
d0ugaloh dear10:33
rakhmerovyes10:33
d0ugalI don't think I have ever paused a task10:33
d0ugal:)10:33
rakhmerovbut this place is very tricky10:33
rakhmerovone thing is when we just pause a WF but it's much more hairy when we have a "pause" command within a on-* list10:34
rakhmerovit used to work but on a completely different architecture10:34
rakhmerovthen I made a total refactoring of the engine and broke it10:34
d0ugaloops :)10:35
rakhmerovit wasn't just covered with any unit tests (shame on me..)10:35
rakhmerovyep10:35
rakhmerovbtw, d0ugal, we still have those two patches that can't merge10:35
rakhmerovfrom OpenStack bot10:35
rakhmerovany idea what we can do with them?10:35
d0ugalIt sounds interesting, I look forward to seeing the fix.10:35
rakhmerovit's TripleO job10:35
d0ugalYeah, I noticed that. It is very strange.10:35
d0ugalI'll try looking agian10:36
rakhmerovyes, will try to deliver the fix soon10:36
d0ugalagain I do the release10:36
rakhmerovok10:36
d0ugaland maybe after the schedule :-D10:36
rakhmerovsure10:36
d0ugalToo many tasks :)10:36
*** gkadam__ has joined #openstack-mistral10:54
*** gkadam_ has quit IRC10:57
*** EmilienM has quit IRC10:59
*** EmilienM has joined #openstack-mistral11:00
*** akazakov has joined #openstack-mistral11:24
*** shardy has quit IRC11:42
*** jaosorior has quit IRC11:45
*** jaosorior has joined #openstack-mistral11:45
*** thrash|g0ne is now known as thrash12:10
*** akazakov has quit IRC12:52
*** hrybacki has quit IRC12:53
*** josecastroleon has joined #openstack-mistral12:54
*** hrybacki has joined #openstack-mistral12:54
*** josecastroleon has quit IRC13:00
*** josecastroleon has joined #openstack-mistral13:00
*** dprince has joined #openstack-mistral13:16
*** akazakov has joined #openstack-mistral13:16
*** hardikjasani has quit IRC13:31
*** yangyapeng has joined #openstack-mistral13:34
*** ykarel is now known as ykarel|away13:35
*** yangyapeng has quit IRC13:39
*** shardy has joined #openstack-mistral13:48
*** mwhahaha has quit IRC13:54
*** oidgar has quit IRC13:55
*** mwhahaha has joined #openstack-mistral13:56
*** oidgar has joined #openstack-mistral14:11
*** toure|gone is now known as toure14:16
*** openstackgerrit has joined #openstack-mistral14:25
openstackgerritAndras Kovi proposed openstack/mistral master: Hard code top package name  https://review.openstack.org/54700114:25
*** ykarel|away has quit IRC14:35
*** josecastroleon has quit IRC14:45
*** josecastroleon has joined #openstack-mistral14:46
*** josecastroleon has quit IRC14:53
*** gkadam__ has quit IRC14:53
*** rbrady has quit IRC14:56
*** josecastroleon has joined #openstack-mistral14:58
*** ykarel|away has joined #openstack-mistral14:59
toured0ugal ping15:07
d0ugaltoure: Hey15:12
toured0ugal quick question in regards to mistral-extra and zuul gate test.15:13
d0ugaltoure: sure15:14
toureshould there be test added for the new port of ansible actions15:14
toureor should I place that in tripleo-common?15:14
*** rbrady has joined #openstack-mistral15:14
d0ugaltoure: ideally there should be a test in mistral itself15:14
toureack15:14
d0ugalbut testing via tripleo is better than nothing15:15
d0ugalWe can always add tests to mistral-extra afterwards15:15
tourecool, let me look at mistral to see what in there15:15
d0ugalI wont be a huge issue initially, but in the future if we support ansible and 20 other external integrations then having them all tested will be key15:15
toure+115:16
d0ugaltoure: have you had any thoughts about how we will handle the recent additions to the tripleo version? i.e. the zaqar support.15:17
d0ugaltoure: https://github.com/openstack/tripleo-common/blob/master/tripleo_common/actions/ansible.py#L470-L49815:17
* toure looks15:18
d0ugaltoure: ideally we don't want the ansible action to depend on zaqar :) but we also want to make it possible to extend it.15:18
toureright, so my port just shifted the action to a new repo and keep the tripleo messaging deps in tack15:19
toureoops15:20
d0ugaltoure: I think your version is older15:20
toureyeah15:20
* toure will refactor15:20
toure:(15:20
d0ugaltoure: yeah, it will be a bit of a pain15:21
d0ugalthrash: any idea what is up with the patch and CI failures? https://review.openstack.org/#/c/544318/15:21
thrashd0ugal: looking15:22
d0ugalIt has been rechecked and rebased a few times :)15:22
d0ugalthrash: the patch based on it passes fine: https://review.openstack.org/#/c/544319/215:22
thrashd0ugal: the tripleo failure looks like a repo issue15:22
d0ugaloh, no tripleo on that patch15:23
d0ugalweird15:23
thrashthere should be....15:23
thrashoh. maybe not. The tripleo job might ignore tox.ini changes.15:23
thrashWhich makes sense.15:23
d0ugalsort-of15:23
d0ugalI'd rather it ignored .gitreview files :)15:24
d0ugalokay, so maybe CI is bust for queens? is that because we don't have a release yet?15:24
thrashperhaps?15:24
* d0ugal asks in #tripleo15:25
d0ugalthrash: I gave us rechecking :)15:25
d0ugal10th time lucky?15:25
thrashhehe.. Why not.15:25
d0ugalthrash: yeah, so, failing due to the lack of branches it seems.15:31
thrashI had that same problem with the client zuul backport.15:31
d0ugalso if we need to backport anything we will have to disable tripleo on stable/queens15:31
thrashthat's easy enough. We could just turn the job to non-voting15:32
d0ugalyeah15:32
d0ugalI might just do that now to land those two patches actually15:32
thrashin .zuul.yaml, then revert that change when it works15:32
thrashdo it in a separate change to make it easier to revert15:32
d0ugalyup15:33
d0ugalthrash: looks okay? https://review.openstack.org/#/c/547032/15:36
thrashd0ugal: aside from the indentation... :)15:37
thrash4 spaces please.15:37
d0ugalthrash: it seems to increase by two in other places?15:38
thrashyou have 3.15:38
d0ugaloh yeah15:38
d0ugallol15:38
thrashor 1 depending on where you are looking15:38
thrash:D15:38
d0ugalFine, I'll round up15:39
openstackgerritDougal Matthews proposed openstack/mistral master: Temporarily disable TripleO CI in stable/queens  https://review.openstack.org/54703315:39
d0ugalgah15:39
d0ugaland that is why we need the .gitreview patch15:40
thrashlol15:40
*** josecastroleon has quit IRC15:46
*** itlinux has joined #openstack-mistral16:03
*** oidgar has quit IRC16:35
*** d0ugal has quit IRC16:35
*** d0ugal has joined #openstack-mistral16:41
*** oidgar has joined #openstack-mistral16:50
openstackgerritAndras Kovi proposed openstack/mistral master: Ensure workflow check is retried on DB error  https://review.openstack.org/54706917:03
*** jpich has quit IRC17:04
*** AlexeyAbashkin has quit IRC17:09
*** dprince has quit IRC17:28
*** dprince has joined #openstack-mistral17:55
*** oidgar has quit IRC18:14
*** ykarel|away has quit IRC18:16
*** openstackgerrit has quit IRC18:33
*** AlexeyAbashkin has joined #openstack-mistral18:36
*** AlexeyAbashkin has quit IRC18:40
*** openstackgerrit has joined #openstack-mistral18:55
openstackgerritMerged openstack/mistral stable/queens: Update .gitreview for stable/queens  https://review.openstack.org/54431818:55
openstackgerritMerged openstack/mistral stable/queens: Update UPPER_CONSTRAINTS_FILE for stable/queens  https://review.openstack.org/54431918:55
*** dprince has quit IRC20:27
d0ugalrakhmerov: wtf. https://review.openstack.org/#/c/467799/20:29
*** dprince has joined #openstack-mistral21:18
*** jtomasek has quit IRC21:27
*** thrash is now known as thrash|g0ne22:19
*** itlinux has quit IRC22:30
*** dprince has quit IRC22:33
*** apetrich has quit IRC23:25
*** itlinux has joined #openstack-mistral23:29

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