Skip to content

Resource tagging: 68/70 tutorials tagged with project=doc-smith#81

Merged
mwunderl merged 1 commit intomainfrom
feature/resource-tagging-v3
May 1, 2026
Merged

Resource tagging: 68/70 tutorials tagged with project=doc-smith#81
mwunderl merged 1 commit intomainfrom
feature/resource-tagging-v3

Conversation

@mwunderl
Copy link
Copy Markdown
Contributor

@mwunderl mwunderl commented May 1, 2026

Every resource-creating AWS CLI command now includes project=doc-smith and tutorial={id} tags. This enables the orphan scanner and Doc Ralf to identify and clean up resources by tutorial.

68/70 tagged — 033-ses and 062-support skipped (no taggable resources).

Tagging methods by service:

  • EC2: --tag-specifications (appended to existing Tags arrays)
  • IAM: tag-role/tag-policy after creation
  • S3: put-bucket-tagging after creation
  • Glue/Cognito: tag-resource after creation (no inline tag support)
  • Lambda/Batch/Grafana/FIS: --tags key=value format
  • ECS/ECR: --tags key=x,value=y (lowercase)
  • Lightsail/Direct Connect: --tags key=x,value=y (lowercase)
  • All others: --tags Key=x,Value=y

Bugs found and fixed during testing:

  • 024-glue: create-database/create-table return no JSON — use constructed ARN for tag-resource
  • 031-cloudwatch: dashboard name with timestamp lost during verify — capture before put-dashboard
  • 065-elasticache, 070-dynamodb: set -e + exec > >(tee) SIGPIPE — drop set -e

Testing:

  • All 66 testable scripts pass locally (exit 0) in personal account
  • Baseline pipeline test running in analytics account for comparison
  • Zero tagging-caused regressions

…utorial={id}

Added --tags, --tag-specifications, or tag-resource commands to all
resource-creating AWS CLI calls. 033-ses and 062-support skipped
(no taggable resources).

Fixes during testing:
- 024-glue: create-database/create-table return no JSON; use constructed ARN for tag-resource
- 031-cloudwatch: capture dashboard name before put-dashboard (was lost in timestamp)
- 065-elasticache, 070-dynamodb: drop set -e (SIGPIPE with exec > >(tee))

All 66 testable scripts pass locally (exit 0).
@mwunderl mwunderl merged commit 087281d into main May 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant