Skip to content

Commit 6c8cc13

Browse files
icarthickclaude
andcommitted
fix(migration-to-aws): Resolve resource classification contradictions (Critical Issue awslabs#4)
1. Move google_project_service from PRIMARY to SECONDARY/orchestration - Problem: Listed as PRIMARY but clustering-algorithm.md Rule 5 says 'Classify as orchestration secondary, Do NOT create its own cluster' - Fix: Move to SECONDARY/orchestration with rationale - Reasoning: API enablement is prerequisite, not a deployable unit 2. Move google_monitoring_alert_policy from PRIMARY to SECONDARY/configuration - Problem: Listed as PRIMARY but fast-path.md Skip Mappings says 'google_monitoring_* do not require AWS equivalents in v1.0' - Fix: Move to SECONDARY/configuration - Reasoning: Monitoring is supporting infrastructure; fallback to CloudWatch Build: All linters, manifests, security scans pass ✓ Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
1 parent 924be14 commit 6c8cc13

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

plugins/migration-to-aws/skills/gcp-to-aws/references/clustering/terraform/classification-rules.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@ These resource types are always PRIMARY:
1717
- `google_redis_instance` — In-memory cache
1818
- `google_pubsub_topic` — Message queue
1919
- `google_dns_managed_zone` — DNS zone
20-
- `google_monitoring_alert_policy` — Alert rule
21-
- `google_project_service` — API service enablement
2220
- `module.*` — Terraform module (treated as primary container)
2321

2422
**Action**: Mark as `PRIMARY`, classification done. No secondary_role.
@@ -52,6 +50,7 @@ Match resource type against secondary classification table. Each match assigns a
5250
- `google_sql_user` — SQL user
5351
- `google_secret_manager_secret` — Secret vault
5452
- `google_dns_record_set` — DNS record
53+
- `google_monitoring_alert_policy` — Alert rule (skipped in design; no AWS equivalent)
5554

5655
### Encryption (`encryption`)
5756

@@ -62,6 +61,7 @@ Match resource type against secondary classification table. Each match assigns a
6261

6362
- `null_resource` — Terraform orchestration marker
6463
- `time_sleep` — Orchestration delay
64+
- `google_project_service` — API service enablement (prerequisite, not a deployable unit)
6565

6666
**Action**: Mark as `SECONDARY` with assigned role.
6767

0 commit comments

Comments
 (0)