Skip to content

feat: Allow the configuration of the security plugin#117

Merged
siegfriedweber merged 65 commits intomainfrom
feat/security-config
Mar 5, 2026
Merged

feat: Allow the configuration of the security plugin#117
siegfriedweber merged 65 commits intomainfrom
feat/security-config

Conversation

@siegfriedweber
Copy link
Member

@siegfriedweber siegfriedweber commented Feb 19, 2026 •
edited
Loading

Description

Allow the configuration of the security plugin

Part of #43

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes

Author

Reviewer

  • Code contains useful comments
  • Code contains useful logging statements
  • (Integration-)Test cases added
  • Documentation added or updated. Follows the style guide.
  • Changelog updated
  • Cargo.toml only contains references to git tags (not specific commits or branches)

Acceptance

  • Feature Tracker has been updated
  • Proper release label has been added
  • Links to generated (nightly) docs added
  • Release note snippet added
  • Add type/deprecation label & add to the deprecation schedule
  • Add type/experimental label & add to the experimental features tracker

@siegfriedweber
test: Add security-config integration test
e9e527f
@siegfriedweber
feat: Add securityConfig to the CRD; Deploy the initial security conf…
419447c 
…iguration
@siegfriedweber
Deploy security config files only if completely managed by the API
d12a691
@siegfriedweber
test(smoke): Use securityConfig
a0382d0
@siegfriedweber
Deploy security config files only to the managing role group
0b8ab8a
@siegfriedweber
Create admin certificate in init container
dce4c40
@siegfriedweber
Add update-security-config container
f6387a9
@siegfriedweber
Configure DN of the admin certificate
fefcfdb
@siegfriedweber
Allow only one pod to manage the security configuration
5aa8f8b
@siegfriedweber
Validate the security configuration; Fix all unit tests
d9e4787
@siegfriedweber
Allow to disable the security plugin
5d5cf8a
@siegfriedweber
Use a structure for the validated security configuration
96745ee
@siegfriedweber
Declare security init containers
b2c611d
@siegfriedweber
test(backup-restore): Use securityConfig
88cb259
@siegfriedweber
test(external-access): Use securityConfig
c3d403e
@siegfriedweber
test(ldap): Use securityConfig
0d31398
@siegfriedweber
test(logging): Use securityConfig
a8c71c2
@siegfriedweber
test(metrics): Use securityConfig
dbd5eae
@siegfriedweber
test(opensearch-dashboards): Use securityConfig
ad31980
@siegfriedweber
Rename clusterConfig.security.config to clusterConfig.security.settin…
1b4e332 
…gs; Fix admin DN; Fix integration tests
@siegfriedweber
Merge branch 'main' into feat/security-config
6530f2d
@siegfriedweber
Update the CRD documentation
fd55eda
@siegfriedweber
Rename admin_dn() to super_admin_dn()
f78b37f
@siegfriedweber
Do not use overrides to determine if TLS is enabled
732b082
@siegfriedweber siegfriedweber self-assigned this Feb 19, 2026
@siegfriedweber siegfriedweber mentioned this pull request Feb 19, 2026
Open
3 tasks
@siegfriedweber siegfriedweber requested a review from a team March 3, 2026 13:55
@siegfriedweber siegfriedweber moved this to Development: Waiting for Review in Stackable Engineering Mar 3, 2026
@siegfriedweber siegfriedweber marked this pull request as ready for review March 3, 2026 13:55
@maltesander maltesander self-requested a review March 3, 2026 14:47
@maltesander maltesander moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Mar 3, 2026
@siegfriedweber siegfriedweber mentioned this pull request Mar 3, 2026
Open
maltesander
maltesander requested changes Mar 5, 2026
View reviewed changes
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

Just minor things, LGTM otherwrise!

maltesander
maltesander approved these changes Mar 5, 2026
View reviewed changes
Copy link
Member

@maltesander maltesander left a comment

Choose a reason for hiding this comment

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

LGTM!

@siegfriedweber siegfriedweber added this pull request to the merge queue Mar 5, 2026
Merged via the queue into main with commit 6938474 Mar 5, 2026
12 checks passed
@siegfriedweber siegfriedweber deleted the feat/security-config branch March 5, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maltesander maltesander maltesander approved these changes

Assignees

@siegfriedweber siegfriedweber

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@siegfriedweber @maltesander