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.

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

Tune aws-nuke: skip S3Objects, include RedShift #117

Merged
merged 6 commits into from
Jun 26, 2021
Merged

Tune aws-nuke: skip S3Objects, include RedShift #117

merged 6 commits into from
Jun 26, 2021

Conversation

Nuru
Copy link
Contributor

@Nuru Nuru commented Jun 26, 2021

what

  • Tune aws-nuke: skip S3Objects, delete RedShift resources, ECS Task Definitions, and OIDC connectors
  • Update Geodesic to current and install Terraform 0.15 and 1.x
  • Disable nightly Docker builds

why

  • Action was timing out after 6 hours, probably because of the overhead of deleting S3 objects one at a time. Not necessary to delete S3 Objects because we will delete entire S3 Buckets which will cause S3 Objects to be deleted as well.
  • We now have a tests that provision RedShift clusters and OIDC connectors, but we were not deleting them.
  • Support current Terraform version as well as legacy version.
  • Image is stable, does not change with nightly builds, and testing now uses curl to pull latest configuration anyway.

@Nuru Nuru requested review from osterman and aknysh June 26, 2021 06:04
@Nuru Nuru requested a review from a team as a code owner June 26, 2021 06:04
@@ -5,8 +5,8 @@ on:
release:
types:
- created
schedule:
- cron: '0 0 * * *'
# schedule:
Copy link
Member

Choose a reason for hiding this comment

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

Why disable?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know why we want to build this nightly. It is pinned to a Geodesic version and we do not update Geodesic that often. It just seems a waste of resources to build it nightly.

@osterman
Copy link
Member

Aws does not allow deletion of bucket with objects.

I don't believe the hypothesis on too many objects

@osterman
Copy link
Member

I rather exit 0 than run manually. Our testing infra orphans way too much to make this manuAl, and even if it fails deleting some stuff is better than not at all.

@Nuru Nuru merged commit f219c9e into master Jun 26, 2021
@Nuru Nuru deleted the nuke branch June 26, 2021 18:41
@Nuru
Copy link
Contributor Author

Nuru commented Jun 26, 2021

Aws does not allow deletion of bucket with objects.

I don't believe the hypothesis on too many objects

The only evidence I have for the hypothesis is

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