Skip to content

Commit

Permalink
fix the invalid fmt of dataload cronJob helm template which leads to …
Browse files Browse the repository at this point in the history
…YAML parse error (#4360)

Signed-off-by: jiuyu <[email protected]>
Co-authored-by: jiuyu <[email protected]>
  • Loading branch information
Syspretor and jiuyu authored Oct 16, 2024
1 parent c0cf719 commit 3e38c94
Show file tree
Hide file tree
Showing 34 changed files with 83 additions and 146 deletions.
2 changes: 1 addition & 1 deletion charts/alluxio/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,5 +29,5 @@ maintainers:
email: [email protected]
dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../library"
2 changes: 1 addition & 1 deletion charts/efc/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,5 @@ maintainers:

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../library"
2 changes: 1 addition & 1 deletion charts/fluid-databackup/alluxio/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@ appVersion: 0.1.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
2 changes: 1 addition & 1 deletion charts/fluid-databackup/goosefs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@ appVersion: 0.1.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
5 changes: 4 additions & 1 deletion charts/fluid-dataloader/alluxio/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,7 @@
- Dataload support Affinity,Tolerations,NodeSelector,SchedulerName

### 0.10.0
- Support cron dataload
- Support cron dataload

### 0.10.3
- Fix incorrect indentation of cron dataload template
4 changes: 2 additions & 2 deletions charts/fluid-dataloader/alluxio/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.3.0
version: 0.10.3

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand All @@ -24,5 +24,5 @@ appVersion: 0.3.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
21 changes: 1 addition & 20 deletions charts/fluid-dataloader/alluxio/templates/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,26 +61,7 @@ spec:
{{- end }}
{{- end }}
spec:
{{- if .Values.dataloader.schedulerName }}
schedulerName: {{ .Values.dataloader.schedulerName }}
{{- end }}
{{- with .Values.dataloader.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.affinity }}
affinity:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
restartPolicy: Never
{{- with .Values.dataloader.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- include "library.fluid.dataload.cronJobCommonTemplateSpec" . | nindent 10 }}
containers:
- name: dataloader
image: {{ required "Dataloader image should be set" .Values.dataloader.image }}
Expand Down
5 changes: 4 additions & 1 deletion charts/fluid-dataloader/goosefs/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,7 @@
- Support for setting affinity,tolerations,nodeSelector,schedulerName

### 0.10.0
- Support cron dataload
- Support cron dataload

### 0.10.3
- Fix incorrect indentation of cron dataload template
4 changes: 2 additions & 2 deletions charts/fluid-dataloader/goosefs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.0
version: 0.10.3

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand All @@ -24,5 +24,5 @@ appVersion: 0.1.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
21 changes: 1 addition & 20 deletions charts/fluid-dataloader/goosefs/templates/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,26 +48,7 @@ spec:
cronjob: {{ printf "%s-job" .Release.Name }}
targetDataset: {{ required "targetDataset should be set" .Values.dataloader.targetDataset }}
spec:
{{- if .Values.dataloader.schedulerName }}
schedulerName: {{ .Values.dataloader.schedulerName }}
{{- end }}
{{- with .Values.dataloader.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.affinity }}
affinity:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
restartPolicy: Never
{{- with .Values.dataloader.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- include "library.fluid.dataload.cronJobCommonTemplateSpec" . | nindent 10 }}
containers:
- name: dataloader
image: {{ required "Dataloader image should be set" .Values.dataloader.image }}
Expand Down
5 changes: 4 additions & 1 deletion charts/fluid-dataloader/jindo/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,7 @@
- Support for setting affinity,tolerations,nodeSelector,schedulerName

### 0.10.0
- Support cron dataload
- Support cron dataload

### 0.10.3
- Fix incorrect indentation of cron dataload template
4 changes: 2 additions & 2 deletions charts/fluid-dataloader/jindo/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.2.0
version: 0.10.3

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand All @@ -24,5 +24,5 @@ appVersion: 0.2.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
21 changes: 1 addition & 20 deletions charts/fluid-dataloader/jindo/templates/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,26 +51,7 @@ spec:
cronjob: {{ printf "%s-job" .Release.Name }}
targetDataset: {{ required "targetDataset should be set" .Values.dataloader.targetDataset }}
spec:
{{- if .Values.dataloader.schedulerName }}
schedulerName: {{ .Values.dataloader.schedulerName }}
{{- end }}
{{- with .Values.dataloader.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.affinity }}
affinity:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
restartPolicy: Never
{{- with .Values.dataloader.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- include "library.fluid.dataload.cronJobCommonTemplateSpec" . | nindent 10 }}
containers:
- name: dataloader
image: {{ required "Dataloader image should be set" .Values.dataloader.image }}
Expand Down
3 changes: 3 additions & 0 deletions charts/fluid-dataloader/jindocache/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
### 0.10.3
- Fix incorrect indentation of cron dataload template

### 0.10.1
- support force load data option

Expand Down
4 changes: 2 additions & 2 deletions charts/fluid-dataloader/jindocache/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.5.0
version: 0.10.3

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand All @@ -24,5 +24,5 @@ appVersion: 0.5.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
21 changes: 1 addition & 20 deletions charts/fluid-dataloader/jindocache/templates/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,26 +61,7 @@ spec:
{{- end }}
{{- end }}
spec:
{{- if .Values.dataloader.schedulerName }}
schedulerName: {{ .Values.dataloader.schedulerName }}
{{- end }}
{{- with .Values.dataloader.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.affinity }}
affinity:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
restartPolicy: Never
{{- with .Values.dataloader.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- include "library.fluid.dataload.cronJobCommonTemplateSpec" . | nindent 10 }}
containers:
- name: dataloader
image: {{ required "Dataloader image should be set" .Values.dataloader.image }}
Expand Down
2 changes: 2 additions & 0 deletions charts/fluid-dataloader/jindofsx/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
### 0.10.3
- Fix incorrect indentation of cron dataload template

### 0.10.0
- Support cron dataload
Expand Down
4 changes: 2 additions & 2 deletions charts/fluid-dataloader/jindofsx/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.5.0
version: 0.10.3

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand All @@ -24,5 +24,5 @@ appVersion: 0.5.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
21 changes: 1 addition & 20 deletions charts/fluid-dataloader/jindofsx/templates/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,26 +61,7 @@ spec:
{{- end }}
{{- end }}
spec:
{{- if .Values.dataloader.schedulerName }}
schedulerName: {{ .Values.dataloader.schedulerName }}
{{- end }}
{{- with .Values.dataloader.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.affinity }}
affinity:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
restartPolicy: Never
{{- with .Values.dataloader.imagePullSecrets }}
imagePullSecrets:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- include "library.fluid.dataload.cronJobCommonTemplateSpec" . | nindent 10 }}
containers:
- name: dataloader
image: {{ required "Dataloader image should be set" .Values.dataloader.image }}
Expand Down
5 changes: 4 additions & 1 deletion charts/fluid-dataloader/juicefs/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,7 @@
- Support for setting affinity,tolerations,nodeSelector,schedulerName

### 0.10.0
- Support cron dataload
- Support cron dataload

### 0.10.3
- Fix incorrect indentation of cron dataload template
4 changes: 2 additions & 2 deletions charts/fluid-dataloader/juicefs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.2.0
version: 0.10.3

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
Expand All @@ -24,5 +24,5 @@ appVersion: 0.2.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
17 changes: 1 addition & 16 deletions charts/fluid-dataloader/juicefs/templates/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,22 +61,7 @@ spec:
{{- end }}
{{- end }}
spec:
{{- if .Values.dataloader.schedulerName }}
schedulerName: {{ .Values.dataloader.schedulerName }}
{{- end }}
{{- with .Values.dataloader.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.affinity }}
affinity:
{{- toYaml . | nindent 8 }}
{{- end }}
{{- with .Values.dataloader.tolerations }}
tolerations:
{{- toYaml . | nindent 8 }}
{{- end }}
restartPolicy: Never
{{- include "library.fluid.dataload.cronJobCommonTemplateSpec" . | nindent 10 }}
{{- range $key, $val := .Values.dataloader.options }}
{{- if eq $key "runtimeName" }}
serviceAccountName: {{ printf "%s-loader" $val | quote }}
Expand Down
2 changes: 1 addition & 1 deletion charts/fluid-datamigrate/juicefs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@ appVersion: 0.1.0

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
2 changes: 1 addition & 1 deletion charts/fluid-dataprocess/common/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,5 @@ appVersion: "1.16.0"

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../../library"
2 changes: 1 addition & 1 deletion charts/goosefs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ maintainers:
email: [email protected]
dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../library"
2 changes: 1 addition & 1 deletion charts/jindocache/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,5 +20,5 @@ name: jindofs

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../library"
2 changes: 1 addition & 1 deletion charts/jindofs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,5 +20,5 @@ name: jindofs

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../library"
2 changes: 1 addition & 1 deletion charts/jindofsx/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,5 +20,5 @@ name: jindofs

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../library"
2 changes: 1 addition & 1 deletion charts/juicefs/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,5 @@ maintainers:

dependencies:
- name: library
version: "0.1.0"
version: "0.2.0"
repository: "file://../library"
Loading

0 comments on commit 3e38c94

Please sign in to comment.