Pchelolo
User

Projects (8)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Jun 24 2015, 10:23 AM (109 w, 2 d)
Availability
Available
LDAP User
Unknown
MediaWiki User
Pchelolo

Recent Activity

Yesterday

Pchelolo added a comment to T163380: Support posting Jobs to EventBus simultaneously with normal job processing.

Ok, the code is life on all wikis, now it's time to start gradually enabling it. First step - enable in labs and I've created the following config change for that: https://gerrit.wikimedia.org/r/#/c/368206/

Thu, Jul 27, 6:31 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, User-mobrovac, Analytics, ChangeProp, Services (later), EventBus

Wed, Jul 26

Pchelolo added a comment to T171788: On wikis without changeprop enabled, lint errors don't update after page edits.

So EventBus/Change-Prop is not enabled on those wikis because RESTBase is not enabled, and as CP right now is mostly used to update restbase, there was no need to enable EventBus or CP. However, as EventBus is going to replace the JobQueue eventually, we need to enable it on all wikis (or close to all wikis). Technically, even if we don't need or want RESTBase on a wiki, we can craft special rules for change-prop that will directly hit Parsoid.

Wed, Jul 26, 9:36 PM · wikitech.wikimedia.org, MediaWiki-extensions-Linter, Services, Parsoid
Pchelolo added a comment to T171788: On wikis without changeprop enabled, lint errors don't update after page edits.

I would guess that the issue would be present on all wikis where EventBus is not enabled: https://github.com/wikimedia/operations-mediawiki-config/blob/master/wmf-config/InitialiseSettings.php#L19000

Wed, Jul 26, 9:00 PM · wikitech.wikimedia.org, MediaWiki-extensions-Linter, Services, Parsoid

Tue, Jul 25

Pchelolo added a comment to T171650: RESTBase fails to start in MW-Vagrant.

After the patch above is merged, do git pull on vagrant repo to pull the latest puppet, and after running vagrant provision RESTBase should start working again.

Tue, Jul 25, 10:38 PM · Patch-For-Review, RESTBase, MediaWiki-Vagrant
Pchelolo committed rMSRDe7adea00a002: Update recommendation-api to 95c9bd1 (authored by Pchelolo).
Update recommendation-api to 95c9bd1
Tue, Jul 25, 8:51 PM
Pchelolo merged T171650: RESTBase fails to start in MW-Vagrant into T171592: Can't start RESTBase, expecting cx_host to be specified.
Tue, Jul 25, 8:39 PM · Services (next), RESTBase
Pchelolo merged task T171650: RESTBase fails to start in MW-Vagrant into T171592: Can't start RESTBase, expecting cx_host to be specified.
Tue, Jul 25, 8:39 PM · Patch-For-Review, RESTBase, MediaWiki-Vagrant
Pchelolo reopened T171592: Can't start RESTBase, expecting cx_host to be specified as "Open".

Errr... Accidentally merged the issues wrong way.

Tue, Jul 25, 8:39 PM · Services (next), RESTBase
Pchelolo added a comment to T171650: RESTBase fails to start in MW-Vagrant.

This is another symptom of T171592

Tue, Jul 25, 8:38 PM · Patch-For-Review, RESTBase, MediaWiki-Vagrant
Pchelolo merged T171592: Can't start RESTBase, expecting cx_host to be specified into T171650: RESTBase fails to start in MW-Vagrant.
Tue, Jul 25, 8:38 PM · Patch-For-Review, RESTBase, MediaWiki-Vagrant
Pchelolo merged task T171592: Can't start RESTBase, expecting cx_host to be specified into T171650: RESTBase fails to start in MW-Vagrant.
Tue, Jul 25, 8:38 PM · Services (next), RESTBase
Pchelolo committed rMSCDc77a542cb0be: Update change-propagation to 9541214 (authored by Pchelolo).
Update change-propagation to 9541214
Tue, Jul 25, 8:22 PM
Pchelolo committed rMSCD444223d6a4bc: Update change-propagation to 15fc2dc (authored by Pchelolo).
Update change-propagation to 15fc2dc
Tue, Jul 25, 8:22 PM
Pchelolo committed rMSTED22967f37f01e: Update trending-edits to 713323e (authored by Pchelolo).
Update trending-edits to 713323e
Tue, Jul 25, 8:17 PM

Mon, Jul 24

Pchelolo added a parent task for T171533: Document ChangeProp and EventBus monitoring: T157088: [EPIC] Develop a JobQueue backend based on EventBus.
Mon, Jul 24, 9:45 PM · EventBus, Analytics, Services (later)
Pchelolo added a subtask for T157088: [EPIC] Develop a JobQueue backend based on EventBus: T171533: Document ChangeProp and EventBus monitoring.
Mon, Jul 24, 9:45 PM · Epic, Services (doing), User-mobrovac, Analytics, ChangeProp, EventBus
Pchelolo created T171533: Document ChangeProp and EventBus monitoring.
Mon, Jul 24, 9:45 PM · EventBus, Analytics, Services (later)

Fri, Jul 21

Pchelolo closed T170902: Don't add the TID by default in storage module as Resolved.

Merged to the next-gem branch. Resolving.

Fri, Jul 21, 5:49 PM · Services (next), RESTBase
Pchelolo closed T170902: Don't add the TID by default in storage module, a subtask of T169938: End of July milestone: Initial implementation & start of testing, as Resolved.
Fri, Jul 21, 5:49 PM · Goal, Epic, Services (doing)

Tue, Jul 18

Pchelolo updated the task description for T171011: Update EventStreams to service-template-node v0.5.2.
Tue, Jul 18, 11:39 PM · Analytics, Wikimedia-Stream, Services (watching)
Pchelolo created T171011: Update EventStreams to service-template-node v0.5.2.
Tue, Jul 18, 11:38 PM · Analytics, Wikimedia-Stream, Services (watching)
Pchelolo created T171002: Resurrect sending original content to Parsoid.
Tue, Jul 18, 11:01 PM · Services (blocked), Parsoid
Pchelolo added a comment to T170997: Stop sending original content to Parsoid.

PR: https://github.com/wikimedia/restbase/pull/837

Tue, Jul 18, 10:59 PM · Parsoid, Services (designing)
Pchelolo added a comment to T170997: Stop sending original content to Parsoid.

Considering T98995, it's probably fine. But if we decide to do that, we should also remove the functionality from Parsoid altogether.

Tue, Jul 18, 10:46 PM · Parsoid, Services (designing)
Pchelolo created T170997: Stop sending original content to Parsoid.
Tue, Jul 18, 10:02 PM · Parsoid, Services (designing)

Mon, Jul 17

Pchelolo added a comment to T170902: Don't add the TID by default in storage module.

Right now we can get away from this by using the timeuuid as an index in the timeline tables as we can create TIDs for arbitrary dates. However, the plan for the key_value buckets was to start simply overwriting the content, so this automatic generation of a tid will get in our way..

Mon, Jul 17, 11:50 PM · Services (next), RESTBase
Pchelolo added subtasks for T156199: Low-latency current revision storage: T170902: Don't add the TID by default in storage module, T170901: Expose default table TTL.
Mon, Jul 17, 11:05 PM · User-mobrovac, Cassandra, Services (designing), Wikimedia-Incident, RfC, RESTBase
Pchelolo added a parent task for T170901: Expose default table TTL: T156199: Low-latency current revision storage.
Mon, Jul 17, 11:05 PM · Services (next), RESTBase
Pchelolo added a parent task for T170902: Don't add the TID by default in storage module: T156199: Low-latency current revision storage.
Mon, Jul 17, 11:04 PM · Services (next), RESTBase
Pchelolo created T170902: Don't add the TID by default in storage module.
Mon, Jul 17, 11:02 PM · Services (next), RESTBase
Pchelolo created T170901: Expose default table TTL.
Mon, Jul 17, 11:00 PM · Services (next), RESTBase
Pchelolo committed rMSCP6ab4e64ff610: Extend blacklisting for derived updates (authored by Pchelolo).
Extend blacklisting for derived updates
Mon, Jul 17, 8:03 PM

Fri, Jul 14

Pchelolo added a comment to T169929: iOS app requests summaries with non-normalized titles.

@JoeWalsh RESTBase uses / for routing, so all the titles in the request should be URI-encoded like encodeURIComponent(title). So the / becomes %2F

Fri, Jul 14, 6:53 PM · iOS-app-v5.6.0-Goat-On-A-Train, Services (watching), Wikipedia-iOS-App-Backlog, iOS-app-Bugs
Pchelolo added a comment to T170692: Return common URLs in summary API so clients do not have to perform bug prone string manipulation .

This looks exactly like the concept of REST HATEOAS - the response here is the driver for the client to discover more APIs.

Fri, Jul 14, 4:38 PM · Reading-Infrastructure-Team-Backlog, Services (watching)

Thu, Jul 13

Pchelolo committed rMSCD03a3591e995e: Config: Only update restbase if the revision content changed. (authored by Pchelolo).
Config: Only update restbase if the revision content changed.
Thu, Jul 13, 11:25 PM
Pchelolo closed T170629: Treat 'null' as 'undefined' when verifying the event schemas as Declined.

We don't really have any place where we use 'null' meaningfully yet, but I do agree it might be the case in the future. I'll decline my own task :)

Thu, Jul 13, 8:20 PM · EventBus, Services (watching), Analytics
Pchelolo added a comment to T170629: Treat 'null' as 'undefined' when verifying the event schemas.

@Ottomata I was more wondering if we can just treat null as undefined generically and whether that's a good idea. So that if the field is optional, then null is allowed by default

Thu, Jul 13, 8:12 PM · EventBus, Services (watching), Analytics
Pchelolo created T170629: Treat 'null' as 'undefined' when verifying the event schemas.
Thu, Jul 13, 8:06 PM · EventBus, Services (watching), Analytics
Pchelolo closed T161935: Vagrant git-update error for event logging as Invalid.

Ok, seems like something was corrupted in my vagrant setup. After deleting the virtualenv and reprovisioning it finally started to work correctly.

Thu, Jul 13, 7:36 PM · Analytics, Analytics-EventLogging, MediaWiki-Vagrant, Services (watching)
Pchelolo added a comment to T162468: Restbase install broken on Vagrant (Cannot find module 'npmlog').

@Krinkle Interestingly, for me vagrant git-update also works with my patch.

Thu, Jul 13, 6:07 PM · Patch-For-Review, Services (next), RESTBase, MediaWiki-Vagrant
Pchelolo renamed T170606: Add Accept header to webrequest logs from Add Accept header to web request Eventlogging to Add Accept header to webrequest table.
Thu, Jul 13, 5:29 PM · Services (watching), Analytics
Pchelolo added a comment to T170602: License for pageview data.

There's a line in the top description specifying Apache2

Thu, Jul 13, 5:21 PM · Patch-For-Review, Analytics-Kanban, Services (watching), WMF-Legal
Pchelolo created T170606: Add Accept header to webrequest logs.
Thu, Jul 13, 5:18 PM · Services (watching), Analytics
Pchelolo added a comment to T162468: Restbase install broken on Vagrant (Cannot find module 'npmlog').

The git-update portion (as usual) failed at RESTBase with Maximum call stack size exceeded . But then inside the vm, the following worked.

Thu, Jul 13, 3:54 AM · Patch-For-Review, Services (next), RESTBase, MediaWiki-Vagrant
Pchelolo added a comment to T162468: Restbase install broken on Vagrant (Cannot find module 'npmlog').

Found a workaround that seems to work for me - downgrading npm to version 2. Uglyish solution, I know, but I couldn't find anything nicer in tens on npm bugs related to this.

Thu, Jul 13, 1:06 AM · Patch-For-Review, Services (next), RESTBase, MediaWiki-Vagrant
Pchelolo added a comment to T162468: Restbase install broken on Vagrant (Cannot find module 'npmlog').

@GWicke This is still happening for me. I've tried today to test various npm versions and find answers on google, but no luck for now.

Thu, Jul 13, 12:16 AM · Patch-For-Review, Services (next), RESTBase, MediaWiki-Vagrant

Tue, Jul 11

Pchelolo updated the task description for T157088: [EPIC] Develop a JobQueue backend based on EventBus.
Tue, Jul 11, 8:42 PM · Epic, Services (doing), User-mobrovac, Analytics, ChangeProp, EventBus
Pchelolo added a comment to T163380: Support posting Jobs to EventBus simultaneously with normal job processing.

What's the desired end state? I imagine you won't use JobQueue::pop / JobQueue::ack as using a hook wouldn't make sense then. So JobQueueEventBus will push to kafka on push, everything else would be a noop, and something else would be responsible for actually running the jobs?

Tue, Jul 11, 8:39 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, User-mobrovac, Analytics, ChangeProp, Services (later), EventBus
Pchelolo added a comment to T157088: [EPIC] Develop a JobQueue backend based on EventBus.

Here's the presentation from the discussion of the options on the Developer Summit 2017: https://commons.wikimedia.org/wiki/File:Asynchronous_processing_on_the_WMF_cluster.pdf

Tue, Jul 11, 8:36 PM · Epic, Services (doing), User-mobrovac, Analytics, ChangeProp, EventBus
Pchelolo moved T164006: Suggestion: API for fetching lint errors for a specific revision from doing to blocked on the Services board.
Tue, Jul 11, 8:20 PM · Services (blocked), RESTBase-API, Parsing-Team, MediaWiki-extensions-Linter
Pchelolo moved T169939: End of August milestone: Cassandra 3 cluster in production from doing to next on the Services board.
Tue, Jul 11, 8:20 PM · Services (later), Operations, Goal, Epic
Pchelolo moved T170319: End of August milestone: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure. from doing to next on the Services board.
Tue, Jul 11, 8:20 PM · Services (later), User-Joe, User-Elukey, Goal, Epic
Pchelolo closed T157089: Add storage to Change-Prop for deduplication as Resolved.

Redis was added to ChangeProp nodes and is already successfully used for blacklisting unparseable pages. This is done.

Tue, Jul 11, 8:09 PM · Services (done), Operations, ChangeProp
Pchelolo closed T157089: Add storage to Change-Prop for deduplication, a subtask of T157088: [EPIC] Develop a JobQueue backend based on EventBus, as Resolved.
Tue, Jul 11, 8:09 PM · Epic, Services (doing), User-mobrovac, Analytics, ChangeProp, EventBus
Pchelolo created T170319: End of August milestone: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure..
Tue, Jul 11, 8:07 PM · Services (later), User-Joe, User-Elukey, Goal, Epic
Pchelolo updated the task description for T170296: End of July milestone: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure..
Tue, Jul 11, 8:03 PM · Goal, Epic, Services (doing)
Pchelolo added a comment to T169930: MCS requests content from RESTBase using non-normalized titles.

I believe the issue is really that it uses normalized titles when it should be using db titles (getPrefixedDBKey) instead. (Underscores instead of spaces)

Tue, Jul 11, 8:00 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Services (watching), Mobile-Content-Service
Pchelolo updated subscribers of T163380: Support posting Jobs to EventBus simultaneously with normal job processing.

I've heavily edited the task description to incorporate input from @GWicke and @mobrovac

Tue, Jul 11, 7:24 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, User-mobrovac, Analytics, ChangeProp, Services (later), EventBus
Pchelolo renamed T163380: Support posting Jobs to EventBus simultaneously with normal job processing from Create a hook when job is posted to JobQueue to Support posting Jobs to EventBus simultaneously with normal job processing.
Tue, Jul 11, 7:22 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, User-mobrovac, Analytics, ChangeProp, Services (later), EventBus
Pchelolo added a comment to T163380: Support posting Jobs to EventBus simultaneously with normal job processing.

IIRC the other option you looked into before was to create a "wrapper" JobQueue class that delegates to Redis *and* the new backend class instances?

Tue, Jul 11, 5:32 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, User-mobrovac, Analytics, ChangeProp, Services (later), EventBus
Pchelolo renamed T163380: Support posting Jobs to EventBus simultaneously with normal job processing from Support posting to multiple JobQueues simultaneously to Create a hook when job is posted to JobQueue.
Tue, Jul 11, 5:23 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, User-mobrovac, Analytics, ChangeProp, Services (later), EventBus
Pchelolo renamed T170296: End of July milestone: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure. from End of July milestone to End of July milestone: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure..
Tue, Jul 11, 5:13 PM · Goal, Epic, Services (doing)
Pchelolo created T170296: End of July milestone: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure..
Tue, Jul 11, 5:12 PM · Goal, Epic, Services (doing)
Pchelolo added a subtask for T169937: Services Q1 2017/18 goal: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure.: T157088: [EPIC] Develop a JobQueue backend based on EventBus.
Tue, Jul 11, 5:10 PM · Operations, User-Joe, User-Elukey, Goal, Epic, Services (doing)
Pchelolo added a parent task for T157088: [EPIC] Develop a JobQueue backend based on EventBus: T169937: Services Q1 2017/18 goal: Begin migrating job queue processing to multi-DC enabled eventbus infrastructure..
Tue, Jul 11, 5:10 PM · Epic, Services (doing), User-mobrovac, Analytics, ChangeProp, EventBus

Mon, Jul 10

Pchelolo added a comment to T170145: Add parsedcomment to recentchange stream.

Hm, not as trivial as I thought. In EventBus we rely on the core RecentChange behavior, using the default formatter and sending it to the EventBus. WE could generically add parsedcomment to the core RecentChange, but I don't think that's a good idea - inside MediaWiki the comment could be easily parsed if needed. WE could add some custom logic in the formatter, which I suppose is a better way to go.

Mon, Jul 10, 9:10 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, Services (watching), EventBus, Analytics, Wikimedia-Stream
Pchelolo added a comment to T170145: Add parsedcomment to recentchange stream.

It seems so from the code Lemme try it in vagrant

Mon, Jul 10, 8:46 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, Services (watching), EventBus, Analytics, Wikimedia-Stream
Pchelolo added a comment to T170145: Add parsedcomment to recentchange stream.

So, in my understanding, @Ottomata is -0.5, @mobrovac is also -0.5, I'm +0.5 and I guess @Nirmos who requested this is +1.0. Overall we have +0.5 for including. Did I summarize correctly?

Mon, Jul 10, 8:12 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, Services (watching), EventBus, Analytics, Wikimedia-Stream
Pchelolo added a comment to T170145: Add parsedcomment to recentchange stream.

I don't have a firm stand on this, but it seems like verbose information that is already present in the comment field. Should we then get rid of comment?

Mon, Jul 10, 8:05 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, Services (watching), EventBus, Analytics, Wikimedia-Stream
Pchelolo added a comment to T169998: RFC: Container path conventions.

We should also take into consideration the development cycle with mounted paths. In case of development, the code is over mounted on top of the sources from inside the container, and the entry point changes to install new node_modules (or update old already installed) and then start the service with the config, that's also over-mounted from the cloned repo. So we need something like /bin/reinstall_deps_and_start

Mon, Jul 10, 5:05 PM · Release-Engineering-Team (Watching / External), MediaWiki-Containers, User-mobrovac, Kubernetes, Services (designing)
Pchelolo added a comment to T170145: Add parsedcomment to recentchange stream.

We don't include any HTML-formatted content in our events, so I'm with you: I don't think we should include parsedcomment.

Mon, Jul 10, 4:57 PM · MW-1.30-release-notes (WMF-deploy-2017-07-25_(1.30.0-wmf.11)), Patch-For-Review, Services (watching), EventBus, Analytics, Wikimedia-Stream

Fri, Jul 7

Pchelolo added a comment to T158100: Deprecate and remove the public title/{title} endpoint.

Yes, the revision visibility change :) One can change the visibility of various parts of a revision, and without exposing the revision in any way in the REST API, I don't see how we can update that.

Fri, Jul 7, 3:48 PM · Services (doing), RESTBase-API, RESTBase

Thu, Jul 6

Pchelolo edited projects for T169911: RESTBase, Change-Prop and MobileApps got in a loop, added: Services (blocked); removed Services (next).

Setting this to blocked until we get more info when (if) this happens again

Thu, Jul 6, 10:50 PM · Services (blocked), ChangeProp, RESTBase
Pchelolo added a comment to T169911: RESTBase, Change-Prop and MobileApps got in a loop.

We've enabled automatic blacklisting for all the derived content updates, so even if this issue happens again it won't kill RESTBase, so for now I'll stop investigating the root cause as we clearly don't have enough information

Thu, Jul 6, 10:00 PM · Services (blocked), ChangeProp, RESTBase
Pchelolo committed rMSCD61a39f9d1491: Config: Extend automatic blacklisting to derived content updates. (authored by Pchelolo).
Config: Extend automatic blacklisting to derived content updates.
Thu, Jul 6, 9:36 PM
Pchelolo created T169930: MCS requests content from RESTBase using non-normalized titles.
Thu, Jul 6, 9:29 PM · Patch-For-Review, Reading-Infrastructure-Team-Backlog (Kanban), Services (watching), Mobile-Content-Service
Pchelolo created T169929: iOS app requests summaries with non-normalized titles.
Thu, Jul 6, 9:25 PM · iOS-app-v5.6.0-Goat-On-A-Train, Services (watching), Wikipedia-iOS-App-Backlog, iOS-app-Bugs
Pchelolo committed rMSCDf9eb6fa94e87: Config: Temporarily block ceb.wikipedia.org (authored by Pchelolo).
Config: Temporarily block ceb.wikipedia.org
Thu, Jul 6, 6:43 PM
Pchelolo added a comment to T169912: Propagating changes doesn't work in beta cluster.

The root cause identified - somehow ChangeProp config in beta cluster got pointed to the production RESTBase instance. The patch above fixes that.

Thu, Jul 6, 6:20 PM · ChangeProp, Services (done), RESTBase
Pchelolo added a comment to T168941: Remove spacing between list items.

@Jdlrobson created T169912 to track not working beta cluster as it seems unrelated to this. Will have a look today.

Thu, Jul 6, 5:38 PM · MW-1.30-release-notes (WMF-deploy-2017-07-11_(1.30.0-wmf.9)), Reading-Web-Kanban-Board, Page-Previews, Reading-Web-Backlog
Pchelolo created T169912: Propagating changes doesn't work in beta cluster.
Thu, Jul 6, 5:37 PM · ChangeProp, Services (done), RESTBase
Pchelolo created T169911: RESTBase, Change-Prop and MobileApps got in a loop.
Thu, Jul 6, 5:32 PM · Services (blocked), ChangeProp, RESTBase
Pchelolo added a project to T169898: Managing size of page-create and revision-create tables in storage. Aggregation? : Services (watching).
Thu, Jul 6, 4:26 PM · Patch-For-Review, Services (watching), Analytics-Kanban, MW-1.30-release-notes (WMF-deploy-2017-06-27_(1.30.0-wmf.7)), EventBus, Wikimedia-Hackathon-2017, Analytics, Easy, Contributors-Analysis, Analytics-EventLogging
Pchelolo added a comment to T164291: Make title-related properties consistent.

This is true. I'd hope that any RESTful service running on a non-English Wikipedia to accept Talk:Foo and respond with 303 See Other [0] with the appropriate Location HTTP header. The client would then request the resource pointed to by that header.

Thu, Jul 6, 3:57 PM · Reading-Infrastructure-Team-Backlog, RESTBase-API, Services (designing)

Fri, Jun 30

Pchelolo committed rMSCDbd89eae0bd25: Config: Only update restbase if the revision content changed. (authored by Pchelolo).
Config: Only update restbase if the revision content changed.
Fri, Jun 30, 6:48 PM

Thu, Jun 29

Pchelolo closed T133221: Support red links updates in change-propagation, a subtask of T39902: RFC: Implement rendering of redlinks in Parsoid HTML as post-processor, as Resolved.
Thu, Jun 29, 8:48 PM · Patch-For-Review, Services (blocked), RfC, Services-next, User-mobrovac, Parsing-Team, ArchCom-RfC, Parsoid
Pchelolo closed T133221: Support red links updates in change-propagation as Resolved.

Deployed and confirmed to work. Resolving the ticket.

Thu, Jun 29, 8:48 PM · Services (done), ChangeProp
Pchelolo committed rMSCD350076cf51be: Config: Support red links in ChangeProp (authored by Pchelolo).
Config: Support red links in ChangeProp
Thu, Jun 29, 7:56 PM

Wed, Jun 28

Pchelolo added a project to T169141: Scap3 considers service restarts to be deploys: Services (watching).
Wed, Jun 28, 10:02 PM · Release-Engineering-Team (Kanban), Services (watching), Scap
Pchelolo created T169140: Enhance the if-modified-since support.
Wed, Jun 28, 9:20 PM · ChangeProp, RESTBase, Services (designing)
Pchelolo added a comment to T146944: Mobile-sections replacement.

We can add support for that server-side if needed.

Any updates on this?
This endpoint came up as a possible solution to T168625. We are basically recreating the logic of the intro field in the MCS endpoint...

Wed, Jun 28, 6:51 PM · Reading-Infrastructure-Team-Backlog, Reading Epics (Platform JS CSS and HTML consolidation), User-mobrovac, RESTBase-API, Services (watching), Wikipedia-Android-App-Backlog, Reading-Web-Backlog, Mobile-Content-Service

Jun 27 2017

Pchelolo added a member for MediaWiki-Containers: Pchelolo.
Jun 27 2017, 11:20 PM
Pchelolo added a comment to T164006: Suggestion: API for fetching lint errors for a specific revision.

@mobrovac I've proposed that option to match the Parsoid API, but providing a GET endpoint works too.

Jun 27 2017, 6:40 PM · Services (blocked), RESTBase-API, Parsing-Team, MediaWiki-extensions-Linter
Pchelolo added a comment to T164006: Suggestion: API for fetching lint errors for a specific revision.

Filed a subtask for Parsoid to figure out correct redirects in case only the title is provided but not the revision. This is blocked on it for the time being.

Jun 27 2017, 6:18 PM · Services (blocked), RESTBase-API, Parsing-Team, MediaWiki-extensions-Linter
Pchelolo created T169006: Correctly redirect in Parsoid /transform/wikitext/to/lint endpoint.
Jun 27 2017, 6:17 PM · Services (watching), Parsing-Team, MediaWiki-extensions-Linter
Pchelolo edited projects for T164006: Suggestion: API for fetching lint errors for a specific revision, added: Services (doing); removed Services.

Parsoid accepts just title/revision in it's wikitext/to/lint API, so all we need to do it to make wikitext an optional parameter as well, and check that either wikitext or a title is provided. Easy.

Jun 27 2017, 5:50 PM · Services (blocked), RESTBase-API, Parsing-Team, MediaWiki-extensions-Linter
Pchelolo removed projects from T133221: Support red links updates in change-propagation: Services (doing), Patch-For-Review, RESTBase.

The task descriptionis brutally outdated, we've implemented a very different approach to dependent pages updates. However, this still needs to be supported an PR for it is awaiting mediawiki train deployment: https://github.com/wikimedia/change-propagation/pull/191

Jun 27 2017, 5:30 PM · Services (done), ChangeProp
Pchelolo updated the task description for T168922: Refactor Change-Prop to use filters.
Jun 27 2017, 12:44 AM · ChangeProp, Services (designing)
Pchelolo created T168922: Refactor Change-Prop to use filters.
Jun 27 2017, 12:42 AM · ChangeProp, Services (designing)

Jun 23 2017

Pchelolo committed rMSCDffabd1312c53: Revert "Temporary disable ORES rules" (authored by Pchelolo).
Revert "Temporary disable ORES rules"
Jun 23 2017, 7:38 PM