dashboard icon indicating copy to clipboard operation
dashboard copied to clipboard

Add Seed Readiness

Open tedteng opened this issue 4 years ago • 10 comments

What would you like to be added:

Hi, @jguipi can you elaborate/update the idea as you mention from the DoD log. you want to add a Seed status for Operator Role in Readiness correct me if I am missing something .

image

Why is this needed: The Readiness display API, control plane, node, and SC but without Seed status. sometimes operator takes most of the attention to check the issue start from the shoot cluster/CP when the issue raised at the beginning. However, the root cause may come from the Seed level (eg quota issue or others. ) Whether we have Readiness for Seed on the Readiness will help for people for draft troubleshooting at the beginning.

tedteng avatar Apr 19 '21 10:04 tedteng

You described it well @tedteng! It will help DOD pinpoint the root cause of an issue faster and provide good information at the dashboard level. We just need to make sure that this Readiness badge is only visible to DOD to not alarm the customer if they see some red, and it doesn’t affect their cluster.

jguipi avatar Apr 22 '21 14:04 jguipi

Yes this will be an operator only feature, as we will need to access seed data which should not be visible to regular users.

grolu avatar Apr 22 '21 15:04 grolu

@dkistner @dguendisch @rfranzke I guess this would be feasible - at least for the all projects view, as in this case we have the seed status (in case it is a managed seed). Do you think this is a valuable feature?

grolu avatar Jun 02 '22 15:06 grolu

qq: so it would be visible in all projects but not in a dedicated shoot view? It would be better than nothing but is there anything making the dedicated shoot view more complicated to have a seed readiness chip as well?

dguendisch avatar Jun 19 '22 14:06 dguendisch

We can also have it on the should details page. We just need to think how we can get the seed status as we are not subscribed to it in this case. The question is if this feature really adds value. If yes, we will proceed and come up with an implementation proposal. But if you consider it as not really useful I would consider closing the ticket as this feature will require some effort so I would not classify it as a "quick win"

grolu avatar Sep 13 '22 07:09 grolu

The question is if this feature really adds value.

Personally I'd see it as a nice to have. It would speed up some few cases by some seconds => nice. But not needed at every price, so if this one is more complicated I think we can also live without it.

dguendisch avatar Sep 13 '22 07:09 dguendisch

Thank you for your assessment. I will look into it time-boxed. Depending on the effort we will postpone it.

grolu avatar Sep 13 '22 08:09 grolu

On the shoot detail page we have the information:

Screen Shot 2022-09-13 at 17 15 37 Screen Shot 2022-09-13 at 17 16 09

For the cluster list we currently do not have the status of the seed. But I guess we need it there, too?!

grolu avatar Sep 13 '22 15:09 grolu

The shoot detail page looks very nice! For the cluster list it would be nice, again not at every dev effort, but nice to have.

dguendisch avatar Sep 13 '22 15:09 dguendisch

Ok. Good to hear that we understood your requirement. However, in the PoC the seed readiness is not reactive and we do not have it on the shoot list. In order to have this, we need to watch the seed resource in the frontend. Doing so, we can also show the seed resources to operators. We first need to do some changes to our backend connection. We will continue with this ticket once we have implemented the foundation.

grolu avatar Sep 14 '22 12:09 grolu