From 6443ac5efa9816da580fe8546942af0cdf4ed271 Mon Sep 17 00:00:00 2001 From: Volker Theile Date: Wed, 9 Sep 2020 18:08:17 +0200 Subject: [PATCH] cephadm: Remove assignment to member variable in ServiceSpecs Remove unnecessary assignment to member variable 'preview_only', this is done in the constructor of the derived ServiceSpec class. Fixes: https://tracker.ceph.com/issues/47384 Signed-off-by: Volker Theile --- src/python-common/ceph/deployment/service_spec.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/python-common/ceph/deployment/service_spec.py b/src/python-common/ceph/deployment/service_spec.py index 25d9fa5f525c2..05a1389f694b1 100644 --- a/src/python-common/ceph/deployment/service_spec.py +++ b/src/python-common/ceph/deployment/service_spec.py @@ -589,8 +589,6 @@ def __init__(self, #: RADOS namespace where NFS client recovery data is stored in the pool. self.namespace = namespace - self.preview_only = preview_only - def validate(self): super(NFSServiceSpec, self).validate() @@ -660,7 +658,6 @@ def __init__(self, self.rgw_frontend_ssl_certificate = rgw_frontend_ssl_certificate self.rgw_frontend_ssl_key = rgw_frontend_ssl_key self.ssl = ssl - self.preview_only = preview_only def get_port(self): if self.rgw_frontend_port: @@ -724,7 +721,6 @@ def __init__(self, self.api_secure = api_secure self.ssl_cert = ssl_cert self.ssl_key = ssl_key - self.preview_only = preview_only if not self.api_secure and self.ssl_cert and self.ssl_key: self.api_secure = True