prometheus / pushgateway

Push acceptor for ephemeral and batch jobs.
Apache License 2.0
3k stars 466 forks source link

/usr/local/bin/pushgateway --version. to stderr #648

Closed kunalmehta-eve closed 4 months ago

kunalmehta-eve commented 4 months ago

Is there a good reason to have --version go to stderr? I've been trying to patch up some ansible roles to install pushgateway and I spend some time trying to figure out why I couldn't grab the output the other day.

So I wanted to make a ticket and ask.

I can confirm it's stderr (just wondering if this is maybe bug or intentional):

Feature request

Use case. Why is this important?

“Nice to have” is not a good use case. :)

Bug Report

What did you do?

What did you expect to see? Output should be redirected to stdout not stderr

What did you see instead? Under which circumstances?

Environment

/usr/local/bin/pushgateway --version pushgateway, version 1.8.0 (branch: HEAD, revision: 5aef87129c8d9ecebcc8b2fb0e97a31980680e8c) build user: root@7f0b27f823b5 build date: 20240403-17:11:45 go version: go1.22.1 platform: linux/amd64 tags: unknown

beorn7 commented 4 months ago

No good reason. It's just the Kingpin default behavior. But the Prometheus server, also using Kingpin, changes the default behavior to stdout. I think we should be consistent within the Prometheus ecosystem.

651 changes the behavior. Please have a look.