Page MenuHomePhabricator

bd808 (Bryan Davis)
Principal Software Engineer, Technical EngagementAdministrator

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
Oct 3 2014, 2:36 PM (363 w, 4 d)
Roles
Administrator
Availability
Available
IRC Nick
bd808
LDAP User
BryanDavis
MediaWiki User
BDavis (WMF) [ Global Accounts ]

I'm BDavis (WMF) on wiki, bd808 on irc, and BryanDavis on Gerrit and Wikitech.

I've got a thing for ūü¶Ąs. Don't judge.

I work for or provide services to the Wikimedia Foundation, but this is my only Phabricator account. Edits, statements, or other contributions made from this account are my own, and may not reflect the views of the Foundation.

Recent Activity

Yesterday

bd808 added a comment to T291447: Toolhub container in staging clsuter cannot reach meta.wikimedia.org to complete OAuth handshake.

Ok. I went the long way around to test this as can be seen by the pile of patches that it took. Ultimately trying to complete the handshake fails in the same way which I think means that @Majavah was correct in T291447#7369589 that the url-downloader proxy will not handle this use case.

Tue, Sep 21, 11:13 PM · Toolhub
bd808 closed T291442: docker-registry.wikimedia.org/prometheus-mcrouter-exporter:latest (0.0.1-2-20210919) failing to start in Toolhub deployment to staging cluster as Resolved.

The immediate problem is fixed. See T291530: Get mcrouter & prometheus-mcrouter-exporter tags for helmfile.d from upstream config for a related follow up task.

Tue, Sep 21, 10:17 PM · User-bd808, Toolhub
bd808 created T291530: Get mcrouter & prometheus-mcrouter-exporter tags for helmfile.d from upstream config.
Tue, Sep 21, 10:16 PM · serviceops, Toolhub
bd808 added a comment to T291447: Toolhub container in staging clsuter cannot reach meta.wikimedia.org to complete OAuth handshake.

url-downloader blocks access to internal networks,

Tue, Sep 21, 4:53 PM · Toolhub
bd808 added a comment to T291447: Toolhub container in staging clsuter cannot reach meta.wikimedia.org to complete OAuth handshake.

Social-core delegates HTTP connection handling to requests. Requests in turn delegates proxy configuration from envars to urllib.request.getproxies() which does the envvar checks. It is also possible to explicitly configure social-core to pass proxy configuration to requests via the SOCIAL_AUTH_PROXIES Django setting. That setting seems to only be documented in the code and not in any of the user docs that I can find. The Django setting should be a dict matching the requests proxies kwarg format. Something like:

SOCIAL_AUTH_PROXIES = {
    "http": "http://url-downloader.eqiad.wikimedia.org:8080"
}

Introduced in social-core v3.4.0 by https://github.com/python-social-auth/social-core/commit/25ed3b6242e89f644b3d4a4d235496905b4bc9c1

Tue, Sep 21, 4:34 PM · Toolhub
bd808 added a comment to T291447: Toolhub container in staging clsuter cannot reach meta.wikimedia.org to complete OAuth handshake.

https://github.com/python-social-auth/social-core/issues/146 reports the normal HTTP_PROXY envvar (https://docs.python-requests.org/en/master/user/advanced/#proxies) not working for some from a venv runtime. More investigation needed. Would be easier to carry out experiments with T290357: Maintenance environment needed for running one-off commands, but I will try to find other means in the near term.

Tue, Sep 21, 3:44 PM · Toolhub

Mon, Sep 20

bd808 created T291447: Toolhub container in staging clsuter cannot reach meta.wikimedia.org to complete OAuth handshake.
Mon, Sep 20, 11:53 PM · Toolhub
bd808 claimed T291442: docker-registry.wikimedia.org/prometheus-mcrouter-exporter:latest (0.0.1-2-20210919) failing to start in Toolhub deployment to staging cluster.

Changing the pull policy fixed the deployment. The open question though is if this will cause more harm than good in the longer term. There are three ideas in tension here:

  1. deployments should use the latest working version of a sidecar image
  2. deployments should protect themselves from unexpected external changes
  3. deployments should be as little work as possible
Mon, Sep 20, 11:44 PM · User-bd808, Toolhub
bd808 updated subscribers of T291442: docker-registry.wikimedia.org/prometheus-mcrouter-exporter:latest (0.0.1-2-20210919) failing to start in Toolhub deployment to staging cluster.

Chat with @Legoktm while working on this:

[22:37]  <  legoktm> hmm, that was supposed to be fixed in I90cc6babab81790711e43d71850d2191dd7de25c
[22:38]  <    bd808> yeah... `docker image inspect` seems to show "User": "65534" when I pull locally
[22:38]  <    bd808> I wonder if the chart is not re-pulling?
[22:39] bd808 digs around some more
[22:39]  <  legoktm> it's pullIfNotPresent, so :latest probably has an older copy
[22:39]  <  legoktm> what if you try :0.0.1-2-20210919 ?
[22:42]  <  legoktm> 1001 has docker-registry.discovery.wmnet/prometheus-mcrouter-exporter                         latest                              1957b8c51410        10 months ago       80.5MB                                                                       
[22:43]  <  legoktm> 1002 has docker-registry.wikimedia.org/prometheus-mcrouter-exporter                           latest                         c1cf680608ca        3 weeks ago         79.7MB
Mon, Sep 20, 10:48 PM · User-bd808, Toolhub
bd808 created T291442: docker-registry.wikimedia.org/prometheus-mcrouter-exporter:latest (0.0.1-2-20210919) failing to start in Toolhub deployment to staging cluster.
Mon, Sep 20, 10:43 PM · User-bd808, Toolhub

Sat, Sep 18

bd808 placed T254700: Citoid requests for YouTube metadata is giving 429: too many requests HTTP error up for grabs.
Sat, Sep 18, 12:18 AM · Citoid

Fri, Sep 17

bd808 moved T288832: Content license for user supplied Toolhub data from In Progress to Review on the Toolhub board.
Fri, Sep 17, 7:13 PM · WMF-Legal, Developer-Advocacy (Jul-Sep 2021), Toolhub

Thu, Sep 16

bd808 added a comment to T276700: Content localization infrastructure setup (export, import, deploy).

Python lib for Markdown translation via PO files: https://mdpo.readthedocs.io/en/master/index.html

Thu, Sep 16, 10:14 PM · I18n, Wikimedia-Developer-Portal
bd808 added a comment to T276708: Define storage format for listed entries (JSON / YAML / Markdown etc) and their placement.

A page is the third level of content organization. Each page will be in a format prescribed by the static site generation software that is chosen for T287175: Decide on most suitable underlying technical platform. It is likely that these will take the form of either YAML documents or Markdown documents with a YAML preamble as appropriate for the tooling. A page should have a title, heading(s), and prose text as well as include one or more categories which in turn include one or more document descriptors. We want the inclusion of categories to work like a template transclusion in MediaWiki: somehow in the page's source document we want to say "put the code quality category content here". How that technically works is :waves hands: an implementation detail that we do not know yet, but we will figure it out! The static site framework we choose will either have native support for transclusions, or we will make our own implementation as either a native extension of the framework or as a pre-processor step of some kind.

Thu, Sep 16, 9:55 PM · Wikimedia-Developer-Portal
bd808 added a comment to T276708: Define storage format for listed entries (JSON / YAML / Markdown etc) and their placement.

@Aklapper and I talked through this ticket in a conference call today. We got a bit into technical weeds and then back out to a higher level again, but I think we may be closing in on a reasonable initial data model. I will try to describe it here a bit for others to comment on. Note there are terms being defined to make this description. The actual words used in the longer term may not be the same, but the concepts they describe feel fairly stable.

Thu, Sep 16, 9:39 PM · Wikimedia-Developer-Portal
bd808 awarded T167973: Move database for wikitech (labswiki) to a main cluster section a Party Time token.
Thu, Sep 16, 3:55 PM · Patch-For-Review, wikitech.wikimedia.org, DBA

Wed, Sep 15

bd808 added a comment to T271480: Setup production database for Toolhub.

@bd808 were you able to find any timeline for this? thanks!

Wed, Sep 15, 4:59 AM · database-backups, serviceops, DBA, Toolhub

Tue, Sep 14

mmodell awarded T273020: Security Readiness Review For Toolhub a Love token.
Tue, Sep 14, 7:16 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 closed T273020: Security Readiness Review For Toolhub as Resolved.

@bd808 - Do you mind if we resolve this task given the long-running sub-tasks still attached? I don't mind keeping this task open for those, but if it's not a big deal, it'd be nice to resolve this task and move it on our board while keeping the sub-tasks open. Thanks.

Tue, Sep 14, 5:32 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 closed T273020: Security Readiness Review For Toolhub, a subtask of T271483: Prepare for production deployment of Toolhub, as Resolved.
Tue, Sep 14, 5:31 PM · Developer-Advocacy (Jul-Sep 2021), Goal, Epic, Toolhub

Mon, Sep 13

bd808 added a comment to T290357: Maintenance environment needed for running one-off commands.

That being said, we don't have yet implemented this functionality and we 'd like to see how to best implement it. kubectl exec is a valid solution, but once one ends up in one of our containers and tries to actually debug something, it turns to pain.

Mon, Sep 13, 8:15 PM · Prod-Kubernetes, Kubernetes, serviceops, Toolhub
bd808 added a comment to T183758: Create backups of user tables from decommissioned database servers.

@bd808 Hi, any update on purging them? :-)

Mon, Sep 13, 4:07 PM · cloud-services-team (Kanban), Data-Services, DBA

Fri, Sep 3

bd808 closed T290341: Toolhub container missing memcached Python lib needed for production as Resolved.
Fri, Sep 3, 10:24 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 triaged T275971: The Toolhub message ", \" probably doesn't need translation as Low priority.
Fri, Sep 3, 9:29 PM · Toolhub
bd808 moved T290357: Maintenance environment needed for running one-off commands from Backlog to Research needed on the Toolhub board.
Fri, Sep 3, 9:26 PM · Prod-Kubernetes, Kubernetes, serviceops, Toolhub
bd808 triaged T290357: Maintenance environment needed for running one-off commands as High priority.
Fri, Sep 3, 9:26 PM · Prod-Kubernetes, Kubernetes, serviceops, Toolhub
bd808 updated subscribers of T290357: Maintenance environment needed for running one-off commands.

@Legoktm has kindly offered to bring up this desire in a future serviceops team meeting.

Fri, Sep 3, 9:26 PM · Prod-Kubernetes, Kubernetes, serviceops, Toolhub
bd808 created T290357: Maintenance environment needed for running one-off commands.
Fri, Sep 3, 9:24 PM · Prod-Kubernetes, Kubernetes, serviceops, Toolhub
bd808 added a comment to T273020: Security Readiness Review For Toolhub.

@sbassett would you like to re-check any of the things that I have attempted to fix before Toolhub is deployed into production?

I can, sure. I assume HEAD on master is fine to look at?

Fri, Sep 3, 7:17 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 added a comment to T273020: Security Readiness Review For Toolhub.

@sbassett would you like to re-check any of the things that I have attempted to fix before Toolhub is deployed into production? I have just done my first deployment into the staging cluster and hope to move to production in the week of 2021-09-13 pending resolution of T288832: Content license for user supplied Toolhub data.

Fri, Sep 3, 5:32 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 triaged T286164: Improve Toolhub's UX as Medium priority.
Fri, Sep 3, 5:31 PM · Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 moved T288832: Content license for user supplied Toolhub data from Research needed to In Progress on the Toolhub board.
Fri, Sep 3, 5:23 PM · WMF-Legal, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 claimed T290341: Toolhub container missing memcached Python lib needed for production.
Fri, Sep 3, 4:57 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 closed T290283: `helmfile -e staging -i apply` fails for Toolhub due to missing ConfigMap, a subtask of T280881: New Service Request Toolhub, as Resolved.
Fri, Sep 3, 4:57 PM · Patch-For-Review, serviceops, Toolhub, Service-deployment-requests, Services, SRE
bd808 closed T290283: `helmfile -e staging -i apply` fails for Toolhub due to missing ConfigMap as Resolved.

Thank you very much for figuring out what was missing in my chart @Legoktm!

Fri, Sep 3, 4:57 PM · User-bd808, Toolhub
bd808 created T290341: Toolhub container missing memcached Python lib needed for production.
Fri, Sep 3, 4:56 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 awarded T290130: Incident response tools operational readiness review a Love token.
Fri, Sep 3, 2:34 PM · SRE, wikitech.wikimedia.org, Sustainability (Incident Followup)

Thu, Sep 2

bd808 added a comment to T290283: `helmfile -e staging -i apply` fails for Toolhub due to missing ConfigMap.

Attempting to verify that the ConfigMap is well formed produces an interesting error message:

$ kubectl apply --validate=true -f /home/bd808/T290283-configmap.yaml
Error from server (Forbidden): error when creating "/home/bd808/T290283-configmap.yaml": configmaps is forbidden: User "toolhub" cannot create resource "configmaps" in API group "" in the namespace "toolhub"
Thu, Sep 2, 10:22 PM · User-bd808, Toolhub
bd808 claimed T290283: `helmfile -e staging -i apply` fails for Toolhub due to missing ConfigMap.
Thu, Sep 2, 10:13 PM · User-bd808, Toolhub
bd808 created T290283: `helmfile -e staging -i apply` fails for Toolhub due to missing ConfigMap.
Thu, Sep 2, 10:13 PM · User-bd808, Toolhub

Mon, Aug 30

bd808 added a project to T288539: Add `force_escape` filter to Django template translation content: Developer-Advocacy (Jul-Sep 2021).
Mon, Aug 30, 10:03 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 added a project to T276374: Figure out what production logging config needs to look like: Developer-Advocacy (Jul-Sep 2021).
Mon, Aug 30, 10:03 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 added a project to T286849: If the logo + title combined with the description is big enough it clips through the author list: Developer-Advocacy (Jul-Sep 2021).
Mon, Aug 30, 10:03 PM · Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 added a project to T289508: Link commons images to commons file page for attribution: Developer-Advocacy (Jul-Sep 2021).
Mon, Aug 30, 10:03 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 closed T276405: Setup periodic crawling, a subtask of T271483: Prepare for production deployment of Toolhub, as Resolved.
Mon, Aug 30, 10:00 PM · Developer-Advocacy (Jul-Sep 2021), Goal, Epic, Toolhub
bd808 closed T276405: Setup periodic crawling as Resolved.
Mon, Aug 30, 10:00 PM · User-bd808, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 closed T289508: Link commons images to commons file page for attribution as Resolved.
Mon, Aug 30, 10:00 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 closed T276374: Figure out what production logging config needs to look like, a subtask of T271483: Prepare for production deployment of Toolhub, as Resolved.
Mon, Aug 30, 10:00 PM · Developer-Advocacy (Jul-Sep 2021), Goal, Epic, Toolhub
bd808 closed T276374: Figure out what production logging config needs to look like as Resolved.
Mon, Aug 30, 10:00 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 moved T280881: New Service Request Toolhub from Radar to Review on the Toolhub board.
Mon, Aug 30, 9:59 PM · Patch-For-Review, serviceops, Toolhub, Service-deployment-requests, Services, SRE

Thu, Aug 26

bd808 added a comment to P17089 Quarry query runs per day (2021-01-01 through 2021-08-26).
MariaDB [quarry]> select date_format(timestamp, '%Y'), count(*) from query_run group by YEAR(timestamp) order by timestamp desc;
+------------------------------+----------+
| date_format(timestamp, '%Y') | count(*) |
+------------------------------+----------+
| 2021                         |    59394 |
| 2020                         |    96501 |
| 2019                         |   104460 |
| 2018                         |    92115 |
| 2017                         |    85557 |
| 2016                         |    84788 |
| 2015                         |    50245 |
| 2014                         |     5053 |
+------------------------------+----------+
8 rows in set (0.38 sec)
Thu, Aug 26, 9:22 PM · Quarry
bd808 added a comment to P17089 Quarry query runs per day (2021-01-01 through 2021-08-26).
MariaDB [quarry]> select date_format(timestamp, '%Y-%m'), count(*) from query_run group by YEAR(timestamp), month(timestamp) order by timestamp desc;
+---------------------------------+----------+
| date_format(timestamp, '%Y-%m') | count(*) |
+---------------------------------+----------+
| 2021-08                         |     5586 |
| 2021-07                         |     7897 |
| 2021-06                         |     7944 |
| 2021-05                         |     7749 |
| 2021-04                         |     6691 |
| 2021-03                         |     7638 |
| 2021-02                         |     7426 |
| 2021-01                         |     8463 |
| 2020-12                         |     5385 |
| 2020-11                         |     6378 |
| 2020-10                         |     7555 |
| 2020-09                         |     5717 |
| 2020-08                         |     6129 |
| 2020-07                         |     7596 |
| 2020-06                         |     8584 |
| 2020-05                         |     7178 |
| 2020-04                         |    17728 |
| 2020-03                         |     8288 |
| 2020-02                         |     8073 |
| 2020-01                         |     7890 |
| 2019-12                         |     4680 |
| 2019-11                         |     5949 |
| 2019-10                         |     6573 |
| 2019-09                         |     5766 |
| 2019-08                         |     5602 |
| 2019-07                         |     6138 |
| 2019-06                         |     8696 |
| 2019-05                         |     9998 |
| 2019-04                         |    11084 |
| 2019-03                         |    13917 |
| 2019-02                         |    15018 |
| 2019-01                         |    11039 |
| 2018-12                         |     8044 |
| 2018-11                         |     7832 |
| 2018-10                         |     7568 |
| 2018-09                         |     6682 |
| 2018-08                         |     7568 |
| 2018-07                         |     7877 |
| 2018-06                         |     7102 |
| 2018-05                         |     5946 |
| 2018-04                         |     9781 |
| 2018-03                         |     8879 |
| 2018-02                         |     7769 |
| 2018-01                         |     7067 |
| 2017-12                         |     6217 |
| 2017-11                         |     5987 |
| 2017-10                         |     6933 |
| 2017-09                         |     6188 |
| 2017-08                         |     6238 |
| 2017-07                         |     5804 |
| 2017-06                         |     7505 |
| 2017-05                         |     8349 |
| 2017-04                         |     6256 |
| 2017-03                         |     7977 |
| 2017-02                         |    10187 |
| 2017-01                         |     7916 |
| 2016-12                         |     9299 |
| 2016-11                         |     8042 |
| 2016-10                         |     6018 |
| 2016-09                         |     6022 |
| 2016-08                         |     6253 |
| 2016-07                         |     7153 |
| 2016-06                         |     5747 |
| 2016-05                         |     8049 |
| 2016-04                         |     6747 |
| 2016-03                         |     9087 |
| 2016-02                         |     6890 |
| 2016-01                         |     5481 |
| 2015-12                         |     4236 |
| 2015-11                         |     4510 |
| 2015-10                         |     5209 |
| 2015-09                         |     4165 |
| 2015-08                         |     5744 |
| 2015-07                         |     3862 |
| 2015-06                         |     3261 |
| 2015-05                         |     3722 |
| 2015-04                         |     4324 |
| 2015-03                         |     4591 |
| 2015-02                         |     3865 |
| 2015-01                         |     2756 |
| 2014-12                         |     1156 |
| 2014-11                         |      968 |
| 2014-10                         |     1159 |
| 2014-09                         |      691 |
| 2014-08                         |     1079 |
+---------------------------------+----------+
85 rows in set (0.63 sec)
Thu, Aug 26, 9:21 PM · Quarry
bd808 added a project to P17089 Quarry query runs per day (2021-01-01 through 2021-08-26): Quarry.
Thu, Aug 26, 9:18 PM · Quarry
bd808 created P17089 Quarry query runs per day (2021-01-01 through 2021-08-26).
Thu, Aug 26, 9:17 PM · Quarry
bd808 closed T287716: Create helm chart for kubernetes deployment as Resolved.
Thu, Aug 26, 8:44 PM · Developer-Advocacy (Jul-Sep 2021), Toolhub, User-bd808
bd808 closed T287716: Create helm chart for kubernetes deployment, a subtask of T271483: Prepare for production deployment of Toolhub, as Resolved.
Thu, Aug 26, 8:44 PM · Developer-Advocacy (Jul-Sep 2021), Goal, Epic, Toolhub
bd808 added a comment to T289789: VPS instance for Scholia production env.

@Thewillyhuman https://wikitech.wikimedia.org/wiki/Help:Adding_Disk_Space_to_Cloud_VPS_instances gives instructions on how to add new cinder volumes to an instance. We have phased out having base images with different local storage sizes in favor of this more flexible approach.

Thu, Aug 26, 4:49 PM · Cloud-VPS (Quota-requests)

Wed, Aug 25

bd808 added a comment to T289658: Audit puppet usage in cloud hosts.

following on from brokes script i create a small python script to give us a list of used classes

Wed, Aug 25, 3:03 PM · User-jbond, Infrastructure-Foundations, Puppet, Cloud-VPS, cloud-services-team (Kanban)
bd808 awarded T289639: Document how k8s logging works a Like token.
Wed, Aug 25, 12:28 AM · Kubernetes, serviceops
bd808 claimed T276374: Figure out what production logging config needs to look like.
Wed, Aug 25, 12:21 AM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 updated subscribers of T276374: Figure out what production logging config needs to look like.

I was wondering what needed to be done for log shipping, but @Legoktm pointed me to T207200: Revisit the logging work done on Q1 2017-2018 for the standard pod setup which seems to indicate that the stderr logging will be picked up on the exec nodes and shipped to the ELK stack for me by some fancy rsyslog magic.

Wed, Aug 25, 12:20 AM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub

Tue, Aug 24

bd808 added a comment to T273020: Security Readiness Review For Toolhub.

Comments on the warnings:

  • rule:security.semgrep-rules.generic.dockerfile.best-practice.missing-image-version -- linter complaint about not using an explict image tag and instead falling back to the default "latest" which makes repeatable builds difficult. Using exact version tags for all images would increase our ability to repeat and verify builds after the fact, but would also necessitate version bumps to be propagated downwards to all dependent builds when we rebuilt an upstream container to apply security patches. For now I am inclined to leave this alone.
Tue, Aug 24, 5:36 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 closed T289596: Consider adding `--no-install-recommends` to generated `apt-get install ...` commands as Declined.

As noted by @Legoktm, this is handled by global config for apt that is baked into our base images. This makes potentially makes Blubber a bit less reusable with arbitrary base images, but as long as it is a Wikimedia specific tool this is fine.

Tue, Aug 24, 4:59 PM · Release Pipeline (Blubber)
bd808 created T289596: Consider adding `--no-install-recommends` to generated `apt-get install ...` commands.
Tue, Aug 24, 4:39 PM · Release Pipeline (Blubber)

Mon, Aug 23

bd808 changed the status of T284352: Upgrade Toolhub ui container from nodejs10 to nodejs12, a subtask of T288551: Investigate replacing vue-cli with vite and webpack with rollup for Toolhub, from Stalled to Open.
Mon, Aug 23, 9:33 PM · Toolhub
bd808 changed the status of T284352: Upgrade Toolhub ui container from nodejs10 to nodejs12 from Stalled to Open.
Mon, Aug 23, 9:33 PM · Toolhub
bd808 removed a watcher for Wikimedia-Hackathon-2021: bd808.
Mon, Aug 23, 7:13 PM
bd808 closed T286874: "Authorized apps" shows empty list even when the raw API shows data as Resolved.
Mon, Aug 23, 5:17 PM · User-bd808, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 claimed T289508: Link commons images to commons file page for attribution.
Mon, Aug 23, 5:17 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 created T289508: Link commons images to commons file page for attribution.
Mon, Aug 23, 4:13 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub

Aug 22 2021

bd808 added a comment to T289365: video2commons stashfailed.

That error message is the api-error-publishfailed localization key. It is used in PublishStashedFileJob.php when any exception is caught while moving the stashed file into primary storage.

Aug 22 2021, 10:48 PM · video2commons

Aug 20 2021

bd808 claimed T288977: Provide resonable method for bots to authenticate to Toolhub API.
Aug 20 2021, 7:57 PM · User-bd808, Toolhub
bd808 awarded T289329: +2 in mediawiki/ for Majavah a Love token.
Aug 20 2021, 3:39 PM · MediaWiki-Gerrit-Group-Requests

