Adding ipmi port to BaseboardManagement type

api/v1alpha1/baseboardmanagement_types.go

@@ -64,6 +64,10 @@ type Connection struct {
 	// +kubebuilder:validation:MinLength=1
 	Host string `json:"host"`
 
+	// Port is the IPMI port of the BaseboardManagement.
+	// +kubebuilder:validation:MinLength=1
+	Port string `json:"port"`
+
 	// AuthSecretRef is the SecretReference that contains authentication information of the BaseboardManagement.
 	// The Secret must contain username and password keys.
 	AuthSecretRef corev1.SecretReference `json:"authSecretRef"`

config/crd/bases/bmc.tinkerbell.org_baseboardmanagements.yaml

@@ -65,10 +65,15 @@ spec:
                   insecureTLS:
                     description: InsecureTLS specifies trusted TLS connections.
                     type: boolean
+                  port:
+                    description: Port is the IPMI port of the BaseboardManagement.
+                    minLength: 1
+                    type: string
                 required:
                 - authSecretRef
                 - host
                 - insecureTLS
+                - port
                 type: object
               power:
                 description: Power is the desired power state of the BaseboardManagement.

config/samples/bmc_v1alpha1_baseboardmanagement.yaml

@@ -5,6 +5,7 @@ metadata:
 spec:
   connection:
     host: 0.0.0.0
+    port: 623
     authSecretRef:
       name: bm-auth
       namespace: sample

controllers/baseboardmanagement_controller.go

@@ -114,9 +114,8 @@ func (r *BaseboardManagementReconciler) reconcile(ctx context.Context, bm *bmcv1
 		return ctrl.Result{Requeue: true}, fmt.Errorf("resolving BaseboardManagement %s/%s SecretReference: %v", bm.Namespace, bm.Name, err)
 	}
 
-	// TODO (pokearu): Remove port hardcoding
 	// Initializing BMC Client
-	bmcClient, err := r.bmcClientFactory(ctx, bm.Spec.Connection.Host, "623", username, password)
+	bmcClient, err := r.bmcClientFactory(ctx, bm.Spec.Connection.Host, bm.Spec.Connection.Port, username, password)
 	if err != nil {
 		logger.Error(err, "BMC connection failed", "host", bm.Spec.Connection.Host)
 		result, setConditionErr := r.setCondition(ctx, bm, bmPatch, bmcv1alpha1.ConnectionError, err.Error())

controllers/baseboardmanagement_controller_test.go

@@ -221,6 +221,7 @@ func getBaseboardManagement() *bmcv1alpha1.BaseboardManagement {
 		Spec: bmcv1alpha1.BaseboardManagementSpec{
 			Connection: bmcv1alpha1.Connection{
 				Host: "0.0.0.0",
+				Port: "623",
 				AuthSecretRef: corev1.SecretReference{
 					Name:      "test-bm-auth",
 					Namespace: "test-namespace",