Kubeadm: Pindahkan semua konstanta ke satu `const (`

Dibuat pada 11 Feb 2019  ·  4Komentar  ·  Sumber: kubernetes/kubeadm

Pindahkan semua const pada file cmd/kubeadm/app/cmd/options/constant.go ke satu const (

yaitu:

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

/ kind cleanup
/ prioritas backlog
/Tolong
/ good-first-issue

good first issue help wanted kincleanup lifecyclactive prioritbacklog

Komentar yang paling membantu

Hai @yagonobre , saya ingin menangani masalah ini.

Dapatkah Anda memberi tahu saya bagaimana saya dapat menguji ini setelah saya selesai memindahkan semua konstanta menjadi satu const ?

Terima kasih.

Semua 4 komentar

@yagonob :
Permintaan ini telah ditandai sebagai sesuai untuk kontributor baru.

Harap pastikan permintaan tersebut memenuhi persyaratan yang tercantum di sini .

Jika permintaan ini tidak lagi memenuhi persyaratan tersebut, label dapat dihapus
dengan berkomentar menggunakan perintah /remove-good-first-issue .

Menanggapi ini :

Pindahkan semua const pada file cmd/kubeadm/app/cmd/options/constant.go ke satu const (

yaitu:

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

/ kind cleanup
/ prioritas backlog
/Tolong
/ good-first-issue

Instruksi untuk berinteraksi dengan saya menggunakan komentar PR tersedia di sini . Jika Anda memiliki pertanyaan atau saran terkait dengan perilaku saya, harap ajukan masalah ke kubernetes / test-infra repository.

penurunan harga tidak berfungsi untuk title: smile:

Hai @yagonobre , saya ingin menangani masalah ini.

Dapatkah Anda memberi tahu saya bagaimana saya dapat menguji ini setelah saya selesai memindahkan semua konstanta menjadi satu const ?

Terima kasih.

Terima kasih @rumshenoy ,
Anda dapat menjalankan pengujian ./build/run.sh make test WHAT=./cmd/kubeadm/app/cmd/ , dan mencoba membangun kubeadm ./build/run.sh make build kubeadm

/ siklus hidup aktif

Apakah halaman ini membantu?
0 / 5 - 0 peringkat