sensu / uchiwa-chef

Other
33 stars 40 forks source link

this part of the config is now an array of hashes #35

Closed slyness closed 9 years ago

slyness commented 9 years ago

Cookbook was creating invalid uchiwa config file..

previously creating

{
  "uchiwa": {
    "user": "admin",
    "pass": "supersecret",
    "refresh": 5,
    "host": "0.0.0.0",
    "port": 3000
  },
  "sensu":  {
      "name": "Sensu",
      "host": "localhost",
      "url": "http://localhost:4567"
    }
}

However, as the documentation states the "sensu" section should now be an array of hashes.

{
  "uchiwa": {
    "user": "admin",
    "pass": "supersecret",
    "refresh": 5,
    "host": "0.0.0.0",
    "port": 3000
  },
  "sensu": [
    {
      "name": "Sensu",
      "host": "localhost",
      "url": "http://localhost:4567"
    }
  ]
}

http://docs.uchiwa.io/en/latest/configuration/examples/#complete-configuration

portertech commented 9 years ago

@slyness thanks dawg :+1:

@amdprophet have time tomorrow to build on this to support multiple Sensu APIs (DCs)?

amdprophet commented 9 years ago

@slyness thanks! I'll merge this in tomorrow.

@portertech yes

tas50 commented 9 years ago

@amdprophet Can this get merged and released?

amdprophet commented 9 years ago

Merging and will fix the test suites myself. Release will come shortly after.