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

Use chartFileRef if is defined #32

Closed
wants to merge 1 commit into from
Closed

Use chartFileRef if is defined #32

wants to merge 1 commit into from

Conversation

renanqts
Copy link

@renanqts renanqts commented Mar 14, 2020 •
edited
Loading

This PR cover the case below.

  valuesFrom:
  - chartFileRef:
      path: another-file.yaml

This is part of #25

@renanqts renanqts mentioned this pull request Mar 14, 2020
Open
@renanqts
Copy link
Author

@stefanprodan could you please take a look?

@stefanprodan
Copy link
Owner

Wouldn't this break the releases coming from Helm repos as the file is inside the chart tar.gz and not in git?

@renanqts
Copy link
Author

renanqts commented Mar 21, 2020 •
edited
Loading

You mean when is specified valuesFrom in this case? @stefanprodan

@renanqts renanqts closed this May 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@renanqts @stefanprodan