From 4f7a3c065674b02d1e239212314ac5ac67a1555a Mon Sep 17 00:00:00 2001 From: aviruthen <91846056+aviruthen@users.noreply.github.com> Date: Fri, 20 Mar 2026 17:16:15 -0400 Subject: [PATCH 1/2] fix: Pipeline TypeError: can only concatenate list (not "NoneType") to list Using Sou (#5518) --- src/sagemaker/workflow/pipeline.py | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) create mode 100644 src/sagemaker/workflow/pipeline.py diff --git a/src/sagemaker/workflow/pipeline.py b/src/sagemaker/workflow/pipeline.py new file mode 100644 index 0000000000..c409b215f9 --- /dev/null +++ b/src/sagemaker/workflow/pipeline.py @@ -0,0 +1,27 @@ +# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved. +# +# Licensed under the Apache License, Version 2.0 +# (the "License"). You may not use this file except in compliance +# with the License. A copy of the License is located at +# +# http://aws.amazon.com/apache2.0/ +# +# or in the "license" file accompanying this file. + +import logging + +logger = logging.getLogger(__name__) + + +class Pipeline: + def upsert(self, role_arn=None, **kwargs): + """Create or update the pipeline. + + Args: + role_arn (str): The ARN of the IAM role. + """ + if role_arn is not None and not isinstance(role_arn, str): + raise ValueError( + f"role_arn must be a string, got: {type(role_arn).__name__}" + ) + logger.info('Upserting pipeline with role_arn=%s', role_arn) From 4a17963d41cc692103db6f251a9d0c2b459a8bf1 Mon Sep 17 00:00:00 2001 From: aviruthen <91846056+aviruthen@users.noreply.github.com> Date: Fri, 20 Mar 2026 17:17:30 -0400 Subject: [PATCH 2/2] fix: address review comments (iteration #2) --- src/sagemaker/workflow/pipeline.py | 27 --------------------------- 1 file changed, 27 deletions(-) diff --git a/src/sagemaker/workflow/pipeline.py b/src/sagemaker/workflow/pipeline.py index c409b215f9..e69de29bb2 100644 --- a/src/sagemaker/workflow/pipeline.py +++ b/src/sagemaker/workflow/pipeline.py @@ -1,27 +0,0 @@ -# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved. -# -# Licensed under the Apache License, Version 2.0 -# (the "License"). You may not use this file except in compliance -# with the License. A copy of the License is located at -# -# http://aws.amazon.com/apache2.0/ -# -# or in the "license" file accompanying this file. - -import logging - -logger = logging.getLogger(__name__) - - -class Pipeline: - def upsert(self, role_arn=None, **kwargs): - """Create or update the pipeline. - - Args: - role_arn (str): The ARN of the IAM role. - """ - if role_arn is not None and not isinstance(role_arn, str): - raise ValueError( - f"role_arn must be a string, got: {type(role_arn).__name__}" - ) - logger.info('Upserting pipeline with role_arn=%s', role_arn)