goharbor / harbor-helm

The helm chart to deploy Harbor
Apache License 2.0
1.14k stars 745 forks source link

feat: .Value.jobservice.logLevel #1764

Closed mmclane closed 2 weeks ago

mmclane commented 1 month ago

This PR is in response to https://github.com/goharbor/harbor-helm/issues/1763

Today you can only specify logLevel once for the whole chart and that loglevel gets used in all of the services.

With the PR you can override .logLevel for the jobservice by setting a value for jobservice.logLevel. I believe this is working and documented and just needs a quick review. I don't see why this same pattern couldn't be done for all services but I just wanted to get it done for jobservice first as that addresses my current problem.