Hello Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12471

to look at the new patch set (#5).

Change subject: [build-support] KUDU-2699 introduce TIDY build type
......................................................................

[build-support] KUDU-2699 introduce TIDY build type

This changelist introduces a new TIDY build type to facilitate running
clang_tidy_gerrit.py from the kudu-tidy-bot Jenkins job by
jenkins.kudu.apache.org.  The idea is to make sure all auto-generated
header files are present before running the clang-tidy tool on the
Kudu C++ source files.

As Adar noticed, our development process is such that new work goes
into master and any commits to release branches are pure backports.
There's not much point in running clang-tidy on those backports.
So, the proposal is to simply run the kudu-tidy-but job only on
the changes in the master branch.  That entails updating the
configuration of the kudu-tidy-but Jenkins job after this change
is pushed into the repo.

The corresponding command for the config for the kudu-tidy-but job is:
----------------------------------------------------------------------
\#!/bin/bash
export TIDYBOT_PASSWORD=<hidden>
export PATH=/usr/lib/ccache:$PATH
ccache -M 50G

set -ex

export BUILD_TYPE=TIDY
build-support/jenkins/build-and-test.sh
----------------------------------------------------------------------

Change-Id: I9b364620c9fb92202cacf0286df292dd4cc9952b
---
M CMakeLists.txt
M build-support/jenkins/build-and-test.sh
2 files changed, 28 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/71/12471/5
--
To view, visit http://gerrit.cloudera.org:8080/12471
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9b364620c9fb92202cacf0286df292dd4cc9952b
Gerrit-Change-Number: 12471
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to