Skip to content

Add DBConnector::getall() method and refactor ConfigDBConnector_Native::get_config().#692

Merged
liuh-80 merged 1 commit intosonic-net:masterfrom
liuh-80:dev/liuh/refact-getall
Sep 29, 2022
Merged

Add DBConnector::getall() method and refactor ConfigDBConnector_Native::get_config().#692
liuh-80 merged 1 commit intosonic-net:masterfrom
liuh-80:dev/liuh/refact-getall

Conversation

@liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Sep 28, 2022

Why I did it

Add DBConnector::getall() for profile provider feature.

How I did it

Add DBConnector::getall() method.
Move content of ConfigDBConnector_Native::get_config() to DBConnector::getall()

How to verify it

Pass all existing UT and E2E test.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Add DBConnector::getall() method and refactor ConfigDBConnector_Native::get_config().

Link to config_db schema for YANG module changes

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

@liuh-80 liuh-80 requested a review from qiluo-msft September 28, 2022 08:47
@liuh-80 liuh-80 marked this pull request as ready for review September 28, 2022 08:47
@liuh-80
Copy link
Contributor Author

liuh-80 commented Sep 29, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80 liuh-80 merged commit 23d7c12 into sonic-net:master Sep 29, 2022
@liuh-80 liuh-80 deleted the dev/liuh/refact-getall branch September 29, 2022 10:23
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