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

[newrelic-logging] Allow customization of init container when running in persistentVolume mode #1445

Open
nluedema opened this issue Jul 26, 2024 · 1 comment
Labels
feature request Categorizes issue or PR as related to a new feature or enhancement.

Comments

@nluedema
Copy link
Contributor

Description

When running newrelic-logging with a persistent volume for storing the fluent-bit database file, an init container is deployed to delete old database files.
This init container can currently not be modified using the values.yaml file.
Being able to modify the definition of this init container would help to stay compliant with automatic security checks that might be performed on a Kubernetes cluster.
In our case, it would be very helpful if we could define the security context and the resources of the init container.

Any other required init container can be fully defined by the user, only the init container created when running in persistentVolume mode cannot be modified. A good solution in my opinion would be to allow the users to add any extra options under fluentBit.persistence.persistentVolume.extra.initContainer. This would allow to define extra options for the init container in the same way as for the persistent volume or the claim, all of which are specific to running in persistentVolume mode.

If you like this proposal I'm happy to contribute with a PR.

Acceptance Criteria

A user is able to add extra options to the definition of the init container that gets created when running in persistentVolume mode.

Describe Alternatives

An alternative could be to make the init container creation optional, so that users can define their own init container for deleting old database files.

Dependencies

None

Additional context

What else should we know about this story that might not fit into the other categories?

Estimates

S

For Maintainers Only or Hero Triaging this bug

Suggested Priority (P1,P2,P3,P4,P5):
Suggested T-Shirt size (S, M, L, XL, Unknown):

@nluedema nluedema added the feature request Categorizes issue or PR as related to a new feature or enhancement. label Jul 26, 2024
@workato-integration
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Categorizes issue or PR as related to a new feature or enhancement.
Projects
None yet
Development

No branches or pull requests

1 participant