| openstackgerrit | Merged openstack/mistral master: Drop pyflakes from the test requirements https://review.openstack.org/515362 | 00:08 |
|---|---|---|
| *** yangyape_ has quit IRC | 00:13 | |
| *** yangyapeng has joined #openstack-mistral | 00:13 | |
| *** yangyapeng has quit IRC | 00:18 | |
| *** yangyapeng has joined #openstack-mistral | 00:21 | |
| *** bobh has quit IRC | 00:21 | |
| *** yangyapeng has quit IRC | 00:25 | |
| *** bobh has joined #openstack-mistral | 00:25 | |
| *** bobh has quit IRC | 00:29 | |
| *** zhurong has joined #openstack-mistral | 00:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral master: Updated from global requirements https://review.openstack.org/517481 | 00:58 |
| *** AlexeyAbashkin has joined #openstack-mistral | 01:09 | |
| *** AlexeyAbashkin has quit IRC | 01:13 | |
| *** yangyapeng has joined #openstack-mistral | 01:27 | |
| *** yangyapeng has quit IRC | 01:29 | |
| *** yangyapeng has joined #openstack-mistral | 01:29 | |
| *** yangyape_ has joined #openstack-mistral | 01:31 | |
| *** yangyapeng has quit IRC | 01:31 | |
| *** yangyapeng has joined #openstack-mistral | 01:51 | |
| *** yangyape_ has quit IRC | 01:52 | |
| *** zhurong has quit IRC | 02:30 | |
| *** yangyapeng has quit IRC | 02:37 | |
| *** yangyapeng has joined #openstack-mistral | 02:38 | |
| *** zhurong has joined #openstack-mistral | 02:39 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 03:10 | |
| *** AlexeyAbashkin has quit IRC | 03:10 | |
| *** yangyapeng has quit IRC | 03:11 | |
| *** yangyapeng has joined #openstack-mistral | 03:11 | |
| *** yangyapeng has quit IRC | 03:45 | |
| *** yangyapeng has joined #openstack-mistral | 03:46 | |
| openstackgerrit | Winson Chan proposed openstack/mistral master: Implement notification of execution events https://review.openstack.org/467799 | 04:15 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Re-raise DB errors when evaluating expressions https://review.openstack.org/517277 | 04:18 |
| openstackgerrit | Winson Chan proposed openstack/mistral master: Implement notification of execution events https://review.openstack.org/467799 | 04:23 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Re-raise DB errors when evaluating expressions https://review.openstack.org/517277 | 04:24 |
| rakhmerov | d0ugal: good morning, can you please review https://review.openstack.org/#/c/517427/ and https://review.openstack.org/#/c/517277/ ? | 04:39 |
| rakhmerov | it's quite urgent for us | 04:39 |
| *** zhurong has quit IRC | 04:43 | |
| openstackgerrit | Merged openstack/mistral master: [API] Support project_id filter in cron_triggers API https://review.openstack.org/516668 | 05:06 |
| openstackgerrit | Merged openstack/mistral master: [API] Support get/delete cron triggers in any projects for admin https://review.openstack.org/516692 | 05:13 |
| openstackgerrit | Merged openstack/mistral master: Updated from global requirements https://review.openstack.org/517481 | 05:23 |
| *** zhurong has joined #openstack-mistral | 05:57 | |
| *** jaosorior has joined #openstack-mistral | 06:46 | |
| *** jtomasek has joined #openstack-mistral | 06:50 | |
| d0ugal | rakhmerov: done. | 07:32 |
| d0ugal | rakhmerov: it would be great if I could get more eys on some of my patches, I can never really find reviews :( | 07:32 |
| rakhmerov | d0ugal: hey | 07:33 |
| rakhmerov | find reviews? What do you mean? | 07:33 |
| rakhmerov | d0ugal: please this too: https://review.openstack.org/#/c/517277/ | 07:33 |
| d0ugal | We need more reviewers :) | 07:33 |
| rakhmerov | ooh, that thing.. | 07:34 |
| rakhmerov | yes | 07:34 |
| rakhmerov | d0ugal: btw, please pay attention that some of your patches are in -1 state | 07:34 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Add a json_dump expression function https://review.openstack.org/514589 | 07:35 |
| rakhmerov | basically, I see only two that I +2ed and that are not in -1 | 07:35 |
| d0ugal | rakhmerov: sure, I can recheck a bunch of them. I believe they are mostly due to CI issues | 07:35 |
| rakhmerov | please do, yes | 07:35 |
| *** zhurong has quit IRC | 07:36 | |
| d0ugal | but getting a second reviewer is difficult. | 07:36 |
| rakhmerov | we have Andras and Nikolay but both are not in IRC | 07:37 |
| d0ugal | That is the problem :) | 07:37 |
| rakhmerov | I'll ask them to be here as well | 07:37 |
| d0ugal | or part of the problem I guess | 07:38 |
| rakhmerov | I'm hoping to see apetrich as a core reviewer soon ;)) | 07:38 |
| apetrich | :) | 07:38 |
| d0ugal | rakhmerov: btw did you see the email: "Regarding Queens Community goal: Tempest Plugin Split" | 07:38 |
| d0ugal | rakhmerov: me too! (re: apetrich!) | 07:39 |
| rakhmerov | yep ) | 07:41 |
| rakhmerov | d0ugal: ooh, email no | 07:41 |
| rakhmerov | I've not been following ML for a while ) | 07:42 |
| d0ugal | rakhmerov: chandankumar sent it to just me and you. To your gmail iirc | 07:42 |
| rakhmerov | really? I think I missed it then somehow.. | 07:42 |
| rakhmerov | or simply forgot | 07:42 |
| rakhmerov | what's the essense? | 07:42 |
| d0ugal | rakhmerov: oh, I think he got your email wrong. | 07:43 |
| rakhmerov | main poing | 07:43 |
| rakhmerov | point | 07:43 |
| d0ugal | actually, I think it is correct. I'll forward it anyway | 07:43 |
| rakhmerov | ok | 07:43 |
| d0ugal | rakhmerov: https://governance.openstack.org/tc/goals/queens/split-tempest-plugins.html | 07:43 |
| d0ugal | rakhmerov: chandankumar is offering to do the work to split mistral's tempest tests out from the main repo | 07:43 |
| d0ugal | as per that queens goal | 07:44 |
| d0ugal | so we would have mistral-tempest-plugin | 07:44 |
| d0ugal | and chandankumar wanted to check it was okay and that nobody else had started | 07:44 |
| rakhmerov | d0ugal: well, yes, there was already the patch that decoupled mistral tempest tests from the main source code (don't remember though if we merged it yet) | 07:44 |
| d0ugal | oh? I didn't see that. | 07:45 |
| d0ugal | I thought they were already decoupled | 07:45 |
| rakhmerov | wait a sec.. | 07:46 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add default configuration files to data_files https://review.openstack.org/507774 | 07:47 |
| rakhmerov | d0ugal: https://review.openstack.org/#/c/500344/ | 07:47 |
| rakhmerov | it mostly does the job | 07:47 |
| rakhmerov | after it's merged all that's left is to create a new repo and move the test over there | 07:48 |
| rakhmerov | the tests.. | 07:48 |
| rakhmerov | so you can just review it as the first step | 07:48 |
| d0ugal | Right | 07:48 |
| d0ugal | oh I remember that patch now, I missed the reply to my questions. | 07:50 |
| rakhmerov | np | 07:50 |
| rakhmerov | I'll reply to his email now | 07:51 |
| d0ugal | btw, I noticed we have a bunch of broken actions yesterday | 07:54 |
| rakhmerov | OpenStack actions? | 07:55 |
| d0ugal | Yeah | 07:55 |
| rakhmerov | hm.. | 07:55 |
| rakhmerov | probably new client versions? | 07:55 |
| d0ugal | A number of the nova actions are deprecated and removed from novaclient | 07:55 |
| rakhmerov | ooh yeah | 07:55 |
| rakhmerov | I see | 07:55 |
| d0ugal | and I think in other projects too | 07:55 |
| d0ugal | I want to try and setup something to automatically check | 07:56 |
| d0ugal | rakhmerov: https://review.openstack.org/#/c/515749/ | 07:59 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Pass the new ActionContext to mistral-lib https://review.openstack.org/506185 | 08:01 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Use the new action context in MistralHTTPAction https://review.openstack.org/505508 | 08:01 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Remove the __init__ method from the test action https://review.openstack.org/513061 | 08:01 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Rename task_id to task_execution_id https://review.openstack.org/514688 | 08:01 |
| rakhmerov | d0ugal: ok | 08:01 |
| *** AlexeyAbashkin has joined #openstack-mistral | 08:06 | |
| *** tung_doan has quit IRC | 08:07 | |
| *** gkadam has joined #openstack-mistral | 08:17 | |
| rakhmerov | d0ugal: woow, this is pretty serious ) | 08:17 |
| d0ugal | rakhmerov: the mistral-lib patch? | 08:18 |
| rakhmerov | yep | 08:18 |
| d0ugal | It is and it isn't - mistral isn't passing the new context yet. | 08:18 |
| rakhmerov | anyway, I somehow overlooked it too | 08:18 |
| d0ugal | so I found it in time :) | 08:18 |
| rakhmerov | yes | 08:18 |
| d0ugal | but, I did feel embarrassed when I did find it :/ | 08:19 |
| rakhmerov | :)) haha | 08:20 |
| apetrich | and still no promotion :( | 08:28 |
| apetrich | we were so close on the 1st | 08:28 |
| d0ugal | :( | 08:47 |
| d0ugal | rakhmerov: btw this series of patches fail because I need the mistral-lib patches to merge and be released: https://review.openstack.org/#/c/506185/ | 09:04 |
| d0ugal | so I shouldn't have rechecked them really | 09:05 |
| * d0ugal will bbiab | 09:05 | |
| *** jpich has joined #openstack-mistral | 09:12 | |
| *** jtomasek has quit IRC | 09:15 | |
| *** jtomasek has joined #openstack-mistral | 09:24 | |
| rakhmerov | d0ugal: I see | 09:26 |
| openstackgerrit | Merged openstack/mistral master: Do not parse updated_at for task if it was not updated https://review.openstack.org/517427 | 09:32 |
| openstackgerrit | Merged openstack/mistral master: Re-raise DB errors when evaluating expressions https://review.openstack.org/517277 | 09:42 |
| *** yangyapeng has quit IRC | 09:52 | |
| openstackgerrit | Merged openstack/mistral master: Decoupling of Mistral tempest test from Mistral code base https://review.openstack.org/500344 | 09:59 |
| *** daidv has quit IRC | 10:10 | |
| *** mcdoker181818 has joined #openstack-mistral | 11:03 | |
| mcdoker181818 | d0ugal: Hi. https://review.openstack.org/#/c/516575/3 I don't know how do merge this fix to the pike branch. Can you explain? Maybe link? Thanks) | 11:09 |
| d0ugal | mcdoker181818: in gerrit, click "cherry pick" and enter stable/pike | 11:11 |
| mcdoker181818 | Thanks again) https://review.openstack.org/#/c/517604/ | 11:13 |
| d0ugal | mcdoker181818: I can't merge pike patches btw | 11:19 |
| d0ugal | so you need rakhmerov for that | 11:20 |
| mcdoker181818 | Okay, I will bother him :D | 11:30 |
| *** mcdoker181818 has quit IRC | 11:38 | |
| *** katkapilatova1 has joined #openstack-mistral | 11:59 | |
| *** katkapilatova1 has quit IRC | 12:00 | |
| *** zhenguo has quit IRC | 12:05 | |
| *** gkadam has quit IRC | 12:47 | |
| *** toure_biab is now known as toure | 12:51 | |
| *** Dinesh_Bhor has quit IRC | 13:08 | |
| *** bobh has joined #openstack-mistral | 13:50 | |
| *** jkilpatr has quit IRC | 13:52 | |
| *** mcdoker181818 has joined #openstack-mistral | 14:06 | |
| *** jkilpatr has joined #openstack-mistral | 14:12 | |
| *** yangyapeng has joined #openstack-mistral | 14:40 | |
| *** jkilpatr has quit IRC | 15:01 | |
| *** jkilpatr has joined #openstack-mistral | 15:15 | |
| *** jaosorior has quit IRC | 15:27 | |
| *** AlexeyAbashkin has quit IRC | 15:53 | |
| *** livelace-link has joined #openstack-mistral | 16:12 | |
| *** livelace-link has quit IRC | 16:19 | |
| *** livelace-link has joined #openstack-mistral | 16:21 | |
| *** jtomasek has quit IRC | 17:12 | |
| *** jkilpatr has quit IRC | 17:16 | |
| *** jtomasek has joined #openstack-mistral | 17:25 | |
| *** jkilpatr has joined #openstack-mistral | 17:28 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 17:45 | |
| *** AlexeyAbashkin has quit IRC | 17:50 | |
| *** jpich has quit IRC | 17:58 | |
| *** weshay is now known as weshay_brb | 18:30 | |
| *** weshay_brb is now known as weshay | 19:20 | |
| *** jrist has quit IRC | 19:25 | |
| *** jrist has joined #openstack-mistral | 19:36 | |
| *** jtomasek has quit IRC | 19:38 | |
| *** jrist_ has joined #openstack-mistral | 19:49 | |
| *** jrist_ has quit IRC | 19:49 | |
| *** jrist has quit IRC | 19:49 | |
| *** jrist has joined #openstack-mistral | 19:50 | |
| *** jrist has quit IRC | 19:50 | |
| *** jrist has joined #openstack-mistral | 19:50 | |
| *** mcdoker181818 has quit IRC | 20:45 | |
| *** mcdoker181818 has joined #openstack-mistral | 20:54 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 21:06 | |
| *** AlexeyAbashkin has quit IRC | 21:10 | |
| *** d0ugal has quit IRC | 22:06 | |
| *** d0ugal has joined #openstack-mistral | 22:10 | |
| *** jrist has quit IRC | 22:11 | |
| *** harlowja has quit IRC | 22:12 | |
| *** jrist has joined #openstack-mistral | 22:15 | |
| *** jrist has quit IRC | 22:15 | |
| *** jrist has joined #openstack-mistral | 22:15 | |
| *** thrash is now known as thrash|g0ne | 22:24 | |
| *** bobh has quit IRC | 22:47 | |
| *** harlowja has joined #openstack-mistral | 23:04 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 23:07 | |
| *** AlexeyAbashkin has quit IRC | 23:11 | |
| *** toure is now known as toure_biab | 23:15 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!