-
Notifications
You must be signed in to change notification settings - Fork 31
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
Redis Master does not come up #8
Comments
Hi! Is there anything in logs of the operator? |
We had the same problem. We noticed that the probem happens when the instances have some data to load at start, while loading the dataset a redis instance is live (respond PONG to PING), ma does not accept commands. We solved adding the initialDelaySeconds and setting it to 120 seconds. |
thanks for the explanation @step76! What size is the dataset? I wonder if I could reproduce this behaviour. Also, your fork looks quite interesting! I examined it a bit and have a small question: why did you decide to add an init container for disabling transparent huge pages instead of doing the same via a DaemonSet with properly set up Pod Priorities? I mean there are multiple ways to deal with kernel settings. I am curious why you chose the |
Our dataset is about 1GB and on our GKE cluster takes 30/40 seconds to load on startup. I don't like the DaemonSet approach because it tends to introduce too many dependencies. For example, in the article you mentioned I don't like the approach of assigning a label to the node. initContainers guarantees the order of execution, moreover the POD dies before the main one starts and I have everything in a single YAML file. But now I want to investigate better the Pod Priorities that I didn't know about. If you want I can submit PR. |
yep, a PR would be great. I would be happy to know your findings about Pod Priorities as well. We are currently relying on them for ordered scheduling of pods. |
fixed in #9 |
I ran through the steps, but there are no redis instances available.
Additional Info
Can you pls take a look ?
Let me know if you need any logs/additional detail.
The text was updated successfully, but these errors were encountered: