Kubeadm: Mova todas as constantes para um único `const (`

Criado em 11 fev. 2019 · 4Comentários · Fonte: kubernetes/kubeadm

Mova todos os const do arquivo cmd/kubeadm/app/cmd/options/constant.go para um único const (

ie:

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

/ tipo limpeza
/ lista de pendências de prioridade
/ajuda
/ good-first-issue

good first issue help wanted kincleanup lifecyclactive prioritbacklog

Fonte

yagonobre

👍3 🚀1

Comentários muito úteis

Olá @yagonobre , gostaria de trabalhar neste assunto.

Você pode me dizer como posso testar isso depois de terminar de mover todas as constantes em uma const ?

Obrigada.

rumshenoy em 11 fev. 2019

🚀2

Todos 4 comentários

@yagonobre :
Esta solicitação foi marcada como adequada para novos colaboradores.

Certifique-se de que a solicitação atenda aos requisitos listados aqui .

Se esta solicitação não atender mais a esses requisitos, o rótulo pode ser removido
comentando com o comando /remove-good-first-issue .

Em resposta a isso :

Mova todos os const do arquivo cmd/kubeadm/app/cmd/options/constant.go para um único const (

ie:

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

/ tipo limpeza
/ lista de pendências de prioridade
/ajuda
/ good-first-issue

Instruções para interagir comigo usando comentários de RP estão disponíveis aqui . Se você tiver dúvidas ou sugestões relacionadas ao meu comportamento, registre um problema no repositório kubernetes / test-infra .

k8s-ci-robot em 11 fev. 2019

o markdown não funciona para o título: smile:

MalloZup em 11 fev. 2019

😄2

Olá @yagonobre , gostaria de trabalhar neste assunto.

Você pode me dizer como posso testar isso depois de terminar de mover todas as constantes em uma const ?

Obrigada.

rumshenoy em 11 fev. 2019

🚀2

Obrigado @rumshenoy ,
você pode executar os testes ./build/run.sh make test WHAT=./cmd/kubeadm/app/cmd/ e tentar construir o kubeadm ./build/run.sh make build kubeadm

/ lifecycle ativo

yagonobre em 11 fev. 2019

👍1

Esta página foi útil?

0 / 5 - 0 avaliações