* Toggle for provider-specific RBAC + Added missing RBAC rules
Signed-off-by: Richard Johansson <richard.jimmy.johansson@gmail.com>
* Updated docs with new Helm values
Signed-off-by: Richard Johansson <richard.jimmy.johansson@gmail.com>
* Added a general flag providerRBAC.enabled to toggle all of the providers
Signed-off-by: Richard Johansson <richard.jimmy.johansson@gmail.com>
* Aligned with main
Signed-off-by: Richard Johansson <richard.jimmy.johansson@gmail.com>
* Corrected inline comments
Signed-off-by: Richard Johansson <richard.jimmy.johansson@gmail.com>
* Fixed incorrect inline comments
Signed-off-by: Richard Johansson <richard.jimmy.johansson@gmail.com>
Signed-off-by: Richard Johansson <richard.jimmy.johansson@gmail.com>
* fix(argo-rollouts): generated value for app.kubernetes.io/version label for argo-rollouts will not exceed maximum length and support passing a SHA digest.
Signed-off-by: Pedro Salgado <steenzout@users.noreply.github.com>
* fix: Add missing double quote in changelog
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Signed-off-by: Pedro Salgado <steenzout@users.noreply.github.com>
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Standardize helm chart maintainer to be
```yaml
maintainers:
- name: argoproj
url: https://argoproj.github.io/
```
We'll use CODEOWNERS to notify individual maintainers about PRs to the helm charts and will be our source-of-truth for that information
Signed-off-by: jmeridth <jmeridth@gmail.com>
* feat(argo-rollouts): Add flag to optionally disable the creation of cluster roles (even when running in cluster mode)
Signed-off-by: Joshua Helton <jdoghelton@gmail.com>
* fix README.md with helm-docs.sh
Signed-off-by: Joshua Helton <jdoghelton@gmail.com>
Signed-off-by: Joshua Helton <jdoghelton@gmail.com>
Co-authored-by: JM <jmeridth@gmail.com>
* feat(argo-rollouts) Add controller.extraEnv
Ability to specify environment variables for rollouts-controller from values.yaml
Signed-off-by: Ben Hayden <hayden767@gmail.com>
* Also implement extraEnv for dashboard component
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* Reword changelog line
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat(argo-rollouts): add dashboard permission on Deployments
- needed for Rollouts with workload referencing
Signed-off-by: Ben Poland <ben.poland@faire.com>
* Add podtemplates to match controller role
Signed-off-by: Ben Poland <ben.poland@faire.com>
* feat(argo-rollouts): upgrade to v1.2.0-rc1
Signed-off-by: Hui Kang <hui.kang@salesforce.com>
* [argo-rollouts]:bump minor version and add release version
Signed-off-by: Hui Kang <hui.kang@salesforce.com>
* fixing the error on the argo rollouts controller needing the notifications configmap and rollout
Signed-off-by: Ian Muge <ian.muge@ardoq.com>
* bump chart version
Signed-off-by: Ian Muge <ian.muge@ardoq.com>
* updating changelog
Signed-off-by: Ian Muge <ian.muge@ardoq.com>
* fix structure to be simialr to argocd-notification as suggested
Signed-off-by: Ian Muge <ian.muge@ardoq.com>
* Bump Minor version as we are adding a new feature
Signed-off-by: Ian Muge <ian.muge@ardoq.com>
* Updated helm docs, all tests are green, locally
Signed-off-by: Ian Muge <ian.muge@ardoq.com>
* Apply changes 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-rollouts): Add ability to define dashboard service node port
Signed-off-by: Jean Mainguy <9969006+jhandguy@users.noreply.github.com>
* Apply changes 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-rollouts): +LB service.type for dashboard svc
Signed-off-by: Maciej Lasyk <maciej@lasyk.info>
* Apply changes from code review
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* feat: Allow setting the type for the dashboard service. (We have a requirement
to run it as NodePort). Default to what it is if not set.
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Bump Chart version.
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Add serviceType into values.yaml
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Address review comments:
Remove old changes annotation.
Rename serviceType to service.type.
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Bump Chart version.
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Fix typo
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* FIx indentation (lint error)
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Update README
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Update charts/argo-rollouts/README.md
Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
Signed-off-by: Dan Tulovsky <dant@lightstep.com>
* Apply suggestions from code review
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* fix changelog
Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>
* docs: Add warning about exposing the Dashboard via Service type
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>
* 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>
* 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>
* 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>
Aikawa
Cyril Jouve
Tal Yitzhak
aniekgul
Marco Kilchhofer
Khanh Ngo
Jason Meridth
Richard Johansson
Pedro
Kiyoshi Muranaka
Luke
jphelton
Vlad Voloshyn
Ben Hayden
foxtel-temujincabigao
Ward Loos
amirschw
Ben Poland
cskh
Ian Muge
Jean Mainguy
Oliver Bähler
Sergey Shaykhullin
Lucas Bickel
Maciej Lasyk
Tung Huynh
MrWetsnow
Donovan Muller
Brian Johnson
Sebastian J
Amrit Gill
Artem Yakimenko
siddharth
cskh
Aditya Menon
Pedro Arvela