Support for Graylog 1.1

[linuxprofessor]

Support for Graylog 1.1

• Changed variable mongodb_host to mongodb_hosts as it will now contain one or more hosts
• Added support for MongoDB URI instead of legacy MongoDB configuration method
• Removed the need for mongodb_port as this will be included in mongodb_hosts
• Changed development env to support Graylog 1.1 and ElasticSearch 1.5.2

[bernd]

@linuxprofessor Thank you very much for the PR.

I would like to keep backwards compatibility with 1.0 and suggest to keep the mongodb_host and mongodb_port options.

We can build the new mongodb_hosts option from them if they are in use. Also, when they are in use we can log a warning that these options are deprecated.

What do you think?

[linuxprofessor]

Could do that, yes. Is there a good way to determine which Graylog version that's used without the user specifying it? If graylog2::repo is used it's trivial, but I don't suppose one can count on everyone using it?

[linuxprofessor]

I've re-added support for v1.0 and updated README.md a bit.

Thinking about mongodb_replica_set, as it has totally different meanings if you're running the deprecated configuration method or the newer mongodb_uri method. Right now the content of that variable should be a number of hosts/ports in the old method and a replica set name in the new one. Can I leave it like this, or should they be separated?

[bernd]

Thanks for the updates. Good point with the replica set setting. I will think about that.

[bernd]

@linuxprofessor Thanks for you hard work! I merged it and will test a bit and then decide what to do with the replica set issue.

[bernd]

@linuxprofessor What do you think about just adding a mongodb_uri config option and deprecating the rest?

[linuxprofessor]

@bernd that would be alright, can prior versions handle mongodb_uri?

[bernd]

that would be alright, can prior versions handle mongodb_uri ?

No, older versions don't know about that. My idea is as follows.

Using mongodb_uri will add the mongodb_uri setting to the server config.

class { 'graylog::server':
  mongodb_uri => 'mongodb://localhost/graylog',
}

Using the old options like mongodb_database and mongodb_host etc. will just write those (deprecated) options to the server config.

class { 'graylog::server':
  mongodb_database => 'graylog',
  mongodb_host     => 'localhost',
}

Usage of any of the deprecated options will trigger a warning in the puppet log.

[linuxprofessor]

Go for it, seems reasonable.