CephNFS: Some fixes for mounting CephNFS using Kerberos auth #12086
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BlaineEXE
approved these changes
Apr 13, 2023
|
Simple. I love it. @spuiuk the commitlint bot is complaining about the capital |
Adding the search domain localhost causes lookup failures. Many DNS servers have a wildcard entry *.localhost which returns 127.0.0.1. Adding a "search localhost" to resolv.conf appends a ".localhost" for every host lookup for which the DNS server returns 127.0.0.1 which causes failures further down the execution path. example: with localhost in the search domains # ping google.com PING google.com.localhost (127.0.0.1) 56(84) bytes of data. 64 bytes from localhost (127.0.0.1): icmp_seq=1 ttl=64 time=0.071 ms Closes: rook#11870 Signed-off-by: Sachin Prabhu <sprabhu@redhat.com>
When using sssd, the /etc/nsswitch.conf should also include local files for lookups. This is required when looking up uid 0 and the root user which are not handled by ldap or other sources. Closes: rook#11925 Signed-off-by: Sachin Prabhu <sprabhu@redhat.com>
|
@BlaineEXE , Thanks. My first encounter with commitlint. There seems to be a flaky test since the only change I made was to the commit summary. |
BlaineEXE
added a commit
that referenced
this pull request
Apr 18, 2023
CephNFS: Some fixes for mounting CephNFS using Kerberos auth (backport #12086)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While testing kerberos mounting of CephNFS resources, I came across a few problems which prevent the use of the CephNFS share being used with kerberos authentication. This set of changes fix a part of the problems we see with mounting cephNFS shares using kerberos.
Which issue is resolved by this Pull Request:
#11870
#11925
Checklist:
skip-cion the PR.