-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
helm deploy failed: matching build results to chart values: no build present for <image> #1748
Comments
@WakeupTsai , i think the meaningful error is last line,
Can you provide your helm Charts config or point us to your code? |
I run the example code
|
After a few tests, I found the problem is caused by
|
I am seeing this same error now on my branch where i am testing helm. |
Should be fixed in #2105. |
Came across the same issue, tested it on v0.28 - v.033 |
We decided not to use #2105. I am no longer working on this. |
@alejandrox1 can you share your setup? Also, we have merged some fixes since then, it would be worthwhile trying to reproduce this on master / bleeding edge (https://skaffold.dev/docs/getting-started/#installing-skaffold). |
I'm going to assume this was fixed with the most recent changes to the helm deployer. Please reopen this issue if that's not the case. Thanks a lot for your contribution! |
Please reopen this issue, I'm experiencing this issue with the newest bleeding-edge release. |
@petr-buchyn I am experiencing this issue as well. Setting my values with the |
@balopat I'm also having trouble installing a helm chart with skaffold. With helm 3 on microk8s with the
Other details:
|
As a workaround I sent the values through a file referenced from deploy:
helm:
releases:
- name: r
chartPath: path/to/helm/chart
namespace: skaffold
valuesFiles:
- path/to/helm/values.yaml |
I have a feeling this issue wasn't really resolved the first time around, so I'm reopening until I feel a little more confident about it. would anyone here be willing to try and repro this again on the latest skaffold version and tell me if the issue is still there? we've made some updated to helm in the last few months but I suspect there are still issues with the way the |
Not sure if it's the same, but my issue in #3948 seems similar. |
With a fresh kubernetes environment and Skaffold never being executed before, this fails when running eddit:// I'm on skaffold 1.7.0 and helm v3.1.2 |
Sorry, for the comment flood, did some more research:
fails with:
Removing the
What's interesting though is that I have another release which uses an image built in the build phase which also uses
In fact, I believe that this problem appears when the image is not built with Skaffold but pulled from e.g. Docker Hub. To summarize:
I think it is legit to tag this as helm/v3 as well. edit:// edit2:// This workaround does not fix the cleanup stage bug. It only allows skaffold to start. If you have a failed release or try to clean up with delete, Skaffold still fails.
|
Ok, sorry for digging this up! After some serious bug hunting it appeared that a chart we use upgraded but had also introduced regression issues. This caused the release to fail, culminating in the logs above. Because Now that everything is cleaned up, apart from the two linked issues, everything appears to be working. In fact, I believe that #1748 (comment) may cause #3948 . I believe that I've run into issues using the In conclusion, if you end up with this issue, it's probably because your deployment failed due to a serious error (e.g. helm template broken, using wrong image, ...). If you check your helm releases using Thank you for your patience! |
@aeneasr - thank you for the detailed feedback! I'm going to close this in the mean-time, but I suspect your research/debugging will prove useful for others to find. If you can think of any ways for us to improve this error message, please open a new issue with the enhancement idea. Thank you! |
I'm trying to run the helm chart example, but seems the skaffold doesn't install the helm chart, and delete it directly.
The text was updated successfully, but these errors were encountered: