diff options
author | Thanh Ha <thanh.ha@linuxfoundation.org> | 2016-10-14 00:39:37 -0400 |
---|---|---|
committer | Thanh Ha <thanh.ha@linuxfoundation.org> | 2016-10-14 00:51:17 -0400 |
commit | 21316a6aa857071e89e4d512613a785eddac2ae5 (patch) | |
tree | 0812243ad2b1e7025359e2b4ad88d5364e52cd92 /jjb/csit | |
parent | 61ff560afc8381d983571ad7395c4d9db1ca998f (diff) |
Use fdio-infra-wrappers to simplify common wrapper use
- Move timestamps to fdio-infra-wrappers as every job is using it
- Move ssh-credentials into fdio-infra-wrappers as every job is using it
- Ensuer ALL jobs are using fdio-infra-wrappers (this should be policy!)
FD.io has this convenient wrapper called fdio-infra-wrappers which is a
simple macro to configure common wrapper functionality that all jobs
should be configured with all jobs are already configuring it too so
simplify job configuration and ensuring we don't have duplication
everywhere.
Less lines of code and more code reuse makes for easier maintenance :)
Change-Id: Ia73b09dfc556f684c43853797231da84a9c517af
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Diffstat (limited to 'jjb/csit')
-rw-r--r-- | jjb/csit/csit-vpp-honeycomb.yaml | 5 | ||||
-rw-r--r-- | jjb/csit/csit-vpp-pylint.yaml | 5 | ||||
-rw-r--r-- | jjb/csit/csit.yaml | 20 |
3 files changed, 4 insertions, 26 deletions
diff --git a/jjb/csit/csit-vpp-honeycomb.yaml b/jjb/csit/csit-vpp-honeycomb.yaml index 01235dfb3..81589ee24 100644 --- a/jjb/csit/csit-vpp-honeycomb.yaml +++ b/jjb/csit/csit-vpp-honeycomb.yaml @@ -34,9 +34,8 @@ branch: '{branch}' wrappers: - - ssh-agent-credentials: - users: - - 'jenkins-gerrit-credentials' + - fdio-infra-wrappers: + build-timeout: '{build-timeout}' triggers: - build-result: diff --git a/jjb/csit/csit-vpp-pylint.yaml b/jjb/csit/csit-vpp-pylint.yaml index 03de48d4f..0d3068945 100644 --- a/jjb/csit/csit-vpp-pylint.yaml +++ b/jjb/csit/csit-vpp-pylint.yaml @@ -40,9 +40,8 @@ choosing-strategy: 'gerrit' wrappers: - - ssh-agent-credentials: - users: - - 'jenkins-gerrit-credentials' + - fdio-infra-wrappers: + build-timeout: '{build-timeout}' triggers: - gerrit-trigger-patch-submitted: diff --git a/jjb/csit/csit.yaml b/jjb/csit/csit.yaml index 0abd0cc59..7ddc89846 100644 --- a/jjb/csit/csit.yaml +++ b/jjb/csit/csit.yaml @@ -57,10 +57,6 @@ choosing-strategy: 'gerrit' wrappers: - - timestamps - - ssh-agent-credentials: - users: - - 'jenkins-gerrit-credentials' - fdio-infra-wrappers: build-timeout: '{build-timeout}' @@ -109,10 +105,6 @@ choosing-strategy: 'gerrit' wrappers: - - timestamps - - ssh-agent-credentials: - users: - - 'jenkins-gerrit-credentials' - fdio-infra-wrappers: build-timeout: 1400 @@ -294,10 +286,6 @@ branch: '{branch}' wrappers: - - timestamps - - ssh-agent-credentials: - users: - - 'jenkins-gerrit-credentials' - fdio-infra-wrappers: build-timeout: '480' @@ -352,10 +340,6 @@ branch: '{branch}' wrappers: - - timestamps - - ssh-agent-credentials: - users: - - 'jenkins-gerrit-credentials' - fdio-infra-wrappers: build-timeout: 480 @@ -403,10 +387,6 @@ branch: '{branch}' wrappers: - - timestamps - - ssh-agent-credentials: - users: - - 'jenkins-gerrit-credentials' - fdio-infra-wrappers: build-timeout: '1440' |