Merge pull request #139 from codefresh-io/CR-28096-argo-cd-event-reporter-removal

feat(argo-cd): event-reporter removal

Author: oleksandr-codefresh

charts/argo-cd/Chart.yaml

@@ -3,7 +3,7 @@ appVersion: v2.14.9-2025-04-23-4de04dd8
 kubeVersion: ">=1.25.0-0"
 description: A Helm chart for Argo CD, a declarative, GitOps continuous delivery tool for Kubernetes.
 name: argo-cd
-version: 7.8.23-2-cap-v2.14.9-2025-04-23-4de04dd8
+version: 7.8.23-3-cap-v2.14.9-2025-04-23-4de04dd8
 home: https://github.com/argoproj/argo-helm
 icon: https://argo-cd.readthedocs.io/en/stable/assets/logo.png
 sources:
@@ -27,54 +27,4 @@ annotations:
     url: https://argoproj.github.io/argo-helm/pgp_keys.asc
   artifacthub.io/changes: |
     - kind: changed
-      description: downgraded helm/chart-releaser-action to v1.5.0 to avoid issue https://github.com/helm/chart-releaser-action/issues/171
-    - kind: changed
-      description: bumped argo-cd to v2.14.9-2025-04-20-584fc7f3
-    - kind: fixed
-      description: Secret for embedded Redis deployments now always uses the same name and key
-    - kind: fixed
-      description: Make code clearer when Redis Secret is optional
-    - kind: fixed
-      description: Correct oliver006/redis_exporter image tag
-    - kind: fixed
-      description: Moved to oliver006/redis_exporter to support multi-arch images (also for non-HA redis)
-    - kind: fixed
-      description: Remove quotation marks around loglevel and logformat CLI args
-    - kind: fixed
-      description: Fixed the log level and log format override for components "dexserver" and "notificationscontroller"
-    - kind: deprecated
-      description: Value `notifications.logLevel` is depreacted in favor of `configs.params."notificationscontroller.log.level"`
-    - kind: deprecated
-      description: Value `notifications.logFormat` is depreacted in favor of `configs.params."notificationscontroller.log.format"`
-    - kind: deprecated
-      description: Value `dex.logLevel` is depreacted in favor of `configs.params."dexserver.log.level"`
-    - kind: deprecated
-      description: Value `dex.logFormat` is depreacted in favor of `configs.params."dexserver.log.format"`
-    - kind: changed
-      description: Upgrade redis-ha
-    - kind: changed
-      description: Bump redis-exporter to 1.69.0
-    - kind: fixed
-      description: Fixed broken topologySpreadConstraints template in commitServer component
-    - kind: added
-      description: Added a comment how to pass values into argocd-cm ConfigMap
-    - kind: added
-      description: Support controller.sync.timeout.seconds
-    - kind: changed
-      description: Bump dex version to v2.42.0
-    - kind: fixed
-      description: Create ServiceMonitor for dex only if dex is enabled
-    - kind: fixed
-      description: Fixed typo in README
-    - kind: fixed
-      description: remove required function in $cluster_key from clusterCredentials
-    - kind: changed
-      description: Bump redis to 7.4.2
-    - kind: added
-      description: Add feature toggle to ArgoCD configmap allowing sync impersonation
-    - kind: fixed
-      description: Address ingress regression of PR #3081, use toString before tpl function
-    - kind: added
-      description: Added conditional around argocd-tls-certs-cm.yaml helm creation
-    - kind: fixed
-      description: Correct ingress listen-ports example
+      description: removed event-reporter resources

charts/argo-cd/README.md

@@ -719,80 +719,6 @@ NAME: my-release
 | crds.keep | bool | `true` | Keep CRDs on chart uninstall |
 | createAggregateRoles | bool | `false` | Create aggregated roles that extend existing cluster roles to interact with argo-cd resources |
 | createClusterRoles | bool | `true` | Create cluster roles for cluster-wide installation. |
-| eventReporter.affinity | object | `{}` (defaults to global.affinity preset) | Assign custom [affinity] rules to the deployment |
-| eventReporter.clusterRoleRules.enabled | bool | `false` | Enable custom rules for the event reporter's ClusterRole resource |
-| eventReporter.clusterRoleRules.rules | list | `[]` | List of custom rules for the event reporter's ClusterRole resource |
-| eventReporter.containerPorts.health | int | `8088` |  |
-| eventReporter.containerPorts.metrics | int | `8087` | Metrics container port |
-| eventReporter.containerSecurityContext | object | See [values.yaml] | Event reporter container-level security context |
-| eventReporter.dnsConfig | object | `{}` | [DNS configuration] |
-| eventReporter.dnsPolicy | string | `"ClusterFirst"` | Alternative DNS policy for event reporter pods |
-| eventReporter.enabled | bool | `true` |  |
-| eventReporter.env | list | `[]` | Environment variables to pass to event reporter |
-| eventReporter.envFrom | list | `[]` (See [values.yaml]) | envFrom to pass to event reporter |
-| eventReporter.extraArgs | list | `[]` | Additional command line arguments to pass to event reporter |
-| eventReporter.extraContainers | list | `[]` | Additional containers to be added to the event reporter pod |
-| eventReporter.hostNetwork | bool | `false` | Host Network for event reporter pods |
-| eventReporter.image.imagePullPolicy | string | `""` (defaults to global.image.imagePullPolicy) | Image pull policy for the event reporter |
-| eventReporter.image.repository | string | `""` (defaults to global.image.repository) | Repository to use for the event reporter |
-| eventReporter.image.tag | string | `""` (defaults to global.image.tag) | Tag to use for the event reporter |
-| eventReporter.imagePullSecrets | list | `[]` (defaults to global.imagePullSecrets) | Secrets with credentials to pull images from a private registry |
-| eventReporter.initContainers | list | `[]` | Init containers to add to the event reporter pod |
-| eventReporter.metrics.applicationLabels.enabled | bool | `false` | Enables additional labels in argocd_app_labels metric |
-| eventReporter.metrics.applicationLabels.labels | list | `[]` | Additional labels |
-| eventReporter.metrics.enabled | bool | `false` | Deploy metrics service |
-| eventReporter.metrics.rules.additionalLabels | object | `{}` | PrometheusRule labels |
-| eventReporter.metrics.rules.annotations | object | `{}` | PrometheusRule annotations |
-| eventReporter.metrics.rules.enabled | bool | `false` | Deploy a PrometheusRule for the event reporter |
-| eventReporter.metrics.rules.namespace | string | `""` | PrometheusRule namespace |
-| eventReporter.metrics.rules.selector | object | `{}` | PrometheusRule selector |
-| eventReporter.metrics.rules.spec | list | `[]` | PrometheusRule.Spec for the event reporter |
-| eventReporter.metrics.service.annotations | object | `{}` | Metrics service annotations |
-| eventReporter.metrics.service.clusterIP | string | `""` | Metrics service clusterIP. `None` makes a "headless service" (no virtual IP) |
-| eventReporter.metrics.service.labels | object | `{}` | Metrics service labels |
-| eventReporter.metrics.service.portName | string | `"http-metrics"` | Metrics service port name |
-| eventReporter.metrics.service.servicePort | int | `8087` | Metrics service port |
-| eventReporter.metrics.service.type | string | `"ClusterIP"` | Metrics service type |
-| eventReporter.metrics.serviceMonitor.additionalLabels | object | `{}` | Prometheus ServiceMonitor labels |
-| eventReporter.metrics.serviceMonitor.annotations | object | `{}` | Prometheus ServiceMonitor annotations |
-| eventReporter.metrics.serviceMonitor.enabled | bool | `false` | Enable a prometheus ServiceMonitor |
-| eventReporter.metrics.serviceMonitor.interval | string | `"30s"` | Prometheus ServiceMonitor interval |
-| eventReporter.metrics.serviceMonitor.metricRelabelings | list | `[]` | Prometheus [MetricRelabelConfigs] to apply to samples before ingestion |
-| eventReporter.metrics.serviceMonitor.namespace | string | `""` | Prometheus ServiceMonitor namespace |
-| eventReporter.metrics.serviceMonitor.relabelings | list | `[]` | Prometheus [RelabelConfigs] to apply to samples before scraping |
-| eventReporter.metrics.serviceMonitor.scheme | string | `""` | Prometheus ServiceMonitor scheme |
-| eventReporter.metrics.serviceMonitor.selector | object | `{}` | Prometheus ServiceMonitor selector |
-| eventReporter.metrics.serviceMonitor.tlsConfig | object | `{}` | Prometheus ServiceMonitor tlsConfig |
-| eventReporter.name | string | `"event-reporter"` |  |
-| eventReporter.nodeSelector | object | `{}` (defaults to global.nodeSelector) | [Node selector] |
-| eventReporter.pdb.annotations | object | `{}` | Annotations to be added to event reporter pdb |
-| eventReporter.pdb.enabled | bool | `false` | Deploy a [PodDisruptionBudget] for the event reporter |
-| eventReporter.pdb.labels | object | `{}` | Labels to be added to event reporter pdb |
-| eventReporter.pdb.maxUnavailable | string | `""` | Number of pods that are unavailable after eviction as number or percentage (eg.: 50%). |
-| eventReporter.pdb.minAvailable | string | `""` (defaults to 0 if not specified) | Number of pods that are available after eviction as number or percentage (eg.: 50%) |
-| eventReporter.podAnnotations | object | `{}` | Annotations to be added to event reporter pods |
-| eventReporter.podLabels | object | `{}` | Labels to be added to event reporter pods |
-| eventReporter.priorityClassName | string | `""` (defaults to global.priorityClassName) | Priority class for the event reporter pods |
-| eventReporter.readinessProbe.failureThreshold | int | `3` | Minimum consecutive failures for the [probe] to be considered failed after having succeeded |
-| eventReporter.readinessProbe.initialDelaySeconds | int | `10` | Number of seconds after the container has started before [probe] is initiated |
-| eventReporter.readinessProbe.periodSeconds | int | `10` | How often (in seconds) to perform the [probe] |
-| eventReporter.readinessProbe.successThreshold | int | `1` | Minimum consecutive successes for the [probe] to be considered successful after having failed |
-| eventReporter.readinessProbe.timeoutSeconds | int | `1` | Number of seconds after which the [probe] times out |
-| eventReporter.replicas | int | `3` |  |
-| eventReporter.resources | object | `{}` | Resource limits and requests for the event reporter pods |
-| eventReporter.serviceAccount.annotations | object | `{}` | Annotations applied to created service account |
-| eventReporter.serviceAccount.automountServiceAccountToken | bool | `true` | Automount API credentials for the Service Account |
-| eventReporter.serviceAccount.create | bool | `true` | Create a service account for the event reporter |
-| eventReporter.serviceAccount.labels | object | `{}` | Labels applied to created service account |
-| eventReporter.serviceAccount.name | string | `"event-reporter"` | Service account name |
-| eventReporter.sourcesServer.baseURL | string | `"http://sources-server"` | base URL of sources-server |
-| eventReporter.sourcesServer.enabled | bool | `false` | Use sources-server logic instead of repo-server |
-| eventReporter.statefulsetAnnotations | object | `{}` | Annotations for the event reporter StatefulSet |
-| eventReporter.terminationGracePeriodSeconds | int | `30` | terminationGracePeriodSeconds for container lifecycle hook |
-| eventReporter.tolerations | list | `[]` (defaults to global.tolerations) | [Tolerations] for use with node taints |
-| eventReporter.topologySpreadConstraints | list | `[]` (defaults to global.topologySpreadConstraints) | Assign custom [TopologySpreadConstraints] rules to the event reporter |
-| eventReporter.volumeMounts | list | `[]` | Additional volumeMounts to the event reporter main container |
-| eventReporter.volumes | list | `[]` | Additional volumes to the event reporter pod |
 | extraObjects | list | `[]` | Array of extra K8s manifests to deploy |
 | fullnameOverride | string | `""` | String to fully override `"argo-cd.fullname"` |
 | kubeVersionOverride | string | `""` | Override the Kubernetes version, which is used to evaluate certain manifests |

charts/argo-cd/templates/_helpers.tpl

@@ -316,34 +316,6 @@ optional: true
     {{- end -}}
 {{- end -}}
 
-{{/*
-Create event reporter name and version as used by the chart label.
-*/}}
-{{- define "argo-cd.event-reporter.fullname" -}}
-{{- printf "%s-%s" (include "argo-cd.fullname" .) .Values.eventReporter.name | trunc 63 | trimSuffix "-" -}}
-{{- end -}}
-{{/*
-Create the name of the Argo CD server service account to use
-*/}}
-{{- define "argo-cd.eventReporterServiceAccountName" -}}
-{{- if .Values.eventReporter.serviceAccount.create -}}
-    {{ default (include "argo-cd.event-reporter.fullname" .) .Values.eventReporter.serviceAccount.name }}
-{{- else -}}
-    {{ default "default" .Values.eventReporter.serviceAccount.name }}
-{{- end -}}
-{{- end -}}
-
-{{/*
-Helper template to set argocd server url in event reporter
-*/}}
-{{- define "argo-cd.eventReporter.argocd-server-adress" -}}
-  {{- $port := .Values.server.service.servicePortHttps }}
-  {{- if (index .Values.configs.params "server.insecure") }}
-    {{- $port = .Values.server.service.servicePortHttp }}
-  {{- end }}
-  {{- printf "%s:%v" (include "argo-cd.server.fullname" .) $port }}
-{{- end -}}
-
 {{/*
   Create acr controller name and version as used by the chart label.
 */}}