plushu / plushu-apps

Miscellaneous app management commands
MIT License
0 stars 0 forks source link

`apps:logs` is not a real log solution #1

Closed stuartpb closed 9 years ago

stuartpb commented 10 years ago

Plushu deserves a proper, first-level logs plugin and command.

What's wrong with docker logs -t $(cat $app_dir/proc/web.cid)? Compare it to heroku logs $app. To sum it up, the solution here only gives logs from the running instance, and doesn't include any other logs- like, say, logs from a crash. (Oh yeah- we're not restarting the app when it crashes. That's another issue.) It doesn't include build/release output. It doesn't include request logs. It doesn't include timestamps. Frankly, it sucks.

This might be doable within the scope of continuing to use progrium/buildstep (at least for building), or it might involve overhauling the inside of the container too.

Right now, I'm unconcerned enough with logging to let this slide.

stuartpb commented 10 years ago

Some relevant literature: https://blog.logentries.com/2014/03/the-state-of-logging-on-docker/

stuartpb commented 10 years ago

You know, if sort can do timestamps, this miiight be doable with what we have now, although there are still questions:

stuartpb commented 9 years ago

Further discussion is being held in plushu/plushu-logs#1. (Was apps:logs ever even implemented?)