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

New variables added to the manifest don't get generated #733

Closed
f0rmiga opened this issue Dec 5, 2019 · 3 comments · Fixed by #745
Closed

New variables added to the manifest don't get generated #733

f0rmiga opened this issue Dec 5, 2019 · 3 comments · Fixed by #745
Labels
accepted bug Something isn't working

Comments

@f0rmiga
Copy link
Member

f0rmiga commented Dec 5, 2019

The issue

While investigating the rotate-cc-database-key errand on kubecf (cloudfoundry-incubator/kubecf#203), I ended up creating a mechanism to add new variables: https://github.com/SUSE/kubecf/blob/44c9f5d280937682dde9c50d995662abe9a7f9dd/deploy/helm/kubecf/assets/operations/instance_groups/api.yaml#L46-L66.

The end result was:

database_encryption:
  current_key_label: encryption_key_1
  keys:
    encryption_key_0: Tw2nYSfrUOUeKGz15S6YVRLwrKzmQrgHukNe0PpTDyVeLnRKmVeZlxh2ptNyiLeq
    encryption_key_1: ((encryption_key_1))

WIP: https://github.com/SUSE/kubecf/tree/f0rmiga/rotate-cc-database-key

I can see that the new variable was added to the with-ops manifest, but cf-operator didn't create a new secret for that.

Expected behaviour

I expected that the variable would be created, and the ((encryption_key_1)) would be rendered to its value.

@f0rmiga f0rmiga added the bug Something isn't working label Dec 5, 2019
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/170142795

The labels on this github issue will be updated when the story is started.

@loewenstein
Copy link

That also means updating from cf-deployment 12.x to some later version (that might add variables) is currently broken?

@f0rmiga
Copy link
Member Author

f0rmiga commented Dec 6, 2019

@loewenstein Most likely.

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

Successfully merging a pull request may close this issue.

3 participants