Skip to content

Commit

Permalink
Simplify the role, remove variable _package_state.
Browse files Browse the repository at this point in the history
  • Loading branch information
robertdebock committed Mar 11, 2019
1 parent e129762 commit fd68e25
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 7 deletions.
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,9 +64,6 @@ httpd_ssl_servername: "{{ ansible_fqdn }}"

# For SSL a TCP port is required.
httpd_ssl_port: 443

# To update all packages installed by this roles, set `httpd_package_state` to `latest`.
httpd_package_state: present
```

Requirements
Expand Down
3 changes: 0 additions & 3 deletions defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,3 @@ httpd_ssl_servername: "{{ ansible_fqdn }}"

# For SSL a TCP port is required.
httpd_ssl_port: 443

# To update all packages installed by this roles, set `httpd_package_state` to `latest`.
httpd_package_state: present
2 changes: 1 addition & 1 deletion tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
- name: install apache httpd
package:
name: "{{ httpd_packages }}"
state: "{{ httpd_package_state }}"
state: present
register: httpd_install_apache_httpd
until: httpd_install_apache_httpd is succeeded
retries: 3
Expand Down

0 comments on commit fd68e25

Please sign in to comment.