Skip to content

chore: remove proxy containaer#11039

Merged
loks0n merged 1 commit into1.8.xfrom
chore-remove-proxy-containaer
Dec 30, 2025
Merged

chore: remove proxy containaer#11039
loks0n merged 1 commit into1.8.xfrom
chore-remove-proxy-containaer

Conversation

@loks0n
Copy link
Copy Markdown
Member

@loks0n loks0n commented Dec 29, 2025

What does this PR do?

(Provide a description of what this PR does and why it's needed.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

  • (Related PR or issue)

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Dec 29, 2025 •
edited
Loading

📝 Walkthrough

Walkthrough

The openruntimes-proxy service definition was removed from the docker-compose.yml file, including its container configuration, network settings, environment variables, and logging configuration. The rest of the Docker Compose stack configuration remains unchanged. No alternative services were added, and no modifications were made to other service definitions.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description contains only the repository's contribution template with unfilled placeholders, providing no meaningful information about the changeset or its motivation. Fill in the description template with details about why the proxy container is being removed, how it was tested, and any related issues or PRs.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: remove proxy containaer' clearly describes the main change—removing the openruntimes-proxy service from docker-compose.yml, matching the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c4fcec0 and 7581019.

📒 Files selected for processing (1)
  • docker-compose.yml
💤 Files with no reviewable changes (1)
  • docker-compose.yml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: scan

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.

@loks0n loks0n requested a review from ChiragAgg5k December 29, 2025 18:53
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 29, 2025

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
libpng 1.6.51-r0 CVE-2025-66293 HIGH
libpng-dev 1.6.51-r0 CVE-2025-66293 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 29, 2025

✨ Benchmark results

  • Requests per second: 1,200
  • Requests with 200 status code: 216,077
  • P99 latency: 0.161993021

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,200 1,262
200 216,077 227,243
P99 0.161993021 0.162382238

@loks0n loks0n requested a review from Meldiron December 29, 2025 19:46
@loks0n loks0n merged commit 4046829 into 1.8.x Dec 30, 2025
72 of 73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChiragAgg5k ChiragAgg5k ChiragAgg5k approved these changes

@Meldiron Meldiron Awaiting requested review from Meldiron

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@loks0n @ChiragAgg5k