Comment 15 for bug 1241376

Revision history for this message
costinel (costinel) wrote :

I don't think procps should return an error when one of the sysctl.d/ files contains incorrect data.

Printing the error would be useful especially if the exact file and value is mentioned, otherwise saying "oh, we failed" when in fact only one attribute failed to load is cumbersome.

Just because iscsitarget package upgrade failed to remove it's sysctl.d config file is not fair to let other packages fail upgrade because of dependency on procps which did not start but otherwise did NOT affect it in any way (munin, for example, failed to be configured by apt-get dist-upgrade because of that)

Don't know if this is the proper place to address this comment to package authors, so if any of you reading this consider I've written in the wrong place please direct me to the correct site/link!