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

Retire Airflow Dev EKS Cluster #6539

Merged
merged 10 commits into from
Jan 21, 2025
Merged

Retire Airflow Dev EKS Cluster #6539

merged 10 commits into from
Jan 21, 2025

Conversation

BrianEllwood
Copy link
Contributor

@BrianEllwood BrianEllwood commented Jan 10, 2025

Pull Request Objective

This piece of work is being tracked in this GitHub Issue.

The above ticket requires that the Dev Airflow Cluster to be retired. This PR is to remove all objects I can without the Terraform plan giving errors.

The changes were applied locally and some objects had to be deleted via the console and the code amended appropriately.

The Terraform plan created in this pr reflects that the changes were applied locally.

Copy link
Contributor

github-actions bot commented Jan 10, 2025

Terraform Component 🧱: aws-analytical-platform-data-production-airflow

Checkov 🛂: failure

Trivy 🛂: failure

Static Analysis Override Label 🏷️: true

Pusher: @BrianEllwood, Action: pull_request, Working Directory: terraform/aws/analytical-platform-data-production/airflow, Workflow: Terraform, Marker: aws-analytical-platform-data-production-airflow_static_analysis

Copy link
Contributor

github-actions bot commented Jan 10, 2025

Terraform Component 🧱: aws-analytical-platform-data-production-airflow

Terraform Initialization ⚙️: success

Terraform Validation 🤖: success

Terraform Plan 🛠️: success

Pusher: @BrianEllwood, Action: pull_request, Working Directory: terraform/aws/analytical-platform-data-production/airflow, Workflow: Terraform, Marker: aws-analytical-platform-data-production-airflow_plan

@BrianEllwood BrianEllwood marked this pull request as ready for review January 21, 2025 13:26
@BrianEllwood BrianEllwood requested a review from a team as a code owner January 21, 2025 13:26
@BrianEllwood
Copy link
Contributor Author

BrianEllwood commented Jan 21, 2025

Plan as expected as all changes applied locally - just the two s3 buckets to add which is a know issue.

Added override static analysis as alerts not related to this work

@BrianEllwood BrianEllwood merged commit 334d6f4 into main Jan 21, 2025
15 checks passed
@BrianEllwood BrianEllwood deleted the retire-airflow-dev-eks branch January 21, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants