Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added ability to import ansible_port if set on the inventory file #281

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

numiralofe
Copy link

Hi all,

I have the need when importing hosts from the ansible inventory file to also set custom ssh ports per host, since this feature was not available i made the following changes, my code is perhaps not the best, feel free to propose / change anything if needed.

Introduced changes:
When using ansible inventory files with different ssh ports per host defined within the ansible_port option the plugin was not importing that into rundeck, according to rundeck documentation it is expecting the following format:

hostname:port

The proposed change reads, if set, the ansible_port value from the inventory file and change the hostname to defined format, if not set defaults back to port 22.

@CorvetteCole
Copy link

would love for support of this to be merged, now that rundeck itself maintains this. Have a few workflows that require it

@huapox
Copy link

huapox commented Dec 1, 2023

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants