Update comments and docs

skypilot-org · Sep 9, 2024 · 6ca0429 · 6ca0429
1 parent 71315a9
commit 6ca0429
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/docs/source/reference/config.rst b/docs/source/reference/config.rst
@@ -90,7 +90,7 @@ Available fields and semantics:
   # Advanced AWS configurations (optional).
   # Apply to all new instances but not existing ones.
   aws:
-    # Tags to assign to all instances launched by SkyPilot (optional).
+    # Tags to assign to all instances and buckets created by SkyPilot (optional).
     #
     # Example use case: cost tracking by user/team/project.
     #

diff --git a/sky/data/storage.py b/sky/data/storage.py
@@ -1448,7 +1448,8 @@ def _create_s3_bucket(self,
             logger.info(
                 f'Created S3 bucket {bucket_name!r} in {region or "us-east-1"}')
 
-            # add tags to the bucket
+            # Add AWS tags configured in config.yaml to the bucket.
+            # This is useful for cost tracking and external cleanup.
             bucket_tags = skypilot_config.get_nested(('aws', 'labels'), {})
             if bucket_tags:
                 s3_client.put_bucket_tagging(