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

Move InfluxDB configuration to it's own part of the Butler config file #670

Closed
mountaindude opened this issue May 28, 2023 · 0 comments · Fixed by #669, #682, #685, #690 or #691
Closed

Move InfluxDB configuration to it's own part of the Butler config file #670

mountaindude opened this issue May 28, 2023 · 0 comments · Fixed by #669, #682, #685, #690 or #691

Comments

@mountaindude
Copy link
Collaborator

mountaindude commented May 28, 2023

🚨 This will be a breaking change.

Background

Previously the only use of InfluxDB in Butler has been to store uptime and Butler memory usage information.
With the addition of monitoring of Windows services that changes.

As there are now several parts of the Butler config file that use InfluxDB settings, those settings should be in their own influxDb section under the root Butler section in the YAML config file

Implementation

Conceptual YAML config file:

Butler:
  uptimeMonitor:
    storeInInfluxdb: 
      enable: true

  serviceMonitor:
    alertDestination:
      influxDb:                     # Send service alerts to InfluxDB
        enable: true

  influxDb:
    enable: true
    hostIP: 192.168.100.20
    hostPort: 8086
    auth:
      enable: false
      username: user_joe
      password: joesecret
    dbName: butler
    instanceTag: DEV              # Tag that can be used to differentiate data from multiple Butler instances
    # Default retention policy that should be created in InfluxDB when Butler creates a new database there. 
    # Any data older than retention policy threshold will be purged from InfluxDB.
    retentionPolicy:
      name: 10d
      duration: 10d   
mountaindude pushed a commit to mountaindude/butler that referenced this issue May 28, 2023
Implements ptarmiganlabs#670

feat: Add InfluxDB as destination for Windows service status monitoring
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment