Current implementation (see https://review.openstack.org/#/c/93178/) assumes that developers explicitly provide log files to be included into diagnostic snapshot archive. I consider this to be a very bad practice, when system relies on memory of developers. We’ve already forgotten about a several important log files, and I envision even more issues like this in the future (log file renamed, but config is not updated - so log doesn’t get included into snapshot, etc.).
It has to be reimplemented in a way when the whole folder gets included. If we need to, we can introduce a list of excludes.
I tried to add entire /var/log directory, but for some reasons it fails
Warning: get() encountered an exception while downloading '/var/log/'
Underlying exception:
Failure
I didn't have enough time to debug it
Here is config [1] which you can use for debugging, use it in mcollective container like `shotgun -c /tmp/cfg`
[1] http:// paste.openstack .org/show/ 84386/