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

Ensure IO limits are written after first workspace is stopped #9404

Merged
merged 1 commit into from
Apr 19, 2022

Conversation

Furisto
Copy link
Member

@Furisto Furisto commented Apr 19, 2022

Description

IO limits stopped being written after the first workspace was stopped because the plugin is a singleton and we set IO limits to unlimited when the workspace is stopped.

Related Issue(s)

Fixes #9341

How to test

  • Use workspace-preview or ephemeral cluster
  • Open workspace -> observe IO limits are written
  • Close workspace
  • Open another workspace on the same node
  • IO limits should be applied for the new workspace

Release Notes

Fixed a bug where IO limits were not applied to the workspace

@Furisto Furisto requested a review from a team April 19, 2022 13:42
csweichel
csweichel previously approved these changes Apr 19, 2022
@aledbf aledbf self-assigned this Apr 19, 2022
@aledbf aledbf self-requested a review April 19, 2022 14:19
@aledbf aledbf removed their assignment Apr 19, 2022
@roboquat roboquat merged commit d31bb31 into main Apr 19, 2022
@roboquat roboquat deleted the fo/fix-io-limits branch April 19, 2022 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IO limits are not applied
4 participants