Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lots of little fixes #502

Open
wants to merge 24 commits into
base: develop
Choose a base branch
from

Conversation

ITJamie
Copy link
Contributor

@ITJamie ITJamie commented Jun 18, 2023

we found several issues that made alcali unusable on a large salt instance.
the minions list or conformity would not load at all.
there was an assumption that it would also have a dict but some of the responses were strings (unhandled failures on the state.apply on minion side)

we have a postgres db with over 20gb of history that pointed out some of the load issues.

the "users" logic on search is the most broken piece, it does not scale and we removed it as it was unworkable. an indexing task to index jobs to pull out userid into its own column would help a lot

@mattLLVW mattLLVW mentioned this pull request Oct 20, 2023
@cyberplant
Copy link

cyberplant commented Jan 9, 2024

+1 to this PR, I've been trying alcali lately but been unable to get even one minion on the list. I see alcali gathering the data on the minions but nothing appears ever on the page, so I guess it's because of the problems mentioned here.

I have some hundreds minions on my salt master, increased all the timeouts that were failing initially but still nothing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants