Skip to content

Comments

[202505] Align the deploy script for smartswitch dark mode#21029

Merged
roy-sror merged 1 commit intosonic-net:202505from
JibinBao:cp_dp_dark_202505
Oct 27, 2025
Merged

[202505] Align the deploy script for smartswitch dark mode#21029
roy-sror merged 1 commit intosonic-net:202505from
JibinBao:cp_dp_dark_202505

Conversation

@JibinBao
Copy link
Contributor

Description of PR

Align the deploy script for smartswitch dark mode. Because when smartswitch is dark mode, the dpus are shutdown, so we need to skip the config related to the dpus.

Summary:
Fixes # (issue)

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?

Align the deploy script with community for smartswitch dark mode

How did you do it?

When smartswitch is dark mode, skip the config related to dpu.

How did you verify/test it?

deploy smartswitch with dark mode

Any platform specific information?

Smartswitch

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

* Align the deploy script with community for smartswitch dark mode

* fix issue deploy_mg issue

We don't need to handle the parameters for the Ansible playbook individually; they can be passed to Ansible as a whole using $@.
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Contributor Author

Hi @prabhataravind ,
It is cherry-picked from #20415,
Could you please review it?

@JibinBao
Copy link
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@roy-sror roy-sror merged commit f7b9950 into sonic-net:202505 Oct 27, 2025
17 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