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

[action] [PR:306] Set the execute bit on sysDescr_pass.py (#306) #307

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

mssonicbld
Copy link
Collaborator

Why I did it

Slim images may have a sysDescr_pass.py script that does not correctly have the execute bit set resulting in any request for the sysDescr.0 OID via SNMP would fail. By setting the execute bit directly in source control, this script is no longer reliant on any deployment script to fix the permissions after the fact.

How I did it

Set the execute bit directly within source control

How to verify it

Testing on a platform which uses a slim image should reliably pass the snmp/test_snmp_v2mib.py test.

#### Why I did it
Slim images may have a sysDescr_pass.py script that does not correctly have the execute bit set resulting in any request for the sysDescr.0 OID via SNMP would fail.  By setting the execute bit directly in source control, this script is no longer reliant on any deployment script to fix the permissions after the fact.

#### How I did it
Set the execute bit directly within source control

#### How to verify it
Testing on a platform which uses a slim image should reliably pass the snmp/test_snmp_v2mib.py test.
@mssonicbld
Copy link
Collaborator Author

Original PR: #306

@mssonicbld mssonicbld merged commit b0a4bcc into sonic-net:202305 Jan 18, 2024
3 checks passed
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