Skip to content
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

Merged
merged 24 commits into from
Aug 27, 2018
Merged

Large refactor to support Ansible 2.5 #976

merged 24 commits into from
Aug 27, 2018

Conversation

jackivanov
Copy link
Collaborator

@jackivanov jackivanov commented May 30, 2018

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

Cloud provider fixes

Usability fixes

Changes to the server

@zoonderkins

This comment has been minimized.

@jackivanov
Copy link
Collaborator Author

@ookangzheng did you try it from scratch? Did you update the requirements? It worked and the latest travis build says it works

@zoonderkins
Copy link

Nope, I reinstall on my current vultr 18.04 ubuntu
I already manually uninstall Wireguard.. still stuck at there

@jackivanov jackivanov force-pushed the ansible2.5 branch 4 times, most recently from ace839b to 1961a44 Compare June 1, 2018 15:02
@aglorei

This comment has been minimized.

@TC1977

This comment has been minimized.

@jackivanov

This comment has been minimized.

@TC1977

This comment has been minimized.

@jackivanov

This comment has been minimized.

@TC1977

This comment has been minimized.

@TC1977

This comment has been minimized.

@TC1977

This comment has been minimized.

@jackivanov
Copy link
Collaborator Author

@TC1977 I can't reproduce this one. Ensure that the requirements are correctly installed

@digeratus
Copy link

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

@TC1977
Copy link
Contributor

TC1977 commented Jun 28, 2018

@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 install certificates and update shell profile commands, then made sure I was using 2.7.15 with python --version. I didn't install homebrew to do it. Algo install still fails at the same "no regions found" step. I also tried changing the ansible version in requirements.txt to 2.5.4. Still fails at the same step.

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?

@jackivanov
Copy link
Collaborator Author

@TC1977 did you try it on a clean python virtual environment?

@jackivanov
Copy link
Collaborator Author

jackivanov commented Jul 30, 2018

@davidemyers Yes, definitely, it's covered here and here

@dnesting

This comment has been minimized.

@jackivanov

This comment has been minimized.

@dguido
Copy link
Member

dguido commented Aug 12, 2018

How is this coming? I have used this branch a few times and did not encounter any errors.

@digeratus
Copy link

@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.

@TC1977
Copy link
Contributor

TC1977 commented Aug 17, 2018

@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.

@msfjarvis
Copy link

FWIW I just deployed a vultr machine with this branch and everything went through perfectly

@dguido
Copy link
Member

dguido commented Aug 27, 2018

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.

@dguido dguido merged commit e8947f3 into master Aug 27, 2018
@dguido dguido deleted the ansible2.5 branch August 27, 2018 14:05
@Hultner
Copy link

Hultner commented Aug 27, 2018

Yay proper FreeBSD support in mainline again

eyecat pushed a commit to eyecat/algo that referenced this pull request Oct 23, 2018
* 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
faf0 pushed a commit to faf0/algo that referenced this pull request Dec 13, 2018
* 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
TC1977 added a commit to TC1977/algo that referenced this pull request Apr 28, 2019
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.
jackivanov pushed a commit that referenced this pull request Apr 29, 2019
Changes the "region not available" question to reflect Algo behavior since #976. Also addresses #1413.

Adds a couple of quote marks to the Ubuntu error question, which disappeared for some reason.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment