From 33bf14ecb187ba7af98414e4084864c52e495df7 Mon Sep 17 00:00:00 2001 From: Ken Dreyer Date: Thu, 25 Jun 2015 08:40:30 -0600 Subject: [PATCH] pull requests: rm "cron" settings From the Jenkins GitHub Pull Requests plugin docs on "useGitHubHooks": Checking this option will disable regular polling (cron) for changes in GitHub and will try to create a GitHub hook. Creating a GitHub hook requires that the user which is specified in the GitHub Pull Request Builder configuration has admin rights to the specified repository. Based on the documentation above, since our PR jobs have "github-hooks: true", the "cron" values have no effect and can be removed. --- .../config/definitions/ceph-build-pull-requests.yml | 1 - .../config/definitions/ceph-deploy-pull-requests.yml | 1 - .../config/definitions/ceph-qa-suite-pull-requests.yml | 1 - .../config/definitions/jenkins-slave-chef-pull-requests.yml | 1 - .../config/definitions/radosgw-agent-pull-requests.yml | 1 - .../config/definitions/teuthology-pull-requests.yml | 1 - 6 files changed, 6 deletions(-) diff --git a/ceph-build-pull-requests/config/definitions/ceph-build-pull-requests.yml b/ceph-build-pull-requests/config/definitions/ceph-build-pull-requests.yml index 9bca339f..a5e5c5db 100644 --- a/ceph-build-pull-requests/config/definitions/ceph-build-pull-requests.yml +++ b/ceph-build-pull-requests/config/definitions/ceph-build-pull-requests.yml @@ -26,7 +26,6 @@ triggers: - github-pull-request: - cron: '* * * * *' admin-list: - alfredodeza - ktdreyer diff --git a/ceph-deploy-pull-requests/config/definitions/ceph-deploy-pull-requests.yml b/ceph-deploy-pull-requests/config/definitions/ceph-deploy-pull-requests.yml index 9f4cf0f4..ce42827d 100644 --- a/ceph-deploy-pull-requests/config/definitions/ceph-deploy-pull-requests.yml +++ b/ceph-deploy-pull-requests/config/definitions/ceph-deploy-pull-requests.yml @@ -26,7 +26,6 @@ triggers: - github-pull-request: - cron: '* * * * *' admin-list: - alfredodeza - ktdreyer diff --git a/ceph-qa-suite-pull-requests/config/definitions/ceph-qa-suite-pull-requests.yml b/ceph-qa-suite-pull-requests/config/definitions/ceph-qa-suite-pull-requests.yml index f340432d..cd4372b5 100644 --- a/ceph-qa-suite-pull-requests/config/definitions/ceph-qa-suite-pull-requests.yml +++ b/ceph-qa-suite-pull-requests/config/definitions/ceph-qa-suite-pull-requests.yml @@ -26,7 +26,6 @@ triggers: - github-pull-request: - cron: '* * * * *' admin-list: - alfredodeza - ktdreyer diff --git a/jenkins-slave-chef-pull-requests/config/definitions/jenkins-slave-chef-pull-requests.yml b/jenkins-slave-chef-pull-requests/config/definitions/jenkins-slave-chef-pull-requests.yml index 1939d5dc..f705223d 100644 --- a/jenkins-slave-chef-pull-requests/config/definitions/jenkins-slave-chef-pull-requests.yml +++ b/jenkins-slave-chef-pull-requests/config/definitions/jenkins-slave-chef-pull-requests.yml @@ -21,7 +21,6 @@ triggers: - github-pull-request: - cron: '* * * * *' admin-list: - alfredodeza - ktdreyer diff --git a/radosgw-agent-pull-requests/config/definitions/radosgw-agent-pull-requests.yml b/radosgw-agent-pull-requests/config/definitions/radosgw-agent-pull-requests.yml index 1bd6094d..a72f713b 100644 --- a/radosgw-agent-pull-requests/config/definitions/radosgw-agent-pull-requests.yml +++ b/radosgw-agent-pull-requests/config/definitions/radosgw-agent-pull-requests.yml @@ -26,7 +26,6 @@ triggers: - github-pull-request: - cron: '* * * * *' admin-list: - alfredodeza - jdurgin diff --git a/teuthology-pull-requests/config/definitions/teuthology-pull-requests.yml b/teuthology-pull-requests/config/definitions/teuthology-pull-requests.yml index fb751388..2c11b101 100644 --- a/teuthology-pull-requests/config/definitions/teuthology-pull-requests.yml +++ b/teuthology-pull-requests/config/definitions/teuthology-pull-requests.yml @@ -26,7 +26,6 @@ triggers: - github-pull-request: - cron: '* * * * *' admin-list: - alfredodeza - ktdreyer -- 2.39.5