Merge remote-tracking branch 'upstream/master' into argocd-helm-v1.0.0

.github/pull_request_template.md

@@ -3,3 +3,4 @@ Checklist:
 * [ ] I have update the chart version in `Chart.yaml` following Semantic Versioning.
 * [ ] Any new values are backwards compatible and/or have sensible default.
 * [ ] I have followed the testing instructions in the [contributing guide](https://github.com/argoproj/argo-helm/blob/master/CONTRIBUTING.md).
+* [ ] I have signed the CLA.

charts/argo-events/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v1
 description: A Helm chart to install Argo-Events in k8s Cluster
 name: argo-events
-version: 0.4.3
+version: 0.5.0
 keywords:
 - argo-events
 - sensor-controller
@@ -11,5 +11,5 @@ sources:
 maintainers:
 - name: Vaibhav Page
 - name: Matt Magaldi
-appVersion: 0.9.2
-icon: https://raw.githubusercontent.com/argoproj/argo/master/argo.png
+appVersion: 0.10
+icon: https://raw.githubusercontent.com/argoproj/argo/master/argo.png

charts/argo-events/README.md

@@ -1,8 +1,8 @@
 # Argo-Events Chart
 
 This is a **community maintained** chart. It installs the [argo-events](https://github.com/argoproj/argo-events) application. This application comes packaged with:
-- Sensor Custom Resource Definition
-- Gateway Custom Resource Definition
+- Sensor Custom Resource Definition (See CRD Notes)
+- Gateway Custom Resource Definition (See CRD Notes)
 - Sensor Controller Deployment
 - Sensor Controller ConfigMap
 - Gateway Controller Deployment
@@ -10,3 +10,7 @@ This is a **community maintained** chart. It installs the [argo-events](https://
 - Service Account
 - Cluster Roles
 - Cluster Role Bindings
+
+## Notes on CRD Installation
+
+Some users would prefer to install the CRDs _outside_ of the chart. You can disable the CRD installation of this chart by using `--set installCRD=false` when installing the chart.

charts/argo-events/templates/gateway-crd.yaml

@@ -1,3 +1,4 @@
+{{- if .Values.installCRD }}
 # Define a "gateway" custom resource definition
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
@@ -12,3 +13,4 @@ spec:
     singular: gateway
   scope: Namespaced
   version: v1alpha1
+{{- end }}

charts/argo-events/templates/sensor-crd.yaml

@@ -1,3 +1,4 @@
+{{- if .Values.installCRD }}
 # Define a "sensor" custom resource definition
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
@@ -12,3 +13,4 @@ spec:
     singular: sensor
   scope: Namespaced
   version: v1alpha1
+{{- end }}

charts/argo-events/values.yaml

@@ -4,6 +4,9 @@ registry: argoproj
 # The image pull policy
 imagePullPolicy: Always
 
+# If set to false, skip installing the CRDs. Requires user to have them installed prior to helm chart installation.
+installCRD: true
+
 # ServiceAccount to use for running controller.
 serviceAccount: argo-events-sa
 # Create service accounts in additional namespaces specified
@@ -13,7 +16,7 @@ additionalSaNamespaces: []
 
 instanceID: argo-events
 
-# set `singleNamespace` to false to have the controllers 
+# set `singleNamespace` to false to have the controllers
 # listen on all namespaces.  Otherwise the controllers will listen
 # on the namespace provided
 namespace: argo-events
@@ -23,11 +26,11 @@ singleNamespace: true
 sensorController:
   name: sensor-controller
   image: sensor-controller
-  tag: v0.9.2
+  tag: v0.10
   replicaCount: 1
 
 gatewayController:
   name: gateway-controller
   image: gateway-controller
-  tag: v0.9.2
+  tag: v0.10
   replicaCount: 1

charts/argo/Chart.yaml

@@ -1,6 +1,6 @@
 apiVersion: v1
-appVersion: "v2.3.0"
+appVersion: "v2.4.2"
 description: A Helm chart for Argo Workflows
 name: argo
-version: 0.5.4
+version: 0.6.1
 icon: https://raw.githubusercontent.com/argoproj/argo/master/argo.png

charts/argo/README.md

@@ -8,7 +8,7 @@ If you want your deployment of this helm chart to most closely match the [argo C
 This chart uses an install hook to configure the CRD definition.  Installation of CRDs is a somewhat privileged process in itself and in RBAC enabled clusters the `default` service account for namespaces does not typically have the ability to do create these.
 
 A few options are:
-- Setup the CRD yourself manually and use the `--no-hooks` options of `helm install`
+- Setup the CRD yourself manually and use `--set installCRD=false` when installing the helm chart. Find the CRDs in the [argo codebase](https://raw.githubusercontent.com/argoproj/argo/master/manifests/base/crds/workflow-crd.yaml)
 - Manually create a ServiceAccount in the Namespace which your release will be deployed w/ appropriate bindings to perform this action and set the `init.serviceAccount` attribute
 - Augment the `default` ServiceAccount permissions in the Namespace in which your Release is deployed to have the appropriate permissions
 

charts/argo/templates/ui-cluster-role.yaml

@@ -1,4 +1,4 @@
-{{- if .Values.ui.enabled -}}
+{{- if .Values.ui.enabled }}
 apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
 metadata:
@@ -24,8 +24,9 @@ rules:
   - argoproj.io
   resources:
   - workflows
+  - workflowtemplates
   verbs:
   - get
   - list
   - watch
-  {{- end -}}
+{{- end }}

charts/argo/templates/workflow-aggregate-roles.yaml

@@ -14,11 +14,12 @@ rules:
   resources:
   - workflows
   - workflows/finalizers
+  - workflowtemplates
+  - workflowtemplates/finalizers
   verbs:
   - get
   - list
   - watch
-
 ---
 apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
@@ -35,6 +36,8 @@ rules:
   resources:
   - workflows
   - workflows/finalizers
+  - workflowtemplates
+  - workflowtemplates/finalizers
   verbs:
   - create
   - delete
@@ -44,7 +47,6 @@ rules:
   - patch
   - update
   - watch
-
 ---
 apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
@@ -61,6 +63,8 @@ rules:
   resources:
   - workflows
   - workflows/finalizers
+  - workflowtemplates
+  - workflowtemplates/finalizers
   verbs:
   - create
   - delete

charts/argo/templates/workflow-controller-clusterrole.yaml

@@ -42,4 +42,21 @@ rules:
   - watch
   - update
   - patch
-  - delete
+  - delete
+- apiGroups:
+  - argoproj.io
+  resources:
+  - workflowtemplates
+  - workflowtemplates/finalizers
+  verbs:
+  - get
+  - list
+  - watch
+- apiGroups:
+  - ""
+  resources:
+  - serviceaccounts
+  verbs:
+  - get
+  - list
+

charts/argo/templates/workflow-controller-config-map.yaml

@@ -43,3 +43,6 @@ data:
     {{- if .Values.controller.telemetryConfig.enabled }}
     telemetryConfig:
 {{ toYaml .Values.controller.telemetryConfig | indent 6}}{{- end }}
+    {{- if .Values.controller.persistence }}
+    persistence:
+{{ toYaml .Values.controller.persistence | indent 6 }}{{- end }}

charts/argo/templates/workflow-crd.yaml

@@ -1,3 +1,4 @@
+{{- if .installCRD }}
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
@@ -14,3 +15,4 @@ spec:
     plural: workflows
     shortNames:
     - wf
+{{- end }}

charts/argo/templates/workflow-template-crd.yaml

@@ -0,0 +1,18 @@
+{{- if .installCRD }}
+apiVersion: apiextensions.k8s.io/v1beta1
+kind: CustomResourceDefinition
+metadata:
+  name: workflowtemplates.argoproj.io
+  annotations:
+    helm.sh/hook: crd-install
+    helm.sh/hook-delete-policy: before-hook-creation
+spec:
+  group: argoproj.io
+  version: v1alpha1
+  scope: Namespaced
+  names:
+    kind: WorkflowTemplate
+    plural: workflowtemplates
+    shortNames:
+    - wftmpl
+{{- end }}

charts/argo/values.yaml

@@ -4,9 +4,10 @@ images:
   ui: argoui
   executor: argoexec
   pullPolicy: Always
-  tag: v2.3.0
+  tag: v2.4.2
 
 crdVersion: v1alpha1
+installCRD: true
 
 init:
   # By default the installation will not set an explicit one, which will mean it uses `default` for the namespace the chart is
@@ -28,6 +29,24 @@ controller:
     enabled: false
     path: /metrics
     port: 8080
+    persistence: {}
+    #  connectionPool:
+    #    maxIdleConns: 100
+    #    maxOpenConns: 0
+    #  # save the entire workflow into etcd and DB
+    #  nodeStatusOffLoad: false
+    #  postgresql:
+    #    host: localhost
+    #    port: 5432
+    #    database: postgres
+    #    tableName: argo_workflows
+    #    # the database secrets must be in the same namespace of the controller
+    #    userNameSecret:
+    #      name: argo-postgres-config
+    #      key: username
+    #    passwordSecret:
+    #      name: argo-postgres-config
+    #      key: password
   telemetryConfig:
     enabled: false
     path: /telemetry

scripts/publish.sh

@@ -1,16 +1,17 @@
 #!/bin/bash
+set -eux
 
 SRCROOT="$(cd "$(dirname "$0")/.." && pwd)"
 GIT_PUSH=${GIT_PUSH:-true}
 
 rm -rf $SRCROOT/output && git clone -b gh-pages git@github.com:argoproj/argo-helm.git $SRCROOT/output
 
-cd $SRCROOT/charts
-for dir in *;
+for dir in $(find $SRCROOT/charts -mindepth 1 -maxdepth 1 -type d);
 do
  echo "Processing $dir"
  helm package $dir
 done
+cp $SRCROOT/*.tgz output/
 cd $SRCROOT/output && helm repo index .
 
 cd $SRCROOT/output && git status