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

Update ecs_service.html.markdown #4

Closed
wants to merge 1 commit into from

Conversation

network-charles
Copy link
Owner

Original Sentences

  • capacity_provider_strategy - (Optional) Capacity provider strategies to use for the service. Can be one or more. These can be updated without destroying and recreating the service only if force_new_deployment = true and not changing from 0 capacity_provider_strategy blocks to greater than 0, or vice versa. See below.

  • launch_type - (Optional) Launch type on which to run your service. The valid values are EC2, FARGATE, and EXTERNAL. Defaults to EC2.

Issues in the Original Sentences

Lack of Outlining Conflicts: These two argument references conflict with each other.

Final Sentences

  • capacity_provider_strategy - (Optional) Capacity provider strategies to use for the service. Can be one or more. These can be updated without destroying and recreating the service only if force_new_deployment = true and not changing from 0 capacity_provider_strategy blocks to greater than 0, or vice versa. See below. Conflicts with launch_type.
  • launch_type - (Optional) Launch type on which to run your service. The valid values are EC2, FARGATE, and EXTERNAL. Defaults to EC2. Conflicts with capacity_provider_strategy.
    Relations

Conclusion

Outlining these conflicts helps the users understand the resource better, it also prevents them from going back and forth after applying the resource.

Closes #0000

References

https://docs.aws.amazon.com/AmazonECS/latest/APIReference/API_CreateService.html

@network-charles network-charles deleted the network-charles-patch-1 branch January 21, 2024 07:54
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.

1 participant