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

Attempt to delete a workspace namespace even if the namespace is not managed #15686

Closed
metlos opened this issue Jan 14, 2020 · 0 comments
Closed
Assignees
Labels
area/che-server kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Milestone

Comments

@metlos
Copy link
Contributor

metlos commented Jan 14, 2020

Describe the bug

A confusing error message in the logs after a workspace delete.

Che version

latest

Steps to reproduce

  1. Set up Che with default configuration, namely <username>-che as the namespace default (or actually any value not containing <workspaceid>)
  2. Create a workspace
  3. Delete the workspace
  4. Observe a warning in the logs claiming that the namespace/project cannot be deleted because it is not managed.

Expected behavior

No message in the logs, because the namespace is indeed not managed so Che should not even try to delete it.

@metlos metlos added the kind/bug Outline of a bug - must adhere to the bug report template. label Jan 14, 2020
@skabashnyuk skabashnyuk added this to the 7.8.0 milestone Jan 14, 2020
@metlos metlos added the status/in-progress This issue has been taken by an engineer and is under active development. label Jan 14, 2020
@metlos metlos added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. and removed status/in-progress This issue has been taken by an engineer and is under active development. labels Jan 14, 2020
@l0rd l0rd added the severity/P2 Has a minor but important impact to the usage or development of the system. label Jan 15, 2020
@ibuziuk ibuziuk mentioned this issue Jan 16, 2020
21 tasks
@metlos metlos closed this as completed Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/che-server kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Projects
None yet
Development

No branches or pull requests

3 participants