Skip to content

Revert "[config reload] Fixing config reload when timer based delayed…#2017

Closed
qiluo-msft wants to merge 1 commit intosonic-net:masterfrom
qiluo-msft:qiluo/revert1967
Closed

Revert "[config reload] Fixing config reload when timer based delayed…#2017
qiluo-msft wants to merge 1 commit intosonic-net:masterfrom
qiluo-msft:qiluo/revert1967

Conversation

@qiluo-msft
Copy link
Contributor

… services are disabled (#1967)"

This reverts commit 055ed4f.

What I did

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@dgsudharsan
Copy link
Collaborator

@qiluo-msft May I know the reason why it is reverted?

@qiluo-msft qiluo-msft marked this pull request as draft January 16, 2022 19:09
@qiluo-msft qiluo-msft closed this Jan 16, 2022
ghooo added a commit that referenced this pull request Jan 18, 2022
…#2019)

Refer to #1951 for details

The PR #1951 was reverted in #2016 because it was thought to be causing the build failure in sonic-swss TestWarmReboot UTs. But it seems the failures in TestWarmReboot are still occurring e.g. #2017 [build](https://dev.azure.com/mssonic/build/_build/results?buildId=65653&view=logs&s=859b8d9a-8fd6-5a5c-6f5e-f84f1990894e)

I think we can reapply #1951 and need to investigate TestWarmReboot tests instability
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