Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Page MenuHomePhabricator

FlaggedRevsComponent
ActivePublic

Members (5)

Watchers (12)

Details

Description

Allows for article content management by "editor" and "reviewer" classes (Homepage)

Recent Activity

Today

stjn added a comment to T379218: New pages are not autorevieved despite of autoreview rights.

A regression that affects multiple big wikis. See, for example, https://ru.wikipedia.org/wiki/Служебная:Новые_страницы (Special:NewPages)

Thu, Nov 7, 1:13 PM · Regression, FlaggedRevs
stjn triaged T379218: New pages are not autorevieved despite of autoreview rights as Unbreak Now! priority.
Thu, Nov 7, 1:11 PM · Regression, FlaggedRevs
Wargo removed a project from T373276: mw-fr-revision-details is covered by flaggedrevs_reviewform: Patch-For-Review.

Patch abandoned. Problem still persists.

Thu, Nov 7, 3:13 AM · FlaggedRevs
Wargo closed T373600: FlaggedRevs Pending is displayed instead of Checked as Resolved.

Seems resolved as I mentioned above. Tell, if it is recurring.

Thu, Nov 7, 3:09 AM · Patch-For-Review, Regression, FlaggedRevs
Wargo renamed T379217: Remove references to ParserOutputAccess and investigate its impact from Remove references to ParserOutputAccess and investiaget its impact to Remove references to ParserOutputAccess and investigate its impact.
Thu, Nov 7, 1:56 AM · Parsoid, Patch-For-Review, FlaggedRevs
Wargo renamed T379217: Remove references to ParserOutputAccess and investigate its impact from Remove references to ParserOutputAccess to Remove references to ParserOutputAccess and investiaget its impact.
Thu, Nov 7, 1:55 AM · Parsoid, Patch-For-Review, FlaggedRevs
Wargo added a project to T379217: Remove references to ParserOutputAccess and investigate its impact: Parsoid.
Thu, Nov 7, 1:55 AM · Parsoid, Patch-For-Review, FlaggedRevs
gerritbot added a project to T379217: Remove references to ParserOutputAccess and investigate its impact: Patch-For-Review.
Thu, Nov 7, 1:49 AM · Parsoid, Patch-For-Review, FlaggedRevs
gerritbot added a comment to T379217: Remove references to ParserOutputAccess and investigate its impact.

Change #1088003 had a related patch set uploaded (by Wargo; author: Wargo):

[mediawiki/extensions/FlaggedRevs@master] It is unused now here and may cause issues now or in the future.

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

Thu, Nov 7, 1:48 AM · Parsoid, Patch-For-Review, FlaggedRevs
Wargo created T379218: New pages are not autorevieved despite of autoreview rights.
Thu, Nov 7, 1:47 AM · Regression, FlaggedRevs
Wargo created T379217: Remove references to ParserOutputAccess and investigate its impact.
Thu, Nov 7, 1:46 AM · Parsoid, Patch-For-Review, FlaggedRevs

Tue, Nov 5

Reedy added a project to T379085: Latest accepted revision popup sometimes doesn't close: FlaggedRevs.
Tue, Nov 5, 4:21 PM · FlaggedRevs
Bawolff merged T378276: Mass uploads to Commons doesn't work for me into T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).
Tue, Nov 5, 2:22 AM · FlaggedRevs, serviceops, WMF-JobQueue
Scott_French added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

Thanks, @Bawolff - Yes, indeed, those both fan into the low-traffic consumer. While we don't really have a prioritization mechanism in this context that I'm aware of, it would probably be fairly straightforward to at least move them out of low-traffic to a dedicated consumer, as @Ladsgroup points out. I've opened T379035 to look into that.

Tue, Nov 5, 1:07 AM · FlaggedRevs, serviceops, WMF-JobQueue

Mon, Nov 4

Ladsgroup added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

It shouldn't be too hard to give it a dedicated lane with small concurrency

Mon, Nov 4, 10:12 AM · FlaggedRevs, serviceops, WMF-JobQueue

Thu, Oct 31

Bawolff added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

Just as an aside, I believe PublishStashedFile AssembleUploadChunks are considered low traffic job. Unlike normal jobs these are very latency sensitive, as they don't happen in the background, but the UI actually makes users wait well these jobs complete (See also T378276). It would be really great if somehow these jobs can be prioritized in a job queue backlog situation.

Thu, Oct 31, 9:23 PM · FlaggedRevs, serviceops, WMF-JobQueue

Wed, Oct 30

kostajh closed T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes) as Resolved.

Thanks everyone!

Wed, Oct 30, 3:53 PM · FlaggedRevs, serviceops, WMF-JobQueue
Scott_French added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

Thanks @Ammarpad for the pointer to where this category came from, and thanks @cscott for providing some context. Indeed, this seems likely to be an interaction between the new category and the way flaggedrevs is used on dewiki.

Wed, Oct 30, 2:50 PM · FlaggedRevs, serviceops, WMF-JobQueue
Dreamy_Jazz added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

MediaModeration jobs have gone back to normal processing.

Wed, Oct 30, 1:22 PM · FlaggedRevs, serviceops, WMF-JobQueue

Tue, Oct 29

cscott added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

I don't think they are related. That same tracking category appeared on every wiki, nothing explains why it would have cause a load spike for dewiki only. Unless perhaps flagged revs does something "funny" with categories that would cause much higher load?

Tue, Oct 29, 8:55 PM · FlaggedRevs, serviceops, WMF-JobQueue
Ammarpad added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

Thanks, @kostajh.

FWIW, it looks like Kategorie:Wikipedia:Seite,_die_JsonConfig_verwendet is a brand new category created on the 24th, with a lot of associated pages.

Tue, Oct 29, 8:48 AM · FlaggedRevs, serviceops, WMF-JobQueue

Mon, Oct 28

Scott_French added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

FWIW, it looks like Kategorie:Wikipedia:Seite,_die_JsonConfig_verwendet is a brand new category created on the 24th, with a lot of associated pages.

Mon, Oct 28, 7:21 PM · FlaggedRevs, serviceops, WMF-JobQueue
kostajh added a comment to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes).

Looking at the breakdown by wiki for flaggedrevs_CacheUpdate jobs among the last 1M entries in the executor log on mwlog1002:

$ tail -1000000 JobExecutor.log | grep flaggedrevs_CacheUpdate | cut -f 5 -d ' ' | sort | uniq -c | sort -nr
 261208 dewiki
   3026 arwiki
    111 ukwiki
 ...

So yeah, seems to effectively all be targeting dewiki. Example:

Mon, Oct 28, 7:12 PM · FlaggedRevs, serviceops, WMF-JobQueue
Dogu removed a watcher for FlaggedRevs: Dogu.
Mon, Oct 28, 7:10 PM
kostajh added a project to T378385: Spike in JobQueue job backlog time (500ms -> 4-8 minutes): FlaggedRevs.
Mon, Oct 28, 7:08 PM · FlaggedRevs, serviceops, WMF-JobQueue
kostajh edited projects for T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts, added: Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8); removed Trust and Safety Product Sprint (Sprint Cello (Oct 7 - 18)).
Mon, Oct 28, 11:17 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
kostajh moved T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts from Ready to Needs QA on the Trust and Safety Product Sprint (Sprint Cello (Oct 7 - 18)) board.
Mon, Oct 28, 10:31 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
JuxtaposedJacob added a comment to T378329: Pending Changes - append approval message at the end of edit summaries.

@Aklapper
Oops- was I not supposed to delete the labels for the questions?
Thanks,
JuxtaposedJacob

Mon, Oct 28, 9:49 AM · FlaggedRevs
Aklapper added a comment to T378329: Pending Changes - append approval message at the end of edit summaries.

@JuxtaposedJacob: For future reference, please use the feature request form (linked from the top of the task creation page) to create feature requests, and fill in all the sections in the template. Thanks.

Mon, Oct 28, 9:44 AM · FlaggedRevs
Bugreporter added a project to T378329: Pending Changes - append approval message at the end of edit summaries: FlaggedRevs.
Mon, Oct 28, 8:54 AM · FlaggedRevs

Sat, Oct 26

Od1n added a comment to T349793: Replace legacy breakpoint value tokens in all Wikimedia web codebases.

There are a few legacy breakpoints remaining, but not many: Code search.

Sat, Oct 26, 12:49 AM · MW-1.43-notes (1.43.0-wmf.26; 2024-10-08), MobileFrontend, RelatedArticles, MediaWiki-extensions-WikimediaEvents, wikimedia-page-library, FlaggedRevs, CSS, Patch-For-Review, MediaWiki-General, MinervaNeue, Desktop Improvements (Vector 2022), Design-System-Team, Codex

Wed, Oct 23

Dreamy_Jazz moved T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts from Needs review to Ready on the Trust and Safety Product Sprint (Sprint Cello (Oct 7 - 18)) board.

All patches appear merged, but not sure if this is ready for QA yet. Therefore, moving back to 'Ready'.

Wed, Oct 23, 1:57 PM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts

Sun, Oct 20

Pppery moved T144700: Groups 'autoreview', 'autoreviewed' and 'autoreviewer' should be merged from Analysis / under discussion to Backlog on the Wikimedia-Site-requests board.
Sun, Oct 20, 2:23 AM · FlaggedRevs, Chinese-Sites, Wikimedia-Site-requests, WikimediaMessages

Sat, Oct 19

PatchDemoBot added a comment to T365924: Place flaggedrevs_reviewform or mw-data-after-content in the body content area.

Test wiki created on Patch demo by Doğu using patch(es) linked to this task:
http://patchdemo.wmcloud.org/wikis/4f87e8738b/wiki/Main Page

Sat, Oct 19, 8:29 PM · Patch-For-Review, Desktop Improvements (Vector 2022) (Tracking), FlaggedRevs

Thu, Oct 17

Dogu removed a member for FlaggedRevs: Dogu.
Thu, Oct 17, 9:54 PM
mszabo moved T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts from In Progress to Needs review on the Trust and Safety Product Sprint (Sprint Cello (Oct 7 - 18)) board.
Thu, Oct 17, 9:02 PM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
Maintenance_bot removed a project from T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts: Patch-For-Review.
Thu, Oct 17, 11:30 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080714 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] api: Check "viewmywatchlist" right

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

Thu, Oct 17, 11:27 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080387 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] FlaggedRevsStats: Compute temporary and IP user statistics together

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

Thu, Oct 17, 10:31 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts

Wed, Oct 16

gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080713 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] ApiQueryOldreviewedPages: Add integration tests

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

Wed, Oct 16, 5:27 PM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080714 had a related patch set uploaded (by Máté Szabó; author: Máté Szabó):

[mediawiki/extensions/FlaggedRevs@master] api: Check "viewmywatchlist" right for temp accounts

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

Wed, Oct 16, 1:47 PM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080713 had a related patch set uploaded (by Máté Szabó; author: Máté Szabó):

[mediawiki/extensions/FlaggedRevs@master] ApiQueryOldreviewedPages: Add integration tests

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

Wed, Oct 16, 1:47 PM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
PatchDemoBot added a comment to T365924: Place flaggedrevs_reviewform or mw-data-after-content in the body content area.

Test wiki on Patch demo by Doğu using patch(es) linked to this task was deleted:

Wed, Oct 16, 1:30 PM · Patch-For-Review, Desktop Improvements (Vector 2022) (Tracking), FlaggedRevs
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080386 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] FlaggedRevsStats: Add integration tests

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

Wed, Oct 16, 10:01 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
ReleaseTaggerBot edited projects for T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts, added: MW-1.43-notes (1.43.0-wmf.28; 2024-10-22); removed MW-1.43-notes (1.43.0-wmf.27; 2024-10-15).
Wed, Oct 16, 10:00 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080375 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] RevisionReviewForm: Don't increment counters for temporary users

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

Wed, Oct 16, 9:53 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080374 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] RevisionReviewForm: Add integration tests

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

Wed, Oct 16, 9:50 AM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts

Tue, Oct 15

gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080387 had a related patch set uploaded (by Máté Szabó; author: Máté Szabó):

[mediawiki/extensions/FlaggedRevs@master] FlaggedRevsStats: Compute temporary and IP user statistics together

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

Tue, Oct 15, 10:25 PM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a comment to T326934: Investigate what is needed to update FlaggedRevs extension for Temporary accounts.

Change #1080386 had a related patch set uploaded (by Máté Szabó; author: Máté Szabó):

[mediawiki/extensions/FlaggedRevs@master] FlaggedRevsStats: Add integration tests

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

Tue, Oct 15, 10:25 PM · Trust and Safety Product Sprint (Sprint Accordion October 28 - November 8), MW-1.43-notes (1.43.0-wmf.28; 2024-10-22), Trust and Safety Product Team, FlaggedRevs, Temporary accounts
gerritbot added a project to T377263: FlaggedRevs double-counts revertedEdits param when rejecting revisions: Patch-For-Review.
Tue, Oct 15, 8:48 PM · Patch-For-Review, FlaggedRevs