Skip to content

Fix waagent-network-setup.service failure and update to teardown the specific interface.#17626

Open
mayankfz wants to merge 1 commit into
3.0-devfrom
mayansingh/waagent_update
Open

Fix waagent-network-setup.service failure and update to teardown the specific interface.#17626
mayankfz wants to merge 1 commit into
3.0-devfrom
mayansingh/waagent_update

Conversation

@mayankfz
Copy link
Copy Markdown

@mayankfz mayankfz commented Jun 4, 2026

Fix waagent-network-setup.service failure and update to teardown the specific interface.

Buddy Build - https://dev.azure.com/mariner-org/mariner/_build/results?buildId=1132698&view=results
upstream PR merged - Azure/WALinuxAgent#3608

@mayankfz mayankfz requested a review from a team as a code owner June 4, 2026 09:47
@microsoft-github-policy-service microsoft-github-policy-service Bot added Packaging 3.0-dev PRs Destined for AzureLinux 3.0 labels Jun 4, 2026
@mayankfz mayankfz force-pushed the mayansingh/waagent_update branch from 8c97055 to 5990e29 Compare June 4, 2026 10:37
…specific interface.

Signed-off-by: Mayank Singh <mayansingh@microsoft.com>
@mayankfz mayankfz force-pushed the mayansingh/waagent_update branch from 5990e29 to 23d5507 Compare June 4, 2026 17:57
@jiria
Copy link
Copy Markdown
Member

jiria commented Jun 5, 2026

The acl-scripts aclmain spec uses --lnx-distro='azurecontainerlinux' in %install, which triggers the ACL-specific config/service file installation paths in setup.py. This spec still uses --lnx-distro='mariner'. Just want to confirm the patch code paths get exercised during the ACL build.

@jiria
Copy link
Copy Markdown
Member

jiria commented Jun 5, 2026

Nit: the description mentions "update to teardown the specific interface" — could you expand slightly on what changed? e.g. "
estart_if() now bounces the specific interface via ip link set down/up instead of restarting all of systemd-networkd, and persist_firewall_rules.py now conditionally adds After=systemd-sysext.service only on ACL." Helps reviewers unfamiliar with the change understand the scope quickly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.0-dev PRs Destined for AzureLinux 3.0 Packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants