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

smbcc: Add 'dos charset' option to Global smb.conf defaults #229

Merged

Conversation

anoopcs9
Copy link
Collaborator

@anoopcs9 anoopcs9 commented Jul 5, 2022

dos charset option allows us to specify a particular charset which in turn helps to avoid the following warning message on clients in the absence of 850 charset:

dos charset 'CP850' unavailable - using ASCII

'dos charset' option allows us to specify a particular charset which
in turn helps to avoid the following warning message on clients in the
absence of 850 charset:

"dos charset 'CP850' unavailable - using ASCII"

Signed-off-by: Anoop C S <[email protected]>
@anoopcs9 anoopcs9 linked an issue Jul 5, 2022 that may be closed by this pull request
Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

That was quick! Thanks for taking this one!

@anoopcs9
Copy link
Collaborator Author

anoopcs9 commented Jul 6, 2022

/test centos-ci/sink-clustered/mini-k8s-1.24

@anoopcs9
Copy link
Collaborator Author

anoopcs9 commented Jul 6, 2022

/test centos-ci/sink-clustered/mini-k8s-1.24

There is an ongoing issue with CentOS CI environment where jenkins pod is getting restarted frequently. This can coincide with jobs running around that time which initially caused failure here. Hopefully the next run goes through well.

@phlogistonjohn
Copy link
Collaborator

OK, threre's also an open-coded uses of smbclient package in mount_path_test.go that I didn't notice before. I'm filing an issue on that and will follow up.

@phlogistonjohn
Copy link
Collaborator

Filed #230

@phlogistonjohn phlogistonjohn merged commit 5225fba into samba-in-kubernetes:master Jul 7, 2022
@anoopcs9 anoopcs9 deleted the smbconf-dos-charset branch July 7, 2022 06:04
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.

Fix annoying 'dos charset 'CP850' unavailable' warnings
2 participants