Skip to content

Add waiter config override option for cloudformation deploy#6748

Open
vvucetic wants to merge 1 commit intoaws:developfrom
vvucetic:add-env-vars-cf-waiter
Open

Add waiter config override option for cloudformation deploy#6748
vvucetic wants to merge 1 commit intoaws:developfrom
vvucetic:add-env-vars-cf-waiter

Conversation

@vvucetic
Copy link
Copy Markdown

Description of changes:

This PR adds an option to override waiter config with env vars for cloudformation deploy. Sometimes, some stacks take over 1 hour to provision which makes deploy command exit with failure after 1h although deploy itself remains running.

Mentioned here: #2831

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@stealthycoin
Copy link
Copy Markdown
Contributor

Hi @vvucetic thanks for the PR. Our team just put out a recent proposal in #6828 detailing improvements to the contribution process. We are working through open PRs, this PR lacked an open issue, so I re-opened the original issue. I am putting this in draft until the issue meets the requirements for review outlined in the proposal.

@stealthycoin stealthycoin marked this pull request as draft April 22, 2022 23:04
@stealthycoin stealthycoin marked this pull request as draft April 22, 2022 23:04
@klockk
Copy link
Copy Markdown

klockk commented Aug 31, 2022

What's the status of this request? I'm trying to deploy an RDS Custom instance using a custom lambda resource which is taking over an hour and causing my nested stacks deployment (update on this case) to time out at 1 hr via the cli deploy option.. Need desperately to get around this issue....

@vvucetic vvucetic marked this pull request as ready for review June 14, 2023 12:11
@AlexandraDorey
Copy link
Copy Markdown

Has this merged or any plans to? being unable to configure this is annoying.

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.

7 participants