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

Generate inventory and ssh config dynamically #87

Merged
merged 7 commits into from
Mar 6, 2024

Conversation

xhernandez
Copy link
Collaborator

This change reduces the tasks done by the provisioners to the minimum, moving most of the tasks to other common places. It also builds the inventory in a dynamic way to provide flexibility for the provisioners.

The SSH configuration is also built dynamically and it uses a new key to allow password-less root SSH between all nodes, making it independent of vagrant.

Depends on #86.

@xhernandez xhernandez force-pushed the ssh-config branch 5 times, most recently from c578b73 to 1a61584 Compare February 28, 2024 18:59
@xhernandez xhernandez mentioned this pull request Feb 28, 2024
@xhernandez xhernandez marked this pull request as ready for review February 28, 2024 19:05
Copy link

dpulls bot commented Feb 29, 2024

🎉 All dependencies have been resolved !

Signed-off-by: Xavi Hernandez <[email protected]>
The dynamically created inventory is stored on a file to be used in
other independent playbooks.

Signed-off-by: Xavi Hernandez <[email protected]>
Accesses to 'config.nodes[inventory_hostname]' can now be simplified to
just 'site'.

Signed-off-by: Xavi Hernandez <[email protected]>
Copy link
Collaborator

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for all the reorganizing work, really appreciated.

@anoopcs9 anoopcs9 merged commit 445cf5d into samba-in-kubernetes:main Mar 6, 2024
6 checks passed
@xhernandez xhernandez deleted the ssh-config branch March 7, 2024 21:07
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.

2 participants