-
Notifications
You must be signed in to change notification settings - Fork 23
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 Kali backport #180
base: develop
Are you sure you want to change the base?
Add Kali backport #180
Conversation
…lic facing URL to access
- name: Update apt cache | ||
apt: | ||
update_cache: yes |
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.
Remove this task. This is redundant with the next task.
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.
The only thing that we should have in this file is import_tasks
. Break everything out into their own task file if possible to stick to the standard.
- name: Create VNC user | ||
user: | ||
name: "{{ vnc_username }}" | ||
password: "{{ vnc_password | password_hash('sha512') }}" | ||
shell: /bin/bash | ||
|
||
- name: Set up VNC password | ||
expect: | ||
command: vncserver | ||
responses: | ||
Password: "{{ vnc_password }}" | ||
Verify: "{{ vnc_password }}" | ||
"Would you like to enter a view-only password (y/n)?": "n" | ||
become: yes | ||
become_user: "{{ vnc_username }}" |
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.
This output to the log. Use no_log to avoid leaking secrets to the stackscript log.
kali_package: "{{ 'kali-linux-everything' if '${EVERYTHING}' == 'Yes' else 'kali-linux-headless' if '${HEADLESS}' == 'Yes' else '' }}" | ||
|
||
# VNC settings | ||
vnc_enabled: "{{ '${VNC}' == 'Yes' }}" |
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.
We are making boolean conditional in the playbook. This is a string. We need to translate 'Yes' or 'No' to either be True
or False
in the vars file.
No description provided.