chore: Release 0.2.0 (Orange-OpenSource#4)

* chore: release-0.2.0 (Orange-OpenSource#2) * chore: release-0.2.0 Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> * ci: helm-lint on PRs Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> * fix: YAML linter Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> --------- Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> * ci: release Helm only on main branch (Orange-OpenSource#3) * chore: release-0.2.0 Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> * ci: helm-lint on PRs Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> * fix: YAML linter Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> * ci: release Helm only on main branch Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> --------- Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com> --------- Signed-off-by: Anthony TREUILLIER <anthony.treuillier.ext@orange.com>
antrema · Oct 31, 2024 · 6f270f8 · 6f270f8
1 parent 59d31eb
commit 6f270f8
Show file tree

Hide file tree

Showing 11 changed files with 56 additions and 37 deletions.
diff --git a/.github/workflows/helm-lint.yml b/.github/workflows/helm-lint.yml
@@ -5,6 +5,7 @@ on:
     branches:
       - main
       - release-*
+  pull_request: {}
 
   workflow_dispatch: {}
 

diff --git a/.github/workflows/helm-release.yml b/.github/workflows/helm-release.yml
@@ -4,7 +4,6 @@ on:
   push:
     branches:
       - main
-      - release-*
     paths:
       - charts/powerdns-operator/Chart.yaml
 

diff --git a/charts/powerdns-operator/Chart.yaml b/charts/powerdns-operator/Chart.yaml
@@ -2,8 +2,8 @@ apiVersion: v2
 name: powerdns-operator
 description: Manage PowerDNS resources with Kubernetes
 type: application
-version: 0.1.0
-appVersion: 0.1.0
+version: 0.2.0
+appVersion: 0.2.0
 kubeVersion: '>= 1.21.0-0'
 keywords:
   - powerdns-operator
@@ -33,4 +33,4 @@ annotations:
       description: |
         An RRset is a collection of records that share the same name and type.
         RRsets can be used to represent a collection of records that are managed
-        together.
+        together.
diff --git a/charts/powerdns-operator/README.md b/charts/powerdns-operator/README.md
@@ -2,7 +2,7 @@
 
 [//]: # (README.md generated by gotmpl. DO NOT EDIT.)
 
-![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![Version: 0.1.0](https://img.shields.io/badge/Version-0.1.0-informational?style=flat-square)
+![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![Version: 0.2.0](https://img.shields.io/badge/Version-0.2.0-informational?style=flat-square)
 
 Manage PowerDNS resources with Kubernetes
 
@@ -33,10 +33,10 @@ The command removes all the Kubernetes components associated with the chart and
 | annotations | object | `{}` | Annotations to add to the controller deployment. |
 | commonLabels | object | `{}` |  |
 | credentials.data.PDNS_API_KEY | string | `"secret"` | Specifies the PowerDNS API key used to authenticate |
-| credentials.data.PDNS_API_URL | string | `"http://powerdns.powerdns.svc:8081"` | Specifies the PowerDNS API URL PDNS_API_URL: "https://powerdns.example.local:8081" |
+| credentials.data.PDNS_API_URL | string | `"https://powerdns.example.local:8081"` | Specifies the PowerDNS API URL |
 | credentials.data.PDNS_API_VHOST | string | `"localhost"` | Specifies the PowerDNS VHOST |
 | credentials.existingSecret | string | `""` | Specifies whether to use an existing secret. |
-| credentials.name | string | `""` | Specifies the secret name to create if `existingSecret` is empty. |
+| credentials.name | string | `"powerdns-api-credentials"` | Specifies the secret name to create if `existingSecret` is empty. |
 | dnsConfig | object | `{}` | Specifies `dnsOptions` to deployment |
 | dnsPolicy | string | `"ClusterFirst"` | Specifies `dnsPolicy` to deployment |
 | fullnameOverride | string | `""` |  |

diff --git a/charts/powerdns-operator/crds/dns.cav.enablers.ob_rrsets.yaml b/charts/powerdns-operator/crds/dns.cav.enablers.ob_rrsets.yaml
@@ -15,15 +15,21 @@ spec:
   scope: Namespaced
   versions:
   - additionalPrinterColumns:
-    - jsonPath: .spec.zoneIdRef.name
+    - jsonPath: .spec.zoneRef.name
       name: Zone
       type: string
+    - jsonPath: .status.dnsEntryName
+      name: Name
+      type: string
     - jsonPath: .spec.type
       name: Type
       type: string
     - jsonPath: .spec.ttl
       name: TTL
       type: integer
+    - jsonPath: .status.syncStatus
+      name: Status
+      type: string
     - jsonPath: .spec.records
       name: Records
       type: string
@@ -55,6 +61,12 @@ spec:
               comment:
                 description: Comment on RRSet.
                 type: string
+              name:
+                description: Name of the record
+                type: string
+                x-kubernetes-validations:
+                - message: Value is immutable
+                  rule: self == oldSelf
               records:
                 description: All records in this Resource Record Set.
                 items:
@@ -77,6 +89,7 @@ spec:
                 - name
                 type: object
             required:
+            - name
             - records
             - ttl
             - type
@@ -85,9 +98,15 @@ spec:
           status:
             description: RRsetStatus defines the observed state of RRset
             properties:
+              dnsEntryName:
+                type: string
               lastUpdateTime:
                 format: date-time
                 type: string
+              syncErrorDescription:
+                type: string
+              syncStatus:
+                type: string
             type: object
         type: object
     served: true

diff --git a/charts/powerdns-operator/tests/__snapshot__/controller_test.yaml.snap b/charts/powerdns-operator/tests/__snapshot__/controller_test.yaml.snap
@@ -8,8 +8,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator
       namespace: NAMESPACE
     spec:
@@ -25,8 +25,8 @@ should match snapshot of default values:
             app.kubernetes.io/instance: RELEASE-NAME
             app.kubernetes.io/managed-by: Helm
             app.kubernetes.io/name: powerdns-operator
-            app.kubernetes.io/version: 0.1.0
-            helm.sh/chart: powerdns-operator-0.1.0
+            app.kubernetes.io/version: 0.2.0
+            helm.sh/chart: powerdns-operator-0.2.0
         spec:
           containers:
             - args:
@@ -38,7 +38,7 @@ should match snapshot of default values:
               envFrom:
                 - secretRef:
                     name: powerdns-api-credentials
-              image: ghcr.io/orange-opensource/powerdns-operator:v0.1.0
+              image: ghcr.io/orange-opensource/powerdns-operator:v0.2.0
               imagePullPolicy: IfNotPresent
               livenessProbe:
                 httpGet:

diff --git a/charts/powerdns-operator/tests/__snapshot__/metrics_service_test.yaml.snap b/charts/powerdns-operator/tests/__snapshot__/metrics_service_test.yaml.snap
@@ -7,8 +7,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-metrics
       namespace: NAMESPACE
     spec:

diff --git a/charts/powerdns-operator/tests/__snapshot__/rbac_test.yaml.snap b/charts/powerdns-operator/tests/__snapshot__/rbac_test.yaml.snap
@@ -7,8 +7,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-leader-election-role
       namespace: NAMESPACE
     rules:
@@ -51,8 +51,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-manager-role
     rules:
       - apiGroups:
@@ -92,8 +92,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-rrset-editor-role
     rules:
       - apiGroups:
@@ -122,8 +122,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
         rbac.authorization.k8s.io/aggregate-to-admin: "true"
         rbac.authorization.k8s.io/aggregate-to-edit: "true"
         rbac.authorization.k8s.io/aggregate-to-view: "true"
@@ -151,8 +151,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-zone-editor-role
     rules:
       - apiGroups:
@@ -181,8 +181,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-zone-viewer-role
     rules:
       - apiGroups:
@@ -207,8 +207,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-leader-election-rolebinding
       namespace: NAMESPACE
     roleRef:
@@ -227,8 +227,8 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator-manager-rolebinding
     roleRef:
       apiGroup: rbac.authorization.k8s.io

diff --git a/charts/powerdns-operator/tests/__snapshot__/serviceaccount_test.yaml.snap b/charts/powerdns-operator/tests/__snapshot__/serviceaccount_test.yaml.snap
@@ -7,7 +7,7 @@ should match snapshot of default values:
         app.kubernetes.io/instance: RELEASE-NAME
         app.kubernetes.io/managed-by: Helm
         app.kubernetes.io/name: powerdns-operator
-        app.kubernetes.io/version: 0.1.0
-        helm.sh/chart: powerdns-operator-0.1.0
+        app.kubernetes.io/version: 0.2.0
+        helm.sh/chart: powerdns-operator-0.2.0
       name: RELEASE-NAME-powerdns-operator
       namespace: NAMESPACE
diff --git a/charts/powerdns-operator/values.yaml b/charts/powerdns-operator/values.yaml
@@ -12,7 +12,7 @@ global:
       # disabled - No modification applied.
       adaptSecurityContext: auto
 
-replicaCount: 1 # we do not support more than 1 replica
+replicaCount: 1  # we do not support more than 1 replica
 
 commonLabels: {}
 

diff --git a/docs/README.md b/docs/README.md
@@ -2,7 +2,7 @@
 
 [//]: # (README.md generated by gotmpl. DO NOT EDIT.)
 
-![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![Version: 0.1.0](https://img.shields.io/badge/Version-0.1.0-informational?style=flat-square)
+![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![Version: 0.2.0](https://img.shields.io/badge/Version-0.2.0-informational?style=flat-square)
 
 Manage PowerDNS resources with Kubernetes
 
@@ -33,10 +33,10 @@ The command removes all the Kubernetes components associated with the chart and
 | annotations | object | `{}` | Annotations to add to the controller deployment. |
 | commonLabels | object | `{}` |  |
 | credentials.data.PDNS_API_KEY | string | `"secret"` | Specifies the PowerDNS API key used to authenticate |
-| credentials.data.PDNS_API_URL | string | `"http://powerdns.powerdns.svc:8081"` | Specifies the PowerDNS API URL PDNS_API_URL: "https://powerdns.example.local:8081" |
+| credentials.data.PDNS_API_URL | string | `"https://powerdns.example.local:8081"` | Specifies the PowerDNS API URL |
 | credentials.data.PDNS_API_VHOST | string | `"localhost"` | Specifies the PowerDNS VHOST |
 | credentials.existingSecret | string | `""` | Specifies whether to use an existing secret. |
-| credentials.name | string | `""` | Specifies the secret name to create if `existingSecret` is empty. |
+| credentials.name | string | `"powerdns-api-credentials"` | Specifies the secret name to create if `existingSecret` is empty. |
 | dnsConfig | object | `{}` | Specifies `dnsOptions` to deployment |
 | dnsPolicy | string | `"ClusterFirst"` | Specifies `dnsPolicy` to deployment |
 | fullnameOverride | string | `""` |  |
-Original file line number
+Diff line change
@@ Expand Up / @@ -4,7 +4,6 @@ on: @@
       push:
         branches:
           - main
-          - release-*
         paths:
           - charts/powerdns-operator/Chart.yaml
@@ Expand Down @@