Aug 19 2021

bd808 created T289313: Rollback DNS record creation on dynamicproxy registration failure.
Aug 19 2021, 9:42 PM · cloud-services-team (Kanban), Horizon
bd808 added a comment to T288962: Cannot create web proxy because of Duplicate RecordSet.

Here is the record that @Bstorm found shown in the Horizon UI:

Screen Shot 2021-08-19 at 10.56.57 AM.png (1√ó1 px, 253 KB)

Aug 19 2021, 5:07 PM · cloud-services-team (Kanban)

Aug 18 2021

bd808 added a comment to T288233: Completion index fails to build on labswiki: MessageCache.php: Process cache for 'en' should be set by now.

I'm not sure that we should call this ticket resolved though. The fix allows the index builds to complete, but all it really does is hide the error and return to how it was operating prior to wmf.15. The root problem to deal with is that the index rebuild currently runs on a machine that isn't able to run mediawiki against wikitech without errors.

Aug 18 2021, 9:11 PM · MW-1.37-notes (1.37.0-wmf.19; 2021-08-16), Discovery-Search (Current work), CirrusSearch, wikitech.wikimedia.org
bd808 added a comment to T288906: Remove CentralAuth SUL migration code.

cc @Legoktm in case you have any thoughts about this as you were involved in the original SUL migration and @bd808 in case you have thoughts about Wikitech

Aug 18 2021, 4:09 PM · MediaWiki-extensions-CentralAuth

Aug 17 2021

