8000 Revert "Fix: add missing sites bandwidth to project usage" by lohanidamodar · Pull Request #11397 · appwrite/appwrite · GitHub
[go: up one dir, main page]

Skip to content

Revert "Fix: add missing sites bandwidth to project usage"#11397

Merged
lohanidamodar merged 1 commit into1.8.xfrom
revert-11387-feat-update-project-usage
Feb 25, 2026
Merged

Revert "Fix: add missing sites bandwidth to project usage"#11397
lohanidamodar merged 1 commit into1.8.xfrom
revert-11387-feat-update-project-usage

Conversation

@lohanidamodar
Copy link
Member

Reverts #11387

@coderabbitai
Copy link
Contributor
coderabbitai bot commented Feb 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ac0aa81 and 244606f.

📒 Files selected for processing (1)
  • app/controllers/api/project.php
💤 Files with no reviewable changes (1)
  • app/controllers/api/project.php

📝 Walkthrough

Walkthrough

This change modifies the network bandwidth calculation logic in the project API controller by removing two metrics: METRIC_SITES_INBOUND and METRIC_SITES_OUTBOUND. Specifically, the metrics are deleted from the total metrics list and the period metrics array, along with the associated aggregation logic that previously contributed these metrics to the projectBandwidth computation. The net result is a reduction of 12 lines of code with no changes to exported or public entity declarations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: reverting a previous fix that added sites bandwidth to project usage metrics.
Description check ✅ Passed The description specifies which PR is being reverted (#11387), which directly relates to the changeset removing sites bandwidth metrics.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-11387-feat-update-project-usage

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
libecpg 18.1-r0 CVE-2026-2004 HIGH
libecpg 18.1-r0 CVE-2026-2005 HIGH
libecpg 18.1-r0 CVE-2026-2006 HIGH
libecpg 18.1-r0 CVE-2026-2007 HIGH
libecpg-dev 18.1-r0 CVE-2026-2004 HIGH
libecpg-dev 18.1-r0 CVE-2026-2005 HIGH
libecpg-dev 18.1-r0 CVE-2026-2006 HIGH
libecpg-dev 18.1-r0 CVE-2026-2007 HIGH
libpng 1.6.54-r0 CVE-2026-25646 HIGH
libpng-dev 1.6.54-r0 CVE-2026-25646 HIGH
libpq 18.1-r0 CVE-2026-2004 HIGH
libpq 18.1-r0 CVE-2026-2005 HIGH
libpq 18.1-r0 CVE-2026-2006 HIGH
libpq 18.1-r0 CVE-2026-2007 HIGH
libpq-dev 18.1-r0 CVE-2026-2004 HIGH
libpq-dev 18.1-r0 CVE-2026-2005 HIGH
libpq-dev 18.1-r0 CVE-2026-2006 HIGH
libpq-dev 18.1-r0 CVE-2026-2007 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2004 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2005 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2006 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2007 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

@github-actions
Copy link
github-actions bot commented Feb 25, 2026

🔄 PHP-Retry Summary

Flaky tests detected across commits:

Commit 244606f - 6 flaky tests
Test Retries Total Time Details
UsageTest::testDatabaseStatsCollectionsAPI 1 10.18s Logs
LegacyConsoleClientTest::testOperators 1 240.67s Logs
LegacyCustomClientTest::testOneToManyRelationship 1 240.86s Logs
TablesDBCustomServerTest::testCreateAttributes 1 242.32s Logs
TablesDBCustomServerTest::testInvalidDocumentStructure 1 240.67s Logs
TablesDBTransactionsConsoleClientTest::testIncrementDecrementEndpointsWithTransaction 1 240.79s Logs
< 8000 include-fragment loading="lazy" src="/appwrite/appwrite/issue_comments/3957323168/edit_form?textarea_id=issuecomment-3957323168-body&comment_context=" data-nonce="v2:e1fca5d2-bb73-1785-3edf-29a544e64daf" data-view-component="true" class="previewable-comment-form js-comment-edit-form-deferred-include-fragment">

@github-actions
Copy link

✨ Benchmark results

  • Requests per second: 1,948
  • Requests with 200 status code: 350,757
  • P99 latency: 0.08175517

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,948 1,190
200 350,757 214,178
P99 0.08175517 0.175119146

@lohanidamodar lohanidamodar merged commit b4a32c8 into 1.8.x Feb 25, 2026
154 of 155 checks passed
@lohanidamodar lohanidamodar deleted the revert-11387-feat-update-project-usage branch February 25, 2026 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0