-
Notifications
You must be signed in to change notification settings - Fork 17
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
README.adoc: Add a standalone setup part
Signed-off-by: Erwann Roussy <[email protected]>
- Loading branch information
Showing
1 changed file
with
12 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
53e4548
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To avoid confusion in the naming scheme, would it not make sense if the setup playbook would be called something like system_setup_debian.yml instead of cluster_setup_debian.yml to reflect that it handles both cases - standalone and cluster? Same for other cluster* files. Also, what happens if you omit the --limit flag? Is that anything valid or will it constantly result in errors from playing the playbooks?
53e4548
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding the naming, yes you are right, we should change that. There are other playbooks to rename like the CI playbooks
It is part of the Ansible refactoring (see #441 ). It will be done there.
Regarding the
--limit
flag, you don't need it here, in the inventory you provided because you only describe one standalone machine.But it is useful when you have for example VMs or cluster machines all described in the same inventory.
It is just to avoid confusion.