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

feat(deadline): add Secret Management support for Repository #514

Merged

Conversation

kozlove-aws
Copy link
Contributor

@kozlove-aws kozlove-aws commented Jul 29, 2021

Problem

We need to have possibility to enable Secret Management in repository construct, also Secret Management should be enabled by default.

Testing

  • Added unit tests
  • Deployed example and validated that Secret Management is enabled.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@kozlove-aws kozlove-aws changed the title [WIP] feat(repository): add Secret Management support feat: add Secret Management support Jul 29, 2021
@kozlove-aws kozlove-aws changed the title feat: add Secret Management support WIP feat: add Secret Management support Jul 29, 2021
@kozlove-aws kozlove-aws changed the title WIP feat: add Secret Management support [WIP] feat: add Secret Management support Jul 29, 2021
@kozlove-aws kozlove-aws force-pushed the use_sm_for_repository branch 2 times, most recently from 8241a11 to a97c6e4 Compare July 30, 2021 13:29
@kozlove-aws kozlove-aws changed the title [WIP] feat: add Secret Management support feat: add Secret Management support Jul 30, 2021
@kozlove-aws kozlove-aws changed the title feat: add Secret Management support [WIP] feat: add Secret Management support Jul 30, 2021
@kozlove-aws kozlove-aws changed the title [WIP] feat: add Secret Management support feat: add Secret Management support Jul 30, 2021
@kozlove-aws kozlove-aws changed the title feat: add Secret Management support feat: add Secret Management support for Repository Jul 30, 2021
@jusiskin jusiskin added the contribution/core This is a PR that came from AWS. label Aug 2, 2021
@horsmand horsmand self-requested a review August 3, 2021 22:37
@jusiskin jusiskin changed the title feat: add Secret Management support for Repository feat(deadline): add Secret Management support for Repository Aug 5, 2021
@kozlove-aws kozlove-aws force-pushed the use_sm_for_repository branch 7 times, most recently from 2fbc8cf to edd489d Compare August 11, 2021 23:45
@kozlove-aws kozlove-aws changed the base branch from mainline to feature_enable_secret_manager August 13, 2021 15:46
Copy link
Contributor

@jericht jericht 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, thanks!

Copy link
Contributor

@horsmand horsmand left a comment

Choose a reason for hiding this comment

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

Really small stuff that should be quick to fix and not require re-testing.

packages/aws-rfdk/lib/deadline/lib/repository.ts Outdated Show resolved Hide resolved
packages/aws-rfdk/lib/deadline/lib/repository.ts Outdated Show resolved Hide resolved
packages/aws-rfdk/lib/deadline/lib/repository.ts Outdated Show resolved Hide resolved
packages/aws-rfdk/lib/deadline/test/repository.test.ts Outdated Show resolved Hide resolved
@horsmand horsmand merged commit 8c7dda6 into aws:feature_enable_secret_manager Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants