Skip to content

fix: temporarily disable sai validation#23346

Merged
yejianquan merged 1 commit intosonic-net:masterfrom
cyw233:temp-disable-sai-validation
Jul 16, 2025
Merged

fix: temporarily disable sai validation#23346
yejianquan merged 1 commit intosonic-net:masterfrom
cyw233:temp-disable-sai-validation

Conversation

@cyw233
Copy link
Copy Markdown
Contributor

@cyw233 cyw233 commented Jul 15, 2025

Why I did it

Temporarily disable SAI validation for now as it is not compatible with the change in sonic-net/sonic-mgmt#19263 due to the usage of concurrent.futures. We will refactor the SAI validation and re-enable it later. Microsoft ADO to track the progress: 33758029.

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

How I did it

How to verify it

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

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

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)

@cyw233 cyw233 requested a review from lguohan as a code owner July 15, 2025 22:39
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yejianquan yejianquan merged commit 5174dbc into sonic-net:master Jul 16, 2025
20 checks passed
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.

4 participants