kubernetes-sigs/cluster-api

Return CLC conversion warnings to user using warnings of validation webhook

Ouverte

#8 944 ouverte le 30 juin 2023

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)Go (1 532 forks)auto 404
help wantedkind/featurepriority/backlogtriage/accepted

Métriques du dépôt

Stars
 (4 267 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

As a user I would like to be informed while creating a cluster using Ignition that my configuration has warnings.

Detailed Description

As a follow up to #4279, now that we have warnings support in validation webhooks, we should make CLC conversion code added in https://github.com/kubernetes-sigs/cluster-api/pull/4172#discussion_r584669828 return warnings to the user.

Anything else you would like to add?

No response

Label(s) to be applied

/kind feature One or more /area label. See https://github.com/kubernetes-sigs/cluster-api/labels?q=area for the list of labels.

Guide contributeur

Direction de recherche
Examinez le code de conversion CLC existant et la gestion des avertissements dans les webhooks de validation. Modifiez la fonction de conversion pour inclure les avertissements dans la réponse du webhook, en veillant à ce qu'ils soient affichés à l'utilisateur.
Stack technique
go
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoKubernetesCluster API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70