Skip to content

[INFRA-351] fix: correct directory and command for space program in supervisor.conf#8838

Merged
sriramveeraghanta merged 1 commit intopreviewfrom
fix-aio-community
Mar 31, 2026
Merged

[INFRA-351] fix: correct directory and command for space program in supervisor.conf#8838
sriramveeraghanta merged 1 commit intopreviewfrom
fix-aio-community

Conversation

@akshat5302
Copy link
Copy Markdown
Member

@akshat5302 akshat5302 commented Mar 31, 2026 •
edited by coderabbitai Bot
Loading

Description

This pull request updates the supervisor configuration for the space program to ensure the correct working directory and server entry point are used during deployment.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Summary by CodeRabbit

  • Chores
    • Updated deployment configuration for the application startup process.

Copilot AI review requested due to automatic review settings March 31, 2026 13:16
@makeplane
Copy link
Copy Markdown

makeplane Bot commented Mar 31, 2026

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 31, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4c1b335e-053c-41f2-a3b3-0d1e933c9272

📥 Commits

Reviewing files that changed from the base of the PR and between 5747dc6 and fd8b651.

📒 Files selected for processing (1)
  • deployments/aio/community/supervisor.conf
📝 Walkthrough

Walkthrough

The supervisord configuration for the [program:space] entry was updated to reflect a directory structure change. The working directory was moved from a build subdirectory to the parent directory, and the startup command was adjusted to use an explicit relative path to the build artifact.

Changes

Cohort / File(s) Summary
Supervisord Configuration
deployments/aio/community/supervisor.conf		 Updated [program:space] working directory from /app/space/apps/space/build/server to /app/space/apps/space, and adjusted startup command to explicitly reference ./build/server/index.js instead of index.js.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 The space service takes flight,
From nested paths to one clean height,
Supervisord now knows the way,
To serve the build both night and day!
✨🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: correcting the directory and command for the space program in supervisor.conf, with proper context via the ticket reference.
Description check ✅ Passed The description covers the required sections with the key change explained and the type of change properly marked. However, Test Scenarios and References sections are incomplete.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-aio-community

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.

Copilot AI reviewed Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the AIO community supervisor.conf to run the space service from the correct working directory and with the correct React Router server entrypoint, aligning the supervisor start command with how the Space image is built/run.

Changes:

  • Change space program working directory to /app/space/apps/space.
  • Update space program command to serve ./build/server/index.js via react-router-serve.

@sriramveeraghanta sriramveeraghanta merged commit febf98e into preview Mar 31, 2026
25 checks passed
@sriramveeraghanta sriramveeraghanta deleted the fix-aio-community branch March 31, 2026 13:23
dadenegarco added a commit to dadenegarco/plane-fa that referenced this pull request May 8, 2026
@dadenegarco @claude
fix(infra): backport upstream Q1-Q2 2026 infra fixes
87eac90 
Three small upstream infra fixes — no conflict with our setup.

- nginx (web/admin/space): real_ip_header typo X-Forward-For
  → X-Forwarded-For. Without this, $remote_addr stays at the proxy
  IP so rate limiting and access logs see the proxy, not the actual
  client (makeplane#8935)
- space Dockerfile: healthcheck targets /spaces/ instead of /, and
  add curl to the runtime image so the healthcheck can run (makeplane#8674)
- aio supervisor.conf: correct space program directory and entry
  point — /app/space/apps/space + ./build/server/index.js (makeplane#8838)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@sriramveeraghanta sriramveeraghanta sriramveeraghanta approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@akshat5302 @sriramveeraghanta