latenighttales/alcali

Minion listing should not be coupled with Alcali key db

mattLLVW opened this issue · 2 comments

We should just do something like test.ping to list minions connected
See:

alcali/api/views/alcali.py

Lines 115 to 119 in a78052d

accepted_minions = Keys.objects.filter(status="accepted").values_list(
"minion_id", flat=True
)
for minion in accepted_minions:
ret = refresh_minion(minion)

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

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