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

Set owner after restoring counters folder during warmboot #13507

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

oleksandrx-kolomeiets
Copy link
Contributor

Signed-off-by: Oleksandr Kolomeiets oleksandrx.kolomeiets@intel.com

Why I did it

After warm reboot, show environment prints the following error:
failed to import plugin show.plugins.macsec: [Errno 13] Permission denied: '/tmp/cache/macsec'

How I did it

Set owner back to admin after restoring counters folder.

How to verify it

sudo warm-reboot, then ensure show environement does not print errors.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Set owner after restoring counters folder during warmboot.

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Oleksandr Kolomeiets <oleksandrx.kolomeiets@intel.com>
@oleksandrx-kolomeiets
Copy link
Contributor Author

@lguohan,
please validate the fix above or suggest the better approach if any.

@lguohan
Copy link
Collaborator

lguohan commented Feb 6, 2023

@yxieca , can you check this one?

1 similar comment
@lguohan
Copy link
Collaborator

lguohan commented Mar 27, 2023

@yxieca , can you check this one?

@yxieca yxieca merged commit 4da51b0 into sonic-net:master Mar 27, 2023
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Mar 27, 2023
…13507)

Why I did it
After warm reboot, show environment prints the following error:
failed to import plugin show.plugins.macsec: [Errno 13] Permission denied: '/tmp/cache/macsec'

How I did it
Set owner back to admin after restoring counters folder.

How to verify it
sudo warm-reboot, then ensure show environement does not print errors.

Signed-off-by: Oleksandr Kolomeiets <oleksandrx.kolomeiets@intel.com>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #14435

yxieca pushed a commit that referenced this pull request Mar 27, 2023
…14435)

Why I did it
After warm reboot, show environment prints the following error:
failed to import plugin show.plugins.macsec: [Errno 13] Permission denied: '/tmp/cache/macsec'

How I did it
Set owner back to admin after restoring counters folder.

How to verify it
sudo warm-reboot, then ensure show environement does not print errors.

Signed-off-by: Oleksandr Kolomeiets <oleksandrx.kolomeiets@intel.com>
Co-authored-by: oleksandrx-kolomeiets <oleksandrx.kolomeiets@intel.com>
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.

5 participants