Bug 1372258

Summary:	sssctl config-check: Add path to files when pattern matching fails
Product:	Red Hat Enterprise Linux 7	Reporter:	Marc Muehlfeld <mmuehlfe>
Component:	ding-libs	Assignee:	Michal Zidek <mzidek>
Status:	CLOSED ERRATA	QA Contact:	Madhuri <mupadhye>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	7.4	CC:	grajaiya, jhrozek, lslebodn, mkosek, mzidek, pbrezina, sgoveas
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	ding-libs-0.6.1-28.el7	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-04-10 16:24:47 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:

Description Marc Muehlfeld 2016-09-01 09:21:24 UTC

Description of problem:
sssctl check-config is inconsistent when on the output:
Messages generated during configuration merging: 2
File /etc/sssd/conf.d/wrong-file-permissions.conf did not pass access check. Skipping.
File configuration.conf.disabled did not match provided patterns. Skipping.

For the second item, only the file name is shown - without the full path.



Version-Release number of selected component (if applicable):
sssd-tools-1.14.0-27.el7.x86_64



Steps to Reproduce:
1. Add a file /etc/sssd/conf.d/configuration.conf.disabled
2. Run
   # sssctl config-check



Actual results:
File configuration.conf.disabled did not match provided patterns. Skipping.



Expected results:
File /etc/sssd/conf.d/configuration.conf.disabled did not match provided patterns. Skipping.

Comment 1 Michal Zidek 2016-09-01 09:29:00 UTC

Upstream ticket:
https://fedorahosted.org/sssd/ticket/3166

Comment 3 Jakub Hrozek 2016-09-05 07:34:21 UTC

Upstream ticket:
https://fedorahosted.org/sssd/ticket/3171

Comment 4 Jakub Hrozek 2016-09-05 07:37:57 UTC

Upstream ticket:
https://fedorahosted.org/sssd/ticket/3166

Comment 5 Michal Zidek 2016-09-13 15:07:34 UTC

ding-libs master:
* 72ca1a3552fdc33b7fa7e832ecfc235dd685a7e7

Comment 12 Madhuri 2018-01-15 08:03:43 UTC

Verified with

# rpm -qa sssd
sssd-1.16.0-14.el7.x86_64

Verification Steps:

1. Configured ldap server.

2. Configure sssd client.

3. Add /etc/sssd/conf.d/test2.conf and /etc/sssd/conf.d/test3.conf files with permission 0777.

4. Run #sssctl config-check

# sssctl config-check
Issues identified by validators: 0

Messages generated during configuration merging: 2
File /etc/sssd/conf.d/test2.conf did not pass access check. Skipping.
File /etc/sssd/conf.d/test3.conf did not pass access check. Skipping.

Used configuration snippet files: 1
/etc/sssd/conf.d/test1.conf

Showing full path of second file, hence verified

Comment 15 errata-xmlrpc 2018-04-10 16:24:47 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, 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-2018:0898