8000 Adding HealthCheckGracePeriodSeconds into ECS Service by andydoTech · Pull Request #966 · cloudtools/troposphere · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Adding HealthCheckGracePeriodSeconds into ECS Service #966

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

Merged
merged 3 commits into from
Jan 24, 2018
Merged

Adding HealthCheckGracePeriodSeconds into ECS Service #966

merged 3 commits into from
Jan 24, 2018

Conversation

andydoTech
Copy link
Contributor

@phobologic
Copy link
Member

I don't see this mentioned in the Cloudformation docs yet:

https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-ecs-service.html

Are you sure this is accepted by Cloudformation?

@phobologic
Copy link
Member

It's also not currently listed in the resource schema specification:

https://d1uauaxba7bl26.cloudfront.net/latest/gzip/CloudFormationResourceSpecification.json

@andydoTech
Copy link
Contributor Author

Ah, you are right. Let me close and submit it when they are ready.

@andydoTech andydoTech closed this Jan 11, 2018
@phobologic
Copy link
Member

Thanks @andydoTech - I can't wait for this feature as well. It's going to fix so many issues for us :)

@phobologic
Copy link
Member

Looks like this was added, and we have another PR for it: #971 :)

@phobologic
Copy link
Member

Re-opening since this should be good now, thanks @andydoTech !

@phobologic
Copy link
Member

Ok, made a minor ordering change, but I'm going to accept this now. Thanks @andydoTech !

@phobologic phobologic merged commit cbaad3b into cloudtools:master Jan 24, 2018
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
0