Skip to content

[slow disk] pass module into the workaround function#2717

Merged
yxieca merged 1 commit intosonic-net:masterfrom
yxieca:hung_task
Dec 30, 2020
Merged

[slow disk] pass module into the workaround function#2717
yxieca merged 1 commit intosonic-net:masterfrom
yxieca:hung_task

Conversation

@yxieca
Copy link
Collaborator

@yxieca yxieca commented Dec 30, 2020

Summary:
Address regression caused by PR #2715 .

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

PR #2715 introduced an issue where the workaround function doesn't have module parameter to call exec_command with.

How did you do it?

Pass in the module parameter so that workaround function can execute command properly.

How did you verify/test it?

Run upgrade_sonic task against an DUT.

@yxieca yxieca requested a review from a team December 30, 2020 08:08
@yxieca yxieca merged commit 20d729d into sonic-net:master Dec 30, 2020
@yxieca yxieca deleted the hung_task branch December 30, 2020 16:39
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…net#14540)

Why I did it
src/linkmgrd

* 2b81c83 - (HEAD -> 202205, origin/202205) [active-standby] Enforce switchover based on heartbeats when mux probe keeps failing (sonic-net#184) (5 hours ago) [Jing Zhang]
src/sonic-platform-common

* c401866 - (HEAD -> 202205, origin/202205) Add set/get_lpmode APIs for sff8436 and sff8636 optics (sonic-net#353) (4 hours ago) [longhuan-cisco]
* ea8331e - implement CMIS set_laser_freq to support 100GHz grid space (sonic-net#294) (4 hours ago) [ChiouRung Haung]
src/sonic-platform-daemons

* cc32493 - (HEAD -> 202205, origin/202205) Retrieve subport from CONFIG_DB to enable breakout support (sonic-net#342) (4 hours ago) [mihirpat1]
* 864e589 - Pass grid parameter while calling set_laser_freq (sonic-net#317) (4 hours ago) [mihirpat1]
src/sonic-swss

* 699477d - (HEAD -> 202205, origin/202205) overlay_dmac change in Vnet configuration. (sonic-net#2721) (4 hours ago) [siqbal1986]
src/sonic-utilities

* f9f3e7e4 - (HEAD -> 202205, origin/202205) [voq] Add fabric_ns to the ns_list when display_option is DISPLAY_ALL. (sonic-net#2717) (4 hours ago) [jfeng-arista]
* eb2d9167 - [GCU] Prohibit removal of PFC_WD POLL_INTERVAL field (sonic-net#2777) (4 hours ago) [isabelmsft]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant