Le nom ne dit pas ce que ca fait. OKDNodeScore ce n'est pas un nom assez descriptif. Quelque chose comme PrepareOKDNodeInstallationScore ou AddNodeToOKDClusterScore. Bref, je n'ai aucune idee de ce que ca fait, ca m'obliger a aller lire le code ce qui ralentis tout ceux qui vont lire ce code dans le futur.
On passe par le score et on call .execute dessus directement. On n'instance pas d'Interpret directement en dehors du get_interpret d'un score.
Fix this comment, not accurate in the context.
For now we can completely delete the storage role, it is not used. We will use worker nodes for storage, and we will autodetect their storage capabilities and orchestrate at a higher level than the OKD role.
Quick review done, overall looks pretty good but deserves some improvements, mostly naming.