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

Make sure db_migrator is run after all config are loaded during load_minigraph #926

Merged
merged 1 commit into from
May 30, 2020

Conversation

abdosi
Copy link
Contributor

@abdosi abdosi commented May 29, 2020

- What I did
Make sure db_migrator is run after all config are loaded during load_minigraph. The behaviour got changed as part of multi-npu change so make it correct again.

- How to verify it
sudo config load_minigrah -y

load_minigraph. The behaviour got changed as part of multi-npu change
so make it correct again.
@abdosi abdosi requested a review from arlakshm May 29, 2020 20:39
@prsunny prsunny requested a review from yxieca May 29, 2020 20:45
@abdosi abdosi merged commit aae4f85 into sonic-net:master May 30, 2020
@abdosi abdosi deleted the loadminigraph_fix branch May 30, 2020 01:46
abdosi added a commit that referenced this pull request Jun 3, 2020
load_minigraph. The behaviour got changed as part of multi-npu change
so make it correct again.
abdosi added a commit to abdosi/sonic-utilities that referenced this pull request Aug 4, 2020
…c-net#926)

load_minigraph. The behaviour got changed as part of multi-npu change
so make it correct again.
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 28, 2022
Make sure db_migrator is run after all config are loaded during (sonic-net#926)
Vnet alias mapping (sonic-net#924)
Changes to make lldp show command for multi-npu platforms. (sonic-net#914)
[Mellanox] Fix thermal control issue: use natural sort for fan
status and thermal status (sonic-net#836)
[Mellanox] add document for thermal control related cli (sonic-net#832)
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