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

chore: ignore INACTIVE cluster in DefaultCluster #2045

Merged
merged 4 commits into from
Mar 12, 2021

Conversation

ota42y
Copy link
Contributor

@ota42y ota42y commented Mar 11, 2021

When we delete ECS cluster, AWS return it as INACTIVE cluster.

INACTIVE
The cluster has been deleted. Clusters with an INACTIVE status may remain discoverable in your account for a period of time. However, this behavior is subject to change in the future, so you should not rely on INACTIVE clusters persisting.

https://docs.aws.amazon.com/AmazonECS/latest/developerguide/clusters.html

We can't run task on INACTIVE cluster so we should ignore it.

When we delete ECS cluster, AWS return it as INACTIVE cluster.

```
INACTIVE
The cluster has been deleted. Clusters with an INACTIVE status may remain discoverable in your account for a period of time. However, this behavior is subject to change in the future, so you should not rely on INACTIVE clusters persisting.
```
https://docs.aws.amazon.com/AmazonECS/latest/developerguide/clusters.html

We can't run task on INACTIVE cluster so we should ignore it.
@ota42y ota42y requested a review from a team as a code owner March 11, 2021 05:17
@ota42y ota42y requested a review from efekarakus March 11, 2021 05:17
Copy link
Contributor

@Lou1415926 Lou1415926 left a comment

Choose a reason for hiding this comment

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

Thank you so much for contributing! A few small comments below ⬇️

internal/pkg/aws/ecs/ecs.go Outdated Show resolved Hide resolved
internal/pkg/aws/ecs/ecs.go Outdated Show resolved Hide resolved
internal/pkg/aws/ecs/ecs.go Outdated Show resolved Hide resolved
@ota42y ota42y changed the title ignore INACTIVE cluster in DefaultCluster fix: ignore INACTIVE cluster in DefaultCluster Mar 12, 2021
@ota42y
Copy link
Contributor Author

ota42y commented Mar 12, 2021

I have completed the fix!

Copy link
Contributor

@efekarakus efekarakus left a comment

Choose a reason for hiding this comment

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

Awesome, thank you!

@efekarakus efekarakus changed the title fix: ignore INACTIVE cluster in DefaultCluster chore: ignore INACTIVE cluster in DefaultCluster Mar 12, 2021
Copy link
Contributor

@Lou1415926 Lou1415926 left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thank you so much for your contribution!

@mergify mergify bot merged commit 11b4274 into aws:mainline Mar 12, 2021
Sprint 🏃‍♀️ automation moved this from In review to Pending release Mar 12, 2021
@ota42y ota42y deleted the feature/ignore_inactive_cluster branch March 13, 2021 00:47
thrau pushed a commit to localstack/copilot-cli-local that referenced this pull request Dec 9, 2022
When we delete ECS cluster, AWS return it as INACTIVE cluster.

```
INACTIVE
The cluster has been deleted. Clusters with an INACTIVE status may remain discoverable in your account for a period of time. However, this behavior is subject to change in the future, so you should not rely on INACTIVE clusters persisting.
```
https://docs.aws.amazon.com/AmazonECS/latest/developerguide/clusters.html

We can't run task on INACTIVE cluster so we should ignore it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Sprint 🏃‍♀️
  
Pending release
Development

Successfully merging this pull request may close these issues.

None yet

3 participants