rights to use them
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Zabe
	Mar 25 2022, 6:54 PM

Description

MariaDB [centralauth_p]> select * from localuser limit 1;
ERROR 1356 (HY000): View 'centralauth_p.localuser' references invalid table(s) or column(s) or function(s) or definer/invoker of view lack rights to use them
MariaDB [centralauth_p]>

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	dbproxy: Repool clouddb10(17\|18\|19\|20)	operations/puppet	production	+25 -25

Customize query in gerrit

Related Objects

Mentioned In: T304986: Tool "Stalk toy" doesn't work correct.
T301674: Recreate views for globaluser table
Mentioned Here: rOPUP74aa414272b6: wmcs: stop accessing gu_hidden in maintain-views
T301674: Recreate views for globaluser table

Event Timeline

Zabe created this task.Mar 25 2022, 6:54 PM

Restricted Application edited projects, added cloud-services-team (Kanban); removed cloud-services-team. · View Herald TranscriptMar 25 2022, 6:54 PM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Zabe triaged this task as High priority.Mar 25 2022, 6:54 PM

I think this is just a missed rebuild of the localuser view as part of T301674: Recreate views for globaluser table.

Yeah, before https://gerrit.wikimedia.org/r/c/operations/puppet/+/760953/3/modules/profile/templates/wmcs/db/wikireplicas/maintain-views.yaml the localuser table was checking for gu_hidden='', but since the gu_hidden column has been removed, this no longer works.

Zabe mentioned this in T301674: Recreate views for globaluser table.Mar 25 2022, 8:04 PM

Yes, the views need to be recreated with those columns removed

This probably requires some more expertise eyes from WMCS. I have tried to recreate the view but the following keeps appearing despite of not showing up on maintain-views config

and `centralauth`.`globaluser`.`gu_hidden` = ''`

Zabe mentioned this in T304986: Tool "Stalk toy" doesn't work correct..Mar 29 2022, 7:23 PM

Harej subscribed.Mar 31 2022, 4:37 PM

In T304733#7809592, @Marostegui wrote:
This probably requires some more expertise eyes from WMCS. I have tried to recreate the view but the following keeps appearing despite of not showing up on maintain-views config
and `centralauth`.`globaluser`.`gu_hidden` = ''`

I'm not sure how that could happen other than by a stale /etc/maintain-views.yaml on the host where you are running /usr/local/sbin/maintain-views. That part of the where clause was removed by @Zabe in rOPUP74aa414272b6: wmcs: stop accessing gu_hidden in maintain-views.

bd808 moved this task from Backlog to ToolsDB on the Data-Services board.Apr 12 2022, 11:23 PM

bd808 moved this task from ToolsDB to Wiki replicas on the Data-Services board.

Urbanecm subscribed.Apr 13 2022, 7:01 PM

Pathoschild subscribed.Apr 17 2022, 4:04 PM

MarioGom subscribed.Apr 21 2022, 5:03 PM

OwenBlacker subscribed.Apr 23 2022, 2:29 PM

Stalktoy is still down due to this error (see, for example https://meta.toolforge.org/stalktoy/OwenBlacker).

Is there an ETA for a fix being released?

• EChetty moved this task from Incoming (new tickets) to Event Platform Backlog on the Data-Engineering board.Apr 24 2022, 6:54 AM

alaa subscribed.May 4 2022, 8:23 AM

DarkMatterMan4500 subscribed.May 4 2022, 2:42 PM

• MusikAnimal awarded a token.May 4 2022, 9:21 PM

• MusikAnimal subscribed.

This also breaks Event Metrics. Has anyone other than @Marostegui attempted to rebuild the view? Or could we try again? :)

I'm actually doing it for another reason right now. It'll be done soon.

It seems it's fixed now?

root@clouddb1018:s7[centralauth_p]> select * from localuser limit 1;
+---------+----------------------+-----------------------+--------------------+-------------+--------------+
| lu_wiki | lu_name              | lu_attached_timestamp | lu_attached_method | lu_local_id | lu_global_id |
+---------+----------------------+-----------------------+--------------------+-------------+--------------+
| aawiki  | ! lash.rubberea.com! | 20130411192301        | primary            |          34 |     16773808 |
+---------+----------------------+-----------------------+--------------------+-------------+--------------+
1 row in set (0.002 sec)

It's not pooled yet, I'll pool it soon.

Change 789308 had a related patch set uploaded (by Ladsgroup; author: Amir Sarabadani):

[operations/puppet@production] dbproxy: Repool clouddb10(17|18|19|20)

https://gerrit.wikimedia.org/r/789308

gerritbot added a project: Patch-For-Review.May 4 2022, 10:20 PM

DarkMatterMan4500 unsubscribed.May 4 2022, 10:21 PM

Change 789308 merged by Ladsgroup:

[operations/puppet@production] dbproxy: Repool clouddb10(17|18|19|20)

https://gerrit.wikimedia.org/r/789308

Ladsgroup closed this task as Resolved.May 4 2022, 10:28 PM

Ladsgroup claimed this task.

Ladsgroup edited projects, added DBA; removed Patch-For-Review, Data-Persistence (work done).