Skip to content

[ACL] Record service ports individually and add them to the acl_table_ports#19080

Merged
StormLiangMS merged 2 commits intosonic-net:masterfrom
Gfrom2016:fix_acl
Jun 24, 2025
Merged

[ACL] Record service ports individually and add them to the acl_table_ports#19080
StormLiangMS merged 2 commits intosonic-net:masterfrom
Gfrom2016:fix_acl

Conversation

@Gfrom2016
Copy link
Copy Markdown
Contributor

@Gfrom2016 Gfrom2016 commented Jun 18, 2025

ports

Description of PR

Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

https://elastictest.org/scheduler/testplan/6852a36a7f408d23fa4bdbac

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@sdszhang sdszhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm.

Signed-off-by: ziitngguo-ms <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit 183a4a6 into sonic-net:master Jun 24, 2025
13 checks passed
@mssonicbld
Copy link
Copy Markdown
Collaborator

@Gfrom2016 PR conflicts with 202411 branch

StormLiangMS pushed a commit that referenced this pull request Jun 25, 2025
…_ports (#19168)

This is a manual cherry-pick of #19080

Signed-off-by: zitingguo-ms [email protected]
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jun 25, 2025
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202505: #19183

mssonicbld pushed a commit that referenced this pull request Jun 25, 2025
…_ports (#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Aug 7, 2025
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports

Signed-off-by: opcoder0 <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports

Signed-off-by: Guy Shemesh <[email protected]>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports

Signed-off-by: Aharon Malkin <[email protected]>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports

Signed-off-by: Guy Shemesh <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports

Signed-off-by: Guy Shemesh <[email protected]>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports

Signed-off-by: Yael Tzur <[email protected]>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
…_ports (sonic-net#19080)

Description of PR
Summary:
Fixes # (issue)
Record service ports individually and add them to the acl_table_ports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants