Skip to content

chore(dogfood): update the dogfood template to add workspace CPU and RAM usage from cgroup #7529

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
May 19, 2023
Merged

chore(dogfood): update the dogfood template to add workspace CPU and RAM usage from cgroup #7529

merged 9 commits into from
May 19, 2023

Conversation

matifali
Copy link
Member

@matifali matifali commented May 15, 2023 •
edited
Loading

This currently does not work on US Pittsburgh because of being on Ubuntu Focal and Kernel 5.4.
But works fine in all other dogfood regions
See related issue: #7076

@matifali
chore(dogfood): Update dogfood to add cgroup CPU and RAM usage
5bd60c0 
This currently does not work on US Pittsburgh because of being on Ubuntu Focal and Kernel 5.4.
@matifali matifali requested review from bpmct and ammario May 15, 2023 13:20
@ammario ammario removed their request for review May 15, 2023 15:35
@matifali matifali requested review from kylecarbs and ammario and removed request for ammario May 15, 2023 18:31
@matifali matifali changed the title chore(dogfood): Update dogfood to add cgroup CPU and RAM usage chore(dogfood): update the dogfood template to add CPU and RAM usage from cgroup May 18, 2023
@matifali matifali changed the title chore(dogfood): update the dogfood template to add CPU and RAM usage from cgroup chore(dogfood): update the dogfood template to add workspace CPU and RAM usage from cgroup May 18, 2023
@matifali matifali requested a review from johnstcn May 18, 2023 09:33
@matifali matifali enabled auto-merge (squash) May 18, 2023 13:26
johnstcn
johnstcn reviewed May 18, 2023
View reviewed changes
johnstcn
johnstcn reviewed May 18, 2023
View reviewed changes
johnstcn
johnstcn reviewed May 18, 2023
View reviewed changes
johnstcn
johnstcn reviewed May 18, 2023
View reviewed changes
@matifali matifali requested review from johnstcn and removed request for bpmct and kylecarbs May 18, 2023 15:11
johnstcn
johnstcn reviewed May 18, 2023
View reviewed changes
johnstcn
johnstcn reviewed May 18, 2023
View reviewed changes
johnstcn
johnstcn reviewed May 18, 2023
View reviewed changes
johnstcn
johnstcn approved these changes May 18, 2023
View reviewed changes
Copy link
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

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

I think this should be fine for the moment!
However, it's tricky to get all this information using bash scripts so more of a short-term solution.

matifali and others added 3 commits May 18, 2023 19:57
@matifali
Fix typo
2764d30
@matifali @johnstcn
Update dogfood/main.tf
85675c1 
Co-authored-by: Cian Johnston <cian@coder.com>
@matifali @johnstcn
Update dogfood/main.tf
92d1851 
Co-authored-by: Cian Johnston <cian@coder.com>
@matifali matifali disabled auto-merge May 18, 2023 17:07
@matifali matifali enabled auto-merge (squash) May 19, 2023 11:11
@matifali matifali disabled auto-merge May 19, 2023 11:14
@matifali matifali merged commit ef635f1 into main May 19, 2023
@matifali matifali deleted the dogfood-agent-meta-data-cgroups branch May 19, 2023 12:15
@github-actions github-actions bot locked and limited conversation to collaborators May 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@johnstcn johnstcn johnstcn approved these changes

Assignees

@matifali matifali

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@matifali @johnstcn