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(vault): add file system (fs) vault #12584

Closed
wants to merge 1 commit into from
Closed

feat(vault): add file system (fs) vault #12584

wants to merge 1 commit into from

Conversation

bungle
Copy link
Member

@bungle bungle commented Feb 19, 2024

Summary

Adds a file system vault.

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

KAG-3736

### Summary

Adds a file system vault.

Signed-off-by: Aapo Talvensaari <[email protected]>
@bungle bungle marked this pull request as ready for review February 21, 2024 09:33
@mheap
Copy link
Member

mheap commented Feb 21, 2024

@bungle News from the red cord. Could we hold off on this for a little while until we clarify requirements? They've shifted since you built this

@bungle bungle added the pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release) label Feb 23, 2024
@team-eng-enablement team-eng-enablement added author/community PRs from the open-source community (not Kong Inc) and removed author/community PRs from the open-source community (not Kong Inc) labels Feb 24, 2024
@kikito kikito marked this pull request as draft March 12, 2024 18:28
@kikito
Copy link
Member

kikito commented Mar 12, 2024

Converting to draft, pending final requirements

@GGabriele
Copy link
Contributor

@kikito @bungle is this going to be included in 3.7?

@bungle
Copy link
Member Author

bungle commented Sep 2, 2024

@GGabriele we never got final requirements. Should we go ahead and merge this?

@GGabriele
Copy link
Contributor

That would be fine to me, but maybe @mheap knows more?

@mheap
Copy link
Member

mheap commented Sep 3, 2024

Let’s close until there’s a concrete need

@mheap mheap closed this Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release) schema-change-noteworthy size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants