Skip to content

Conversation

@yvthepief
Copy link
Contributor

Allow update timeout when being used in the deploy stage within codepipeline, Allow read of environment_variables when using codebuild in the deployment stage in codepipeline

Issue #, if available:
#306

Description of changes:
Added the possibility to set timeout and environment_variables when using CodeBuild in the deployment stage of CodePipeline.

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

…ipeline, Allow read of environment_variables when using codebuild in the deployment stage in codepipeline
Copy link
Collaborator

@sbkok sbkok left a comment

Choose a reason for hiding this comment

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

Thanks for contributing these fixes!
Looks good to me.

@sbkok sbkok added this to the v3.1.1 milestone Feb 16, 2021
@sbkok
Copy link
Collaborator

sbkok commented Feb 16, 2021

Hi @yvthepief, could you rebase your changes on top of the latest master commit?
I would love to get your fix deployed in the next release of ADF, v3.1.1.
Best regards, Simon

@yvthepief
Copy link
Contributor Author

Hi @yvthepief, could you rebase your changes on top of the latest master commit?
I would love to get your fix deployed in the next release of ADF, v3.1.1.
Best regards, Simon

Done!

Copy link
Collaborator

@thomasmcgannon thomasmcgannon 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!

@sbkok sbkok merged commit 27a49d8 into awslabs:master Feb 16, 2021
sbkok added a commit to sbkok/aws-deployment-framework that referenced this pull request Mar 3, 2021
**Fixes 🐞**

* Fixes `timeout` and `environment_variables` to be used when defined in
  the default CodeBuild Deployment provider properties awslabs#307, closes awslabs#306.
* Fixes intrinsic functions for account_region param files awslabs#333, closes awslabs#147.
* Fixes use of deployment from source directly when build stage is
  disabled awslabs#334, closes awslabs#236 and closes awslabs#318.
@sbkok sbkok mentioned this pull request Mar 3, 2021
sbkok added a commit that referenced this pull request Mar 3, 2021
**Fixes 🐞**

* Fixes `timeout` and `environment_variables` to be used when defined in
  the default CodeBuild Deployment provider properties #307, closes #306.
* Fixes intrinsic functions for account_region param files #333, closes #147.
* Fixes use of deployment from source directly when build stage is
  disabled #334, closes #236 and closes #318.
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