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

HLD: Management interface for Sonic Config Management operations #33

Merged
merged 7 commits into from
Nov 22, 2019

Conversation

bhaveshdell
Copy link
Collaborator

No description provided.

@bhaveshdell bhaveshdell changed the title Management interface for Sonic Config operations HLD. HLD: Management interface for Sonic Config operations Oct 22, 2019
@bhaveshdell bhaveshdell changed the title HLD: Management interface for Sonic Config operations HLD: Management interface for Sonic Config Management operations Oct 22, 2019
EXEC Level <br>
Write to file /etc/sonic/config_db.json <br>

2. Copy configuration from file to running and reload. <br>

This comment was marked as outdated.

doc/mgmt/Sonic_Config_Mgmt_HLD.md Outdated Show resolved Hide resolved
EXEC Level <br>
Write to file /etc/sonic/config_db.json <br>

2. Copy configuration from file to running and reload. <br>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When "config reload" is executed, all the containers are reloaded. Does it mean the sonic-mgmt-framework container is also reloaded. This may kill the "copy" session initiated. Ensure that even if the session is terminated, user gets some feedback and does not loose connectivity forever. Also care needs to be taken that the "config reload" session initiated on the host o/s finishes and is not interrupted because management-container is restarted.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The session will terminate due to mgmt-framework restart. User is prompted for confirmation with the warning message about connectivity loss. The session has to be re-initiated by the user.
mgmt-framework is the last service to restart.

@bhaveshdell bhaveshdell merged commit 5672065 into master Nov 22, 2019
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