Good day, While this is a good idea, we at least need the option to display a diff regardless. It's an important piece of config mgmt / file integrity mgmt in knowing what changes were made to a file that was replaced.
Is there a command line flag that will allow this? Thanks! -josh -----Original Message----- From: spacewalk-devel-boun...@redhat.com [mailto:spacewalk-devel-boun...@redhat.com] On Behalf Of Michael Mraka Sent: Monday, October 29, 2012 9:47 AM To: spacewalk-devel@redhat.com Subject: Re: [Spacewalk-devel] [PATCH] proposed for rhncfg in nightly repo to check stat of local file instead of temp file 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 _______________________________________________ Spacewalk-devel mailing list Spacewalk-devel@redhat.com https://www.redhat.com/mailman/listinfo/spacewalk-devel
