Skip to content

Commit 3d8b560

Browse files
committed
fix(apache.service.running): further improve circular requisites
This is a follow-on to PR saltstack-formulas#388: - When the main Apache config file changes, reload Apache - Installing modules only needs the module dir to exist, it doesn't need the entire config. Tightening up this requisite avoids potential circular dependencies.
1 parent 20654f3 commit 3d8b560

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

apache/config/file.sls

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,8 @@ apache-config-file-managed:
9797
- sls: {{ sls_package_install }}
9898
- context:
9999
apache: {{ apache | json }}
100+
- watch_in:
101+
- service: apache-service-running
100102
101103
{%- if grains.os_family in ('Debian', 'FreeBSD') %}
102104

apache/config/modules/install.sls

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ apache-config-modules-{{ module }}-enable:
4141
{%- endif %}
4242
- order: 225
4343
- require:
44-
- sls: {{ sls_config_file }}
44+
- file: apache-config-file-directory-moddir
4545
- watch_in:
4646
- module: apache-service-running-restart
4747
- require_in:

0 commit comments

Comments
 (0)