-
-
Notifications
You must be signed in to change notification settings - Fork 182
Add TrueNAS VM #1294
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add TrueNAS VM #1294
Conversation
Greptile SummaryThis PR adds a TrueNAS Community Edition VM installation script with automatic ISO fetching and optional disk import functionality. The implementation is well-structured with proper error handling and user interaction flows. Key Changes:
Issues Found:
Confidence Score: 4/5
Important Files Changed
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 files reviewed, 7 comments
| # Scrapes the TrueNAS download portal for ISO paths from the current and previous year, | ||
| # filtering out nightlies/alphas and returning the latest stable releases for each major | ||
| # version along with any beta or RC pre-releases. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Remove comments from shell script body. Comments only allowed in file headers.
| # Scrapes the TrueNAS download portal for ISO paths from the current and previous year, | |
| # filtering out nightlies/alphas and returning the latest stable releases for each major | |
| # version along with any beta or RC pre-releases. | |
| function truenas_iso_lookup() { |
Context Used: Rule from dashboard - What: Shell script files in ct/ and /install directories must not contain comments except for file h... (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| # This function checks the version of Proxmox Virtual Environment (PVE) and exits if the version is not supported. | ||
| # Supported: Proxmox VE 8.0.x – 8.9.x and 9.0 – 9.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Remove comments from shell script body.
| # This function checks the version of Proxmox Virtual Environment (PVE) and exits if the version is not supported. | |
| # Supported: Proxmox VE 8.0.x – 8.9.x and 9.0 – 9.1 | |
| pve_check() { |
Context Used: Rule from dashboard - What: Shell script files in ct/ and /install directories must not contain comments except for file h... (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| fi | ||
| done | ||
|
|
||
| # Fetching iso list from TrueNAS downloads for whiptail radiolist |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Remove comment.
| # Fetching iso list from TrueNAS downloads for whiptail radiolist | |
| ISOARRAY=() |
Context Used: Rule from dashboard - What: Shell script files in ct/ and /install directories must not contain comments except for file h... (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| if [ -z "${SELECTED_ISO:-}" ]; then | ||
| # Fallback: Find the latest stable release only (excluding RC/BETA for safety) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Remove comment.
| if [ -z "${SELECTED_ISO:-}" ]; then | |
| # Fallback: Find the latest stable release only (excluding RC/BETA for safety) | |
| if [ -z "${SELECTED_ISO:-}" ]; then |
Context Used: Rule from dashboard - What: Shell script files in ct/ and /install directories must not contain comments except for file h... (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| -scsihw virtio-scsi-single -cdrom local:iso/$ISO_NAME -vga virtio >/dev/null | ||
| msg_ok "Created VM shell" | ||
|
|
||
| # Optional step to import onboard disks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: Remove comment.
| # Optional step to import onboard disks | |
| if [ "$IMPORT_DISKS" == "yes" ]; then |
Context Used: Rule from dashboard - What: Shell script files in ct/ and /install directories must not contain comments except for file h... (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
|
I intentionally added the inline comments for reviewing purposes. I can gladly remove them. |
Scripts wich are clearly AI generated and not further revied by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review.
✍️ Description
This PR adds a TrueNAS Community Edition VM script and json for frontend. TrueNAS CE (Formerly TrueNAS SCALE) is a popular open source NAS software built on Debian (CORE versions are discontinued).
The script prepares the VM with the installation ISO image, which is automatically fetched from the official TrueNAS downloads URL. The user also has the option to import onboard disks.
After running the script, the user has to proceed with installing TrueNas via VM console.
🔗 Related PR / Issue
Link: #
✅ Prerequisites (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)📋 Additional Information (optional)
📦 Application Requirements (for new scripts)