http://bugzilla.suse.com/show_bug.cgi?id=1087891 http://bugzilla.suse.com/show_bug.cgi?id=1087891#c24 --- Comment #24 from Maximilian Meister <mmeister@suse.com> --- (In reply to Klaus Kämpf from comment #22)
(In reply to Maximilian Meister from comment #19)
Thorsten, i also found out there are some more places upstream besides returners and modules that didnt have the drop in replacement mechanism yet. e.g. salt/pillar/mysql.py or salt/auth/mysql.py
i updated the patch in https://build.opensuse.org/package/show/home:m_meister:branches: systemsmanagement:saltstack/salt
Awaiting your pull request at https://github.com/openSUSE/salt ;-)
klaus, can i also wait for https://github.com/saltstack/salt/pull/46890 to get merged and backport it to the openSUSE fork, or what's the usual workflow? (In reply to Thorsten Kukuk from comment #23)
Update: I have fixed the patch so that PyMySQL will be loaded now, but of course the interface is not 100% compatible:
08:50:08,200 [ERROR ] Could not store events - returner 'mysql.event_return' raised exception: module 'pymysql.connections' has no attribute 'OperationalError' 08:50:14,489 [CRITICAL] Specified ext_pillar interface velum is unavailable
thorsten, can you check which version of PyMySQL is installed? the OperationalError should get imported here: https://github.com/PyMySQL/PyMySQL/blob/master/pymysql/connections.py#L25 if that doesn't work maybe we need to import the err.py explicitly... but i am no python expert, to me it looks like it should be loaded already i have also changed again the patch after some upstream feedback to fix linting issues (the undefined HAS_MYSQLDB and an import statement) -- You are receiving this mail because: You are on the CC list for the bug.