| *** harlowja has quit IRC | 00:46 | |
| *** bobh has joined #openstack-mistral | 01:56 | |
| *** sulvone5WEXU3 has joined #openstack-mistral | 02:25 | |
| *** sulvone5WEXU3 has quit IRC | 02:31 | |
| *** jaosorior has quit IRC | 02:42 | |
| *** jaosorior has joined #openstack-mistral | 02:57 | |
| *** threestrands has joined #openstack-mistral | 02:59 | |
| *** zhurong has joined #openstack-mistral | 02:59 | |
| *** bobh has quit IRC | 03:04 | |
| *** bobh has joined #openstack-mistral | 03:22 | |
| *** bobh has quit IRC | 03:23 | |
| *** bobh has joined #openstack-mistral | 03:23 | |
| *** bobh has quit IRC | 03:28 | |
| *** hardikjasani has joined #openstack-mistral | 04:18 | |
| rakhmerov | mcdoker181818, d0ugal: no-no, there will be a single object in this case | 04:22 |
|---|---|---|
| *** zhurong has quit IRC | 04:24 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/queens: Add '__task_execution' structure to task execution context on the fly https://review.openstack.org/562929 | 04:29 |
| rakhmerov | d0ugal: hi, is it ok to approve the backport https://review.openstack.org/562929? | 04:58 |
| *** jaosorior_ has joined #openstack-mistral | 05:17 | |
| *** jaosorior has quit IRC | 05:21 | |
| *** jaosorior_ is now known as jaosorior | 05:48 | |
| *** shardy has joined #openstack-mistral | 06:29 | |
| *** pgaxatte has joined #openstack-mistral | 06:31 | |
| *** pgaxatte has left #openstack-mistral | 06:41 | |
| *** pgaxatte has joined #openstack-mistral | 06:42 | |
| apetrich | d0ugal, how was it that we run mistral-tempest-plugin? | 06:51 |
| rakhmerov | apetrich: hi, just curious. What's on mistral-tempest-plugin? Can you give a context? | 06:56 |
| apetrich | rakhmerov, yeah as if we didn't have enough issues we decided that we needed a chicken and egg problem so the *-tempest-plugin were created | 06:58 |
| apetrich | rakhmerov, we removed the tempest tests from our source code and moved them into a different repo | 06:58 |
| apetrich | CI when does the tempest uses those repos | 06:58 |
| rakhmerov | yes, so? | 06:58 |
| rakhmerov | any problem with that? | 06:58 |
| apetrich | so I have a change that in mistral. (the force delete of finished execs) tempest is failing because the tests try to clean the execs they created and that failed for lacking of the force param. | 07:00 |
| *** threestrands has quit IRC | 07:00 | |
| apetrich | so I need to create a change to mistral-tempest-plugin make my patch depends on that so my tests pass CI so it can be merged so that later the change to the mistral-tempest-plyugin can be merged | 07:01 |
| apetrich | that is the opposite of the order of an usual depends-on works | 07:02 |
| apetrich | or I do the other way around and make the m-t-p change depends on mistral change but the mistral change will never pass CI | 07:02 |
| apetrich | but right now I just want to test in live env if my change to m-t-p is working | 07:03 |
| apetrich | rakhmerov, ^^ | 07:03 |
| rakhmerov | hm.. | 07:06 |
| rakhmerov | ok | 07:06 |
| *** shardy is now known as shardy_afk | 07:09 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 07:37 | |
| d0ugal | rakhmerov: sure | 07:41 |
| rakhmerov | ok | 07:41 |
| d0ugal | rakhmerov: we also need the release note patch | 07:41 |
| rakhmerov | yes, forgot about it | 07:41 |
| rakhmerov | thanks | 07:41 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/queens: Release note for not persisting '__task_execution' in DB https://review.openstack.org/562967 | 07:42 |
| d0ugal | thanks | 07:42 |
| d0ugal | apetrich: what requires a tempest change? | 07:42 |
| d0ugal | apetrich: technically you shouldn't ever need to change something in tempest, unless it turns out we were testing the incorrect behaviour | 07:42 |
| apetrich | d0ugal, http://logs.openstack.org/02/560802/4/check/mistral-devstack/386313e/job-output.txt.gz#_2018-04-19_20_06_51_391961 | 07:43 |
| d0ugal | Looking | 07:43 |
| apetrich | the new behaviour is crashing with the tearDown | 07:43 |
| d0ugal | apetrich: aha, so in this case we were indeed testing the bad behaviour :) | 07:43 |
| apetrich | the new normal | 07:43 |
| d0ugal | apetrich: I would say the "correct" fix is to have tempest wait for executions to complete before they are deleted. | 07:45 |
| apetrich | d0ugal, some of them are just Delayed | 07:45 |
| apetrich | but that is besides the point I'm just trying to get it tempest to run on my live env | 07:46 |
| apetrich | and failing at that :) | 07:46 |
| d0ugal | apetrich: oh, I see | 07:46 |
| d0ugal | apetrich: I forget how you do that, it is kinda hacky | 07:46 |
| apetrich | crap :) that was what I needed | 07:46 |
| d0ugal | apetrich: tripleo does it, you could find what they do in quickstart | 07:47 |
| d0ugal | might be easiest since you are familiar with it | 07:47 |
| apetrich | oh true | 07:47 |
| apetrich | d0ugal, cheers | 07:47 |
| *** jpich has joined #openstack-mistral | 07:50 | |
| d0ugal | #startmeeting mistral | 08:00 |
| openstack | Meeting started Fri Apr 20 08:00:06 2018 UTC and is due to finish in 60 minutes. The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:00 |
| *** openstack changes topic to " (Meeting topic: mistral)" | 08:00 | |
| openstack | The meeting name has been set to 'mistral' | 08:00 |
| d0ugal | It is the Friday office hour! | 08:00 |
| d0ugal | yay! | 08:00 |
| d0ugal | d0ugal,rakhmerov,apetrich,bobh, mcdoker18: ping :) | 08:01 |
| d0ugal | https://etherpad.openstack.org/p/mistral-office-hours | 08:01 |
| apetrich | here we are | 08:01 |
| rakhmerov | d0ugal: yep, here | 08:01 |
| d0ugal | So, the only interesting news I have is that Mistral R-1 has been released! | 08:02 |
| rakhmerov | d0ugal: cool ) | 08:02 |
| d0ugal | mistral-lib and python-mistralclient releases will happen soon, I made a mistake in the original patches. | 08:03 |
| d0ugal | With R-1 out, I am going to spend this hour moving all the R-1 bugs to R-2 | 08:03 |
| d0ugal | and if I get time after that I am going to try and do some triage | 08:03 |
| apetrich | nice | 08:03 |
| rakhmerov | yep | 08:04 |
| d0ugal | I made a copy/paste fail - I tried to release mistral-lib with a mistralclient sha and a mistralclient release with a mistral-extra sha :/ | 08:05 |
| d0ugal | Thankfully the tools block me. | 08:05 |
| d0ugal | If anyone has anything to discuss, do let me know - there is time for that. | 08:06 |
| d0ugal | My goal with the office hours is to take care of project/PTL duties so that I am here and active, but if somebody else is here and wants to chat then that is more important. | 08:06 |
| rakhmerov | d0ugal: I don't have much, you've probably seen my memory/performance improvements that have already been merged. That's all I have. Plus I'm investigating other issues now that we're seeing in our env | 08:07 |
| d0ugal | rakhmerov: other performance issues? | 08:07 |
| rakhmerov | well, yes | 08:07 |
| rakhmerov | I can tell briefly | 08:07 |
| d0ugal | Sure, if you think they would be interesting | 08:08 |
| rakhmerov | one thing is that we do deletion of resources (wf execs and defs) and for some reason get huge memory footprint during that | 08:08 |
| rakhmerov | I have no idea yet why this happens but it's consistently reproducible | 08:08 |
| rakhmerov | the other thing is that we run a workflow with some subworkflows, so totally we have 4-5 wf executions | 08:09 |
| d0ugal | rakhmerov: when deleting an object, do we load the full model from the database? | 08:09 |
| rakhmerov | BUT the internal caches that cache WF specifications by executions get full, all 100 entries | 08:09 |
| rakhmerov | which is I'm sure a bug | 08:09 |
| rakhmerov | d0ugal: no, we shouldn't. We just do a delete query | 08:10 |
| rakhmerov | I'll check that anyway | 08:10 |
| d0ugal | I guess we don't, but I can't think why else memory usage would be high | 08:10 |
| rakhmerov | d0ugal: yeah, me too | 08:10 |
| d0ugal | I'm very interested to hear what you find. | 08:11 |
| rakhmerov | yep | 08:11 |
| rakhmerov | I need some more time though, hopefully today I'll have more info | 08:11 |
| rakhmerov | because I can't reproduce those issues locally on my mac | 08:12 |
| rakhmerov | only in our distributed env in the lab | 08:12 |
| rakhmerov | that's all I have I guess ) | 08:12 |
| rakhmerov | keep working.. | 08:12 |
| d0ugal | Thanks for sharing :) | 08:12 |
| d0ugal | So in R-1 we fixed 13 bugs, but we have 8 in progress and 25 more triaged | 08:13 |
| rakhmerov | but https://review.openstack.org/#/c/562554 makes a really big difference memory wise ) | 08:13 |
| rakhmerov | d0ugal: ok | 08:13 |
| rakhmerov | let me take a look at LP | 08:13 |
| d0ugal | R-2 already has 15 bugs assigned. We either need to reduce the bug for R-2 or solve lots of things :) | 08:14 |
| d0ugal | s/solve/fix/ | 08:14 |
| rakhmerov | :) | 08:14 |
| rakhmerov | so looking at R-1.. | 08:14 |
| rakhmerov | https://bugs.launchpad.net/mistral/+bug/1757966 | 08:14 |
| openstack | Launchpad bug 1757966 in Mistral "Mistral server consumes more memory over time" [Critical,In progress] - Assigned to Renat Akhmerov (rakhmerov) | 08:14 |
| rakhmerov | my latest patches were addressing it | 08:15 |
| rakhmerov | but I'd prefer not to close it yet and do some more investigation | 08:15 |
| d0ugal | Sure | 08:15 |
| rakhmerov | so let's move it to R-2 | 08:15 |
| rakhmerov | ok for you? | 08:15 |
| rakhmerov | ok | 08:15 |
| rakhmerov | https://bugs.launchpad.net/mistral/+bug/1720797 | 08:16 |
| openstack | Launchpad bug 1720797 in Mistral "Migrate Mistral CI jobs to new Zuul v3" [High,In progress] - Assigned to Brad P. Crochet (brad-9) | 08:16 |
| rakhmerov | I thought this was done, no? :) | 08:16 |
| d0ugal | There has been some recent activity there | 08:16 |
| rakhmerov | ok | 08:16 |
| d0ugal | I thought... let me check. | 08:16 |
| d0ugal | https://review.openstack.org/#/c/509184/ | 08:16 |
| d0ugal | so it seems that mistralclient isn't finished. | 08:16 |
| rakhmerov | ooh, yeah | 08:17 |
| rakhmerov | ok | 08:17 |
| rakhmerov | d0ugal: I'll step out for 5 mins.. Sorry. | 08:17 |
| d0ugal | rakhmerov: np | 08:17 |
| rakhmerov | i'm here | 08:23 |
| d0ugal | apetrich: Did you ever hear more from Winson about documentation for the execution events? | 08:24 |
| apetrich | d0ugal, not really | 08:24 |
| d0ugal | apetrich: oops | 08:24 |
| d0ugal | rakhmerov: ^ that question was for you? | 08:24 |
| rakhmerov | nope | 08:24 |
| d0ugal | :) | 08:24 |
| rakhmerov | i can ping him | 08:24 |
| d0ugal | Thanks | 08:25 |
| d0ugal | Even if he could provide something rough we can then polish that would be useful | 08:25 |
| rakhmerov | yes | 08:25 |
| rakhmerov | sent an email to him.. | 08:27 |
| d0ugal | Thanks! | 08:27 |
| rakhmerov | yeah, a number of bugs for R-2 is impressive ) | 08:29 |
| d0ugal | Yeah :/ | 08:29 |
| rakhmerov | https://bugs.launchpad.net/mistral/+bug/1664030 is marked as Fixed | 08:29 |
| openstack | Launchpad bug 1664030 in Mistral "docker image build crashing" [High,Fix released] - Assigned to Brad P. Crochet (brad-9) | 08:29 |
| d0ugal | At some point I am going to go through that list and remove some | 08:29 |
| rakhmerov | why is it in R-2? | 08:29 |
| d0ugal | rakhmerov: oh, my mistake. | 08:30 |
| *** gkadam has joined #openstack-mistral | 08:30 | |
| d0ugal | Moved it back | 08:30 |
| d0ugal | I didn't realise thrash|g0ne had opened a new bug. | 08:30 |
| d0ugal | https://bugs.launchpad.net/mistral/+bug/1765421 | 08:31 |
| openstack | Launchpad bug 1765421 in Mistral "docker image build job not working" [Undecided,In progress] - Assigned to Brad P. Crochet (brad-9) | 08:31 |
| d0ugal | That is the new one, I'll mark that for R-2 | 08:31 |
| rakhmerov | ok | 08:31 |
| d0ugal | Okay, I have moved everything that wasn't complete to R-2 | 08:32 |
| d0ugal | I am going to send an email to the list asking people to check the bugs and blueprints that they care about | 08:32 |
| d0ugal | and make sure they are all correctly assigned | 08:32 |
| rakhmerov | ok | 08:33 |
| rakhmerov | I can confirm that I should be able to complete those that are assigned to me | 08:34 |
| d0ugal | Great, thanks | 08:34 |
| d0ugal | rakhmerov: the blueprints also? :) | 08:34 |
| rakhmerov | and their statues are up to date | 08:35 |
| rakhmerov | on BPs, let me check.. | 08:35 |
| rakhmerov | https://blueprints.launchpad.net/mistral/+spec/mistral-async-workflows probably won't be in my focus | 08:35 |
| rakhmerov | I'd move it to R-3 | 08:36 |
| d0ugal | Sure | 08:36 |
| rakhmerov | I want to get it done in Rocky, but not in R-2 | 08:36 |
| rakhmerov | I'll be working on Scheduler | 08:36 |
| d0ugal | Great | 08:36 |
| rakhmerov | (not even sure if we have a ticket for that) | 08:36 |
| rakhmerov | yes, as we talked at the PTG | 08:36 |
| rakhmerov | and some docs that we also discussed | 08:36 |
| d0ugal | Good point. We should open a bug/blueprint - whichever you think is most sensible. | 08:36 |
| rakhmerov | ok, I'll do | 08:37 |
| rakhmerov | #action rakhmerov: open a ticket for the Scheduler refactoring work | 08:37 |
| rakhmerov | https://blueprints.launchpad.net/mistral/+spec/mistral-readonly-db-transactions | 08:37 |
| rakhmerov | this one btw is a low hanging fruit that could be given to someone who's getting up to speed | 08:38 |
| rakhmerov | fairly easy but also internally (code wise) useful | 08:38 |
| d0ugal | rakhmerov: https://review.openstack.org/#/c/529513/ | 08:38 |
| d0ugal | it is completed alreay? :) | 08:39 |
| d0ugal | Or is there something else we need to do? | 08:39 |
| rakhmerov | Hah! | 08:39 |
| d0ugal | Looks like apetrich done it in Queens! | 08:39 |
| rakhmerov | looking... | 08:39 |
| rakhmerov | I even approved it :)) | 08:39 |
| d0ugal | haha | 08:39 |
| apetrich | lol | 08:39 |
| d0ugal | We are good at this. | 08:39 |
| d0ugal | :P | 08:39 |
| rakhmerov | ok, but then the status is not up to date :) (who to blame? :)) ) | 08:39 |
| d0ugal | everything in this meeting :-D | 08:40 |
| d0ugal | everyone* /me is tired | 08:40 |
| apetrich | probably me. I must have forgotten to add the blueprint for the automation to pick it up | 08:40 |
| rakhmerov | ok, np ) | 08:40 |
| d0ugal | apetrich: blueprints are not automatically closed AFAIK | 08:40 |
| apetrich | really? I assumed they were | 08:40 |
| rakhmerov | let's mark it Implemented though? I don't we we can move it back to a closed milestone | 08:41 |
| d0ugal | and I guess when you did it you didn't have permission on launchpad | 08:41 |
| d0ugal | I'll update it | 08:41 |
| apetrich | oh that's true | 08:41 |
| d0ugal | I'll just have to leave it as rocky-1 | 08:41 |
| d0ugal | apetrich: so now you can claim credit for it in queens and rocky. | 08:41 |
| rakhmerov | ok | 08:41 |
| rakhmerov | :) | 08:42 |
| d0ugal | brb, 2 mins | 08:42 |
| rakhmerov | d0ugal: I have reasonable doubts that someone will get https://blueprints.launchpad.net/mistral/+spec/mistral-secure-sensitive-data done ) | 08:42 |
| d0ugal | back - sorry about that | 08:48 |
| d0ugal | rakhmerov: Indeed, I agree. | 08:48 |
| rakhmerov | ) | 08:48 |
| d0ugal | So we actually fixed 22 bugs in R-1, I found a number that had merged but didn't have a milestone | 08:49 |
| rakhmerov | ooh | 08:50 |
| rakhmerov | can we assign a milestone to them now? | 08:50 |
| d0ugal | Yes, I have done that | 08:50 |
| d0ugal | https://launchpad.net/mistral/+milestone/rocky-1 | 08:50 |
| d0ugal | The bug list is longer! | 08:50 |
| rakhmerov | ok | 08:51 |
| rakhmerov | awesome | 08:51 |
| rakhmerov | I guess we need to remind someone to set a milestone :) | 08:51 |
| d0ugal | Indeed! | 08:51 |
| rakhmerov | Hardik probably? | 08:51 |
| d0ugal | A number of people :) | 08:51 |
| rakhmerov | ok | 08:51 |
| rakhmerov | I'll tell Hardik (my colleague) | 08:52 |
| d0ugal | but I am not sure that everyone can do that? | 08:52 |
| d0ugal | bobh had missed a few | 08:52 |
| rakhmerov | I thought for bugs it should be possible, hm.. but may be not | 08:52 |
| d0ugal | This one for example didn't have a milestone: https://bugs.launchpad.net/mistral/+bug/1761246 | 08:52 |
| openstack | Launchpad bug 1761246 in Mistral "cut_list() needs to work the same as cut_dict()" [Low,Fix released] - Assigned to Bob Haddleton (bob-haddleton) | 08:52 |
| rakhmerov | ok | 08:53 |
| rakhmerov | d0ugal: if only a limited circle of people can do assignments maybe we just need to check it on review | 08:53 |
| d0ugal | Yup, we should look into that. | 08:54 |
| d0ugal | https://bugs.launchpad.net/mistral/+bug/1765349 | 08:54 |
| openstack | Launchpad bug 1765349 in Mistral "doc: missing minor version release notes" [Undecided,New] | 08:54 |
| d0ugal | We have had a request to go and write historical release notes :/ | 08:55 |
| d0ugal | We have been getting better are release notes slowly, but I don't think it is realistic to write and backport release notes for previous cycles. | 08:55 |
| rakhmerov | d0ugal: yes, Hardik confirmed that he can't assign a milestone | 08:56 |
| rakhmerov | d0ugal: ooh gosh, no! | 08:56 |
| d0ugal | rakhmerov: I'll add milestone reviewing to the regular lost of tasks on the etherpad. | 08:56 |
| rakhmerov | I'll eat an entire milestone probably :) | 08:57 |
| d0ugal | lol | 08:57 |
| rakhmerov | I'll = it'll | 08:57 |
| pgaxatte | rakhmerov: I can't imagine what you take for breakfast :) | 08:58 |
| *** apetrich has quit IRC | 08:59 | |
| rakhmerov | :) | 08:59 |
| rakhmerov | pgaxatte: reviews mostly ) | 08:59 |
| *** jaosorior has quit IRC | 08:59 | |
| hardikjasani | Haha | 08:59 |
| d0ugal | lol | 09:00 |
| d0ugal | Okay, we are our of time! That was a good R-1 cleanup | 09:00 |
| d0ugal | Thanks for the help | 09:00 |
| d0ugal | Next week I'll start looking at the R-2 goals and making sure they seem realistic. Hopefully I can reduce the work we have planned to someting almost possible. | 09:01 |
| d0ugal | #endmeeting | 09:01 |
| *** openstack changes topic to " (Meeting topic: test)" | 09:01 | |
| openstack | Meeting ended Fri Apr 20 09:01:18 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-04-20-08.00.html | 09:01 |
| rakhmerov | thanks! | 09:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-04-20-08.00.txt | 09:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-04-20-08.00.log.html | 09:01 |
| d0ugal | rakhmerov: https://bugs.launchpad.net/mistral/+bug/1676411 | 09:22 |
| openstack | Launchpad bug 1676411 in Mistral "Mistral serialiser and deserialiser can only work with entities in utf-8" [Medium,Confirmed] | 09:22 |
| d0ugal | Since we serialise to JSON, doesn't that mean it must be UTF-8? | 09:23 |
| d0ugal | iirc JSON must be UTF-8/16 | 09:23 |
| d0ugal | Yeah, I think that is correct (wikipedia agrees with me) | 09:24 |
| rakhmerov | hm.. I didn't think that JSON must be UTF-8/16 actually | 09:24 |
| d0ugal | oh | 09:24 |
| d0ugal | I'll try and find a better source :) | 09:24 |
| rakhmerov | UTF-8 is just common for all text-like communication protocols | 09:25 |
| d0ugal | https://tools.ietf.org/html/rfc8259#section-8.1 | 09:25 |
| d0ugal | "JSON text exchanged between systems that are not part of a closedecosystem MUST be encoded using UTF-8" | 09:25 |
| rakhmerov | hah | 09:25 |
| rakhmerov | interesting | 09:25 |
| rakhmerov | yes | 09:26 |
| d0ugal | and we output the serialised result via the API, so it isn't a closed ecosystem | 09:26 |
| rakhmerov | ok | 09:26 |
| rakhmerov | then it's invalid? The ticket I mean | 09:26 |
| d0ugal | I believe so | 09:26 |
| rakhmerov | ok | 09:26 |
| rakhmerov | just wonder why someone would need other enconding | 09:27 |
| d0ugal | It is something that we should document perhaps, but we have bigger issues with out documentation about custom actions :) | 09:27 |
| d0ugal | Indeed, I was wondering that too. Intergrating a legacy system? | 09:27 |
| rakhmerov | ooh, it's an old bug filed by my former colleague Anastasia | 09:28 |
| rakhmerov | ok, probably she came across it when testing something within Mistral itself | 09:28 |
| rakhmerov | so I think we can safely close it | 09:28 |
| *** mfedosin has quit IRC | 09:29 | |
| d0ugal | Okay, thanks! | 09:29 |
| d0ugal | rakhmerov: do you understand this bug? https://bugs.launchpad.net/mistral/+bug/1666565 | 09:30 |
| openstack | d0ugal: Error: malone bug 1666565 not found | 09:30 |
| d0ugal | It is the last "NEW" bug! :) | 09:30 |
| d0ugal | https://bugs.launchpad.net/mistral/+bugs?search=Search&field.status=New&orderby=id&start=0 | 09:30 |
| rakhmerov | looking.. | 09:30 |
| *** mfedosin has joined #openstack-mistral | 09:31 | |
| rakhmerov | d0ugal: ooh, yeah, I think it's really a bug | 09:33 |
| d0ugal | Yeah | 09:33 |
| rakhmerov | yes, I do understand it ) | 09:33 |
| d0ugal | It is surprising this hasn't come up before | 09:33 |
| rakhmerov | yep | 09:33 |
| d0ugal | Actually, it is over a year old | 09:33 |
| d0ugal | We need to look at securiy bugs more | 09:33 |
| rakhmerov | I'd show a thumb up if I could ) | 09:34 |
| d0ugal | lol | 09:34 |
| rakhmerov | yes | 09:34 |
| rakhmerov | low hanging fruit actually | 09:34 |
| d0ugal | Yeah, I think so | 09:34 |
| rakhmerov | just needs thorough verification | 09:34 |
| d0ugal | 👍 | 09:35 |
| d0ugal | https://bugs.launchpad.net/mistral/+bugs?search=Search&field.status=New&orderby=id&start=0 | 09:36 |
| d0ugal | yay! | 09:36 |
| rakhmerov | 👍 :) | 09:45 |
| *** apetrich has joined #openstack-mistral | 09:46 | |
| openstackgerrit | Merged openstack/mistral stable/queens: Add '__task_execution' structure to task execution context on the fly https://review.openstack.org/562929 | 09:49 |
| openstackgerrit | Merged openstack/mistral stable/queens: Release note for not persisting '__task_execution' in DB https://review.openstack.org/562967 | 09:49 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/queens: Optimize final workflow context evaluation with a batch request https://review.openstack.org/563014 | 09:51 |
| openstackgerrit | Tovin Seven proposed openstack/mistral master: Trivial: Update pypi url to new url https://review.openstack.org/563025 | 09:57 |
| openstackgerrit | Tovin Seven proposed openstack/python-mistralclient master: Trivial: Update pypi url to new url https://review.openstack.org/563057 | 10:21 |
| *** apetrich has quit IRC | 10:51 | |
| d0ugal | rakhmerov: it would have been to have a bug and a release note for https://review.openstack.org/#/q/I1f016a9c0d3add35e2c5059a29a2eca00e45af42 | 11:04 |
| d0ugal | since you are backporting it | 11:04 |
| rakhmerov | :) | 11:06 |
| rakhmerov | I can do a release note | 11:06 |
| rakhmerov | np | 11:06 |
| rakhmerov | the bug kind of exists but I didn't specify "Closes-Bug" because I'm not sure if it closes it | 11:07 |
| d0ugal | rakhmerov: Related-Bug works in those situations | 11:07 |
| rakhmerov | I could have done something like "Related-Bug" though.. ok | 11:07 |
| rakhmerov | yes | 11:07 |
| d0ugal | :) | 11:07 |
| rakhmerov | shame on me | 11:07 |
| rakhmerov | (and on reviewers, haha ) | 11:07 |
| d0ugal | it mergd so quickly. | 11:09 |
| *** jaosorior has joined #openstack-mistral | 11:29 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Release note for batched evaluation of final workflow context https://review.openstack.org/563088 | 11:32 |
| *** thrash|g0ne is now known as thrash | 12:05 | |
| openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Fix docker image build job https://review.openstack.org/562588 | 12:06 |
| pgaxatte | hi d0ugal, I read your comment on the bug I filed (https://bugs.launchpad.net/mistral/+bug/1764742) | 12:11 |
| openstack | Launchpad bug 1764742 in Mistral "Cannot disable actions without rebuilding" [Medium,Confirmed] | 12:11 |
| pgaxatte | do you think it would be possible to have a small mechanism while waiting for these features? | 12:11 |
| pgaxatte | I made a small patch to add options to disable some actions in the configuration | 12:12 |
| pgaxatte | it's on validation on our side but is there any chance to see something like this merged? | 12:13 |
| openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Fix docker image build job https://review.openstack.org/562588 | 12:34 |
| *** apetrich has joined #openstack-mistral | 12:46 | |
| openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Fix docker image build job https://review.openstack.org/562588 | 12:59 |
| thrash | d0ugal: ^^^ that *hopefully* is the last one :) | 12:59 |
| apetrich | thrash, thanks for tackling that | 12:59 |
| thrash | apetrich: no problemo. :) | 12:59 |
| thrash | anyone have any idea what that "BUILD_V8EVAL" would be used for? I guess the JS action? | 13:00 |
| thrash | d0ugal: apetrich woot! Success! | 13:09 |
| apetrich | neat! | 13:09 |
| openstackgerrit | Merged openstack/mistral master: Release note for batched evaluation of final workflow context https://review.openstack.org/563088 | 13:13 |
| *** apetrich has quit IRC | 13:15 | |
| *** apetrich has joined #openstack-mistral | 13:17 | |
| *** hardikjasani has quit IRC | 13:28 | |
| *** toure|gone is now known as toure | 13:33 | |
| openstackgerrit | Jose Castro Leon proposed openstack/mistral master: [Event-engine] Allow event_engine to work in HA https://review.openstack.org/548044 | 13:33 |
| *** doriftoshoes____ has quit IRC | 13:41 | |
| *** doriftoshoes____ has joined #openstack-mistral | 13:42 | |
| d0ugal | pgaxatte: if the change is small then yeah, I think we could do something like that. | 13:43 |
| d0ugal | thrash: yeah, so if you have that enabled the docker build will take much longer | 13:43 |
| d0ugal | thrash: it downloads and builds all the v8 javascript engine stuff. it is veryyyy slow | 13:43 |
| thrash | d0ugal: It is disabled for the moment, since it failed. | 13:43 |
| d0ugal | works for me! | 13:43 |
| thrash | \o/ | 13:43 |
| d0ugal | thrash: I am amazed the build only takes 5 mins | 13:46 |
| thrash | d0ugal: That actually doesn't surprise me. I think they have a local docker repo. | 13:47 |
| d0ugal | oh, I guess that does make sense | 13:47 |
| thrash | d0ugal: and python artifacts as well. | 13:47 |
| thrash | d0ugal: so, mostly just pulling wheels is my guess. | 13:48 |
| pgaxatte | d0ugal, alright I'll see with the team to push it upstream if they're ok | 13:53 |
| *** hrybacki has quit IRC | 14:06 | |
| *** hrybacki has joined #openstack-mistral | 14:06 | |
| *** apetrich has quit IRC | 14:23 | |
| *** nguyenhai has quit IRC | 14:33 | |
| *** jaosorior has quit IRC | 14:37 | |
| d0ugal | pgaxatte: so you are running a Mistral fork at the moment? | 14:48 |
| pgaxatte | d0ugal: no not yet, I still have to deploy tha changes I made but I need validation before doing that | 14:49 |
| d0ugal | I see, thanks | 14:49 |
| pgaxatte | then I'll backport my changes to pike and run it on our staging env | 14:49 |
| d0ugal | jpich: Hey launchpad expert, I have a question! | 14:53 |
| d0ugal | jpich: is there a way to have a link for all your bugs that refers to "self" - i.e. if you visit it you see bugs assigned to you and I would see only bugs assigned to me | 14:53 |
| d0ugal | In the UI when I click on bugs assigned to me it gives me a link with my username :( | 14:53 |
| *** apetrich has joined #openstack-mistral | 14:54 | |
| jpich | d0ugal: Not that I'm aware of, I use my username :/ | 14:58 |
| d0ugal | jpich: nevermind, thanks. I just wanted to share a link that worked for everyone | 14:58 |
| jpich | too easy | 14:58 |
| *** livelace-link has joined #openstack-mistral | 15:16 | |
| toure | d0ugal ping | 15:18 |
| d0ugal | toure: Hey | 15:18 |
| toure | got a sec, have a bit of trouble with the api | 15:18 |
| toure | d0ugal http://paste.openstack.org/show/719635/ | 15:19 |
| d0ugal | toure: I am tied up a bit with something else, might have to wait until next week. | 15:19 |
| toure | ack | 15:19 |
| *** livelace-link has quit IRC | 15:19 | |
| *** livelace-link has joined #openstack-mistral | 15:22 | |
| openstackgerrit | Merged openstack/mistral master: Trivial: Update pypi url to new url https://review.openstack.org/563025 | 15:35 |
| *** shardy_afk has quit IRC | 15:46 | |
| *** shardy has joined #openstack-mistral | 15:48 | |
| d0ugal | toure: so it looks like you are returning a dict when it expects the resource model? | 15:48 |
| openstackgerrit | Merged openstack/python-mistralclient master: Trivial: Update pypi url to new url https://review.openstack.org/563057 | 15:49 |
| *** jtomasek has quit IRC | 15:53 | |
| *** apetrich has quit IRC | 15:57 | |
| *** AlexeyAbashkin has quit IRC | 16:09 | |
| *** jtomasek has joined #openstack-mistral | 16:11 | |
| *** jtomasek has quit IRC | 16:12 | |
| openstackgerrit | Jose Castro Leon proposed openstack/mistral master: [Event-engine] Allow event_engine to work in HA https://review.openstack.org/548044 | 16:23 |
| *** shardy has quit IRC | 16:29 | |
| *** jpich has quit IRC | 16:35 | |
| *** gkadam has quit IRC | 16:42 | |
| openstackgerrit | Vitalii Solodilov proposed openstack/mistral master: WIP: Check a shadowing of python builtin https://review.openstack.org/551437 | 17:21 |
| *** rbrady is now known as rbrady-afk | 17:35 | |
| *** rbrady-afk has quit IRC | 17:41 | |
| openstackgerrit | Merged openstack/mistral stable/queens: Optimize final workflow context evaluation with a batch request https://review.openstack.org/563014 | 17:45 |
| *** thrash is now known as thrash|biab | 18:42 | |
| *** jtomasek has joined #openstack-mistral | 18:45 | |
| openstackgerrit | Paul Belanger proposed openstack/mistral master: Stop using slave_scripts/install-distro-packages.sh https://review.openstack.org/563225 | 18:48 |
| *** harlowja has joined #openstack-mistral | 19:02 | |
| *** thrash|biab is now known as thrash | 19:49 | |
| openstackgerrit | Vitalii Solodilov proposed openstack/mistral master: Rename the variables that were shadowed with python builtin https://review.openstack.org/551437 | 21:09 |
| *** thrash is now known as thrash|g0ne | 21:10 | |
| openstackgerrit | Vitalii Solodilov proposed openstack/mistral master: WIP: Created check_and_complete delayed call if it is latest task in branch https://review.openstack.org/555233 | 21:21 |
| openstackgerrit | Vitalii Solodilov proposed openstack/mistral master: Added 'safe-rerun' policy to task-defaults section https://review.openstack.org/559576 | 21:21 |
| *** jtomasek has quit IRC | 21:21 | |
| openstackgerrit | Vitalii Solodilov proposed openstack/mistral master: Minor `update_on_match` improvement: https://review.openstack.org/561490 | 21:31 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!