Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15481 )
Change subject: [build] Skip builds with no functional changes ...................................................................... Patch Set 4: (6 comments) http://gerrit.cloudera.org:8080/#/c/15481/4//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15481/4//COMMIT_MSG@9 PS4, Line 9: This patch introduces the option to skip builds in pre-commit when : the changes are in files or directories that do not impact the build : or test. I appreciate the motivation behind this, but are you sure this is desirable long term? It can actually be somewhat difficult to determine whether a given file/directory affects the build or a test (see my comment about 'examples'), and it's also going to be annoying to keep this list up-to-date with the reality in the repo. http://gerrit.cloudera.org:8080/#/c/15481/4//COMMIT_MSG@13 PS4, Line 13: This functionality is disabled by default but can be enabled on our Jenkins instance by setting `KUDU_ALLOW_SKIPPED_TESTS=1`. Too long. http://gerrit.cloudera.org:8080/#/c/15481/4//COMMIT_MSG@17 PS4, Line 17: precomit pre-commit http://gerrit.cloudera.org:8080/#/c/15481/4/build-support/jenkins/build-and-test.sh File build-support/jenkins/build-and-test.sh: http://gerrit.cloudera.org:8080/#/c/15481/4/build-support/jenkins/build-and-test.sh@96 PS4, Line 96: examples We build the C++ example in client-examples.sh, so we can't exclude the entire directory. http://gerrit.cloudera.org:8080/#/c/15481/4/build-support/jenkins/build-and-test.sh@96 PS4, Line 96: grep -qvE "^\.dockerignore|^docker/|^docs/|^examples/|^kubernetes/|LICENSE\.txt|NOTICE\.txt|.*\.adoc|.*\.md"; then Any way we could set this up with each entry on its own line? It'll be much easier to maintain that way. http://gerrit.cloudera.org:8080/#/c/15481/4/build-support/jenkins/build-and-test.sh@96 PS4, Line 96: .dockerignore Could add .gitignore and .ycm_extra_conf.py too. -- To view, visit http://gerrit.cloudera.org:8080/15481 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If56db6593ce7aa64b562a9fb277a337bffc7b532 Gerrit-Change-Number: 15481 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 18 Mar 2020 20:32:38 +0000 Gerrit-HasComments: Yes
