Skip to content

[docs] Updated install instructions for openwrt 25.12 (apk)#274

Merged
nemesifier merged 1 commit intomasterfrom
update-openwrt-install-instructions
Apr 10, 2026
Merged

[docs] Updated install instructions for openwrt 25.12 (apk)#274
nemesifier merged 1 commit intomasterfrom
update-openwrt-install-instructions

Conversation

@nemesifier
Copy link
Copy Markdown
Member

Updated install instructions for openwrt 25.12 (apk).

@nemesifier nemesifier self-assigned this Apr 10, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 10, 2026

Warning

Rate limit exceeded

@nemesifier has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 23 minutes and 55 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 23 minutes and 55 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9e036220-b66c-4633-9a51-150bf516a8f5

📥 Commits

Reviewing files that changed from the base of the PR and between 7afbaf3 and 86a87c6.

📒 Files selected for processing (1)
  • user/configure-device.rst
📝 Walkthrough

Walkthrough

The documentation file for OpenWISP device configuration was updated to change the installation method for OpenWISP agents. The instructions previously required manually downloading and installing .ipk packages using wget and opkg install commands. The new instructions install packages directly from OpenWrt repositories using apk for OpenWrt versions 25.12 and later, and opkg for versions 24.10 and earlier. Notes regarding wget SSL issues and related workarounds were removed, and a reference to downloads.openwisp.io for obtaining latest builds was added.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: updating OpenWrt installation instructions to use the new apk package manager for version 25.12, which directly matches the changeset.
Description check ✅ Passed The description is directly related to the changeset, describing the updated installation instructions for OpenWrt 25.12, though it provides minimal detail.
Bug Fixes ✅ Passed This PR adds documentation for OpenWrt 25.12 support, not a bug fix in core user-facing functionality, so the conditional check does not apply.

✏️ 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 update-openwrt-install-instructions

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.

@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented Apr 10, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The PR correctly updates OpenWrt installation instructions:

  • Adds apk package manager commands for OpenWrt 25.12+
  • Retains opkg commands for OpenWrt 24.10 and earlier
  • Fixes grammatical issue in note text
  • Changes HTTP to HTTPS for downloads.openwisp.io (security improvement)

The simplified package manager approach is much cleaner than the previous manual download method.

Files Reviewed (1 file)
  • user/configure-device.rst - Documentation update with version-specific installation instructions

Reviewed by kimi-k2.5-0127 · 122,730 tokens

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@user/configure-device.rst`:
- Around line 61-63: The sentence "To use the latest features available, we
recommend downloading and installing." is incomplete; update the note text in
configure-device.rst to complete the instruction by replacing that fragment with
a full sentence such as "To use the latest features available, we recommend
downloading and installing the latest builds from downloads.openwisp.io." Ensure
the updated text keeps the existing link markup (`downloads.openwisp.io
<http://downloads.openwisp.io/>`__) and reads as a complete actionable sentence.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: f2d9e093-5038-4aee-8b87-bd6485577179

📥 Commits

Reviewing files that changed from the base of the PR and between 093bd99 and 9f8439a.

📒 Files selected for processing (1)
  • user/configure-device.rst
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: build
  • GitHub Check: Kilo Code Review
🔇 Additional comments (1)
user/configure-device.rst (1)

51-57: No review comment was provided. Please share the review comment that needs to be rewritten within <review_comment> tags, along with any relevant context about the codebase and changes being reviewed.

Comment thread user/configure-device.rst Outdated
@nemesifier nemesifier force-pushed the update-openwrt-install-instructions branch from 9f8439a to 7afbaf3 Compare April 10, 2026 23:30
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

♻️ Duplicate comments (1)
user/configure-device.rst (1)

61-63: ⚠️ Potential issue | 🟡 Minor

Use HTTPS for downloads.openwisp.io link.

The note is now complete, but the URL still uses http://; please switch to https:// for secure and consistent docs linking.

✏️ Proposed fix
-    installing the latest builds published on `downloads.openwisp.io
-    <http://downloads.openwisp.io/>`__.
+    installing the latest builds published on `downloads.openwisp.io
+    <https://downloads.openwisp.io/>`__.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@user/configure-device.rst` around lines 61 - 63, Update the link in the
configure-device.rst sentence that currently uses
`http://downloads.openwisp.io/` to use HTTPS: replace
`http://downloads.openwisp.io/` with `https://downloads.openwisp.io/` in the
line that reads "installing the latest builds published on
`downloads.openwisp.io <http://downloads.openwisp.io/>`__." to ensure the
documentation uses a secure URL.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Duplicate comments:
In `@user/configure-device.rst`:
- Around line 61-63: Update the link in the configure-device.rst sentence that
currently uses `http://downloads.openwisp.io/` to use HTTPS: replace
`http://downloads.openwisp.io/` with `https://downloads.openwisp.io/` in the
line that reads "installing the latest builds published on
`downloads.openwisp.io <http://downloads.openwisp.io/>`__." to ensure the
documentation uses a secure URL.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: fb7c1790-ab75-4186-b15a-82239a2e6409

📥 Commits

Reviewing files that changed from the base of the PR and between 9f8439a and 7afbaf3.

📒 Files selected for processing (1)
  • user/configure-device.rst
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: build
  • GitHub Check: Kilo Code Review
🔇 Additional comments (2)
user/configure-device.rst (2)

47-47: Clear intro line for the new install flow.

This wording cleanly introduces the command split by OpenWrt version and is easier to follow than the prior package-download sequence.


51-57: The version-to-package-manager mapping is accurate per official OpenWrt documentation.

OpenWrt 25.12 and later use apk for package management (confirmed by official release announcements), and OpenWrt 24.10 and earlier use opkg (confirmed by release documentation). There are no intermediate stable versions between 24.10.x and 25.12.0, so the thresholds (>= 25.12 and <= 24.10) have no gaps or ambiguities. The installation instructions correctly route users to the appropriate package manager for their release.

coderabbitai[bot]
coderabbitai Bot previously approved these changes Apr 10, 2026
@nemesifier nemesifier force-pushed the update-openwrt-install-instructions branch from 7afbaf3 to 86a87c6 Compare April 10, 2026 23:33
@nemesifier nemesifier merged commit 295ebe1 into master Apr 10, 2026
4 checks passed
@nemesifier nemesifier deleted the update-openwrt-install-instructions branch April 10, 2026 23:53
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