Fix image properties propagation

Signed-off-by: Matej Feder <[email protected]>

Author: matofeder

api/v1alpha1/openstacknodeimagerelease_types.go

@@ -41,8 +41,57 @@ type OpenStackNodeImage struct {
 	CreateOpts *CreateOpts `json:"createOpts" yaml:"createOpts"`
 }
 
+// TODO: Reimplement logic to import `images.CreateOpts` from Gophercloud with a `Properties` override.
+//       The current `images.CreateOpts` defines `json:"-"` for the `Properties` field, making it unsuitable
+//       for proper serialization/deserialization required for `CreateOpts` used in Kubernetes resources
+//       and `nodeimagerelease.yaml`.
+//
+//       One solution is to create a new struct specifically for serialization/deserialization
+//       that embeds `images.CreateOpts` and redefines the `Properties` field to allow proper handling.
+//       Alternatively, consider overriding the default marshalling/unmarshalling functions to achieve the desired behavior.
+//
 // CreateOpts represents options used to create an image.
-type CreateOpts images.CreateOpts
+type CreateOpts struct {
+	// Name is the name of the new image.
+	Name string `json:"name" required:"true"`
+
+	// Id is the the image ID.
+	ID string `json:"id,omitempty"`
+
+	// Visibility defines who can see/use the image.
+	Visibility *images.ImageVisibility `json:"visibility,omitempty"`
+
+	// Hidden is whether the image is listed in default image list or not.
+	Hidden *bool `json:"os_hidden,omitempty"`
+
+	// Tags is a set of image tags.
+	Tags []string `json:"tags,omitempty"`
+
+	// ContainerFormat is the format of the
+	// container. Valid values are ami, ari, aki, bare, and ovf.
+	ContainerFormat string `json:"container_format,omitempty"`
+
+	// DiskFormat is the format of the disk. If set,
+	// valid values are ami, ari, aki, vhd, vmdk, raw, qcow2, vdi,
+	// and iso.
+	DiskFormat string `json:"disk_format,omitempty"`
+
+	// MinDisk is the amount of disk space in
+	// GB that is required to boot the image.
+	MinDisk int `json:"min_disk,omitempty"`
+
+	// MinRAM is the amount of RAM in MB that
+	// is required to boot the image.
+	MinRAM int `json:"min_ram,omitempty"`
+
+	// protected is whether the image is not deletable.
+	Protected *bool `json:"protected,omitempty"`
+
+	// properties is a set of properties, if any, that
+	// are associated with the image.
+	Properties map[string]string `json:"properties,omitempty"`
+}
+
 
 // OpenStackNodeImageReleaseStatus defines the observed state of OpenStackNodeImageRelease.
 type OpenStackNodeImageReleaseStatus struct {

config/crd/bases/infrastructure.clusterstack.x-k8s.io_openstacknodeimagereleases.yaml

@@ -114,6 +114,13 @@ spec:
                         description: Hidden is whether the image is listed in default
                           image list or not.
                         type: boolean
+                      properties:
+                        additionalProperties:
+                          type: string
+                        description: |-
+                          properties is a set of properties, if any, that
+                          are associated with the image.
+                        type: object
                       protected:
                         description: protected is whether the image is not deletable.
                         type: boolean