Skip to content

stack: drop env-driven filehost, switch panel to ValwareIRC webpanel#5

Merged
ValwareIRC merged 1 commit into
mainfrom
cleanup-filehost-env-and-panel-image
Jun 8, 2026
Merged

stack: drop env-driven filehost, switch panel to ValwareIRC webpanel#5
ValwareIRC merged 1 commit into
mainfrom
cleanup-filehost-env-and-panel-image

Conversation

@ValwareIRC

@ValwareIRC ValwareIRC commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

FILEHOST_URL was a vestige of the obbyircd container's pre-runtime
filehost render. obbyircd now expects an operator-edited filehosts {}
block in obbyircd.conf, so the env var has no consumer. Dropped from
.env.example and replaced the comment with a pointer to the conf block.

Panel image: obbyworld/webpanel:0.1.0 was the legacy panel. Repointed at
ghcr.io/valwareirc/unrealircd-webpanel-2:latest (modern Go+React rewrite,
multi-arch). Source: ValwareIRC/unrealircd-webpanel-2.

Summary by CodeRabbit

  • Documentation

    • Updated environment configuration examples with the latest web panel image reference.
    • Clarified that filehost settings are configured at runtime rather than through environment variables.
  • Chores

    • Updated default container image source and version for the web panel service.

…d-webpanel-2

FILEHOST_URL was a vestige of the obbyircd container's pre-runtime
filehost render. obbyircd now expects an operator-edited filehosts {}
block in obbyircd.conf, so the env var has no consumer. Dropped from
.env.example and replaced the comment with a pointer to the conf block.

Panel image: obbyworld/webpanel:0.1.0 was the legacy panel. Repointed at
ghcr.io/valwareirc/unrealircd-webpanel-2:latest (modern Go+React rewrite,
multi-arch). Source: ValwareIRC/unrealircd-webpanel-2.
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR updates container image references and environment variable documentation. The FILEHOST_URL environment variable is removed from .env.example with guidance that filehosts are configured at runtime. The obby-panel service image reference is updated from obbyworld/webpanel to ghcr.io/valwareirc/unrealircd-webpanel-2 across both configuration files.

Changes

Configuration and Image Reference Update

Layer / File(s) Summary
Image references and environment variable updates
.env.example, compose.yaml
FILEHOST_URL removed with documentation indicating runtime configuration via obbyircd.conf; OBBY_PANEL_IMAGE example updated to ghcr.io/valwareirc/unrealircd-webpanel-2:latest in both .env.example and compose.yaml service definition with renovate metadata updated.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A registry switch, clean and bright,
From obbyworld to the ghcr light,
Filehosts find their rightful place,
In config files, setting the pace! 🔧

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the two main changes: removing the env-driven filehost configuration and switching the panel image to ValwareIRC's webpanel.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 cleanup-filehost-env-and-panel-image

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.

@ValwareIRC ValwareIRC merged commit b5c2362 into main Jun 8, 2026
3 of 4 checks passed
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.

1 participant