feat: Add custom-tags support for customer-provided tags (#444)

newman911 · web-flow · commit dfd7f1544796 · 2026-03-18T10:28:42.000-04:00
* feat: Add custom-tags support for customer-provided tags

Add custom_tags parameter to all module settings and app.py files
so that customer-provided tags flow through from capability deployment
to CDK resources. This matches the pattern used by Data Foundations
solution kit.

Affected modules:
- sagemaker-templates
- sagemaker-studio
- sagemaker-notebook
- sagemaker-ground-truth-labeling
- sagemaker-model-monitoring
- ray-operator
- ray-cluster
- ray-orchestrator

* chore: Release 3.2.7

---------

Co-authored-by: Nuri Boardman &lt;nurboard@amazon.com&gt;
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -11,6 +11,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### **Changed**
 
+## v3.2.7
+
+### **Added**
+
+- added optional `custom_tags` parameter to all modules for customer-provided tag support
+
 ## v3.2.6
 
 ### **Changed**
diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-3.2.6
+3.2.7
diff --git a/modules/eks/ray-cluster/app.py b/modules/eks/ray-cluster/app.py
@@ -50,6 +50,10 @@
     for tag_key, tag_value in app_settings.parameters.tags.items():
         Tags.of(app).add(tag_key, tag_value)
 
+if app_settings.parameters.custom_tags:
+    for tag_key, tag_value in app_settings.parameters.custom_tags.items():
+        Tags.of(app).add(tag_key, tag_value)
+
 Tags.of(app).add("SeedFarmerDeploymentName", app_settings.settings.deployment_name)
 Tags.of(app).add("SeedFarmerModuleName", app_settings.settings.module_name)
 Tags.of(app).add("SeedFarmerProjectName", app_settings.settings.project_name)
diff --git a/modules/eks/ray-cluster/settings.py b/modules/eks/ray-cluster/settings.py
@@ -62,6 +62,7 @@ class SeedFarmerParameters(CdkBaseSettings):
     worker_labels: Dict[str, str] = Field(default={})
     permissions_boundary_name: Optional[str] = Field(default=None)
     tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
 
 class SeedFarmerSettings(CdkBaseSettings):
diff --git a/modules/eks/ray-cluster/tests/test_app.py b/modules/eks/ray-cluster/tests/test_app.py
@@ -42,3 +42,8 @@ def test_eks_cluster_name(stack_defaults):
 
     with pytest.raises(ValidationError):
         import app  # noqa: F401
+
+
+def test_app_with_custom_tags(stack_defaults):
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
diff --git a/modules/eks/ray-operator/app.py b/modules/eks/ray-operator/app.py
@@ -55,6 +55,10 @@
     for tag_key, tag_value in app_settings.parameters.tags.items():
         Tags.of(app).add(tag_key, tag_value)
 
+if app_settings.parameters.custom_tags:
+    for tag_key, tag_value in app_settings.parameters.custom_tags.items():
+        Tags.of(app).add(tag_key, tag_value)
+
 Tags.of(app).add("SeedFarmerDeploymentName", app_settings.settings.deployment_name)
 Tags.of(app).add("SeedFarmerModuleName", app_settings.settings.module_name)
 Tags.of(app).add("SeedFarmerProjectName", app_settings.settings.project_name)
diff --git a/modules/eks/ray-operator/settings.py b/modules/eks/ray-operator/settings.py
@@ -38,6 +38,7 @@ class SeedFarmerParameters(CdkBaseSettings):
     data_bucket_name: Optional[str] = Field(default=None)
     permissions_boundary_name: Optional[str] = Field(default=None)
     tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
 
 class SeedFarmerSettings(CdkBaseSettings):
diff --git a/modules/eks/ray-operator/tests/test_app.py b/modules/eks/ray-operator/tests/test_app.py
@@ -44,3 +44,8 @@ def test_eks_cluster_name(stack_defaults):
 
     with pytest.raises(ValidationError):
         import app  # noqa: F401
+
+
+def test_app_with_custom_tags(stack_defaults):
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
diff --git a/modules/eks/ray-orchestrator/app.py b/modules/eks/ray-orchestrator/app.py
@@ -56,6 +56,10 @@
     for tag_key, tag_value in app_settings.parameters.tags.items():
         Tags.of(app).add(tag_key, tag_value)
 
+if app_settings.parameters.custom_tags:
+    for tag_key, tag_value in app_settings.parameters.custom_tags.items():
+        Tags.of(app).add(tag_key, tag_value)
+
 Tags.of(app).add("SeedFarmerDeploymentName", app_settings.settings.deployment_name)
 Tags.of(app).add("SeedFarmerModuleName", app_settings.settings.module_name)
 Tags.of(app).add("SeedFarmerProjectName", app_settings.settings.project_name)
diff --git a/modules/eks/ray-orchestrator/settings.py b/modules/eks/ray-orchestrator/settings.py
@@ -40,6 +40,7 @@ class SeedFarmerParameters(CdkBaseSettings):
     dra_export_path: str = Field(default="/ray/export/")
     permissions_boundary_name: Optional[str] = Field(default=None)
     tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
 
 class SeedFarmerSettings(CdkBaseSettings):
diff --git a/modules/eks/ray-orchestrator/tests/test_app.py b/modules/eks/ray-orchestrator/tests/test_app.py
@@ -45,3 +45,8 @@ def test_eks_cluster_name(stack_defaults):
 
     with pytest.raises(ValidationError):
         import app  # noqa: F401
+
+
+def test_app_with_custom_tags(stack_defaults):
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
diff --git a/modules/sagemaker/sagemaker-ground-truth-labeling/app.py b/modules/sagemaker/sagemaker-ground-truth-labeling/app.py
@@ -65,6 +65,14 @@
 
 aws_cdk.Aspects.of(app).add(cdk_nag.AwsSolutionsChecks(log_ignores=True))
 
+if app_settings.module_settings.tags:
+    for tag_key, tag_value in app_settings.module_settings.tags.items():
+        aws_cdk.Tags.of(app).add(tag_key, tag_value)
+
+if app_settings.module_settings.custom_tags:
+    for tag_key, tag_value in app_settings.module_settings.custom_tags.items():
+        aws_cdk.Tags.of(app).add(tag_key, tag_value)
+
 aws_cdk.Tags.of(app).add("SeedFarmerDeploymentName", app_settings.seedfarmer_settings.deployment_name)
 aws_cdk.Tags.of(app).add("SeedFarmerModuleName", app_settings.seedfarmer_settings.module_name)
 aws_cdk.Tags.of(app).add("SeedFarmerProjectName", app_settings.seedfarmer_settings.project_name)
diff --git a/modules/sagemaker/sagemaker-ground-truth-labeling/settings.py b/modules/sagemaker/sagemaker-ground-truth-labeling/settings.py
@@ -60,6 +60,8 @@ class ModuleSettings(CdkBaseSettings):
     verification_task_price: Dict[str, Dict[str, int]] = Field(default={})
     labeling_workflow_schedule: str = Field(default="cron(0 12 * * ? *)")
     permissions_boundary_name: Optional[str] = Field(default=None)
+    tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
 
 class SeedFarmerSettings(CdkBaseSettings):
diff --git a/modules/sagemaker/sagemaker-ground-truth-labeling/tests/test_app.py b/modules/sagemaker/sagemaker-ground-truth-labeling/tests/test_app.py
@@ -92,3 +92,8 @@ def test_labeling_task_keywords() -> None:
 
     with pytest.raises(ValidationError):
         import app  # noqa: F401
+
+
+def test_app_with_custom_tags() -> None:
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
diff --git a/modules/sagemaker/sagemaker-model-monitoring/app.py b/modules/sagemaker/sagemaker-model-monitoring/app.py
@@ -31,6 +31,10 @@
     for tag_key, tag_value in app_settings.module_settings.tags.items():
         aws_cdk.Tags.of(app).add(tag_key, tag_value)
 
+if app_settings.module_settings.custom_tags:
+    for tag_key, tag_value in app_settings.module_settings.custom_tags.items():
+        aws_cdk.Tags.of(app).add(tag_key, tag_value)
+
 aws_cdk.Tags.of(app).add("SeedFarmerDeploymentName", app_settings.seedfarmer_settings.deployment_name)
 aws_cdk.Tags.of(app).add("SeedFarmerModuleName", app_settings.seedfarmer_settings.module_name)
 aws_cdk.Tags.of(app).add("SeedFarmerProjectName", app_settings.seedfarmer_settings.project_name)
diff --git a/modules/sagemaker/sagemaker-model-monitoring/sagemaker_model_monitoring/settings.py b/modules/sagemaker/sagemaker-model-monitoring/sagemaker_model_monitoring/settings.py
@@ -102,6 +102,7 @@ class ModuleSettings(CdkBaseSettings):
 
     permissions_boundary_name: Optional[str] = Field(default=None)
     tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
 
 class SeedFarmerSettings(CdkBaseSettings):
diff --git a/modules/sagemaker/sagemaker-model-monitoring/sagemaker_model_monitoring/stack.py b/modules/sagemaker/sagemaker-model-monitoring/sagemaker_model_monitoring/stack.py
@@ -1,4 +1,4 @@
-from typing import Any, List, Optional
+from typing import Any, Dict, List, Optional
 
 import constructs
 from aws_cdk import Stack, Tags
@@ -91,6 +91,7 @@ def __init__(
         model_explainability_probability_attribute: Optional[str],
         model_explainability_schedule_expression: str,
         permissions_boundary_name: Optional[str] = None,
+        custom_tags: Optional[Dict[str, str]] = None,
         **kwargs: Any,
     ) -> None:
         super().__init__(scope, id, **kwargs)
diff --git a/modules/sagemaker/sagemaker-model-monitoring/tests/test_app.py b/modules/sagemaker/sagemaker-model-monitoring/tests/test_app.py
@@ -39,3 +39,8 @@ def test_all_disabled(stack_defaults):
 
     with pytest.raises(Exception, match="At least one of enable_data_quality_monitor, .+ must be True"):
         import app  # noqa: F401
+
+
+def test_app_with_custom_tags(stack_defaults):
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
diff --git a/modules/sagemaker/sagemaker-model-monitoring/tests/test_stack.py b/modules/sagemaker/sagemaker-model-monitoring/tests/test_stack.py
@@ -55,6 +55,7 @@ def stack_model_package_input(
         enable_model_explainability_monitor=enable_model_explainability_monitor,
         baseline_training_data_s3_uri=baseline_training_data_s3_uri,
         baseline_output_data_s3_uri=baseline_output_data_s3_uri,
+        custom_tags={"CustomerTag": "test-value"},
     )
 
     return stack.SageMakerModelMonitoringStack(
diff --git a/modules/sagemaker/sagemaker-notebook/sagemaker_notebook/settings.py b/modules/sagemaker/sagemaker-notebook/sagemaker_notebook/settings.py
@@ -42,6 +42,7 @@ class SeedFarmerParameters(CdkBaseSettings):
     role_arn: Optional[str] = Field(default=None)
     permissions_boundary_name: Optional[str] = Field(default=None)
     tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
     @field_validator("notebook_name")
     @classmethod
diff --git a/modules/sagemaker/sagemaker-notebook/sagemaker_notebook/stack.py b/modules/sagemaker/sagemaker-notebook/sagemaker_notebook/stack.py
@@ -31,6 +31,7 @@ def __init__(
         additional_code_repositories: Optional[List[str]] = None,
         role_arn: Optional[str] = None,
         tags: Optional[Dict[str, str]] = None,
+        custom_tags: Optional[Dict[str, str]] = None,
         permissions_boundary_name: Optional[str] = None,
         **kwargs: Any,
     ) -> None:
@@ -91,6 +92,7 @@ def __init__(
         self.kms_key_arn = kms_key_arn
         self.role_arn = role_arn
         self.additional_tags = tags
+        self.custom_tags = custom_tags
 
         self.additional_code_repositories = (
             ["https://github.com/aws/amazon-sagemaker-examples.git"]
@@ -149,6 +151,8 @@ def setup_tags(self) -> None:
         Tags.of(self).add("sagemaker:deployment-stage", Stack.of(self).stack_name)
         for k, v in (self.additional_tags or {}).items():
             Tags.of(self).add(k, v)
+        for k, v in (self.custom_tags or {}).items():
+            Tags.of(self).add(k, v)
 
     def setup_outputs(self) -> None:
         """Setups outputs and metadata."""
diff --git a/modules/sagemaker/sagemaker-notebook/tests/test_app.py b/modules/sagemaker/sagemaker-notebook/tests/test_app.py
@@ -23,3 +23,8 @@ def stack_defaults():
 
 def test_app(stack_defaults):
     import app  # noqa: F401
+
+
+def test_app_with_custom_tags(stack_defaults):
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
diff --git a/modules/sagemaker/sagemaker-studio/app.py b/modules/sagemaker/sagemaker-studio/app.py
@@ -73,6 +73,10 @@
     for tag_key, tag_value in app_settings.parameters.tags.items():
         aws_cdk.Tags.of(app).add(tag_key, tag_value)
 
+if app_settings.parameters.custom_tags:
+    for tag_key, tag_value in app_settings.parameters.custom_tags.items():
+        aws_cdk.Tags.of(app).add(tag_key, tag_value)
+
 aws_cdk.Tags.of(app).add("SeedFarmerDeploymentName", app_settings.settings.deployment_name)
 aws_cdk.Tags.of(app).add("SeedFarmerModuleName", app_settings.settings.module_name)
 aws_cdk.Tags.of(app).add("SeedFarmerProjectName", app_settings.settings.project_name)
diff --git a/modules/sagemaker/sagemaker-studio/settings.py b/modules/sagemaker/sagemaker-studio/settings.py
@@ -61,6 +61,7 @@ class SeedFarmerParameters(CdkBaseSettings):
     mlflow_artifact_store_bucket_prefix: str = Field(default="/")
 
     tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
 
 class SeedFarmerSettings(CdkBaseSettings):
diff --git a/modules/sagemaker/sagemaker-studio/tests/test_app.py b/modules/sagemaker/sagemaker-studio/tests/test_app.py
@@ -58,3 +58,8 @@ def test_default_synthesizer(stack_defaults):
     os.environ["SEEDFARMER_PARAMETER_LOOKUP_ROLE_ARN"] = "dummy"
 
     import app as my_app  # noqa: F401
+
+
+def test_app_with_custom_tags(stack_defaults):
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
diff --git a/modules/sagemaker/sagemaker-templates/app.py b/modules/sagemaker/sagemaker-templates/app.py
@@ -23,6 +23,14 @@
     batch_inference_project_settings=app_settings.batch_inference_project_settings,
 )
 
+if app_settings.module_settings.tags:
+    for tag_key, tag_value in app_settings.module_settings.tags.items():
+        aws_cdk.Tags.of(app).add(tag_key, tag_value)
+
+if app_settings.module_settings.custom_tags:
+    for tag_key, tag_value in app_settings.module_settings.custom_tags.items():
+        aws_cdk.Tags.of(app).add(tag_key, tag_value)
+
 aws_cdk.Tags.of(app).add("SeedFarmerDeploymentName", app_settings.seedfarmer_settings.deployment_name)
 aws_cdk.Tags.of(app).add("SeedFarmerModuleName", app_settings.seedfarmer_settings.module_name)
 aws_cdk.Tags.of(app).add("SeedFarmerProjectName", app_settings.seedfarmer_settings.project_name)
diff --git a/modules/sagemaker/sagemaker-templates/settings.py b/modules/sagemaker/sagemaker-templates/settings.py
@@ -78,6 +78,7 @@ class ModuleSettings(CdkBaseSettings):
     permissions_boundary_name: Optional[str] = Field(default=None)
     s3_access_logs_bucket_arn: Optional[str] = Field(default=None)
     tags: Optional[Dict[str, str]] = Field(default=None)
+    custom_tags: Optional[Dict[str, str]] = Field(default=None)
 
 
 class XGBoostAbaloneProjectSettings(CdkBaseSettings):
diff --git a/modules/sagemaker/sagemaker-templates/stack.py b/modules/sagemaker/sagemaker-templates/stack.py
@@ -2,7 +2,7 @@
 # SPDX-License-Identifier: Apache-2.0
 
 import os
-from typing import Any, List, Optional, Tuple, cast
+from typing import Any, Dict, List, Optional, Tuple, cast
 
 import cdk_nag
 from aws_cdk import BundlingOptions, BundlingOutput, DockerImage, Stack
@@ -52,6 +52,7 @@ def __init__(
         permissions_boundary_name: Optional[str] = None,
         s3_access_logs_bucket_arn: Optional[str] = None,
         cross_account_external_id: Optional[str] = None,
+        custom_tags: Optional[Dict[str, str]] = None,
         **kwargs: Any,
     ) -> None:
         super().__init__(scope, id, **kwargs)
diff --git a/modules/sagemaker/sagemaker-templates/tests/test_app.py b/modules/sagemaker/sagemaker-templates/tests/test_app.py
@@ -33,3 +33,8 @@ def stack_defaults():
 
 def test_app() -> None:
     import app  # noqa: F401
+
+
+def test_app_with_custom_tags(stack_defaults):
+    os.environ["SEEDFARMER_PARAMETER_CUSTOM_TAGS"] = '{"CustomerTag": "test-value"}'
+    import app  # noqa: F401
