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

lvm-localpv lvmPlugin does not support loglevel #300

Open
ferhatguneri opened this issue May 7, 2024 · 1 comment
Open

lvm-localpv lvmPlugin does not support loglevel #300

ferhatguneri opened this issue May 7, 2024 · 1 comment

Comments

@ferhatguneri
Copy link

openebs-lvm-plugin container does not support loglevel.

What did you expect to happen:
openebs-lvm-localpv-controller and openebs-lvm-localpv-node does support and loglevel can be defined for them but sidecar container of controller which is openebs-lvm-plugin does not support --v=3 parameter.

Log Level can be configured for lvmNode and lvmController
https://github.com/openebs/lvm-localpv/blob/develop/deploy/helm/charts/values.yaml#L56
https://github.com/openebs/lvm-localpv/blob/develop/deploy/helm/charts/values.yaml#L73

But it cannot be configured for lvmPlugin. and I tried to pass --v=3 to the sidecard container just like the others but, it crashed the container.
https://github.com/openebs/lvm-localpv/blob/develop/deploy/helm/charts/values.yaml#L145

Environment:

  • openebs helm chart version 3.10.0
  • v1.27.8
  • Linux
@dsharma-dc dsharma-dc added enhancement New feature or request Backlog and removed enhancement New feature or request labels Jun 6, 2024
@avishnu
Copy link
Member

avishnu commented Sep 19, 2024

Further investigation is needed, scoped for v4.3.

@avishnu avishnu added this to the v4.3 milestone Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants