Skip to content

Fix to correct the parameters passed to this API get_macsec_counters#16978

Merged
judyjoseph merged 2 commits intosonic-net:masterfrom
judyjoseph:fix_clear_counter
Feb 17, 2025
Merged

Fix to correct the parameters passed to this API get_macsec_counters#16978
judyjoseph merged 2 commits intosonic-net:masterfrom
judyjoseph:fix_clear_counter

Conversation

@judyjoseph
Copy link
Copy Markdown
Contributor

Summary:
Two of the PR's passed different set of parameters to the API get_macsec_counters.

#15257
#15905

Fix is to use the latest change made as per #15257

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@judyjoseph judyjoseph requested a review from yxieca February 14, 2025 20:24
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@judyjoseph judyjoseph enabled auto-merge (squash) February 15, 2025 04:37
@judyjoseph judyjoseph merged commit 221012b into sonic-net:master Feb 17, 2025
17 checks passed
nnelluri-cisco pushed a commit to nnelluri-cisco/sonic-mgmt that referenced this pull request Mar 15, 2025
…onic-net#16978)

* Fix to correct the parameters passed to this API get_macsec_counters
arlakshm pushed a commit to Azure/sonic-mgmt.msft that referenced this pull request Jun 24, 2025
This PR is to cherry-pick macsec changes done in master to get the
macsec nightly pipeline run.

Below are the commits included.

sonic-net/sonic-mgmt#15905
sonic-net/sonic-mgmt#16048
sonic-net/sonic-mgmt#16978
sonic-net/sonic-mgmt#17087

---------

Co-authored-by: Tejaswini Chadaga <tchadaga@microsoft.com>
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.

4 participants