-
Notifications
You must be signed in to change notification settings - Fork 635
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
[ISSUE #4415] Add Promethus source connector #4493
Conversation
.../src/main/java/org/apache/eventmesh/connector/prometheus/server/PrometheusConnectServer.java
Outdated
Show resolved
Hide resolved
...va/org/apache/eventmesh/connector/prometheus/source/connector/PrometheusSourceConnector.java
Outdated
Show resolved
Hide resolved
...va/org/apache/eventmesh/connector/prometheus/source/connector/PrometheusSourceConnector.java
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #4493 +/- ##
============================================
+ Coverage 15.46% 15.47% +0.01%
Complexity 1452 1452
============================================
Files 691 691
Lines 28101 28106 +5
Branches 2624 2626 +2
============================================
+ Hits 4345 4349 +4
- Misses 23309 23312 +3
+ Partials 447 445 -2
... and 8 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
...va/org/apache/eventmesh/connector/prometheus/source/connector/PrometheusSourceConnector.java
Show resolved
Hide resolved
@pandaapo hi may I ask why the package guava-retrying-2.0.0.jar is failed in License Check? It is normal in local |
Could you show me the detailed CI error information? I can't see it. Additionally, you can try updating the file 'knowledge-dependencies.txt'. |
it is like this as follow. And where is the file 'knowledge-dependencies.txt'? |
It's my typo error. It should be 'known-dependencies.txt'. |
I found it and I'll try again. Thanks |
@xwm1992 please review again, thanks |
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
Fixes #4415.
Motivation
Add Promethus source connector.
Modifications
Add new source connector module for prometheus
Documentation