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

Fix version in db_migrator for PORT_QOS_MAP|global #2289

Conversation

bingwang-ms
Copy link
Contributor

What I did

This PR is to fix the db_migrator version for table PORT_QOS_MAP|global.

Because the migrator version for table PORT_QOS_MAP|global is version_2_0_0 in 202012 branch (PR #2215), we should use the same version in master and 202205 branch to be compatible.

How I did it

Change the migrator version for table PORT_QOS_MAP|global from version_3_0_5 to version_2_0_0 .

How to verify it

Verified by UT.

collected 78 items                                                                                                                                                                                    

tests/db_migrator_test.py ..............................................................................                                                                                        [100%]

===================================================================================== 78 passed in 21.88 seconds ======================================================================================

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)

@bingwang-ms bingwang-ms requested review from vaibhavhd and yxieca August 2, 2022 11:24
@bingwang-ms bingwang-ms merged commit c6794b5 into sonic-net:master Aug 3, 2022
yxieca pushed a commit that referenced this pull request Aug 8, 2022
* Fix db_migrator version for PORT_QOS_MAP|global
dprital added a commit to dprital/sonic-buildimage that referenced this pull request Aug 9, 2022
Update sonic-utilities submodule pointer to include the following:
* Fix GCU bug when backend service modifying config ([sonic-net#2295](sonic-net/sonic-utilities#2295))
* Fix issues for sonic_installer upgrade-docker and sonic_installer rollback-docker ([sonic-net#2278](sonic-net/sonic-utilities#2278))
* [crm] add checking for CRM interval range ([sonic-net#2293](sonic-net/sonic-utilities#2293))
* Fix the issue that sonic_platform is not installed on vs image ([sonic-net#2300](sonic-net/sonic-utilities#2300))
* Add FEC correctable and uncorrectable port stats ([sonic-net#2027](sonic-net/sonic-utilities#2027))
* Add CLI to configure YANG config validation ([sonic-net#2147](sonic-net/sonic-utilities#2147))
* Add override testcase to verify removal ([sonic-net#2288](sonic-net/sonic-utilities#2288))
* Fix version in db_migrator  for  ([sonic-net#2289](sonic-net/sonic-utilities#2289))
* [intfutil] Check whether the FEC mode is supported on the platform before configuring it to CONFIG_DB ([sonic-net#2223](sonic-net/sonic-utilities#2223))
* Transfer organization from Azure to sonic-net ([sonic-net#2284](sonic-net/sonic-utilities#2284))
* [watermarkstat] Fix CLI script for unconfigured PG counters ([sonic-net#2239](sonic-net/sonic-utilities#2239))
* Improve the way to check port type of RJ45 port ([sonic-net#2249](sonic-net/sonic-utilities#2249))

Signed-off-by: dprital <drorp@nvidia.com>
preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Nov 21, 2022
* Fix db_migrator version for PORT_QOS_MAP|global
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants