-
Notifications
You must be signed in to change notification settings - Fork 37
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
Update 7.1.0 release notes #245
Conversation
WalkthroughThe release notes for Unraid OS version 7.1.0 have been updated with new documentation. A section on known VM snapshot issues has been added, noting that the "please wait" indicator might persist after a snapshot is created and advising a one-minute wait before refreshing. Additionally, a subsection now outlines wireless networking limitations. This details that only a single wireless NIC (wlan0) is supported (which cannot be bonded) and provides specific configuration instructions for Docker (using ipvlan network type with restricted host access) and VMs (requiring manual static IP setup and ipvtap configuration). No public entity declarations were modified. Changes
Poem
Tip ⚡🧪 Multi-step agentic review comment chat (experimental)
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (3)
docs/unraid-os/release-notes/7.1.0.md (3)
11-15
: VM Snapshots Section Addition Looks Good
The newly added "#### VM Snapshots" section clearly explains the persistent "please wait" issue following snapshot creation. Consider, if available, adding a reference or link to troubleshooting documentation for users who might need additional guidance.
54-66
: Wireless Networking Limitations Section is Comprehensive
The additional subsection covering wireless limitations is detailed and informative. It explains the restrictions on the wireless NIC (wlan0)—including its MAC address limitation, Docker container requirements, and VM configuration notes—clearly. Ensure that this language is consistent with other related documentation and that the technical instructions (like configuring the ipvtap interface) have been verified with actual behavior.🧰 Tools
🪛 LanguageTool
[duplication] ~56-~56: Possible typo: you repeated a word.
Context: ... * Only one wireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * ...(ENGLISH_WORD_REPEAT_RULE)
[style] ~57-~57: As a shorter alternative for ‘able to’, consider using “can not”.
Context: ...ireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * Docker containe...(BE_ABLE_TO)
56-57
: Refine Wording for Wireless NIC Restrictions
There is a minor wording improvement suggested here:
- Instead of "* Only one wireless NIC is supported, wlan0", consider using "* Only a single wireless NIC (wlan0) is supported" for clarity and to avoid a potential duplicate mention.
- Also, change "* wlan0 is not able to participate in a bond" to "* wlan0 cannot participate in a bond" for brevity.
Below is a diff suggestion for these lines:
-* Only one wireless NIC is supported, wlan0 +* Only a single wireless NIC (wlan0) is supported -* wlan0 is not able to participate in a bond +* wlan0 cannot participate in a bond🧰 Tools
🪛 LanguageTool
[duplication] ~56-~56: Possible typo: you repeated a word.
Context: ... * Only one wireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * ...(ENGLISH_WORD_REPEAT_RULE)
[style] ~57-~57: As a shorter alternative for ‘able to’, consider using “can not”.
Context: ...ireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * Docker containe...(BE_ABLE_TO)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/unraid-os/release-notes/7.1.0.md
(2 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/unraid-os/release-notes/7.1.0.md
[duplication] ~56-~56: Possible typo: you repeated a word.
Context: ... * Only one wireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * ...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~57-~57: As a shorter alternative for ‘able to’, consider using “can not”.
Context: ...ireless NIC is supported, wlan0 * wlan0 is not able to participate in a bond * Docker containe...
(BE_ABLE_TO)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: build
- GitHub Check: AWS Amplify Console Web Preview
🔇 Additional comments (1)
docs/unraid-os/release-notes/7.1.0.md (1)
21-21
: Updated Comparison Header is Correct
The header now correctly compares changes versus version 7.0.1. Please verify that all internal cross-references and links in this context have been updated consistently.
This pull request is automatically being deployed by Amplify Hosting (learn more). |
Before Submitting This PR, Please Ensure You Have Completed The Following:
Summary by CodeRabbit