RegisteredLimit Tests and Config Samples

On-behalf-of: SAP nils.gondermann@sap.com

Author: gndrmnn

config/samples/openstack_v1alpha1_registeredlimit.yaml

@@ -1,14 +1,29 @@
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
+kind: Service
+metadata:
+  name: registeredlimit-sample
+spec:
+  cloudCredentialsRef:
+    cloudName: openstack-admin
+    secretName: openstack-clouds
+  managementPolicy: unmanaged
+  import:
+    filter:
+      type: "compute"
+---
+apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: RegisteredLimit
 metadata:
   name: registeredlimit-sample
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: Sample RegisteredLimit
-    # TODO(scaffolding): Add all fields the resource supports
+    serviceRef: registeredlimit-sample
+    resourceName: vcpus
+    defaultLimit: 5
+    regionID: RegionOne

internal/controllers/registeredlimit/tests/registeredlimit-create-full/00-assert.yaml

@@ -5,9 +5,9 @@ metadata:
   name: registeredlimit-create-full
 status:
   resource:
-    name: registeredlimit-create-full-override
     description: RegisteredLimit from "create full" test
-    # TODO(scaffolding): Add all fields the resource supports
+    resourceName: vcpus
+    defaultLimit: 5
   conditions:
   - type: Available
     status: "True"
@@ -30,4 +30,4 @@ resourceRefs:
 assertAll:
     - celExpr: "registeredlimit.status.id != ''"
     - celExpr: "registeredlimit.status.resource.serviceID == service.status.id"
-    # TODO(scaffolding): Add more checks
+    - celExpr: "!has(registeredlimit.status.resource.name)"

internal/controllers/registeredlimit/tests/registeredlimit-create-full/00-create-resource.yaml

@@ -5,25 +5,25 @@ metadata:
   name: registeredlimit-create-full
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
-  managementPolicy: managed
-  # TODO(scaffolding): Add the necessary fields to create the resource
-  resource: {}
+  managementPolicy: unmanaged
+  import:
+    filter:
+      type: "compute"
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: RegisteredLimit
 metadata:
   name: registeredlimit-create-full
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     name: registeredlimit-create-full-override
     description: RegisteredLimit from "create full" test
     serviceRef: registeredlimit-create-full
-    # TODO(scaffolding): Add all fields the resource supports
+    resourceName: vcpus
+    defaultLimit: 5

internal/controllers/registeredlimit/tests/registeredlimit-create-minimal/00-assert.yaml

@@ -5,8 +5,8 @@ metadata:
   name: registeredlimit-create-minimal
 status:
   resource:
-    name: registeredlimit-create-minimal
-    # TODO(scaffolding): Add all fields the resource supports
+    resourceName: vcpus
+    defaultLimit: 5
   conditions:
   - type: Available
     status: "True"
@@ -29,4 +29,4 @@ resourceRefs:
 assertAll:
     - celExpr: "registeredlimit.status.id != ''"
     - celExpr: "registeredlimit.status.resource.serviceID == service.status.id"
-    # TODO(scaffolding): Add more checks
+    - celExpr: "!has(registeredlimit.status.resource.name)"

internal/controllers/registeredlimit/tests/registeredlimit-create-minimal/00-create-resource.yaml

@@ -5,24 +5,23 @@ metadata:
   name: registeredlimit-create-minimal
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
-  managementPolicy: managed
-  # TODO(scaffolding): Add the necessary fields to create the resource
-  resource: {}
+  managementPolicy: unmanaged
+  import:
+    filter:
+      type: "compute"
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: RegisteredLimit
 metadata:
   name: registeredlimit-create-minimal
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
   managementPolicy: managed
-  # TODO(scaffolding): Only add the mandatory fields. It's possible the resource
-  # doesn't have mandatory fields, in that case, leave it empty.
   resource:
     serviceRef: registeredlimit-create-minimal
+    resourceName: vcpus
+    defaultLimit: 5

internal/controllers/registeredlimit/tests/registeredlimit-import/00-import-resource.yaml

@@ -1,5 +1,18 @@
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
+kind: Service
+metadata:
+  name: registeredlimit-import-external
+spec:
+  cloudCredentialsRef:
+    cloudName: openstack-admin
+    secretName: openstack-clouds
+  managementPolicy: unmanaged
+  import:
+    filter:
+      name: compute
+---
+apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: RegisteredLimit
 metadata:
   name: registeredlimit-import
@@ -10,6 +23,6 @@ spec:
   managementPolicy: unmanaged
   import:
     filter:
-      name: registeredlimit-import-external
       description: RegisteredLimit registeredlimit-import-external from "registeredlimit-import" test
-      # TODO(scaffolding): Add all fields supported by the filter
+      serviceRef: registeredlimit-import-external
+      resourceName: vcpus

internal/controllers/registeredlimit/tests/registeredlimit-import/01-assert.yaml

@@ -16,7 +16,7 @@ status:
   resource:
     name: registeredlimit-import-external-not-this-one
     description: RegisteredLimit registeredlimit-import-external from "registeredlimit-import" test
-    # TODO(scaffolding): Add fields necessary to match filter
+    resourceName: vcpus
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: RegisteredLimit

internal/controllers/registeredlimit/tests/registeredlimit-import/01-create-trap-resource.yaml

@@ -5,12 +5,11 @@ metadata:
   name: registeredlimit-import-external-not-this-one
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
   managementPolicy: managed
-  # TODO(scaffolding): Add the necessary fields to create the resource
-  resource: {}
+  resource:
+    type: endpoint-import-external-not-this-one
 ---
 # This `registeredlimit-import-external-not-this-one` resource serves two purposes:
 # - ensure that we can successfully create another resource which name is a substring of it (i.e. it's not being adopted)
@@ -21,11 +20,12 @@ metadata:
   name: registeredlimit-import-external-not-this-one
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: RegisteredLimit registeredlimit-import-external from "registeredlimit-import" test
     serviceRef: registeredlimit-import-external-not-this-one
-    # TODO(scaffolding): Add fields necessary to match filter
+    regionID: RegionOne
+    resourceName: vcpus
+    defaultLimit: 5

internal/controllers/registeredlimit/tests/registeredlimit-import/02-assert.yaml

@@ -28,6 +28,6 @@ status:
       status: "False"
       reason: Success
   resource:
-    name: registeredlimit-import-external
     description: RegisteredLimit registeredlimit-import-external from "registeredlimit-import" test
-    # TODO(scaffolding): Add all fields the resource supports
+    resourceName: vcpus
+    defaultLimit: 5

internal/controllers/registeredlimit/tests/registeredlimit-import/02-create-resource.yaml

@@ -5,24 +5,24 @@ metadata:
   name: registeredlimit-import
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
-  managementPolicy: managed
-  # TODO(scaffolding): Add the necessary fields to create the resource
-  resource: {}
+  managementPolicy: unmanaged
+  import:
+    filter:
+      name: compute
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: RegisteredLimit
 metadata:
   name: registeredlimit-import-external
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
-    cloudName: openstack
+    cloudName: openstack-admin
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: RegisteredLimit registeredlimit-import-external from "registeredlimit-import" test
     serviceRef: registeredlimit-import
-    # TODO(scaffolding): Add fields necessary to match filter
+    resourceName: vcpus
+    defaultLimit: 5