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

Used CPU of a plugin is higher than limit on Developer Sandbox #20768

Closed
Tracked by #20809
l0rd opened this issue Nov 15, 2021 · 13 comments
Closed
Tracked by #20809

Used CPU of a plugin is higher than limit on Developer Sandbox #20768

l0rd opened this issue Nov 15, 2021 · 13 comments
Assignees
Labels
area/plugins kind/bug Outline of a bug - must adhere to the bug report template. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@l0rd
Copy link
Contributor

l0rd commented Nov 15, 2021

Describe the bug

On developer sandbox the memory used by a plugin goes above the limit:

image

That should not happen. A container cannot use more CPU than its limit. Is the limit not applied? Is the used CPU mis-calculated?

Che version

7.36

Steps to reproduce

Use a devfile that includes the openshift connector and specify a low CPU limit for it. Start the workspace on developer sandbox.

Expected behavior

The used CPU should never go above the limit.

Runtime

OpenShift

Screenshots

No response

Installation method

other (please specify in additional context)

Environment

Dev Sandbox (workspaces.openshift.com)

Eclipse Che Logs

No response

Additional context

No response

@l0rd l0rd added 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. area/plugins labels Nov 15, 2021
@l0rd l0rd changed the title Used memory of a plugin is higher than limit on Developer Sandbox Used CPU of a plugin is higher than limit on Developer Sandbox Nov 15, 2021
@l0rd
Copy link
Contributor Author

l0rd commented Nov 15, 2021

I have reproduced that for the Java extension too #20769

@che-bot
Copy link
Contributor

che-bot commented May 14, 2022

Issues go stale after 180 days of inactivity. lifecycle/stale issues rot after an additional 7 days of inactivity and eventually close.

Mark the issue as fresh with /remove-lifecycle stale in a new comment.

If this issue is safe to close now please do so.

Moderators: Add lifecycle/frozen label to avoid stale mode.

@che-bot che-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 14, 2022
@l0rd
Copy link
Contributor Author

l0rd commented May 14, 2022

/remove-lifecycle stale

@che-bot che-bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 14, 2022
@vinokurig vinokurig self-assigned this Aug 18, 2022
@vinokurig
Copy link
Contributor

vinokurig commented Aug 18, 2022

Could not overdraw the limit on dog-fooding with che-code editor:
screenshot-che-dogfooding apps che-dev x6e0 p1 openshiftapps com-2022 08 18-16_41_44

@svor
Copy link
Contributor

svor commented Aug 18, 2022

@vinokurig try to reproduce it on developer sandbox as it is described in the description

@l0rd
Copy link
Contributor Author

l0rd commented Aug 18, 2022

Note that this issue was reproduced in Theia, not VS Code.

@vinokurig
Copy link
Contributor

Could not reproduce it on devspaces 3.0 with che-theia neither:
screenshot-nimbusweb me-2022 08 19-10_59_39

@ibuziuk
Copy link
Member

ibuziuk commented Aug 19, 2022

Could not overdraw the limit on dog-fooding with che-code editor:

On dogfooding we do apply neither limits nor quotas, it should be verified on the Developer Sandbox staging enviroment

@vinokurig
Copy link
Contributor

I've tried it on the Developer Sandbox with this devfile:

apiVersion: 1.0.0
metadata:
  name: bash-psfc
projects:
  - name: bash
    source:
      location: 'https://github.com/che-samples/bash'
      type: git
      branch: main
components:
  - mountSources: true
    command:
      - tail
    args:
      - '-f'
      - /dev/null
    memoryLimit: 64Mi
    type: dockerimage
    alias: dev
    image: 'registry.access.redhat.com/ubi8-minimal:8.3'
  - id: mads-hartmann/bash-ide-vscode/latest
    type: chePlugin
    registryUrl: 'https://eclipse-che.github.io/che-plugin-registry/7.42.0/v3/'
  - id: rogalmic/bash-debug/latest
    type: chePlugin
    registryUrl: 'https://eclipse-che.github.io/che-plugin-registry/7.42.0/v3/'
  - id: timonwong/shellcheck/latest
    preferences:
      shellcheck.executablePath: /bin/shellcheck
    type: chePlugin
    registryUrl: 'https://eclipse-che.github.io/che-plugin-registry/7.42.0/v3/'
  - id: redhat/vscode-openshift-connector/latest
    memoryLimit: 500Mi
    type: chePlugin

The workspcae failed to start because of the Pod crash loop back-off error see the video:
ezgif-3-a6208c33c8

@svor
Copy link
Contributor

svor commented Aug 22, 2022

It seems the problem with memory limit, did you detect which container was failed with OOM? Try to increase memoryLimit for that component

@vinokurig
Copy link
Contributor

After the developer sandbox has been updated to 3.1 I still can't reproduce the issue. When I setupt the limit to less then 750 Mi, workspaces crashes when it reaches the maximum of RAM, but if setup more than 750 Mi, I can't overload the containers RAM limit.

@svor
Copy link
Contributor

svor commented Oct 11, 2022

I've reproduced it on the dogfooding instance:

  • Start a Quarkus workspace with VSCode editor
  • Install all recommended plugins
  • Enable the normal Java LS
  • See that Java LS is building the project
  • Open resource monitor info

Screenshot from 2022-10-11 12-48-24

@che-bot
Copy link
Contributor

che-bot commented Apr 9, 2023

Issues go stale after 180 days of inactivity. lifecycle/stale issues rot after an additional 7 days of inactivity and eventually close.

Mark the issue as fresh with /remove-lifecycle stale in a new comment.

If this issue is safe to close now please do so.

Moderators: Add lifecycle/frozen label to avoid stale mode.

@che-bot che-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 9, 2023
@ibuziuk ibuziuk added the lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. label Apr 9, 2023
@che-bot che-bot closed this as completed Apr 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plugins kind/bug Outline of a bug - must adhere to the bug report template. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

5 participants