Repository navigation
Conversation
Contributor
|
✅ Requirements check passed. Thanks for fixing the description — the |
github-actions
Bot
added
requirements missing
Contributor
Try this script
COMMUNITY_SCRIPTS_URL=https://raw.githubusercontent.com/freakms/DevScripts/add-xwiki \
bash -c "$(curl -fsSL https://raw.githubusercontent.com/freakms/DevScripts/add-xwiki/ct/xwiki.sh)"
Against a core branch as wellAdd Useful flags while testing
|
github-actions
Bot
added
new script
and removed
requirements missing
Member
|
/test |
michelroegl-brunner
requested changes
Nov 11, 2026
michelroegl-brunner
left a comment
michelroegl-brunner
left a comment
Member
There was a problem hiding this comment.
Looks overall not to bad. We could have way less clutter in there if we just pick a database.
Why do we need to give the option? It should be sufficent if the app selects the best deafult. If a user wants to differ, they can either fork the install or install it on there own.
So please remove the DB switch stuff and pick one, and remove the inline comments.
Contributor
Proxmox script test@michelroegl-brunner @freakms ✅ 2/2 passed at
📸 ct/xwiki.sh (pve) — http://10.99.0.113:8080/xwiki: after install / after update
📸 ct/xwiki.sh (incus) — http://10.98.0.128:8080/xwiki: after install / after update
Not built:
📄 ct/xwiki.sh (pve) — install log (1585 lines)🔄 ct/xwiki.sh (pve) — update log (29 lines)📄 ct/xwiki.sh (incus) — install log (804 lines)🔄 ct/xwiki.sh (incus) — update log (24 lines)Full logs: workflow run (artifacts |
Author
|
Thanks for the review! Removed the DB switch (PostgreSQL only), removed the inline comments and switched to create_backup/restore_backup. The XWiki repo is now added via setup_deb822_repo. Re-tested install and update on PVE 9.2.26. |
michelroegl-brunner
approved these changes
Nov 12, 2026
michelroegl-brunner
left a comment
michelroegl-brunner
left a comment
Member
There was a problem hiding this comment.
Perfect. That looks way cleaner then before and is more readable.
Thank you for the changes.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✍️ Description
Adds XWiki, an open source enterprise wiki (Confluence alternative), installed from the official XWiki Debian repository (maven.xwiki.org, added via
setup_deb822_repo) with Tomcat 10, OpenJDK 21 from Debian 13 and PostgreSQL.Deviation from the standard pattern: XWiki is distributed as signed .deb packages (database created by the package via dbconfig-common), so install and
update_scriptuse apt instead offetch_and_deploy_gh_release; the update usescreate_backup/restore_backupfor /etc/xwiki.Tested on Proxmox VE 9.2.26: install, Distribution Wizard, update (existing pages kept), container reboot.
🔗 Related PR / Issue
Link: community-scripts/ProxmoxVE#12671
✅ Prerequisites (X in brackets)
🏗️ arm64 Support (X in brackets)
🖥️ Tested on (X in brackets)
🛠️ Type of Change (X in brackets)
README,AppName.md,CONTRIBUTING.md, or other docs.🔍 Code & Security Review (X in brackets)
CODE-AUDIT.md&CONTRIBUTING.mdguidelinesAppName.sh,AppName-install.sh,AppName.json)fetch_and_deploy_gh_release,fetch_and_deploy_codeberg_release,fetch_and_deploy_gl_release, orfetch_and_deploy_from_urlinstead ofgit pull.📋 Additional Information (optional)
No git pull is used; updates go through apt because XWiki ships official Debian packages (see Description). The XWiki APT repo is a flat repository, so its official .list file is used instead of
setup_deb822_repo. Happy to change this if a helper fits better.Release tarballs: XWiki publishes versioned official releases (currently 18.9.0) with download archives (WAR, Jetty distribution) on xwiki.org and tagged source tarballs on GitHub. The script intentionally uses the official signed Debian packages from maven.xwiki.org instead, as recommended by the XWiki installation guide.
📦 Application Requirements (for new scripts)
🌐 Source
🤖 AI Assistance (X in brackets)
AGENTS.mdand.github/agents/pve-script-creator.agent.mdas guidance, and the output has been reviewed and corrected to match those guidelines.Please describe to which degree, if any, an LLM was used in creating this pull request.
Claude (Anthropic) via the Claude app: Claude Sonnet 5.5 drafted the three files from my standalone XWiki installer, following AGENTS.md; Claude Opus 5.5 fixed the package filter in update_script after my test. Reasoning level: default. I tested everything on my Proxmox VE 9.2.26 host and fixed the issues found during testing.