Conversation
Update only the DHCP relay and monitor gitlinks to the final merged 202411 source commits for the Fungible F2 delivery. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: c23940f6-b749-4479-84f2-c094d432a4e0 Signed-off-by: Xichen Lin <lukelin0907@gmail.com>
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Collaborator
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
yijingyan2
pushed a commit
to Azure/sonic-mgmt.msft
that referenced
this pull request
Sep 10, 2026
## Summary - use `uv` for the Elastictest trigger that runs on `sonic-ubuntu-1c` - move the three Pre-test jobs to clean Microsoft-hosted `ubuntu-latest` agents ## Why The self-hosted `sonic-ubuntu-1c` agents no longer provide `pip`, causing every Elastictest trigger in Azure/sonic-buildimage-msft#3073 to fail. Their persistent workspaces can also contain root-owned pytest artifacts, causing checkout cleanup failures. This backports the relevant approach from sonic-net/sonic-mgmt#25721 to the 202412 pipeline. --------- Signed-off-by: Lun Yue <17232861+lunyue-ms@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
5 tasks
Lun Yue (lunyue-ms)
added a commit
to Azure/sonic-mgmt.msft
that referenced
this pull request
Sep 11, 2026
### Description of PR Backport sonic-net/sonic-mgmt#19258 to update legacy Ansible module shebangs from `#!/usr/bin/env python` to `#!/usr/bin/python`. Recent Elastictest runs for Azure/sonic-buildimage-msft#3073 use `ansible-core 2.20.9` and fail during `Read duts minigraph` because Ansible executes `/usr/bin/env python` literally, while the current sonic-mgmt container has no `python` alias. The supported `/usr/bin/python` shebang allows Ansible interpreter discovery to select the available Python 3 interpreter. `testbed_vm_info.py` already had the corrected shebang on this branch, so this backport updates the remaining 23 modules. ### Type of change - [x] Bug fix - [ ] Testbed and Framework (new/improvement) - [ ] New test case - [ ] Test case improvement ### Back port request - [x] 202503 ### Validation - Reproduced the Elastictest failure with `ansible-core 2.20.9` and an `#!/usr/bin/env python` custom module. - Confirmed that changing the shebang to `#!/usr/bin/python` enables interpreter discovery and succeeds with the available Python 3 interpreter. - Confirmed the PR contains only the upstream shebang changes. Signed-off-by: Lun Yue <17232861+lunyue-ms@users.noreply.github.com> Co-authored-by: Yutong Zhang <90831468+yutongzhang-microsoft@users.noreply.github.com>
Contributor
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
4 tasks
Dashuai Zhang (sdszhang)
marked this pull request as ready for review
September 18, 2026 05:10
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Contributor
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
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
5 participants
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.
Why I did it
Deliver the three merged Fungible F2 DHCP fixes from public
202411into Microsoft SONiC202503: physical-member DHCPv6 relay ingress, VLAN-backed PortChannel monitor attribution, and prevention of pre-VLAN false relay-disparity alerts.This is a manual, submodule-only sync attempt. All source changes have already merged:
Work item tracking
How I did it
Update exactly two gitlinks on
202503:src/dhcprelay13d7f5aafea8418512c3258807ab9e920bc0e40fbe8671589c4eccca384de06ed28196dc5b94fd78src/dhcpmon2519a88abd7f244d48026e8fee31ff1e7a1feb5cee138055e160cdfc99b3aa8b4bb2a72b97ff3991The new dhcpmon commit includes both source PRs. No other file,
.gitmodulesdeclaration, package metadata, dependency, CLI/service configuration, counter schema, or event contract is changed here.How to verify it
061711f362345c4715fec5c2a480530da4c24b39; only the gitlink commit identities differ from that candidate.Which release branch to backport (provide reason below if selected)
None requested. This PR directly targets Microsoft
202503.Tested branch (Please provide the tested image version)
202503candidate:validate-f2-dhcp-202503-rx-dedupe-1828720a.1214622-5d1a77a10, with identical component source trees; see the linked evidence and its limitations.Description for the changelog
Sync merged F2 DHCP relay and monitor fixes into Microsoft SONiC 202503.
Link to config_db schema for YANG module changes
Not applicable; this PR updates gitlinks only.
A picture of a cute animal (not mandatory but encouraged)
Not included.