-
Notifications
You must be signed in to change notification settings - Fork 95
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
Document VCS integration in foreman_ansible plugin #2920
base: master
Are you sure you want to change the base?
Document VCS integration in foreman_ansible plugin #2920
Conversation
The PR preview for 44ccc02 is available at theforeman-foreman-documentation-preview-pr-2920.surge.sh The following output files are affected by this PR: |
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.
any chance to handle this using foreman-instaler?
fd51ddb
to
44ccc02
Compare
Note that you can only clone Ansible roles through `http` and `https`. | ||
|
||
* Click *Examine* to select a git revision from the list of branches and tags. | ||
* Toggle between *Skipping existing* and *Update existing* to allow {Project} to override a given role if it is already present on the {SmartProxy}. |
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.
* Toggle between *Skipping existing* and *Update existing* to allow {Project} to override a given role if it is already present on the {SmartProxy}. | |
* Toggle between *Skipping existing* and *Update existing* to allow {Project} to override a given role if it is already present on your {SmartProxy}. |
@@ -18,6 +18,7 @@ endif::[] | |||
* `/usr/share/ansible/roles` | |||
* `/etc/ansible/collections` | |||
* `/usr/share/ansible/collections` | |||
* `(optional) VCS_INTEGRATION_PATH/ansible/roles` |
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.
where does "VCS_INTEGRATION_PATH" come from? Is this something a user has to replace?
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.
For nightly, I think these manualy steps are more or less OK. For the next release, I think it would be best to do this via foreman-installer
.
Thanks for documenting this feature! I followed the guide and dont have the vcs capability in the Ansible feature. is there something else need doing (other than the steps in the doc)? |
This pull request adds documentation for VCS integration in foreman_ansible.
Still subject to change.
Requires: theforeman/foreman_ansible#676
Requires: theforeman/smart_proxy_ansible#85