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

Can we give better errors in spinapp status? #307

Open
lann opened this issue Aug 19, 2024 · 2 comments
Open

Can we give better errors in spinapp status? #307

lann opened this issue Aug 19, 2024 · 2 comments

Comments

@lann
Copy link
Contributor

lann commented Aug 19, 2024

Context: https://discord.com/channels/926888690310053918/950022897160839248/1275126957322801192 and https://www.youtube.com/watch?v=2CFj4_wt2qo&t=3609s

They were attempting to deploy a Wasm OCI artifact which is not yet directly supported by Spin.

Currently they just get the standard k8s "Deployment does not have minimum availability" status, but I wonder if we could give more useful feedback, especially when the app configuration has fatal errors.

@endocrimes
Copy link
Contributor

That would be cool - but I'm not sure how we can sensibly emit those from the Shim in the way that the operator can reliably parse and proxy into the object - I wonder if @devigned has any ideas?

@devigned
Copy link

I think we should be able to pass that back as pod events via containerd.

More broadly, any terminal error which causes us to fail running a workload should be bubbled up to users in some coherent way. We should never leave a user to guess at the error which caused the failure of their workload.

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

No branches or pull requests

3 participants