search

celestiaorg / test-infra

Testing infrastructure for the Celestia Network
Apache License 2.0
25 stars 10 forks source link

fix: Error logs to stderr stream #196

Closed tty47 closed 1 year ago

tty47 commented 1 year ago

Overview

Hello team!

Tiny change, as a best practice, I've moved the errors stream to stderr instead of using the stdout which is the default log stream but not for errors. I realised that we were using the same stream for both use cases.

Was it set to stdout in purpose?

Thanks in advance! 🚀

Checklist

Bidon15 commented 1 year ago

Was it set to stdout in purpose?

Yes it was back in the local:docker days. While now we don't care much about hacking our way out as before due to:

  1. containers are deployed in pods
  2. we will soon have logging into file for comet-bft /app