Merge pull request #833 from dosaboy/wsgi-socket-rotation

Rename wsgi-rotation to wsgi-socket-rotation
juju · Aug 31, 2023 · 77bb1c8 · 77bb1c8
2 parents 02ee3d1 + 1a90eb0
commit 77bb1c8
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/charmhelpers/contrib/openstack/context.py b/charmhelpers/contrib/openstack/context.py
@@ -1748,9 +1748,9 @@ def __init__(self, name=None, script=None, admin_script=None,
 
     def __call__(self):
         total_processes = _calculate_workers()
-        enable_wsgi_rotation = config('wsgi-rotation')
-        if enable_wsgi_rotation is None:
-            enable_wsgi_rotation = True
+        enable_wsgi_socket_rotation = config('wsgi-socket-rotation')
+        if enable_wsgi_socket_rotation is None:
+            enable_wsgi_socket_rotation = True
         ctxt = {
             "service_name": self.service_name,
             "user": self.user,
@@ -1764,7 +1764,7 @@ def __call__(self):
             "public_processes": int(math.ceil(self.public_process_weight *
                                               total_processes)),
             "threads": 1,
-            "wsgi_rotation": enable_wsgi_rotation,
+            "wsgi_socket_rotation": enable_wsgi_socket_rotation,
         }
         return ctxt
 

diff --git a/charmhelpers/contrib/openstack/templates/wsgi-openstack-api.conf b/charmhelpers/contrib/openstack/templates/wsgi-openstack-api.conf
@@ -12,7 +12,7 @@ Listen {{ admin_port }}
 Listen {{ public_port }}
 {% endif -%}
 
-{% if wsgi_rotation -%}
+{% if wsgi_socket_rotation -%}
 WSGISocketRotation On
 {% else -%}
 WSGISocketRotation Off

diff --git a/tests/contrib/openstack/test_os_contexts.py b/tests/contrib/openstack/test_os_contexts.py
@@ -3570,7 +3570,7 @@ def test_loglevel_context_unset(self):
     def test_wsgi_worker_config_context(self,
                                         _calculate_workers):
         self.config.side_effect = fake_config({
-            'worker-multiplier': 2, 'non-defined-wsgi-rotation': True
+            'worker-multiplier': 2, 'non-defined-wsgi-socket-rotation': True
         })
         _calculate_workers.return_value = 8
         service_name = 'service-name'
@@ -3588,15 +3588,15 @@ def test_wsgi_worker_config_context(self,
             "admin_processes": 2,
             "public_processes": 6,
             "threads": 1,
-            "wsgi_rotation": True,
+            "wsgi_socket_rotation": True,
         }
         self.assertEqual(expect, ctxt())
 
     @patch.object(context, '_calculate_workers')
     def test_wsgi_worker_config_context_user_and_group(self,
                                                        _calculate_workers):
         self.config.side_effect = fake_config({
-            'worker-multiplier': 1, 'wsgi-rotation': False
+            'worker-multiplier': 1, 'wsgi-socket-rotation': False
         })
         _calculate_workers.return_value = 1
         service_name = 'service-name'
@@ -3618,7 +3618,7 @@ def test_wsgi_worker_config_context_user_and_group(self,
             "admin_processes": 1,
             "public_processes": 1,
             "threads": 1,
-            "wsgi_rotation": False,
+            "wsgi_socket_rotation": False,
         }
         self.assertEqual(expect, ctxt())