Skip to content

Comments

[202506] Align the deploy script for smartswitch dark mode#770

Open
JibinBao wants to merge 1 commit intoAzure:202506from
JibinBao:cp_pr_21029_dark_mode
Open

[202506] Align the deploy script for smartswitch dark mode#770
JibinBao wants to merge 1 commit intoAzure:202506from
JibinBao:cp_pr_21029_dark_mode

Conversation

@JibinBao
Copy link

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

@JibinBao
Copy link
Author

Hi @prabhataravind
Cherry-pick sonic-net/sonic-mgmt#20415, could you please review it?

@prabhataravind
Copy link

@JibinBao could you help fix conflicts?

* 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 $@.
@JibinBao JibinBao force-pushed the cp_pr_21029_dark_mode branch from dca9322 to 1421bc8 Compare November 19, 2025 05:11
@JibinBao
Copy link
Author

@JibinBao could you help fix conflicts?

hi @prabhataravind Done, please review it

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.

2 participants