Skip to content

[feature] Added support for Ubuntu 26.04 #633#634

Merged
nemesifier merged 2 commits into
masterfrom
issues/633-ubuntu-26.04
Jul 2, 2026
Merged

[feature] Added support for Ubuntu 26.04 #633#634
nemesifier merged 2 commits into
masterfrom
issues/633-ubuntu-26.04

Conversation

@nemesifier

Copy link
Copy Markdown
Member

Checklist

  • I have read the OpenWISP Contributing Guidelines.
  • I have manually tested the changes proposed in this pull request.
  • I have written new test cases for new code and/or updated existing tests for changes to existing code.
  • I have updated the documentation.

Reference to Existing Issue

Closes #633

Description of Changes

Added support for Ubuntu 26.04.

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 56a58060-fd94-445f-8064-f00511cfc909

📥 Commits

Reviewing files that changed from the base of the PR and between cc5ecab and 281d141.

📒 Files selected for processing (1)
  • tasks/freeradius.yml
📜 Recent review details
⏰ Context from checks skipped due to timeout. (5)
  • GitHub Check: Build ubuntu2204
  • GitHub Check: Build ubuntu2404
  • GitHub Check: Build debian13
  • GitHub Check: Build debian12
  • GitHub Check: Build ubuntu2604
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{yml,yaml,j2}

📄 CodeRabbit inference engine (AGENTS.md)

Avoid unnecessary blank lines inside Jinja, YAML, and shell blocks.

Files:

  • tasks/freeradius.yml
**/*.{yml,yaml,j2,sh}

📄 CodeRabbit inference engine (AGENTS.md)

Write comments only when they explain why code is shaped a certain way. Put comments before the relevant block instead of scattering them inside it.

Files:

  • tasks/freeradius.yml
🔇 Additional comments (1)
tasks/freeradius.yml (1)

47-47: LGTM!

Also applies to: 62-75


📝 Walkthrough

Walkthrough

This pull request adds Ubuntu 26.04 (codename resolute) support to the ansible-openwisp2 role. Changes include a new vars/ubuntu-26.yml file with package overrides, a conditional include in tasks/main.yml, a new FreeRADIUS APT repository task for resolute in tasks/freeradius.yml, an added ubuntu2604 entry in the CI build matrix and molecule local platforms, a new Ubuntu version entry in meta/main.yml's galaxy_info.platforms, and documentation updates across README.md, docs/index.rst, docs/user/system-requirements.rst, docs/developer/installation.rst, and a whitespace normalization in docs/user/role-variables.rst.

Estimated code review effort: 2 (Simple) | ~15 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Ansible as Ansible Playbook
  participant TasksMain as tasks/main.yml
  participant VarsFile as vars/ubuntu-26.yml
  participant FreeRadiusTask as tasks/freeradius.yml

  Ansible->>TasksMain: run role tasks
  TasksMain->>TasksMain: check distribution == Ubuntu, release == resolute
  TasksMain->>VarsFile: include_vars ubuntu-26.yml
  VarsFile-->>TasksMain: package variable overrides
  Ansible->>FreeRadiusTask: run FreeRADIUS setup
  FreeRadiusTask->>FreeRadiusTask: check release == resolute
  FreeRadiusTask->>FreeRadiusTask: write networkradius.list APT source
Loading

Related issues: #633

Suggested labels: enhancement, documentation, ci

Suggested reviewers: nemesifier, jieter


Caution

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

  • Ignore

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Out of Scope Changes check ❌ Error Formatting-only edits in docs/user/role-variables.rst and tasks/freeradius.yml are unrelated to Ubuntu 26.04 support. Remove or split out the unrelated formatting-only changes so the PR stays focused on Ubuntu 26.04 support.
✅ Passed checks (7 passed)
Check name Status Explanation
Title check ✅ Passed The title is descriptive, uses an allowed type, and clearly matches the Ubuntu 26.04 support change.
Description check ✅ Passed It includes the required checklist, issue reference, and change summary; only the screenshot section is missing.
Linked Issues check ✅ Passed The CI, docs, Galaxy metadata, and Ubuntu 26.04-specific task/vars updates satisfy #633's support request.
Bug Fixes ✅ Passed N/A: this PR adds Ubuntu 26.04 support and docs/CI/metadata updates; it doesn’t fix a core bug, so regression-test requirements don’t apply.
Features ✅ Passed Issue #633 exists and is assigned/boarded to org member nemesifier; docs and CI/molecule updates add Ubuntu 26.04 support.
Changes ✅ Passed Docs/README/system requirements updated; CI/Molecule add Ubuntu 26.04 coverage, and no UI or public API changes were introduced.
General Rules ✅ Passed PASS: Issue #633 is properly triaged, and the PR consistently updates CI/docs/Molecule/vars/metadata for Ubuntu 26.04 with no obvious security, perf, or naming issues.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issues/633-ubuntu-26.04

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.

@nemesifier nemesifier moved this from Backlog to In progress in 26.06 Release Jul 2, 2026
@nemesifier
nemesifier merged commit 5ee8f12 into master Jul 2, 2026
7 checks passed
@nemesifier
nemesifier deleted the issues/633-ubuntu-26.04 branch July 2, 2026 20:18
@github-project-automation github-project-automation Bot moved this from In progress to Done in 26.06 Release Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

[feature] Add support for Ubuntu 26.04

1 participant