Skip to content

Commit

Permalink
Merge pull request #106 from jenkins-infra/INFRA-2210-hotfix
Browse files Browse the repository at this point in the history
`buildPlugin()` - Temporarily disable Windows builds and force ACI
  • Loading branch information
oleg-nenashev authored Aug 15, 2019
2 parents f23fffe + c3fe5b6 commit 9419093
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions vars/buildPlugin.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,13 @@ def call(Map params = [:]) {
def repo = params.containsKey('repo') ? params.repo : null
def failFast = params.containsKey('failFast') ? params.failFast : true
def timeoutValue = params.containsKey('timeout') ? params.timeout : 60
//TODO: revert once https://groups.google.com/forum/#!topic/jenkinsci-dev/v9d-XosOp2s is fixed
def useAci = params.containsKey('useAci') ? params.useAci : false
if (!useAci) {
echo "WARNING: Forcing Azure Container Instance, because standard agents are not stable at the moment. See INFRA-2210"
useAci = true
}

if(timeoutValue > 180) {
echo "Timeout value requested was $timeoutValue, lowering to 180 to avoid Jenkins project's resource abusive consumption"
timeoutValue = 180
Expand All @@ -31,6 +37,14 @@ def call(Map params = [:]) {
String javaLevel = config.javaLevel

String stageIdentifier = "${label}-${jdk}${jenkinsVersion ? '-' + jenkinsVersion : ''}"

//TODO: revert once https://groups.google.com/forum/#!topic/jenkinsci-dev/v9d-XosOp2s is fixed
if ("windows".equals(label)) {
echo "WARNING: Skipping ${stageIdentifier}, because Windows agents are not stable at the moment. See INFRA-2210"
return;
}


boolean first = tasks.size() == 1
boolean runFindbugs = first && params?.findbugs?.run
boolean runCheckstyle = first && params?.checkstyle?.run
Expand Down

0 comments on commit 9419093

Please sign in to comment.