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

Fix instructions for encryption-config #373

Merged
merged 1 commit into from Jan 29, 2019

Conversation

@jaimegag
Copy link
Collaborator

@jaimegag jaimegag commented Jan 22, 2019

What this PR does / why we need it:
Since this commit the instructions provided are no longer true. When using a --var-file a | is introduced and that breaks the to_yaml parsing.

How can this PR be verified?
Testing the manifest/ops-files/enable-encryption-config.yml config

Is there any change in kubo-release?
This is consequence of a change in kubo-release

Is there any change in kubo-ci?
NONE

Does this affect upgrade, or is there any migration required?
NONE

Which issue(s) this PR fixes:
NONE

Release note:
NONE

@carlo-colombo carlo-colombo changed the base branch from master to develop Jan 29, 2019
@carlo-colombo carlo-colombo merged commit 0e83f54 into cloudfoundry-incubator:develop Jan 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants