* feat(argo-workflows): Moved `.Values.images` values to global
Signed-off-by: yu-croco <yuki.kita22@gmail.com>
* Revert "feat(argo-workflows): Moved `.Values.images` values to global"
This reverts commit 67b4294675.
Signed-off-by: yu-croco <yuki.kita22@gmail.com>
* feat(argo-workflows): Add global tag
Signed-off-by: yu-croco <yuki.kita22@gmail.com>
* feat(argo-workflows): Assign common labels to controller resources
Signed-off-by: kenchan0130 <tt.tanishi100@gmail.com>
* feat(argo-workflows): Assign common labels to the server sa resource
Signed-off-by: kenchan0130 <tt.tanishi100@gmail.com>
* Update argo-workflows Chart.yaml
Signed-off-by: kenchan0130 <tt.tanishi100@gmail.com>
* fix(argo-workflows): add missing rules for workflow role to run HTTP/plugin templates
Signed-off-by: yu-croco <yuki.kita22@gmail.com>
* chore(argo-workflows): refactor role rules simpler
Signed-off-by: yu-croco <yuki.kita22@gmail.com>
* feat(argo-workflows): add support for Deployment annotations
bump version and add changelog
update documentation
Signed-off-by: snooyen <sean@bright.ai>
* 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>
* Added short tag function and updated templates
Signed-off-by: John Clark <clark.john.d@outlook.com>
* Chart.yaml update (version, changes)
Signed-off-by: John Clark <clark.john.d@outlook.com>
* Removed unnecessary conditional logic
Signed-off-by: John Clark <clark.john.d@outlook.com>
* Final newline to _helpers
Signed-off-by: John Clark <clark.john.d@outlook.com>
* Rework without template function
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Fix typos and add missing words in README.md
No real content changes, just textual improvements.
* Bump version and add changelog
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Sync changes README.md -> README.md.gotmpl
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* argo-workflows: update to Argo Workflows version 3.3.1
* Add WorkflowTaskResult CRD
* Add associated permissions to Workflow Controller Role
Signed-off-by: Ira W. Snyder <ira@descarteslabs.com>
* Address review comments
Signed-off-by: Ira W. Snyder <ira@descarteslabs.com>
* Update to Argo Workflows v3.3.2
A new release is out.
Signed-off-by: Ira W. Snyder <ira@descarteslabs.com>
* Add list
Signed-off-by: Frank Bagherzadeh <frankbzd@gmail.com>
* bump version and add change log
Signed-off-by: Frank Bagherzadeh <frankbzd@gmail.com>
* fix changelog
Signed-off-by: Frank Bagherzadeh <frankbzd@gmail.com>
* optional role
Signed-off-by: Carl Raiden Worley <carl.worley@10gen.com>
* match existing setting
Signed-off-by: Carl Raiden Worley <carl.worley@10gen.com>
* fix lint
Signed-off-by: Carl Raiden Worley <carl.worley@10gen.com>
* feat(argo-workflows): Add controller volumes
Signed-off-by: John Clark <clark.john.d@outlook.com>
* Chart.yaml mispelling (paramaters > parameters)
Signed-off-by: John Clark <clark.john.d@outlook.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-workflows): argo workflows chart to allow set the namespace of Prometheus ServiceMonitor.
Signed-off-by: mugioka <okamugi0722@gmail.com>
* Bump minor chart version
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Add missing WorkflowTasksets RBAC to controller
Signed-off-by: Paul Ayling <paulayling.dev@gmail.com>
* Add WorkflowTaskSet RBAC to admin role
Signed-off-by: Paul Ayling <paulayling.dev@gmail.com>
* 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>
* 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>
* 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>
* 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>
* 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>