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

Add support for validation of HelmRelease using ".spec.valuesFrom" feature of FluxCD #57

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jozala
Copy link
Contributor

@jozala jozala commented Dec 22, 2020

This one would solve issue #25, where this functionality is described in more details.

@jozala
Copy link
Contributor Author

jozala commented Dec 22, 2020

The tests are failing because of the old location to stable chart repository in Ghost requirements in fluxcd/flux-get-started repository. I've created a pull request to update this: fluxcd/flux-get-started#60 .

@jozala
Copy link
Contributor Author

jozala commented Jan 14, 2021

Could the tests be rerun for this pull request? I think I don't have privileges to rerun them.

Pull request fluxcd/flux-get-started#60 is now merged so I believe tests should pass now.

@jozala
Copy link
Contributor Author

jozala commented Jan 14, 2021

Thanks for re-running the tests.
I got caught by another problem. This time it is 'requirements.lock' in flux-get-started repository.
I will fix that and make sure everything works before going further with this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant