| *** anilvenkata has joined #openstack-rally | 05:26 | |
| *** tosky has joined #openstack-rally | 08:00 | |
| *** dmellado has quit IRC | 08:01 | |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Try to turn treat warning as errors at tox https://review.opendev.org/714924 | 08:03 |
|---|---|---|
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Try to fix existing users job https://review.opendev.org/718087 | 08:04 |
| *** dmellado has joined #openstack-rally | 08:05 | |
| *** rcernin has quit IRC | 08:27 | |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Try to turn treat warning as errors at tox https://review.opendev.org/714924 | 09:42 |
| *** lucasagomes has joined #openstack-rally | 09:56 | |
| lucasagomes | hi all, the rally job on the neutron and networking-ovn stable branches are broken due to this error here: http://paste.openstack.org/show/791794/ | 09:58 |
| lucasagomes | full logs: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_352/718351/1/check/networking-ovn-rally-task/352be03/job-output.txt | 09:58 |
| lucasagomes | I've been trying to figure out what it could be, we already have the override-checkout in zuul pointing to the 1.7.0 tag in the rally-openstack repository | 09:58 |
| lucasagomes | but it's still failing | 09:58 |
| andreykurilin | lucasagomes: hi | 09:59 |
| andreykurilin | strange | 09:59 |
| lucasagomes | I suspect this may be related to https://opendev.org/openstack/rally-openstack/commit/c4f0526dd8025956bacfedcd24c8a0e5a1b6005b ? | 09:59 |
| lucasagomes | andreykurilin, yeah | 09:59 |
| lucasagomes | I also tried this https://review.opendev.org/#/c/718351/ but it also didn't work, even tho I can see that the local_branch for the enabled plugins is now pointing to 1.7.0 | 09:59 |
| lucasagomes | do you guys have any idea how could I fix the problem > | 09:59 |
| lucasagomes | ?* | 09:59 |
| andreykurilin | lucasagomes: as a quick solution (to unblock your gates), I can revert changes to osresources... | 10:01 |
| andreykurilin | lucasagomes: do you have anything critical to merge at the moment or I have several hours for experiments ? | 10:03 |
| lucasagomes | andreykurilin, well we can set the rally job for non voting as well... or pehaps just propose the revert and we can add a depends-on to see if that really will fix the problem | 10:06 |
| lucasagomes | I will continue to investigate it today, I was looking some more ideas | 10:06 |
| lucasagomes | I don't understand why it may seem to be using the master branch of rally-openstack on the stable branches even tho I can clearly see the override-checkout pointing to the 1.7.0 tag | 10:07 |
| andreykurilin | lucasgomes: revert will fix the issue. as you may see from execution of previous rally commands, there are a bunch of deprecation warnings. These warnings came from rally-openstack master and they were not released yet. | 10:07 |
| lucasagomes | andreykurilin, fair yeah | 10:08 |
| lucasagomes | if you don't mind then, perhaps reverting is the quickest solution at the moment | 10:08 |
| andreykurilin | lucasgomes: let me try to reach zuul team to get more details. If I do not get answer quickly, I'll revert the change | 10:08 |
| lucasagomes | andreykurilin, sounds good, thank you very much! | 10:09 |
| andreykurilin | lucasgomes: sorry for this issue... | 10:09 |
| lucasagomes | it's all good, happens :-) | 10:09 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Try to fix existing users job https://review.opendev.org/718087 | 10:13 |
| andreykurilin | lucasagomes: can you restore your change? would like to push another revision for testing and do not want to create a lot of gerrit changes :) | 10:19 |
| lucasagomes | andreykurilin, sure | 10:21 |
| lucasagomes | andreykurilin, done | 10:21 |
| lucasagomes | feel free to push any changes you need there | 10:22 |
| andreykurilin | lucasagomes: well, new patch-set will not fix the issue, it should just hide some warnings. starting work on change to rally-openstack repo | 10:37 |
| lucasagomes | cool thanks a lot for looking into this | 10:38 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Revert a part of osresources changes https://review.opendev.org/718383 | 10:42 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Turn treat warning as errors at tox https://review.opendev.org/714924 | 11:06 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Turn treat warning as errors at tox https://review.opendev.org/714924 | 11:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Turn treat warning as errors at tox https://review.opendev.org/714924 | 11:59 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Revert a part of osresources changes https://review.opendev.org/718383 | 12:01 |
| lucasagomes | thanks andreykurilin for ^ | 12:18 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Get back jobs https://review.opendev.org/718412 | 12:25 |
| openstackgerrit | Merged openstack/rally-openstack master: [ci] Revert a part of osresources changes https://review.opendev.org/718383 | 13:05 |
| openstackgerrit | Merged openstack/rally-openstack master: [ci] Turn treat warning as errors at tox https://review.opendev.org/714924 | 13:24 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Move osresources script under ci roles https://review.opendev.org/718452 | 13:37 |
| andreykurilin | lucasagomes: revert helped | 14:34 |
| lucasagomes | andreykurilin, it absolutely did man! Thank you | 14:34 |
| lucasagomes | andreykurilin, I can see that test patch already passed the part it failed before | 14:35 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Move osresources script under ci roles https://review.opendev.org/718452 | 14:36 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Move osresources script under ci roles https://review.opendev.org/718452 | 15:32 |
| openstackgerrit | Merged openstack/rally-openstack master: [ci] Get back jobs https://review.opendev.org/718412 | 15:44 |
| *** lucasagomes has quit IRC | 16:03 | |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Move osresources script under ci roles https://review.opendev.org/718452 | 17:08 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Move osresources script under ci roles https://review.opendev.org/718452 | 17:49 |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Move osresources script under ci roles https://review.opendev.org/718452 | 18:24 |
| *** anilvenkata has quit IRC | 18:48 | |
| *** anilvenkata has joined #openstack-rally | 18:50 | |
| *** anilvenkata has quit IRC | 19:03 | |
| openstackgerrit | Andrey Kurilin proposed openstack/rally-openstack master: [ci] Move osresources script under ci roles https://review.opendev.org/718452 | 20:45 |
| *** rcernin has joined #openstack-rally | 22:32 | |
| *** threestrands has joined #openstack-rally | 23:10 | |
| *** tosky has quit IRC | 23:20 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!