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

[dashboard] Add workspace status indicator color red for failed status #4378

Merged
merged 1 commit into from
Jun 11, 2021

Conversation

corneliusludmann
Copy link
Contributor

Fixes #3635

@corneliusludmann corneliusludmann force-pushed the corneliusludmann/errored-workspaces-3635 branch 2 times, most recently from e3d7220 to d387483 Compare June 7, 2021 14:15
@corneliusludmann corneliusludmann requested a review from gtsiolis June 7, 2021 14:16
@corneliusludmann corneliusludmann marked this pull request as ready for review June 7, 2021 14:16
@gtsiolis
Copy link
Contributor

gtsiolis commented Jun 9, 2021

/werft run

👍 started the job as gitpod-build-corneliusludmann-errored-workspaces-3635.5

Copy link
Contributor

@gtsiolis gtsiolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @corneliusludmann, do you know if there's an easy way to reproduce this erroneous state on the workspaces list?

@corneliusludmann
Copy link
Contributor Author

Hey @corneliusludmann, do you know if there's an easy way to reproduce this erroneous state on the workspaces list?

Unfortunately, not. 😞

What you can do is to alter the database to add a failed condition (as JSON content, unfortunately, I don't have a query handy). However, it seems to me that the value of this is pretty limited since you don't really test how a failed workspace behaves.

@gtsiolis
Copy link
Contributor

gtsiolis commented Jun 11, 2021

However, it ...

DEAL. Safe to merge this as is once conflicts are resolved? 🥝

@corneliusludmann corneliusludmann force-pushed the corneliusludmann/errored-workspaces-3635 branch from d387483 to a79783a Compare June 11, 2021 15:39
@corneliusludmann
Copy link
Contributor Author

I think it should be safe.

@corneliusludmann corneliusludmann merged commit 34a5b02 into main Jun 11, 2021
@corneliusludmann corneliusludmann deleted the corneliusludmann/errored-workspaces-3635 branch June 11, 2021 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Errored workspaces don't use the correct status indicator
2 participants