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

Adding snowflake as a bundled database secrets plugin #10603

Merged
merged 6 commits into from
Jan 7, 2021

Conversation

Valarissa
Copy link
Contributor

This PR seeks to add the (Snowflake db secrets plugin)[https://github.com/hashicorp/vault-plugin-database-snowflake] as a bundled plugin.

It is targeted for release with Vault 1.7

vault/plugin_catalog_test.go Outdated Show resolved Hide resolved
vault/testing.go Outdated Show resolved Hide resolved
@Valarissa Valarissa requested a review from calvn January 5, 2021 00:36
Copy link
Contributor

@calvn calvn left a comment

Choose a reason for hiding this comment

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

One last tiny thing, otherwise LGTM!

@@ -3,6 +3,8 @@ package vault
import (
"context"
"fmt"
"github.com/hashicorp/vault/sdk/helper/consts"
Copy link
Contributor

Choose a reason for hiding this comment

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

The import here seems to be off. Can you revert this?

@Valarissa Valarissa merged commit 636d037 into master Jan 7, 2021
@kalafut kalafut deleted the bundle_snowflake_db_plugin branch January 8, 2021 04:48
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.

2 participants