Kubeadm: Verschieben Sie alle Konstanten in eine einzelne `const (`

Erstellt am 11. Feb. 2019  ·  4Kommentare  ·  Quelle: kubernetes/kubeadm

Verschieben Sie alle const in der Datei cmd/kubeadm/app/cmd/options/constant.go in ein einzelnes const (

dh:

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

/ Art Aufräumen
/ Prioritätsstau
/Hilfe
/ gute erste Ausgabe

good first issue help wanted kincleanup lifecyclactive prioritbacklog

Hilfreichster Kommentar

Hallo @yagonobre , ich würde gerne an diesem Thema arbeiten.

Können Sie mir sagen, wie ich dies testen kann, nachdem ich alle Konstanten in ein const verschoben habe?

Vielen Dank.

Alle 4 Kommentare

@yagonobre :
Diese Anfrage wurde als für neue Mitwirkende geeignet markiert.

Bitte stellen Sie sicher, dass die Anfrage die hier aufgeführten Anforderungen erfüllt.

Wenn diese Anforderung diese Anforderungen nicht mehr erfüllt, kann das Etikett entfernt werden
durch Kommentieren mit dem Befehl /remove-good-first-issue .

Als Antwort darauf :

Verschieben Sie alle const in der Datei cmd/kubeadm/app/cmd/options/constant.go in ein einzelnes const (

dh:

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

/ Art Aufräumen
/ Prioritätsstau
/Hilfe
/ gute erste Ausgabe

Anweisungen zur Interaktion mit mir mithilfe von PR-Kommentaren finden Sie hier . Wenn Sie Fragen oder Anregungen zu meinem Verhalten haben, reichen Sie bitte ein Problem mit dem Repository

Markdown funktioniert nicht für Titel: smile:

Hallo @yagonobre , ich würde gerne an diesem Thema arbeiten.

Können Sie mir sagen, wie ich dies testen kann, nachdem ich alle Konstanten in ein const verschoben habe?

Vielen Dank.

Danke @rumshenoy ,
Sie können die Tests ./build/run.sh make test WHAT=./cmd/kubeadm/app/cmd/ ausführen und versuchen, kubeadm ./build/run.sh make build kubeadm zu erstellen

/ Lebenszyklus aktiv

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen