ytproxy bug and resources (#20)

* ytproxy bug and resources

fixed missing .ytproxy in pod template and fixed resources

* Update Chart.yaml
* Changelog and readme

---------

Co-authored-by: Skyler Mäntysaari <samip5@users.noreply.github.com>

charts/apps/piped/Chart.yaml

@@ -9,7 +9,7 @@ sources:
   - https://github.com/TeamPiped/piped-proxy
 keywords:
   - streaming
-version: 2.3.0
+version: 2.4.0
 appVersion: latest
 kubeVersion: ">=1.22.0-0"
 maintainers:
@@ -26,5 +26,10 @@ dependencies:
     condition: postgresql.enabled
 annotations:
   artifacthub.io/changes: |-
+    - kind: fixed
+      description: Missing .ytproxy in pod template
+    - kind: fixed
+      description: resource definitions in templates
     - kind: changed
-      description: Upgraded `postgresql` chart dependency to version 12.4.2
+      description: Added resources limits to container definitions
+

charts/apps/piped/README.md

@@ -1,6 +1,6 @@
 # piped
 
-![Version: 2.3.0](https://img.shields.io/badge/Version-2.3.0-informational?style=flat-square) ![AppVersion: latest](https://img.shields.io/badge/AppVersion-latest-informational?style=flat-square)
+![Version: 2.4.0](https://img.shields.io/badge/Version-2.4.0-informational?style=flat-square) ![AppVersion: latest](https://img.shields.io/badge/AppVersion-latest-informational?style=flat-square)
 
 Piped is an alternative privacy-friendly YouTube frontend which is efficient by design.
 
@@ -160,7 +160,7 @@ helm install piped TeamPiped/piped -f values.yaml
 
 ## Changelog
 
-### Version 2.3.0
+### Version 2.4.0
 
 #### Added
 
@@ -168,11 +168,12 @@ N/A
 
 #### Changed
 
-* Upgraded `postgresql` chart dependency to version 12.4.2
+* Added resources limits to container definitions
 
 #### Fixed
 
-N/A
+* Missing .ytproxy in pod template
+* resource definitions in templates
 
 ## Support
 

charts/apps/piped/templates/backend/lib/_container.tpl

@@ -3,6 +3,10 @@
 - name: {{ include "backend.names.fullname" . }}
   image: {{ printf "%s:%s" .Values.backend.image.repository (default .Chart.AppVersion .Values.backend.image.tag) | quote }}
   imagePullPolicy: {{ .Values.backend.image.pullPolicy }}
+  {{- with .Values.backend.resources }}
+  resources:
+    {{- toYaml . | nindent 4 }}
+  {{- end }}
   {{- with .Values.backend.command }}
   command:
     {{- if kindIs "string" . }}

charts/apps/piped/templates/backend/lib/_pod.tpl

@@ -102,8 +102,4 @@ topologySpreadConstraints:
 tolerations:
     {{- toYaml . | nindent 2 }}
   {{- end }}
-  {{- with .Values.backend.resources }}
-resources:
-   {{- toYaml . | nindent 2 }}
-  {{- end }}
 {{- end -}}

charts/apps/piped/templates/frontend/lib/_container.tpl

@@ -3,6 +3,10 @@
 - name: {{ include "frontend.names.fullname" . }}
   image: {{ printf "%s:%s" .Values.frontend.image.repository (default .Chart.AppVersion .Values.frontend.image.tag) | quote }}
   imagePullPolicy: {{ .Values.frontend.image.pullPolicy }}
+  {{- with .Values.frontend.resources }}
+  resources:
+    {{- toYaml . | nindent 4 }}
+  {{- end }}
   {{- with .Values.frontend.command }}
   command:
     {{- if kindIs "string" . }}

charts/apps/piped/templates/frontend/lib/_pod.tpl

@@ -98,8 +98,4 @@ topologySpreadConstraints:
 tolerations:
     {{- toYaml . | nindent 2 }}
   {{- end }}
-  {{- with .Values.frontend.resources }}
-resources:
-   {{- toYaml . | nindent 2 }}
-  {{- end }}
 {{- end -}}

charts/apps/piped/templates/ytproxy/lib/_container.tpl

@@ -3,6 +3,10 @@
 - name: {{ include "ytproxy.names.fullname" . }}
   image: {{ printf "%s:%s" .Values.ytproxy.image.repository (default .Chart.AppVersion .Values.ytproxy.image.tag) | quote }}
   imagePullPolicy: {{ .Values.ytproxy.image.pullPolicy }}
+  {{- with .Values.ytproxy.resources }}
+  resources:
+    {{- toYaml . | nindent 4 }}
+  {{- end }}
   {{- with .Values.ytproxy.command }}
   command:
     {{- if kindIs "string" . }}

charts/apps/piped/templates/ytproxy/lib/_pod.tpl

@@ -82,28 +82,24 @@ containers:
 volumes:
     {{- nindent 2 . }}
   {{- end }}
-  {{- with .Values.hostAliases }}
+  {{- with .Values.ytproxy.hostAliases }}
 hostAliases:
     {{- toYaml . | nindent 2 }}
   {{- end }}
-  {{- with .Values.nodeSelector }}
+  {{- with .Values.ytproxy.nodeSelector }}
 nodeSelector:
     {{- toYaml . | nindent 2 }}
   {{- end }}
-  {{- with .Values.affinity }}
+  {{- with .Values.ytproxy.affinity }}
 affinity:
     {{- toYaml . | nindent 2 }}
   {{- end }}
-  {{- with .Values.topologySpreadConstraints }}
+  {{- with .Values.ytproxy.topologySpreadConstraints }}
 topologySpreadConstraints:
     {{- toYaml . | nindent 2 }}
   {{- end }}
-  {{- with .Values.tolerations }}
+  {{- with .Values.ytproxy.tolerations }}
 tolerations:
     {{- toYaml . | nindent 2 }}
-    {{- end -}}
-    {{- with .Values.resources }}
-resources:
-   {{- toYaml . | nindent 2 }}
-  {{- end }}
+  {{- end -}}
 {{- end -}}