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

Fix PreserveSMBAttr for linux #2530

Merged
merged 6 commits into from Jan 17, 2024
Merged

Conversation

nakulkar-msft
Copy link
Member

@nakulkar-msft nakulkar-msft commented Jan 9, 2024

Preserve smb-permissions to linux was not working because of a restrictive interface. Cleaned the interface. The same tests running on windows have been made to run in linux. There is one test which fails, I'm yet to understand and I've disabled it on linux currently. Will check with Adele.

@gapra-msft
Copy link
Member

point PR to dev

Copy link
Contributor

@tasherif-msft tasherif-msft left a comment

Choose a reason for hiding this comment

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

can we add a small test for this that ensures this is now corrected on linux

@nakulkar-msft nakulkar-msft changed the base branch from main to dev January 10, 2024 02:53
vibhansa-msft
vibhansa-msft previously approved these changes Jan 10, 2024
@nakulkar-msft nakulkar-msft merged commit 98add5d into dev Jan 17, 2024
17 checks passed
@nakulkar-msft nakulkar-msft deleted the nakulkar/private/preserveSMB1 branch January 20, 2024 02:40
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.

None yet

5 participants