From d6da30801344ce88bfe393537aa491f3f1ea8398 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20de=20Saint=20Martin?= Date: Fri, 9 Aug 2019 11:41:30 +0200 Subject: [PATCH] [stable/elasticsearch-curator] Update to 5.7.6 by changing docker repository (#15669) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * [stable/elasticsearch-curator] Update to 5.7.6 by changing docker repository. Also add desaintmartin to maintainers. Signed-off-by: Cédric de Saint Martin * Update stable/elasticsearch-curator/README.md Co-Authored-By: Carlos Rodríguez Hernández Signed-off-by: Cédric de Saint Martin * [stable/elasticsearch-curator] Remove @tmestdagh from OWNERS. Signed-off-by: Cédric de Saint Martin --- stable/elasticsearch-curator/Chart.yaml | 6 +++--- stable/elasticsearch-curator/OWNERS | 6 +++--- stable/elasticsearch-curator/README.md | 23 +++++++++++++++++------ stable/elasticsearch-curator/values.yaml | 6 +++--- 4 files changed, 26 insertions(+), 15 deletions(-) diff --git a/stable/elasticsearch-curator/Chart.yaml b/stable/elasticsearch-curator/Chart.yaml index 6e3fdf84ba..e908b0d086 100644 --- a/stable/elasticsearch-curator/Chart.yaml +++ b/stable/elasticsearch-curator/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v1 appVersion: "5.5.4" description: A Helm chart for Elasticsearch Curator name: elasticsearch-curator -version: 1.5.0 +version: 2.0.0 home: https://github.com/elastic/curator keywords: - curator @@ -12,7 +12,7 @@ sources: - https://github.com/kubernetes/charts/elasticsearch-curator - https://github.com/pires/docker-elasticsearch-curator maintainers: - - name: tmestdagh - email: mestdagh.tom@gmail.com + - name: desaintmartin + email: cedric.dsm@gmail.com - name: gianrubio email: gianrubio@gmail.com diff --git a/stable/elasticsearch-curator/OWNERS b/stable/elasticsearch-curator/OWNERS index d8c0ba052c..89df1c0092 100644 --- a/stable/elasticsearch-curator/OWNERS +++ b/stable/elasticsearch-curator/OWNERS @@ -1,6 +1,6 @@ approvers: - - tmestdagh + - desaintmartin - gianrubio reviewers: - - tmestdagh - - gianrubio \ No newline at end of file + - desaintmartin + - gianrubio diff --git a/stable/elasticsearch-curator/README.md b/stable/elasticsearch-curator/README.md index b020309e0c..2057b85dd3 100644 --- a/stable/elasticsearch-curator/README.md +++ b/stable/elasticsearch-curator/README.md @@ -23,6 +23,17 @@ To install the chart, use the following: $ helm install stable/elasticsearch-curator ``` +## Upgrading an existing Release to a new major version + +A major chart version change (like v1.2.3 -> v2.0.0) indicates that there is an +incompatible breaking change needing manual actions. + +### To 2.0.0 + +v2.0.0 uses docker image from `elasticsearch-curator` author, which differs in its way to install curator. + +If you have a hardcoded `command` value, please update it to follow the new `curator` executable path: `/curator/curator` (which is not in PATH). + ## Configuration The following table lists the configurable parameters of the docker-registry chart and @@ -31,8 +42,8 @@ their default values. | Parameter | Description | Default | | :----------------------------------- | :---------------------------------------------------------- | :------------------------------------------- | | `image.pullPolicy` | Container pull policy | `IfNotPresent` | -| `image.repository` | Container image to use | `quay.io/pires/docker-elasticsearch-curator` | -| `image.tag` | Container image tag to deploy | `5.5.4` | +| `image.repository` | Container image to use | `untergeek/curator` | +| `image.tag` | Container image tag to deploy | `5.7.6` | | `hooks` | Whether to run job on selected hooks | `{ "install": false, "upgrade": false }` | | `cronjob.schedule` | Schedule for the CronJob | `0 1 * * *` | | `cronjob.annotations` | Annotations to add to the cronjob | {} | @@ -45,15 +56,15 @@ their default values. | `envFromSecrets` | Environment variables from secrets to the cronjob container | {} | | `envFromSecrets.*.from.secret` | - `secretKeyRef.name` used for environment variable | | | `envFromSecrets.*.from.key` | - `secretKeyRef.key` used for environment variable | | -| `command` | Command to execute | ["curator"] | -| `configMaps.action_file_yml` | Contents of the Curator action_file.yml | See values.yaml | -| `configMaps.config_yml` | Contents of the Curator config.yml (overrides config) | See values.yaml | +| `command` | Command to execute | ["/curator/curator"] | +| `configMaps.action_file_yml` | Contents of the Curator action_file.yml | See values.yaml | +| `configMaps.config_yml` | Contents of the Curator config.yml (overrides config) | See values.yaml | | `resources` | Resource requests and limits | {} | | `priorityClassName` | priorityClassName | `nil` | | `extraVolumeMounts` | Mount extra volume(s), | | | `extraVolumes` | Extra volumes | | | `extraInitContainers` | Init containers to add to the cronjob container | {} | -| `securityContext` | Configure PodSecurityContext | `false` | +| `securityContext` | Configure PodSecurityContext | `false` | | `rbac.enabled` | Enable RBAC resources | `false` | | `psp.create` | Create pod security policy resources | `false` | | `serviceAccount.create` | Create a default serviceaccount for elasticsearch curator | `true` | diff --git a/stable/elasticsearch-curator/values.yaml b/stable/elasticsearch-curator/values.yaml index cc28c65654..460f2a46ed 100644 --- a/stable/elasticsearch-curator/values.yaml +++ b/stable/elasticsearch-curator/values.yaml @@ -30,8 +30,8 @@ psp: create: false image: - repository: quay.io/pires/docker-elasticsearch-curator - tag: 5.5.4 + repository: untergeek/curator + tag: 5.7.6 pullPolicy: IfNotPresent hooks: @@ -41,7 +41,7 @@ hooks: # run curator in dry-run mode dryrun: false -command: ["curator"] +command: ["/curator/curator"] env: {} configMaps: