Merge "Keystone: Remove unnecessary binds for admin endpoint"

openstack-archive · Jun 14, 2022 · e1afca3 · e1afca3
2 parents 4cb5386 + 2de73e7
commit e1afca3
Showing 1 changed file with 5 additions and 15 deletions.
diff --git a/deployment/keystone/keystone-container-puppet.yaml b/deployment/keystone/keystone-container-puppet.yaml
@@ -499,7 +499,6 @@ outputs:
         '111 keystone':
           dport:
             - 5000
-            - {get_param: [EndpointMap, KeystoneAdmin, port]}
       firewall_frontend_rules:
         '100 keystone_public_haproxy_frontend':
           dport:
@@ -582,9 +581,6 @@ outputs:
             keystone::service_name: 'httpd'
             keystone::enable_ssl: {get_param: EnableInternalTLS}
             keystone::wsgi::apache::access_log_format: 'forwarded'
-            keystone::wsgi::apache::api_port:
-              - 5000
-              - {get_param: [EndpointMap, KeystoneAdmin, port]}
             keystone::wsgi::apache::ssl: {get_param: EnableInternalTLS}
             keystone::wsgi::apache::servername:
               str_replace:
@@ -603,18 +599,12 @@ outputs:
             # internal_api -> IP
             # internal_api_uri -> [IP]
             # internal_api_subnet - > IP/CIDR
-            # NOTE: this applies to all 2 bind IP settings below...
             keystone::wsgi::apache::bind_host:
-              - str_replace:
-                  template:
-                    "%{lookup('$NETWORK')}"
-                  params:
-                    $NETWORK: {get_param: [ServiceNetMap, KeystonePublicApiNetwork]}
-              - str_replace:
-                  template:
-                    "%{lookup('$NETWORK')}"
-                  params:
-                    $NETWORK: {get_param: [ServiceNetMap, KeystoneAdminApiNetwork]}
+              str_replace:
+                template:
+                  "%{lookup('$NETWORK')}"
+                params:
+                  $NETWORK: {get_param: [ServiceNetMap, KeystonePublicApiNetwork]}
           - keystone::cache::enabled: {get_param: EnableCache}
             keystone::cache::tls_enabled: {get_param: MemcachedTLS}
           - if: