| *** d0ugal has quit IRC | 01:24 | |
| *** d0ugal has joined #openstack-mistral | 01:35 | |
| *** larsks has quit IRC | 05:05 | |
| *** larsks has joined #openstack-mistral | 05:12 | |
| *** eyalb1 has joined #openstack-mistral | 06:48 | |
| *** eyalb2 has joined #openstack-mistral | 07:04 | |
| *** eyalb1 has quit IRC | 07:07 | |
| *** eyalb1 has joined #openstack-mistral | 07:14 | |
| *** eyalb2 has quit IRC | 07:17 | |
| rakhmerov | d0ugal: hey | 07:33 |
|---|---|---|
| rakhmerov | have a few minutes? | 07:33 |
| *** pgaxatte has joined #openstack-mistral | 07:56 | |
| d0ugal | rakhmerov: Sure | 08:36 |
| rakhmerov | d0ugal: we're finally moving OpenStack actions from the main mistral repo | 08:37 |
| rakhmerov | the patches are on review | 08:37 |
| d0ugal | oh, cool | 08:37 |
| rakhmerov | eyalb1 is working on it | 08:37 |
| d0ugal | Hows it going? | 08:37 |
| rakhmerov | yeah, cool ) | 08:38 |
| rakhmerov | however, we're having an issue with TripleO | 08:38 |
| rakhmerov | eyalb1: Eyal, can you please explain d0ugal what needs to be done in TripleO when those patches are merged? | 08:38 |
| d0ugal | Can you link me to the patch? | 08:39 |
| eyalb1 | since I moved some files from mistral to mistral-extra | 08:39 |
| eyalb1 | it fails on building the rpm dlrn | 08:39 |
| rakhmerov | https://review.opendev.org/#/c/703972/ | 08:39 |
| eyalb1 | it looks for the mapping.json file that was moved to mistral-extra | 08:40 |
| rakhmerov | https://review.opendev.org/#/c/703296/ | 08:40 |
| rakhmerov | these are the two main patches | 08:40 |
| rakhmerov | d0ugal: so we're trying to understand how to deal with it on the TripleO side | 08:40 |
| rakhmerov | we can temporarily make this CI job non-voting, but in the mean time we need to fix it in TripleO | 08:41 |
| d0ugal | Looking | 08:41 |
| d0ugal | I guess it will break all of tripleo :) so making it non-voting wouldn't be great for us | 08:41 |
| d0ugal | I'll see if I can handle it | 08:41 |
| rakhmerov | d0ugal: I understand, yes. But it has to de done one day anyway :) | 08:44 |
| rakhmerov | we were supposed to do that long ago ) | 08:44 |
| rakhmerov | we're just thinking how to mitigate the consequences for 3rd parties | 08:44 |
| d0ugal | well of crouse | 08:44 |
| eyalb1 | also there is a new version of mistral-lib that should be used so I guess there is a need to fix the rpm requirements | 08:46 |
| d0ugal | eyalb1, rakhmerov: http://paste.openstack.org/show/789088/ | 08:46 |
| d0ugal | That seems to be the error | 08:46 |
| d0ugal | Is that something new in mistral-lib? has it been released? | 08:46 |
| d0ugal | ah | 08:46 |
| d0ugal | you just said that :) | 08:46 |
| d0ugal | so maybe we don't have a rpm for that yet... checking | 08:47 |
| rakhmerov | yeah | 08:47 |
| d0ugal | I don't think we even have a rpm for mistral-extra yet | 08:48 |
| rakhmerov | yes | 08:49 |
| d0ugal | We have a mistral-lib updated rpm now, so I rechecked it | 08:49 |
| rakhmerov | but is this doable within a reasonable amount of time? | 08:49 |
| d0ugal | but I guess things will still fail because we don't have mistral-extra | 08:49 |
| d0ugal | We can try | 08:49 |
| rakhmerov | it needs to be added in the requirements I guess | 08:50 |
| d0ugal | How much time is reasonable? :) | 08:50 |
| rakhmerov | mistral-extra | 08:50 |
| rakhmerov | d0ugal: well, till the end of the U cycle is fine :) | 08:50 |
| rakhmerov | I mean, I understand technically it is easy | 08:50 |
| rakhmerov | but we're wondering if it's going to make some other problems for you | 08:51 |
| rakhmerov | organisational etc. | 08:51 |
| d0ugal | Only technical problems, and they are mostly easy but I don't know much about rpms :) so I need help there | 08:52 |
| rakhmerov | yes | 08:52 |
| rakhmerov | we're asking for help ) | 08:52 |
| d0ugal | Really I think mistral should depend on mistral-extra | 08:53 |
| d0ugal | otherwise I think you are removing all the actions without any deprecation? | 08:53 |
| d0ugal | but I don't have strong feelings about that ^ | 08:55 |
| d0ugal | I think there are two things that need to happen in TripleO | 08:55 |
| d0ugal | 1. We have picked up the new mistral-lib, but we have not promoted the package yet. Something is failing there, but I don't think it is Mistral related. People are working on this. However, that means CI is using the old version for now. | 08:56 |
| d0ugal | 2. We need to create a new mistral-extra package and add that to tripleo, otherwise it will fail due to the missing actions. | 08:56 |
| openstackgerrit | Eyal proposed openstack/mistral-lib master: Fix requirements https://review.opendev.org/705615 | 09:03 |
| eyalb1 | @d0gal: Do you run mistral tempest test on tripleo ? | 09:05 |
| eyalb1 | because it will fail only if you ran mistral workflow with openstack actions | 09:05 |
| eyalb1 | @d0ugal | 09:05 |
| d0ugal | eyalb1: I don't think we do, but I am not certain | 09:09 |
| d0ugal | I guess the tempest tests will be updated to depend on mistra-extra? | 09:09 |
| eyalb1 | yes | 09:09 |
| *** apetrich has joined #openstack-mistral | 09:11 | |
| eyalb1 | at the moment the tempest are running only on devstack so in the requirments there is no dependency | 09:11 |
| eyalb1 | not on mistral/mistrtal-lib | 09:11 |
| eyalb1 | and I think it is on purpose | 09:11 |
| eyalb1 | when building the gate then there is a dependency | 09:12 |
| eyalb1 | once mistral-extra is merged I might add tempest test just for openstack actions | 09:13 |
| d0ugal | Okay, makes sense | 09:14 |
| openstackgerrit | Eyal proposed openstack/mistral-lib master: Remove python2 support https://review.opendev.org/705615 | 09:16 |
| eyalb1 | d0ugal: it looks for the mapping.json file that was moved to mistral-extra | 10:00 |
| eyalb1 | install: cannot stat './mistral/actions/openstack/mapping.json': No such file or directory | 10:00 |
| eyalb1 | DEBUG: RPM build errors: | 10:00 |
| eyalb1 | DEBUG: error: Bad exit status from /var/tmp/rpm-tmp.QtUUbG (%install) | 10:00 |
| eyalb1 | DEBUG: Bad exit status from /var/tmp/rpm-tmp.QtUUbG (%install) | 10:00 |
| eyalb1 | DEBUG: Child return code was: 1 | 10:00 |
| eyalb1 | INFO: EXCEPTION: [Error()] | 10:00 |
| d0ugal | eyalb1: k, thanks | 10:00 |
| d0ugal | Currently writing a rpm spec for mistral-extra :) | 10:01 |
| eyalb1 | also it looks like it installed mistral-lib 1.3.0 and not 1.4.0 | 10:03 |
| eyalb1 | https://zuul.opendev.org/t/openstack/build/01df24513a6248169910373ce147bf60/log/logs/undercloud/home/zuul/install_packages.sh.log#3370 | 10:03 |
| d0ugal | eyalb1: yeah, I mentioned that above. The new version has been built but not promoted yet | 10:03 |
| d0ugal | Tripleo promotes packages when we have a full set of versions all working together, soemthing caused a failure so it hasn't happened yet | 10:04 |
| d0ugal | rakhmerov, eyalb1: Is the mistral-extra patch ready to merge? | 10:15 |
| d0ugal | https://review.opendev.org/#/c/703296/ | 10:15 |
| d0ugal | I think it will make the rpm easier to verify if there is code there :) | 10:16 |
| rakhmerov | d0ugal: I still need to review it | 10:16 |
| d0ugal | ok | 10:16 |
| rakhmerov | will do till the end of the day | 10:16 |
| d0ugal | okay, thanks! | 10:19 |
| d0ugal | Please let me know if there are any big changes as I am writing a spec based on the requirements etc. listed there | 10:19 |
| *** apetrich has quit IRC | 11:14 | |
| *** pgaxatte has quit IRC | 11:14 | |
| *** mmethot_ has quit IRC | 11:14 | |
| *** szaher has quit IRC | 11:14 | |
| *** irclogbot_2 has quit IRC | 11:15 | |
| *** irclogbot_1 has joined #openstack-mistral | 11:17 | |
| *** apetrich has joined #openstack-mistral | 11:18 | |
| *** pgaxatte has joined #openstack-mistral | 11:18 | |
| *** mmethot_ has joined #openstack-mistral | 11:18 | |
| *** szaher has joined #openstack-mistral | 11:18 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Set the delayed call "key" field to the right value https://review.opendev.org/705635 | 11:39 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Set the delayed call "key" field to the right value https://review.opendev.org/705635 | 11:45 |
| openstackgerrit | Eyal proposed openstack/mistral-tempest-plugin master: Drop support in py2 https://review.opendev.org/705643 | 12:08 |
| openstackgerrit | Eyal proposed openstack/mistral-tempest-plugin master: Remove some pycharm warnings https://review.opendev.org/705651 | 12:45 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral-extra master: Correct the mistral-extra package name https://review.opendev.org/705676 | 13:33 |
| d0ugal | rakhmerov, eyalb1: ^ | 13:33 |
| d0ugal | You might notice that if you install mistral-extra from pypi it has no package :) | 13:33 |
| *** eyalb1 has quit IRC | 13:44 | |
| *** jtomasek has joined #openstack-mistral | 15:02 | |
| *** jtomasek has quit IRC | 15:27 | |
| *** jtomasek has joined #openstack-mistral | 15:32 | |
| *** pgaxatte has quit IRC | 16:55 | |
| *** rakhmerov has quit IRC | 18:52 | |
| *** jtomasek has quit IRC | 19:04 | |
| *** jtomasek has joined #openstack-mistral | 19:08 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!