From 61647074d04cdcccfbe86f08fe0a4cca5f169086 Mon Sep 17 00:00:00 2001 From: Kenan Erdogan Date: Wed, 20 Oct 2021 09:02:20 +0200 Subject: [PATCH 1/2] add missing labels key into roles.yaml Signed-off-by: Kenan Erdogan --- charts/opensearch/CHANGELOG.md | 35 ++++++++++++++++++++++++++- charts/opensearch/Chart.yaml | 2 +- charts/opensearch/templates/role.yaml | 1 + 3 files changed, 36 insertions(+), 2 deletions(-) diff --git a/charts/opensearch/CHANGELOG.md b/charts/opensearch/CHANGELOG.md index 324755cf..bb0e7510 100644 --- a/charts/opensearch/CHANGELOG.md +++ b/charts/opensearch/CHANGELOG.md @@ -13,6 +13,36 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Fixed ### Security +--- +## [1.2.1] +### Added +### Changed +### Deprecated +### Removed +### Fixed +- Missing `labels` key is added into role.yaml. +### Security + +--- +## [1.2.0] +### Added +### Changed +- Change labels of opensearch cart to match standard recommendations. +### Deprecated +### Removed +### Fixed +### Security + +--- +## [1.1.0] +### Added +- Enable using externally created secret for securityConfig. +### Changed +### Deprecated +### Removed +### Fixed +### Security + --- ## [1.0.8] ### Added @@ -79,7 +109,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Fixed ### Security -[Unreleased]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.8...HEAD +[Unreleased]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.2.1...HEAD +[1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.2.0...opensearch-1.2.1 +[1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.1.0...opensearch-1.2.0 +[1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.8...opensearch-1.1.0 [1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.7...opensearch-1.0.8 [1.0.7]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.6...opensearch-1.0.7 [1.0.6]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.5...opensearch-1.0.6 diff --git a/charts/opensearch/Chart.yaml b/charts/opensearch/Chart.yaml index 6e28d0ab..1579e58c 100644 --- a/charts/opensearch/Chart.yaml +++ b/charts/opensearch/Chart.yaml @@ -15,7 +15,7 @@ type: application # This is the chart version. This version number should be incremented each time you make changes # to the chart and its templates, including the app version. # Versions are expected to follow Semantic Versioning (https://semver.org/) -version: 1.2.0 +version: 1.2.1 # This is the version number of the application being deployed. This version number should be # incremented each time you make changes to the application. Versions are not expected to diff --git a/charts/opensearch/templates/role.yaml b/charts/opensearch/templates/role.yaml index 7f7890ea..cba3cf99 100644 --- a/charts/opensearch/templates/role.yaml +++ b/charts/opensearch/templates/role.yaml @@ -4,6 +4,7 @@ apiVersion: rbac.authorization.k8s.io/v1 kind: Role metadata: name: {{ $fullName | quote }} + labels: {{- include "opensearch.labels" . | nindent 4 }} rules: - apiGroups: From a796c453d07f251c13c69e8ac5f4216a00870cd6 Mon Sep 17 00:00:00 2001 From: Kenan Erdogan Date: Wed, 20 Oct 2021 10:17:45 +0200 Subject: [PATCH 2/2] Apply suggestions from code review Co-authored-by: Oliver Hartl --- charts/opensearch/CHANGELOG.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/opensearch/CHANGELOG.md b/charts/opensearch/CHANGELOG.md index bb0e7510..51946cda 100644 --- a/charts/opensearch/CHANGELOG.md +++ b/charts/opensearch/CHANGELOG.md @@ -110,9 +110,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ### Security [Unreleased]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.2.1...HEAD -[1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.2.0...opensearch-1.2.1 -[1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.1.0...opensearch-1.2.0 -[1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.8...opensearch-1.1.0 +[1.2.1]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.2.0...opensearch-1.2.1 +[1.2.0]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.1.0...opensearch-1.2.0 +[1.1.0]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.8...opensearch-1.1.0 [1.0.8]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.7...opensearch-1.0.8 [1.0.7]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.6...opensearch-1.0.7 [1.0.6]: https://github.com/opensearch-project/helm-charts/compare/opensearch-1.0.5...opensearch-1.0.6