Merge "Use oslo_config new type PortOpt for port options"

ironic/api/__init__.py

@@ -21,10 +21,9 @@ API_SERVICE_OPTS = [
     cfg.StrOpt('host_ip',
                default='0.0.0.0',
                help=_('The IP address on which ironic-api listens.')),
-    cfg.IntOpt('port',
-               default=6385,
-               min=1, max=65535,
-               help=_('The TCP port on which ironic-api listens.')),
+    cfg.PortOpt('port',
+                default=6385,
+                help=_('The TCP port on which ironic-api listens.')),
     cfg.IntOpt('max_limit',
                default=1000,
                help=_('The maximum number of items returned in a single '

ironic/common/image_service.py

@@ -48,10 +48,9 @@ glance_opts = [
     cfg.StrOpt('glance_host',
                default='$my_ip',
                help=_('Default glance hostname or IP address.')),
-    cfg.IntOpt('glance_port',
-               default=9292,
-               min=1, max=65535,
-               help=_('Default glance port.')),
+    cfg.PortOpt('glance_port',
+                default=9292,
+                help=_('Default glance port.')),
     cfg.StrOpt('glance_protocol',
                default='http',
                help=_('Default protocol to use when connecting to glance. '

ironic/drivers/modules/ilo/common.py

@@ -48,10 +48,9 @@ opts = [
     cfg.IntOpt('client_timeout',
                default=60,
                help=_('Timeout (in seconds) for iLO operations')),
-    cfg.IntOpt('client_port',
-               default=443,
-               min=1, max=65535,
-               help=_('Port to be used for iLO operations')),
+    cfg.PortOpt('client_port',
+                default=443,
+                help=_('Port to be used for iLO operations')),
     cfg.StrOpt('swift_ilo_container',
                default='ironic_ilo_container',
                help=_('The Swift iLO container to store data.')),

ironic/drivers/modules/irmc/common.py

@@ -26,10 +26,10 @@ from ironic.common.i18n import _
 scci = importutils.try_import('scciclient.irmc.scci')
 
 opts = [
-    cfg.IntOpt('port',
-               default=443,
-               help=_('Port to be used for iRMC operations, either 80 or '
-                      '443')),
+    cfg.PortOpt('port',
+                default=443,
+                help=_('Port to be used for iRMC operations, either 80 or '
+                       '443')),
     cfg.StrOpt('auth_method',
                default='basic',
                help=_('Authentication method to be used for iRMC operations, '

ironic/drivers/modules/virtualbox.py

@@ -46,10 +46,9 @@ VIRTUALBOX_TO_IRONIC_POWER_MAPPING = {
 }
 
 opts = [
-    cfg.IntOpt('port',
-               default=18083,
-               min=1, max=65535,
-               help=_('Port on which VirtualBox web service is listening.')),
+    cfg.PortOpt('port',
+                default=18083,
+                help=_('Port on which VirtualBox web service is listening.')),
 ]
 CONF = cfg.CONF
 CONF.register_opts(opts, group='virtualbox')