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

Download workspace shows error while any workpace is running for the first time #4977

Closed
tyler36 opened this issue Jul 28, 2021 · 11 comments
Closed
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

@tyler36
Copy link

tyler36 commented Jul 28, 2021

Bug description

When I clicked the download link on the dashboard, the page reloaded with an error

Oh, no! Something went wrong!
Error

I tried 3 times but with same results. Unfortunately, the error is generic and therefore I am unsure what the problem might be.

My first attempts was with a gitpod session spawned by DrupalPod on this issues
I also attempted a gitpod session spawened by DrupalPod on this issue

Finally, in order to have "steps to repoduce" I attempted it with a default template, see below.

Steps to reproduce

  1. Visit https://gitpod.io/workspaces/
  2. Click New workspace
  3. Click Examples tab
  4. Click template-php-drupal-ddev
  5. Click Gitpod button in bottom left, and select Gitpod: open dashboard
  6. Click virtual dots (have we decided on a name for these yet?) for the workspace and select download
  7. New page loads, with generic error (see screenshot)
    gitpod-download-error

Expected behavior

Obviously a download would be preferable.

Example repository

No response

Anything else?

Besides the download feature not working, perhaps the error message could be improved. Eg.

  • "Your account does not have permsission"
  • "We seem to be having a network issue"
  • "This workspace is currently locked"
  • "The download feature is unavailable at this time"

It's hard to know if it's a gitpod issues, template issue, feature issues, temporary glitch in the matrix issue, permissions issue ... etc.

Since I'm on the "basic" account with limited hours, I'm hesitant to randomly spin up different workspaces and try downloading them. There really isn't any information to begin debugging though.

@pawlean pawlean added the type: bug Something isn't working label Jul 28, 2021
@gtsiolis
Copy link
Contributor

gtsiolis commented Aug 9, 2021

Thanks for opening this issue @tyler36! FYI, downloading is currently not possible while the workspace is running. Providing a better error message could help here.

@shaal
Copy link
Contributor

shaal commented Aug 12, 2021

This problem can be solved at the frontend.
The download link should be disabled when a workspace is running and a popup message should explain that a workspace must be stopped before it can be downloaded.

@tyler36
Copy link
Author

tyler36 commented Aug 17, 2021

@shaal , disabling the download link makes sense. Adding a tooltip / popup explaining would improve the user experience.

At the moment it feels "broken".

@stale
Copy link

stale bot commented Nov 15, 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 Nov 15, 2021
@gtsiolis
Copy link
Contributor

Re-opening as this is still relevant.

@gtsiolis gtsiolis reopened this Nov 15, 2021
@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Nov 15, 2021
@gtsiolis gtsiolis added meta: never-stale This issue can never become stale meta: stale This issue/PR is stale and will be closed soon team: webapp Issue belongs to the WebApp team user experience labels Nov 15, 2021
@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Nov 15, 2021
@jldec jldec changed the title Download cause an error Download workspace shows error while any workpace is running for the first time Nov 27, 2021
@jldec
Copy link
Contributor

jldec commented Nov 27, 2021

This feels broken.

I believe this only applies during the very first workspace start. If you stop and then restart a workspace, the download will return the last saved state even while the workspace is running.

@JanKoehnlein
Copy link
Contributor

I am inclined to always disable the download button if the workspace is running, as the user might expect the current state not the last backup. Unfortunately, this is often the last resort to get at least some data when workspaces are not stopping for whatever reason. Maybe labeling it "Download (latest) backup" would be better and disabling if there is no backup?

@gtsiolis
Copy link
Contributor

I am inclined to always disable the download button if the workspace is running

Agree. We could disable and provide some context on why the button is disabled through a tooltip or a label inline.

Alternatively, we coud remove the download button when downloading is not possible.

FWIW, we have a similar but lower-impact situation when deleting workspaces in PREPARING phase, see #4173 and #5186. Cc @mrsimonemms

@tyler36
Copy link
Author

tyler36 commented Dec 1, 2021

I prefer the option of a disabled button with text explaining why. This shows that the feature is usually available but currently is not because .

Removing buttons could be seen as removing features.

@jldec
Copy link
Contributor

jldec commented Jan 6, 2022

Let's fix the error with minimal additional complexity for now.

  • Change Action to say "Download Backup".
  • Don't change the behavior except when there is no backup.
  • In the case of no backup, disable the "Download Backup" action, and show a tooltip which says
    No backups available until a workspace is stopped at least once.

@gtsiolis
Copy link
Contributor

gtsiolis commented Nov 9, 2022

Closing this as we're removing the workspace download functionality in the scope of workspace file durability (#7901).

See also #14364 (comment):

Besides ensuring workspace files reliability, this could push us towards figuring out new ways to allow users to export or back up workspace files and more importantly file changes, like exporting in branch, downloading a patch, and more.

Cc @kylos101 @atduarte

@gtsiolis gtsiolis closed this as completed Nov 9, 2022
@gtsiolis gtsiolis moved this to In Validation in 🍎 WebApp Team Nov 9, 2022
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: In Validation
Development

No branches or pull requests

6 participants