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

Not possible to limit the number of workspaces per user #21860

Closed
Tracked by #21925
ibuziuk opened this issue Dec 2, 2022 · 7 comments
Closed
Tracked by #21925

Not possible to limit the number of workspaces per user #21860

ibuziuk opened this issue Dec 2, 2022 · 7 comments
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator area/dashboard area/doc Issues related to documentation kind/bug Outline of a bug - must adhere to the bug report template. new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes severity/P1 Has a major impact to usage or development of the system. sprint/current status/release-notes-review-needed Issues that needs to be reviewed by the doc team for the Release Notes wording
Milestone

Comments

@ibuziuk
Copy link
Member

ibuziuk commented Dec 2, 2022

Describe the bug

spec:
  components:
    cheServer:
      extraProperties:
        CHE_LIMITS_USER_WORKSPACES_COUNT: "<kept_workspaces_limit>" 

configuration does not work

Che version

7.57@latest

Steps to reproduce

Follow the docs for Limiting the number of workspaces that a user can keep https://www.eclipse.org/che/docs/stable/administration-guide/limiting-the-number-of-workspaces-that-a-user-can-keep/

ERROR: the procedure does not work, user can create unlimited number of workspaces

Expected behavior

Introduce new CR property, since it is not expected to configure the limit of DevWorkspace using che-server extraProperties

devWorkspace:
totalLimitPerUser: '5'
runningLimit: '2'

In general, I believe we need to consider deprecation and removal of the DevWorksace field from the CR and have the workspace limits configured in DevEnvironments

Runtime

OpenShift

Screenshots

No response

Installation method

OperatorHub

Environment

Amazon

Eclipse Che Logs

No response

Additional context

Outdated docs - https://www.eclipse.org/che/docs/stable/administration-guide/limiting-the-number-of-workspaces-that-a-user-can-keep/

In general, I believe we need to consider deprecation and removal of the DevWorksace field from the CR and have the workspace limits configured in DevEnvironments

Release Notes Text

Title: The maximum number of running workspace per user is configurable
A new CheCluster parameter allows administrators to specify the maximum number or running workspaces per user. It can be configured editing the value spec.devEnvironments.maxNumberOfRunningWorkspacesPerUser of CheCluster custom resource.

@ibuziuk ibuziuk added the kind/bug Outline of a bug - must adhere to the bug report template. label Dec 2, 2022
@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 Dec 2, 2022
@ibuziuk ibuziuk added area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator area/dashboard severity/P1 Has a major impact to usage or development of the system. sprint/next and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Dec 2, 2022
@ibuziuk ibuziuk mentioned this issue Dec 5, 2022
82 tasks
@ibuziuk ibuziuk added the area/doc Issues related to documentation label Dec 23, 2022
@tolusha tolusha added this to the 7.60 milestone Jan 19, 2023
@tolusha tolusha closed this as completed Jan 19, 2023
@l0rd
Copy link
Contributor

l0rd commented Jan 19, 2023

We need to update the doc

@l0rd l0rd reopened this Jan 19, 2023
@tolusha
Copy link
Contributor

tolusha commented Jan 19, 2023

The doc was updated in this PR
eclipse-che/che-docs#2538

@l0rd
Copy link
Contributor

l0rd commented Jan 19, 2023

@tolusha sorry about that, we were looking at the wrong version of the doc

@l0rd l0rd closed this as completed Jan 19, 2023
@l0rd l0rd added new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes status/release-notes-review-needed Issues that needs to be reviewed by the doc team for the Release Notes wording labels Jan 19, 2023
@dmytro-ndp
Copy link
Contributor

@ibuziuk : I was able to create 5 workspaces in DS 3.5.0.RC-03-10 having CHE_LIMITS_USER_WORKSPACES_COUNT: '3' in CheCluster config:

spec:
  components:
    devWorkspace:
      runningLimit: '2'
    cheServer:
      debug: false
      logLevel: INFO
      extraProperties:
        CHE_LIMITS_USER_WORKSPACES_COUNT: '3'

Screenshot from 2023-03-14 19-27-22

@l0rd
Copy link
Contributor

l0rd commented Mar 14, 2023

@dmytro-ndp this is not the right field. Please refer to the doc

@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Mar 14, 2023

@l0rd: thanks for the hint.
It worked as expected in DS 3.5.0.RC-03-10 following the article you shared:

spec:
  devEnvironments:
    maxNumberOfWorkspacesPerUser: 3
    maxNumberOfRunningWorkspacesPerUser: 2

Screenshot from 2023-03-15 00-46-59

Screenshot from 2023-03-15 11-28-31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/che-operator Issues and PRs related to Eclipse Che Kubernetes Operator area/dashboard area/doc Issues related to documentation kind/bug Outline of a bug - must adhere to the bug report template. new&noteworthy For new and/or noteworthy issues that deserve a blog post, new docs, or emphasis in release notes severity/P1 Has a major impact to usage or development of the system. sprint/current status/release-notes-review-needed Issues that needs to be reviewed by the doc team for the Release Notes wording
Projects
None yet
Development

No branches or pull requests

6 participants