search

maistra / istio-workspace

Safely develop and test on any Kubernetes cluster without affecting others.
Apache License 2.0
57 stars 14 forks source link

Validate cluster-side installation #331

Open aslakknutsen opened 4 years ago

aslakknutsen commented 4 years ago
What can be improved

Check if the operator is installed in the current namespace/cluster. If it's not the case we should present a solution to the user by e.g. pointing to docs or steps required to fix the problem.

Why

Be helpful. :)

aslakknutsen commented 4 years ago

.. wouldn't we know this when we get an Error from trying to apply a Kind:Session that does not exist?

bartoszmajsak commented 4 years ago

Review for an epic related ux for cli

bartoszmajsak commented 2 years ago

@aslakknutsen We removed ike install command in favor of the operator bundle. Shall we close this issue or offer a hint on how to install our stuff in case of error?

aslakknutsen commented 2 years ago

Yeah, that sounds better.

aslakknutsen commented 2 years ago

Let's detect a failure to apply Session and give a hint to why. That should at least detect missing Session CRD due to missing Operator install.

I'm wondering if maybe an auth issue could trigger the exact same condition, but not sure. We'll have to test it. If you don't have the role to create a session does that show up as a session object doesn't exist as well