Project

General

Profile

Review #706

ansible-role-httpd_php_fpm: dont_check_potentially_non_existent_path

Added by Pierre-Louis Bonicoli 3 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
Normal
Category:
-
Start date:
2020-07-01
Branch:
ansible-role-httpd_php_fpm@dont_check_potentially_non_existent_path

Description

Repository/branch: ansible-role-httpd_php_fpm/dont_check_potentially_non_existent_path

Don't check existence of potentially nonexistent paths.

Some paths might be created later, for example: /etc/zabbix/zabbix.conf.php. This file can not be rendered before since owner is created in the following task.
Nonexistent path mentioned in open_basedir php configuration seems to be without any consequence.


Related issues

Blocks DuckCorp Infrastructure - Review #704: duckcorp-infra: move supervision serverResolved2020-07-01Actions

History

#1

Updated by Pierre-Louis Bonicoli 3 months ago

  • Copied from Review #705: ansible-role-httpd_php_fpm: create Unix group used for pool workers added
#2

Updated by Pierre-Louis Bonicoli 3 months ago

  • Copied from deleted (Review #705: ansible-role-httpd_php_fpm: create Unix group used for pool workers)
#3

Updated by Pierre-Louis Bonicoli 3 months ago

  • Blocks Review #704: duckcorp-infra: move supervision server added
#4

Updated by Pierre-Louis Bonicoli 3 months ago

  • Copied to Review #707: ansible-role-zabbix: ignore debian bugs #909750 added
#5

Updated by Pierre-Louis Bonicoli 3 months ago

  • Copied to deleted (Review #707: ansible-role-zabbix: ignore debian bugs #909750)
#6

Updated by Marc Dequènes 3 months ago

  • Assignee changed from Marc Dequènes to Pierre-Louis Bonicoli

This was added because paths tend to change in between major Debian releases and it helps discover problems. I would prefer we change it into a warning.

#7

Updated by Pierre-Louis Bonicoli 3 months ago

  • Assignee changed from Pierre-Louis Bonicoli to Marc Dequènes
Updated:
  • warning added using fail module and ignore_errors keyword
  • remove ansible_check_mode exception (since the task will not fail when a path doesn't exist)
#8

Updated by Marc Dequènes 3 months ago

  • Status changed from In Progress to Resolved

Merged, thanks.

Also available in: Atom PDF