Skip to content

Commit 789bb9a

Browse files
Adrien "ze" UrbanAdrien "ze" Urban
authored andcommitted
fix(repo): do not override service from package
Package probably provide a valid service, with proper service path. Do not overwrite it. Note: the version that would be written uses path build with /usr/local and minor version directory, while package (on Debian) tend to install in /bin or /usr/bin, ...
1 parent b61cbef commit 789bb9a

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

mongodb/install.sls

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -215,6 +215,7 @@ include:
215215
{%- if 'service' in software and software['service'] %}
216216
{%- set service = software['service'] %}
217217
218+
{%- if package != 'repo' %}
218219
{{ formula }}-{{ comp }}-{{ service.name }}-install-service-directory:
219220
file.directory:
220221
- name: {{ d.dir.var }}/{{ name }}
@@ -226,7 +227,7 @@ include:
226227
- sls: {{ sls_config_users }}
227228
- file: {{ formula }}-install-prerequisites
228229
- require_in:
229-
{%- if grains.kernel == 'Linux' %}
230+
{%- if grains.kernel == 'Linux' %}
230231
- file: {{ formula }}-{{ comp }}-{{ service.name }}-install-service-systemd
231232
232233
{{ formula }}-{{ comp }}-{{ service.name }}-install-service-systemd:
@@ -251,10 +252,9 @@ include:
251252
start: {{ software['path'] }}/bin/{{ name }}
252253
- watch_in:
253254
- cmd: {{ formula }}-{{ comp }}-{{ service.name }}-install-service-systemd
254-
cmd.run:
255+
cmd.wait:
255256
- name: systemctl daemon-reload
256-
257-
{%- elif grains.kernel == 'Darwin' %}
257+
{%- elif grains.kernel == 'Darwin' %}
258258
{%- set servicename = name if 'name' not in service else service.name %}
259259
- require_in:
260260
- file: {{ formula }}-{{ comp }}-{{ servicename }}-install-service-launched
@@ -275,9 +275,10 @@ include:
275275
user: {{ software['user'] }}
276276
limits: {{ d.limits }}
277277
278-
{%- endif %} {# linux/darwin #}
278+
{%- endif %} {# linux/darwin #}
279+
{%- endif %} {# !repo #}
279280
{%- endif %} {# service #}
280281
{%- endif %} {# wanted #}
281282
{%- endfor %} {# component #}
282283
{%- endif %} {# wanted #}
283-
{%- endfor %} {# components #}
284+
{%- endfor %} {# components #}

0 commit comments

Comments
 (0)