+1 for dropping it. Enrico, have you created a Jenkins view which gathers all Maven based builds?
Andor On Fri, Jun 14, 2019 at 7:44 AM Enrico Olivelli <[email protected]> wrote: > Il ven 14 giu 2019, 00:50 Michael Han <[email protected]> ha scritto: > > > I see now. This was a job I set up long time ago for my own testing > purpose > > only; it was not intended to be part of official commit workflow. I am > not > > sure how this job becomes the official post commit job though. If we have > > daily builds (do we) - then probably we don't need this. Though keep it > > does not hurt, either. > > > Actually it is the last tool that is using ant and it is the only one that > needs jira credentials. > > Personally I would drop it. > > Enrico > > > I have no strong preference one way or the other. > > > > > > On Wed, Jun 12, 2019 at 1:50 AM Andor Molnar <[email protected] > > > > wrote: > > > > > It might give some additional feedback to the committer/contributor > > whether > > > the patch has been successfully integrated into the target branch. > We've > > > seen issues when the merge build was outdated, showing green light on > the > > > pull request and the commit eventually broke master after merging. > > > > > > I'm not 100% sure we need it, maybe other folks can give some more > color > > to > > > this. > > > > > > Btw. do we already have a View in jenkins for Maven builds like this > one > > > for Ant: https://builds.apache.org/view/S-Z/view/ZooKeeper/ > > > > > > Andor > > > > > > > > > > > > On Wed, Jun 12, 2019 at 10:33 AM Enrico Olivelli <[email protected]> > > > wrote: > > > > > > > Michael > > > > it seems that the job is trying to send emails to you at > > > [email protected] > > > > but there is a typo. > > > > > > > > Do we still need this job? > > > > As we are dropping ANT we have to migrate it, but I don't find much > > value > > > > as we are already performing daily builds and precommits. > > > > Having such comment in jira is not so important > > > > > > > > Enrico > > > > > > > > Enrico > > > > > > > > Il mer 12 giu 2019, 10:04 Enrico Olivelli <[email protected]> ha > > > > scritto: > > > > > > > > > Self answered from the message on jira... > > > > > > > > > > It is zookeeper-trunk-single-thread > > > > > > > > > > Thanks Michael > > > > > > > > > > Enrico > > > > > > > > > > SUCCESS: Integrated in Jenkins build Zookeeper-trunk-single-thread > > #388 > > > > > (See [ > > https://builds.apache.org/job/Zookeeper-trunk-single-thread/388/ > > > ]) > > > > > > > > > > ZOOKEEPER-3366: Pluggable metrics system for ZooKeeper - move > > remaining > > > > > (nkalmar: rev 6af151a7e44760c33165e58d3b33a4d2760283a4) > > > > > * (edit) > zookeeper-server/src/main/java/org/apache/zookeeper/metrics/ > > > > > MetricsContext.java > > > > > * (edit) > zookeeper-server/src/main/java/org/apache/zookeeper/server/ > > > > > command/MonitorCommand.java > > > > > * (edit) > zookeeper-server/src/main/java/org/apache/zookeeper/server/ > > > > > > > > > > > > > > > > > > > > Il mer 12 giu 2019, 02:00 Michael Han <[email protected]> ha > scritto: > > > > > > > > > >> Hi Enrico, > > > > >> > > > > >> I don't think it's a post commit job that adds the "Issue resolved > > by > > > > pull > > > > >> request...." comment (if this is the comment you were referring > > to.). > > > > This > > > > >> was added by the merge script through JIRA api. > > > > >> > > > > >> On Mon, Jun 10, 2019 at 1:34 PM Enrico Olivelli < > > [email protected]> > > > > >> wrote: > > > > >> > > > > >> > Hi ZooKeepers, > > > > >> > I am looking for the Post-Commit job, the one that adds that > JIRA > > > > >> comment > > > > >> > after we merge a PR. > > > > >> > > > > > >> > I would like to understand it and decide whether to drop it or > to > > > move > > > > >> it > > > > >> > to Maven. > > > > >> > Personally I don't find much value in that task. > > > > >> > > > > > >> > This is my search [1] (I hope that the link works for you) > > > > >> > > > > > >> > Enrico > > > > >> > > > > > >> > [1] > > > > >> > > > > > >> > > > > > >> > > > > > > > > > > https://builds.apache.org/job/Zookeeper-trunk-single-thread/search/?q=zookeeper&Jenkins-Crumb=1e1a1d3bbc6a541e903422747a16c759 > > > > >> > > > > > >> > > > > > > > > > > > > > > >
