Apply suggestions from code review

Co-authored-by: janiskemper <63146658+janiskemper@users.noreply.github.com>
Signed-off-by: Matej Feder <matej.feder@dnation.cloud>

README.md

@@ -93,7 +93,7 @@ Get the latest CSPO release version and apply CSPO manifests to the management c
 # Get the latest CSPO release version
 CSPO_VERSION=$(curl https://api.github.com/repos/SovereignCloudStack/cluster-stack-provider-openstack/releases/latest -s | jq .name -r)
 # Apply CSPO manifests
-curl -sSL https://github.com/sovereignCloudStack/cluster-stack-provider-openstack/releases/download/${CSPO_VERSION}/cspo-infrastructure-components.yaml | /tmp/envsubst | kubectl apply -f -
+curl -sSL https://github.com/SovereignCloudStack/cluster-stack-provider-openstack/releases/download/${CSPO_VERSION}/cspo-infrastructure-components.yaml | /tmp/envsubst | kubectl apply -f -
 ```
 
 ## Create the workload cluster

api/v1alpha1/conditions_const.go

@@ -22,7 +22,7 @@ import (
 
 const (
 	// ClusterStackReleaseAssetsReadyCondition reports on whether the download of cluster stack release assets is complete.
-	ClusterStackReleaseAssetsReadyCondition clusterv1beta1.ConditionType = "ClusterStackReleaseDownloaded"
+	ClusterStackReleaseAssetsReadyCondition clusterv1beta1.ConditionType = "ClusterStackReleaseAssetsReady"
 
 	// ReleaseAssetsNotDownloadedYetReason is used when release assets are not yet downloaded.
 	ReleaseAssetsNotDownloadedYetReason = "ReleaseAssetsNotDownloadedYet"
@@ -65,7 +65,7 @@ const (
 
 const (
 	// OpenStackImageImportStartCondition reports the image import start.
-	OpenStackImageImportStartCondition = "OpenStackImageImportStart"
+	OpenStackImageImportStartedCondition = "OpenStackImageImportStarted"
 
 	// OpenStackImageImportNotStartReason is used when image import does not start yet.
 	OpenStackImageImportNotStartReason = "OpenStackImageImportNotStartReason"

api/v1alpha1/openstackclusterstackrelease_types.go

@@ -27,7 +27,7 @@ import (
 
 // OpenStackClusterStackReleaseSpec defines the desired state of OpenStackClusterStackRelease.
 type OpenStackClusterStackReleaseSpec struct {
-	// The name of the cloud to use from the clouds secret
+	// CloudName is the name of the cloud to use from the cloud's secret.
 	CloudName string `json:"cloudName"`
 	// IdentityRef is a reference to a identity to be used when reconciling this cluster
 	IdentityRef *apiv1alpha7.OpenStackIdentityReference `json:"identityRef"`

api/v1alpha1/openstacknodeimagerelease_types.go

@@ -28,7 +28,7 @@ import (
 
 // OpenStackNodeImageReleaseSpec defines the desired state of OpenStackNodeImageRelease.
 type OpenStackNodeImageReleaseSpec struct {
-	// The name of the cloud to use from the clouds secret
+	// CloudName is the name of the cloud to use from the cloud's secret.
 	CloudName string `json:"cloudName"`
 	// IdentityRef is a reference to a identity to be used when reconciling this cluster
 	IdentityRef *apiv1alpha7.OpenStackIdentityReference `json:"identityRef"`

internal/controller/openstacknodeimagerelease_controller.go

@@ -175,7 +175,7 @@ func (r *OpenStackNodeImageReleaseReconciler) Reconcile(ctx context.Context, req
 		}
 
 		conditions.MarkTrue(openstacknodeimagerelease, apiv1alpha1.OpenStackImageImportStartCondition)
-		// requeue to make sure that image ID can be find by image name
+		// requeue to make sure that image ID can be found via image name
 		return ctrl.Result{Requeue: true}, nil
 	}