Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/kibana,metricbeat,filebeat] default config is merged to user defined config #12199

Closed
kimxogus opened this issue Mar 14, 2019 · 6 comments
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.

Comments

@kimxogus
Copy link
Contributor

Is this a request for help?: no


Is this a BUG REPORT or FEATURE REQUEST? (choose one):

Version of Helm and Kubernetes:

Client: &version.Version{SemVer:"v2.13.0", GitCommit:"79d07943b03aea2b76c12644b4b54733bc5958d6", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.0", GitCommit:"79d07943b03aea2b76c12644b4b54733bc5958d6", GitTreeState:"clean"}

Which chart: stable/filebeat stable/kibana stable/metricbeat

What happened: As default kibana.yml, metricbeat.yml and filebeat.yml config values are defined in yaml, those values are automatically merged to user defined configs.
As a result, my kibana.yml become mixed as below

elasticsearch:
  hosts:
  - user-defined-es-host
elasticsearch.url: http://elasticsearch:9200
server:
  host: "0"
  rewriteBasePath: true
server.host: "0"
server.name: kibana
xpack:
  canvas.enabled: false
  infra.enabled: false
  logstash.enabled: false
  monitoring:
    enabled: true
    kibana:
      collection:
        enabled: true

What you expected to happen: Only user defined config values are set in config file. Default config values needs to be empty

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know:

@puetzp
Copy link

puetzp commented Mar 14, 2019

I am experiencing the same issue. When I define an output destination for metricbeat in values.yaml file like so:

...
  config:
    metricbeat.config:
      modules:
        path: ${path.config}/modules.d/*.yml
        reload.enabled: false
    processors:
    - add_cloud_metadata:
    output.logstash:
      hosts: ["<snip>:<snip>"]
  modules:
    system:
      enabled: true
      config:
...

the decoded metricbeat.yml from the kube-secret includes both the custom and default config:

    metricbeat.config:
      modules:
        path: ${path.config}/modules.d/*.yml
        reload.enabled: false
    output.file:
      filename: metricbeat
      number_of_files: 5
      path: /usr/share/metricbeat/data
      rotate_every_kb: 10000
    output.logstash:
      hosts:
      - <snip>:<snip>
    processors:
    - add_cloud_metadata: null

Edit: I'm using v2.13.0 as well

@cdenneen
Copy link
Contributor

Had same problem here with stable/filebeat
helm/helm#5184 (comment)

Trying to remove the values for output.file didn't work causing issues.

@dperetti
Copy link

dperetti commented Apr 4, 2019

The workaround is to use overrideConfig.

@pjanuario
Copy link

@dperetti overrideConfig is a solution for the stable/filebeat but not for stable/kibana, for Kibana I couldnt found a workaround yet.

@stale
Copy link

stale bot commented May 20, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@stale stale bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 20, 2019
@stale
Copy link

stale bot commented Jun 3, 2019

This issue is being automatically closed due to inactivity.

@stale stale bot closed this as completed Jun 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants