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.

Sign up for GitHub

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

Jump to bottom

[vcluster]: Add audit feature to apiserver #133

Merged
merged 5 commits into from
Oct 7, 2024
Merged

[vcluster]: Add audit feature to apiserver #133

merged 5 commits into from
Oct 7, 2024

Conversation

adberger
Copy link
Contributor

@adberger adberger commented Sep 30, 2024 •
edited
Loading

What this PR does:

Adds the ability to opt-in for audit logging of the apiServer.

Checklist:

  • Pull Request title in format [chart]: Changed Something
  • Updated documentation in the README.md.gotmpl file and executed helm-docs
  • Chart Version bumped
  • All commits are signed-off

@adberger adberger requested a review from a team as a code owner September 30, 2024 13:44
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 30, 2024
@adberger
[vcluster]: Add audit feature to apiserver
e43a02c 
Signed-off-by: Adrian Berger <adrian.berger@bedag.ch>
@github-actions github-actions bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 7, 2024
@adberger
fix: Do not run chart-testing (install) in GitHub workflows
d4c1ae1 
Signed-off-by: Adrian Berger <adrian.berger@bedag.ch>
@github-actions github-actions bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 7, 2024
@adberger
fix: Switch to https for dependency in vcluster again
068442e 
Signed-off-by: Adrian Berger <adrian.berger@bedag.ch>
@adberger
fix: casting of integers in vclusters
3ae8528 
Signed-off-by: Adrian Berger <adrian.berger@bedag.ch>
@adberger
fix: Remove subPath from configMap file mount in vcluster
e9bf266 
Signed-off-by: Adrian Berger <adrian.berger@bedag.ch>
larsgerber
larsgerber approved these changes Oct 7, 2024
View reviewed changes
Copy link

@larsgerber larsgerber left a comment

Choose a reason for hiding this comment

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

All issues have been resolved.

@adberger adberger merged commit 708ecbb into bedag:master Oct 7, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@larsgerber larsgerber larsgerber approved these changes

@malahayatii malahayatii Awaiting requested review from malahayatii malahayatii is a code owner automatically assigned from bedag/sre

Assignees
No one assigned
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adberger @larsgerber