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

[db_migrator] Fix the target db version of the port-channel key test #1842

Merged
merged 1 commit into from
Oct 18, 2021

Conversation

stephenxs
Copy link
Collaborator

What I did

The port-channel key migrator was introduced in version 2_0_2 so the expected database version of the test case should be 2_0_2.
It was modified to 2_0_3 when the new version was introduced by mistake. This won't fail the test but disable the require its database version to be updated every time a new version is introduced. (Refer #1566 and #1614 for details)
This is to correct it by changing it back to 2_0_2.

Signed-off-by: Stephen Sun stephens@nvidia.com

How I did it

How to verify it

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)

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@liat-grozovik
Copy link
Collaborator

@yxieca could you please help to review or suggest assignee?

@yxieca yxieca merged commit 00948d0 into sonic-net:master Oct 18, 2021
@stephenxs stephenxs deleted the fix-db-migrator-lacp branch October 18, 2021 23:10
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.

4 participants