Revert "Fix apply of stx-openstack when host is locked"

The commit that we are reverting broke the normal lock/unlock
case when stx-openstack is applied. More specifically,
the mariadb pod failed to start when stx-openstack
was applied automatically after unlock.

This reverts commit 754a1d33de.

Change-Id: I0f1e5854d22ed54747d0237153ada3985f29ef96
This commit is contained in:
Mihnea Saracin 2020-08-12 15:38:42 +00:00
parent cc42f7cf54
commit d73c7c494d
22 changed files with 39 additions and 70 deletions

View File

@ -26,8 +26,7 @@ Patch08: 0008-Fix-rabbitmq-could-not-bind-port-to-ipv6-address-iss.patch
Patch09: 0009-Enable-override-of-mariadb-server-probe-parameters.patch Patch09: 0009-Enable-override-of-mariadb-server-probe-parameters.patch
Patch10: 0010-Mariadb-use-utf8_general_ci-collation-as-default.patch Patch10: 0010-Mariadb-use-utf8_general_ci-collation-as-default.patch
Patch11: 0011-Add-mariadb-database-config-override-to-support-ipv6.patch Patch11: 0011-Add-mariadb-database-config-override-to-support-ipv6.patch
Patch12: 0012-Add-tolerations-to-rabbitmq-chart.patch Patch12: 0012-enable-Values.conf.database.config_override-for-mari.patch
Patch13: 0013-enable-Values.conf.database.config_override-for-mari.patch
BuildRequires: helm BuildRequires: helm
BuildRequires: chartmuseum BuildRequires: chartmuseum
@ -49,7 +48,6 @@ Openstack Helm Infra charts
%patch10 -p1 %patch10 -p1
%patch11 -p1 %patch11 -p1
%patch12 -p1 %patch12 -p1
%patch13 -p1
%build %build
# Host a server for the charts # Host a server for the charts

View File

@ -1,26 +0,0 @@
From c8f3a96fad3344cfdb058c7c0fee77431f77a001 Mon Sep 17 00:00:00 2001
From: Mihnea Saracin <Mihnea.Saracin@windriver.com>
Date: Wed, 22 Jul 2020 15:53:25 +0300
Subject: [PATCH] Add tolerations to rabbitmq chart
Signed-off-by: Mihnea Saracin <Mihnea.Saracin@windriver.com>
---
rabbitmq/templates/statefulset.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/rabbitmq/templates/statefulset.yaml b/rabbitmq/templates/statefulset.yaml
index 9e40a103..cf5d1811 100644
--- a/rabbitmq/templates/statefulset.yaml
+++ b/rabbitmq/templates/statefulset.yaml
@@ -85,6 +85,8 @@ spec:
{{ tuple $envAll "rabbitmq" "server" | include "helm-toolkit.snippets.kubernetes_pod_anti_affinity" | indent 8 }}
nodeSelector:
{{ $envAll.Values.labels.server.node_selector_key }}: {{ $envAll.Values.labels.server.node_selector_value | quote }}
+ tolerations:
+{{ toYaml .Values.tolerations | indent 8 }}
initContainers:
{{ tuple $envAll "rabbitmq" list | include "helm-toolkit.snippets.kubernetes_entrypoint_init_container" | indent 8 }}
- name: rabbitmq-password
--
2.17.1

View File

@ -39,10 +39,10 @@ class AodhHelm(openstack.OpenstackBaseHelm):
def _get_pod_overrides(self): def _get_pod_overrides(self):
overrides = { overrides = {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers(), 'api': self._num_controllers(),
'evaluator': self._num_provisioned_controllers(), 'evaluator': self._num_controllers(),
'listener': self._num_provisioned_controllers(), 'listener': self._num_controllers(),
'notifier': self._num_provisioned_controllers() 'notifier': self._num_controllers()
} }
} }
return overrides return overrides

View File

@ -23,7 +23,7 @@ class BarbicanHelm(openstack.OpenstackBaseHelm):
common.HELM_NS_OPENSTACK: { common.HELM_NS_OPENSTACK: {
'pod': { 'pod': {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers() 'api': self._num_controllers()
} }
}, },
'endpoints': self._get_endpoints_overrides(), 'endpoints': self._get_endpoints_overrides(),

View File

@ -41,8 +41,8 @@ class CeilometerHelm(openstack.OpenstackBaseHelm):
def _get_pod_overrides(self): def _get_pod_overrides(self):
return { return {
'replicas': { 'replicas': {
'central': self._num_provisioned_controllers(), 'central': self._num_controllers(),
'notification': self._num_provisioned_controllers() 'notification': self._num_controllers()
} }
} }

View File

@ -49,10 +49,10 @@ class CinderHelm(openstack.OpenstackBaseHelm):
} }
}, },
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers(), 'api': self._num_controllers(),
'volume': self._num_provisioned_controllers(), 'volume': self._num_controllers(),
'scheduler': self._num_provisioned_controllers(), 'scheduler': self._num_controllers(),
'backup': self._num_provisioned_controllers() 'backup': self._num_controllers()
} }
}, },
'conf': { 'conf': {

View File

@ -27,7 +27,7 @@ class FmRestApiHelm(openstack.OpenstackBaseHelm):
'endpoints': self._get_endpoints_overrides(), 'endpoints': self._get_endpoints_overrides(),
'pod': { 'pod': {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers() 'api': self._num_controllers()
}, },
}, },
} }

View File

@ -51,7 +51,7 @@ class GlanceHelm(openstack.OpenstackBaseHelm):
replicas_count = 1 replicas_count = 1
ceph_backend = self._get_primary_ceph_backend() ceph_backend = self._get_primary_ceph_backend()
if ceph_backend: if ceph_backend:
replicas_count = self._num_provisioned_controllers() replicas_count = self._num_controllers()
return { return {
'replicas': { 'replicas': {

View File

@ -39,7 +39,7 @@ class GnocchiHelm(openstack.OpenstackBaseHelm):
def _get_pod_overrides(self): def _get_pod_overrides(self):
return { return {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers() 'api': self._num_controllers()
} }
} }

View File

@ -38,10 +38,10 @@ class HeatHelm(openstack.OpenstackBaseHelm):
def _get_pod_overrides(self): def _get_pod_overrides(self):
return { return {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers(), 'api': self._num_controllers(),
'cfn': self._num_provisioned_controllers(), 'cfn': self._num_controllers(),
'cloudwatch': self._num_provisioned_controllers(), 'cloudwatch': self._num_controllers(),
'engine': self._num_provisioned_controllers() 'engine': self._num_controllers()
} }
} }

View File

