Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

Metrics Prometheus Push Gateway Support #638

Merged
merged 4 commits into from
Jan 23, 2019
Merged

Conversation

shemnon
Copy link
Contributor

@shemnon shemnon commented Jan 22, 2019

PR description

Sometimes metrics are hard to poll (docker containers with varying ip
addresses). Because of that the push gateway exists. This extends the
metrics system to support push or pull mode for metrics (but not both
at the same time).

Three new flags
--metrics-mode=<push|pull> - Whether we are in pull mode (the default) where
prometheus is expected to poll or push mode where pantheon pushes to
a push gateway.

--metrics-push-interval=<integer> the frequency, in seconds, between pushes to
the push gateway. Only relevant in push mode

--metrics-prometheus-job=<string> The name of the job to report in the push gateway

Also, --metrics-host= and --metrics-port= gain new meaning in push mode. Instead of the
server they are opening up it is the host and the port of the push gateway it should push to.

Sometimes metrics are hard to poll (docker containers with varying ip
addresses).  Because of that the push gateway exists.  This extends the
metrics system to support push or pull mode for metrics (but not both
at the same time).
Sometimes metrics are hard to poll (docker containers with varying ip
addresses).  Because of that the push gateway exists.  This extends the
metrics system to support push or pull mode for metrics (but not both
at the same time).
Copy link
Contributor

@ajsutton ajsutton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@shemnon shemnon merged commit f02c98b into PegaSysEng:master Jan 23, 2019
@shemnon shemnon added the doc next release Doc needed for next release (ie, dev work done) label Jan 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
doc next release Doc needed for next release (ie, dev work done)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants