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

[fluentd-elasticsearch] Support for securityContext #106

Merged
merged 5 commits into from
Apr 20, 2022

Conversation

nzolot
Copy link
Contributor

@nzolot nzolot commented Apr 14, 2022

Which chart

fluentd-elasticsearch

What this PR does / why we need it

Feature request: #93

Which issue this PR fixes

  • none

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • All variables are documented in the charts README

@nzolot nzolot requested a review from monotek as a code owner April 14, 2022 14:00
Copy link
Member

@monotek monotek left a comment

Choose a reason for hiding this comment

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

Please raise minor chart version and sign dco.

@nzolot
Copy link
Contributor Author

nzolot commented Apr 15, 2022

hi @monotek , thanks for checking the PR. I have signed-off my commits and bumped minor version

@@ -1,6 +1,6 @@
apiVersion: v2
name: fluentd-elasticsearch
version: 13.3.0
version: 13.3.1
Copy link
Member

@monotek monotek Apr 15, 2022

Choose a reason for hiding this comment

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

new feature should raise minor version -> 13.5.0.

@monotek monotek merged commit 33e79ee into kokuwaio:main Apr 20, 2022
cpjolly pushed a commit to cpjolly/kokuwaio-helm-charts that referenced this pull request Jul 4, 2022
* Added ability to specify securityContext

Signed-off-by: Nick Zolotarov <[email protected]>

* Update README.md

Signed-off-by: Nick Zolotarov <[email protected]>

* [fluentd-elasticsearch] Added ability to specify securityContext (kokuwaio#93) - version bump

Signed-off-by: Nick Zolotarov <[email protected]>

Co-authored-by: André Bauer <[email protected]>
Signed-off-by: Chris <[email protected]>
monotek added a commit that referenced this pull request Jul 4, 2022
* [fluentd-elasticsearch] Support for securityContext (#106)

* Added ability to specify securityContext

Signed-off-by: Nick Zolotarov <[email protected]>

* Update README.md

Signed-off-by: Nick Zolotarov <[email protected]>

* [fluentd-elasticsearch] Added ability to specify securityContext (#93) - version bump

Signed-off-by: Nick Zolotarov <[email protected]>

Co-authored-by: André Bauer <[email protected]>
Signed-off-by: Chris <[email protected]>

* [mysqldump] Updated Bitnami MariaDB chart version

* Signed commit

Signed-off-by: Chris <[email protected]>

Co-authored-by: Nick Zolotarov <[email protected]>
Co-authored-by: André Bauer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants