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

UX: Dead end when image-build fails during prebuild #4879

Closed
Tracked by #9291 ...
geropl opened this issue Jul 20, 2021 · 15 comments
Closed
Tracked by #9291 ...

UX: Dead end when image-build fails during prebuild #4879

geropl opened this issue Jul 20, 2021 · 15 comments
Labels
component: dashboard meta: never-stale This issue can never become stale team: webapp Issue belongs to the WebApp team type: bug Something isn't working user experience

Comments

@geropl
Copy link
Member

geropl commented Jul 20, 2021

Bug description

When an image build for a preview fails, the user is left on alone on a HeadlessLogView, without any error being displayed.

The gap here stems from the fact that the HeadlessLogView assumes we will navigate away from it on workspace start, no matter the prebuild result. But we'll only do so if ideUrl is set for that instance - which is not the case during image builds.

Context:

Steps to reproduce

  1. apply a commit like this to a testing branch
  2. start a prebuild which triggers an image build taking longer than that period

Expected behavior

The exact behavior is up to discussion, but we should make sure we unstuck the user here!

Example repository

No response

Anything else?

No response

@csweichel
Copy link
Contributor

/schedule

@geropl
Copy link
Member Author

geropl commented Jul 20, 2021

/cc @jankeromnes @gtsiolis

@csweichel
Copy link
Contributor

If the image build for a prebuild fails, the following options come to mind:

  1. restart the workspace with default image and no prebuild:
  • pro: starts a workspace immediately
  • con: it's not very clear to the user what just happened. They end up in a workspace with an unexpected image and no prebuild
  1. restart the workspace with no prebuild:
  • pro: starts a workspace immediately
  • con: it's likely that the image build will fail here, too, so the user will be left with an error after all. At least now they can understand what the error is
  1. show an option button giving the user a choice to "Skip Prebuild" and/or "Skip Prebuild and use Default Image"
  • pro: the least magical choice
  • con: really feels like the "boot in rescue mode"

Either way, we should probably show the image build error irregardless :)

@csweichel
Copy link
Contributor

Let's clarify on how we want to do this, and then re-schedule (due to its priority it will end up at the top of the list anyways).
I'll remove this from groundwork until then.

@geropl
Copy link
Member Author

geropl commented Jul 21, 2021

Regarding 2:

con: it's likely that the image build will fail here, too, so the user will be left with an error after all. At least now they can understand what the error is

This is exactly what happened in the incident where this was discovered. So no viable option IMO.

Regarding 3: Given the above "Skip Prebuild and use Default Image" feels like the only non-magic option that helps the user.

Another option:

  1. show a button "Open Docker Editor" bringing the user into the fast-feedback-loop we're implementing for the onboarding flow to fix this issue quickly. 😍 But that might be out-of-scope for this, depending on the availability/maturity of said editor.

@stale
Copy link

stale bot commented Oct 20, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Oct 20, 2021
@shaal
Copy link
Contributor

shaal commented Oct 20, 2021

Is there a solution for this issue?

@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Oct 20, 2021
@stale
Copy link

stale bot commented Jan 18, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Jan 18, 2022
@shaal
Copy link
Contributor

shaal commented Jan 18, 2022

Is there a solution for this issue?

@gtsiolis
Copy link
Contributor

Haven't seen any recent discussions about this. Probably a fallback action during workspace start could suffice. Do you still run into this, @shaal?

It could be interesting to see some metrics about the error rates here and decide about the severity of this. Cc @jldec @jldec

@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Jan 18, 2022
@gtsiolis gtsiolis added meta: stale This issue/PR is stale and will be closed soon team: webapp Issue belongs to the WebApp team labels Jan 18, 2022
@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Jan 18, 2022
@shaal
Copy link
Contributor

shaal commented Jan 18, 2022

@gtsiolis It's hard to know what is the cause of the issue, but I'm still experiencing prebuilds are "stuck".
Without status/warning/error messages, this process can be frustrating/confusing.

@jldec jldec added meta: never-stale This issue can never become stale and removed priority: highest (user impact) Directly user impacting roadmap item: new dashboard labels Feb 8, 2022
@jldec
Copy link
Contributor

jldec commented Feb 8, 2022

Prioritizing as part of #7812

@AlexTugarev
Copy link
Member

This might be solved in 5a6250c.

The "How to test" from #10357 should be sufficient to verify this issue on a preview environment. Maybe the procedure would require to append RUN exit 1 to the Dockerfile to let the image build fail intentionally.

@laushinka
Copy link
Contributor

Tested this with an image build that fails and I was able to continue. I will close this.
@shaal I hope this has worked for you as well.

Screenshot 2022-06-10 at 09 58 40

@shaal
Copy link
Contributor

shaal commented Jun 10, 2022

Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: dashboard meta: never-stale This issue can never become stale team: webapp Issue belongs to the WebApp team type: bug Something isn't working user experience
Projects
Status: No status
Development

No branches or pull requests

8 participants