-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Default namespace strategy should be 1 namespace per user #18381
Comments
this is mostly operator issue with permissions. IMHO it is duplicate of this #15493 which even has open PR eclipse-che/che-operator#166 which I've started almost a year ago eclipse-che/che-operator#137 :) I believe it was not done because it works on openshift with oauth and we don't want to have so wide permissions for che-operator. It's probably worth to revisit. |
Thanks @sparkoo! Is the procedure with permission documented? |
I don't think so and I agree that this should be at least documented. Something like "if you want to have namespace per user strategy, grant these cluster permissions |
In my opinion, during installation, |
@sunix I disagree with this. We should set our defaults to work as smooth as possible. Having a installer that by default does not install working product and link me to documentation is big UX flaw. We can add a warning to do some manual steps, but don't set such cases as defaults. |
@sparkoo the user is expecting to have 1 namespace for all his workspace. This is recommended, and some features rely on it (for instance using user secrets) and we have to be consistent in all our installations as much as we can. The installer should:
I don't see any UX problem with that. |
Issues go stale after Mark the issue as fresh with If this issue is safe to close now please do so. Moderators: Add |
Already implemented |
Describe the bug
Following the documentation, https://www.eclipse.org/che/docs/che-7/installation-guide/installing-che-on-google-cloud-platform/
The default namespace strategy was 1 namespace for all
che
:This installation was done with chectl nightly-build 2 months ago (September 2020).
Expected behavior
Have a namespace per user strategy by default, something like:
The text was updated successfully, but these errors were encountered: