latenighttales / alcali

Featureful Saltstack GUI
https://alcali.dev
MIT License
356 stars 61 forks source link

Minion listing should not be coupled with Alcali key db #326

Closed mattLLVW closed 3 years ago

mattLLVW commented 3 years ago

We should just do something like test.ping to list minions connected See: https://github.com/latenighttales/alcali/blob/a78052d1bb28f7a5eac467921252a92e345c90c1/api/views/alcali.py#L115-L119

ggiesen commented 3 years ago

I haven't delved into the code, but is there any reason to not use the manage.status runner?

mattLLVW commented 3 years ago

@ggiesen manage.status is publishing test.ping and needs @runner permissions.