wcannon / salt-check

Other
12 stars 1 forks source link

Current client handling is unix specific #3

Closed mchugh19 closed 6 years ago

mchugh19 commented 6 years ago

https://github.com/wcannon/salt-check/blob/343e2105aaf53473512c7600ce15a46ff7d812ee/salt/_modules/saltcheck.py#L308

I don't have a development environment handy to submit a PR, but this could instead read:

        local_opts = salt.config.minion_config(__opts__['conf_file'])
        local_opts['file_client'] = 'local'
        self.salt_lc = salt.client.Caller(mopts=local_opts)

This both removes the linux file hardcoding as well as doesn't overwrite the current __opts__

mchugh19 commented 6 years ago

Fixed in #4