Fix rabbitmq chart duplicated probes key

This change updates our openstack-helm-infra patches to rabbitmq chart
so that no duplicate keys are placed in a yaml file.

Initial stx-openstack tests on Debian showed that the most updated
platform version fails to upload an app that has duplicated keys, during
the sysinv map construction step. So this fix is required to update
stx-openstack with the best yaml syntax practices.

Test Plan:
PASS - Build openstack-helm-infra package
PASS - Build stx-openstack helm charts app
PASS - Upload stx-openstack on Debian

Closes-Bug: 1997564

Signed-off-by: Thales Elero Cervi <thaleselero.cervi@windriver.com>
Change-Id: Ib7f63fee2868b2ccea3ab95a8c83edba96af1c7e
This commit is contained in:
Thales Elero Cervi
2022-11-23 11:54:39 -03:00
parent 009080586c
commit ca2a1d0fae
4 changed files with 36 additions and 32 deletions

View File

@@ -8,13 +8,15 @@ liveness and readiness probes.
Change-Id: I8d2685118eb4ce3b8c27952892f7ad553fc5de77
Signed-off-by: Gerry Kopec <Gerry.Kopec@windriver.com>
[ fix duplicate yaml keys ]
Signed-off-by: Thales Elero Cervi <thaleselero.cervi@windriver.com>
---
rabbitmq/templates/statefulset.yaml | 10 ++++++----
rabbitmq/values.yaml | 9 +++++++++
2 files changed, 15 insertions(+), 4 deletions(-)
rabbitmq/values.yaml | 8 ++++++++
2 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/rabbitmq/templates/statefulset.yaml b/rabbitmq/templates/statefulset.yaml
index e0e61e6..f71dc3e 100644
index 45f8271a..cd9f39d6 100644
--- a/rabbitmq/templates/statefulset.yaml
+++ b/rabbitmq/templates/statefulset.yaml
@@ -225,14 +225,16 @@ spec:
@@ -39,14 +41,13 @@ index e0e61e6..f71dc3e 100644
command:
- /tmp/rabbitmq-liveness.sh
diff --git a/rabbitmq/values.yaml b/rabbitmq/values.yaml
index 9cd3a91..2bae4cb 100644
index 6a1bd85f..0d37e673 100644
--- a/rabbitmq/values.yaml
+++ b/rabbitmq/values.yaml
@@ -145,6 +145,15 @@ pod:
limits:
memory: "1024Mi"
cpu: "2000m"
+ probes:
@@ -48,6 +48,14 @@ images:
pod:
probes:
+ readiness:
+ initialDelaySeconds: 10
+ periodSeconds: 10
@@ -55,9 +56,9 @@ index 9cd3a91..2bae4cb 100644
+ initialDelaySeconds: 30
+ periodSeconds: 10
+ timeoutSeconds: 10
conf:
enabled_plugins:
prometheus_rabbitmq_exporter:
rabbitmq_exporter:
readiness:
--
2.7.4
2.25.1

View File

@@ -26,10 +26,11 @@ Ref:
Signed-off-by: Maik Catrinque <maik.wandercatrinqueandrade@windriver.com>
Co-authored-by: Andrew Martins Carletti <Andrew.MartinsCarletti@windriver.com>
Change-Id: I56e966ea64e8881ba436213f0c9e1cbe547098e3
Signed-off-by: Thales Elero Cervi <thaleselero.cervi@windrive.com>
---
rabbitmq/templates/bin/_rabbitmq-start.sh.tpl | 3 +++
rabbitmq/values.yaml | 6 ++++++
4 files changed, 11 insertions(+), 1 deletion(-)
2 files changed, 9 insertions(+)
diff --git a/rabbitmq/templates/bin/_rabbitmq-start.sh.tpl b/rabbitmq/templates/bin/_rabbitmq-start.sh.tpl
index 794f0919..4ef849fd 100644
@@ -44,7 +45,7 @@ index 794f0919..4ef849fd 100644
+{{- end}}
exec rabbitmq-server
diff --git a/rabbitmq/values.yaml b/rabbitmq/values.yaml
index c593966f..8eb51c64 100644
index 90ed6d21..65a9526e 100644
--- a/rabbitmq/values.yaml
+++ b/rabbitmq/values.yaml
@@ -46,6 +46,12 @@ images:
@@ -59,7 +60,7 @@ index c593966f..8eb51c64 100644
+
pod:
probes:
prometheus_rabbitmq_exporter:
readiness:
--
2.25.1

View File

@@ -8,13 +8,15 @@ liveness and readiness probes.
Change-Id: I8d2685118eb4ce3b8c27952892f7ad553fc5de77
Signed-off-by: Gerry Kopec <Gerry.Kopec@windriver.com>
[ fix duplicate yaml keys ]
Signed-off-by: Thales Elero Cervi <thaleselero.cervi@windriver.com>
---
rabbitmq/templates/statefulset.yaml | 10 ++++++----
rabbitmq/values.yaml | 9 +++++++++
2 files changed, 15 insertions(+), 4 deletions(-)
rabbitmq/values.yaml | 8 ++++++++
2 files changed, 14 insertions(+), 4 deletions(-)
diff --git a/rabbitmq/templates/statefulset.yaml b/rabbitmq/templates/statefulset.yaml
index e0e61e6..f71dc3e 100644
index 45f8271a..cd9f39d6 100644
--- a/rabbitmq/templates/statefulset.yaml
+++ b/rabbitmq/templates/statefulset.yaml
@@ -225,14 +225,16 @@ spec:
@@ -39,14 +41,13 @@ index e0e61e6..f71dc3e 100644
command:
- /tmp/rabbitmq-liveness.sh
diff --git a/rabbitmq/values.yaml b/rabbitmq/values.yaml
index 9cd3a91..2bae4cb 100644
index 6a1bd85f..0d37e673 100644
--- a/rabbitmq/values.yaml
+++ b/rabbitmq/values.yaml
@@ -145,6 +145,15 @@ pod:
limits:
memory: "1024Mi"
cpu: "2000m"
+ probes:
@@ -48,6 +48,14 @@ images:
pod:
probes:
+ readiness:
+ initialDelaySeconds: 10
+ periodSeconds: 10
@@ -55,9 +56,9 @@ index 9cd3a91..2bae4cb 100644
+ initialDelaySeconds: 30
+ periodSeconds: 10
+ timeoutSeconds: 10
conf:
enabled_plugins:
prometheus_rabbitmq_exporter:
rabbitmq_exporter:
readiness:
--
2.7.4
2.25.1

View File

@@ -26,10 +26,11 @@ Ref:
Signed-off-by: Maik Catrinque <maik.wandercatrinqueandrade@windriver.com>
Co-authored-by: Andrew Martins Carletti <Andrew.MartinsCarletti@windriver.com>
Change-Id: I56e966ea64e8881ba436213f0c9e1cbe547098e3
Signed-off-by: Thales Elero Cervi <thaleselero.cervi@windrive.com>
---
rabbitmq/templates/bin/_rabbitmq-start.sh.tpl | 3 +++
rabbitmq/values.yaml | 6 ++++++
4 files changed, 11 insertions(+), 1 deletion(-)
2 files changed, 9 insertions(+)
diff --git a/rabbitmq/templates/bin/_rabbitmq-start.sh.tpl b/rabbitmq/templates/bin/_rabbitmq-start.sh.tpl
index 794f0919..4ef849fd 100644
@@ -44,7 +45,7 @@ index 794f0919..4ef849fd 100644
+{{- end}}
exec rabbitmq-server
diff --git a/rabbitmq/values.yaml b/rabbitmq/values.yaml
index c593966f..8eb51c64 100644
index 90ed6d21..65a9526e 100644
--- a/rabbitmq/values.yaml
+++ b/rabbitmq/values.yaml
@@ -46,6 +46,12 @@ images:
@@ -59,7 +60,7 @@ index c593966f..8eb51c64 100644
+
pod:
probes:
prometheus_rabbitmq_exporter:
readiness:
--
2.25.1