| openstackgerrit | Vlad Gusev proposed openstack/mistral stable/stein: Fix error validate token when run cron trigger https://review.opendev.org/695332 | 01:06 |
|---|---|---|
| openstackgerrit | Vlad Gusev proposed openstack/mistral stable/stein: Add release note for fix error of cron trigger run https://review.opendev.org/695333 | 01:07 |
| *** ricolin has joined #openstack-mistral | 01:55 | |
| *** apetrich has quit IRC | 03:08 | |
| rakhmerov | d0ugal: hey | 06:25 |
| rakhmerov | just noticed your ping | 06:25 |
| openstackgerrit | Merged openstack/mistral stable/stein: Fix error validate token when run cron trigger https://review.opendev.org/695332 | 07:24 |
| openstackgerrit | Merged openstack/mistral stable/stein: Add release note for fix error of cron trigger run https://review.opendev.org/695333 | 07:24 |
| rakhmerov | d0ugal: I approved everything I could submitted by yourself | 07:27 |
| *** apetrich has joined #openstack-mistral | 07:34 | |
| *** pgaxatte has joined #openstack-mistral | 07:44 | |
| d0ugal | rakhmerov: Great, thanks | 08:40 |
| d0ugal | rakhmerov: I was going to ask for help debbung the CI faiures. I really don't know where to start with it | 08:40 |
| d0ugal | but maybe we just need to leave it non-voting? There isn't much happening in Rocky now. | 08:41 |
| d0ugal | rakhmerov: Did you see Jean's question on https://review.opendev.org/#/c/694781/ ? | 08:47 |
| d0ugal | I guess he didn't understand why the other patches were backported. Should it be a feature release? | 08:47 |
| rakhmerov | no, I didn't see that question | 08:56 |
| rakhmerov | will have a look a bit later today.. | 08:56 |
| rakhmerov | d0ugal: I replied to it | 09:00 |
| rakhmerov | yeah, it's kind of my fault but I didn't think it would be an issue | 09:01 |
| d0ugal | rakhmerov: no problem, I guess he was just curious | 09:04 |
| openstackgerrit | Merged openstack/mistral stable/rocky: Make the mistral devstack jobs non-voting in Rocky https://review.opendev.org/695240 | 09:16 |
| openstackgerrit | Merged openstack/mistral stable/rocky: Mask sensitive data when logging action results https://review.opendev.org/694733 | 09:16 |
| *** ricolin has quit IRC | 09:22 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Remove the TripleO standalone job https://review.opendev.org/695378 | 09:29 |
| rakhmerov | d0ugal: I quickly glanced at those failures in Rocky, they actually look like some logical problems either with the tests or some relevant changes | 09:30 |
| rakhmerov | Details: {'debuginfo': None, 'faultstring': "Invalid input [name=wf1, class=ReverseWorkflowSpec, missing=['farewell']]", 'faultcode': 'Client'} | 09:31 |
| rakhmerov | a missing input parameter | 09:31 |
| rakhmerov | very weird | 09:31 |
| d0ugal | rakhmerov: I tried to look and the code looks fine | 09:31 |
| d0ugal | They also all passed at least once | 09:32 |
| rakhmerov | hm.. | 09:32 |
| rakhmerov | well, rocky is pretty old, I remember we had lots of issues with ordering etc. leading to race conditions | 09:32 |
| d0ugal | Yeah, I remember that too. I forget how it was fixed :) | 09:33 |
| rakhmerov | but I'm just surprised with a missing parameter, it doesn't look like falling into that category of issues | 09:33 |
| d0ugal | Agreed | 09:34 |
| d0ugal | Maybe the workflow name is reused and sometimes it has the parameter and sometimes it doesn't? | 09:35 |
| rakhmerov | yeah.. it is possible actually! | 09:36 |
| rakhmerov | yeah, maybe something is wrong with clean up | 09:37 |
| rakhmerov | d0ugal: I'm looking at https://review.opendev.org/#/c/695378/1 | 09:37 |
| rakhmerov | how come that this job wasn't used? I'm not sure I understand your commit message | 09:37 |
| rakhmerov | the "...standalone" job that we always see wasn't take from what you removed? | 09:38 |
| d0ugal | rakhmerov: there are two TripleO jobs, one of them doesn't use Mistral | 09:38 |
| d0ugal | oh wait | 09:38 |
| d0ugal | rakhmerov: it is the one I removed, look at the diff | 09:38 |
| d0ugal | it has "-standalone" at the end | 09:38 |
| d0ugal | Sorry, I am used to TripleO devs just referring to it as "standalone" | 09:38 |
| rakhmerov | yes, I understand.. | 09:38 |
| rakhmerov | so you're stating that it never used Mistral? | 09:39 |
| d0ugal | Correct | 09:39 |
| d0ugal | the other tripleo job does use Mistral | 09:39 |
| rakhmerov | oooh damn... | 09:39 |
| rakhmerov | really? ) | 09:39 |
| d0ugal | Yeah, I don't know how I didn't notice this before | 09:39 |
| rakhmerov | we've been running it for years ) | 09:39 |
| d0ugal | I know :/ | 09:39 |
| rakhmerov | ok | 09:39 |
| d0ugal | Maybe it did a long time ago, but I forget | 09:39 |
| rakhmerov | it's good though that you finally noticed that ) | 09:40 |
| rakhmerov | I'd never do | 09:40 |
| d0ugal | haha true | 09:40 |
| rakhmerov | thanks anyway | 09:40 |
| d0ugal | It does install Mistral, but doesn't use it. If you look at the link in the commit message you can see there are no action execution logs | 09:40 |
| d0ugal | well, other than the startup and some debug logs | 09:40 |
| rakhmerov | yeah, ok | 09:40 |
| d0ugal | no executor logs I mean :) | 09:40 |
| rakhmerov | got it | 09:40 |
| rakhmerov | one day I believe I need to learn how that all works, haha ) | 09:41 |
| d0ugal | I hate to think how many hours of CI we have wasted there | 09:41 |
| rakhmerov | yeah | 09:41 |
| d0ugal | rakhmerov: well, we are removing Mistral from TripleO actually. So you might not want to bother learning... | 09:41 |
| rakhmerov | and how that increased the bills for electricity somewhere in a DC :) | 09:41 |
| rakhmerov | d0ugal: woow, really? | 09:42 |
| rakhmerov | can you tell more? | 09:42 |
| d0ugal | rakhmerov: Sure, basically we are replacing it with Ansible | 09:42 |
| d0ugal | There is a effort in tripleo to make things easier and simpler | 09:42 |
| d0ugal | This is part of that | 09:42 |
| rakhmerov | Ansible Tower or just core Ansible? | 09:43 |
| d0ugal | Just core ansible, but it does mean that we can maybe use Tower/AWX in the future. | 09:43 |
| rakhmerov | aah, ok | 09:43 |
| rakhmerov | hopefully the TripleO members understand that these two things are not interchangeable | 09:44 |
| d0ugal | Indeed | 09:44 |
| d0ugal | but we never really took advantage of all the features in Mistral. | 09:45 |
| rakhmerov | for our use cases though it may be fine, I realised so far that you use Mistral in a pretty limited way | 09:45 |
| rakhmerov | yeah | 09:45 |
| rakhmerov | I actually expected that to happen pretty soon, so ok | 09:45 |
| rakhmerov | thanks for letting me know | 09:45 |
| d0ugal | no problem | 09:45 |
| rakhmerov | so when do you think we can remove the TripleO jobs then completely? :) | 09:46 |
| d0ugal | rakhmerov: I guess it will take some time, and we still have some people running Newton (!) so we will have some connection to Mistral for some time | 09:46 |
| rakhmerov | how long may it last? | 09:46 |
| rakhmerov | this transition | 09:46 |
| rakhmerov | ok | 09:46 |
| d0ugal | rakhmerov: Good question. I don't know really, we have lots to do. Maybe by the end of V? | 09:46 |
| d0ugal | Newton was our first release that included Mistral btw | 09:47 |
| rakhmerov | yes :) | 09:48 |
| rakhmerov | ok | 09:48 |
| d0ugal | I think our biggest problem was the development workflow with Mistral. Writing workflows and actions. Updating the workbooks, installing the actions, restarting mistral etc. | 09:49 |
| d0ugal | It was very painful for everyone | 09:49 |
| d0ugal | Lots of people know Ansible and it is generally easier to write | 09:49 |
| d0ugal | but I do appreciate they are not the same thing :) | 09:50 |
| rakhmerov | d0ugal: I totally understand this concern. I keep raising it again and again so that we could get resources to make it all much easier | 10:11 |
| rakhmerov | and better documented etc. | 10:11 |
| rakhmerov | lots of people raise the same when they get to developing something relatively big | 10:11 |
| d0ugal | Yeah | 10:12 |
| d0ugal | I feel like it is hard to do now unfortunately | 10:12 |
| rakhmerov | d0ugal: btw, that's why I really wanted to implement all those initiatives around actions, specifically | 10:12 |
| rakhmerov | d0ugal: it is, I know | 10:12 |
| d0ugal | rakhmerov: Can you look at https://review.opendev.org/#/c/695375/ and https://review.opendev.org/#/c/694786/ | 10:13 |
| rakhmerov | ok | 10:13 |
| d0ugal | rakhmerov: I made them feature releases because there were lots of unreleased changes in stable/rocky and stable/stein | 10:13 |
| d0ugal | they didn't all feel like bug fixes to me | 10:13 |
| rakhmerov | ok | 10:13 |
| d0ugal | stein changes: https://github.com/openstack/mistral/compare/8.0.0...stable/stein | 10:14 |
| d0ugal | rocky changes: https://github.com/openstack/mistral/compare/8.0.0...stable/stein | 10:14 |
| d0ugal | rakhmerov: ^ if you want a easy place to see them | 10:15 |
| rakhmerov | ok | 10:15 |
| d0ugal | oops, the second link is bad | 10:15 |
| d0ugal | rocky changes: https://github.com/openstack/mistral/compare/7.0.4...stable/rocky | 10:15 |
| rakhmerov | d0ugal: is what you told me an official decision yet? | 10:18 |
| d0ugal | rakhmerov: yes, the work has already started | 10:18 |
| rakhmerov | ok | 10:19 |
| d0ugal | rakhmerov: https://review.opendev.org/#/c/679272/ | 10:19 |
| d0ugal | rakhmerov: the spec hasn't merged yet, but I guess it will soon (once I find time to update it!) | 10:19 |
| rakhmerov | it is disappointing, of course, but ok | 10:20 |
| d0ugal | rakhmerov: Indeed, I think it is a shame | 10:22 |
| rakhmerov | well, another lesson.. | 10:22 |
| rakhmerov | hopefully, we'll learn it | 10:22 |
| rakhmerov | it is especially sad to me because I've been trying to convince people in importance of improving all that stuff (extending, plugins, actions etc. etc.) for years | 10:23 |
| rakhmerov | but we keep failing to address that | 10:24 |
| d0ugal | Yup, growing a community around the project is probably one of the hardest but most important things :) | 10:24 |
| rakhmerov | d0ugal: approved your patches | 10:32 |
| d0ugal | rakhmerov: thanks! | 10:32 |
| *** apetrich has quit IRC | 11:29 | |
| *** apetrich has joined #openstack-mistral | 11:59 | |
| *** ricolin has joined #openstack-mistral | 12:27 | |
| *** pgaxatte has quit IRC | 13:30 | |
| *** mmethot has quit IRC | 13:36 | |
| *** pgaxatte has joined #openstack-mistral | 13:47 | |
| *** jtomasek has quit IRC | 14:32 | |
| *** jtomasek has joined #openstack-mistral | 14:35 | |
| *** mmethot has joined #openstack-mistral | 14:43 | |
| *** pgaxatte has quit IRC | 15:25 | |
| *** jtomasek has quit IRC | 15:38 | |
| openstackgerrit | Merged openstack/mistral-dashboard master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/694565 | 15:47 |
| *** pgaxatte has joined #openstack-mistral | 15:57 | |
| openstackgerrit | Merged openstack/mistral-extra master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/694564 | 16:11 |
| *** ricolin has quit IRC | 16:39 | |
| *** pgaxatte has quit IRC | 17:14 | |
| *** openstack has joined #openstack-mistral | 21:15 | |
| *** ChanServ sets mode: +o openstack | 21:15 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!