Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Support configurable log level in weave-npc #2628

Closed
awh opened this issue Nov 8, 2016 · 2 comments · Fixed by #3683
Closed

Support configurable log level in weave-npc #2628

awh opened this issue Nov 8, 2016 · 2 comments · Fixed by #3683

Comments

@awh
Copy link
Contributor

awh commented Nov 8, 2016

From @awh on October 14, 2016 8:43

Via a configmap or an environment variable.

Copied from original issue: weaveworks-experiments/weave-npc#20

@bboreham
Copy link
Contributor

I believe the crux of this issue is that while the weave-npc program supports our usual command-line flag --log-level, it is nontrivial to edit this into the yaml for the DaemonSet.

Our config generator at cloud.weave.works allows you to add environment variables, so perhaps that is the way to go.

@bboreham bboreham changed the title Support configurable log level Support configurable log level in weave-npc Feb 19, 2018
@hairyhenderson
Copy link
Contributor

For anyone else who came to this from a google search, this can be accomplished by patching the PodSpec in the DaemonSet to add --log-level=error to the args list.

I did something like this:

weave-patch.yml:

kind: DaemonSet
metadata:
  name: weave-net
  namespace: kube-system
spec:
  template:
    spec:
      containers:
        - name: weave-npc
          args:
            - --log-level=error
$ kubectl patch -n kube-system daemonset weave-net --patch "$(< weave-patch.yml)"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants