New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable metrics #94
Enable metrics #94
Conversation
go.mod
Outdated
@@ -21,3 +21,5 @@ require ( | |||
sigs.k8s.io/controller-tools v0.2.9-0.20200331153640-3c5446d407dd | |||
sigs.k8s.io/yaml v1.2.0 | |||
) | |||
|
|||
replace github.com/openshift/machine-api-operator v0.2.1-0.20200527204437-14e5e0c7d862 => github.com/Danil-Grigorev/machine-api-operator v0.2.1-0.20200604073207-04068f243c6b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer this to be moved back to master openshift before merge
Hold until we can identify whether this is absolutely necessary
/hold
d074718
to
5dcc563
Compare
/hold Waiting on openshift/machine-api-operator#590 |
/hold cancel |
/retest |
This PR needs a rebase And I think we need to revendor MAO here so that we can bring in the metrics package right? |
5dcc563
to
d9b85dd
Compare
d9b85dd
to
ae54e26
Compare
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
go.mod
Outdated
replace ( | ||
sigs.k8s.io/cluster-api-provider-aws => github.com/openshift/cluster-api-provider-aws v0.2.1-0.20200618031251-e16dd65fdd85 | ||
sigs.k8s.io/cluster-api-provider-azure => github.com/openshift/cluster-api-provider-azure v0.1.0-alpha.3.0.20200618001858-af08a66b92de | ||
sigs.k8s.io/cluster-api-provider-gcp => github.com/openshift/cluster-api-provider-gcp v0.0.1-0.20200618110630-09cef98ae787 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this needed at all? this repo is not even a fork.
Can you please include a link to the mao counter PR in the PR desc and the commit desc? |
ae54e26
to
34469da
Compare
34469da
to
3a57e2e
Compare
- Include PR openshift/machine-api-operator#609 introducing metrics support
3a57e2e
to
e99eda8
Compare
/retest |
@enxebre @JoelSpeed @alexander-demichev I'd like to merge those today. Please approve and LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
📖 Add helm chart to getting started doc
Working on https://issues.redhat.com/browse/OCPCLOUD-784
This PR is adding support for reporting following prometheus metrics and also starting controller-runtime metrics server to make these metrics available for prometheus servers:
mapi_instance_create_failed
: Total count of "create" cloud api errorsmapi_instance_update_failed
: Total count of "update" cloud api errorsmapi_instance_delete_failed
: Total count of "delete" cloud api errorsLabels on these metrics (for GCP):
Depends on PR openshift/machine-api-operator#609 introducing metrics support