From 2efc06140fda47211c0e5cc387505f60a5a0444f Mon Sep 17 00:00:00 2001
From: Drew Fisher <drew.fisher@oracle.com>
Date: Wed, 15 Mar 2017 06:58:35 -0600
Subject: [PATCH] Issue warning when old Swift ports are used

https://bugs.launchpad.net/swift/+bug/1521339 changed the default ports
from 6000-6002 to 6200-6202 so as not to conflict with X or other
services. This patch is to prepare operators for the eventual change of
the default values.

Change-Id: I84501747af7b3ab46c22f1d76adfe3c629144378
---
 manifests/storage/all.pp                             | 12 ++++++++++++
 .../notes/swift-port-warning-0ac39e6905f977b5.yaml   |  8 ++++++++
 2 files changed, 20 insertions(+)
 create mode 100644 releasenotes/notes/swift-port-warning-0ac39e6905f977b5.yaml

diff --git a/manifests/storage/all.pp b/manifests/storage/all.pp
index 1b081367..e723e96c 100644
--- a/manifests/storage/all.pp
+++ b/manifests/storage/all.pp
@@ -125,6 +125,18 @@ class swift::storage::all(
 
   include ::swift::deps
 
+  if ($object_port == '6000') {
+    warning('The default port for the object storage server has changed from 6000 to 6200 and will be changed in a later release')
+  }
+
+  if ($container_port == '6001') {
+    warning('The default port for the container storage server has changed from 6001 to 6201 and will be changed in a later release')
+  }
+
+  if ($account_port == '6002') {
+    warning('The default port for the account storage server has changed from 6002 to 6202 and will be changed in a later release')
+  }
+
   class { '::swift::storage':
     storage_local_net_ip => $storage_local_net_ip,
   }
diff --git a/releasenotes/notes/swift-port-warning-0ac39e6905f977b5.yaml b/releasenotes/notes/swift-port-warning-0ac39e6905f977b5.yaml
new file mode 100644
index 00000000..9af4e9d7
--- /dev/null
+++ b/releasenotes/notes/swift-port-warning-0ac39e6905f977b5.yaml
@@ -0,0 +1,8 @@
+---
+features:
+  - Adds a warning when ports 6000-6002 are used for object, container,
+    and/or account storage servers. LP#1521339
+upgrade:
+  - Operators shoud prepare for the port change by specifying ports for the
+    object storage server, container storage server, and account storage
+    server.