Add more descriptive Message to DrainingFailedReason and DrainingReason

[adilGhaffarDev]

What would you like to be added (User Story)?

As a user/operator when draining I would like to have more descriptive message in the machine status.

Detailed Description

In DrainingReason we can show in the message which pod is being evicted right now we are getting this information from the callback OnPodDeletionOrEvictionStarted and similarly when draining fails we can also get the information of failed to evict pod from OnPodDeletionOrEvictionFinished.

Anything else you would like to add?

Right now we show information regarding evicted pod in the machine controller logs using OnPodDeletedOrEvicted callback. Showing more info in the status using other callbacks(OnPodDeletionOrEvictionStarted and OnPodDeletionOrEvictionFinished) will help end user more, checking logs is bit cumbersome.

Label(s) to be applied

/kind feature

[k8s-ci-robot]

This issue is currently awaiting triage.

If CAPI contributors determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[fabriziopandini]

Generally, +1 if we get that info back from the drain library and this won't be too noisy
Less also links the previous issue #7887 and if possible explains the difference with the current one

/triage accepted

[adilGhaffarDev]

Generally, +1 if we get that info back from the drain library

We are getting this information now. It was changed in this PR: kubernetes/kubernetes#117502

this won't be too noisy

I need to check this.

Less also links the previous issue #7887 and if possible explains the difference with the current one

This PR was asking for too much, as discussed there I agree with Fabrizio we don't need this much information regarding pods on machine status. Just more descriptive Message on the reason should be enough.

[adilGhaffarDev]

/assign

[fabriziopandini]

/priority backlog

[sbueringer]

Just to keep an overview over the implementation + docs etc.:

• ✨ Refactor node drain #11074
• 📖 Add Node drain documentation #11121
• 🌱 Improve node drain e2e test #11127