Setup from scratch needs two runs about mysql-formula HOT 8 CLOSED

@cboltz did you test the latest version of this formula?
it works on opensuse Leap. without issues.

from mysql-formula.

not if you define a salt_user ;-) - see my mail on the openSUSE heroes ML for details.

from mysql-formula.

@cboltz
isn't it easier to extend the delete anonymouse stuff with
if the salt_user isn't the mysql_root_user require the salt-user sls state first

something like

servers.sls

{% for host in ['localhost', 'localhost.localdomain', salt['grains.get']('fqdn')] %}
mysql_delete_anonymous_user_{{ host }}:
  mysql_user:
    - absent
    - host: {{ host or "''" }}
    - name: ''
    - connection_host: '{{ mysql_host }}'
    - connection_user: '{{ mysql_salt_user }}'
    {% if mysql_salt_password %}
    - connection_pass: '{{ mysql_salt_password }}'
    {% endif %}
    - connection_charset: utf8
    - require:
      - service: mysqld
      - pkg: mysql_python
      {%- if (mysql_salt_user == mysql_root_user) and mysql_root_password %}
      - cmd: mysql_root_password
      {%- endif %}
      {%- if (mysql_salt_user != mysql_root_user)
      - sls: salt-user
      {%- endif %}
{% endfor %}
{% endif %}
{% endif %}

from mysql-formula.

I'll take everything that works ;-) - and if it's better than my patch/pull request, I won't object.

from mysql-formula.

I haven't tested it I have tested the default mysql-formula on opensuse tumble and leap without your problems, that's why I was asking if you were running from the latest formula.

from mysql-formula.

@cboltz could you test it in your environment?

from mysql-formula.

Just to be sure - your proposal means to add (near the end of the section you quoted, around line 60 in server.sls)

      {%- if (mysql_salt_user != mysql_root_user)
      - sls: salt-user
      {%- endif %}

Right?

If I change this to - sls: mysql.salt-user (note the added mysql.) and also add mysql.salt-user to init.sls (as in my initial patch), it works :-)

from mysql-formula.

Yes, that's the idea, except I'm not happy to hear that the mysql.salt-user state should be added by default to the init.sls maybe a similar wrap as the mysql.dev state could be achieved here.

{% set mysql_salt_user = salt['pillar.get']('mysql:salt_user:salt_user_name', False) %}
include:
{% if mysql_salt_user %}
  - mysql.salt-user
{% endif %}

from mysql-formula.