Add openSUSE Tumbleweed/Leap support#1166
Merged
angristan merged 8 commits intoangristan:masterfrom Dec 11, 2025
Merged
Conversation
angristan
force-pushed
the
opensuse
branch
2 times, most recently
from
43b1f19 to
066d850
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds support for openSUSE Tumbleweed and openSUSE Leap 16+ distributions to the OpenVPN installer script. The changes enable the script to detect, install, configure, and remove OpenVPN on openSUSE systems using zypper as the package manager.
Key Changes
- Added OS detection logic for openSUSE Tumbleweed and Leap distributions with version validation
- Implemented openSUSE-specific package management commands (zypper) across install, uninstall, and configuration functions
- Extended test infrastructure to include openSUSE distributions in CI/CD pipeline and test suite
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| openvpn-install.sh | Added openSUSE OS detection in checkOS function with version check for Leap 16+; added zypper commands for installing/removing OpenVPN and Unbound; added openSUSE-specific Unbound remote-control configuration |
| test/Dockerfile.server | Added zypper package manager support with necessary dependencies for openSUSE containers |
| README.md | Updated compatibility table to include openSUSE Leap >= 16 and openSUSE Tumbleweed as supported distributions |
| Makefile | Added test-opensuse-leap and test-opensuse-tumbleweed targets for local testing |
| .github/workflows/docker-test.yml | Added openSUSE Leap 16.0 and Tumbleweed to the CI test matrix |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Add version check requiring openSUSE Leap >= 16 - Fix Unbound remote-control SSL cert issue (openSUSE-specific) - Add CI tests for openSUSE Leap 16.0 and Tumbleweed - Add Makefile test targets for openSUSE - Add zypper/gawk support to test Dockerfile - Update README compatibility table
The amzn2023 service name fix should be a separate PR.
The change from /etc/system-release to /etc/os-release (needed for openSUSE support) broke Arch Linux detection because Arch also has /etc/os-release. Now we check for ID="arch" inside that branch.
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.
No description provided.