Comment # 24 on bug 1087891 from Maximilian Meister

(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)