add hostPort and container validation to webhook

Author: Cyril TOVENA

pkg/apis/stable/v1alpha1/types.go

@@ -190,6 +190,24 @@ func (gs *GameServer) ApplyDefaults() {
 func (gs *GameServer) Validate() (bool, []metav1.StatusCause) {
 	var causes []metav1.StatusCause
 
+	// make sure a name is specified when there is multiple containers in the pod.
+	if len(gs.Spec.Container) == 0 && len(gs.Spec.Template.Spec.Containers) > 1 {
+		causes = append(causes, metav1.StatusCause{
+			Type:    metav1.CauseTypeFieldValueInvalid,
+			Field:   "container",
+			Message: "Container is required when using multiple containers in the pod templace",
+		})
+	}
+
+	// no host port when using dynamic PortPolicy
+	if gs.Spec.HostPort > 0 && gs.Spec.PortPolicy == Dynamic {
+		causes = append(causes, metav1.StatusCause{
+			Type:    metav1.CauseTypeFieldValueInvalid,
+			Field:   "hostPort",
+			Message: "HostPort cannot be specified with a Dynamic PortPolicy",
+		})
+	}
+
 	// make sure the container value points to a valid container
 	_, _, err := gs.FindGameServerContainer()
 	if err != nil {

pkg/apis/stable/v1alpha1/types_test.go

@@ -180,14 +180,23 @@ func TestGameServerValidate(t *testing.T) {
 
 	gs = GameServer{
 		Spec: GameServerSpec{
-			Container: "nope",
+			Container:  "",
+			HostPort:   5001,
+			PortPolicy: Dynamic,
 			Template: corev1.PodTemplateSpec{
-				Spec: corev1.PodSpec{Containers: []corev1.Container{{Name: "testing", Image: "testing/image"}}}}},
+				Spec: corev1.PodSpec{Containers: []corev1.Container{
+					{Name: "testing", Image: "testing/image"},
+					{Name: "anothertest", Image: "testing/image"},
+				}}}},
 	}
 	ok, causes = gs.Validate()
+	fields := []string{}
+	for _, f := range causes {
+		fields = append(fields, f.Field)
+	}
 	assert.False(t, ok)
-	assert.Len(t, causes, 1)
-	assert.Equal(t, causes[0].Field, "container")
+	assert.Len(t, causes, 3)
+	assert.Contains(t, fields, "container", "hostPort")
 	assert.Equal(t, causes[0].Type, metav1.CauseTypeFieldValueInvalid)
 }