* 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>
* 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: 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>
* allow workflow to be run in multiple namespaces, bump chart version
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* update helm install instructions
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* fix indents
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* add newlines
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* bump chart version
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* default to "argo" namespace for workflow service account
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* simplify setting service account, fix syntactical error in defaulting to "argo" namespace
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* Bump only the chart version, revert bumping the controller version: 3.0.8 does not exist yet as a workflow-controller image version
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* don't set a namespace if there isn't one provided
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* bump version
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* re-use controller.workflowNamespaces instead of introducing workflow.namespaces
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* re-use controller.workflowNamespaces instead of introducing workflow.namespaces
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* re-use controller.workflowNamespaces instead of introducing workflow.namespaces
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* re-use controller.workflowNamespaces instead of introducing workflow.namespaces, add additional inline comments on rbac.create effect
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* compress namespaces from controller.workflowNamespaces and workflow.namespace into single loop iteration
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* compress namespaces from controller.workflowNamespaces and workflow.namespace into single loop iteration
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* compress namespaces from controller.workflowNamespaces and workflow.namespace into single loop iteration
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* dro now-redundant branch for controller.workflowNamespaces
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* use template helper "argo-workflows.fullname" instead of release name in order to be able to override the service account name
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* add missing "template" keyword
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* drop redundant $namespace assignment that was already moved inside the "range" function
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* replace "define" templates with simple values now that controller.workflowNamespaces and workflow.namespace are unified into a single "range" iteration
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* drop redundant $namespace assignment that was already moved inside the "range" function
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* inline "define" templates with simple values now that controller.workflowNamespaces and workflow.namespace are unified into a single "range" iteration
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* inline "define" templates with simple values now that controller.workflowNamespaces and workflow.namespace are unified into a single "range" iteration
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* add missing "$" to denote target for templating the workflows fullname helper
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* restore `with` blocks for service account name and service account annotations
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* drop validations: considering cases of `workflowNamespaces`, `namespace` and `singleNamespace`
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* include case for single namespaces with default ("") namespace, otherwise (ternary) combine the workflow namespace and workflowNamespaces from the controller settings into a list to iterate over
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* wrap namespace configuration in `with` block to prevent setting it to ""
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* fix append vs. concat bug: append workflowNamespace to (possibly 0-lenght) list of controller.workflowNamespaces
Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* fix namespace range with additional check for either single namespace being false, or append to the workflow namespaces: the first non-null (coalesce) from the workflow namespace or the namespace Argo is being deployed to.
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* add changelog entries
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
* unwrap (inline) with-block for service account name
Signed-off-by: reinvantveer <reinvantveer@gmail.com>
Co-authored-by: Vlad Losev <vladlosev@users.noreply.github.com>
* fix(argo-workflows): Observe 'controller.serviceAccount.create'
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* chore: Ignore ci/ directory in final package
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* chore(argo-workflows): Add missing value
Signed-off-by: yu-croco <yuki.kita22@gmail.com>
* reflect review: make it shorter and clearer.
Signed-off-by: yu-croco <yuki.kita22@gmail.com>
* Apply suggestions from code review
Signed-off-by: Marco Kilchhofer <mkilchhofer@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>
* Update example for extraPaths for usage with k8s>=1.19 with "networking.k8s.io/v1"
Signed-off-by: sturman <4456572+sturman@users.noreply.github.com>
* Fix typos in values.yaml
Signed-off-by: sturman <4456572+sturman@users.noreply.github.com>
* Bump charts version
Signed-off-by: sturman <4456572+sturman@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>
Oliver Bähler
66li
Rein van 't Veer
Paul Ayling
Michiel De Mey
Clément
Tuan Anh Tran
Vlad Losev
Louis Haußknecht
Sanjay Tiwari
Marco Kilchhofer
Peter O
Julien Duchesne
bderdiarian
Elliot Maincourt
Karsten Siemer
Peter Smit
Rein van 't Veer
Matt Carr
Mike Bryant
Thom
Niels ten Boom
kostas-theo
Zadkiel
Rein van 't Veer
Jakub Bielawski
Alex Sears
Sevan
Aikawa
Roman
Tadayuki Onishi
g-linville
Abhinav Khanna
chgl