-
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 file install method #242
Conversation
Hi @juju4 - thanks for opening the PR. Would you mind expanding on your specific scenario in order to better understand the ask here? |
This allows to deploy directly from orchestrator system without need for console credentials or additional system (web server to store files) |
@juju4 We can probably refine this a bit to be more inline with the project. For example, we're not going to implement a copy feature as that can be done in a pre-tasks outside the collection itself. However, we do believe in supporting local file path installations for all operating systems so this is something we would like to add. If you agree with the above, then we can contribute our recommended changes to your PR. |
4a47ea5
to
a15b959
Compare
@juju4 Let me know if you are on board with the current proposed changes I have pushed to your PR. Basically, this is how you would use it to get that same functionality you are after: ---
- hosts: all
tasks:
- name: Copy falcon-sensor to remote
ansible.builtin.copy:
src: <path to falcon-sensor on orchestrator>
dest: /tmp/falcon.deb
roles:
- role: crowdstrike.falcon.falcon_install
vars:
falcon_install_method: file
falcon_localfile_path: /tmp/falcon.deb
falcon_localfile_cleanup: yes |
LGTM. Thanks! |
a15b959
to
bd5180c
Compare
Allows to install from a file local to orchestrator