[FEATURE] - Resource Count Delection Skipping

When then configuration has not provisioned any ressources, we can skip the deletion job
appvia · Jan 18, 2024 · f24b418 · f24b418
1 parent 7f883e5
commit f24b418
Show file tree

Hide file tree

Showing 16 changed files with 1,633 additions and 431 deletions.
diff --git a/charts/terranetes-controller/crds/terraform.appvia.io_cloudresources.yaml b/charts/terranetes-controller/crds/terraform.appvia.io_cloudresources.yaml
diff --git a/charts/terranetes-controller/crds/terraform.appvia.io_configurations.yaml b/charts/terranetes-controller/crds/terraform.appvia.io_configurations.yaml
diff --git a/charts/terranetes-controller/crds/terraform.appvia.io_contexts.yaml b/charts/terranetes-controller/crds/terraform.appvia.io_contexts.yaml
@@ -2,7 +2,7 @@ apiVersion: apiextensions.k8s.io/v1
 kind: CustomResourceDefinition
 metadata:
   annotations:
-    controller-gen.kubebuilder.io/version: v0.13.0
+    controller-gen.kubebuilder.io/version: v0.14.0
   name: contexts.terraform.appvia.io
 spec:
   group: terraform.appvia.io
@@ -25,10 +25,19 @@ spec:
           description: Context is the schema for the context type
           properties:
             apiVersion:
-              description: 'APIVersion defines the versioned schema of this representation of an object. Servers should convert recognized schemas to the latest internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
+              description: |-
+                APIVersion defines the versioned schema of this representation of an object.
+                Servers should convert recognized schemas to the latest internal value, and
+                may reject unrecognized values.
+                More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources
               type: string
             kind:
-              description: 'Kind is a string value representing the REST resource this object represents. Servers may infer this from the endpoint the client submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
+              description: |-
+                Kind is a string value representing the REST resource this object represents.
+                Servers may infer this from the endpoint the client submits requests to.
+                Cannot be updated.
+                In CamelCase.
+                More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds
               type: string
             metadata:
               type: object
@@ -39,7 +48,10 @@ spec:
                   additionalProperties:
                     type: object
                     x-kubernetes-preserve-unknown-fields: true
-                  description: Variables is a list of variables which can be used globally by Context resources. The structure of the variables is a map of key/value pairs, which MUST have both a description and a value.
+                  description: |-
+                    Variables is a list of variables which can be used globally by Context resources.
+                    The structure of the variables is a map of key/value pairs, which MUST have both
+                    a description and a value.
                   type: object
               required:
                 - variables
@@ -53,27 +65,41 @@ spec:
                     description: Condition is the current observed condition of some aspect of a resource
                     properties:
                       detail:
-                        description: Detail is any additional human-readable detail to understand this condition, for example, the full underlying error which caused an issue
+                        description: |-
+                          Detail is any additional human-readable detail to understand this condition, for example,
+                          the full underlying error which caused an issue
                         type: string
                       lastTransitionTime:
-                        description: LastTransitionTime is the last time the condition transitioned from one status to another. This should be when the underlying condition changed.  If that is not known, then using the time when the API field changed is acceptable.
+                        description: |-
+                          LastTransitionTime is the last time the condition transitioned from one status to another.
+                          This should be when the underlying condition changed.  If that is not known, then using the time when the API field changed is acceptable.
                         format: date-time
                         type: string
                       message:
-                        description: Message is a human readable message indicating details about the transition. This may be an empty string.
+                        description: |-
+                          Message is a human readable message indicating details about the transition.
+                          This may be an empty string.
                         maxLength: 32768
                         type: string
                       name:
                         description: Name is a human-readable name for this condition.
                         minLength: 1
                         type: string
                       observedGeneration:
-                        description: ObservedGeneration represents the .metadata.generation that the condition was set based upon. For instance, if .metadata.generation is currently 12, but the .status.conditions[x].observedGeneration is 9, the condition is out of date with respect to the current state of the instance.
+                        description: |-
+                          ObservedGeneration represents the .metadata.generation that the condition was set based upon.
+                          For instance, if .metadata.generation is currently 12, but the .status.conditions[x].observedGeneration is 9, the condition is out of date
+                          with respect to the current state of the instance.
                         format: int64
                         minimum: 0
                         type: integer
                       reason:
-                        description: Reason contains a programmatic identifier indicating the reason for the condition's last transition. Producers of specific condition types may define expected values and meanings for this field, and whether the values are considered a guaranteed API. The value should be a CamelCase string. This field may not be empty.
+                        description: |-
+                          Reason contains a programmatic identifier indicating the reason for the condition's last transition.
+                          Producers of specific condition types may define expected values and meanings for this field,
+                          and whether the values are considered a guaranteed API.
+                          The value should be a CamelCase string.
+                          This field may not be empty.
                         maxLength: 1024
                         minLength: 1
                         pattern: ^[A-Za-z]([A-Za-z0-9_,:]*[A-Za-z0-9_])?$
@@ -86,7 +112,12 @@ spec:
                           - Unknown
                         type: string
                       type:
-                        description: Type of condition in CamelCase or in foo.example.com/CamelCase. --- Many .condition.type values are consistent across resources like Available, but because arbitrary conditions can be useful (see .node.status.conditions), the ability to deconflict is important. The regex it matches is (dns1123SubdomainFmt/)?(qualifiedNameFmt)
+                        description: |-
+                          Type of condition in CamelCase or in foo.example.com/CamelCase.
+                          ---
+                          Many .condition.type values are consistent across resources like Available, but because arbitrary conditions can be
+                          useful (see .node.status.conditions), the ability to deconflict is important.
+                          The regex it matches is (dns1123SubdomainFmt/)?(qualifiedNameFmt)
                         maxLength: 316
                         pattern: ^([a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*/)?(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])$
                         type: string
