Remove 'fold' from barn-agent loop.

The fold makes the code safer but I think it is at the cost of
the code being readable for someone new to the project. I've
removed it to hopefully make the steps easier to see.

This change also ensures that barn-agent doesn't failover when it's got a
'flacky' connection i.e. where it's managing to ship some files out of the
entire list. In production barn-agent can often end up running like this.
Add logging directory to tests.
Send metric if barn-agent is shipping all log files.

This is a good sign that something has gone wrong. Either barn-agent has
failed over or something has gone wrong with the middle node.