Browse code

Use swift port variable in keystone and cinder

This patch alows specifiying a deviation of the swift default port 8080 with
variable SWIFT_DEFAULT_BIND_PORT. The created endpoints in keystone for
object-store and the backup_swift_url in cinder.conf will use variable
SWIFT_DEFAULT_BIND_PORT instead of the fixed port 8080.

Change-Id: I47bbcf77368c430718fb8f29b7de1ff305e64422
Closes-Bug: #1489767

Falk Reimann authored on 2015/08/28 19:40:19
Showing 3 changed files
... ...
@@ -256,7 +256,7 @@ function configure_cinder {
256 256
     fi
257 257
 
258 258
     if is_service_enabled swift; then
259
-        iniset $CINDER_CONF DEFAULT backup_swift_url "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:8080/v1/AUTH_"
259
+        iniset $CINDER_CONF DEFAULT backup_swift_url "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:$SWIFT_DEFAULT_BIND_PORT/v1/AUTH_"
260 260
     fi
261 261
 
262 262
     if is_service_enabled ceilometer; then
... ...
@@ -266,9 +266,9 @@ function configure_keystone {
266 266
 
267 267
         # Add swift endpoints to service catalog if swift is enabled
268 268
         if is_service_enabled s-proxy; then
269
-            echo "catalog.RegionOne.object_store.publicURL = http://%SERVICE_HOST%:8080/v1/AUTH_\$(tenant_id)s" >> $KEYSTONE_CATALOG
270
-            echo "catalog.RegionOne.object_store.adminURL = http://%SERVICE_HOST%:8080/" >> $KEYSTONE_CATALOG
271
-            echo "catalog.RegionOne.object_store.internalURL = http://%SERVICE_HOST%:8080/v1/AUTH_\$(tenant_id)s" >> $KEYSTONE_CATALOG
269
+            echo "catalog.RegionOne.object_store.publicURL = http://%SERVICE_HOST%:$SWIFT_DEFAULT_BIND_PORT/v1/AUTH_\$(tenant_id)s" >> $KEYSTONE_CATALOG
270
+            echo "catalog.RegionOne.object_store.adminURL = http://%SERVICE_HOST%:$SWIFT_DEFAULT_BIND_PORT/" >> $KEYSTONE_CATALOG
271
+            echo "catalog.RegionOne.object_store.internalURL = http://%SERVICE_HOST%:$SWIFT_DEFAULT_BIND_PORT/v1/AUTH_\$(tenant_id)s" >> $KEYSTONE_CATALOG
272 272
             echo "catalog.RegionOne.object_store.name = Swift Service" >> $KEYSTONE_CATALOG
273 273
         fi
274 274
 
... ...
@@ -44,6 +44,7 @@ SWIFT_APACHE_WSGI_DIR=${SWIFT_APACHE_WSGI_DIR:-/var/www/swift}
44 44
 SWIFT3_DIR=$DEST/swift3
45 45
 
46 46
 SWIFT_SERVICE_PROTOCOL=${SWIFT_SERVICE_PROTOCOL:-$SERVICE_PROTOCOL}
47
+SWIFT_DEFAULT_BIND_PORT=${SWIFT_DEFAULT_BIND_PORT:-8080}
47 48
 SWIFT_DEFAULT_BIND_PORT_INT=${SWIFT_DEFAULT_BIND_PORT_INT:-8081}
48 49
 SWIFT_SERVICE_LOCAL_HOST=${SWIFT_SERVICE_LOCAL_HOST:-$SERVICE_LOCAL_HOST}
49 50
 SWIFT_SERVICE_LISTEN_ADDRESS=${SWIFT_SERVICE_LISTEN_ADDRESS:-$SERVICE_LISTEN_ADDRESS}
... ...
@@ -62,7 +63,7 @@ SWIFT_CONF_DIR=${SWIFT_CONF_DIR:-/etc/swift}
62 62
 if is_service_enabled s-proxy && is_service_enabled swift3; then
63 63
     # If we are using ``swift3``, we can default the S3 port to swift instead
64 64
     # of nova-objectstore
65
-    S3_SERVICE_PORT=${S3_SERVICE_PORT:-8080}
65
+    S3_SERVICE_PORT=${S3_SERVICE_PORT:-$SWIFT_DEFAULT_BIND_PORT}
66 66
 fi
67 67
 
68 68
 if is_service_enabled g-api; then
... ...
@@ -187,7 +188,7 @@ function _cleanup_swift_apache_wsgi {
187 187
 # _config_swift_apache_wsgi() - Set WSGI config files of Swift
188 188
 function _config_swift_apache_wsgi {
189 189
     sudo mkdir -p ${SWIFT_APACHE_WSGI_DIR}
190
-    local proxy_port=${SWIFT_DEFAULT_BIND_PORT:-8080}
190
+    local proxy_port=${SWIFT_DEFAULT_BIND_PORT}
191 191
 
192 192
     # copy proxy vhost and wsgi file
193 193
     sudo cp ${SWIFT_DIR}/examples/apache2/proxy-server.template $(apache_site_config_for proxy-server)
... ...
@@ -348,7 +349,7 @@ function configure_swift {
348 348
     local csyncfile=${SWIFT_CONF_DIR}/container-sync-realms.conf
349 349
     cp ${SWIFT_DIR}/etc/container-sync-realms.conf-sample ${csyncfile}
350 350
     iniset ${csyncfile} realm1 key realm1key
351
-    iniset ${csyncfile} realm1 cluster_name1 "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:8080/v1/"
351
+    iniset ${csyncfile} realm1 cluster_name1 "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:$SWIFT_DEFAULT_BIND_PORT/v1/"
352 352
 
353 353
     iniuncomment ${SWIFT_CONFIG_PROXY_SERVER} DEFAULT user
354 354
     iniset ${SWIFT_CONFIG_PROXY_SERVER} DEFAULT user ${STACK_USER}
... ...
@@ -369,7 +370,7 @@ function configure_swift {
369 369
     if is_service_enabled tls-proxy; then
370 370
         iniset ${SWIFT_CONFIG_PROXY_SERVER} DEFAULT bind_port ${SWIFT_DEFAULT_BIND_PORT_INT}
371 371
     else
372
-        iniset ${SWIFT_CONFIG_PROXY_SERVER} DEFAULT bind_port ${SWIFT_DEFAULT_BIND_PORT:-8080}
372
+        iniset ${SWIFT_CONFIG_PROXY_SERVER} DEFAULT bind_port ${SWIFT_DEFAULT_BIND_PORT}
373 373
     fi
374 374
 
375 375
     if is_ssl_enabled_service s-proxy; then
... ...
@@ -621,9 +622,9 @@ function create_swift_accounts {
621 621
         get_or_create_service "swift" "object-store" "Swift Service"
622 622
         get_or_create_endpoint "object-store" \
623 623
             "$REGION_NAME" \
624
-            "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:8080/v1/AUTH_\$(tenant_id)s" \
625
-            "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:8080" \
626
-            "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:8080/v1/AUTH_\$(tenant_id)s"
624
+            "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:$SWIFT_DEFAULT_BIND_PORT/v1/AUTH_\$(tenant_id)s" \
625
+            "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:$SWIFT_DEFAULT_BIND_PORT" \
626
+            "$SWIFT_SERVICE_PROTOCOL://$SERVICE_HOST:$SWIFT_DEFAULT_BIND_PORT/v1/AUTH_\$(tenant_id)s"
627 627
     fi
628 628
 
629 629
     local swift_tenant_test1=$(get_or_create_project swifttenanttest1 default)
... ...
@@ -764,7 +765,7 @@ function start_swift {
764 764
         swift-init --run-dir=${SWIFT_DATA_DIR}/run ${type} stop || true
765 765
     done
766 766
     if is_service_enabled tls-proxy; then
767
-        local proxy_port=${SWIFT_DEFAULT_BIND_PORT:-8080}
767
+        local proxy_port=${SWIFT_DEFAULT_BIND_PORT}
768 768
         start_tls_proxy '*' $proxy_port $SERVICE_HOST $SWIFT_DEFAULT_BIND_PORT_INT &
769 769
     fi
770 770
     run_process s-proxy "$SWIFT_DIR/bin/swift-proxy-server ${SWIFT_CONF_DIR}/proxy-server.conf -v"