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

Add some tests for cgroup v2 #8658

Merged
merged 2 commits into from
Mar 11, 2022
Merged

Add some tests for cgroup v2 #8658

merged 2 commits into from
Mar 11, 2022

Conversation

utam0k
Copy link
Contributor

@utam0k utam0k commented Mar 8, 2022

Description

  • Add a test for cgroup v2
  • Add a test for k3s

Related Issue(s)

Fixes #8627

How to test

Run the integration test following README of tests.
(e.g. https://werft.gitpod-dev.com/job/gitpod-custom-to-cgv2-itest.2/logs)
For cgroup v2, you need to prepare a dedicated image to build the environment. core-dev is cgoup v1, so the test for cgroup v2 is skipped. gitpod-k3s-202203090157-cgv2 is the image that set the cgroup v2 setting.

Release Notes

None

Documentation

@codecov
Copy link

codecov bot commented Mar 8, 2022

Codecov Report

Merging #8658 (f07e348) into main (f4826d3) will decrease coverage by 1.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #8658      +/-   ##
==========================================
- Coverage   12.31%   11.17%   -1.14%     
==========================================
  Files          20       18       -2     
  Lines        1161      993     -168     
==========================================
- Hits          143      111      -32     
+ Misses       1014      880     -134     
+ Partials        4        2       -2     
Flag Coverage Δ
components-gitpod-cli-app 11.17% <ø> (ø)
components-local-app-app-darwin-amd64 ?
components-local-app-app-darwin-arm64 ?
components-local-app-app-linux-amd64 ?
components-local-app-app-linux-arm64 ?
components-local-app-app-windows-386 ?
components-local-app-app-windows-amd64 ?
components-local-app-app-windows-arm64 ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
components/local-app/pkg/auth/auth.go
components/local-app/pkg/auth/pkce.go

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Command: "bash",
Args: []string{
"-c",
"while [ ! -e /workspace/k3s ]; do sleep 1; done && sudo chmod +x /workspace/k3s && sleep 20 && sudo /workspace/k3s kubectl wait --for=condition=Ready nodes -l node-role.kubernetes.io/master=true --timeout 30s && sudo /workspace/k3s kubectl get nodes",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a draft now because we need sleep 20. Without it, the Text file will be busy. I plan to fix this.

@utam0k utam0k force-pushed the to/cgv2-itest branch 6 times, most recently from 93d4db5 to 91b36bb Compare March 9, 2022 01:36
@utam0k utam0k marked this pull request as ready for review March 9, 2022 02:48
@utam0k utam0k requested a review from a team March 9, 2022 09:04
@github-actions github-actions bot added the team: workspace Issue belongs to the Workspace team label Mar 9, 2022
test/tests/workspace/cgroup_v2_test.go Outdated Show resolved Hide resolved
test/tests/workspace/cgroup_v2_test.go Outdated Show resolved Hide resolved
@utam0k utam0k force-pushed the to/cgv2-itest branch 3 times, most recently from b170bac to 63741ad Compare March 10, 2022 01:51
@utam0k
Copy link
Contributor Author

utam0k commented Mar 10, 2022

@Furisto PTAL

@Furisto
Copy link
Member

Furisto commented Mar 10, 2022

@utam0k Can you squash your commits? Chris has reminded me that we should not have too many commits.

@utam0k
Copy link
Contributor Author

utam0k commented Mar 11, 2022

@utam0k Can you squash your commits? Chris has reminded me that we should not have too many commits.

@Furisto Sure. I organized some commits. PTAL

@roboquat roboquat merged commit 8a5bd2f into main Mar 11, 2022
@roboquat roboquat deleted the to/cgv2-itest branch March 11, 2022 10:30
@roboquat roboquat added deployed: workspace Workspace team change is running in production deployed Change is completely running in production labels Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed: workspace Workspace team change is running in production deployed Change is completely running in production release-note-none size/L team: workspace Issue belongs to the Workspace team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add integration test for workspacekit cgroup v2 behaviour
3 participants