Skip to content
This repository has been archived by the owner on Mar 24, 2022. It is now read-only.

Complete states #38

Open
rominf opened this issue Sep 28, 2015 · 2 comments
Open

Complete states #38

rominf opened this issue Sep 28, 2015 · 2 comments

Comments

@rominf
Copy link
Contributor

rominf commented Sep 28, 2015

It would be nice to complete state names and arguments.

I propose to add variable:

set salt_minion=localhost

And then gather completions with:

salt $salt_minion sys.state_argspec
@basepi
Copy link
Contributor

basepi commented Sep 28, 2015

The problem is that rarely is the local minion actually called "localhost". We may have to force the user to set the local minion's ID as part of turning this feature on. (It should be off by default, for sure)

@rominf
Copy link
Contributor Author

rominf commented Sep 28, 2015

@basepi
OK, I wasn't to precise with my proposal. The idea was actually to let user choose minion ID.

Actually, I think it's even better to allow user to give full string to allow salt-call too:

set salt_states_completion_string="salt-call --local sys.states_argspec"

@rominf rominf closed this as completed Sep 28, 2015
@rominf rominf reopened this Sep 28, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants