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

Add user home configuration attribute #34

Merged
merged 1 commit into from
Feb 11, 2019

Conversation

jpiron
Copy link

@jpiron jpiron commented Jan 14, 2019

With default to sftp_home_partition + '/' + sftp_user.name.

@johanmeiring
Copy link
Owner

Hi @jpiron, thanks for your patience.

If I run the run-local-tests.sh script locally I get an error result with every single case... in all cases, the directory /home/user2 is not created. Not sure why the Travis integration isn't working anymore (I'll see about fixing that sometime soon). Would you mind double-checking on your end and then fixing it if you find the same behaviour?

With default to sftp_home_partition + '/' + sftp_user.name.
@jpiron
Copy link
Author

jpiron commented Feb 5, 2019

Sorry I forgot to update the run-local-tests.sh with the modified home folder :(

This is now fixed however tests fail on Centos 6 with Ansible 2.2.2.0 (they also fail on master).

@johanmeiring
Copy link
Owner

@jpiron Thanks again for your contribution, I'm happy to merge it. Think I'll remove CentOS 6 support with a major version tag very soon.

@johanmeiring johanmeiring merged commit 7409650 into johanmeiring:master Feb 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants