Add OpenStackServerGroup CRD and Controller

Implements new CRD for OpenstackServerGroup in v1alpha8 to allow managed
Server Groups with standard policies, and adds ServerGroupRef to OpenstackMachine
that references the new CRD and uses it for VM creation.

Closes: #1256

PROJECT

@@ -42,18 +42,21 @@ resources:
   kind: OpenStackClusterTemplate
   version: v1alpha7
 - group: infrastructure
-  version: v1alpha8
   kind: OpenStackCluster
-- group: infrastructure
   version: v1alpha8
-  kind: OpenStackMachine
 - group: infrastructure
+  kind: OpenStackMachine
   version: v1alpha8
+- group: infrastructure
   kind: OpenStackMachineTemplate
+  version: v1alpha8
 - group: infrastructure
   kind: OpenStackClusterTemplate
   version: v1alpha8
 - group: infrastructure
   kind: OpenStackFloatingIPPool
   version: v1alpha1
+- group: infrastructure
+  kind: OpenstackServerGroup
+  version: v1alpha8
 version: "2"

api/v1alpha8/openstackmachine_types.go

@@ -88,6 +88,10 @@ type OpenStackMachineSpec struct {
 	// +optional
 	ServerGroup *ServerGroupFilter `json:"serverGroup,omitempty"`
 
+	// The server group ref to assign the machine to.
+	// +optional
+	ServerGroupRef *ServerGroupRef `json:"serverGroupRef,omitempty"`
+
 	// IdentityRef is a reference to a identity to be used when reconciling this cluster
 	// +optional
 	IdentityRef *OpenStackIdentityReference `json:"identityRef,omitempty"`

api/v1alpha8/openstackservergroup_types.go

@@ -0,0 +1,76 @@
+/*
+Copyright 2019 The Kubernetes Authors.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+	http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package v1alpha8
+
+import (
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+)
+
+const (
+	// ServerGroupFinalizer allows ReconcileOpenStackServerGroup to clean up OpenStack resources associated with OpenStackServerGroup before
+	// removing it from the apiserver.
+	ServerGroupFinalizer = "openstackservergroup.infrastructure.cluster.x-k8s.io"
+)
+
+// OpenStackServerGroupSpec defines the desired state of OpenStackServerGroup
+type OpenStackServerGroupSpec struct {
+	// The name of the cloud to use from the clouds secret
+	// +optional
+	CloudName string `json:"cloudName"`
+
+	// Policy is a string with some valid values; affinity, anti-affinity, soft-affinity, soft-anti-affinity.
+	Policy string `json:"policy"`
+
+	// IdentityRef is a reference to a identity to be used when reconciling this resource
+	// +optional
+	IdentityRef *OpenStackIdentityReference `json:"identityRef,omitempty"`
+}
+
+// OpenStackServerGroupStatus defines the observed state of OpenStackServerGroup
+type OpenStackServerGroupStatus struct {
+	// Ready is true when the resource is created.
+	// +optional
+	Ready bool `json:"ready"`
+
+	// UUID of provisioned ServerGroup
+	ID string `json:"uuid"`
+}
+
+//+kubebuilder:object:root=true
+//+kubebuilder:subresource:status
+
+// OpenStackServerGroup is the Schema for the openstackservergroups API
+type OpenStackServerGroup struct {
+	metav1.TypeMeta   `json:",inline"`
+	metav1.ObjectMeta `json:"metadata,omitempty"`
+
+	Spec   OpenStackServerGroupSpec   `json:"spec,omitempty"`
+	Status OpenStackServerGroupStatus `json:"status,omitempty"`
+}
+
+//+kubebuilder:object:root=true
+
+// OpenStackServerGroupList contains a list of OpenStackServerGroup
+type OpenStackServerGroupList struct {
+	metav1.TypeMeta `json:",inline"`
+	metav1.ListMeta `json:"metadata,omitempty"`
+	Items           []OpenStackServerGroup `json:"items"`
+}
+
+func init() {
+	objectTypes = append(objectTypes, &OpenStackServerGroup{}, &OpenStackServerGroupList{})
+}

api/v1alpha8/types.go

@@ -240,6 +240,12 @@ type ServerGroupFilter struct {
 	Name string `json:"name,omitempty"`
 }
 
+type ServerGroupRef struct {
+	// Name of the OpenStackServerGroup resource to be used.
+	// Must be in the same namespace as the resource(s) being provisioned.
+	Name string `json:"name"`
+}
+
 // BlockDeviceType defines the type of block device to create.
 type BlockDeviceType string
 
@@ -489,7 +495,7 @@ type APIServerLoadBalancer struct {
 
 // ReferencedMachineResources contains resolved references to resources required by the machine.
 type ReferencedMachineResources struct {
-	// ServerGroupID is the ID of the server group the machine should be added to and is calculated based on ServerGroupFilter.
+	// ServerGroupID is the ID of the server group the machine should be added to and is calculated based on ServerGroupFilter or ServerGroupRef.
 	// +optional
 	ServerGroupID string `json:"serverGroupID,omitempty"`
 

config/crd/bases/infrastructure.cluster.x-k8s.io_openstackclusters.yaml

@@ -5268,6 +5268,17 @@ spec:
                           name:
                             type: string
                         type: object
+                      serverGroupRef:
+                        description: The server group ref to assign the machine to.
+                        properties:
+                          name:
+                            description: |-
+                              Name of the OpenStackServerGroup resource to be used.
+                              Must be in the same namespace as the resource(s) being provisioned.
+                            type: string
+                        required:
+                        - name
+                        type: object
                       serverMetadata:
                         description: Metadata mapping. Allows you to create a map
                           of key value pairs to add to the server instance.
@@ -5713,7 +5724,8 @@ spec:
                         type: string
                       serverGroupID:
                         description: ServerGroupID is the ID of the server group the
-                          machine should be added to and is calculated based on ServerGroupFilter.
+                          machine should be added to and is calculated based on ServerGroupFilter
+                          or ServerGroupRef.
                         type: string
                     type: object
                   sshKeyName:

config/crd/bases/infrastructure.cluster.x-k8s.io_openstackclustertemplates.yaml

@@ -2701,6 +2701,18 @@ spec:
                                   name:
                                     type: string
                                 type: object
+                              serverGroupRef:
+                                description: The server group ref to assign the machine
+                                  to.
+                                properties:
+                                  name:
+                                    description: |-
+                                      Name of the OpenStackServerGroup resource to be used.
+                                      Must be in the same namespace as the resource(s) being provisioned.
+                                    type: string
+                                required:
+                                - name
+                                type: object
                               serverMetadata:
                                 description: Metadata mapping. Allows you to create
                                   a map of key value pairs to add to the server instance.

config/crd/bases/infrastructure.cluster.x-k8s.io_openstackmachines.yaml

@@ -2064,6 +2064,17 @@ spec:
                   name:
                     type: string
                 type: object
+              serverGroupRef:
+                description: The server group ref to assign the machine to.
+                properties:
+                  name:
+                    description: |-
+                      Name of the OpenStackServerGroup resource to be used.
+                      Must be in the same namespace as the resource(s) being provisioned.
+                    type: string
+                required:
+                - name
+                type: object
               serverMetadata:
                 description: Metadata mapping. Allows you to create a map of key value
                   pairs to add to the server instance.
@@ -2214,7 +2225,8 @@ spec:
                     type: string
                   serverGroupID:
                     description: ServerGroupID is the ID of the server group the machine
-                      should be added to and is calculated based on ServerGroupFilter.
+                      should be added to and is calculated based on ServerGroupFilter
+                      or ServerGroupRef.
                     type: string
                 type: object
             type: object

config/crd/bases/infrastructure.cluster.x-k8s.io_openstackmachinetemplates.yaml

@@ -1743,6 +1743,17 @@ spec:
                           name:
                             type: string
                         type: object
+                      serverGroupRef:
+                        description: The server group ref to assign the machine to.
+                        properties:
+                          name:
+                            description: |-
+                              Name of the OpenStackServerGroup resource to be used.
+                              Must be in the same namespace as the resource(s) being provisioned.
+                            type: string
+                        required:
+                        - name
+                        type: object
                       serverMetadata:
                         description: Metadata mapping. Allows you to create a map
                           of key value pairs to add to the server instance.