Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ERROR] unmarshal error from configmap.yaml #194

Closed
daegeun-ha opened this issue Oct 10, 2019 · 4 comments · Fixed by #198
Closed

[ERROR] unmarshal error from configmap.yaml #194

daegeun-ha opened this issue Oct 10, 2019 · 4 comments · Fixed by #198

Comments

@daegeun-ha
Copy link

@daegeun-ha daegeun-ha commented Oct 10, 2019

There is an error from configmap.yaml, line number 184
after 'namespaces', neither include nor ignore isn't appear.
Maybe it is changed like below

namesapces:
include:
-all
ignore:
-

@daegeun-ha daegeun-ha changed the title unmarshal error from configmap.yaml [Bunmarshal error from configmap.yaml Oct 10, 2019
@daegeun-ha daegeun-ha changed the title [Bunmarshal error from configmap.yaml unmarshal error from configmap.yaml Oct 10, 2019
@daegeun-ha daegeun-ha changed the title unmarshal error from configmap.yaml [ERROR] unmarshal error from configmap.yaml Oct 10, 2019
@sanketsudake
Copy link
Member

@sanketsudake sanketsudake commented Oct 10, 2019

@daegeun-ha Please include your configmap.yaml complete contents ?

@PrasadG193
Copy link
Member

@PrasadG193 PrasadG193 commented Oct 10, 2019

@daegeun-ha Please make sure you are using the latest helm chart and image: infracloudio/botkube:latest
https://github.com/infracloudio/botkube/tree/develop/helm/botkube

@daegeun-ha
Copy link
Author

@daegeun-ha daegeun-ha commented Oct 11, 2019

Hello, Thank you for fast reply :)
@PrasadG193 latest helm chart also has the same problem
@sanketsudake my configmap.yaml is same as the original.
https://github.com/infracloudio/botkube/blob/develop/helm/botkube/values.yaml
problem line numbers are 183, 184

@PrasadG193
Copy link
Member

@PrasadG193 PrasadG193 commented Oct 11, 2019

Aah, I think we missed this while refactoring.

Correct format is:

    - name: clusterrole
      namespaces:
        include:
          - all
      events:
        - create
        - delete
        - error

I am assuming you are getting this error while executing @Botkube notifier configmap

@daegeun-ha Please feel free to make this change and raise a PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants