Skip to content

Commit fdf2d5d

Browse files
authored
Retry only on CI (#2480)
1 parent a392dfa commit fdf2d5d

File tree

3 files changed

+30
-14
lines changed

3 files changed

+30
-14
lines changed

build.gradle.kts

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import software.aws.toolkits.gradle.IdeVersions
1515
import software.aws.toolkits.gradle.changelog.tasks.GenerateGithubChangeLog
1616
import software.aws.toolkits.gradle.findFolders
1717
import software.aws.toolkits.gradle.getOrCreate
18+
import software.aws.toolkits.gradle.ciOnly
1819
import software.aws.toolkits.gradle.intellij
1920
import software.aws.toolkits.gradle.removeTask
2021
import software.aws.toolkits.gradle.resources.ValidateMessages
@@ -183,10 +184,12 @@ subprojects {
183184
html.isEnabled = true
184185
}
185186

186-
retry {
187-
failOnPassedAfterRetry.set(false)
188-
maxFailures.set(5)
189-
maxRetries.set(2)
187+
ciOnly {
188+
retry {
189+
failOnPassedAfterRetry.set(false)
190+
maxFailures.set(5)
191+
maxRetries.set(2)
192+
}
190193
}
191194
}
192195

@@ -227,10 +230,12 @@ subprojects {
227230

228231
mustRunAfter(tasks.test)
229232

230-
retry {
231-
failOnPassedAfterRetry.set(false)
232-
maxFailures.set(5)
233-
maxRetries.set(2)
233+
ciOnly {
234+
retry {
235+
failOnPassedAfterRetry.set(false)
236+
maxFailures.set(5)
237+
maxRetries.set(2)
238+
}
234239
}
235240
}
236241

@@ -251,7 +256,7 @@ subprojects {
251256
systemProperty("ide.show.tips.on.startup.default.value", false)
252257

253258
systemProperty("aws.telemetry.skip_prompt", "true")
254-
if (System.getenv("CI") != null) {
259+
ciOnly {
255260
systemProperty("aws.sharedCredentialsFile", "/tmp/.aws/credentials")
256261
}
257262

@@ -303,7 +308,6 @@ intellij {
303308
version = ideProfile.community.sdkVersion
304309
pluginName = "aws-jetbrains-toolkit"
305310
updateSinceUntilBuild = false
306-
downloadSources = System.getenv("CI") == null
307311
}
308312

309313
tasks.getByName<PrepareSandboxTask>("prepareSandbox") {

buildSrc/src/software/aws/toolkits/gradle/BuildScriptUtils.kt

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,12 @@ fun SourceSetContainer.getOrCreate(sourceSet: String, block: SourceSet.() -> Uni
3737
create(sourceSet).block()
3838
}
3939
}
40+
41+
/**
42+
* Only run the given block if this build is running within a CI system (e.g. GitHub actions, CodeBuild etc)
43+
*/
44+
fun Project.ciOnly(block: () -> Unit) {
45+
if (System.getenv("CI") != null) {
46+
block()
47+
}
48+
}

ui-tests/build.gradle.kts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
// Copyright 2020 Amazon.com, Inc. or its affiliates. All Rights Reserved.
22
// SPDX-License-Identifier: Apache-2.0
33

4+
import software.aws.toolkits.gradle.ciOnly
45
import java.net.URI
56

67
val remoteRobotPort: String by project
@@ -58,9 +59,11 @@ tasks.register<Test>("uiTestCore") {
5859
}
5960

6061
// uiTestCore needs its own version of this since it's not part of normal test tasks
61-
retry {
62-
failOnPassedAfterRetry.set(false)
63-
maxFailures.set(5)
64-
maxRetries.set(2)
62+
ciOnly {
63+
retry {
64+
failOnPassedAfterRetry.set(false)
65+
maxFailures.set(5)
66+
maxRetries.set(2)
67+
}
6568
}
6669
}

0 commit comments

Comments
 (0)