Results 603 comments of Tim Gross

I don't have the bandwidth right now to submit a fix to sirupsen/logrus. Ultimately this is an end-user issue that we've documented a warning for, and fixing the upstream problem...

Another workaround option for someone running into this kind of problem will be to use the `passthru` logging option described in #462 #437

> I think this is another reasonable solution to use cases similar to #504. It's also useful in much more general cases. IIRC this was in discussion when we made...

> Two issues that I can think of are that the CP config for watches assumes only services, so we'll have to figure out a path through that. I'm pretty...

The groundwork for this is done in https://github.com/joyent/containerpilot/pull/418 but a subsequent PR will need to be done to allow (and test) the configuration change.

Noting for myself that there's a lot of under-the-hook implementation overlap between the issues in #435, #416, and #396

The big picture need for this seems sound. The details look complicated. I think we need to explore the edge cases, particularly around `each` vs `once` and some of the...

The bus is a dumb publisher. Each job tracks its own state (via things like [`restartsRemain`](https://github.com/joyent/containerpilot/blob/e7fda21871b001a359a6840297671159f667a914/jobs/jobs.go#L71) or [`startEvent`](https://github.com/joyent/containerpilot/blob/e7fda21871b001a359a6840297671159f667a914/jobs/jobs.go#L60)), which is why we did things like set the start event to...

> Is there any other initiative to add state tracking to CP? I'm happy to keep using my "solution" if that's the way it is. I just thought it was...

Noting for myself that there's a lot of under-the-hook implementation overlap between the issues in #435, #416, and #396