Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

apache and nginx basic and role examples should allow for the port to defined. #92

Closed
tima opened this issue Dec 19, 2016 · 1 comment
Closed

apache and nginx basic and role examples should allow for the port to defined. #92

tima opened this issue Dec 19, 2016 · 1 comment
Labels
enhancement help wanted

Comments

@tima
Copy link
Collaborator

tima commented Dec 19, 2016

Add a variable for the port the web server (nginx or apache) in the playbook and configuration template to which port the web server will bind. This effects basic and role versions of the apache and nginx examples.
@tima tima added the v2 label Dec 19, 2016
@tima tima mentioned this issue Dec 23, 2016
Closed
@tima
Copy link
Collaborator Author

tima commented Jan 4, 2017 •
edited
Loading

I'd like to append this issue to suggest we remove apache_max_keep_alive_requests and nginx_keepalive_timeout from these examples. It's too hard to demonstrate or test (and therefore of little value in this context) as opposed to a port.

@tima tima mentioned this issue Jul 13, 2017
Closed
@tima tima removed the v2 label Sep 19, 2017
bbaassssiiee added a commit to bbaassssiiee/lightbulb that referenced this issue Dec 13, 2017
@bbaassssiiee
Fix for ansible#92
e843f13
@bbaassssiiee bbaassssiiee mentioned this issue Dec 13, 2017
Merged
@tima tima closed this as completed in 2d3bcc9 Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tima