Fix collectorDataSource retention env var conditions (#336)

* Fix collectorDataSource retention env var conditions

The condition checks for retention10m, retention1h, and retention1d were
incorrectly checking for non-existent field names (retentionResolution10m,
retentionResolution1h, retentionResolution1d), causing the env vars to never
be set even when the values were configured in values.yaml.

* Bump chart version to 2.5.9

Author: dag-andersen

charts/opencost/Chart.yaml

@@ -9,7 +9,7 @@ keywords:
 - finops
 - monitoring
 - opencost
-version: 2.5.8
+version: 2.5.9
 maintainers:
   - name: jessegoodier
   - name: toscott

charts/opencost/templates/deployment.yaml

@@ -165,15 +165,15 @@ spec:
             - name: NETWORK_PORT
               value: {{ .networkPort | quote }}
             {{- end }}
-            {{- if .retentionResolution10m }}
+            {{- if .retention10m }}
             - name: COLLECTOR_RESOLUTION_10M_RETENTION
               value: {{ .retention10m | quote }}
             {{- end }}
-            {{- if .retentionResolution1h }}
+            {{- if .retention1h }}
             - name: COLLECTOR_RESOLUTION_1H_RETENTION
               value: {{ .retention1h | quote }}
             {{- end }}
-            {{- if .retentionResolution1d }}
+            {{- if .retention1d }}
             - name: COLLECTOR_RESOLUTION_1D_RETENTION
               value: {{ .retention1d | quote }}
             {{- end }}