Page MenuHomePhabricator

dancy (Ahmon Dancy)
Staff Software Engineer, Release EngineeringAdministrator

Projects (7)

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Jun 27 2020, 12:14 AM (207 w, 3 d)
Roles
Administrator
Availability
Available
IRC Nick
dancy
LDAP User
Ahmon Dancy
MediaWiki User
ADancy (WMF) [ Global Accounts ]

Recent Activity

Today

dancy created T367922: Scap release pipeline failing with blubber/buildkit:experimental-native-llb.
Tue, Jun 18, 8:28 PM · Release Pipeline (Blubber)
dancy committed rMSCA1909b349cc63: Release 4.88.0-1.
Release 4.88.0-1
Tue, Jun 18, 6:50 PM

Fri, Jun 14

dancy changed the status of T159991: If aborting a scap due to test canary error rate, output some errors for reference from Open to In Progress.
Fri, Jun 14, 9:58 PM · Patch-For-Review, Release-Engineering-Team, Scap
dancy changed the status of T183999: Scap canary has a shifting baseline from Open to In Progress.
Fri, Jun 14, 9:58 PM · Patch-For-Review, Release-Engineering-Team (Seen), Scap
dancy changed the status of T367131: Did retrying canary checks do anything? from Open to In Progress.
Fri, Jun 14, 9:57 PM · Patch-For-Review, Release-Engineering-Team, Scap

Wed, Jun 12

dancy created T367352: Buildkit v0.14.0 released.
Wed, Jun 12, 7:08 PM · GitLab (CI & Job Runners), Release-Engineering-Team
dancy added a project to T367302: Confusing scap backport behavior on gate-and-submit failure: Release-Engineering-Team.
Wed, Jun 12, 6:55 PM · Release-Engineering-Team, Scap
dancy closed T360550: Gerrit 3.7.8: CI has completed checks. Reload the change view? RELOAD button doesn't work as Resolved.

I have tried it live (waiting for the result of a postmerge to complete) and clicking RELOAD fixed it. Additionally the new code is slightly easier/nicer since it does not rely on crawling the DOM to trigger the event from the ChangeView, but instead triggers it directly from document.

I am leaving the task open pending confirmation by another party 🎉

Wed, Jun 12, 4:08 PM · Gerrit (Gerrit 3.7)
dancy awarded T360550: Gerrit 3.7.8: CI has completed checks. Reload the change view? RELOAD button doesn't work a Party Time token.
Wed, Jun 12, 4:08 PM · Gerrit (Gerrit 3.7)
dancy committed rMSCA85b81eb8c488: deploy-local: Remove "symlink is current" check.
deploy-local: Remove "symlink is current" check
Wed, Jun 12, 3:01 PM

Tue, Jun 11

dancy committed rMSCA9356a86b2ba1: utils.py: write_file_if_needed, temp_to_permanent_file, get_umask.
utils.py: write_file_if_needed, temp_to_permanent_file, get_umask
Tue, Jun 11, 2:50 PM

Mon, Jun 10

dancy created T367128: PHP Deprecated: Use of QuickSurveys survey with link parameter was deprecated in MediaWiki 1.43. [Called from QuickSurveys\SurveyFactory::factoryExternal].
Mon, Jun 10, 10:00 PM · Patch-For-Review, QuickSurveys, Wikimedia-production-error
dancy reopened T360550: Gerrit 3.7.8: CI has completed checks. Reload the change view? RELOAD button doesn't work as "Open".
Mon, Jun 10, 9:27 PM · Gerrit (Gerrit 3.7)
dancy added a comment to T360550: Gerrit 3.7.8: CI has completed checks. Reload the change view? RELOAD button doesn't work.

I think that one is fixed now that we have upgraded to Gerrit 3.9.5 and RELOAD does reload the change view!

Hi @hashar. I wasn't able to confirm this for myself. I'm testing with https://gerrit.wikimedia.org/r/c/operations/puppet/+/1041230. I get the "CI has completed checks. Reload the change view? RELOAD/DISMISS" notice, I click RELOAD, and nothing happens.

Mon, Jun 10, 9:26 PM · Gerrit (Gerrit 3.7)
dancy added a comment to T360550: Gerrit 3.7.8: CI has completed checks. Reload the change view? RELOAD button doesn't work.

I think that one is fixed now that we have upgraded to Gerrit 3.9.5 and RELOAD does reload the change view!

Mon, Jun 10, 9:22 PM · Gerrit (Gerrit 3.7)
dancy closed T366856: `UNIQUE constraint failed: blob.name` during manifest upload to Reggie as Resolved.

Fixed in Reggie v1.12.0.

Mon, Jun 10, 7:48 PM · Release-Engineering-Team
dancy closed T366844: Don't just append names with "and" as Resolved.

Resolved as of scap 4.87.0 (deployed today).

Mon, Jun 10, 5:45 PM · Scap
dancy closed T366217: unable to create revert commit from scap as Resolved.

Resolved as of scap 4.87.0 (deployed today).

Mon, Jun 10, 5:42 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy added a comment to T366217: unable to create revert commit from scap.

Why does scap not use the same user here it uses to +2 any approved backports?

This is certainly a possibility and would require no additional setup for users. The only downside is that the name of the account (TrainBranchBot) wouldn't match its use for this case.

Mon, Jun 10, 5:41 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy closed T364309: deployment: fix-staging-perms fails to finish as Resolved.

fix-staging-perms has been fixed. The umask issue is resolved in scap 4.87.0 (deployed today).

Mon, Jun 10, 5:38 PM · Release-Engineering-Team, Scap
dancy committed rMSCAbfc9ded33e93: Release 4.87.0-1.
Release 4.87.0-1
Mon, Jun 10, 5:23 PM
dancy claimed T364309: deployment: fix-staging-perms fails to finish.
Mon, Jun 10, 3:08 PM · Release-Engineering-Team, Scap
dancy committed rMSCA77daa3218f46: deploy: set umask followup.
deploy: set umask followup
Mon, Jun 10, 3:04 PM
dancy committed rMSCA4409447124fd: Set umask at beginning of sync-*, backport, and deploy.
Set umask at beginning of sync-*, backport, and deploy
Mon, Jun 10, 2:41 PM

Fri, Jun 7

dancy committed rMSCA2a03cd3e4d30: pause_after_testserver_sync: Use commas to separate usernames.
pause_after_testserver_sync: Use commas to separate usernames
Fri, Jun 7, 10:26 PM
dancy committed rMSCA981a5dcc035d: test_backport.py: Revised setup_depends_on_included_in().
test_backport.py: Revised setup_depends_on_included_in()
Fri, Jun 7, 10:24 PM
dancy committed rMSCA90aa8c1237d4: Rename utils.move_symlink to utils.update_symlink.
Rename utils.move_symlink to utils.update_symlink
Fri, Jun 7, 9:23 PM
dancy committed rMSCAf82a6acbb014: scap deploy: Fail gracefully if run outside of a git repository.
scap deploy: Fail gracefully if run outside of a git repository
Fri, Jun 7, 9:14 PM
dancy triaged T342162: "scap deploy"'s config-deploy should check for broken symlinks as Low priority.
Fri, Jun 7, 8:58 PM · Release-Engineering-Team (Priority Backlog 📥), Patch-For-Review, Data-Platform-SRE, Scap
dancy committed rMSCA6ea20480ed0c: blubber.yaml: Update 'build-docs' target.
blubber.yaml: Update 'build-docs' target
Fri, Jun 7, 3:33 PM
dancy committed rMSCAaa68d28ba65c: blubber.yaml: Use blubber/buildkit v0.23.0.
blubber.yaml: Use blubber/buildkit v0.23.0
Fri, Jun 7, 3:23 PM
dancy closed T366810: scap fails deploying integration/docroot as Resolved.
Fri, Jun 7, 3:21 PM · Scap
dancy committed rMSCAf7c7c73507db: test_backport.py: Make "unusual submodule" tests more reliable.
test_backport.py: Make "unusual submodule" tests more reliable
Fri, Jun 7, 3:20 PM
dancy committed rMSCA38f3cc3bc324: git.py: Make is_dir() work for submodule directories (v2).
git.py: Make is_dir() work for submodule directories (v2)
Fri, Jun 7, 3:20 PM

Thu, Jun 6

dancy triaged T366856: `UNIQUE constraint failed: blob.name` during manifest upload to Reggie as Medium priority.
Thu, Jun 6, 10:54 PM · Release-Engineering-Team
dancy added a comment to T366856: `UNIQUE constraint failed: blob.name` during manifest upload to Reggie.

Thu, Jun 6, 10:50 PM · Release-Engineering-Team
dancy updated the name of F54984098: Reggie access log from "xxx" to "Reggie access log".
Thu, Jun 6, 10:49 PM
dancy created T366856: `UNIQUE constraint failed: blob.name` during manifest upload to Reggie.
Thu, Jun 6, 10:48 PM · Release-Engineering-Team
dancy updated subscribers of T366817: blubber NEWS is outdated.
Thu, Jun 6, 5:27 PM · Release-Engineering-Team, Release Pipeline (Blubber)
dancy committed rMSCAfbbbaf787090: backport --revert: Use gerrit_push_user to push revert commit to Gerrit.
backport --revert: Use gerrit_push_user to push revert commit to Gerrit
Thu, Jun 6, 5:18 PM
dancy committed rMSCAc6eab8b38f4c: git.py: set_env_vars_for_user: Only set env vars when needed.
git.py: set_env_vars_for_user: Only set env vars when needed
Thu, Jun 6, 5:18 PM
dancy committed rMSCA978c415201ca: Release 4.86.1-1.
Release 4.86.1-1
Thu, Jun 6, 4:18 PM
dancy added a reverting change for rMSCAf8ace7db9093: git.py: Make is_dir() work for submodule directories: rMSCAec31a9cbd54d: Revert "git.py: Make is_dir() work for submodule directories".
Thu, Jun 6, 4:16 PM
dancy committed rMSCAec31a9cbd54d: Revert "git.py: Make is_dir() work for submodule directories".
Revert "git.py: Make is_dir() work for submodule directories"
Thu, Jun 6, 4:16 PM
dancy added a comment to T366810: scap fails deploying integration/docroot.

@hashar Sorry about the bug. I will fix or revert right away.

Thu, Jun 6, 4:11 PM · Scap
dancy committed rMSCA4759bfabaa83: test_backport.py: Use ssh for Gerrit operations that require auth.
test_backport.py: Use ssh for Gerrit operations that require auth
Thu, Jun 6, 2:54 PM

Wed, Jun 5

dancy added a comment to T366217: unable to create revert commit from scap.

I guess I also ran into this. I was about to roll back the train for group0 (not using scap backport --revert as that would have also reverted on test servers within group 0 in my understanding), thus following steps on https://wikitech.wikimedia.org/wiki/Backport_windows/Deployers#Reverting .
Last step listed on the page is git push origin HEAD:refs/for/[BRANCH]%topic=revert-[SHA1] which told me that I lack an SSH key

Wed, Jun 5, 6:00 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy moved T366217: unable to create revert commit from scap from Backlog to In progress on the Release-Engineering-Team (Yakisfaction) board.
Wed, Jun 5, 4:44 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy edited projects for T366217: unable to create revert commit from scap, added: Release-Engineering-Team (Yakisfaction); removed Release-Engineering-Team (Onboarding 🚀).
Wed, Jun 5, 4:43 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy committed rMSCAe724ad34565e: backport.py: _create_reverts() tweaks.
backport.py: _create_reverts() tweaks
Wed, Jun 5, 2:57 PM
dancy committed rMSCA1e37b4dd034d: git.py: Define remote_set_url and remote_get_url.
git.py: Define remote_set_url and remote_get_url
Wed, Jun 5, 2:51 PM
dancy committed rMSCAf8ace7db9093: git.py: Make is_dir() work for submodule directories.
git.py: Make is_dir() work for submodule directories
Wed, Jun 5, 2:46 PM

Tue, Jun 4

dancy changed the status of T366217: unable to create revert commit from scap from Open to In Progress.
Tue, Jun 4, 7:17 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy closed T366480: Clearer warning when someone tries to run scap backport on non-deployment host as Resolved.

scap 4.85.0 has been deployed with the improvement.

Tue, Jun 4, 3:13 PM · Scap
dancy committed rMSCA72807e1c75fb: Release 4.85.0-1.
Release 4.85.0-1
Tue, Jun 4, 3:10 PM
dancy committed rMSCAa8c15c1e65b9: Restrict some scap subcommands to deploy servers only.
Restrict some scap subcommands to deploy servers only
Tue, Jun 4, 3:07 PM

Mon, Jun 3

dancy changed the status of T366480: Clearer warning when someone tries to run scap backport on non-deployment host from Open to In Progress.
Mon, Jun 3, 4:12 PM · Scap
dancy added a comment to T366480: Clearer warning when someone tries to run scap backport on non-deployment host.

There is also the possibility of someone trying to use scap on the deploy host of the inactive DC (which does have a git repo at /srv/mediawiki-staging), which should also be prevented. (Maybe it already is, I never tried. But at least in theory, it seems like the more damaging case for using a wrong host.)

Mon, Jun 3, 3:19 PM · Scap
dancy updated subscribers of T257317: scap deploy --init on deployment server fails on first puppet run.
Mon, Jun 3, 3:10 PM · Scap
dancy committed rMSCAd8faa6bc2805: Release 4.84.0-1.
Release 4.84.0-1
Mon, Jun 3, 3:00 PM
dancy committed rMSCA1f04ca361fdc: kubernetes: Image build errors are now fatal.
kubernetes: Image build errors are now fatal
Mon, Jun 3, 2:59 PM

Fri, May 31

thcipriani awarded T359643: Get rid of the /srv/mediawiki/php symbolic link a Yellow Medal token.
Fri, May 31, 11:48 PM · Release-Engineering-Team (Priority Backlog 📥)
dancy closed T361401: 1.43.0-wmf.7 deployment blockers as Resolved.
Fri, May 31, 4:11 PM · Release-Engineering-Team (Priority Backlog 📥), Release, Train Deployments

Thu, May 30

dancy removed a parent task for T366268: 12 million warnings of Wikimedia\Rdbms\Platform\SQLPlatform::makeList: array key {key} in list of values ignored: T361401: 1.43.0-wmf.7 deployment blockers.
Thu, May 30, 8:16 PM · MW-1.43-notes (1.43.0-wmf.8; 2024-06-04), MediaWiki-libs-Rdbms, Wikimedia-production-error
dancy removed a subtask for T361401: 1.43.0-wmf.7 deployment blockers: T366268: 12 million warnings of Wikimedia\Rdbms\Platform\SQLPlatform::makeList: array key {key} in list of values ignored.
Thu, May 30, 8:16 PM · Release-Engineering-Team (Priority Backlog 📥), Release, Train Deployments
dancy added a subtask for T361401: 1.43.0-wmf.7 deployment blockers: T366268: 12 million warnings of Wikimedia\Rdbms\Platform\SQLPlatform::makeList: array key {key} in list of values ignored.
Thu, May 30, 5:33 PM · Release-Engineering-Team (Priority Backlog 📥), Release, Train Deployments
dancy added a parent task for T366268: 12 million warnings of Wikimedia\Rdbms\Platform\SQLPlatform::makeList: array key {key} in list of values ignored: T361401: 1.43.0-wmf.7 deployment blockers.
Thu, May 30, 5:33 PM · MW-1.43-notes (1.43.0-wmf.8; 2024-06-04), MediaWiki-libs-Rdbms, Wikimedia-production-error
dancy removed a subtask for T361401: 1.43.0-wmf.7 deployment blockers: T366268: 12 million warnings of Wikimedia\Rdbms\Platform\SQLPlatform::makeList: array key {key} in list of values ignored.
Thu, May 30, 3:49 PM · Release-Engineering-Team (Priority Backlog 📥), Release, Train Deployments
dancy removed a parent task for T366268: 12 million warnings of Wikimedia\Rdbms\Platform\SQLPlatform::makeList: array key {key} in list of values ignored: T361401: 1.43.0-wmf.7 deployment blockers.
Thu, May 30, 3:49 PM · MW-1.43-notes (1.43.0-wmf.8; 2024-06-04), MediaWiki-libs-Rdbms, Wikimedia-production-error
dancy added a comment to T366268: 12 million warnings of Wikimedia\Rdbms\Platform\SQLPlatform::makeList: array key {key} in list of values ignored.

Thanks for the report @TheDJ .

Thu, May 30, 3:48 PM · MW-1.43-notes (1.43.0-wmf.8; 2024-06-04), MediaWiki-libs-Rdbms, Wikimedia-production-error
dancy added a comment to T344730: Migrate Data Engineering Pipelinelib repos to GitLab.

Thanks for the reminder @tchin . Is there some restriction on the gitlab trusted runners repo. I an unable to push to the repo. @tchin @dancy

Thu, May 30, 2:51 PM · Patch-For-Review, Data-Engineering (Q4 2024 April 1st - June 30th), GitLab (Pipeline Services Migration🐤), Event-Platform

Wed, May 29

dancy added a comment to T366217: unable to create revert commit from scap.

Why does scap not use the same user here it uses to +2 any approved backports?

Wed, May 29, 10:12 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy added a comment to T366217: unable to create revert commit from scap.

It seems like scap backport --revert currently relies on the user having something like the following in their .gitconfig file:

[url "ssh://<username>@gerrit.wikimedia.org:29418/"]
    pushInsteadOf = https://gerrit.wikimedia.org/r/p/
    pushInsteadOf = https://gerrit.wikimedia.org/r/
Wed, May 29, 9:54 PM · Release-Engineering-Team (Priority Backlog 📥), Scap
dancy closed T359643: Get rid of the /srv/mediawiki/php symbolic link as Resolved.

All is well.

Wed, May 29, 3:43 PM · Release-Engineering-Team (Priority Backlog 📥)
dancy updated the task description for T359643: Get rid of the /srv/mediawiki/php symbolic link.
Wed, May 29, 3:42 PM · Release-Engineering-Team (Priority Backlog 📥)
Jdforrester-WMF awarded T359643: Get rid of the /srv/mediawiki/php symbolic link a Barnstar token.
Wed, May 29, 2:53 PM · Release-Engineering-Team (Priority Backlog 📥)

Tue, May 28

dancy updated the task description for T359643: Get rid of the /srv/mediawiki/php symbolic link.
Tue, May 28, 8:40 PM · Release-Engineering-Team (Priority Backlog 📥)
dancy added a comment to T322462: Flow/TemplateHelper: Trying to access array offset on value of type null.

I saw a burst of these today, shortly after rolling the 1.43.0-wmf.7 train to group0. Looks like they come in batches of 6 per request. Each has a different line number in the range 556-578.

Tue, May 28, 7:06 PM · StructuredDiscussions, Growth-Team, Wikimedia-production-error
dancy added a comment to T355344: GrowthExperiments: PHP Notice: Undefined index: XXXXXX.

@Michael Thanks for investigating!

Tue, May 28, 4:29 PM · MW-1.43-notes (1.43.0-wmf.8; 2024-06-04), User-Michael, Growth-Team (Sprint 15 (Growth Team)), GrowthExperiments-Mentorship, GrowthExperiments-MentorDashboard, Wikimedia-production-error
dancy added a comment to T362518: Deprecate buster-backports.

Just to be completely sure before deleting

Still left to either remove or rebuild:

  • base images
docker-registry.wikimedia.org/docker-gc:1.0.0-20230402              [FAIL]

OK to delete. 1.2.0 is current.

This is replaced by docker-registry.wikimedia.org/repos/releng/docker-gc/docker-gc and I can remove the image that is not in the releng namespace completely?

Tue, May 28, 4:28 PM · Patch-For-Review, Infrastructure-Foundations, Release-Engineering-Team, serviceops

Thu, May 23

dancy added a comment to T318289: Deprecate Blubber's CLI and microservice (blubberoid) interfaces.

Regarding

  • (main branch) Remove cmd/blubber and cmd/blubberoid interfaces from Blubber
Thu, May 23, 5:09 PM · Release-Engineering-Team (Priority Backlog 📥), Epic, Release Pipeline (Blubber)
dancy added a project to T365729: Puppet agent not working on deployment-cache-text08.deployment-prep for a few weeks: Beta-Cluster-Infrastructure.
Thu, May 23, 3:33 PM · Beta-Cluster-Infrastructure
dancy updated the task description for T365729: Puppet agent not working on deployment-cache-text08.deployment-prep for a few weeks.
Thu, May 23, 3:33 PM · Beta-Cluster-Infrastructure
dancy created T365729: Puppet agent not working on deployment-cache-text08.deployment-prep for a few weeks.
Thu, May 23, 3:32 PM · Beta-Cluster-Infrastructure

Wed, May 22

