Skip to content

Delete#11

Closed
sekka1 wants to merge 2 commits intomainfrom
delete
Closed

Delete#11
sekka1 wants to merge 2 commits intomainfrom
delete

Conversation

@sekka1
Copy link
Copy Markdown
Contributor

@sekka1 sekka1 commented Dec 30, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 30, 2025 01:33
@sekka1 sekka1 deployed to production December 30, 2025 01:34 — with GitHub Actions Active
@github-actions
Copy link
Copy Markdown

🍹 destroy on vpc/organization/vpc/staging

Pulumi report
   Destroying (staging):
@ destroying......
Downloading plugin vpc-0.0.2: done
Installing plugin vpc-0.0.2: done
@ destroying....
Downloading plugin aws-7.13.0: starting
Downloading plugin aws-7.9.1: starting
@ destroying....
Downloading plugin aws-7.13.0: done
Installing plugin aws-7.13.0: starting
Downloading plugin aws-7.9.1: done
Installing plugin aws-7.9.1: starting
@ destroying........
Installing plugin aws-7.13.0: done
Installing plugin aws-7.9.1: done
@ destroying.....................

@ destroying.....
-  aws:ec2:RouteTableAssociation staging-vpc-public-rta-2 deleting (0s) 
-  aws:ec2:RouteTableAssociation staging-vpc-public-rta-1 deleting (0s) 
-  aws:ec2:Route staging-vpc-public-route deleting (0s) 
-  aws:ec2:RouteTableAssociation staging-vpc-private-rta-1 deleting (0s) 
-  aws:ec2:RouteTableAssociation staging-vpc-private-rta-3 deleting (0s) 
-  aws:ec2:RouteTableAssociation staging-vpc-public-rta-3 deleting (0s) 
-  aws:ec2:RouteTableAssociation staging-vpc-private-rta-2 deleting (0s) 
@ destroying....
-  aws:ec2:RouteTableAssociation staging-vpc-public-rta-2 deleted (0.86s) 
-  aws:ec2:RouteTableAssociation staging-vpc-private-rta-1 deleted (1s) 
-  aws:ec2:RouteTableAssociation staging-vpc-public-rta-1 deleted (1s) 
@ destroying....
-  aws:ec2:Route staging-vpc-public-route deleted (1s) 
-  aws:ec2:RouteTableAssociation staging-vpc-private-rta-3 deleted (1s) 
-  aws:ec2:RouteTableAssociation staging-vpc-public-rta-3 deleted (1s) 
-  aws:ec2:RouteTableAssociation staging-vpc-private-rta-2 deleted (2s) 
@ destroying.....
-  aws:ec2:InternetGateway staging-vpc-igw deleting (0s) 
-  aws:ec2:RouteTable staging-vpc-private-rt deleting (0s) 
-  aws:ec2:Subnet staging-vpc-public-subnet-2 deleting (0s) 
-  aws:ec2:Subnet staging-vpc-private-subnet-2 deleting (0s) 
-  aws:ec2:Subnet staging-vpc-public-subnet-1 deleting (0s) 
-  aws:ec2:RouteTable staging-vpc-public-rt deleting (0s) 
-  aws:ec2:Subnet staging-vpc-private-subnet-3 deleting (0s) 
-  aws:ec2:SecurityGroup staging-vpc-default-sg deleting (0s) 
-  aws:ec2:Subnet staging-vpc-public-subnet-3 deleting (0s) 
-  aws:ec2:Subnet staging-vpc-private-subnet-1 deleting (0s) 
-  aws:ec2:InternetGateway staging-vpc-igw deleted (0.73s) 
@ destroying....
-  aws:ec2:RouteTable staging-vpc-private-rt deleted (1s) 
-  aws:ec2:Subnet staging-vpc-public-subnet-2 deleted (1s) 
-  aws:ec2:Subnet staging-vpc-private-subnet-2 deleted (1s) 
-  aws:ec2:Subnet staging-vpc-public-subnet-1 deleted (1s) 
@ destroying....
-  aws:ec2:RouteTable staging-vpc-public-rt deleted (2s) 
-  aws:ec2:Subnet staging-vpc-private-subnet-3 deleted (2s) 
-  aws:ec2:SecurityGroup staging-vpc-default-sg deleted (2s) 
-  aws:ec2:Subnet staging-vpc-public-subnet-3 deleted (2s) 
@ destroying....
-  aws:ec2:Subnet staging-vpc-private-subnet-1 deleted (3s) 
@ destroying.....
-  aws:ec2:Vpc staging-vpc-vpc deleting (0s) 
-  aws:ec2:Vpc staging-vpc-vpc deleted (0.78s) 
@ destroying....
-  vpc:index:Vpc staging-vpc deleting (0s) 
-  pulumi:providers:aws aws-provider deleting (0s) 
-  pulumi:providers:aws aws-provider deleted (0.41s) 
-  pulumi:pulumi:Stack vpc-staging deleting (0s) 
@ destroying.....
-  pulumi:pulumi:Stack vpc-staging deleted (0.45s) 
Outputs:
 - default_security_group_id: "sg-04368df1f8aa266a9"
 - internet_gateway_id      : "igw-002aac311c32e1032"
 - nat_gateway_ids          : []
 - private_subnet_ids       : [
 -     [0]: "subnet-055ab1e8dd85f0b99"
 -     [1]: "subnet-0877950117ae4c8ea"
 -     [2]: "subnet-0a6d0726e85dd821a"
   ]
 - public_subnet_ids        : [
 -     [0]: "subnet-08cff8aa8fd0b3217"
 -     [1]: "subnet-06b2c984e57e9d7f9"
 -     [2]: "subnet-0cf3e4082906a08bf"
   ]
 - vpc_arn                  : "arn:aws:ec2:us-east-1:126911164279:vpc/vpc-06dd3b31b26398d3b"
 - vpc_id                   : "vpc-06dd3b31b26398d3b"

Resources:
   - 21 deleted

Duration: 14s

The resources in the stack have been deleted, but the history and configuration associated with the stack are still maintained. 
If you want to remove the stack completely, run `pulumi stack rm staging`.
   

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request appears to be making changes that would destroy staging infrastructure. The PR includes a dangerous modification to the GitHub Actions workflow that changes the Pulumi command from preview to destroy, along with a minor formatting change to a Pulumi configuration file.

  • Changed Pulumi workflow command from preview to destroy in the staging deployment workflow
  • Added a blank line to the end of the VPC Pulumi configuration file

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/pulumi_staging.yaml Modified the Pulumi Preview step to run destroy instead of preview - this will destroy staging infrastructure
pulumi/environments/aws/staging/40-vpc/Pulumi.yaml Added trailing blank line (formatting only)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

always-include-summary: true
cloud-url: ${{ env.PULUMI_CLOUD_URL_STAGING }}
command: preview
command: destroy #preview
Copy link

Copilot AI Dec 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The command has been changed from "preview" to "destroy", which will destroy staging infrastructure instead of previewing changes. This is extremely dangerous as it runs on pull requests and when pushing to main. The step name is "Pulumi Preview" but the actual command is "destroy", creating a critical mismatch. If this change is merged, it will destroy the staging environment when the workflow runs.

Suggested change
command: destroy #preview
command: preview

Copilot uses AI. Check for mistakes.
@sekka1 sekka1 closed this Dec 30, 2025
@sekka1 sekka1 deleted the delete branch December 30, 2025 01:37
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.

2 participants