Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Commit

Permalink
osm-deployment: parametrize osm log level in values.yaml (#2046)
Browse files Browse the repository at this point in the history
* osm-deployment: parametrize log level.

Currently it's hardcoded in deployment. This gives at least some
short term flexibility till we add configMap update, which has some
ongoing discussion as how to tackle.

* address comments
  • Loading branch information
eduser25 authored Nov 13, 2020
1 parent 9f7c8ce commit 2a09728
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion charts/osm/templates/osm-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ spec:
containerPort: 15128
command: ['/osm-controller']
args: [
"--verbosity", "trace",
"--verbosity", "{{.Values.OpenServiceMesh.controllerLogLevel}}",
"--osm-namespace", "{{.Release.Namespace}}",
"--mesh-name", "{{.Values.OpenServiceMesh.meshName}}",
"--init-container-image", "{{.Values.OpenServiceMesh.image.registry}}/init:{{ .Values.OpenServiceMesh.image.tag }}",
Expand Down
1 change: 1 addition & 0 deletions charts/osm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ OpenServiceMesh:
meshName: osm
useHTTPSIngress: false
envoyLogLevel: error
controllerLogLevel: trace
enforceSingleMesh: false

# Set deployJaeger to true to deploy a Jaeger cluster in the
Expand Down

0 comments on commit 2a09728

Please sign in to comment.