Skip to content

Conversation

@arthurzenika
Copy link

@arthurzenika arthurzenika commented Sep 8, 2017

and configure transport when pillar is used.

And enable configuration of redis sentinels.

Copy link

@EvaSDK EvaSDK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good but I wonder if having one sls, let's say transport.sls configuring transport.json, for handling transport configuration wouldn't help in reducing all these if's.

@powellchristoph
Copy link

@EvaSDK that sounds like a great idea. This formula was written a while ago and sensu has made changes. I wonder if a small refactor is in order.

@arthurzenika
Copy link
Author

@EvaSDK I agree transport could gain from being centralized. As long as redis (and sentinel) are options there, that's be great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants