Skip to content

[202012] Replace swsssdk with swsscommon in sonic-host-services and update submodule sonic-swss-common#8513

Merged
qiluo-msft merged 2 commits intosonic-net:202012from
qiluo-msft:qiluo/hostserv_swsscommon202012
Sep 15, 2021
Merged

[202012] Replace swsssdk with swsscommon in sonic-host-services and update submodule sonic-swss-common#8513
qiluo-msft merged 2 commits intosonic-net:202012from
qiluo-msft:qiluo/hostserv_swsscommon202012

Conversation

@qiluo-msft
Copy link
Copy Markdown
Collaborator

@qiluo-msft qiluo-msft commented Aug 18, 2021

Why I did it

Backport #8034 to 202012 branch

sonic-swss-common submodule updating includes below commits

a6b98da 2021-04-29 | Add support for config_db subscribe and unsubscribe python apis (#481) [arlakshm]
2506ca0 2021-08-22 | [ci] Fix azure pipeline DownloadPipelineArtifact source branch (#514) [Qi Luo]

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

Copy link
Copy Markdown
Collaborator

@sujinmkang sujinmkang Aug 18, 2021

Choose a reason for hiding this comment

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

just minor cosmetic changes, same as the changes in other files, can you change this to "from swsscomm import swsscommon"? #Resolved

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I submit another PR on master to solve your comments: #8515.

Keep this PR as simple as backport.

Copy link
Copy Markdown
Collaborator

@sujinmkang sujinmkang Aug 18, 2021

Choose a reason for hiding this comment

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

And also change this line to just "swsscommon.ConfigDBConnector = MockConfigDb"? #Resolved

@qiluo-msft
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@qiluo-msft qiluo-msft force-pushed the qiluo/hostserv_swsscommon202012 branch from 4a1df0a to 9ef593c Compare September 14, 2021 11:58
@qiluo-msft qiluo-msft changed the title [202012] Replace swsssdk with swsscommon in sonic-host-services [202012] Replace swsssdk with swsscommon in sonic-host-services and update submodule sonic-swss-common Sep 15, 2021
@qiluo-msft qiluo-msft merged commit ad8a332 into sonic-net:202012 Sep 15, 2021
@qiluo-msft qiluo-msft deleted the qiluo/hostserv_swsscommon202012 branch September 15, 2021 01:16
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