| lbragstad | rakhmerov do you know if the stable/train gate is broken? | 00:46 |
|---|---|---|
| *** __ministry has joined #openstack-mistral | 02:00 | |
| *** lbragstad has quit IRC | 02:31 | |
| *** apetrich has quit IRC | 02:42 | |
| *** ykarel|away is now known as ykarel | 04:08 | |
| *** eyalb has joined #openstack-mistral | 04:49 | |
| *** threestrands has joined #openstack-mistral | 05:27 | |
| *** threestrands has quit IRC | 05:33 | |
| *** eyalb has quit IRC | 05:47 | |
| *** eyalb has joined #openstack-mistral | 05:51 | |
| *** eyalb has quit IRC | 05:54 | |
| *** eyalb has joined #openstack-mistral | 05:54 | |
| *** eyalb1 has joined #openstack-mistral | 05:56 | |
| *** pgaxatte has joined #openstack-mistral | 05:58 | |
| *** eyalb has quit IRC | 05:59 | |
| *** eyalb2 has joined #openstack-mistral | 06:14 | |
| *** eyalb1 has quit IRC | 06:17 | |
| rakhmerov | mgariepy: not sure if you've seen my reply (depends on whether you have a bouncer). So, it should have been backported, checking it now | 06:30 |
| rakhmerov | mgariepy: so for train here it is: https://review.opendev.org/#/c/718375/ | 06:34 |
| rakhmerov | it's just not exactly the same patch | 06:35 |
| *** apetrich has joined #openstack-mistral | 07:11 | |
| *** eyalb1 has joined #openstack-mistral | 07:15 | |
| *** eyalb2 has quit IRC | 07:18 | |
| *** eyalb2 has joined #openstack-mistral | 07:30 | |
| *** eyalb1 has quit IRC | 07:33 | |
| *** nightmare_unreal has joined #openstack-mistral | 07:41 | |
| apetrich | Morning rakhmerov! | 07:59 |
| rakhmerov | apetrich: hey | 07:59 |
| apetrich | rakhmerov, how are you doing? | 07:59 |
| rakhmerov | good! Still alive :) | 07:59 |
| apetrich | tell me about it :) | 08:00 |
| apetrich | so there's a cve patch that for queens that somehow hasn't been merged https://review.opendev.org/#/c/734830/1 and zuul is all messed up | 08:00 |
| apetrich | do you know if there's anything we can do about it? | 08:01 |
| rakhmerov | what is "cve"? | 08:01 |
| rakhmerov | looking at the patch.. | 08:02 |
| rakhmerov | ok, I see it's been backported to stein, rocky and train | 08:03 |
| rakhmerov | but there's something wrong with queens | 08:03 |
| rakhmerov | hm.. | 08:04 |
| apetrich | rakhmerov, very | 08:04 |
| rakhmerov | I haven't looked at it closely | 08:04 |
| apetrich | rakhmerov, cve is a security patch. sorry I didn't know that it was redhat centric term. | 08:04 |
| rakhmerov | yet | 08:04 |
| rakhmerov | that's ok, I got it | 08:04 |
| rakhmerov | ok, have you looked into the failures yet? | 08:05 |
| rakhmerov | I see a lot of "RETRY_LIMIT" statuses | 08:05 |
| rakhmerov | usually when it happens it means something major is broken in the CI itself | 08:05 |
| apetrich | yeah | 08:06 |
| rakhmerov | hm... queens falls into the 4 last releases so it should still be supported | 08:07 |
| rakhmerov | 2020-06-11 08:14:37.897898 | TASK [test-setup : Run tools/test-setup.sh] | 08:09 |
| rakhmerov | 3802020-06-11 08:14:39.175600 | ubuntu-xenial | + DB_ROOT_PW=insecure_slave | 08:09 |
| rakhmerov | 3812020-06-11 08:14:39.175739 | ubuntu-xenial | + DB_USER=openstack_citest | 08:09 |
| rakhmerov | 3822020-06-11 08:14:39.175761 | ubuntu-xenial | + DB_PW=openstack_citest | 08:09 |
| rakhmerov | 3832020-06-11 08:14:39.175775 | ubuntu-xenial | + sudo -H mysqladmin -u root password insecure_slave | 08:09 |
| rakhmerov | 3842020-06-11 08:14:39.180452 | ubuntu-xenial | sudo: mysqladmin: command not found | 08:09 |
| rakhmerov | 3852020-06-11 08:14:39.638229 | ubuntu-xenial | ERROR | 08:09 |
| rakhmerov | it can't fine mysqladmin for some reason | 08:09 |
| rakhmerov | I'll try to clarify it with the infra team | 08:09 |
| eyalb2 | bindep maybe ? | 08:09 |
| rakhmerov | trying to recheck it in the meantime | 08:10 |
| rakhmerov | eyalb2: what exactly about bindep? | 08:10 |
| eyalb2 | I remember we had to add it so it will install mysql stuff | 08:11 |
| eyalb2 | it used to install it by default | 08:11 |
| eyalb2 | but then we had to add this file | 08:11 |
| rakhmerov | yeah.. I don't remember the details though | 08:12 |
| rakhmerov | yes, you may be right | 08:13 |
| rakhmerov | there's no such thing as bindep.txt in this branch | 08:13 |
| rakhmerov | let me try to add it.. | 08:13 |
| *** ykarel is now known as ykarel|lunch | 08:13 | |
| *** eyalb1 has joined #openstack-mistral | 08:15 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/queens: Add bindep.txt with the CI dependencies https://review.opendev.org/736129 | 08:17 |
| *** eyalb2 has quit IRC | 08:18 | |
| rakhmerov | apetrich: ^ hopefully, it will help | 08:18 |
| apetrich | thanks a lot rakhmerov and eyalb1! | 08:19 |
| eyalb1 | its failing now on the uwsgi issue | 08:30 |
| rakhmerov | eyalb1: what is failing? So far so good | 08:32 |
| rakhmerov | I'm going to rebase that patch on top of mine | 08:32 |
| rakhmerov | everything is green so far | 08:32 |
| *** eyalb2 has joined #openstack-mistral | 08:32 | |
| rakhmerov | apetrich: so adding bindep.txt is helping | 08:32 |
| rakhmerov | eyalb1: aah, yes, devstack gates | 08:33 |
| rakhmerov | it was solved recently for master | 08:33 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/queens: Mask sensitive data when logging action results https://review.opendev.org/734830 | 08:34 |
| *** eyalb1 has quit IRC | 08:35 | |
| apetrich | thanks! | 08:40 |
| rakhmerov | apetrich: so I asked in openstack-infra regarding this "uwsgi" issue and they said that stable branches are still in progress | 08:43 |
| rakhmerov | so we may have to wait a little bit | 08:44 |
| *** eyalb1 has joined #openstack-mistral | 08:45 | |
| apetrich | thanks again! | 08:47 |
| *** eyalb2 has quit IRC | 08:48 | |
| *** eyalb1 has quit IRC | 08:54 | |
| *** ykarel|lunch is now known as ykarel | 09:03 | |
| *** apetrich has quit IRC | 09:17 | |
| *** apetrich has joined #openstack-mistral | 09:26 | |
| *** eyalb has joined #openstack-mistral | 10:04 | |
| mgariepy | rakhmerov, train has a backport but Stein doesn't and it is vulnerable to this issue. | 10:33 |
| *** eyalb1 has joined #openstack-mistral | 10:33 | |
| *** eyalb has quit IRC | 10:36 | |
| rakhmerov | mgariepy: no, it's not | 10:36 |
| rakhmerov | this patch is just one of the series of patches | 10:37 |
| rakhmerov | there was another patch also backported to train (because it was very much needed there) that adds a config option to disable conversion of YAQL expression results | 10:38 |
| rakhmerov | yaql.convert_output_data | 10:38 |
| rakhmerov | by default it's set to True to keep compatibility | 10:38 |
| rakhmerov | and this sanitizing is needed only if this conversion is disabled | 10:38 |
| rakhmerov | so it doesn't affect versions prior to Train | 10:39 |
| mgariepy | i tested the wf on rocky yesterday and it did crash | 10:40 |
| mgariepy | i'll setup a dev instance and retest the wf on theses. i'll keep you posted on the resullt. | 10:46 |
| mgariepy | probably not today but surely this week. | 10:46 |
| *** eyalb2 has joined #openstack-mistral | 10:48 | |
| rakhmerov | ok | 10:49 |
| rakhmerov | mgariepy: maybe it crashed for a different reason? | 10:50 |
| rakhmerov | it, please share the results | 10:50 |
| rakhmerov | we'll see if something really needs to be backported. But before Train this mechanism just worked differently. So I assume there might have been just a different issue | 10:51 |
| *** eyalb1 has quit IRC | 10:51 | |
| mgariepy | it's the yaml expansion that did filled the ram on my machine. | 10:55 |
| rakhmerov | mgariepy: that is a completely different issue I believe | 10:57 |
| rakhmerov | I need to check where the corresponding fix is backported to | 10:58 |
| mgariepy | ok | 10:59 |
| rakhmerov | maybe you confuse two different things: YAML and YAQL | 10:59 |
| mgariepy | maybe ;) | 11:00 |
| rakhmerov | yeah, I found a patch: https://review.opendev.org/#/c/698714/ | 11:01 |
| rakhmerov | this is probably what you're hitting | 11:01 |
| rakhmerov | and yes, I believe it's not been backported.. | 11:01 |
| mgariepy | ha | 11:01 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/train: Disable the use of anchors when parsing yaml https://review.opendev.org/736158 | 11:02 |
| rakhmerov | mgariepy: yeah, it was supposed to be since it's a severe issue | 11:02 |
| rakhmerov | we overlooked, I admit | 11:02 |
| mgariepy | no worries. | 11:02 |
| mgariepy | can it be backported back to rocky ? | 11:03 |
| rakhmerov | so thanks for paying attention to this, I'll come up with the backports shortly | 11:03 |
| rakhmerov | I believe so | 11:03 |
| mgariepy | ok | 11:03 |
| rakhmerov | here it is for train: https://review.opendev.org/#/c/736158/ | 11:03 |
| rakhmerov | keep doing for others.. | 11:03 |
| mgariepy | ok perfect. | 11:03 |
| rakhmerov | for stein and rocky there are merge conflicts, but we'll do it anyway. It will just take some time to adjust em | 11:04 |
| mgariepy | the code change quite a bit between releases :) ahah | 11:06 |
| rakhmerov | yep | 11:06 |
| rakhmerov | that's a normal thing | 11:06 |
| mgariepy | yep. when it's stalled it's never a good thing :) | 11:07 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/stein: Disable the use of anchors when parsing yaml https://review.opendev.org/736161 | 11:11 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/rocky: Disable the use of anchors when parsing yaml https://review.opendev.org/736163 | 11:15 |
| mgariepy | great thanks for the patches. | 11:17 |
| *** eyalb1 has joined #openstack-mistral | 11:18 | |
| *** eyalb2 has quit IRC | 11:20 | |
| *** eyalb2 has joined #openstack-mistral | 11:48 | |
| *** eyalb1 has quit IRC | 11:51 | |
| *** eyalb1 has joined #openstack-mistral | 12:19 | |
| *** eyalb2 has quit IRC | 12:22 | |
| *** lbragstad has joined #openstack-mistral | 12:39 | |
| *** eyalb2 has joined #openstack-mistral | 13:06 | |
| *** eyalb1 has quit IRC | 13:08 | |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/mistral-lib stable/queens: Ensure we mask sensitive data from Mistral Action logs https://review.opendev.org/736217 | 13:28 |
| *** eyalb1 has joined #openstack-mistral | 13:36 | |
| *** eyalb2 has quit IRC | 13:38 | |
| *** eyalb2 has joined #openstack-mistral | 13:51 | |
| *** eyalb1 has quit IRC | 13:54 | |
| *** eyalb1 has joined #openstack-mistral | 14:06 | |
| *** eyalb2 has quit IRC | 14:08 | |
| *** eyalb1 has quit IRC | 14:24 | |
| openstackgerrit | Merged openstack/mistral stable/train: Disable the use of anchors when parsing yaml https://review.opendev.org/736158 | 15:50 |
| *** ykarel is now known as ykarel|away | 16:33 | |
| *** pgaxatte has quit IRC | 16:40 | |
| *** nightmare_unreal has quit IRC | 17:24 | |
| *** irclogbot_3 has quit IRC | 20:14 | |
| *** amorin has quit IRC | 20:18 | |
| *** amorin has joined #openstack-mistral | 20:20 | |
| *** irclogbot_0 has joined #openstack-mistral | 20:21 | |
| *** csatari has quit IRC | 20:39 | |
| *** csatari has joined #openstack-mistral | 21:39 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!