Kubeadm: Mueva todas las constantes a un solo `const (`

Creado en 11 feb. 2019 · 4Comentarios · Fuente: kubernetes/kubeadm

Mover todos los const del archivo cmd/kubeadm/app/cmd/options/constant.go a un solo const (

es decir:

- // 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"
+)

/ limpieza amable
/ cartera de pedidos prioritaria
/ayuda
/ buena-primera-edición

good first issue help wanted kincleanup lifecyclactive prioritbacklog

Fuente

yagonobre

👍3 🚀1

Comentario más útil

Hola @yagonobre , me gustaría trabajar en este tema.

¿Puede decirme cómo puedo probar esto después de que termine de mover todas las constantes en un const ?

Gracias.

rumshenoy en 11 feb. 2019

🚀2

Todos 4 comentarios

@yagonobre :
Esta solicitud se ha marcado como adecuada para nuevos colaboradores.

Asegúrese de que la solicitud cumpla con los requisitos enumerados aquí .

Si esta solicitud ya no cumple con estos requisitos, la etiqueta se puede quitar
comentando con el comando /remove-good-first-issue .

En respuesta a esto :

Mover todos los const del archivo cmd/kubeadm/app/cmd/options/constant.go a un solo const (

es decir:

- // 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"
+)

/ limpieza amable
/ cartera de pedidos prioritaria
/ayuda
/ buena-primera-edición

Las instrucciones para interactuar conmigo usando comentarios de relaciones públicas están disponibles aquí . Si tiene preguntas o sugerencias relacionadas con mi comportamiento, presente un problema en el repositorio de kubernetes / test-infra .

k8s-ci-robot en 11 feb. 2019

markdown no funciona para el título: sonrisa:

MalloZup en 11 feb. 2019

😄2

Hola @yagonobre , me gustaría trabajar en este tema.

¿Puede decirme cómo puedo probar esto después de que termine de mover todas las constantes en un const ?

Gracias.

rumshenoy en 11 feb. 2019

🚀2

Gracias @rumshenoy ,
puede ejecutar las pruebas ./build/run.sh make test WHAT=./cmd/kubeadm/app/cmd/ e intentar construir kubeadm ./build/run.sh make build kubeadm

/ ciclo de vida activo

yagonobre en 11 feb. 2019

👍1

¿Fue útil esta página

0 / 5 - 0 calificaciones