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

Workspace is not failing if requirement is not satisfied #685

Closed
Tracked by #20830
benoitf opened this issue Nov 24, 2021 · 2 comments
Closed
Tracked by #20830

Workspace is not failing if requirement is not satisfied #685

benoitf opened this issue Nov 24, 2021 · 2 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@benoitf
Copy link
Collaborator

benoitf commented Nov 24, 2021

Description

Start a devWorkspace with high "cpuLimit/memoryLimit"
We see logs about too many cpu requirements, etc but when listing the DevWorkspaces command is displaying "Waiting for workspace deployment"

$ oc get dw
NAME                    DEVWORKSPACE ID             PHASE      INFO
spring-petclinic-bnoj   workspace26699a51196d4b4c   Starting   Waiting for workspace deployment

but events of the platform are reporting

11m         Warning   FailedCreate             replicaset/workspace26699a51196d4b4c-6d58f65b64   Error creating: pods "workspace26699a51196d4b4c-6d58f65b64-k2n48" is forbidden: maximum cpu usage per Pod is 4, but limit is 6
11m         Warning   FailedCreate             replicaset/workspace26699a51196d4b4c-6d58f65b64   Error creating: pods "workspace26699a51196d4b4c-6d58f65b64-qx89t" is forbidden: maximum cpu usage per Pod is 4, but limit is 6
11m         Warning   FailedCreate             replicaset/workspace26699a51196d4b4c-6d58f65b64   Error creating: pods "workspace26699a51196d4b4c-6d58f65b64-5v2ms" is forbidden: maximum cpu usage per Pod is 4, but limit is 6

How To Reproduce

In Che, clicking on https://che-host#https://github.com/benoitf/spring-petclinic/tree/error-cpu

Expected behavior

DevWorkspace should report failure

Additional context

@ibuziuk
Copy link
Contributor

ibuziuk commented Dec 7, 2021

PR - sent #701

@ibuziuk
Copy link
Contributor

ibuziuk commented Dec 13, 2021

PR has been merged. Closing

@ibuziuk ibuziuk closed this as completed Dec 13, 2021
@amisevsk amisevsk added this to the v0.12.0 milestone Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants