Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sonic-config-engine uses libswsscommon instead of swsssdk #6406

Merged
merged 3 commits into from
Jan 20, 2021

Conversation

qiluo-msft
Copy link
Collaborator

- Why I did it
swsssdk will be deprecated. Migrate sonic-config-engine to use libswsscommon library instead

- How I did it

- How to verify it
Unit test

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Copy link
Collaborator

@lguohan lguohan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you check vs test failure?

@qiluo-msft
Copy link
Collaborator Author

Retest vs please

@lguohan
Copy link
Collaborator

lguohan commented Jan 12, 2021

looks like vstest failed again. does not seem to be random event, need investigation.

@qiluo-msft
Copy link
Collaborator Author

ACK, noticing the bug and working on it.

@qiluo-msft qiluo-msft marked this pull request as draft January 12, 2021 21:27
Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>
Includes commits:

```
2f79bb4 2021-01-16 | Fix ConfigDBPipeConnector.get_config() implementation (sonic-net#446) [Qi Luo]
fc64e3a 2021-01-12 | [dbconnector] protect db information with mutex (sonic-net#423) [Ying Xie]
e8a03de 2021-01-12 | [ci]: use sonicbld pool (sonic-net#445) [lguohan]
```
@qiluo-msft qiluo-msft marked this pull request as ready for review January 19, 2021 19:37
@qiluo-msft qiluo-msft merged commit 3c16f80 into sonic-net:master Jan 20, 2021
@qiluo-msft qiluo-msft deleted the qiluo/configcommon branch January 20, 2021 20:06
daall pushed a commit that referenced this pull request Jan 22, 2021
**- Why I did it**
swsssdk will be deprecated. Migrate sonic-config-engine to use libswsscommon library instead

**- How to verify it**
Unit test
lguohan pushed a commit that referenced this pull request Jan 23, 2021
…MMON (#6538)

Otherwise all the docker image derived from docker-config-engine-stretch will have broken SONIC_CONFIG_ENGINE_PY2
The bug is introduced in #6406
lguohan pushed a commit that referenced this pull request Jan 24, 2021
…MMON (#6538)

Otherwise all the docker image derived from docker-config-engine-stretch will have broken SONIC_CONFIG_ENGINE_PY2
The bug is introduced in #6406
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.

3 participants