search

netdata / msi-installer

Netdata installer for Windows using WSL2
GNU General Public License v3.0
21 stars 2 forks source link

Fix /etc/netdata/go.d/wmi.conf #2

Closed cakrit closed 1 year ago

cakrit commented 1 year ago

It now says

jobs:
  - name: DESKTOP-C7OKV71
    url: http://127.0.0.1:9182/metrics

name should have the output of $(hostname) url should have the output of $(hostname).local

The localhost IP doesn't work in my case, not sure if it works anywhere, as it references the linux instance. 

DESKTOP-C7OKV71:/usr/libexec/netdata/plugins.d# ./go.d.plugin -d -m wmi_exporter
...
[ ERROR ] wmi[DESKTOP-C7OKV71] wmi.go:91 Get "http://127.0.0.1:9182/metrics": dial tcp 127.0.0.1:9182: connect: connection refused
[ ERROR ] wmi[DESKTOP-C7OKV71] job.go:191 check failed
cakrit commented 1 year ago

The new nightly has the improvement from https://github.com/netdata/netdata/issues/13716, so from tomorrow we shouldn't need this config file at all. Please remove it @dfpr

ilyam8 commented 1 year ago

Added in https://github.com/netdata/go.d.plugin/pull/941

jobs:
  - name: wsl-local
    url: http://hostname.local:9182/metrics

"hostname" will be replaced with hostname result when reading the config.

dfpr commented 1 year ago

The new nightly has the improvement from netdata/netdata#13716, so from tomorrow we shouldn't need this config file at all. Please remove it @dfpr

I will remove tomorrow and test then.

dfpr commented 1 year ago

Removed the file and tested it is working.