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

[helm] namingPolicies in values.yaml is not reflected in configmap #41

Closed
zoetrope opened this issue Dec 8, 2021 · 1 comment
Closed
Labels
bug Something isn't working

Comments

@zoetrope
Copy link
Member

zoetrope commented Dec 8, 2021

Describe the bug

NamingPolicy is now supported in #17.
However, namingpolicies in values.yaml is not reflected in configmap.

Because namingpolicies is not described in configmap.yaml.
https://github.com/cybozu-go/accurate/blob/da7508cf549fd7c49668d515e759345d6c6824a6/charts/accurate/templates/configmap.yaml

Environments

  • Version: 0.3.0
  • Chart Version: 0.2.2

To Reproduce
Steps to reproduce the behavior:

  1. Prepare a values.yaml containing namingPolicies as below:
controller:
  config:
    namingPolicies:
      - root:  foo
        match: foo_.*
  1. Deploy accurate using Helm with the values.yaml.
helm install --create-namespace --namespace accurate accurate -f values.yaml accurate/accurate
  1. The generated configmap does not contain namingPolicies

Expected behavior
The configmap should contain namingPolicies.

@zoetrope zoetrope added the bug Something isn't working label Dec 8, 2021
@ymmt2005
Copy link
Member

Fixed in #42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants