Page MenuHomePhabricator
Create Task
Maniphest T278128

Investigate Kartographer workaround for FlaggedRevs
Closed, ResolvedPublic
Actions

Description

We want to explore potential solutions for Kartotherian always pulling the latest revision of maps metadata, when it should be pulling the latest *published* revision on FlaggedRevs wikis.

Relevant documents:

Details

SubjectRepoBranchLines +/-
[WIP] Pull stable revision of mapdatamediawiki/extensions/Kartographermaster+24 -2
Revert "Allow Kartotherian to connect back to a demo server"maps/kartotherian/deploymaster+0 -2
[POC] Allow mapdata API to fetch data from old revisionsmediawiki/extensions/Kartographermaster+25 -7
Allow Kartotherian to connect back to a demo servermaps/kartotherian/deploymaster+2 -0
[POC] Allow mapdataLoader to fetch data from old revisionsmediawiki/services/kartotherianmaster+11 -9
[POC] Allow DataLoader to fetch mapdata from old revisionsmapdatamaster+8 -4
Fix obvious copy-paste error in HybridGroup.parse()mapdatamaster+4 -4
Customize query in gerrit

Event Timeline

awight created this task.Mar 22 2021, 2:01 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 22 2021, 2:01 PM
gerritbot added a comment.Mar 22 2021, 2:05 PM

Change 674023 had a related patch set uploaded (by Awight; owner: Awight):
[maps/kartotherian/deploy@master] Allow Kartotherian to connect back to a demo server

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

gerritbot added a project: Patch-For-Review.Mar 22 2021, 2:05 PM
gerritbot added a comment.Mar 23 2021, 7:17 AM

Change 674093 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Kartographer@master] [POC] Allow mapdata API to fetch data from old revisions

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

gerritbot added a comment.Mar 23 2021, 7:30 AM

Change 674023 merged by jenkins-bot:
[maps/kartotherian/deploy@master] Allow Kartotherian to connect back to a demo server

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

gerritbot added a comment.Mar 23 2021, 7:49 AM

Change 674252 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mapdata@master] Fix obvious copy-paste error in HybridGroup.parse()

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

gerritbot added a comment.Mar 23 2021, 8:20 AM

Change 674257 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mapdata@master] [POC] Allow DataLoader to fetch mapdata from old revisions

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

gerritbot added a comment.Mar 23 2021, 8:27 AM

Change 674259 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/services/kartotherian@master] [POC] Allow mapdataLoader to fetch data from old revisions

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

gerritbot added a comment.Mar 23 2021, 10:33 AM

Change 674276 had a related patch set uploaded (by Awight; owner: Awight):
[mediawiki/extensions/Kartographer@master] [WIP] Avoid caching unpublished content

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

thiemowmde updated the task description. (Show Details)Mar 23 2021, 4:15 PM
thiemowmde updated the task description. (Show Details)Mar 23 2021, 4:20 PM
gerritbot added a comment.Apr 28 2021, 10:45 AM

Change 674252 merged by jenkins-bot:

[mapdata@master] Fix obvious copy-paste error in HybridGroup.parse()

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

gerritbot added a comment.Oct 19 2021, 8:46 AM

Change 674257 abandoned by Thiemo Kreuz (WMDE):

[mapdata@master] [POC] Allow DataLoader to fetch mapdata from old revisions

Reason:

Superseded by If98623f.

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

gerritbot added a comment.Oct 26 2021, 11:57 AM

Change 674259 abandoned by Thiemo Kreuz (WMDE):

[mediawiki/services/kartotherian@master] [POC] Allow mapdataLoader to fetch data from old revisions

Reason:

Replaced by I2889b7e.

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

thiemowmde closed this task as Resolved.Nov 9 2021, 12:54 PM
thiemowmde claimed this task.
gerritbot added a comment.Nov 29 2021, 10:37 AM

Change 742255 had a related patch set uploaded (by Awight; author: Awight):

[maps/kartotherian/deploy@master] Revert \"Allow Kartotherian to connect back to a demo server\"

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

gerritbot added a comment.Dec 3 2021, 9:06 AM

Change 674093 abandoned by Thiemo Kreuz (WMDE):

[mediawiki/extensions/Kartographer@master] [POC] Allow mapdata API to fetch data from old revisions

Reason:

This is done in I7097a39 as well as I9f7ecce now.

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

gerritbot added a comment.Feb 8 2022, 1:14 PM

Change 742255 abandoned by MSantos:

[maps/kartotherian/deploy@master] Revert \"Allow Kartotherian to connect back to a demo server\"

Reason:

Already done in another patch

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

gerritbot added a comment.May 8 2024, 11:17 AM

Change #674276 abandoned by Thiemo Kreuz (WMDE):

[mediawiki/extensions/Kartographer@master] [WIP] Pull stable revision of mapdata

Reason:

I think what we ended implementing via I58af8d7 makes this obsolete.

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

Maintenance_bot removed a project: Patch-For-Review.May 8 2024, 11:31 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL