Bug 1892184

Summary: autofs: return a connection failure until maps have been fetched
Product: Red Hat Enterprise Linux 8 Reporter: Ian Kent <ikent>
Component: autofsAssignee: Ian Kent <ikent>
Status: CLOSED ERRATA QA Contact: Kun Wang <kunwan>
Severity: medium Docs Contact:
Priority: low    
Version: 8.1CC: abokovoy, arusso, atikhono, b.prins, dlavu, dpal, esa, grajaiya, ikent, jhrozek, jhunt, lslebodn, mbogdano, mkosek, mniranja, mrichter, muhammad.zali, mzidek, orion, pbrezina, rvdwees, sbose, sgoveas, thalman, tscherf, xzhou
Target Milestone: pre-dev-freezeKeywords: Triaged
Target Release: 8.1Flags: pm-rhel: mirror+
Hardware: All   
OS: Linux   
Whiteboard: sync-to-jira
Fixed In Version: autofs-5.1.4-48.el8 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1113639 Environment:
Last Closed: 2021-05-18 14:55:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1113639    
Bug Blocks: 1101782, 1679810, 1689138    
Attachments:
Description Flags
Patch - fix lookup_nss_read_master() nsswicth check return
none
Patch - fix sss_master_map_wait timing
none
Patch - improve lookup_sss() error handling
none
Patch - fix lookup_nss_read_master() nsswicth check return
none
Patch - fix typo in open_sss_lib()
none
Patch - fix sss_master_map_wait timing
none
Patch - add sss ECONREFUSED return handling
none
Patch - use mapname in sss context for setautomntent()
none
Patch - add support for new sss autofs proto version call
none
Patch - fix retries check in setautomntent_wait()
none
Patch - refactor sss setautomntent()
none
Patch - improve sss setautomntent() error handling
none
Patch - refactor sss getautomntent()
none
Patch - improve sss getautomntent() error handling
none
Patch - sss introduce calculate_retry_count() function
none
Patch - move readall into struct master
none
Patch - sss introduce a flag to indicate map being read
none
Patch - update sss timeout documentation
none
Patch - refactor sss getautomntbyname()
none
Patch - improve sss getautomntbyname() error handling
none
Patch - use a valid timeout in lookup_prune_one_cache()
none
Patch - dont prune offset map entries
none
Patch - simplify sss source stale check none

Comment 12 Ian Kent 2020-11-03 00:18:50 UTC
Created attachment 1726069 [details]
Patch - fix lookup_nss_read_master() nsswicth check return

Comment 13 Ian Kent 2020-11-03 00:20:07 UTC
Created attachment 1726070 [details]
Patch - fix sss_master_map_wait timing

Comment 14 Ian Kent 2020-11-03 00:22:44 UTC
Created attachment 1726071 [details]
Patch - improve lookup_sss() error handling

There are a number of comments in this patch and the description
also claims to understand the various cases.

Please verify my understanding of what sss is doing is correct.

Comment 20 Ian Kent 2020-11-30 09:06:48 UTC
Created attachment 1734755 [details]
Patch - fix lookup_nss_read_master() nsswicth check return

Comment 21 Ian Kent 2020-11-30 09:08:25 UTC
Created attachment 1734756 [details]
Patch - fix typo in open_sss_lib()

Comment 22 Ian Kent 2020-11-30 09:10:03 UTC
Created attachment 1734757 [details]
Patch - fix sss_master_map_wait timing

Comment 23 Ian Kent 2020-11-30 09:11:13 UTC
Created attachment 1734758 [details]
Patch - add sss ECONREFUSED return handling

Comment 24 Ian Kent 2020-11-30 09:12:29 UTC
Created attachment 1734759 [details]
Patch - use mapname in sss context for setautomntent()

Comment 25 Ian Kent 2020-11-30 09:13:37 UTC
Created attachment 1734760 [details]
Patch - add support for new sss autofs proto version call

Comment 26 Ian Kent 2020-11-30 09:14:41 UTC
Created attachment 1734761 [details]
Patch - fix retries check in setautomntent_wait()

Comment 27 Ian Kent 2020-11-30 09:15:50 UTC
Created attachment 1734762 [details]
Patch - refactor sss setautomntent()

Comment 28 Ian Kent 2020-11-30 09:17:08 UTC
Created attachment 1734763 [details]
Patch - improve sss setautomntent() error handling

Comment 29 Ian Kent 2020-11-30 09:18:35 UTC
Created attachment 1734764 [details]
Patch - refactor sss getautomntent()

Comment 30 Ian Kent 2020-11-30 09:19:57 UTC
Created attachment 1734765 [details]
Patch - improve sss getautomntent() error handling

Comment 31 Ian Kent 2020-11-30 09:21:19 UTC
Created attachment 1734766 [details]
Patch - sss introduce calculate_retry_count() function

Comment 32 Ian Kent 2020-11-30 09:22:13 UTC
Created attachment 1734767 [details]
Patch - move readall into struct master

Comment 33 Ian Kent 2020-11-30 09:23:14 UTC
Created attachment 1734768 [details]
Patch - sss introduce a flag to indicate map being read

Comment 34 Ian Kent 2020-11-30 09:24:21 UTC
Created attachment 1734769 [details]
Patch - update sss timeout documentation

Comment 35 Ian Kent 2020-11-30 09:25:34 UTC
Created attachment 1734770 [details]
Patch - refactor sss getautomntbyname()

Comment 36 Ian Kent 2020-11-30 09:26:46 UTC
Created attachment 1734771 [details]
Patch - improve sss getautomntbyname() error handling

Comment 37 Ian Kent 2020-11-30 09:27:46 UTC
Created attachment 1734772 [details]
Patch - use a valid timeout in lookup_prune_one_cache()

Comment 38 Ian Kent 2020-11-30 09:28:57 UTC
Created attachment 1734773 [details]
Patch - dont prune offset map entries

Comment 39 Ian Kent 2020-11-30 09:29:57 UTC
Created attachment 1734774 [details]
Patch - simplify sss source stale check

Comment 40 Pavel Březina 2020-12-04 10:57:29 UTC
SSSD Part wash pushed upstream:
Pushed PR: https://github.com/SSSD/sssd/pull/5343

* `master`
    * 075519bceca7a8f4fa28a0b7c538f2f50d552d13 - configure: check for stdatomic.h
    * 8a22d4ad45f5fc8e888be693539495093c2b3c35 - autofs: correlate errors for different protocol versions
    * 34c519a4851194164befc150df8e768431e66405 - autofs: disable fast reply
    * 9098108a7142513fa04afdf92a2c1b3ac002c56e - autofs: translate ERR_OFFLINE to EHOSTDOWN
    * e50258da70b67ff1b0f928e2e7875bc2fa32dfde - autofs: return ERR_OFFLINE if we fail to get information from backend and cache is empty
    * 3f0ba4c2dcf9126b0f94bca4a056b516759d25c1 - cache_req: allow cache_req to return ERR_OFFLINE if all dp request failed

Comment 48 errata-xmlrpc 2021-05-18 14:55:46 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (autofs bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2021:1621