* feat(argo-workflows) Include pod name in key format for GCS logs
A better default would be to add the `pod.name` to the key format of the Argo logs.
I noticed that when not including the pod name, it would constantly overwrite the same log from from different pods.
Including it makes sure that each pod will have its own log file.
Signed-off-by: Michiel De Mey <de.mey.michiel@gmail.com>
* Bump chart version and added changelog
Signed-off-by: Michiel De Mey <de.mey.michiel@gmail.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* [v0.2.2] Add missing config map key for git commit-message-template
Signed-off-by: Niklas Jönsson <niklas@pej.se>
* Sync some other things from upstream manifests
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Quote bools
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix: Rerun helm-docs
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Use helm-docs for argo-workflows
Signed-off-by: Tuan Anh Tran <me@tuananh.org>
* bump version and update changelog
Signed-off-by: Tuan Anh Tran <me@tuananh.org>
* ci: add step to check if docs is staled and needs update
Signed-off-by: Tuan Anh Tran <me@tuananh.org>
* docs: run helm-docs to generate readme
Signed-off-by: Tuan Anh Tran <me@tuananh.org>
* Add missing parameter documentation
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add .helmdocsignore for charts which not yet use helm-docs
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Do not exclude argocd-notifications anymore
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* docs(argocd-notifications): Use helm-docs
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Use chart appVersion as default image tag
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add signature keys for GPG in additionalProjects
Signed-off-by: Jakub Faber <jakub.faber@camabeh.dev>
* Fix whitespace
Signed-off-by: Jakub Faber <jakub.faber@camabeh.dev>
* Fix lint - Newline at the end of file
Signed-off-by: Jakub Faber <jakub.faber@camabeh.dev>
* feat(argocd-image-updater): Optionally expose metrics
Signed-off-by: Jan Pieper <kontakt@jan-pieper.info>
* Apply suggestions from code review
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add missing link references to README
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat: Allow additional containers in controller deployment
Our local metrics setup uses an extra container for the controller
deployment. Adding this in case others have a similar use case.
Signed-off-by: Brian Johnson <b2jrock@gmail.com>
* Apply suggesstions from code review
Signed-off-by: Brian Johnson <b2jrock@gmail.com>
* Bump minor version
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix(helm3): install does not have flag --name
Signed-off-by: Cyril Corbon <corboncyril@gmail.com>
* Update charts/argo-cd/Chart.yaml
Co-authored-by: Marko Bevc <marko.bevc@gmail.com>
Signed-off-by: Cyril Corbon <corboncyril@gmail.com>
Co-authored-by: Marko Bevc <marko.bevc@gmail.com>
* Update image and tag for argo events deployments
Signed-off-by: Paul Ayling <paul.ayling@sky.uk>
* Update Nats images
Signed-off-by: Paul Ayling <paul.ayling@sky.uk>
* Update sensorImage and eventSourceImage
Signed-off-by: Paul Ayling <paul.ayling@sky.uk>
* Update Chart.yaml
Signed-off-by: Paul Ayling <paul.ayling@sky.uk>
* Place new argument at end to work with old images
Signed-off-by: Paul Ayling <paul.ayling@sky.uk>
* Update chart appversion and RBAC
Signed-off-by: Paul Ayling <paulayling.dev@gmail.com>
* Apply suggestions from code review
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Signed-off-by: Paul Ayling <paulayling.dev@gmail.com>
* Default to Chart AppVersion if tag does not exist
Signed-off-by: Paul Ayling <paulayling.dev@gmail.com>
* Update Chart AppVersion to v1.5.0
Signed-off-by: Paul Ayling <paulayling.dev@gmail.com>
* Update changelog
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix(argo-cd): Update to app version v2.1.5
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* chore: Use chart appVersion as global default imageTag
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-cd): Add ability to pass finalizers to AppProject
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* chore: Use 'with' instead of 'if' and beautify indent style
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* docs: Add finalizers to example in values.yaml
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* chore: Reconfigure KinD to 1 control-plane and 3 worker nodes
This prepares chart testing of 'argo-cd' chart with redis-ha enabled. Redis-ha defines hard pod anti-affinity and requires multiple worker nodes.
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* chore(argo-cd): Add chart tests for HA
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add HA chapter to README
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Skip HPA tests of ArgoCD
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-rollouts): Support extraArgs for controller and dashboard
Adding support for additional arguments that are passed to the controller and the dashboard.
Following the pattern of other Argo charts by re-using the name `extraArgs` for the value.
Refs: https://github.com/argoproj/argo-helm/issues/969
Signed-off-by: Sebastian J <zeeman@zeeman.de>
* Update charts/argo-rollouts/templates/controller/deployment.yaml
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Signed-off-by: Sebastian J <zeeman@zeeman.de>
* Update charts/argo-rollouts/templates/dashboard/deployment.yaml
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Signed-off-by: Sebastian J <zeeman@zeeman.de>
* PR comments
Signed-off-by: Sebastian J <zeeman@zeeman.de>
* Fix newly added parameters in README
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Fix changelog keyword
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-workflows): Bump appVersion to 3.2.0
Signed-off-by: Louis Haußknecht <lh@haussknecht.com>
* feat(argo-workflows): Add WorkflowTaskSet to CRD
Signed-off-by: Louis Haußknecht <lh@haussknecht.com>
* feat(argo-workflows): Synced CRDs with https://github.com/argoproj/argo-workflows/tree/v3.2.0/manifests/base/crds/minimal.
Signed-off-by: Louis Haußknecht <lh@haussknecht.com>
* Bump Chart version
Signed-off-by: Marko Bevc <marko@scalefactory.com>
Co-authored-by: Marko Bevc <marko@scalefactory.com>
* Create optional serviceAnnotations value for redis in helm chart
Signed-off-by: arunkumar.anandhan <arunkumar.anandhan@niceincontact.com>
* Uncomment accidentally commented lines
Signed-off-by: arunkumar.anandhan <arunkumar.anandhan@niceincontact.com>
* update chart version
Signed-off-by: arunkumar.anandhan <arunkumar.anandhan@niceincontact.com>
* update service annotation key for redis
Signed-off-by: arunkumar.anandhan <arunkumar.anandhan@niceincontact.com>
* Update L24 with appropriate description
Signed-off-by: arunkumar.anandhan <arunkumar.anandhan@niceincontact.com>
* Move redis.service key occurence in values
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Also implement additional labels
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add documentation
- Changelog annotations
- Add new values to README
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix: Use correct value for redis service targetPort
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* support annotations on secret objects
Signed-off-by: Marcial White <marcial@kaskada.com>
* true-up artifacthub changes
Signed-off-by: Marcial White <marcial@kaskada.com>
Co-authored-by: Marko Bevc <marko@scalefactory.com>
* fix issue 943
Signed-off-by: Artem Kosenko <artem.kosenko@thomsonreuters.com>
* Apply changes from code review
Changes in detail:
- Add missing examples/documentation in values.yaml
- Beautify indentation
- Use 'with' instead of 'if' to don't repeat variable names
- Fix changelog annotation
- Bump minor chart version as this adds a feature
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Update README
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marko Bevc <marko@scalefactory.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat: Add hostAliases to dex/deployment
based on https://github.com/argoproj/argo-helm/pull/274
Signed-off-by: Sinhyeok <sinhyeok@gmail.com>
* Bumped version number of argo-cd chart
Signed-off-by: Sinhyeok <sinhyeok@gmail.com>
* Beautify indentation
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Beautify indentation of existing implementation
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Update changelog
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: attar.sh <attar.sh@kakaobank.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Allow NodePort to be specified for server service.
Signed-off-by: Peter Oruba <peter@oruba.de>
* Different version number and different condition in NodePort setting evaluation, both as per PR review suggested.
Signed-off-by: Peter Oruba <peter@oruba.de>
* docs: update default values example for extraArgs
Signed-off-by: Tuan Anh Tran <me@tuananh.org>
* Fix noeol in Chart.yaml
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-cd): Ability to provide cluster role to repo-server
Signed-off-by: Roman Rudenko <3kmnazapad@gmail.com>
* custom clusterRoleRules
Signed-off-by: Roman Rudenko <3kmnazapad@gmail.com>
* chore: beautify indentation
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix: noeol in clusterrolebinding.yaml
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Roman Rudenko <roman.rudenko@telekom.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Allow setting the metrics port name
Here, we scrape every pod that has a .*-metrics port here
But the helm chart is locked to `metrics` which our Prometheus doesn't see
This PR will make it configurable with a default value of `metrics`
Signed-off-by: Julien Duchesne <julien.duchesne@grafana.com>
* Bump version
Signed-off-by: Julien Duchesne <julien.duchesne@grafana.com>
* Bump minor version
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-cd): Add ability to scrape redis metrics
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Bump chart version
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-cd): Add support for topologySpreadConstraints
Signed-off-by: Sander van Schie <sandervanschie@gmail.com>
* Apply suggestions from code review
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-cd): add extraContainers to pods besides server
Signed-off-by: mikutas <23391543+mikutas@users.noreply.github.com>
* Apply suggestions from code review
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix(argo-rollouts): Use 'controller.component' parameter consistently
Signed-off-by: Donovan Muller <donovan.muller@gmail.com>
* Fix second occurrence of controller.component
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* reenable static assets for argo-cd server
Signed-off-by: Geisler, James D <geislerjamesd@gmail.com>
* add fix to changelog
Signed-off-by: Geisler, James D <geislerjamesd@gmail.com>
* feat(argo-cd): Upgrade argo-cd image to 2.1.1
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Bump chart version
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Bump fix version
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
Co-authored-by: Geisler, James D <geislerjamesd@gmail.com>
* Allow service monitor relabeling configs
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Revert scrape interval to optional field
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Revert path to original place
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Fix trailing character on README
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add probes for Dex server >= 2.28.0
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Enable metrics port and allow user to configure only retries
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Update README
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* Place probe configuration on top-level
Signed-off-by: Petr Drastil <petr.drastil@gmail.com>
* refactor: Parameterizing dex ports names due to istio 403 errors
Signed-off-by: Piotr Kowalczyk <kowalczyk.p.4@pg.com>
* fix: updating default http port name to http
Signed-off-by: Piotr Kowalczyk <kowalczyk.p.4@pg.com>
* Bump appVersion to 3.1.0
Signed-off-by: Matt Carr <mcarr@kinaxis.com>
* Update charts/argo-workflows/Chart.yaml
Co-authored-by: Oliver Bähler <oliverbaehler@hotmail.com>
Signed-off-by: Matt Carr <mcarr@kinaxis.com>
* Fix conflicts with upstream
Signed-off-by: Matt Carr <mcarr@kinaxis.com>
* Update changelog annotations and bump minor version
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Oliver Bähler <oliverbaehler@hotmail.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-rollouts): add dashboard deployment and service
credit: the implemention is inspired by the work from
Andrii Perenesenko <andrii.perenesenko@gmail.com>.
Signed-off-by: Hui Kang <hui.kang@salesforce.com>
* Apply suggestions from code review
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add components to selectlabels
Signed-off-by: Hui Kang <hui.kang@salesforce.com>
* Bump major version and add upgrading note
Signed-off-by: Hui Kang <hui.kang@salesforce.com>
* fix: Documentation on dashboard values, change annotations
Signed-off-by: Hui Kang <hui.kang@salesforce.com>
* feat: Enable dashboard during chart testing
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Hui Kang <hui.kang@salesforce.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-cd): Add support for aggregation clusterroles
* feat: Add support for aggregation clusterroles
Signed-off-by: Christian Strack <github@abyth.de>
* fix(argo-cd): Remove redundant whitespace from README.md
Signed-off-by: Christian Strack <github@abyth.de>
Co-authored-by: Marko Bevc <marko.bevc@gmail.com>
* fix(argo-cd): Remove component labels from aggregation roles
Signed-off-by: Christian Strack <github@abyth.de>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix(argo-cd): Use value `createAggregateRoles` to control role creation
* fix: Use value `createAggregateRoles` to control role creation
* bump version to 3.12.0
* update chart README.md accordingly
Signed-off-by: Christian Strack <github@abyth.de>
Co-authored-by: Marko Bevc <marko.bevc@gmail.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat: Add support for extra containers in server & controller deployment
Signed-off-by: Mike Bryant <mikebryant@bulb.co.uk>
* Update charts/argo-workflows/templates/controller/workflow-controller-deployment.yaml
Co-authored-by: Oliver Bähler <oliverbaehler@hotmail.com>
Signed-off-by: Mike Bryant <mike@mikebryant.me.uk>
* Update charts/argo-workflows/templates/server/server-deployment.yaml
Co-authored-by: Oliver Bähler <oliverbaehler@hotmail.com>
Signed-off-by: Mike Bryant <mike@mikebryant.me.uk>
Co-authored-by: Oliver Bähler <oliverbaehler@hotmail.com>
* feat(argo-cd):Add sessionAffinity to server
Signed-off-by: Rafael Tovar <rafael.tovar@taotesting.com>
* feat(argo-cd): update changelog new chart
Signed-off-by: Rafael Tovar <rafael.tovar@taotesting.com>
