Skip to content

Conversation

@yiannistri
Copy link
Contributor

@yiannistri yiannistri commented Oct 5, 2020

  • Add interval to a couple of examples in docs as it's now a required field

@stefanprodan
Copy link
Member

The e2e tests are failing because Bitnami introduced a chart dependency named common and source controller doesn't support dependencies in Git repos that point to Helm repos. We need to use a repo that we control cc @hiddeco

@hiddeco
Copy link
Member

hiddeco commented Oct 5, 2020

@yiannistri created #165 to solve the test issue. While we wait for the chart release, can you please amend your commit message to include a bit of the information from the PR title? Thanks 🧁

@yiannistri
Copy link
Contributor Author

Yes sure 👍

@hiddeco
Copy link
Member

hiddeco commented Oct 6, 2020

@yiannistri if you rebase this it should now pass :-)

@stefanprodan stefanprodan merged commit 198303a into fluxcd:main Oct 6, 2020
@yiannistri yiannistri deleted the docs/add-interval branch October 6, 2020 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants