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

bug: KubeSteller setup scripts do not wait for KubeFlex control plane sync #2074

Open
MikeSpreitzer opened this issue Apr 23, 2024 · 6 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/medium Must be staffed and worked on either currently, or very soon, ideally in time for the next release.

Comments

@MikeSpreitzer
Copy link
Contributor

Describe the bug

A KubeFlex ControlPlane is like a Kubernetes Pod or CustomResourceDefinition --- merely creating the API object is not enough, you generally have to wait for the asynchronous processing to finish. For example, kflex create ... -p SomePCH will complete successfully before any attempt is even made to find or apply the named PCH. See kubestellar/kubeflex#235 .

Steps To Reproduce

This is from code examination.

Expected Behavior

Waiting where completion of asynchronous processing is needed.

Additional Context

No response

@MikeSpreitzer MikeSpreitzer added the kind/bug Categorizes issue or PR as related to a bug. label Apr 23, 2024
@MikeSpreitzer
Copy link
Contributor Author

/cc @pdettori
/cc @francostellari
/cc @dumb0002
/cc @effi-ofer

@effi-ofer
Copy link
Contributor

The bottom line is that we need to wait both for the SYNCED and the READY to be set to true?

@pdettori
Copy link
Contributor

That's necessary but not sufficient. "SYNCED" will only tell if the templates have been applied without errors, but not if the objects encoded in those templates are up and running. While kflex can tell if a control plane is up checking the status of the API server, it cannot tell if arbitrary objects part of a PCH are up and running. Consider the case of the OCM PCH for example, there is no general way to know that OCM is up and running unless we write use-case specific logic for it.

@pdettori
Copy link
Contributor

Note that even helm (as an example) will not tell you if all objects in the release are in a READY state. It will only tell if all objects have been applied without errors.

@MikeSpreitzer
Copy link
Contributor Author

MikeSpreitzer commented Apr 24, 2024

The good news here is that our setup-kubestellar.sh script has constant (known to us developers) needs from the PCHs and so can specifically wait for that.

@ezrasilvera ezrasilvera added the priority/medium Must be staffed and worked on either currently, or very soon, ideally in time for the next release. label Apr 27, 2024
@ezrasilvera
Copy link
Contributor

@effi-ofer Any update on this ? (CC: @pdettori )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/medium Must be staffed and worked on either currently, or very soon, ideally in time for the next release.
Projects
Status: No status
Development

No branches or pull requests

4 participants