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

fix(backend): サーバーメトリクスのメモリ使用率が不正確になることがある不具合の修正 #10728

Merged
merged 2 commits into from
Apr 29, 2023

Conversation

nexryai
Copy link
Contributor

@nexryai nexryai commented Apr 28, 2023

What

#10295 の修正
メモリの使用量をused - buffers - cachedではなくtotal - availableで求めるように

Why

  • ディストーションによって正しく計測できていなかったため
    • たまに極端に高い値が表示され心臓に良くないため

Additional info (optional)

misskey.sda1.net に実際に適用したところ、UbuntuとCentOSの両方で正常に動作していることを確認しました。
Screenshot from 2023-04-28 09-40-01

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Apr 28, 2023
@github-actions github-actions bot requested review from rinsuki and syuilo April 28, 2023 01:06
@codecov
Copy link

codecov bot commented Apr 28, 2023

Codecov Report

Merging #10728 (6b3504b) into develop (5124db5) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop   #10728   +/-   ##
========================================
  Coverage    76.08%   76.08%           
========================================
  Files          898      898           
  Lines        88742    88742           
  Branches      6268     6264    -4     
========================================
  Hits         67517    67517           
  Misses       21225    21225           
Impacted Files Coverage Δ
packages/backend/src/daemons/ServerStatsService.ts 97.93% <100.00%> (ø)

@nexryai nexryai changed the title FIX: サーバーメトリクスのメモリ使用率が不正確になることがある不具合の修正 fix(backend): サーバーメトリクスのメモリ使用率が不正確になることがある不具合の修正 Apr 28, 2023
@syuilo
Copy link
Member

syuilo commented Apr 29, 2023

もともとシンプルにtotal - availableだったのを何か問題があってused - buffers - cachedに変更した経緯があったような気がするけどよく覚えてないわね

@acid-chicken
Copy link
Member

もともとシンプルにtotal - availableだったのを何か問題があってused - buffers - cachedに変更した経緯があったような気がするけどよく覚えてないわね

Related to #3437

@acid-chicken
Copy link
Member

ややこしいので整理すると、要するに問題があったのは active のことで、当時想定していたバージョンの free にまだ存在しなかった available が現在は有用ということかな

@syuilo syuilo merged commit 7de59a8 into misskey-dev:develop Apr 29, 2023
@syuilo
Copy link
Member

syuilo commented Apr 29, 2023

🙏

sasagar pushed a commit to sasagar/misskey that referenced this pull request May 9, 2023
* FIX: サーバーメトリクスのメモリ使用率が不正確になることがある不具合の修正

* Update CHANGELOG
na2na-p pushed a commit to na2na-p/misskey that referenced this pull request May 10, 2023
* FIX: サーバーメトリクスのメモリ使用率が不正確になることがある不具合の修正

* Update CHANGELOG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants