search

sensu-plugins / sensu-plugins-kubernetes

Sensu plugins for Kubernetes
http://sensu-plugins.io
MIT License
28 stars 36 forks source link

add authentication options #14

Closed mickfeech closed 8 years ago

mickfeech commented 8 years ago

Pull Request Checklist

Is this in reference to an existing issue?

General

Checks fail if SSL is enabled and security is enabled on the API service

Known Compatablity Issues

sstarcher commented 8 years ago

@mickfeech could you pull the common code out into a lib and squash your commits.

An example would be https://github.com/sensu-plugins/sensu-plugins-aws/blob/master/lib/sensu-plugins-aws/common.rb

mickfeech commented 8 years ago

@sstarcher They are really just options being passed from the command line.

sstarcher commented 8 years ago

they are also 20 lines of options

sstarcher commented 8 years ago

I believe these changes are superseded by @GKTheOne added a pull request. If I'm incorrect free free to reopen the pull request.