Skip to content
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

Add prometheus provider #3555

Merged
merged 1 commit into from
Jun 5, 2022
Merged

Conversation

Lucaber
Copy link
Contributor

@Lucaber Lucaber commented May 31, 2022

Allows to load a scalar metrics from prometheus

Example config:

meters:
  - name: pv
    type: custom
    power:
      source: prometheus
      address: http://prometheus:9090
      query: "scalar(p_realtime)"

provider/prometheus.go Outdated Show resolved Hide resolved
provider/prometheus.go Outdated Show resolved Hide resolved
provider/prometheus.go Outdated Show resolved Hide resolved
provider/prometheus.go Outdated Show resolved Hide resolved
provider/prometheus.go Outdated Show resolved Hide resolved
@andig
Copy link
Member

andig commented Jun 3, 2022

Danke für den PR! Ich denke die Lösung richtet sich eher an die Techniker als an Anwender, ist aber auch minimalinvasiv umsetzbar- das würde ich sehr gerne unterstützen.

@andig andig added the enhancement New feature or request label Jun 3, 2022
@Lucaber Lucaber force-pushed the feature/provider-prometheus branch 2 times, most recently from a7e11e1 to cb8fa1d Compare June 4, 2022 13:21
@Lucaber Lucaber requested a review from andig June 4, 2022 13:22
@Lucaber
Copy link
Contributor Author

Lucaber commented Jun 4, 2022

Danke für den PR! Ich denke die Lösung richtet sich eher an die Techniker als an Anwender, ist aber auch minimalinvasiv umsetzbar- das würde ich sehr gerne unterstützen.

Techniker möchtes es gerne auch anwenden :) Ich schiebe bei mir alles ins Prometheus, auch gibt mein Sromzähler leider keinen aktuellen Verbrauch zurück, durch eine rate() query im Prometheus jedoch einfach auslesbar/berechenbar.
Ich könnte evcc leider noch nicht mit einer Wallbox testen, da der eebus support der VW Wallbox noch sehr frisch ist und hier wohl schon andere dran sind.
Sromzähler und PV laufen aber schon, leider nur etwas verzögert (bis zu 30sec-10minuten :( ), mal schauen wie gut evcc damit klar kommt.

provider/prometheus.go Outdated Show resolved Hide resolved
Allows to load a scalar metrics from prometheus

Example config:
```
meters:
  - name: pv
    type: custom
    power:
      source: prometheus
      address: http://prometheus:9090
      query: "scalar(p_realtime)"
```
@Lucaber Lucaber force-pushed the feature/provider-prometheus branch from cb8fa1d to 5865cdb Compare June 4, 2022 15:06
@andig andig merged commit c5f6ffc into evcc-io:master Jun 5, 2022
@andig
Copy link
Member

andig commented Jun 5, 2022

Danke für den PR! Bitte letzte Änderung beachten- address ist jetzt wie sonst auch uri.

@Lucaber
Copy link
Contributor Author

Lucaber commented Jun 5, 2022

Ich sehe keine letzte Änderung, aktuell wird die Prometheus config mit api.Config geladen
c5f6ffc#diff-be5ede6f2bcf69c0b1a22cb558ba1a12c243621f99b4ff9e0c315a725cfa2fdbR31

@andig
Copy link
Member

andig commented Jun 5, 2022

Ich seh auch warum:

Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
To https://github.com/Lucaber/evcc.git
 ! [remote rejected]   feature/provider-prometheus -> feature/provider-prometheus (permission denied)
error: failed to push some refs to 'https://github.com/Lucaber/evcc.git'

Ich repariers in master.

@Lucaber Lucaber deleted the feature/provider-prometheus branch June 6, 2022 09:14
dontbyte pushed a commit to dontbyte/evcc that referenced this pull request Aug 2, 2022
Allows to load a scalar metrics from prometheus

Example config:
```
meters:
  - name: pv
    type: custom
    power:
      source: prometheus
      uri: http://prometheus:9090
      query: "scalar(p_realtime)"
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants