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

Raft config refactor for mount entry size limit #25992

Merged
merged 5 commits into from
Mar 19, 2024
Merged

Raft config refactor for mount entry size limit #25992

merged 5 commits into from
Mar 19, 2024

Conversation

banks
Copy link
Member

@banks banks commented Mar 18, 2024

This is a small config parsing refactor for the raft backend as we continue to add new configurations.

The only noticeable changes for Community Edition:

  • A new config key can now be parsed max_mount_and_namespace_table_entry_size although a warning will be emitted since it's configuration that only takes effect in Vault Enterprise.
  • Several existing configurations that only apply to Enterprise features such as AP upgrades, Redundancy Zones and Non-voting servers will now also provide a more helpful warning in CE rather than being silent no-ops.

@banks banks added this to the 1.17.0-rc milestone Mar 18, 2024
@banks banks requested a review from miagilepner March 18, 2024 16:30
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Mar 18, 2024
@banks banks changed the title F/mount limit Raft config refactor for mount entry size limit Mar 18, 2024
Copy link

github-actions bot commented Mar 18, 2024

CI Results:
All Go tests succeeded! ✅

Copy link

github-actions bot commented Mar 18, 2024

Build Results:
All builds succeeded! ✅

@banks banks merged commit 3a2a922 into main Mar 19, 2024
82 of 83 checks passed
@banks banks deleted the f/mount-limit branch March 19, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants