Skip to content

Commit ecf8493

Browse files
fix - remove archiveLogs=true as it breaks workflow execution with no configured artifact storage (#507)
1 parent a073e2f commit ecf8493

File tree

3 files changed

+19
-11
lines changed

3 files changed

+19
-11
lines changed

charts/gitops-runtime/README.md

Lines changed: 18 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ argo-workflows:
2525
controller:
2626
workflowDefaults:
2727
spec:
28+
archiveLogs: true
2829
artifactRepository:
2930
configMap: codefresh-workflows-log-store
3031
key: codefresh-workflows-log-store
@@ -210,14 +211,14 @@ sealed-secrets:
210211
| app-proxy.image-enrichment.serviceAccount.name | string | `"codefresh-image-enrichment-sa"` | Name of the service account to create or the name of the existing one to use |
211212
| app-proxy.image.pullPolicy | string | `"IfNotPresent"` | |
212213
| app-proxy.image.repository | string | `"quay.io/codefresh/cap-app-proxy"` | |
213-
| app-proxy.image.tag | string | `"1.3470.0"` | |
214+
| app-proxy.image.tag | string | `"1.3514.0"` | |
214215
| app-proxy.imagePullSecrets | list | `[]` | |
215216
| app-proxy.initContainer.command[0] | string | `"./init.sh"` | |
216217
| app-proxy.initContainer.env | object | `{}` | |
217218
| app-proxy.initContainer.extraVolumeMounts | list | `[]` | Extra volume mounts for init container |
218219
| app-proxy.initContainer.image.pullPolicy | string | `"IfNotPresent"` | |
219220
| app-proxy.initContainer.image.repository | string | `"quay.io/codefresh/cap-app-proxy-init"` | |
220-
| app-proxy.initContainer.image.tag | string | `"1.3470.0"` | |
221+
| app-proxy.initContainer.image.tag | string | `"1.3514.0"` | |
221222
| app-proxy.initContainer.resources.limits | object | `{}` | |
222223
| app-proxy.initContainer.resources.requests.cpu | string | `"0.2"` | |
223224
| app-proxy.initContainer.resources.requests.memory | string | `"256Mi"` | |
@@ -252,6 +253,9 @@ sealed-secrets:
252253
| app-proxy.serviceAccount.annotations | object | `{}` | |
253254
| app-proxy.serviceAccount.create | bool | `true` | |
254255
| app-proxy.serviceAccount.name | string | `"cap-app-proxy"` | |
256+
| app-proxy.serviceMonitor.enabled | bool | `false` | |
257+
| app-proxy.serviceMonitor.labels | object | `{}` | |
258+
| app-proxy.serviceMonitor.name | string | `""` | |
255259
| app-proxy.tolerations | list | `[]` | |
256260
| argo-cd.applicationVersioning.enabled | bool | `true` | Enable application versioning |
257261
| argo-cd.applicationVersioning.useApplicationConfiguration | bool | `true` | Extract application version based on ApplicationConfiguration CRD |
@@ -278,17 +282,19 @@ sealed-secrets:
278282
| argo-rollouts.enabled | bool | `true` | |
279283
| argo-rollouts.fullnameOverride | string | `"argo-rollouts"` | |
280284
| argo-rollouts.installCRDs | bool | `true` | |
281-
| argo-workflows.controller.workflowDefaults.spec.archiveLogs | bool | `true` | |
282285
| argo-workflows.crds.install | bool | `true` | Install and upgrade CRDs |
283286
| argo-workflows.enabled | bool | `true` | |
284287
| argo-workflows.executor.resources.requests.ephemeral-storage | string | `"10Mi"` | |
285288
| argo-workflows.fullnameOverride | string | `"argo"` | |
286289
| argo-workflows.mainContainer.resources.requests.ephemeral-storage | string | `"10Mi"` | |
287290
| argo-workflows.server.authModes | list | `["client"]` | auth-mode needs to be set to client to be able to see workflow logs from Codefresh UI |
288291
| argo-workflows.server.baseHref | string | `"/workflows/"` | Do not change. Workflows UI is only accessed through internal router, changing this values will break routing to workflows native UI from Codefresh. |
289-
| cf-argocd-extras | object | `{"eventReporter":{"affinity":{},"enabled":true,"nodeSelector":{},"tolerations":[]},"libraryMode":true,"sourcesServer":{"affinity":{},"enabled":true,"nodeSelector":{},"tolerations":[]}}` | Codefresh extra services for ArgoCD |
292+
| cf-argocd-extras | object | `{"eventReporter":{"affinity":{},"enabled":true,"nodeSelector":{},"pdb":{"enabled":false},"serviceMonitor":{"main":{"enabled":false}},"tolerations":[]},"libraryMode":true,"sourcesServer":{"affinity":{},"enabled":true,"hpa":{"enabled":true},"nodeSelector":{},"tolerations":[]}}` | Codefresh extra services for ArgoCD |
293+
| cf-argocd-extras.eventReporter.pdb.enabled | bool | `false` | Enable PDB for event reporter |
294+
| cf-argocd-extras.eventReporter.serviceMonitor.main.enabled | bool | `false` | Enable ServiceMonitor for event reporter |
290295
| cf-argocd-extras.libraryMode | bool | `true` | Library mode for the chart. Allows to inject values from gitops runtime chart |
291-
| cf-argocd-extras.sourcesServer | object | `{"affinity":{},"enabled":true,"nodeSelector":{},"tolerations":[]}` | Sources server configuration |
296+
| cf-argocd-extras.sourcesServer | object | `{"affinity":{},"enabled":true,"hpa":{"enabled":true},"nodeSelector":{},"tolerations":[]}` | Sources server configuration |
297+
| cf-argocd-extras.sourcesServer.hpa.enabled | bool | `true` | Enable HPA for sources server |
292298
| codefreshWorkflowLogStoreCM | object | `{"enabled":true,"endpoint":"gitops-workflow-logs.codefresh.io","insecure":false}` | Argo workflows logs storage on Codefresh platform settings. Don't change unless instructed by Codefresh support. |
293299
| event-reporters.rollout.eventSource.affinity | object | `{}` | |
294300
| event-reporters.rollout.eventSource.nodeSelector | object | `{}` | |
@@ -327,13 +333,17 @@ sealed-secrets:
327333
| event-reporters.workflow.sensor.tolerations | list | `[]` | |
328334
| event-reporters.workflow.serviceAccount.create | bool | `true` | |
329335
| gitops-operator.affinity | object | `{}` | |
336+
| gitops-operator.config.commitStatusPollingInterval | string | `"10s"` | Commit status polling interval |
337+
| gitops-operator.config.maxConcurrentReleases | int | `100` | Maximum number of concurrent releases being processed by the operator (this will not affect the number of releases being processed by the gitops runtime) |
338+
| gitops-operator.config.promotionWrapperTemplate | string | `""` | An optional template for the promotion wrapper (empty default will use the embedded one) |
339+
| gitops-operator.config.taskPollingInterval | string | `"10s"` | Task polling interval |
340+
| gitops-operator.config.workflowMonitorPollingInterval | string | `"10s"` | Workflow monitor polling interval |
330341
| gitops-operator.crds | object | `{"additionalLabels":{},"annotations":{},"install":true,"keep":false}` | Codefresh gitops operator crds |
331342
| gitops-operator.crds.additionalLabels | object | `{}` | Additional labels for gitops operator CRDs |
332343
| gitops-operator.crds.annotations | object | `{}` | Annotations on gitops operator CRDs |
333344
| gitops-operator.crds.install | bool | `true` | Whether or not to install CRDs |
334345
| gitops-operator.crds.keep | bool | `false` | Keep CRDs if gitops runtime release is uninstalled |
335346
| gitops-operator.enabled | bool | `true` | |
336-
| gitops-operator.env.TASK_PULLING_INTERVAL | string | `"10s"` | |
337347
| gitops-operator.fullnameOverride | string | `""` | |
338348
| gitops-operator.image | object | `{}` | |
339349
| gitops-operator.imagePullSecrets | list | `[]` | |
@@ -405,7 +415,8 @@ sealed-secrets:
405415
| global.runtime.isConfigurationRuntime | bool | `false` | is the runtime set as a "configuration runtime". |
406416
| global.runtime.name | string | `nil` | Runtime name. Must be unique per platform account. |
407417
| global.tolerations | list | `[]` | Global tolerations for all components |
408-
| installer | object | `{"affinity":{},"argoCdVersionCheck":{"argoServerLabels":{"app.kubernetes.io/component":"server","app.kubernetes.io/part-of":"argocd"}},"image":{"pullPolicy":"IfNotPresent","repository":"quay.io/codefresh/gitops-runtime-installer","tag":""},"nodeSelector":{},"skipValidation":false,"tolerations":[]}` | Runtime installer used for running hooks and checks on the release |
418+
| installer | object | `{"affinity":{},"argoCdVersionCheck":{"argoServerLabels":{"app.kubernetes.io/component":"server","app.kubernetes.io/part-of":"argocd"}},"image":{"pullPolicy":"IfNotPresent","repository":"quay.io/codefresh/gitops-runtime-installer","tag":""},"nodeSelector":{},"skipUsageValidation":false,"skipValidation":false,"tolerations":[]}` | Runtime installer used for running hooks and checks on the release |
419+
| installer.skipUsageValidation | bool | `false` | if set to true, pre-install hook will *not* run |
409420
| installer.skipValidation | bool | `false` | if set to true, pre-install hook will *not* run |
410421
| internal-router.affinity | object | `{}` | |
411422
| internal-router.clusterDomain | string | `"cluster.local"` | |

charts/gitops-runtime/README.md.gotmpl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ argo-workflows:
2525
controller:
2626
workflowDefaults:
2727
spec:
28+
archiveLogs: true
2829
artifactRepository:
2930
configMap: codefresh-workflows-log-store
3031
key: codefresh-workflows-log-store

charts/gitops-runtime/values.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -315,10 +315,6 @@ argo-workflows:
315315
resources:
316316
requests:
317317
ephemeral-storage: 10Mi
318-
controller:
319-
workflowDefaults:
320-
spec:
321-
archiveLogs: true
322318

323319
# -- Argo workflows logs storage on Codefresh platform settings. Don't change unless instructed by Codefresh support.
324320
codefreshWorkflowLogStoreCM:

0 commit comments

Comments
 (0)