diff options
| author | syuilo <4439005+syuilo@users.noreply.github.com> | 2026-01-22 14:32:57 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2026-01-22 14:32:57 +0900 |
| commit | 2fa6ecc7efaaf9b9d189cdd3a3ebbb9171c86078 (patch) | |
| tree | 765b336c2f5b27e1df1f5ee1e14f4bb6a8e54684 /.github/workflows | |
| parent | enhance(dev): improve mem report (#17117) (diff) | |
| download | misskey-2fa6ecc7efaaf9b9d189cdd3a3ebbb9171c86078.tar.gz misskey-2fa6ecc7efaaf9b9d189cdd3a3ebbb9171c86078.tar.bz2 misskey-2fa6ecc7efaaf9b9d189cdd3a3ebbb9171c86078.zip | |
enhance(dev): improve mem report (#17118)
* wip
* wip
* Update report-backend-memory.yml
* Update report-backend-memory.yml
* Update .github/workflows/report-backend-memory.yml
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Diffstat (limited to '.github/workflows')
| -rw-r--r-- | .github/workflows/report-backend-memory.yml | 87 |
1 files changed, 61 insertions, 26 deletions
diff --git a/.github/workflows/report-backend-memory.yml b/.github/workflows/report-backend-memory.yml index 47ec652cfd..451a8cf9e6 100644 --- a/.github/workflows/report-backend-memory.yml +++ b/.github/workflows/report-backend-memory.yml @@ -54,31 +54,48 @@ jobs: BASE_MEMORY=$(cat ./artifacts/memory-base.json) HEAD_MEMORY=$(cat ./artifacts/memory-head.json) - calc() { - BASE=$(echo "$BASE_MEMORY" | jq -r '.memory.'"$1"' // 0') - HEAD=$(echo "$HEAD_MEMORY" | jq -r '.memory.'"$1"' // 0') + variation() { + calc() { + BASE=$(echo "$BASE_MEMORY" | jq -r ".${1}.${2} // 0") + HEAD=$(echo "$HEAD_MEMORY" | jq -r ".${1}.${2} // 0") - DIFF=$((HEAD - BASE)) - if [ "$BASE" -gt 0 ]; then - DIFF_PERCENT=$(echo "scale=2; ($DIFF * 100) / $BASE" | bc) - else - DIFF_PERCENT=0 - fi + DIFF=$((HEAD - BASE)) + if [ "$BASE" -gt 0 ]; then + DIFF_PERCENT=$(echo "scale=2; ($DIFF * 100) / $BASE" | bc) + else + DIFF_PERCENT=0 + fi - # Convert KB to MB for readability - BASE_MB=$(echo "scale=2; $BASE / 1024" | bc) - HEAD_MB=$(echo "scale=2; $HEAD / 1024" | bc) - DIFF_MB=$(echo "scale=2; $DIFF / 1024" | bc) + # Convert KB to MB for readability + BASE_MB=$(echo "scale=2; $BASE / 1024" | bc) + HEAD_MB=$(echo "scale=2; $HEAD / 1024" | bc) + DIFF_MB=$(echo "scale=2; $DIFF / 1024" | bc) - echo "$1-base=$BASE_MB" >> "$GITHUB_OUTPUT" - echo "$1-head=$HEAD_MB" >> "$GITHUB_OUTPUT" - echo "$1-diff=$DIFF_MB" >> "$GITHUB_OUTPUT" - echo "$1-diff_percent=$DIFF_PERCENT" >> "$GITHUB_OUTPUT" + JSON=$(jq -c -n \ + --arg base "$BASE_MB" \ + --arg head "$HEAD_MB" \ + --arg diff "$DIFF_MB" \ + --arg diff_percent "$DIFF_PERCENT" \ + '{base: $base, head: $head, diff: $diff, diff_percent: $diff_percent}') + + echo "$JSON" + } + + JSON=$(jq -c -n \ + --argjson VmRSS "$(calc $1 VmRSS)" \ + --argjson VmHWM "$(calc $1 VmHWM)" \ + --argjson VmSize "$(calc $1 VmSize)" \ + '{VmRSS: $VmRSS, VmHWM: $VmHWM, VmSize: $VmSize}') + + echo "$JSON" } - calc VmRSS - calc VmHWM - calc VmSize + JSON=$(jq -c -n \ + --argjson beforeGc "$(variation beforeGc)" \ + --argjson afterGc "$(variation afterGc)" \ + '{beforeGc: $beforeGc, afterGc: $afterGc}') + + echo "res=$JSON" >> "$GITHUB_OUTPUT" - id: build-comment name: Build memory comment run: | @@ -88,15 +105,33 @@ jobs: echo "$HEADER" > ./output.md echo >> ./output.md - echo "| Metric | base | head | Diff |" >> ./output.md - echo "|--------|------|------|------|" >> ./output.md - echo "| RSS | ${{ steps.compare.outputs.VmRSS-base }} MB | ${{ steps.compare.outputs.VmRSS-head }} MB | ${{ steps.compare.outputs.VmRSS-diff }} MB (${{ steps.compare.outputs.VmRSS-diff_percent }}%) |" >> ./output.md - echo "| HWM | ${{ steps.compare.outputs.VmHWM-base }} MB | ${{ steps.compare.outputs.VmHWM-head }} MB | ${{ steps.compare.outputs.VmHWM-diff }} MB (${{ steps.compare.outputs.VmHWM-diff_percent }}%) |" >> ./output.md - echo "| VMS | ${{ steps.compare.outputs.VmSize-base }} MB | ${{ steps.compare.outputs.VmSize-head }} MB | ${{ steps.compare.outputs.VmSize-diff }} MB (${{ steps.compare.outputs.VmSize-diff_percent }}%) |" >> ./output.md + table() { + line() { + BASE=$(echo "${{ steps.compare.outputs.res }}" | jq -r ".${1}.${2}.base") + HEAD=$(echo "${{ steps.compare.outputs.res }}" | jq -r ".${1}.${2}.head") + DIFF=$(echo "${{ steps.compare.outputs.res }}" | jq -r ".${1}.${2}.diff") + DIFF_PERCENT=$(echo "${{ steps.compare.outputs.res }}" | jq -r ".${1}.${2}.diff_percent") + + echo "| ${2} | ${BASE} MB | ${HEAD} MB | ${DIFF} MB (${DIFF_PERCENT}%) |" >> ./output.md + } + + echo "| Metric | base | head | Diff |" >> ./output.md + echo "|--------|------|------|------|" >> ./output.md + line $1 VmRSS + line $1 VmHWM + line $1 VmSize + } + + echo "### Before GC" >> ./output.md + table beforeGc + echo >> ./output.md + + echo "### After GC" >> ./output.md + table afterGc echo >> ./output.md # Determine if this is a significant change (more than 5% increase) - if [ "$(echo "${{ steps.compare.outputs.VmRSS-diff_percent }} > 5" | bc)" -eq 1 ]; then + if [ "$(echo "${{ steps.compare.outputs.res }}" | jq -r '.afterGc.VmRSS.diff_percent | tonumber > 5')" = "true" ]; then echo "⚠️ **Warning**: Memory usage has increased by more than 5%. Please verify this is not an unintended change." >> ./output.md echo >> ./output.md fi |