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

Issue #214 - Added DeployTimeout to allow pipline timeout override #321

Merged
merged 2 commits into from Aug 28, 2018

Conversation

mneil
Copy link
Contributor

@mneil mneil commented Aug 28, 2018

Addresses issue #214 to add timeout for deploy pipelines. I only parameterized the build steps. The 30 hard minute timeout is still in place for the artifact storage steps.

Allows users to override the default 30 minutes for
deploy timeouts in acceptance, production, and build
pipelines

- Moved the params assignment into an exported method for easier
testing of param assignment
- Added test to ensure timeout was read and set for each pipeline

Issue #214 - Added parameters for deploy timeout

Allows users to override the default 30 minutes for
deploy timeouts in acceptance, production, and build
pipelines

Issue #214 - Pull params building out of upsert for testing

Moved the params assignment into an exported method for easier
testing of param assignment.

Issue #214 - Added test for build timeout in mu yaml

Issue #214 - Renamed DeployTimeout to BuildTimeout to reflect use

Accidentally named the Struct properties BuildTimeout when they
only affect DeployTimeout settings. Renamed to avoid confusion
and allow for having a separate deploy timeout in the future
@cplee cplee merged commit 0cca5d3 into develop Aug 28, 2018
@cplee cplee removed the in progress label Aug 28, 2018
@cplee cplee deleted the issue-214 branch September 19, 2018 20:43
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.

None yet

2 participants