google / mtail

extract internal monitoring data from application logs for collection in a timeseries database
Apache License 2.0
3.82k stars 378 forks source link

fix: LogStreams create and close their own lines channels. #889

Closed jaqx0r closed 2 months ago

jaqx0r commented 2 months ago

This makes it easier to observe logstream state externally because we can read the lines channel until it closes.

The tests are changed slightly to asynchronously drain the output channel at the start of the stream.

Fix some naming issues and log formatting.

github-actions[bot] commented 2 months ago

Unit Test Results

    1 files     27 suites   8m 51s :stopwatch:   649 tests   647 :white_check_mark: 2 :zzz: 0 :x: 1 920 runs  1 914 :white_check_mark: 6 :zzz: 0 :x:

Results for commit 1d5c5e23.