-
Notifications
You must be signed in to change notification settings - Fork 63
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
add ansible example #197
add ansible example #197
Conversation
Since running the bootstrap.py can take quite a while, I'd suggest the addition of a "stat" module call and only run the bootstrap.py if the client does NOT have a .pem file. The bootstrap.py uses this file as a signal to exit but doing this in Ansible eliminates the Python interpreter call altogether. (And the "force_all_plays_to_run" can be set to true to override this check if necessary.)
|
I must admit I don't see the benefit here. Checking that pem is pretty much the first thing bootstrap does. |
Was the PEM check always there? I though an early version I used re-ran it regardless. Adding in the check saves copying the script. Again, not a big deal in the grand scheme of things so in your environment it might make sense to copy and run regardless. |
The PEM check is there since #120, so no, not always :) |
Ok - I retract my comment/addition as it is not directly aligned with the usage of the bootstrap.py script within an Ansible playbook. |
based on Katello/katello-client-bootstrap#197 will always run bootstrap with --force, so be careful ;) if you don't define bootstrap_login/bootstrap_password it will not try to do puppet, otherwise it will
based on Katello/katello-client-bootstrap#197 will always run bootstrap with --force, so be careful ;) if you don't define bootstrap_login/bootstrap_password it will not try to do puppet, otherwise it will
based on Katello/katello-client-bootstrap#197 will always run bootstrap with --force, so be careful ;) if you don't define bootstrap_login/bootstrap_password it will not try to do puppet, otherwise it will
Can this example also delete |
@sean797 - Sure, the example would just need a "file:" module call with "state: absent". Something like this (untested):
|
this has so far only got positive feedback, and given it's not a code change, but an example, I am going to just merge that now :) |
* make bootstrap.py compatible with Puppet 4 (Katello#230) * add ansible example playbook (Katello#197) * remove SAMs CA consumer RPM in remove_obsolete_packages() (Katello#234) * allow FIPS enabled systems to register successfully with puppet (Katello#236) * allow configuring timeout for API calls and subscription-manager (Katello#237) * implement switching proxy / caps without destroying the host in Foreman (Katello#227)
No description provided.