Skip to content

Addition of prober_type field for MUX_CABLE#22633

Merged
kperumalbfn merged 1 commit intosonic-net:masterfrom
harjotsinghpawra:harjosin_icmp
May 27, 2025
Merged

Addition of prober_type field for MUX_CABLE#22633
kperumalbfn merged 1 commit intosonic-net:masterfrom
harjotsinghpawra:harjosin_icmp

Conversation

@harjotsinghpawra
Copy link
Contributor

Why I did it

New prober_type harware is being added to linkmgrd. This is config knob yang addition for the specific field in mUX_CABLE

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 15, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: harjotsinghpawra (6a4a31f)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@harjotsinghpawra
Copy link
Contributor Author

@qiluo-msft Hi, I think these errors are not related to my changes , is there some issues going on with current master

temp.patch Outdated
Copy link
Contributor

@zjswhhh zjswhhh May 22, 2025

Choose a reason for hiding this comment

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

This patch file needs to be fixed.

Copy link
Contributor

Choose a reason for hiding this comment

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

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@zjswhhh zjswhhh left a comment

Choose a reason for hiding this comment

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

lgtm

@zjswhhh
Copy link
Contributor

zjswhhh commented May 26, 2025

Hi @kperumalbfn - please help merge.

@kperumalbfn kperumalbfn merged commit d2e193d into sonic-net:master May 27, 2025
19 checks passed
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #23954

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