Il ven 21 set 2018, 22:53 Michael Han <[email protected]> ha scritto: > >> It is the default config, I can drop. In BK sometimes it is useful in > order to save resources for docs/website changes for instance > > We have separate repo for ZK web site and our doc artifacts only > regenerate when we do a new release. So I think there is not much benefit > of keeping this option, as we always want Jenkins as the gate before > committing any code changes. >
Ok, will drop > >> But it seems to me that the 'Jenkins' line never goes green nor red, > it stays yellow forever > > I see what you mean. I think "*Jenkins *Pending — Jenkins is validating > pull request ..." should be removed, as it's not telling anything new. I > don't know how though. > > > > I can't find any hook anywhere, in CI, in precommit bash scripts/ant targets Does anyone has an explanation? Maybe it is an asf infra configuration Enrico > On Fri, Sep 21, 2018 at 11:20 AM, Enrico Olivelli <[email protected]> > wrote: > >> Il ven 21 set 2018, 19:11 Michael Han <[email protected]> ha scritto: >> >> > Thanks for the work, Enrico! Looks like it's working? >> > >> > >> if the description of the PR contains 'skip ci' validation won't be >> > run from jenkins >> > >> > Do we need this option? >> > >> >> >> It is the default config, I can drop. In BK sometimes it is useful in >> order >> to save resources for docs/website changes for instance >> >> > >> > >> doc/wiki to update and if there is a common place for storing CI Jobs >> > configurations >> > >> > jenkins job config is stored in the job itself. We don't version control >> > it separately. >> > With this update the "Jenkins" section is out of date: >> > https://cwiki.apache.org/confluence/display/ZOOKEEPER/HowToContribute >> > Would you please update the page to reflect latest state of the jenkins >> - >> > github integration? >> > >> >> Ok >> >> > >> > >> It seems that the first line with 'Jenkins' never goes away. >> > >> > Seems ok to me to keep two lines. The second line does not cover things >> > other than test (like find bug checks, release audit checks, etc). >> > >> >> TheI job execution is one, so the informative content is the same: red vs >> green. >> >> But it seems to me that the 'Jenkins' line never goes green nor red, it >> stays yellow forever >> >> >> Enrico >> >> >> > On Fri, Sep 21, 2018 at 5:51 AM, Enrico Olivelli <[email protected]> >> > wrote: >> > >> >> Too early... >> >> It seems that the first line with 'Jenkins' never goes away. >> >> >> >> Patrick, do you have some hint ? >> >> Otherwise I can ask to infra or [email protected] >> >> >> >> Enrico >> >> >> >> Il giorno ven 21 set 2018 alle ore 14:45 Enrico Olivelli >> >> <[email protected]> ha scritto: >> >> > >> >> > Okay >> >> > on github you see two lines, but there is only one job on Jenkins >> >> > >> >> > see >> >> > https://github.com/apache/zookeeper/pull/638 >> >> > >> >> > @Michael Han please double check >> >> > >> >> > If all it is okay I will send a new clean summary in another email >> >> thread >> >> > The short version is: >> >> > - an user sends a PR (against master, branch-3.5 and branch-3.4)> CI >> >> kicks in an validates, on GitHub you see the status and the results >> >> > - an "Apache Committer" (from any project I guess, as I am not a ZK >> >> committer but it works, it is important that he is registered in Apache >> >> Gihub 'organization') can add a comment with only this phrase 'retest >> this >> >> please' and the PR will be re-validated from CI >> >> > - if the description of the PR contains 'skip ci' validation won't be >> >> run from jenkins >> >> > >> >> > configuration is here: >> >> > https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/ >> >> > >> >> > >> >> > Please tell me if there is any doc/wiki to update and if there is a >> >> common place for storing CI Jobs configurations >> >> > >> >> > >> >> > Enrico >> >> > >> >> > Il giorno ven 21 set 2018 alle ore 11:56 Enrico Olivelli < >> >> [email protected]> ha scritto: >> >> >> >> >> >> >> >> >> >> >> >> Il giorno ven 21 set 2018 alle ore 11:27 Enrico Olivelli < >> >> [email protected]> ha scritto: >> >> >>> >> >> >>> I have set up the 'retest this please' trigger. >> >> >>> But now we have two jobs running for each pull request. >> >> >>> >> >> >>> I can't find the flag which makes the validation named 'Jenkins' >> start >> >> >>> >> >> >> https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/configure >> >> >>> >> >> >>> the name of the new one is 'default', and it maps to the same Job >> >> >> >> >> >> >> >> >> Actually the result for the new 'default' validation looks better >> >> >> see it here >> >> >> https://github.com/apache/zookeeper/pull/637 >> >> >> >> >> >> The old one just told 'Looks like there's a problem with this pull >> >> request' >> >> >> the new one reports ' FAILURE 1731 tests run, 1 skipped, 2 failed' >> >> >> >> >> >> Enrico >> >> >> >> >> >> >> >> >>> >> >> >>> is there any other trigger ? >> >> >>> I will continue to search >> >> >>> >> >> >>> Enrico >> >> >>> >> >> >>> Il giorno ven 21 set 2018 alle ore 09:33 Enrico Olivelli < >> >> [email protected]> ha scritto: >> >> >>>> >> >> >>>> I apologize if there will be some spam on the list >> >> >>>> >> >> >>>> When you enable this trigger asfgit starts to add comments like >> this: >> >> >>>> Can one of the admins verify this patch? >> >> >>>> >> >> >>>> I am looking for a solution. We had the same on BookKeeper jobs >> >> >>>> >> >> >>>> Enrico >> >> >>>> >> >> >>>> Il giorno ven 21 set 2018 alle ore 09:31 Enrico Olivelli < >> >> [email protected]> ha scritto: >> >> >>>>> >> >> >>>>> >> >> >>>>> >> >> >>>>> Il giorno ven 21 set 2018 alle ore 03:44 Michael Han < >> >> [email protected]> ha scritto: >> >> >>>>>> >> >> >>>>>> Thanks Enrico for volunteering, it'll be cool if we can fix >> this. >> >> >>>>> >> >> >>>>> >> >> >>>>> Ok, working now on >> >> >>>>> >> https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/ >> >> >>>>> >> >> >>>>> Stay tuned >> >> >>>>> Enrico >> >> >>>>> >> >> >>>>>> >> >> >>>>>> >> >> >>>>>> On Wed, Sep 19, 2018 at 10:42 PM, Enrico Olivelli < >> >> [email protected]> wrote: >> >> >>>>>>> >> >> >>>>>>> Il gio 20 set 2018, 01:17 Michael Han <[email protected]> ha >> >> scritto: >> >> >>>>>>> >> >> >>>>>>> > We've seen lots of confusions recently around Jenkins >> >> pre-commit check >> >> >>>>>>> > status on github pull request as well as questions like how >> to >> >> trigger >> >> >>>>>>> > Jenkins job to get a green build. I added a section "Jenkins >> >> Pre-commit >> >> >>>>>>> > Check" to our wiki page about this matter: >> >> >>>>>>> > >> >> >>>>>>> > >> >> https://cwiki.apache.org/confluence/display/ZOOKEEPER/HowToContribute >> >> >>>>>>> > >> >> >>>>>>> > I suspect the permission issue is about the fact that apache >> >> zookeeper >> >> >>>>>>> > github is read only for most of us including committers. Only >> >> asfbot has >> >> >>>>>>> > write access. When a Jenkins job is triggered through >> "Rebuild" >> >> button the >> >> >>>>>>> > credential it's using is not asfbot thus the final job result >> >> will fail to >> >> >>>>>>> > write back to github. I don't know how to solve this, but >> >> hopefully we have >> >> >>>>>>> > workarounds (which though require PR creator to be more >> >> diligent on >> >> >>>>>>> > obtaining a green build). >> >> >>>>>>> > >> >> >>>>>>> >> >> >>>>>>> In Bookkeeper we have enabled the magic 'retest this please' >> and >> >> it is very >> >> >>>>>>> useful. >> >> >>>>>>> >> >> >>>>>>> I can take a look in order to back port to ZK, I have access to >> >> ASF Jenkins >> >> >>>>>>> so I can check current configuration for ZK. >> >> >>>>>>> >> >> >>>>>>> In Bookkeeper we also have the configuration of Jenkins jobs on >> >> git, using >> >> >>>>>>> Jenkins DSL language. I can also take a look and see how to >> port >> >> to ZK >> >> >>>>>>> >> >> >>>>>>> >> >> >>>>>>> >> >> >>>>>>> Enrico >> >> >>>>>>> >> >> >>>>>>> >> >> >>>>>>> > Michael. >> >> >>>>>>> > >> >> >>>>>>> -- >> >> >>>>>>> >> >> >>>>>>> >> >> >>>>>>> -- Enrico Olivelli >> >> >>>>>> >> >> >>>>>> >> >> >> > >> > -- >> >> >> -- Enrico Olivelli >> > > -- -- Enrico Olivelli
