search

sensu / sensu-omnibus

Build full-stack platform-specific Sensu packages
12 stars 16 forks source link

[REGRESSION] EMBEDDED_RUBY default changed? #252

Closed majormoses closed 6 years ago

majormoses commented 6 years ago

I am submitting this on behalf of a user that reported issues in https://github.com/sensu-plugins/sensu-plugins-mailer/issues/65 after troubleshooting we discovered EMBEDDED_RUBY=false this default changed to true in 0.21. According to the user they installed vanilla sensu 1.1.2 without any CM tools and this was defaulting to false.

cwjohnston commented 6 years ago

This hasn't changed.

See https://github.com/sensu/sensu-omnibus/blob/master/files/sensu-gem/default/sensu which is pulled in by https://github.com/sensu/sensu-omnibus/blob/master/config/software/sensu-gem.rb#L116.

majormoses commented 6 years ago

Ok well I guess I trusted the user that told me they did not touch it.

cwjohnston commented 6 years ago

@majormoses I apologize for seeming brusque in my response; I meant to respond to this question quickly but I realize now I was not very polite in doing so. Thanks for helping to surface potential issues like these. If the issue can be reproduced we certainly want to look into it.

majormoses commented 6 years ago

@cwjohnston no worries I understand and have been guilty of this in the past as well.