Merge pull request #631 from eshulman2/cdrive

Add config drive field to server controller

Author: mandre

api/v1alpha1/server_types.go

@@ -187,6 +187,13 @@ type ServerResourceSpec struct {
 	// +listType=atomic
 	// +optional
 	Metadata []ServerMetadata `json:"metadata,omitempty"`
+
+	// configDrive specifies whether to attach a config drive to the server.
+	// When true, configuration data will be available via a special drive
+	// instead of the metadata service.
+	// +optional
+	// +kubebuilder:validation:XValidation:rule="self == oldSelf",message="configDrive is immutable"
+	ConfigDrive *bool `json:"configDrive,omitempty"`
 }
 
 // ServerMetadata represents a key-value pair for server metadata.
@@ -288,6 +295,10 @@ type ServerResourceStatus struct {
 	// +listType=atomic
 	// +optional
 	Metadata []ServerMetadataStatus `json:"metadata,omitempty"`
+
+	// configDrive indicates whether the server was booted with a config drive.
+	// +optional
+	ConfigDrive bool `json:"configDrive,omitempty"`
 }
 
 // ServerMetadataStatus represents a key-value pair for server metadata in status.

api/v1alpha1/zz_generated.deepcopy.go

@@ -202,6 +202,15 @@ spec:
                     x-kubernetes-validations:
                     - message: availabilityZone is immutable
                       rule: self == oldSelf
+                  configDrive:
+                    description: |-
+                      configDrive specifies whether to attach a config drive to the server.
+                      When true, configuration data will be available via a special drive
+                      instead of the metadata service.
+                    type: boolean
+                    x-kubernetes-validations:
+                    - message: configDrive is immutable
+                      rule: self == oldSelf
                   flavorRef:
                     description: flavorRef references the flavor to use for the server
                       instance.
@@ -455,6 +464,10 @@ spec:
                       server is located.
                     maxLength: 1024
                     type: string
+                  configDrive:
+                    description: configDrive indicates whether the server was booted
+                      with a config drive.
+                    type: boolean
                   hostID:
                     description: hostID is the host where the server is located in
                       the cloud.

cmd/models-schema/zz_generated.openapi.go

@@ -25,3 +25,4 @@ spec:
         value: development
       - key: owner
         value: sample
+    configDrive: true

config/crd/bases/openstack.k-orc.cloud_servers.yaml

@@ -263,6 +263,7 @@ func (actuator serverActuator) CreateResource(ctx context.Context, obj *orcv1alp
 		Tags:             tags,
 		Metadata:         metadata,
 		AvailabilityZone: resource.AvailabilityZone,
+		ConfigDrive:      resource.ConfigDrive,
 	}
 
 	/* keypairs.CreateOptsExt was merged into servers.CreateOpts in gopher cloud V3

config/samples/openstack_v1alpha1_server.yaml

@@ -71,7 +71,8 @@ func (serverStatusWriter) ApplyResourceStatus(log logr.Logger, osResource *osRes
 		WithHostID(osResource.HostID).
 		WithAvailabilityZone(osResource.AvailabilityZone).
 		WithServerGroups(ptr.Deref(osResource.ServerGroups, []string{})...).
-		WithTags(ptr.Deref(osResource.Tags, []string{})...)
+		WithTags(ptr.Deref(osResource.Tags, []string{})...).
+		WithConfigDrive(osResource.ConfigDrive)
 
 	if imageID, ok := osResource.Image["id"]; ok {
 		status.WithImageID(fmt.Sprintf("%s", imageID))

internal/controllers/server/actuator.go

@@ -71,3 +71,4 @@ status:
         value: test
       - key: owner
         value: kuttl
+    configDrive: true

internal/controllers/server/status.go

@@ -53,3 +53,4 @@ spec:
         value: test
       - key: owner
         value: kuttl
+    configDrive: true