ci: make cdbg deploy errors easier to spot

burgerdev commented 3 weeks ago

Context

We had a couple of failed cdbg deploy steps recently, which are incredibly hard to spot in the logs because they are hidden in the log group. Adding an explicit ::error:: should highlight them.

Proposed change(s)

Write an error message if cdbg deploy failed.

Checklist

[x] Run the E2E tests that are relevant to this PR's changes
- [x] https://github.com/edgelesssys/constellation/actions/runs/9512506797
[x] Add labels (e.g., for changelog category)
[x] Is PR title adequate for changelog?
[x] Link to Milestone

netlify[bot] commented 3 weeks ago

Deploy Preview for constellation-docs canceled.

Name	Link
Latest commit	0775318d697aa5e38f0cac717fbb9804a4a04bc3
Latest deploy log	https://app.netlify.com/sites/constellation-docs/deploys/666bf4373b29e80008a60ccf

burgerdev commented 3 weeks ago

Thanks for the review! I just realized that the || echo construction swallows the return code - does ::error:: fail the group regardless?

daniel-weisse commented 3 weeks ago

Pretty sure `echo "::error" only highlights the message in the action output

edgelesssys / constellation