Skip to content

Fix random failure in PR/CI build.#2006

Merged
prsunny merged 1 commit intosonic-net:masterfrom
liushilongbuaa:master
Nov 4, 2021
Merged

Fix random failure in PR/CI build.#2006
prsunny merged 1 commit intosonic-net:masterfrom
liushilongbuaa:master

Conversation

@liushilongbuaa
Copy link
Contributor

What I did
I add step to clean work space.
Thus avoid random failure in azure pipeline.
Why I did it
There are build failures because of work space not clean.
https://dev.azure.com/mssonic/build/_build/results?buildId=48551&view=logs&j=e51ebb60-45fa-5846-0822-3aaf597a9c80&t=29c82d13-0aa4-5ac7-9d6a-e34d59819a1b
How I verified it

Details if related

@prsunny
Copy link
Collaborator

prsunny commented Nov 4, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny prsunny merged commit faa26db into sonic-net:master Nov 4, 2021
liushilongbuaa added a commit to liushilongbuaa/sonic-swss that referenced this pull request Dec 7, 2021
liushilongbuaa added a commit that referenced this pull request Dec 8, 2021
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
…delete whole config move (sonic-net#2006)

#### What I did
Not generating delete whole config move because it is not allowed by JsonPatch library and it is not possible for ConfigDb to be equal to NULL.

This solves the first problem in issue sonic-net/sonic-utilities#2000 (comment)

#### How I did it
- Verified UpperLevelMoveExtender does not generate delete whole config move
- Modified DeleteInsteadOfReplaceMoveExtender to not generate delete whole config move
- DeleteRefsMoveExtender cannot generate delete whole config move, because it deletes the referrer leafs

#### How to verify it
unit-test

#### 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)
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
*step to clean work space, to avoid random failure in azure pipeline.
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