Page MenuHomePhabricator
Create Task
Maniphest T334064

Migrate termbox to mw-api-int
Closed, ResolvedPublic
Actions

Description

Switch backend mw-api calls from baremetal api_appserver cluster to kubernetes hosted mw-api-int

Details

SubjectRepoBranchLines +/-
termbox: Switch to mw-api-int-async on k8soperations/deployment-chartsmaster+15 -6
termbox-test: call mw-api-intoperations/deployment-chartsmaster+13 -2
termbox: Migrate from staging-test to stagingoperations/deployment-chartsmaster+18 -31
InitialiseSettings.php: Change termbox url for testwikidatawikioperations/mediawiki-configmaster+1 -1
termbox: Switch to mw-api-int-async on k8soperations/deployment-chartsmaster+16 -5
Revert "Revert "termbox: Switch to mw-api-int-async on k8s""operations/deployment-chartsmaster+6 -6
termbox: Switch to mw-api-int-async on k8soperations/deployment-chartsmaster+4 -4
Customize query in gerrit

Related Objects
Search...

Event Timeline

Clement_Goubert triaged this task as Medium priority.Apr 5 2023, 9:05 AM
Clement_Goubert created this task.
Clement_Goubert moved this task from Incoming 🐫 to this.quarter 🍕 on the serviceops board.
Clement_Goubert mentioned this in T333120: Migrate internal traffic to k8s.
gerritbot added a comment.Apr 5 2023, 9:17 AM

Change 905944 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):

[operations/deployment-charts@master] termbox: Switch to mw-api-int-async on k8s

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

gerritbot added a project: Patch-For-Review.Apr 5 2023, 9:17 AM
Maintenance_bot added a project: Wikidata.Apr 5 2023, 9:29 AM
Restricted Application added a project: [DEPRECATED] wdwb-tech. · View Herald TranscriptApr 5 2023, 9:30 AM
gerritbot added a comment.Apr 5 2023, 10:10 AM

Change 905944 merged by jenkins-bot:

[operations/deployment-charts@master] termbox: Switch to mw-api-int-async on k8s

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

Maintenance_bot removed a project: Patch-For-Review.Apr 5 2023, 10:10 AM
Lucas_Werkmeister_WMDE subscribed.Apr 5 2023, 10:49 AM
Clement_Goubert added a comment.Apr 5 2023, 10:58 AM

Deploying to the staging-test environment yields errors
https://logstash.wikimedia.org/app/discover#/doc/logstash-*/logstash-k8s-1-7.0.0-1-2023.04.05?id=LGbuUIcBT1HDjeC69jil

Interestingly, the staging-test environment is not using mesh, contrary to the production deployment, and the "standard" staging deployment doesn't have an associated DNS.

Clement_Goubert added a comment.Apr 5 2023, 10:58 AM

Reverting the change until we can investigate those differences further.

gerritbot added a comment.Apr 5 2023, 11:16 AM

Change 905963 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):

[operations/deployment-charts@master] Revert "Revert "termbox: Switch to mw-api-int-async on k8s""

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

gerritbot added a project: Patch-For-Review.Apr 5 2023, 11:16 AM
gerritbot added a comment.Apr 5 2023, 1:52 PM

Change 905963 merged by jenkins-bot:

[operations/deployment-charts@master] Revert "Revert "termbox: Switch to mw-api-int-async on k8s""

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

Maintenance_bot removed a project: Patch-For-Review.Apr 5 2023, 2:11 PM
gerritbot added a comment.Apr 17 2023, 6:40 AM

Change 908955 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):

[operations/deployment-charts@master] termbox: Switch to mw-api-int-async on k8s

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

gerritbot added a project: Patch-For-Review.Apr 17 2023, 6:40 AM
gerritbot added a comment.May 2 2023, 8:59 AM

Change 908955 merged by jenkins-bot:

[operations/deployment-charts@master] termbox: Switch to mw-api-int-async on k8s

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

Maintenance_bot removed a project: Patch-For-Review.May 2 2023, 9:10 AM
gerritbot added a comment.May 2 2023, 9:47 AM

Change 914274 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):

[operations/mediawiki-config@master] InitialiseSettings.php: Change termbox url for testwikidatawiki

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

gerritbot added a project: Patch-For-Review.May 2 2023, 9:47 AM
gerritbot added a comment.May 2 2023, 9:51 AM

Change 914275 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):

[operations/deployment-charts@master] termbox: Migrate from staging-test to staging

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

Clement_Goubert added a comment.May 3 2023, 1:13 PM

There is a discussion to be had on whether we wish to merge these two environments (staging and staging-test) or keep the status quo. Holding the migration until we have had it.

Clement_Goubert changed the task status from Open to Stalled.May 3 2023, 1:13 PM
WMDE-leszek subscribed.Jul 3 2023, 11:02 AM
gerritbot added a comment.Jul 4 2023, 10:48 AM

Change 914275 abandoned by Clément Goubert:

[operations/deployment-charts@master] termbox: Migrate from staging-test to staging

Reason:

Keep current deployments.

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

gerritbot added a comment.Jul 4 2023, 11:13 AM

Change 935397 had a related patch set uploaded (by Clément Goubert; author: Clément Goubert):

[operations/deployment-charts@master] termbox: Switch to mw-api-int-async on k8s

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

gerritbot added a comment.Aug 21 2023, 8:10 AM

Change 951043 had a related patch set uploaded (by Giuseppe Lavagetto; author: Giuseppe Lavagetto):

[operations/deployment-charts@master] termbox-test: call mw-api-int

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

Clement_Goubert moved this task from this.quarter 🍕 to Doing 😎 on the serviceops board.Aug 21 2023, 2:34 PM
Clement_Goubert reassigned this task from Clement_Goubert to Joe.Aug 21 2023, 2:39 PM
gerritbot added a comment.Aug 23 2023, 10:50 AM

Change 951043 merged by jenkins-bot:

[operations/deployment-charts@master] termbox-test: call mw-api-int

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

Joe added a comment.Aug 23 2023, 11:01 AM

Just deployed the change to termbox-test, and I still see my test url

http://termbox-test.staging.svc.eqiad.wmnet:3031/termbox?entity=Q229877&revision=630197&language=en&editLink=%2Fwiki%2FSpecial%3ASetLabelDescriptionAliases%2FQ229877&preferredLanguages=en

return the same content after the redeployment. I'll proceed with the more general change to termbox that @Clement_Goubert prepared.

gerritbot added a comment.Aug 24 2023, 8:14 AM

Change 935397 merged by jenkins-bot:

[operations/deployment-charts@master] termbox: Switch to mw-api-int-async on k8s

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

Joe closed this task as Resolved.Aug 24 2023, 9:21 AM

Termbox has been migrated

Lucas_Werkmeister_WMDE added a comment.Aug 24 2023, 9:53 AM

\o/

Lucas_Werkmeister_WMDE added a comment.Aug 24 2023, 11:07 AM

Hm, doesn’t seem to be fully working :/ T344904: Termbox SSR broken on Test Wikidata (since k8s migration? unclear)

Lucas_Werkmeister_WMDE mentioned this in T355685: Migrate Termbox SSR from Node 16 to 18.Jan 23 2024, 4:08 PM
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