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

Changes in workspace get lost if restarting from local Devfile with typo #22638

Closed
l0rd opened this issue Oct 27, 2023 · 1 comment
Closed

Changes in workspace get lost if restarting from local Devfile with typo #22638

l0rd opened this issue Oct 27, 2023 · 1 comment
Assignees
Labels
area/dashboard kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.

Comments

@l0rd
Copy link
Contributor

l0rd commented Oct 27, 2023

Describe the bug

With the recent changes the default workspace is started in ephemeral mode (aka "safe mode").

This is ok for workspace creation but when it's about a "restart" then all the changes in the workspace get lost.

We should be very careful and avoid proposing to restart in ephemeral mode when it was about the restart of an existing workspace:

  • Restart
  • Restart using defaults (tooltip: using the default component)
  • Edit DevWork### Describe the bug

With the recent changes the default workspace is started in ephemeral mode (aka "safe mode").

This is ok for workspace creation but when it's about a "restart" then all the changes in the workspace get lost.

Che version

next (development version)

Steps to reproduce

  • Start an empty workspace
  • Add a few files to the workspace and then add a devfile with an image that doesn't exist (see example below)
  • Select restart from local Devfile
  • Wait for an error message
  • Click the "Restart with default devfile" button.
  • Wait for the target workspace to restart and open IDE
  • See that the devfile doesn't exist anymore, and all the changes got lost
schemaVersion: 2.2.0
metadata:
  name: devfile-with-typo
components:
  - name: tools
    container:
      image: quay.io/does/not/exist

Expected behavior

The workspace should not be restarted in ephemeral mode

@l0rd l0rd added the kind/bug Outline of a bug - must adhere to the bug report template. label Oct 27, 2023
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Oct 27, 2023
@amisevsk amisevsk added severity/P1 Has a major impact to usage or development of the system. area/dashboard and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Oct 27, 2023
@l0rd l0rd changed the title Changes in workspace get lost if restarting from local Devfile that has a typo Changes in workspace get lost if restarting from local Devfile with typo Oct 29, 2023
@ibuziuk ibuziuk moved this to Unplanned Tasks in Eclipse Che Team A Backlog Oct 30, 2023
@ibuziuk
Copy link
Member

ibuziuk commented Nov 2, 2023

fixed in main for 7.77.0 eclipse-che/che-dashboard#967

@ibuziuk ibuziuk closed this as completed Nov 2, 2023
@svor svor moved this from Unplanned Tasks to ✅ Done in Eclipse Che Team A Backlog Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dashboard kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

4 participants