Error compiling message catalogs following TWN import
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	bd808
	Nov 23 2021, 2:17 PM

Description

The 2021-11-22 export from TWN is causing post-merge build failures when the container build attempts to compile the .po message catalogs:

Execution of msgfmt failed: /srv/app/toolhub/locale/qqq/LC_MESSAGES/django.po:1088: a format specification for argument 'message' doesn't exist in 'msgstr'
msgfmt: found 1 fatal error
CommandError: compilemessages generated one or more errors.

Details

Subject	Repo	Branch	Lines +/-
toolhub: Bump container version to 2021-12-23-121200-production	operations/deployment-charts	master	+1 -1
build: Remove legacy qqq Django locale	wikimedia/toolhub	main	+0 -1 K
Toolhub: Exclude qqq from export	translatewiki	master	+1 -1
build: Exclude qqq pseudo-locale from compilation	wikimedia/toolhub	main	+5 -5

Customize query in gerrit

Related Objects

Mentioned In: rGTWN4b197382e16c: Toolhub: Exclude qqq from export

Event Timeline

bd808 created this task.Nov 23 2021, 2:17 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 23 2021, 2:17 PM

bd808 claimed this task.Nov 23 2021, 2:19 PM

bd808 triaged this task as Unbreak Now! priority.

bd808 changed the subtype of this task from "Task" to "Bug Report".

bd808 added a project: translatewiki.net.

Restricted Application added a project: User-bd808. · View Herald TranscriptNov 23 2021, 2:19 PM

I think we probably should be using --exclude qqq in our makemessages invocation. The qqq pseudo-locale is only meant to provide translation unit documentation for the users at https://translatewiki.net. We do not care if the PO file exported by TWN compiles cleanly into a MO binary.

https://docs.djangoproject.com/en/2.2/ref/django-admin/#django-admin-makemessages

If you don't maintain the documentation file in the repository (only through translatewiki.net), then we could skip exporting it.

In T296301#7523630, @Nikerabbit wrote:

If you don't maintain the documentation file in the repository (only through translatewiki.net), then we could skip exporting it.

That seems like a reasonable thing as well. Following a "belt and suspenders" model I think we could do both.

Change 740844 had a related patch set uploaded (by BryanDavis; author: Bryan Davis):

[wikimedia/toolhub@main] build: Exclude qqq pseudo-locale from compilation

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

gerritbot added a project: Patch-For-Review.Nov 23 2021, 2:40 PM

Change 740844 merged by jenkins-bot:

[wikimedia/toolhub@main] build: Exclude qqq pseudo-locale from compilation

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

Resetting from UBN! status. The merged patch has restored post-merge container builds as hoped.

Maintenance_bot removed a project: Patch-For-Review.Nov 23 2021, 5:10 PM

Change 740896 had a related patch set uploaded (by BryanDavis; author: Bryan Davis):

[translatewiki@master] Toolhub: Exclude qqq from export

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

gerritbot added a project: Patch-For-Review.Nov 23 2021, 5:16 PM

bd808 moved this task from In Progress to Review on the Toolhub board.Nov 23 2021, 5:17 PM

Change 740896 merged by jenkins-bot:

[translatewiki@master] Toolhub: Exclude qqq from export

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

bd808 mentioned this in rGTWN4b197382e16c: Toolhub: Exclude qqq from export.Nov 23 2021, 5:33 PM

Change 740910 had a related patch set uploaded (by BryanDavis; author: Bryan Davis):

[wikimedia/toolhub@main] build: Remove legacy qqq Django locale

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

Change 740910 merged by jenkins-bot:

[wikimedia/toolhub@main] build: Remove legacy qqq Django locale

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

Maintenance_bot removed a project: Patch-For-Review.Nov 24 2021, 3:10 PM

bd808 closed this task as Resolved.Nov 24 2021, 6:00 PM

bd808 added a project: Developer-Advocacy (Oct-Dec 2021).

Change 749220 had a related patch set uploaded (by BryanDavis; author: Bryan Davis):

[operations/deployment-charts@master] toolhub: Bump container version to 2021-12-20-122341-production

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

gerritbot added a project: Patch-For-Review.Dec 21 2021, 5:19 PM

Change 749220 merged by jenkins-bot:

[operations/deployment-charts@master] toolhub: Bump container version to 2021-12-23-121200-production