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

Confusing info box in the Operator documentation #9

Open
jerrinot opened this issue Jan 3, 2022 · 4 comments
Open

Confusing info box in the Operator documentation #9

jerrinot opened this issue Jan 3, 2022 · 4 comments
Assignees

Comments

@jerrinot
Copy link

jerrinot commented Jan 3, 2022

The Getting Started guide has this infobox:
image

I find the text confusing. When it can be inaccurate? Is it just "check-then-act"? What does "informational purposes" means exactly? What is the use-case where I can use this field and when I should NOT rely on it?

@jerrinot jerrinot transferred this issue from another repository Jan 4, 2022
@SeriyBg
Copy link
Contributor

SeriyBg commented Jan 4, 2022

@jerrinot the reason for this is that readyMembers are listening for the MembershipStateChangeds which are fire-and-forget events. Thus we do not recommend to thread it a reliable source of truth. But I agree that this can be described more thoroughly

@JakeSCahill
Copy link
Contributor

How does this affect users?

In what circumstances would this value be inaccurate?

What more accurate ways do we offer to check how many active members are in the cluster?

@SeriyBg
Copy link
Contributor

SeriyBg commented Feb 10, 2022

@JakeSCahill I don't think there is a precise scenario when this value is inaccurate. The more accurate data will be provided in the future with the implementation of advanced cluster status

@JakeSCahill
Copy link
Contributor

So the potential for a user to see an inaccurate value there is negligible?

Maybe the best course of action is to remove the warning.

@rebekah-lawrence rebekah-lawrence self-assigned this Feb 28, 2024
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

No branches or pull requests

4 participants