All users were logged out of Bugzilla on October 13th, 2018

Fix TC linux64 xpcshell system encoding failure

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
5 months ago

People

(Reporter: ahal, Assigned: ahal)

Tracking

unspecified

Firefox Tracking Flags

(firefox45 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Comment 1

3 years ago
Adding LANG=en_US.UTF-8 didn't seem to make a difference:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=bd91d26b8871
(Assignee)

Comment 2

3 years ago
Here's a clue in the log:
21:40:32     INFO -  PROCESS | 3022 | (process:3022): Gtk-WARNING **: Locale not supported by C library.
21:40:32     INFO -  PROCESS | 3022 | 	Using the fallback 'C' locale.

Which after some googling lead to:
http://askubuntu.com/a/359932

So it seems the language pack may need to be installed in the image, and we may need to run 'dpkg-reconfigure locales' to set it up.
(Assignee)

Updated

3 years ago
Assignee: nobody → ahalberstadt
Status: NEW → ASSIGNED
(Assignee)

Comment 3

3 years ago
Some success:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e01926a462c8

We needed the 'language-pack-en-base' package installed, then set LANG. Chunk 2 still fails for other reasons though. Will post some patches later today.
(Assignee)

Comment 4

3 years ago
Created attachment 8681985 [details]
MozReview Request: Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image; set LANG/LC_ALL to en_US.UTF-8, r=dustin

Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image, r=dustin

Also calls `dpkg-reconfigure locales`, but it's not entirely clear if this is
necessary or not.
Attachment #8681985 - Flags: review?(dustin)
(Assignee)

Comment 5

3 years ago
Created attachment 8681986 [details]
MozReview Request: Bug 1218542 - Set LANG and LC_ALL to en_US.UTF-8 for desktop taskcluster tests, r=dustin

Bug 1218542 - Set LANG and LC_ALL to en_US.UTF-8 for desktop taskcluster tests, r=dustin
Attachment #8681986 - Flags: review?(dustin)
Comment on attachment 8681985 [details]
MozReview Request: Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image; set LANG/LC_ALL to en_US.UTF-8, r=dustin

https://reviewboard.mozilla.org/r/23931/#review21367

::: testing/docker/ubuntu1204-test-upd/VERSION:1
(Diff revision 1)
> -0.1.1.20151012044301
> +0.1.2

This should remain a datestamp.  At some point, we'll have an autolander bumping it, and the date gives a good indication of when the image was created (the FROM in desktop-test should be changed, too)

Looks good otherwise.  Do you have permission to push these images, or do you want me to build and push the results?  Greg's working on automatically building images now :)
Attachment #8681985 - Flags: review?(dustin)
Comment on attachment 8681986 [details]
MozReview Request: Bug 1218542 - Set LANG and LC_ALL to en_US.UTF-8 for desktop taskcluster tests, r=dustin

https://reviewboard.mozilla.org/r/23933/#review21369

::: testing/taskcluster/tasks/tests/fx_unittest_base.yml:21
(Diff revision 1)
> +      LC_ALL: 'en_US.UTF-8'

I wonder if it would be better to specify these at the system level somehow?  Maybe in /etc/profile.d?
Attachment #8681986 - Flags: review?(dustin) → review+
(Assignee)

Comment 8

3 years ago
Comment on attachment 8681985 [details]
MozReview Request: Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image; set LANG/LC_ALL to en_US.UTF-8, r=dustin

Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image; set LANG/LC_ALL to en_US.UTF-8, r=dustin

Also calls `dpkg-reconfigure locales`, but it's not entirely clear if this is
necessary or not.
Attachment #8681985 - Attachment description: MozReview Request: Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image, r=dustin → MozReview Request: Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image; set LANG/LC_ALL to en_US.UTF-8, r=dustin
Attachment #8681985 - Flags: review?(dustin)
(Assignee)

Updated

3 years ago
Attachment #8681986 - Attachment is obsolete: true
(Assignee)

Comment 9

3 years ago
> Looks good otherwise.  Do you have permission to push these images, or do
> you want me to build and push the results?  Greg's working on automatically
> building images now :)

I don't think I have permission to push these.. I don't mind doing it if you give me permission, otherwise, yes please push them on my behalf :).
Comment on attachment 8681985 [details]
MozReview Request: Bug 1218542 - Install language-pack-en-base in ubuntu1204-test image; set LANG/LC_ALL to en_US.UTF-8, r=dustin

https://reviewboard.mozilla.org/r/23931/#review21375

Thanks!
Attachment #8681985 - Flags: review?(dustin) → review+
Builds in progress
Builds complete -- you should be good to land this.

Comment 14

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b083ca57314a
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox45: --- → fixed
Resolution: --- → FIXED
removing the b2g 2.5 flag since this commit has been reverted due to an incorrect merge, sorry for the confusion
status-b2g-v2.5: fixed → ---
Component: General Automation → General
Product: Release Engineering → Release Engineering
You need to log in before you can comment on or make changes to this bug.