1Password / connect-helm-charts

Official 1Password Helm Charts
https://developer.1password.com
MIT License
90 stars 73 forks source link

Allow log format parameter #117

Closed Steffen911 closed 1 year ago

Steffen911 commented 1 year ago

Summary

Currently all containers default to the human-readable log format as described here https://developer.1password.com/docs/cli/reference/. It should be possible to set the desired format on the helm chart.

Use cases

Set log format via parameters to ensure that monitoring systems pass the logs correctly.

Proposed solution

Allow a operator.logFormat, connect.logFormat, and sync.logFormat, or global logFormat parameter on the helm chart. Alternatively, accept a set of extraEnvs that are propagated to all pods such that a user can set OP_FORMAT.

Is there a workaround to accomplish this today?

Not that I'm aware of.