Comment 47 for bug 1541390

Revision history for this message
Ihor Kalnytskyi (ikalnytskyi) wrote :

Well, I was wrong. It's indeed about Fabric. My investigations reveal the following things:

- We store snapshots in `/var/log/dump`
- Shotgun tries to download the whole `/var/log`
- Despite the fact Shotgun has `dump/` in exclusion list for `/var/log`, the real exclusion is taking place post-factum (when it's completely downloaded)
- If there're files in `/var/log/dump/`, there's a chance that path to them is too long, and that Fabric fails to retrieve them.
- As we know from my previous explanation above, Fabric tries to download `/var/log` file-by-file recursively, but if it fails to download at least one file - the whole downloading will be aborted.
- So it fails to download file due to long path (File name too long) and we miss docker-logs in the snapshot.

So basically, I see two workarounds:

* Do not use the whole `/var/log` and point to `/var/log/docker-logs` + some other helpful stuff.
* Move `/var/log` to the beginning of objects in shotgun config, so dump/ will be probably empty and we don't catch that error.

Meantime, we MUST to consider re-implementation of shotgun as a part of bug 1543119.