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

Part 4/n - Remove secrets plugins #102

Merged
merged 2 commits into from
Feb 2, 2024

Conversation

cipherboy
Copy link
Member

In alignment with the plugin policy decisions of #64, this removes all non-OSI secrets plugins, preferring to make them external plugins.

To ease maintenance burden, only integrations licensed under OSI licenses will be included in the main repository; other plugins will be community supported and not coupled with the binary or the release process.


This is part of #68, broken up to make review easier.

@naphelps naphelps self-requested a review February 2, 2024 19:30
In alignment with the plugin policy decisions, this removes all non-OSI
secrets plugins, preferring to make them external plugins.

To ease maintenance burden, only integrations licensed under OSI licenses
will be included in the main repository; other plugins will be community
supported and not coupled with the binary or the release process.

Cherry-pick of 946d4e0
             + 2d55f0a.

Signed-off-by: Alexander Scheel <alexander.m.scheel@gmail.com>
This removes various references to non-OSI integrations within the test
suite, usually used to mock backends for testing. Since these are
removed (between the auth and secrets PRs), they're no longer necessary
once both have merged.

Cherry-pick of 8d6796e.

Signed-off-by: Alexander Scheel <alexander.m.scheel@gmail.com>
@cipherboy
Copy link
Member Author

@naphelps Updated :-)

@naphelps naphelps merged commit 40aecd9 into openbao:main Feb 2, 2024
3 of 9 checks passed
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