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

Contrib: Support Vault backend KVv2. Connected to #1331. #1345

Merged
merged 1 commit into from
Sep 28, 2020

Conversation

sgnn7
Copy link
Contributor

@sgnn7 sgnn7 commented Sep 25, 2020

Branch transfer from #1334

Signed-off-by: Meijer [email protected]

What does this PR do?

  • Support secrets of Vault's KV v2 backend which is incompatible with the current implementation in secretless-broker.
  • New behavior is implemented with backwards compatability to current (old) behavior.

What ticket does this PR close?

Connected to #1331

Checklists

Change log

  • The CHANGELOG has been updated, or
  • This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

  • This PR includes new unit and integration tests to go with the code changes, or
  • The changes in this PR do not require tests

Documentation

  • This PR does not require updating any documentation, or
  • Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs

@sgnn7 sgnn7 force-pushed the michael2m-1331-support-kv-v2 branch from 8d56e63 to 1bc6d40 Compare September 25, 2020 20:49
@sgnn7 sgnn7 force-pushed the michael2m-1331-support-kv-v2 branch from 1bc6d40 to 9c9197f Compare September 25, 2020 20:50
@@ -6,6 +6,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

## [Unreleased]

### Added
- The `vault` provider now supports loading secrets from the KV Version 2 secret
Copy link

Choose a reason for hiding this comment

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

Lists should be surrounded by blank lines

@codeclimate
Copy link

codeclimate bot commented Sep 25, 2020

Code Climate has analyzed commit 9c9197f and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Style 1

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 50.2% (0.0% change).

View more on Code Climate.

@sgnn7 sgnn7 marked this pull request as ready for review September 25, 2020 21:21
@sgnn7 sgnn7 requested a review from a team as a code owner September 25, 2020 21:21
Copy link
Contributor

@izgeri izgeri left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for contributing this @michael2m!!

@sgnn7 sgnn7 merged commit a2c522e into master Sep 28, 2020
@sgnn7 sgnn7 deleted the michael2m-1331-support-kv-v2 branch September 28, 2020 19:25
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