Skip to content

[action] [PR:9470] Revert "Set key "asic0" in single-asic scenerio to keep consistent with multi-asic in function core_dump_and_config_check."#9509

Merged
mssonicbld merged 1 commit intosonic-net:202012from
mssonicbld:cherry/202012/9470
Aug 17, 2023
Merged

[action] [PR:9470] Revert "Set key "asic0" in single-asic scenerio to keep consistent with multi-asic in function core_dump_and_config_check."#9509
mssonicbld merged 1 commit intosonic-net:202012from
mssonicbld:cherry/202012/9470

Conversation

@mssonicbld
Copy link
Collaborator

Reverts #8884

In PR #8884, in multi-asic scenrio

node.copy(content=json.dumps(duts_data[node.hostname]["pre_running_config"]["asic0"], indent=4),
 dest='/etc/sonic/config_db.json', verbose=False)

This code will overwrite the global config with asic0 config, which will cause failure. So revert this PR first. And then we will set a more exactly key name.

…multi-asic. (sonic-net#8884)" (sonic-net#9470)

Reverts sonic-net#8884

In PR sonic-net#8884, in multi-asic scenrio
```
node.copy(content=json.dumps(duts_data[node.hostname]["pre_running_config"]["asic0"], indent=4),
              dest='/etc/sonic/config_db.json', verbose=False)
```
This code will overwrite the global config with asic0 config, which will cause failure. So revert this PR first. And then we will set a more exactly key name.

Signed-off-by: Yutong Zhang <[email protected]>
@mssonicbld
Copy link
Collaborator Author

Original PR: #9470

@mssonicbld mssonicbld merged commit 9eb950b into sonic-net:202012 Aug 17, 2023
@mssonicbld mssonicbld deleted the cherry/202012/9470 branch February 4, 2024 09:02
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.

2 participants