[ https://issues.apache.org/jira/browse/HBASE-22924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17643430#comment-17643430 ]
Sean Busbey commented on HBASE-22924: ------------------------------------- that interpretation is correct Duo. this ticket is for a) adding to the jenkinsfile the needed configs to skip non-PR builds AND b) remove the current shell script clause that skips non-PR builds. this ticket would also be obsolete if someone were to combine our build pipelines for PRs with the ones for nightlies. But that would be a lot more work. > GitHUB PR job should use when clause to filter to just PRs. > ----------------------------------------------------------- > > Key: HBASE-22924 > URL: https://issues.apache.org/jira/browse/HBASE-22924 > Project: HBase > Issue Type: Improvement > Components: build, community > Reporter: Sean Busbey > Assignee: Shanky Sharma > Priority: Minor > Labels: beginner > > Right now the GitHub PR checking job uses a Jenkins environment variable to > stop branch builds: > {code} > # If CHANGE_URL is set (e.g., Github Branch Source > plugin), process it. > # Otherwise exit, because we don't want HBase to do a > # full build. We wouldn't normally do this check for > smaller > # projects. :) > if [[ -z "${CHANGE_URL}" ]]; then > echo "Full build skipped" > > "${WORKSPACE}/${PATCHDIR}/report.html" > exit 0 > fi > {code} > instead we should use a Jenkinsfile {{when}} clause > ([ref|https://jenkins.io/doc/book/pipeline/syntax/#when]) and the > {{changeRequest()}} filter to say we want to build PRs. Assuming the eventual > adaptation of the JIRA precommit job properly sets that it's testing a change > this should also help reuse there. > {code} > stage ('precommit-run') { > when { changeRequest() } > steps { > ... > } > } > {code} > it would also be nice to fold things into a single stage so that we skip the > yetus checkout when we're not going to run tests. -- This message was sent by Atlassian Jira (v8.20.10#820010)