Kubeadm: Переместите все константы в один `const (`

Созданный на 11 февр. 2019 · 4Комментарии · Источник: kubernetes/kubeadm

Переместить все const в файле cmd/kubeadm/app/cmd/options/constant.go в один 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.
- 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"
+)

/ добрый убор
/ приоритет отставания
/помощь
/ хороший-первый-выпуск

good first issue help wanted kincleanup lifecyclactive prioritbacklog

Источник

yagonobre

👍3 🚀1

Самый полезный комментарий

Привет, @yagonobre , я хотел бы поработать над этим вопросом.

Можете ли вы сказать мне, как я могу проверить это после того, как переместил все константы в один const ?

Спасибо.

rumshenoy 11 февр. 2019

🚀2

Все 4 Комментарий

@yagonobre :
Этот запрос был отмечен как подходящий для новых участников.

Убедитесь, что запрос соответствует перечисленным здесь требованиям.

Если этот запрос больше не соответствует этим требованиям, ярлык может быть удален.
путем комментирования с помощью команды /remove-good-first-issue .

В ответ на это :

Переместить все const в файле cmd/kubeadm/app/cmd/options/constant.go в один 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.
- 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"
+)

/ добрый убор
/ приоритет отставания
/помощь
/ хороший-первый-выпуск

Инструкции по взаимодействию со мной с помощью PR-комментариев доступны здесь . Если у вас есть вопросы или предложения, связанные с моим поведением, отправьте сообщение о проблеме в репозиторий kubernetes / test-infra .

k8s-ci-robot 11 февр. 2019

уценка не работает для заголовка: smile:

MalloZup 11 февр. 2019

😄2

Привет, @yagonobre , я хотел бы поработать над этим вопросом.

Можете ли вы сказать мне, как я могу проверить это после того, как переместил все константы в один const ?

Спасибо.

rumshenoy 11 февр. 2019

🚀2

Спасибо @rumshenoy ,
вы можете запустить тесты ./build/run.sh make test WHAT=./cmd/kubeadm/app/cmd/ и попробовать собрать kubeadm ./build/run.sh make build kubeadm

/ жизненный цикл активен

yagonobre 11 февр. 2019

👍1

Была ли эта страница полезной?

0 / 5 - 0 рейтинги