dancy closed T346226: Package conflicts for tox causes CI failure, a subtask of T357548: Blubber Python builder: Always use a virtualenv, as Resolved.
Wed, May 22, 6:33 PM · Patch-For-Review, Release-Engineering-Team (Yakisfaction)
dancy closed T346226: Package conflicts for tox causes CI failure as Resolved.
Wed, May 22, 6:33 PM · Release Pipeline (Blubber)
dancy updated the task description for T357548: Blubber Python builder: Always use a virtualenv.
Wed, May 22, 5:27 PM · Patch-For-Review, Release-Engineering-Team (Yakisfaction)
dancy closed T357548: Blubber Python builder: Always use a virtualenv as Resolved.
Wed, May 22, 5:26 PM · Patch-For-Review, Release-Engineering-Team (Yakisfaction)
dancy changed the status of T346226: Package conflicts for tox causes CI failure from Open to In Progress.
Wed, May 22, 5:24 PM · Release Pipeline (Blubber)
dancy changed the status of T346226: Package conflicts for tox causes CI failure, a subtask of T357548: Blubber Python builder: Always use a virtualenv, from Open to In Progress.
Wed, May 22, 5:23 PM · Patch-For-Review, Release-Engineering-Team (Yakisfaction)
dancy added a comment to T346226: Package conflicts for tox causes CI failure.

I did a temporary fix in the repository to unblock CI issue so that we are not blocked by issue https://gerrit.wikimedia.org/r/c/mediawiki/services/machinetranslation/+/958599 - directly call pytest instead of calling via tox.
but black, and ruff checks are not present in it.

Wed, May 22, 5:22 PM · Release Pipeline (Blubber)
dancy moved T357548: Blubber Python builder: Always use a virtualenv from Waiting for review to Done on the Release-Engineering-Team (Yakisfaction) board.
Wed, May 22, 4:30 PM · Patch-For-Review, Release-Engineering-Team (Yakisfaction)
dancy updated the task description for T346090: Blubber: Add workaround for Bookworm's Python pip restrictions.
Wed, May 22, 3:37 PM · Release-Engineering-Team, Release Pipeline (Blubber)
dancy added a comment to T357548: Blubber Python builder: Always use a virtualenv.

docker-registry.discovery.wmnet/repos/releng/blubber/buildkit:v0.23.0 has been published with the changes.

Wed, May 22, 3:35 PM · Patch-For-Review, Release-Engineering-Team (Yakisfaction)
dancy closed T289712: Investigate whether we can/should integrate Git/Reviewers with GitLab as Resolved.
Wed, May 22, 3:12 PM · Patch-For-Review, Release-Engineering-Team (Priority Backlog 📥), GitLab (Integrations), User-brennen, Developer Productivity, Code-Health

Tue, May 21

dancy closed T350478: Investigate docker-gc.service failures on GitLab runners as Resolved.

Closing optimistically.

Tue, May 21, 6:45 PM · Patch-For-Review, collaboration-services
dancy added a comment to T289712: Investigate whether we can/should integrate Git/Reviewers with GitLab.

https://docs.gitlab.com/ee/api/merge_requests.html#unsubscribe-from-a-merge-request

POST /projects/:id/merge_requests/:merge_request_iid/unsubscribe might be the needed magic after adding a note to drop notifications for the bot itself.

Tue, May 21, 6:42 PM · Patch-For-Review, Release-Engineering-Team (Priority Backlog 📥), GitLab (Integrations), User-brennen, Developer Productivity, Code-Health
dancy added a comment to T365350: Implement a bot which subscribes/mentions multiple reviewers for merge requests.

I just added a note to https://phabricator.wikimedia.org/T289712#9818356 that may be relevant here:

There's a problem with the method that is used to add mentions to a merge request. When the gitlab-mentions-bot mentions someone in a note, the bot account itself becomes subscribed to the MR and it receives update notifications. That's no good.
Tue, May 21, 6:08 PM · User-brennen, Release-Engineering-Team (Radar), GitLab (Integrations)
dancy reopened T289712: Investigate whether we can/should integrate Git/Reviewers with GitLab as "Open".

There's a problem with the method that is used to add mentions to a merge request. When the gitlab-mentions-bot mentions someone in a note, the bot account itself becomes subscribed to the MR and it receives update notifications. That's no good.

Tue, May 21, 6:07 PM · Patch-For-Review, Release-Engineering-Team (Priority Backlog 📥), GitLab (Integrations), User-brennen, Developer Productivity, Code-Health
dancy added a comment to T327949: Convert runner-1030.gitlab-runners.eqiad1.wikimedia.cloud to an instance-wide shared runner.

Sorry for reopening this old task. But it seems during the refactoring of the runner registration (T344951) runner-1030 was converted back to a group runner. So all WMCS runners are group runners for /repos again (see admin menu).

@dancy should we make runner-1030 (or more?) instance-wide again?

Tue, May 21, 3:42 PM · collaboration-services, Release-Engineering-Team (GitLab IV: Mise En Place 🍱), GitLab