bd808 added a comment to T289097: CodeEditor extension should provide a Vue component of the Ace editor (so WikiLambda doesn't have to).

An editor component feels to me like something that would fit more with MediaWiki-extensions-CodeMirror than with SyntaxHighlight, but maybe I am missing a connection here?

Aug 17 2021, 10:55 PM · Editing-team, CodeEditor, Design-Systems-team-project, Abstract Wikipedia team

Aug 16 2021

bd808 closed T286121: Requesting contentadmin on Wikitech for Majavah as Resolved.

{{Done}}. Sorry that this slipped through the cracks for so long. I look forward to having your help managing the wiki!

Aug 16 2021, 11:47 PM · User-bd808, wikitech.wikimedia.org
bd808 moved T288547: Account recovery help needed for Wikimedia and Wikitech account Amitie_10g from Backlog to Authn/Authz on the wikitech.wikimedia.org board.
Aug 16 2021, 11:41 PM · wikitech.wikimedia.org, cloud-services-team (Kanban), Trust-and-Safety
bd808 added a comment to T288977: Provide resonable method for bots to authenticate to Toolhub API.

The built-in TokenAuthentication system of Django REST Framework might be the easiest solution to implement for this. I think we would only need to:

  • Add rest_framework.authtoken to INSTALLED_APPS
  • Add rest_framework.authentication.TokenAuthentication to REST_FRAMEWORK["DEFAULT_AUTHENTICATION_CLASSES"]
  • Expose self-service token management (create, revoke) via the Toolhub API
  • Expose token management API via the Toolhub UI. This feels like it could fit in the "Developer settings" screen as new tab.
Aug 16 2021, 5:41 PM · User-bd808, Toolhub
bd808 created T288977: Provide resonable method for bots to authenticate to Toolhub API.
Aug 16 2021, 5:13 PM · User-bd808, Toolhub

Aug 14 2021

bd808 added a comment to T283410: Show localised timestamp on wiki pages such as program pages.

I came back to this task today because of wanting local times displayed on the https://wikimania.wikimedia.org/wiki/2021:Program schedule. I added the <time/> wrapper to the [[Template:Zonestamp]] output there and also added class="zonestamp".

Aug 14 2021, 9:34 PM · Wikimedia-General-or-Unknown, Wikimedia-Hackathon-2021
bd808 closed T216485: Archive/delete tool macommune, a subtask of T133777: Tools that should get archived/deleted (tracking), as Declined.
Aug 14 2021, 3:16 PM · User-bd808, Projects-Cleanup, Tracking-Neverending, Toolforge
bd808 closed T216485: Archive/delete tool macommune as Declined.

Adopted by @Ash_Crow in T288890: Adoption request for macommune, so deletion is no longer needed.

Aug 14 2021, 3:16 PM · Toolforge (Tools to be deleted), Projects-Cleanup, Tools
bd808 closed T288890: Adoption request for macommune as Resolved.
Aug 14 2021, 3:11 PM · User-bd808, Toolforge-standards-committee
bd808 added a comment to T288890: Adoption request for macommune.

The $HOME for this tool has been cleaned already likely as part of filing T216485: Archive/delete tool macommune. The replica.my.cnf credentials are in place, but https://tool-db-usage.toolforge.org/owner/s52894 shows no tools-db stored data so this should be fine.

Aug 14 2021, 3:09 PM · User-bd808, Toolforge-standards-committee

Aug 13 2021

bd808 updated the task description for T286164: Improve Toolhub's UX.
Aug 13 2021, 8:24 PM · Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 added a comment to T288557: Ensure that production deployment includes a Strict-Transport-Security header for toolhub.wikimedia.org.

Looks like the header will be set by Varnish.

Aug 13 2021, 7:58 PM · Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 closed T288539: Add `force_escape` filter to Django template translation content as Resolved.
Aug 13 2021, 7:55 PM · Developer-Advocacy (Jul-Sep 2021), User-bd808, Toolhub
bd808 closed T288539: Add `force_escape` filter to Django template translation content, a subtask of T273020: Security Readiness Review For Toolhub, as Resolved.
Aug 13 2021, 7:55 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 closed T288536: Add timeouts to crawler calls to external URLs, a subtask of T273020: Security Readiness Review For Toolhub, as Resolved.
Aug 13 2021, 7:54 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 closed T288536: Add timeouts to crawler calls to external URLs as Resolved.
Aug 13 2021, 7:54 PM · User-bd808, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 closed T288542: Update nodejs libraries that fall within current semver constraints as Resolved.
Aug 13 2021, 7:54 PM · User-bd808, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 closed T288542: Update nodejs libraries that fall within current semver constraints, a subtask of T273020: Security Readiness Review For Toolhub, as Resolved.
Aug 13 2021, 7:54 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 closed T288532: Update Toolhub python dependencies with known security patches, a subtask of T273020: Security Readiness Review For Toolhub, as Resolved.
Aug 13 2021, 7:54 PM · user-sbassett, Toolhub, Security, secscrum, Security Readiness Reviews
bd808 closed T288532: Update Toolhub python dependencies with known security patches as Resolved.
Aug 13 2021, 7:54 PM · User-bd808, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 added a comment to T288832: Content license for user supplied Toolhub data.

Creative Commons Attribution-ShareAlike 4.0 International License - The most recent version of the CC-BY license family which is used for main namespace content on most (all?) Wikimedia wikis.

I don't know of any WMF wiki that uses CC BY-SA 4.0. Most use CC BY-SA 3.0, and some use others, such as CC BY 2.5.
CC BY-SA 4.0 is not considered compatible with CC BY-SA 3.0 by WMF Legal.

Aug 13 2021, 7:35 PM · WMF-Legal, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 added a comment to T288832: Content license for user supplied Toolhub data.

Happy to chime in, yes. Is there an example of the data we're talking about that I could look at?

Aug 13 2021, 7:29 PM · WMF-Legal, Developer-Advocacy (Jul-Sep 2021), Toolhub
bd808 added a project to T288832: Content license for user supplied Toolhub data: WMF-Legal.

I will check with WMF-Legal to see what policies and processes need to be followed in selecting a content license.

As the WMF-Legal project tag was added to this task, some general information to avoid wrong expectations:
Please note that public tasks in Wikimedia Phabricator are in general not a place where to expect feedback from the Legal Team of the Wikimedia Foundation due to the scope of the team and/or nature of legal topics. See the project tag description.
Please see https://meta.wikimedia.org/wiki/Legal for when and how to contact the Legal Team. Thanks!

Aug 13 2021, 3:21 PM · WMF-Legal, Developer-Advocacy (Jul-Sep 2021), Toolhub