Skip to content

Commit

Permalink
[CI] Update precommit job captions with trigger phrase
Browse files Browse the repository at this point in the history

Descriptions of the changes in this PR:

*Motivation*

Sometime people don't know how to retrigger a bookkeeper precommit job.

*Changes*

This PR is adding the trigger phrase to the job caption. so people will know
how to retrigger the jobs in the PR itself.




Reviewers: Enrico Olivelli <eolivelli@gmail.com>

This closes apache#1881 from sijie/update_jenkins_for_self_instructions
  • Loading branch information
sijie authored Dec 12, 2018
1 parent 9a0da5c commit 2c59032
Show file tree
Hide file tree
Showing 9 changed files with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ freeStyleJob('bookkeeper_precommit_bookie_tests') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'Bookie Tests',
'Bookie Tests (trigger via `run bookkeeper-server bookie tests`)',
'.*(re)?run bookkeeper-server (bookie )?tests.*',
'.*\\[x\\] \\[skip bookkeeper-server (bookie )?tests\\].*',
true)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ freeStyleJob('bookkeeper_precommit_client_tests') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'Client Tests',
'Client Tests (trigger via `run bookkeeper-server client tests`)',
'.*(re)?run bookkeeper-server (client )?tests.*',
'.*\\[x\\] \\[skip bookkeeper-server (client )?tests\\].*',
true)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ freeStyleJob('bookkeeper_precommit_integrationtests') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'Integration Tests',
'Integration Tests (trigger via `run integration tests`)',
'.*(re)?run integration tests.*',
'.*\\[x\\] \\[skip integration tests\\].*')

Expand Down
2 changes: 1 addition & 1 deletion .test-infra/jenkins/job_bookkeeper_precommit_java8.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ mavenJob('bookkeeper_precommit_pullrequest_java8') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'Build (Java 8)',
'Build (Java 8) (trigger via `rebuild java8`)',
'.*(re)?build java8.*',
'.*\\[x\\] \\[skip build java8\\].*')

Expand Down
2 changes: 1 addition & 1 deletion .test-infra/jenkins/job_bookkeeper_precommit_java9.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ mavenJob('bookkeeper_precommit_pullrequest_java9') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'Build (Java 9)',
'Build (Java 9) (trigger via `rebuild java9`)',
'.*(re)?build java9.*',
'.*\\[x\\] \\[skip build java9\\].*')

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ freeStyleJob('bookkeeper_precommit_remaining_tests') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'All Other Tests',
'All Other Tests (trigger via `run bookkeeper-server remaining tests`)',
'.*(re)?run bookkeeper-server (remaining )?tests.*',
'.*\\[x\\] \\[skip bookkeeper-server (remaining )?tests\\].*',
true)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ freeStyleJob('bookkeeper_precommit_replication_tests') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'Replication Tests',
'Replication Tests (trigger via `run bookkeeper-server replication tests`)',
'.*(re)?run bookkeeper-server (replication )?tests.*',
'.*\\[x\\] \\[skip bookkeeper-server (replication )?tests\\].*',
true)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ freeStyleJob('bookkeeper_precommit_tls_tests') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'TLS Tests',
'TLS Tests (trigger via `run bookkeeper-server tls tests`)',
'.*(re)?run bookkeeper-server (tls )?tests.*',
'.*\\[x\\] \\[skip bookkeeper-server (tls )?tests\\].*',
true)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ mavenJob('bookkeeper_precommit_pullrequest_validation') {
// Sets that this is a PreCommit job.
common_job_properties.setPreCommit(
delegate,
'PR Validation',
'PR Validation (trigger via `run pr validation`)',
'.*(re)?run pr validation.*',
'',
true)
Expand Down

0 comments on commit 2c59032

Please sign in to comment.