Skip to content

revert(docker-ptf): restore 202411 compatibility#28609

Open
auspham wants to merge 1 commit into
sonic-net:202411from
auspham:austinpham/38870496-revert-docker-ptf
Open

revert(docker-ptf): restore 202411 compatibility#28609
auspham wants to merge 1 commit into
sonic-net:202411from
auspham:austinpham/38870496-revert-docker-ptf

Conversation

@auspham

@auspham auspham commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Why I did it

PR testing is no longer running for the 202411 branch. The docker-ptf changes introduced by #26326 break the established behavior required by older platform releases. Revert that portion of the change to preserve compatibility when those releases use docker-ptf.

Work item tracking
  • Microsoft ADO (number only): 38870496

How I did it

Reverted only the dockers/docker-ptf changes from commit f1b9ef048d3e991450276bc8146a472b452869c2. The changes made by that commit outside docker-ptf remain intact.

How to verify it

Compared dockers/docker-ptf with the parent of f1b9ef048d3e991450276bc8146a472b452869c2 and confirmed there are no semantic differences. Confirmed the PR changes no paths outside dockers/docker-ptf.

Which release branch to backport (provide reason below if selected)

None. This PR directly targets the 202411 branch.

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • 202411 - image build not run because docker-ptf PR testing is disabled on this branch

Description for the changelog

Restore the pre-#26326 docker-ptf behavior on 202411 for older platform releases.

Link to config_db schema for YANG module changes

N/A

A picture of a cute animal (not mandatory but encouraged)

N/A

Revert only the docker-ptf changes introduced by f1b9ef0. PR testing has been stopped for the 202411 branch, so restore the established behavior required by older platform releases.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7e980f91-752e-4a7a-a99c-377c5e75a089
Signed-off-by: Austin (Ngoc Thang) Pham <austinpham@microsoft.com>
@auspham
auspham requested a review from lguohan as a code owner July 24, 2026 00:03
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants