Skip to content

Fix resources created in apply in aws-py-hub-and-spoke-network#2678

Merged
jkodroff merged 1 commit intomasterfrom
jkodroff/fix-resources-in-apply
Apr 16, 2026
Merged

Fix resources created in apply in aws-py-hub-and-spoke-network#2678
jkodroff merged 1 commit intomasterfrom
jkodroff/fix-resources-in-apply

Conversation

@jkodroff
Copy link
Copy Markdown
Member

@jkodroff jkodroff commented Apr 8, 2026

Summary

  • Refactor spoke and inspection VPCs to stop creating resources inside apply() callbacks, which caused those resources to not appear in pulumi preview
  • Replace get_subnets_output data source lookups with direct VPC component outputs (isolated_subnet_ids, public_subnet_ids) so resources appear in preview even on fresh stacks
  • Use _output variants of data source functions (get_route_table_output, get_subnet_output) to work with Output[str] values
  • Update requirements.txt to latest major versions, fix AWSX subnet strategy warnings, remove unused imports

Test plan

  • pulumi preview succeeds with 138 resources, no errors or warnings
  • All spoke routes, VPC endpoints, and inspection routes appear in preview (previously hidden inside apply)
  • Python formatting passes (make check_python_formatting)

🤖 Generated with Claude Code

Refactor spoke and inspection VPCs to avoid creating resources inside
apply() callbacks, which causes preview to not show those resources.
Use VPC component outputs directly instead of get_subnets_output data
source lookups, and use _output variants of data source functions.

Also updates dependencies to latest major versions, fixes subnet
strategy warnings, removes unused imports, and fixes a README typo.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jkodroff jkodroff requested a review from CamSoper April 8, 2026 22:27
@jkodroff jkodroff requested a review from Frassle April 16, 2026 15:20
@jkodroff jkodroff merged commit c8e324c into master Apr 16, 2026
27 checks passed
@jkodroff jkodroff deleted the jkodroff/fix-resources-in-apply branch April 16, 2026 21:43
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.

3 participants