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
MON-1688: Expose remote write #1308
MON-1688: Expose remote write #1308
Conversation
009424b
to
8216365
Compare
e01fe28
to
e72b0df
Compare
go.mod
Outdated
@@ -29,3 +29,5 @@ require ( | |||
) | |||
|
|||
replace k8s.io/client-go => k8s.io/client-go v0.21.2 | |||
|
|||
replace github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring v0.48.1 => github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring v0.49.1-0.20210803083158-78e0ea966b65 |
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 would stick with v0.49.0 and leave aside Authorization and OAuth2 support for now (it can be added in 4.10 once we upgrade to a version of the Prometheus operator supporting them).
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.
Oh yeah, good point ty.
4e36ab7
to
f9cb9ee
Compare
f9cb9ee
to
653dc91
Compare
6ba8dca
to
5dd1a80
Compare
/lgtm |
/retest |
/test e2e-agnostic-upgrade |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
4 similar comments
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
12 similar comments
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/label qe-approved |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/test e2e-agnostic |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/hold |
/hold cancel |
@jan--f: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest-required Please review the full test history for this PR and help us cut down flakes. |
This is a WIP PR as prometheus-operator/prometheus-operator#4180 is still being reviewedThis exposes
remote_write
to users. CMO gets its ownremote_write
struct in order to hide unsafe fields (likeBearerToken
andTLSConfig
) and to potentially add fields later so simpler specification for this feature.