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

Samba test fix #13

Merged
merged 3 commits into from
Dec 22, 2023
Merged

Samba test fix #13

merged 3 commits into from
Dec 22, 2023

Conversation

moto-timo
Copy link
Member

Revert the ubuntu-20.04 bump as that introduces smb4 and our samba.sh and smb.conf are not ready for that.

Install smbclient on the runner so it knows how to mount a cifs to allow the tests to continue passing

This moves to smb4 and our samba.sh and smb.conf are not ready for that.

This reason for this commit was to fix the failing github actions tests,
but since it was not tested in a pull request, it didn't help nor fix
the failure.

This reverts commit e6bde33.

Signed-off-by: Tim Orling <tim.orling@konsulko.com>
Signed-off-by: Tim Orling <tim.orling@konsulko.com>
@moto-timo moto-timo self-assigned this Dec 22, 2023
Fix:
mount error: cifs filesystem not supported by the system
mount error(19): No such device
Refer to the mount.cifs(8) manual page (e.g. man mount.cifs) and kernel log messages (dmesg)

Signed-off-by: Tim Orling <tim.orling@konsulko.com>
@moto-timo moto-timo merged commit 1e7ce3a into crops:master Dec 22, 2023
1 check passed
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.

1 participant