Commit f16b745f by Brian Coca

now properly uses default port if defined

parent 496186f5
...@@ -19,7 +19,6 @@ ...@@ -19,7 +19,6 @@
from __future__ import (absolute_import, division, print_function) from __future__ import (absolute_import, division, print_function)
__metaclass__ = type __metaclass__ = type
from ansible import constants as C
from ansible.inventory.group import Group from ansible.inventory.group import Group
from ansible.utils.vars import combine_vars from ansible.utils.vars import combine_vars
...@@ -76,7 +75,7 @@ class Host: ...@@ -76,7 +75,7 @@ class Host:
self.ipv4_address = name self.ipv4_address = name
self.ipv6_address = name self.ipv6_address = name
if port and port != C.DEFAULT_REMOTE_PORT: if port:
self.set_variable('ansible_ssh_port', int(port)) self.set_variable('ansible_ssh_port', int(port))
self._gathered_facts = False self._gathered_facts = False
......
...@@ -108,7 +108,7 @@ class InventoryParser(object): ...@@ -108,7 +108,7 @@ class InventoryParser(object):
if len(tokens) == 0: if len(tokens) == 0:
continue continue
hostname = tokens[0] hostname = tokens[0]
port = C.DEFAULT_REMOTE_PORT port = None
# Three cases to check: # Three cases to check:
# 0. A hostname that contains a range pesudo-code and a port # 0. A hostname that contains a range pesudo-code and a port
# 1. A hostname that contains just a port # 1. A hostname that contains just a port
......
...@@ -298,6 +298,10 @@ class PlayContext(Base): ...@@ -298,6 +298,10 @@ class PlayContext(Base):
if variable_name in variables: if variable_name in variables:
setattr(new_info, attr, variables[variable_name]) setattr(new_info, attr, variables[variable_name])
# make sure we get port defaults if needed
if newinfo.port is None and C.DEFAULT_REMOTE_PORT is not None:
newinfo.port = int(C.DEFAULT_REMOTE_PORT)
# become legacy updates # become legacy updates
if not new_info.become_pass: if not new_info.become_pass:
if new_info.become_method == 'sudo' and new_info.sudo_pass: if new_info.become_method == 'sudo' and new_info.sudo_pass:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment