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

Create a shadow ZTP data json file accessible to non-root user #21

Open
wants to merge 1 commit into
base: 201911
Choose a base branch
from

Conversation

rajendra-dendukuri
Copy link
Collaborator

To allow non-root user to view status information, a shadow file for the current
ztp_data.json is created. The shadow file (ztp_data_shadow.json) contains only
status information which does not provide knowledge of url's of the
configuration scripts used for ztp. The ztp_data_shadow.json file is updated
everytume ztp_data.json is updated. The ztp_data_shadow.json is accessible to
non-root user as well while ztp_data.json is accessible only to root user and
has more information about the url's where configuration scripts can be downloaded
from.

To allow non-root user to view status information, a shadow file for the current
ztp_data.json is created. The shadow file (ztp_data_shadow.json) contains only
status information which does not provide knowledge of url's of the
configuration scripts used for ztp. The ztp_data_shadow.json file is updated
everytume ztp_data.json is updated. The ztp_data_shadow.json is accessible to
non-root user as well while ztp_data.json is accessible only to root user and
has more information about the url's where configuration scripts can be downloaded
from.

Signed-off-by: Rajendra Dendukuri <[email protected]>
@rajendra-dendukuri
Copy link
Collaborator Author

Fixes sonic-net/sonic-buildimage#5377

@liat-grozovik
Copy link

Who is the ztp maintainer that can approve and merge it?

@ben-gale
Copy link
Collaborator

Who is the ztp maintainer that can approve and merge it?

@rajendra-dendukuri

@rajendra-dendukuri
Copy link
Collaborator Author

/easycla

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