(beam) branch master updated: Use onlyif (#37578)

[yhu]

This is an automated email from the ASF dual-hosted git repository.

yhu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new cea7923eef6 Use onlyif (#37578)
cea7923eef6 is described below

commit cea7923eef68e3f490fbd7cf6fdd7b24ca126d12
Author: Vitaly Terentyev <vitaly.terentyev....@gmail.com>
AuthorDate: Thu Feb 12 19:11:47 2026 +0400

    Use onlyif (#37578)
---
 website/build.gradle | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/website/build.gradle b/website/build.gradle
index 790afe582eb..2aa04282e67 100644
--- a/website/build.gradle
+++ b/website/build.gradle
@@ -135,10 +135,13 @@ task copyGcpCredsToContainer(type: Exec) {
   dependsOn setupDockerContainer
 
   def gcpCredsHost = providers.gradleProperty("gcpCredsFile").orNull
-  if (gcpCredsHost) {
-    commandLine 'docker', 'cp', gcpCredsHost, 
"${->startDockerContainer.containerId()}:${gcpCredsInContainer}"
-  } else {
-    println "[performance_looks] -PgcpCredsFile not provided, skipping copy"
+
+  onlyIf { gcpCredsHost != null }
+
+  doFirst {
+    commandLine 'docker', 'cp',
+      gcpCredsHost,
+      "${->startDockerContainer.containerId()}:${gcpCredsInContainer}"
   }
 }
 
@@ -146,7 +149,10 @@ task downloadPerformanceLooks(type: Exec) {
   dependsOn setupDockerContainer, copyGcpCredsToContainer
 
   def gcpCredsHost = providers.gradleProperty("gcpCredsFile").orNull
-  if (gcpCredsHost) {
+
+  onlyIf { gcpCredsHost != null }
+
+  doFirst {
     commandLine 'docker', 'exec',
       "${-> setupDockerContainer.containerId()}", '/bin/bash', '-lc',
       """
@@ -173,8 +179,6 @@ task downloadPerformanceLooks(type: Exec) {
         echo "[performance_looks][WARNING] Website build will continue without 
updated images"
       fi
     """
-  } else {
-    println "[performance_looks] Skipping download of performance looks"
   }
 }