| *** sanjayu_ has joined #zuul | 00:00 | |
| *** saneax has quit IRC | 00:03 | |
| *** tosky has quit IRC | 00:08 | |
| *** hashar has quit IRC | 00:11 | |
| *** hamalq has quit IRC | 00:15 | |
| *** zenkuro has quit IRC | 00:30 | |
| *** sanjayu_ has quit IRC | 01:15 | |
| *** jamesmcarthur has joined #zuul | 01:21 | |
| *** jamesmcarthur has quit IRC | 01:32 | |
| *** ikhan has joined #zuul | 02:04 | |
| *** iamweswilson has quit IRC | 02:50 | |
| *** PrinzElvis has quit IRC | 02:51 | |
| *** webknjaz has quit IRC | 02:51 | |
| *** aprice has quit IRC | 02:51 | |
| *** iamweswilson has joined #zuul | 02:51 | |
| *** dcastellani has quit IRC | 02:52 | |
| *** dcastellani has joined #zuul | 02:53 | |
| *** PrinzElvis has joined #zuul | 02:53 | |
| *** webknjaz has joined #zuul | 02:53 | |
| *** aprice has joined #zuul | 02:54 | |
| *** rfolco|bbl has quit IRC | 02:56 | |
| *** bhavikdbavishi has joined #zuul | 03:29 | |
| *** bhavikdbavishi1 has joined #zuul | 03:38 | |
| *** bhavikdbavishi has quit IRC | 03:40 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:40 | |
| *** jamesmcarthur has joined #zuul | 04:33 | |
| *** jamesmcarthur has quit IRC | 04:33 | |
| *** jamesmcarthur has joined #zuul | 04:56 | |
| *** jamesmcarthur has quit IRC | 05:06 | |
| *** jamesmcarthur has joined #zuul | 05:07 | |
| *** jamesmcarthur has quit IRC | 05:12 | |
| *** sanjayu_ has joined #zuul | 05:14 | |
| *** sanjayu_ has quit IRC | 05:19 | |
| *** bhavikdbavishi has quit IRC | 05:28 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:33 | |
| *** jamesmcarthur has joined #zuul | 05:55 | |
| *** bhavikdbavishi has joined #zuul | 06:05 | |
| *** jamesmcarthur has quit IRC | 06:33 | |
| *** bhavikdbavishi has quit IRC | 07:02 | |
| zbr | corvus: i am againt disabling the warning, it is valid, if it annoys someone they should fix the issue instead of hiding the dirt | 07:30 |
|---|---|---|
| openstackgerrit | zbr proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 07:34 |
| *** bhavikdbavishi has joined #zuul | 07:45 | |
| *** bhavikdbavishi has quit IRC | 08:13 | |
| *** vorotech has joined #zuul | 08:51 | |
| *** sanjayu_ has joined #zuul | 09:02 | |
| *** sanjayu_ has quit IRC | 09:14 | |
| *** vorotech has quit IRC | 09:16 | |
| *** jamesmcarthur has joined #zuul | 10:31 | |
| *** jamesmcarthur has quit IRC | 10:36 | |
| *** tosky has joined #zuul | 11:23 | |
| *** zenkuro has joined #zuul | 11:25 | |
| *** rfolco|bbl has joined #zuul | 12:04 | |
| *** rfolco|bbl is now known as rfolco | 12:25 | |
| *** jamesmcarthur has joined #zuul | 12:46 | |
| *** jamesmcarthur has quit IRC | 13:49 | |
| *** tosky_ has joined #zuul | 13:50 | |
| *** tosky is now known as Guest10516 | 13:52 | |
| *** tosky_ is now known as tosky | 13:52 | |
| corvus | zbr: maybe on monday you can chat with avass and find out where it's bothering him | 14:21 |
| zbr | corvus: sure. there are more 208 to fix but I am not going to start them before we get this one in. | 14:22 |
| zbr | getting reviews performed was quite slow, so i preferred to focus on other projects, like molecule. | 14:24 |
| *** vorotech has joined #zuul | 14:47 | |
| *** zenkuro has quit IRC | 15:03 | |
| *** zenkuro has joined #zuul | 15:03 | |
| *** vorotech has quit IRC | 15:40 | |
| *** vorotech has joined #zuul | 15:42 | |
| *** bhavikdbavishi has joined #zuul | 15:55 | |
| *** tosky has quit IRC | 16:12 | |
| *** vorotech has quit IRC | 16:20 | |
| avass | corvus, zbr: what's bothering me is that errors and warnings look exactly the same so the error is hidden amongst the warnings. | 16:35 |
| zbr | avass: file a bug on the linter and I will raise a patch to make it easy to distinguish. | 16:35 |
| corvus | avass: are the warnings for new code you're adding or for unrelated code? | 16:36 |
| zbr | i have few alternatives in mind, one would be to report warnings as W208 instead of E208, another one would be to display them in different sections. | 16:36 |
| avass | corvus: the warnings are for unrelated code | 16:36 |
| zbr | the warnings are for existing code, but the linter is not able to guess what is the new code and what is old code. | 16:37 |
| zbr | in fact I do have PR in review that enables a progressive mode which would sort current issue at https://github.com/ansible/ansible-lint/pull/1065 | 16:37 |
| zbr | but it got stalled due to lack of reviewers | 16:38 |
| avass | It would have helped if the warnings was a different colour, or not grouped together with the errors in the ouput from the linter | 16:38 |
| zbr | next week the lack of reviewers issue is supposed to be addresed. | 16:38 |
| zbr | avass: you are now using the pep8 format, which is not flexible. | 16:39 |
| * corvus has to run | 16:39 | |
| zbr | but if not using pep8 format, we could easily make it easier to distinguish. | 16:39 |
| zbr | avass: in fact I think that you have review rights on linter, so you could help me ship required changes. | 16:40 |
| avass | Having a different color for warnings vs errors should still work though, no? | 16:40 |
| zbr | yes, no problem with color, just that zuul is still color blind, my patches were not merged. | 16:40 |
| avass | zbr: yeah.. don't really have time to keep up with the project so doing a review now and then sometimes takes a bit more effort :) | 16:41 |
| avass | but that would help when running the linter locally at least :) | 16:41 |
| zbr | avass: look at https://github.com/ansible/ansible-lint/pull/1065 -- if is ok, i could make a release. | 16:42 |
| avass | zbr: looking | 16:43 |
| zbr | and regarding visual difference, let me bake something for tomorrow and if you have few minutes to review we may have a fix released before monday. | 16:43 |
| zbr | you may laugh but zuul-jobs repo was the reason why I wrote the progressive mode. | 16:44 |
| zbr | i know current implementation is not perfect, but for zuul usage it should work. | 16:45 |
| zbr | the flaw is if someone tried to test multiple commits, current implementation makes the assumption that previous commit did already passed CI. | 16:46 |
| avass | zbr: i think it looks good, only minor details if you want to fix them | 17:03 |
| avass | I'll be back in a bit the sous vide just finished preparing my steak :) | 17:04 |
| *** jamesmcarthur has joined #zuul | 17:04 | |
| zbr | i will rephrase... | 17:05 |
| *** vorotech has joined #zuul | 17:18 | |
| *** vorotech has quit IRC | 17:23 | |
| *** vorotech has joined #zuul | 17:24 | |
| avass | zuul-jobs-maint: can we get another review on: https://review.opendev.org/#/c/748498/ | 17:38 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add nimble roles and job https://review.opendev.org/747865 | 17:41 |
| *** jamesmcarthur has quit IRC | 17:44 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Enable progressive mode with ansible-lint https://review.opendev.org/760691 | 17:52 |
| zbr | avass: currently linter violations are sorted by file before printing, this means that I cannot separate warnings from errors and when run in monochrome mode with pep8 format, the only difference will be a letter W instead of E. | 18:02 |
| zbr | for other modes it will be easier to distinguish due to coloring, or more text being printed. | 18:03 |
| zbr | i am open to suggestions | 18:03 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Enable progressive mode with ansible-lint https://review.opendev.org/760691 | 18:10 |
| avass | being able to differentiate warnings from errors by color when running on your own machine should be good enough for now. | 18:16 |
| *** jamesmcarthur has joined #zuul | 18:22 | |
| *** jamesmcarthur has quit IRC | 18:32 | |
| *** wuchunyang has joined #zuul | 18:33 | |
| *** wuchunyang has quit IRC | 18:38 | |
| openstackgerrit | Vitaliy Lotorev proposed zuul/zuul-jobs master: emit-job-header: Print username in node information https://review.opendev.org/760692 | 18:59 |
| *** jamesmcarthur has joined #zuul | 19:00 | |
| openstackgerrit | Vitaliy Lotorev proposed zuul/zuul-jobs master: emit-job-header: Print username in node information https://review.opendev.org/760692 | 19:01 |
| *** jamesmcarthur has quit IRC | 19:05 | |
| *** tosky has joined #zuul | 19:07 | |
| openstackgerrit | Vitaliy Lotorev proposed zuul/zuul-jobs master: emit-job-header: Print username in node information https://review.opendev.org/760692 | 19:07 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add option to install kubernetes with kind https://review.opendev.org/740935 | 19:11 |
| *** rfolco has quit IRC | 19:16 | |
| *** bhavikdbavishi has quit IRC | 19:22 | |
| *** vorotech has quit IRC | 19:22 | |
| *** jamesmcarthur has joined #zuul | 19:57 | |
| *** jamesmcarthur has quit IRC | 20:17 | |
| *** zenkuro has quit IRC | 20:23 | |
| *** zenkuro has joined #zuul | 20:23 | |
| *** polls45 has joined #zuul | 20:49 | |
| *** sassyn has joined #zuul | 20:54 | |
| *** jamesmcarthur has joined #zuul | 20:55 | |
| sassyn | Hi everyone. Quick question again: I have around 200 repos. have configured 20 merger... but the speed is still very slow after doing reboot. is there any way to speed this up? | 20:55 |
| sassyn | expect of adding more merger? | 20:56 |
| fungi | in opendev we have around 2000 repos and have 8 dedicated merger servers in addition to our 12 executors, startup time for the scheduler is a few minutes... maybe 5-10? | 20:58 |
| fungi | i guess if you have a higher average number of branches per repository than we do, that could cause it to take much longer | 20:58 |
| sassyn | It takes around 180min after reboot | 21:00 |
| sassyn | and yes we have many many branches | 21:01 |
| sassyn | Hi fungi - hope all is good with you | 21:01 |
| sassyn | thank u for the help! | 21:02 |
| fungi | i suspect our branch average is probably roughly 2 branches per repository | 21:02 |
| fungi | yep, doing well here | 21:02 |
| sassyn | I didn't forget the beer! :-) | 21:02 |
| fungi | heh, i have plenty, but thanks for the thought | 21:02 |
| sassyn | we have around100 bracnhes | 21:02 |
| sassyn | in some of the repos | 21:03 |
| fungi | we have some repos with maybe a dozen branches, but probably the majority have only one | 21:05 |
| avass | we've got something like 800 branches and that takes about 10-20min to start up running 6 executors. but the repo with most of those branches is quite big as well | 21:13 |
| avass | sassyn: the scheduler is reporting number of queued jobs to statsd if you've got that set up. That way you can see how many merge jobs actually run during startup | 21:17 |
| sassyn | avaas thank u | 21:18 |
| sassyn | gearadmin is the tool i used to check the status | 21:18 |
| sassyn | merger:cat 4793 21 21 | 21:19 |
| avass | fungi: do you have a link to your grafana? I always have a hard time finding that :( | 21:19 |
| avass | ah http://grafana.openstack.org/ | 21:19 |
| avass | sassyn: it would looke something like this: http://grafana.openstack.org/d/T6vSHcSik/zuul-status?orgId=1 | 21:20 |
| avass | sassyn: yeah I guess that works as well :) | 21:22 |
| sassyn | very nice dashboard | 21:23 |
| sassyn | with the grafana! | 21:23 |
| sassyn | i plan to do it as well | 21:23 |
| sassyn | thanks | 21:23 |
| sassyn | i just trying to push this | 21:23 |
| sassyn | in the production | 21:23 |
| sassyn | and the team want to have 15 min time if server crash | 21:23 |
| sassyn | before going into prodtucion | 21:23 |
| sassyn | such a great took | 21:23 |
| *** jamesmcarthur has quit IRC | 21:44 | |
| *** sai438 has joined #zuul | 22:32 | |
| *** pmannidi has quit IRC | 22:33 | |
| *** sanjayu_ has joined #zuul | 22:33 | |
| *** jamesmcarthur has joined #zuul | 22:53 | |
| *** sanjayu__ has joined #zuul | 23:00 | |
| *** SotK has quit IRC | 23:02 | |
| *** SotK has joined #zuul | 23:03 | |
| *** sanjayu_ has quit IRC | 23:03 | |
| *** jamesmcarthur has quit IRC | 23:04 | |
| *** jamesmcarthur has joined #zuul | 23:47 | |
| *** sanjayu__ has quit IRC | 23:51 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!