[edot] add prometheus remote write receiver#11937
Conversation
|
This pull request does not have a backport label. Could you fix it @VihasMakwana? 🙏
|
|
This pull request is now in conflicts. Could you fix it? 🙏 |
|
@VihasMakwana Could you please add back and populate the "Why is it important?" section of this PR's description, especially there is no issue related to this PR either? Thanks. |
added the "Why is it important" section above |
|
@AlexanderWert I take this won't be a Core component. Any annotation we'd like to add to this one in |
There was a problem hiding this comment.
LGTM but not sure if @theletterf's question is blocking or not.
|
@ycombinator It's not blocking, just nice to have :) We can proceed from my point of view if we deem this is complete. |
0093283 to
e33c935
Compare
|
This pull request is now in conflicts. Could you fix it? 🙏 |
e817a24 to
d70d40a
Compare
d70d40a to
b732c95
Compare
|
/test |
💚 Build Succeeded
History
|
|
@theletterf I think we should be okay without any additional annotations. For eg. postgresqlreceiver is a part of EDOT, but it is not a core component and not mentioned in |
|
@ycombinator Could I get an approval if you're around? The CI is finally green 😄 |
* add remote write receiver * test and chlog * initial commit (cherry picked from commit b3bfc0f)
What does this PR do?
Add prometheus remote write receiver to EDOT
Why is it important?
Allows users of Prometheus to forward data to Elastic through the EDOT Collector without requiring them to adapt their collection architecture.
Checklist
./changelog/fragmentsusing the changelog toolDisruptive User Impact
How to test this PR locally
Related issues
Questions to ask yourself