From 6d9b1b9df3b89369da1073ba63daef73e0ddc1bf Mon Sep 17 00:00:00 2001 From: Dan Guido Date: Sun, 3 Aug 2025 04:56:06 -0400 Subject: [PATCH] fix: Correct Azure requirements file path to resolve deployment failures (#14781) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * fix: Add IPv6 support for WireGuard endpoint addresses Fixes issue where IPv6 addresses in WireGuard configuration files were not properly formatted with square brackets when used with port numbers. The WireGuard client configuration template now detects IPv6 addresses using the ansible.utils.ipv6 filter and wraps them in brackets as required by the WireGuard configuration format. Example outputs: - IPv4: 192.168.1.1:51820 - IPv6: [2600:3c01::f03c:91ff:fedf:3b2a]:51820 - Hostname: vpn.example.com:51820 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude * fix: Correct Azure requirements file path to fix deployment failures The previous fix in commit 7acdca0 updated to Azure collection v3.7.0 but referenced the incorrect requirements file name. The file is now called requirements.txt instead of requirements-azure.txt in v3.7.0. This fixes the Azure deployment failure where pip cannot find the requirements file, preventing users from deploying VPN servers on Azure. Also added no_log: true to prevent potential credential leakage during the pip installation process. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude * fix: Use simple colon check for IPv6 detection in WireGuard template The previous implementation used `ansible.utils.ipv6` filter which is not available in the current environment, causing the Smart Test Selection workflow to fail with "No filter named 'ansible.utils.ipv6' found." This change replaces the filter with a simple string check for colons (':') which is a reliable way to detect IPv6 addresses since they contain colons while IPv4 addresses and hostnames typically don't. The fix maintains the same functionality: - IPv6 addresses: `[2600:3c01::f03c:91ff:fedf:3b2a]:51820` - IPv4 addresses: `192.168.1.1:51820` - Hostnames: `vpn.example.com:51820` This resolves the failing workflow tests in PR #14781. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude * fix: Remove unrelated changes and clean up Python cache - Remove WireGuard IPv6 change (belongs in separate PR) - Delete committed Python cache file - Add __pycache__/ and *.pyc to .gitignore This PR should only contain the Azure requirements file path fix. --------- Co-authored-by: Claude --- .gitignore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.gitignore b/.gitignore index 018ca1f..78e3df0 100644 --- a/.gitignore +++ b/.gitignore @@ -9,3 +9,5 @@ venvs/* !venvs/.gitinit .vagrant .ansible/ +__pycache__/ +*.pyc