@ -62,8 +62,8 @@ class IngressHelm(base.BaseHelm):
common.HELM_NS_OPENSTACK: { common.HELM_NS_OPENSTACK: {
'pod': { 'pod': {
'replicas': { 'replicas': {
'ingress': self._num_provisioned_controllers(), 'ingress': self._num_controllers(),
'error_page': self._num_provisioned_controllers() 'error_page': self._num_controllers()
}, },
'resources': { 'resources': {
'enabled': limit_enabled, 'enabled': limit_enabled,

View File

@ -45,8 +45,8 @@ class IronicHelm(openstack.OpenstackBaseHelm):
common.HELM_NS_OPENSTACK: { common.HELM_NS_OPENSTACK: {
'pod': { 'pod': {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers(), 'api': self._num_controllers(),
'conductor': self._num_provisioned_controllers() 'conductor': self._num_controllers()
} }
}, },
'network': self._get_network_overrides(), 'network': self._get_network_overrides(),

View File

@ -49,7 +49,7 @@ class KeystoneHelm(openstack.OpenstackBaseHelm):
def _get_pod_overrides(self): def _get_pod_overrides(self):
overrides = { overrides = {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers() 'api': self._num_controllers()
}, },
'lifecycle': { 'lifecycle': {
'termination_grace_period': { 'termination_grace_period': {

View File

@ -24,8 +24,8 @@ class MagnumHelm(openstack.OpenstackBaseHelm):
common.HELM_NS_OPENSTACK: { common.HELM_NS_OPENSTACK: {
'pod': { 'pod': {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers(), 'api': self._num_controllers(),
'conductor': self._num_provisioned_controllers() 'conductor': self._num_controllers()
} }
} }
} }

View File

@ -25,8 +25,8 @@ class MariadbHelm(openstack.OpenstackBaseHelm):
common.HELM_NS_OPENSTACK: { common.HELM_NS_OPENSTACK: {
'pod': { 'pod': {
'replicas': { 'replicas': {
'server': self._num_provisioned_controllers(), 'server': self._num_server_replicas(),
'ingress': self._num_provisioned_controllers() 'ingress': self._num_controllers()
} }
}, },
'endpoints': self._get_endpoints_overrides(), 'endpoints': self._get_endpoints_overrides(),

View File

@ -49,7 +49,7 @@ class NeutronHelm(openstack.OpenstackBaseHelm):
common.HELM_NS_OPENSTACK: { common.HELM_NS_OPENSTACK: {
'pod': { 'pod': {
'replicas': { 'replicas': {
'server': self._num_provisioned_controllers() 'server': self._num_controllers()
}, },
}, },
'conf': { 'conf': {

View File

@ -107,13 +107,13 @@ class NovaHelm(openstack.OpenstackBaseHelm):
} }
}, },
'replicas': { 'replicas': {
'api_metadata': self._num_provisioned_controllers(), 'api_metadata': self._num_controllers(),
'placement': self._num_provisioned_controllers(), 'placement': self._num_controllers(),
'osapi': self._num_provisioned_controllers(), 'osapi': self._num_controllers(),
'conductor': self._num_provisioned_controllers(), 'conductor': self._num_controllers(),
'consoleauth': self._num_provisioned_controllers(), 'consoleauth': self._num_controllers(),
'scheduler': self._num_provisioned_controllers(), 'scheduler': self._num_controllers(),
'novncproxy': self._num_provisioned_controllers() 'novncproxy': self._num_controllers()
} }
}, },
'conf': { 'conf': {

View File

@ -30,7 +30,7 @@ class NovaApiProxyHelm(openstack.OpenstackBaseHelm):
} }
}, },
'replicas': { 'replicas': {
'proxy': self._num_provisioned_controllers() 'proxy': self._num_controllers()
} }
}, },
'conf': { 'conf': {

View File

@ -38,7 +38,7 @@ class PankoHelm(openstack.OpenstackBaseHelm):
def _get_pod_overrides(self): def _get_pod_overrides(self):
overrides = { overrides = {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers() 'api': self._num_controllers()
} }
} }
return overrides return overrides

View File

@ -26,7 +26,7 @@ class PlacementHelm(openstack.OpenstackBaseHelm):
common.HELM_NS_OPENSTACK: { common.HELM_NS_OPENSTACK: {
'pod': { 'pod': {
'replicas': { 'replicas': {
'api': self._num_provisioned_controllers() 'api': self._num_controllers()
} }
}, },
'endpoints': self._get_endpoints_overrides() 'endpoints': self._get_endpoints_overrides()

View File

@ -340,9 +340,6 @@ data:
# filesystem race as seen in the LP. # filesystem race as seen in the LP.
volume: volume:
size: 1Gi size: 1Gi
tolerations:
- effect: NoExecute
operator: Exists
source: source:
type: tar type: tar
location: http://172.17.0.1/helm_charts/starlingx/rabbitmq-0.1.0.tgz location: http://172.17.0.1/helm_charts/starlingx/rabbitmq-0.1.0.tgz