-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Large refactor to support Ansible 2.5 #976
Conversation
9117743
to
ddf4d55
Compare
This comment has been minimized.
This comment has been minimized.
@ookangzheng did you try it from scratch? Did you update the requirements? It worked and the latest travis build says it works |
Nope, I reinstall on my current vultr 18.04 ubuntu |
ace839b
to
1961a44
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@TC1977 I can't reproduce this one. Ensure that the requirements are correctly installed |
I'm seeing a ton of changes the last few days on this branch and I'd like to help test it. Just one question, what's the new way syntax to update users from ansible? In the past/currently, it looks like this: ansible-playbook users.yml -e "server_ip=$server_ip server_user=$server_user ssh_tunneling_enabled=$ssh_tunneling_enabled IP_subject_alt_name=$IP_subject easyrsa_CA_password=$easyrsa_CA_password" -t update-users --skip-tags common |
@jackivanov I'm not sure what I'm doing wrong. The standard install of core dependencies and remaining dependencies on the readme.md is working every time for me. I'm running High Sierra which comes with python 2.7.10. I can install the master branch without any problems at all. I upgraded python on my Mac to the latest 2.7.15 from https://www.python.org/downloads/release/python-2715/ and ran the Is it an issue with AWS? I notice that it fails after I enter my access keys, and on the AWS console it doesn't show that the access keys have been used. Anyone else with a Mac want to try? |
@TC1977 did you try it on a clean python virtual environment? |
@davidemyers Yes, definitely, it's covered here and here |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
How is this coming? I have used this branch a few times and did not encounter any errors. |
@dguido So far, the branch is solid. The only topic I would love to see more clarity on is related to issue #963 for which the temporary fix is to disable dos_protection. Now, to me, I interpret that to be a possible Strongswan bug for which there might be a fix in the future. And that's fine. However, the fact still remains that the server might still be susceptible to DOS attacks. Additionally, it's not clear if the issue happens only when ondemand_cellular=true AND ondemand_wifi=true. Why do the Algo options matter? Well, if say ondemand_cellular=false is used, perhaps the issue goes away and the server is still "protected" from DOS attacks since no changes were made to charon. See where I'm going with this? Perhaps, until the Strongswan bug is fixed, ondemand_cellular/wifi should not be used resulting in the worst case scenario that users will have to manually turn on their VPN. Is that better than having a weakened server? That's for the Algo user to decide, but they should have the information to make an educated decision. Just my opinion. |
@digeratus I think it's somewhat of a moot point, because it doesn't look like disabling dos_protection really solves the problem. See #963. |
FWIW I just deployed a vultr machine with this branch and everything went through perfectly |
This branch has been in testing for long enough so we're going to go ahead and merge it. I'm fully expecting that we'll encounter a few bugs but I think that merging it into master is the only way it will get the exposure needed to ferret them out. |
Yay proper FreeBSD support in mainline again |
* Refactoring, booleans declaration and update users fix * Make server_name more FQDN compatible * Rename variables * Define the default value for store_cakey * Skip a prompt about the SSH user if deploying to localhost * Disable reboot for non-cloud deployments * Enable EC2 volume encryption by default * Add default server value (localhost) for the local installation Delete empty files * Add default region to aws_region_facts * Update docs * EC2 credentials fix * Warnings fix * Update deploy-from-ansible.md * Fix a typo * Remove lightsail from the docs * Disable EC2 encryption by default * rename droplet to server * Disable dependencies * Disable tls_cipher_suite * Convert wifi-exclude to a string. Update-users fix * SSH access congrats fix * 16.04 > 18.04 * Dont ask for the credentials if specified in the environment vars * GCE server name fix
* Refactoring, booleans declaration and update users fix * Make server_name more FQDN compatible * Rename variables * Define the default value for store_cakey * Skip a prompt about the SSH user if deploying to localhost * Disable reboot for non-cloud deployments * Enable EC2 volume encryption by default * Add default server value (localhost) for the local installation Delete empty files * Add default region to aws_region_facts * Update docs * EC2 credentials fix * Warnings fix * Update deploy-from-ansible.md * Fix a typo * Remove lightsail from the docs * Disable EC2 encryption by default * rename droplet to server * Disable dependencies * Disable tls_cipher_suite * Convert wifi-exclude to a string. Update-users fix * SSH access congrats fix * 16.04 > 18.04 * Dont ask for the credentials if specified in the environment vars * GCE server name fix
Changes the "region not available" question to reflect Algo behavior since trailofbits#976. Also addresses trailofbits#1413. Adds a couple of quote marks to the Ubuntu error question, which disappeared for some reason.
The huge update is coming. We need people to get it tested and reviewed. It remains a Work-in-Progress.
Under the hood Ansible fixes
openssl_
native modulesmain.yml
and split it into 3 parts:input.yml
,cloud.yml
andserver.yml
algo
script and moves them toinput.yml
and particular roles (Fixes Register inputs in a playbook #942, Fixes Scaleway Server Won't Allocate Unless Typed With Fingers B/C Of a Dumb WLS Bash Bug #833 and Fixes Fails with virtualenvwrapper #762)Cloud provider fixes
Usability fixes
configs/$server_ip/config.yml
Changes to the server