@@ -113,7 +144,9 @@ spec:
                       type: string
                   type: object
                 lastSuccess:
-                  description: LastSuccess descibes the generation and time of the last reconciliation which resulted in a Success status
+                  description: |-
+                    LastSuccess descibes the generation and time of the last reconciliation which resulted in
+                    a Success status
                   properties:
                     generation:
                       description: Generation is the generation reconciled on the last reconciliation

diff --git a/charts/terranetes-controller/crds/terraform.appvia.io_plans.yaml b/charts/terranetes-controller/crds/terraform.appvia.io_plans.yaml
@@ -2,7 +2,7 @@ apiVersion: apiextensions.k8s.io/v1
 kind: CustomResourceDefinition
 metadata:
   annotations:
-    controller-gen.kubebuilder.io/version: v0.13.0
+    controller-gen.kubebuilder.io/version: v0.14.0
   name: plans.terraform.appvia.io
 spec:
   group: terraform.appvia.io
@@ -28,10 +28,19 @@ spec:
           description: Plan is the schema for the plan type
           properties:
             apiVersion:
-              description: 'APIVersion defines the versioned schema of this representation of an object. Servers should convert recognized schemas to the latest internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
+              description: |-
+                APIVersion defines the versioned schema of this representation of an object.
+                Servers should convert recognized schemas to the latest internal value, and
+                may reject unrecognized values.
+                More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources
               type: string
             kind:
-              description: 'Kind is a string value representing the REST resource this object represents. Servers may infer this from the endpoint the client submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
+              description: |-
+                Kind is a string value representing the REST resource this object represents.
+                Servers may infer this from the endpoint the client submits requests to.
+                Cannot be updated.
+                In CamelCase.
+                More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds
               type: string
             metadata:
               type: object
@@ -64,27 +73,41 @@ spec:
                     description: Condition is the current observed condition of some aspect of a resource
                     properties:
                       detail:
-                        description: Detail is any additional human-readable detail to understand this condition, for example, the full underlying error which caused an issue
+                        description: |-
+                          Detail is any additional human-readable detail to understand this condition, for example,
+                          the full underlying error which caused an issue
                         type: string
                       lastTransitionTime:
-                        description: LastTransitionTime is the last time the condition transitioned from one status to another. This should be when the underlying condition changed.  If that is not known, then using the time when the API field changed is acceptable.
+                        description: |-
+                          LastTransitionTime is the last time the condition transitioned from one status to another.
+                          This should be when the underlying condition changed.  If that is not known, then using the time when the API field changed is acceptable.
                         format: date-time
                         type: string
                       message:
-                        description: Message is a human readable message indicating details about the transition. This may be an empty string.
+                        description: |-
+                          Message is a human readable message indicating details about the transition.
+                          This may be an empty string.
                         maxLength: 32768
                         type: string
                       name:
                         description: Name is a human-readable name for this condition.
                         minLength: 1
                         type: string
                       observedGeneration:
-                        description: ObservedGeneration represents the .metadata.generation that the condition was set based upon. For instance, if .metadata.generation is currently 12, but the .status.conditions[x].observedGeneration is 9, the condition is out of date with respect to the current state of the instance.
+                        description: |-
+                          ObservedGeneration represents the .metadata.generation that the condition was set based upon.
+                          For instance, if .metadata.generation is currently 12, but the .status.conditions[x].observedGeneration is 9, the condition is out of date
+                          with respect to the current state of the instance.
                         format: int64
                         minimum: 0
                         type: integer
                       reason:
-                        description: Reason contains a programmatic identifier indicating the reason for the condition's last transition. Producers of specific condition types may define expected values and meanings for this field, and whether the values are considered a guaranteed API. The value should be a CamelCase string. This field may not be empty.
+                        description: |-
+                          Reason contains a programmatic identifier indicating the reason for the condition's last transition.
+                          Producers of specific condition types may define expected values and meanings for this field,
+                          and whether the values are considered a guaranteed API.
+                          The value should be a CamelCase string.
+                          This field may not be empty.
                         maxLength: 1024
                         minLength: 1
                         pattern: ^[A-Za-z]([A-Za-z0-9_,:]*[A-Za-z0-9_])?$
@@ -97,7 +120,12 @@ spec:
                           - Unknown
                         type: string
                       type:
-                        description: Type of condition in CamelCase or in foo.example.com/CamelCase. --- Many .condition.type values are consistent across resources like Available, but because arbitrary conditions can be useful (see .node.status.conditions), the ability to deconflict is important. The regex it matches is (dns1123SubdomainFmt/)?(qualifiedNameFmt)
+                        description: |-
+                          Type of condition in CamelCase or in foo.example.com/CamelCase.
+                          ---
+                          Many .condition.type values are consistent across resources like Available, but because arbitrary conditions can be
+                          useful (see .node.status.conditions), the ability to deconflict is important.
+                          The regex it matches is (dns1123SubdomainFmt/)?(qualifiedNameFmt)
                         maxLength: 316
                         pattern: ^([a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*/)?(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])$
                         type: string
@@ -124,7 +152,9 @@ spec:
                       type: string
                   type: object
                 lastSuccess:
-                  description: LastSuccess descibes the generation and time of the last reconciliation which resulted in a Success status
+                  description: |-
+                    LastSuccess descibes the generation and time of the last reconciliation which resulted in
+                    a Success status
                   properties:
                     generation:
                       description: Generation is the generation reconciled on the last reconciliation