Skip to content

Commit

Permalink
Merge pull request #2828 from AdamBrousseau/remove_unused_function_se…
Browse files Browse the repository at this point in the history
…t_job_names

Remove unused function set_job_names
  • Loading branch information
llxia committed Sep 11, 2018
2 parents 165d995 + af96720 commit 95b4369
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
10 changes: 0 additions & 10 deletions buildenv/jenkins/common/variables-functions
Original file line number Diff line number Diff line change
Expand Up @@ -444,15 +444,6 @@ def set_artifactory_config() {
}
}

/*
* Sets the names of the downstream build & test jobs
*/
def set_job_names() {
BUILD_JOB_NAME = VARIABLES.build_job_prefix + "${SDK_VERSION}-${SPEC}"
SANITY_JOB_NAME = VARIABLES.sanity_job_prefix + "${SDK_VERSION}-${SPEC}"
EXTENDED_JOB_NAME = VARIABLES.extended_job_prefix + "${SDK_VERSION}-${SPEC}"
}

def set_job_properties(JOB_TYPE) {
/*************************
* Setup Jenkins Properties
Expand Down Expand Up @@ -668,7 +659,6 @@ def set_job_variables(job_type) {
set_sdk_variables()
set_test_targets()
set_slack_channel()
set_job_names()
break
case "wrapper":
//set variable for pipeline all/personal
Expand Down
3 changes: 0 additions & 3 deletions buildenv/jenkins/variables/defaults.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,6 @@ credentials:
github: 'b6987280-6402-458f-bdd6-7affc2e360d4'
test_dependencies_job_name: 'test.getDependency'
slack_channel: '#jenkins'
build_job_prefix: 'Build-JDK-'
sanity_job_prefix: 'Sanity-JDK-'
extended_job_prefix: 'Extended-JDK-'
build_discarder:
logs:
build: 20
Expand Down

0 comments on commit 95b4369

Please sign in to comment.