certbot: Properly evaluate when apache is installed
continuous-integration/drone/push Build is failing Details

Checking the existence of /etc/apache2 is not enough as a condition to
validate the presence of apache.

Indeed, some packages (including certbot!!!), put some files in
/etc/apache2/conf-available even if apache isn't installed.

In those cases, the check is not correct, and we'll enter in the apache
block, and fail when we try to enable the configuration.

With this commit, we now validate the presence apache with the presence
of /usr/sbin/apachectl
This commit is contained in:
Ludovic Poujol 2019-11-26 11:58:52 +01:00
parent dc1c78e08a
commit 0e58f34e18
Signed by: lpoujol
GPG Key ID: 6F563E6A4DD5DCEF
2 changed files with 2 additions and 1 deletions

View File

@ -63,6 +63,7 @@ The **patch** part changes incrementally at each release.
* roundcube: fix typo for roundcube vhost
* tomcat: fix typo for default tomcat_version
* evolinux-base: Fix our zsyslog rotate config that doesn't work on Debian 10
* certbot: Properly evaluate when apache is installed
### Security

View File

@ -21,7 +21,7 @@
- name: Check if Apache is installed
stat:
path: /etc/apache2
path: /usr/sbin/apachectl
register: is_apache
- block: