Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Logs bucket must respect force destroy var #38

Closed
wants to merge 6 commits into from
Closed

Logs bucket must respect force destroy var #38

wants to merge 6 commits into from

Conversation

rafilkmp3
Copy link

@rafilkmp3 rafilkmp3 commented Jun 1, 2020 •
edited
Loading

what

  • Cannot run terraform destroy

why

  • We cannot destroy the env because force_destroy is not passed to sub module with creates s3 logs webserver

references

Error: error deleting S3 Bucket (example-app-logs): BucketNotEmpty: The bucket you tried to delete is not empty. You must delete all versions in the bucket.
	status code: 409, request id: 5CF1672F03ED424B, host id: iaD5GM5k02VH9RDxUphFw62XE/wILC6wZ/jgsd334fsddfo8LBwp/YMhczzrEjTRyg5BdTa9D4=

@osterman
Copy link
Member

/test all

@osterman
Copy link
Member

/rebuild-readme

@rafilkmp3 rafilkmp3 requested a review from a team as a code owner July 14, 2020 05:26
@rafilkmp3 rafilkmp3 requested review from jamengual and brcnblc and removed request for a team July 14, 2020 05:26
@jamengual
Copy link

/rebuild-readme

@jamengual
Copy link

/test all

@jamengual
Copy link

@ rafilkmp3 could you rebase?

@jamengual
Copy link

/rebuild-readme

@rafilkmp3 rafilkmp3 requested a review from a team as a code owner December 26, 2020 22:35
@jamengual
Copy link

/rebuild-readme

@jamengual
Copy link

/test all

@jamengual
Copy link

@rafilkmp3 could you follow this guide and then commit the changes > https://docs.cloudposse.com/community/updating-modules-for-terraform-14/

@maximmi maximmi mentioned this pull request Dec 30, 2020
Merged
@maximmi maximmi closed this in #46 Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamengual jamengual Awaiting requested review from jamengual

@brcnblc brcnblc Awaiting requested review from brcnblc

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rafilkmp3 @osterman @jamengual @actions-bot