[intel-gpu-plugin] Migrate to common v3 (#966)

Signed-off-by: Bᴇʀɴᴅ Sᴄʜᴏʀɢᴇʀs <me@bjw-s.dev>

charts/stable/intel-gpu-plugin/Chart.yaml

@@ -1,6 +1,6 @@
 apiVersion: v2
 name: intel-gpu-plugin
-version: 2.0.0
+version: 3.0.0
 appVersion: 0.20.0
 description: The Intel GPU plugin facilitates offloading the processing of computation intensive workloads to GPU hardware
 keywords:
@@ -15,7 +15,9 @@ sources:
 maintainers:
   - name: billimek
     email: jeff@billimek.com
+  - name: bjw-s
+    email: me@bjw-s.dev
 dependencies:
   - name: common
-    version: 2.5.0
+    version: 3.0.2
     repository: https://library-charts.k8s-at-home.com

charts/stable/intel-gpu-plugin/README.md

@@ -1,6 +1,6 @@
 # intel-gpu-plugin
 
-![Version: 2.0.0](https://img.shields.io/badge/Version-2.0.0-informational?style=flat-square) ![AppVersion: 0.20.0](https://img.shields.io/badge/AppVersion-0.20.0-informational?style=flat-square)
+![Version: 3.0.0](https://img.shields.io/badge/Version-3.0.0-informational?style=flat-square) ![AppVersion: 0.20.0](https://img.shields.io/badge/AppVersion-0.20.0-informational?style=flat-square)
 
 The Intel GPU plugin facilitates offloading the processing of computation intensive workloads to GPU hardware
 
@@ -16,7 +16,7 @@ The Intel GPU plugin facilitates offloading the processing of computation intens
 
 | Repository | Name | Version |
 |------------|------|---------|
-| https://library-charts.k8s-at-home.com | common | 2.5.0 |
+| https://library-charts.k8s-at-home.com | common | 3.0.2 |
 
 ## TL;DR
 
@@ -87,37 +87,17 @@ affinity:
 
 | Key | Type | Default | Description |
 |-----|------|---------|-------------|
-| args[0] | string | `"-shared-dev-num"` |  |
+| args | list | `["-shared-dev-num","1"]` | Override the args for the default container Refer to the [plugin documentation](https://github.com/intel/intel-device-plugins-for-kubernetes/blob/main/cmd/gpu_plugin/README.md) for more information. |
-| args[1] | string | `"1"` |  |
+| controller.type | string | `"daemonset"` | Run this chart as a daemonset. Do not modify unless you know what you are doing. |
-| controllerType | string | `"daemonset"` |  |
+| envValueFrom.NODE_NAME | object | `spec.nodeName` | Sets the NODE_NAME env var to the name of the node where the pod is running. Do not modify unless you know what you are doing. |
-| envValueFrom.NODE_NAME.fieldRef.fieldPath | string | `"spec.nodeName"` |  |
+| image.pullPolicy | string | `"IfNotPresent"` | image pull policy |
-| hostPathMounts[0].emptyDir | bool | `false` |  |
+| image.repository | string | `"intel/intel-gpu-plugin"` | image repository |
-| hostPathMounts[0].enabled | bool | `true` |  |
+| image.tag | string | `"0.20.0"` | image tag |
-| hostPathMounts[0].hostPath | string | `"/dev/dri"` |  |
+| ingress.main.enabled | bool | `false` | Ingress is disabled for this chart. Do not modify unless you know what you are doing. |
-| hostPathMounts[0].mountPath | string | `"/dev/dri"` |  |
+| persistence | object | See values.yaml | Configure persistence settings for the chart under this key. |
-| hostPathMounts[0].name | string | `"devfs"` |  |
+| probes | object | See values.yaml | Disable probes for this chart since there is no service. Do not modify unless you know what you are doing. |
-| hostPathMounts[0].readOnly | bool | `true` |  |
+| service.main.enabled | bool | `false` | Main service is disabled for this chart. Do not modify unless you know what you are doing. |
-| hostPathMounts[1].emptyDir | bool | `false` |  |
+| serviceAccount.create | bool | `true` | Specifies whether a service account should be created |
-| hostPathMounts[1].enabled | bool | `true` |  |
-| hostPathMounts[1].hostPath | string | `"/sys/class/drm"` |  |
-| hostPathMounts[1].mountPath | string | `"/sys/class/drm"` |  |
-| hostPathMounts[1].name | string | `"sysfs"` |  |
-| hostPathMounts[1].readOnly | bool | `true` |  |
-| hostPathMounts[2].emptyDir | bool | `false` |  |
-| hostPathMounts[2].enabled | bool | `true` |  |
-| hostPathMounts[2].hostPath | string | `"/var/lib/kubelet/device-plugins"` |  |
-| hostPathMounts[2].mountPath | string | `"/var/lib/kubelet/device-plugins"` |  |
-| hostPathMounts[2].name | string | `"kubeletsockets"` |  |
-| image.pullPolicy | string | `"IfNotPresent"` |  |
-| image.repository | string | `"intel/intel-gpu-plugin"` |  |
-| image.tag | string | `"0.20.0"` |  |
-| ingress.enabled | bool | `false` |  |
-| probes.liveness.enabled | bool | `false` |  |
-| probes.readiness.enabled | bool | `false` |  |
-| probes.startup.enabled | bool | `false` |  |
-| service.enabled | bool | `false` |  |
-| serviceAccount.create | bool | `true` |  |
-| strategy.type | string | `"Recreate"` |  |
 
 ## Changelog
 
@@ -125,6 +105,13 @@ All notable changes to this application Helm chart will be documented in this fi
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+### [3.0.0]
+
+#### Changed
+
+- **BREAKING**: Upgraded the common library dependency to version 3.0.2. This introduces several breaking changes (`service`, `ingress` and `persistence` keys have been refactored).
+  Be sure to check out the [library chart](https://github.com/k8s-at-home/library-charts/blob/common-3.0.2/charts/stable/common/) for the up-to-date values.
+
 ### [2.0.0]
 
 #### Added

charts/stable/intel-gpu-plugin/README_CHANGELOG.md.gotmpl

@@ -9,6 +9,13 @@ All notable changes to this application Helm chart will be documented in this fi
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+### [3.0.0]
+
+#### Changed
+
+- **BREAKING**: Upgraded the common library dependency to version 3.0.2. This introduces several breaking changes (`service`, `ingress` and `persistence` keys have been refactored).
+  Be sure to check out the [library chart](https://github.com/k8s-at-home/library-charts/blob/common-3.0.2/charts/stable/common/) for the up-to-date values.
+
 ### [2.0.0]
 
 #### Added

charts/stable/intel-gpu-plugin/values.yaml

@@ -6,52 +6,64 @@
 #
 
 image:
+  # -- image repository
   repository: intel/intel-gpu-plugin
-  pullPolicy: IfNotPresent
+  # -- image tag
   tag: 0.20.0
+  # -- image pull policy
+  pullPolicy: IfNotPresent
 
-controllerType: daemonset
+controller:
-
+  # -- Run this chart as a daemonset. Do not modify unless you know what you are doing.
-strategy:
+  type: daemonset
-  type: Recreate
 
 serviceAccount:
+  # -- Specifies whether a service account should be created
   create: true
 
+# -- Override the args for the default container
+# Refer to the [plugin documentation](https://github.com/intel/intel-device-plugins-for-kubernetes/blob/main/cmd/gpu_plugin/README.md) for more information.
 args:
   - -shared-dev-num
   - "1"
 
 envValueFrom:
+  # -- Sets the NODE_NAME env var to the name of the node where the pod is running. Do not modify unless you know what you are doing.
+  # @default -- `spec.nodeName`
   NODE_NAME:
     fieldRef:
       fieldPath: spec.nodeName
 
-hostPathMounts:
+# -- Configure persistence settings for the chart under this key.
-  - name: "devfs"
+# @default -- See values.yaml
+persistence:
+  devfs:
     enabled: true
-    emptyDir: false
+    type: hostPath
-    mountPath: "/dev/dri"
     hostPath: "/dev/dri"
     readOnly: true
-  - name: "sysfs"
+  sysfs:
     enabled: true
-    emptyDir: false
+    type: hostPath
-    mountPath: "/sys/class/drm"
     hostPath: "/sys/class/drm"
     readOnly: true
-  - name: "kubeletsockets"
+  kubeletsockets:
     enabled: true
-    emptyDir: false
+    type: hostPath
-    mountPath: "/var/lib/kubelet/device-plugins"
     hostPath: "/var/lib/kubelet/device-plugins"
 
 ingress:
-  enabled: false
+  main:
+    # -- Ingress is disabled for this chart. Do not modify unless you know what you are doing.
+    enabled: false
 
 service:
-  enabled: false
+  main:
+    # -- Main service is disabled for this chart. Do not modify unless you know what you are doing.
+    enabled: false
 
+# -- Disable probes for this chart since there is no service. Do not modify unless you know what you are doing.
+# @default -- See values.yaml
 probes:
   liveness:
     enabled: false