From ba8b9368e124cbd99d0718adaec4489844dce87e Mon Sep 17 00:00:00 2001 From: jorgee Date: Tue, 7 Jan 2025 13:48:53 +0100 Subject: [PATCH 1/3] adding warining messages when Cloudinfo fails Signed-off-by: jorgee --- .../nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy | 2 ++ 1 file changed, 2 insertions(+) diff --git a/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy b/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy index 4bf66f9860..3ab252363e 100644 --- a/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy +++ b/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy @@ -606,6 +606,7 @@ class GoogleBatchTaskHandler extends TaskHandler implements FusionAwareTask { } } catch (Exception e) { + log.warn "Cannot select machine type using Seqera Cloudinfo for task: `${task.lazyName()}`. This feature can be disabled with setting the environment variable NXF_CLOUDINFO_ENABLED to false." log.debug "[GOOGLE BATCH] Cannot select machine type using Seqera Cloudinfo for task: `${task.lazyName()}` - ${e.message}" } @@ -618,6 +619,7 @@ class GoogleBatchTaskHandler extends TaskHandler implements FusionAwareTask { ) // Fallback to Google Batch automatically deduce from requested resources + log.warn "Using Google Batch default machine type." return null } From 83c87b94ad71e3b3336cf0bba91d71c969296ffd Mon Sep 17 00:00:00 2001 From: Jorge Ejarque Date: Thu, 16 Jan 2025 12:30:52 +0100 Subject: [PATCH 2/3] Update plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy Co-authored-by: Paolo Di Tommaso Signed-off-by: Jorge Ejarque --- .../nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy b/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy index 3ab252363e..a104e5185e 100644 --- a/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy +++ b/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy @@ -606,8 +606,7 @@ class GoogleBatchTaskHandler extends TaskHandler implements FusionAwareTask { } } catch (Exception e) { - log.warn "Cannot select machine type using Seqera Cloudinfo for task: `${task.lazyName()}`. This feature can be disabled with setting the environment variable NXF_CLOUDINFO_ENABLED to false." - log.debug "[GOOGLE BATCH] Cannot select machine type using Seqera Cloudinfo for task: `${task.lazyName()}` - ${e.message}" + log.warn "Cannot determine the machine type to be used for task: `${task.lazyName()}` - If this problem persists disable disable the Cloudinfo service by setting the variable NXF_CLOUDINFO_ENABLED=false in your environment", e } // Check if a specific machine type was provided by the user From 25be3d00a0a4a557cd8a5da010464db40f108199 Mon Sep 17 00:00:00 2001 From: jorgee Date: Thu, 16 Jan 2025 16:23:20 +0100 Subject: [PATCH 3/3] remove unnecesary warning Signed-off-by: jorgee --- .../nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy | 1 - 1 file changed, 1 deletion(-) diff --git a/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy b/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy index a104e5185e..c6e36e5ad5 100644 --- a/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy +++ b/plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchTaskHandler.groovy @@ -618,7 +618,6 @@ class GoogleBatchTaskHandler extends TaskHandler implements FusionAwareTask { ) // Fallback to Google Batch automatically deduce from requested resources - log.warn "Using Google Batch default machine type." return null }