Paresh Mutha wrote:
% Hello,
% 
% Commit 7a18b250b07ff4ed0c34fa48e69029c114ec3ab1 introduced the check
% on ownership & permission of the config file so that it doesn't
% log/display the diff it the config file is not readable by all.
% 
% Here it is checking the stat for source. But source is always the
% temp file which is created based on the config file content received
% from the Server and it is always under root ownership and not
% readable by all. Thus for all config files which are not owned by
% root or are readable by all, the diff result is not displayed.
% Instead there is the message "Differences exist in a file that is
% not readable by all. Re-deployment of configuration file is
% recommended"
% 
% Attached is the proposed patch to check the stat of dst i.e the file
% present on the system. With this change, for the files which are
% readable by all the actual diff result would be displayed.

You're correct :). Reviewed and pushed patch.
Thanks.

% Regards,
% Paresh


--
Michael Mráka
Satellite Engineering, Red Hat

_______________________________________________
Spacewalk-devel mailing list
Spacewalk-devel@redhat.com
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to