Kubeadm: Déplacer toutes les constantes vers un seul `const (`

Créé le 11 févr. 2019 · 4Commentaires · Source: kubernetes/kubeadm

Déplacer tous les const du fichier cmd/kubeadm/app/cmd/options/constant.go vers un seul const (

c'est à dire:

- // APIServerAdvertiseAddress flag sets the IP address the API Server will advertise it's listening on. Specify '0.0.0.0' to use the address of the default network interface.
- const APIServerAdvertiseAddress = "apiserver-advertise-address"

- // APIServerBindPort flag sets the port for the API Server to bind to.
- const APIServerBindPort = "apiserver-bind-port"
+ const (
+    // APIServerAdvertiseAddress flag sets the IP address the API Server will advertise it's listening on. Specify '0.0.0.0' to use the address of the default network interface.
+    APIServerAdvertiseAddress = "apiserver-advertise-address"
+
+    // APIServerBindPort flag sets the port for the API Server to bind to.
+    const APIServerBindPort = "apiserver-bind-port"
+)

/ gentil nettoyage
/ backlog prioritaire
/aider
/ good-first-issue

good first issue help wanted kincleanup lifecyclactive prioritbacklog

Source

yagonobre

👍3 🚀1

Commentaire le plus utile

Salut @yagonobre , j'aimerais travailler sur ce problème.

Pouvez-vous me dire comment je peux tester cela après avoir fini de déplacer toutes les constantes en un seul const ?

Merci.

rumshenoy le 11 févr. 2019

🚀2

Tous les 4 commentaires

@yagonobre :
Cette demande a été marquée comme adaptée aux nouveaux contributeurs.

Veuillez vous assurer que la demande répond aux exigences énumérées ici .

Si cette demande ne répond plus à ces exigences, l'étiquette peut être supprimée
en commentant avec la commande /remove-good-first-issue .

En réponse à cela :

Déplacer tous les const du fichier cmd/kubeadm/app/cmd/options/constant.go vers un seul const (

c'est à dire:

- // APIServerAdvertiseAddress flag sets the IP address the API Server will advertise it's listening on. Specify '0.0.0.0' to use the address of the default network interface.
- const APIServerAdvertiseAddress = "apiserver-advertise-address"

- // APIServerBindPort flag sets the port for the API Server to bind to.
- const APIServerBindPort = "apiserver-bind-port"
+ const (
+    // APIServerAdvertiseAddress flag sets the IP address the API Server will advertise it's listening on. Specify '0.0.0.0' to use the address of the default network interface.
+    APIServerAdvertiseAddress = "apiserver-advertise-address"
+
+    // APIServerBindPort flag sets the port for the API Server to bind to.
+    const APIServerBindPort = "apiserver-bind-port"
+)

/ gentil nettoyage
/ backlog prioritaire
/aider
/ good-first-issue

Les instructions pour interagir avec moi en utilisant les commentaires PR sont disponibles ici . Si vous avez des questions ou des suggestions concernant mon comportement, veuillez signaler un problème avec le

k8s-ci-robot le 11 févr. 2019

markdown ne fonctionne pas pour le titre: sourire:

MalloZup le 11 févr. 2019

😄2

Salut @yagonobre , j'aimerais travailler sur ce problème.

Pouvez-vous me dire comment je peux tester cela après avoir fini de déplacer toutes les constantes en un seul const ?

Merci.

rumshenoy le 11 févr. 2019

🚀2

Merci @rumshenoy ,
vous pouvez exécuter les tests ./build/run.sh make test WHAT=./cmd/kubeadm/app/cmd/ , et essayer de construire kubeadm ./build/run.sh make build kubeadm

/ cycle de vie actif

yagonobre le 11 févr. 2019

👍1

Cette page vous a été utile?

0 / 5 - 0 notes