Description

let's say I have an apache process group I want to control.

• when using supervisord.running, the name must be apache:* (which is the same notation used by supervisor)
• when using supervisord.dead, the name must be apache: (without the *)

As both states mirror each other, I expect them to use the same convention for name, the one from running since it's the convention from supervisor.

The difference is that the final * is stripped in running : https://github.com/saltstack/salt/blob/master/salt/states/supervisord.py#L83 but not in dead : https://github.com/saltstack/salt/blob/master/salt/states/supervisord.py#L278

Implementing the same behaviour from running in dead is backward compatible

Setup

apache is running:
  supervisord.dead:
  - name: 'apache:*'

# KO but what I expect
KO -- apache is stopped:
  supervisord.dead:
  - name: 'apache:*'

# OK but not what I expect
OK -- apache is stopped:
  supervisord.dead:
  - name: 'apache:'

Steps to Reproduce the behavior

setup a supervisor with an apache config group & run the sls from previous section

Expected behavior

running/dead use the same parameters.

Screenshots

Versions Report
salt 3001.1 (both master/minion)

Additional context