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] Don't show workspaces with zero instances as 'active' #3784

Merged
merged 1 commit into from
Apr 9, 2021

Conversation

jankeromnes
Copy link
Contributor

@jankeromnes jankeromnes commented Apr 6, 2021

Fixes #3780

This isn't a normal case, but apparently it can happen that a workspace has zero instances if it fails at the wrong time during creation (as probably happened to me in the bug).

@jankeromnes
Copy link
Contributor Author

jankeromnes commented Apr 9, 2021

/werft run

👍 started the job as gitpod-build-jx-active-no-instance.1

@jankeromnes

This comment has been minimized.

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.

@jankeromnes will this also resolve #3635?

@jankeromnes
Copy link
Contributor Author

Thanks @gtsiolis!

@jankeromnes will this also resolve #3635?

Unfortunately, it won't. This only fixes the situation where workspaces that were never started got shown as "Active".

@jankeromnes jankeromnes removed the request for review from svenefftinge April 9, 2021 11:38
@jankeromnes jankeromnes merged commit 269c9dc into main Apr 9, 2021
@jankeromnes jankeromnes deleted the jx/active-no-instance branch April 9, 2021 11:38
@gtsiolis
Copy link
Contributor

gtsiolis commented Apr 9, 2021

Sounds also good! Thanks for clarifying and fixing this @jankeromnes! 🏀

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.

[dashboard] Random non-active workspace considered active
2 participants