smcavallo

Results 20 comments of smcavallo

> > > Thanks for the pull request. I haven't tried running it, but isn't this similar to what `coverage debug config` shows? If so, maybe we need to make...

> > > Can you elaborate, why these updates are needed for the plugin? > > They are not packaged so it will always use the latest installed version afaik....

@majormoses - The current dependency tree for the sensu cookbook looks like this -> ```sensu - windows - ms_dotnet - windows - rabbitmq - dpkg_autostart - logrotate - erlang -...

@majormoses - yes i'd love for those dependencies to be 'suggests' instead of 'depends' and would give us the flexibility to include/exclude those dependencies in our wrapper cookbooks. afaik the...

If acceptable, i've created a PR for the change from depends to suggests. This is a breaking change. Since the default is now opt-out, consumers would need to implement the...

@tas50 and @majormoses - agreed - closed that PR

@majormoses - agreed with the above, as a consumer, we can live with this 'as-is' for now as sensu 2.0 is coming down the pipeline. would it make sense for...

Many provisioning tools like `cloudformation` and `terraform` allow managing/associating security groups. The use case here is for ec2 instances which are not provisioned with any of those tools. I would...

@majormoses - here is our use case where this cookbook could actually improve security. We'd like this cookbook to keep our ingress/egress rules in sync. There is a possibility that...

@majormoses - See https://github.com/chef-cookbooks/aws/pull/381 Feel free to modify as you feel appropriate.