Skip to content

Commit

Permalink
feat(servers_config): craft states ids with server instead of `loop…
Browse files Browse the repository at this point in the history
….index0`

to get states easier to extend

BREAKING CHANGES: All `server_conf_{int}` and `server_state_{int}` will be renamed
to `nginx_{server}_server_config_file` and `nginx_{server}_server_state_symlink`
respectively
  • Loading branch information
sticky-note committed Dec 30, 2022
1 parent 982705f commit f580eeb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions nginx/servers_config.sls
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ nginx_server_available_dir:
# Managed enabled/disabled state for servers
{% for server, settings in nginx.servers.managed.items() %}
{% set conf_state_id = 'server_conf_' ~ loop.index0 %}
{%- set conf_state_id = 'nginx_' ~ server ~ '_server_config_file' %}
{% if 'deleted' in settings and settings.deleted %}
{{ conf_state_id }}:
file.absent:
Expand Down Expand Up @@ -143,7 +143,7 @@ nginx_server_available_dir:
{% endif %}
{% if settings.enabled != None %}
{% set status_state_id = 'server_state_' ~ loop.index0 %}
{%- set status_state_id = 'nginx_' ~ server ~ '_server_state_symlink' %}
{%- set enabled_dir = path_join(server, nginx.servers.managed.get(server).get('enabled_dir', nginx.lookup.server_enabled)) -%}
{%- set available_dir = path_join(server, nginx.servers.managed.get(server).get('available_dir', nginx.lookup.server_available)) -%}
{%- if enabled_dir != available_dir %}
Expand Down

0 comments on commit f580eeb

Please sign in to comment.