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."#9510

Merged
mssonicbld merged 1 commit intosonic-net:202205from
mssonicbld:cherry/202205/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."#9510
mssonicbld merged 1 commit intosonic-net:202205from
mssonicbld:cherry/202205/9470

Conversation

@mssonicbld
Copy link
Collaborator

@mssonicbld mssonicbld commented Aug 16, 2023

Reverts #8884 to fix #9015

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 71748f2 into sonic-net:202205 Aug 17, 2023
@mssonicbld mssonicbld deleted the cherry/202205/9470 branch February 4, 2024 09:24
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