MediaWiki core tests fail with HHVM 3.6.1+dfsg1-1+wm2 : gmp_init(): Unable to convert variable to GMP - string is not an integer
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Assigned To

Authored By

	hashar
	May 12 2015, 3:30 PM

Description

I have upgraded HHVM on the CI slaves from 3.3.1+dfsg1-1+wm3.1 to 3.6.1+dfsg1-1+wm2 and mediawiki/core tests started to fail:

HHVM 3.6.1 is installed.

There were 7 errors:

1) WfBaseConvertTest::testDigitToBase2 with data set #0 ('0', '0', '0', '0', '0', '0', '0')
gmp_init(): Unable to convert variable to GMP - string is not an integer

includes/GlobalFunctions.php:3356
tests/phpunit/includes/GlobalFunctions/wfBaseConvertTest.php:53
tests/phpunit/MediaWikiTestCase.php:131

2) WfBaseConvertTest::testDigitToBase3 with data set #0 ('0', '0', '0', '0', '0', '0', '0')
gmp_init(): Unable to convert variable to GMP - string is not an integer

includes/GlobalFunctions.php:3356
tests/phpunit/includes/GlobalFunctions/wfBaseConvertTest.php:65
tests/phpunit/MediaWikiTestCase.php:131

3) WfBaseConvertTest::testDigitToBase5 with data set #0 ('0', '0', '0', '0', '0', '0', '0')
gmp_init(): Unable to convert variable to GMP - string is not an integer

includes/GlobalFunctions.php:3356
tests/phpunit/includes/GlobalFunctions/wfBaseConvertTest.php:77
tests/phpunit/MediaWikiTestCase.php:131

4) WfBaseConvertTest::testDigitToBase8 with data set #0 ('0', '0', '0', '0', '0', '0', '0')
gmp_init(): Unable to convert variable to GMP - string is not an integer

includes/GlobalFunctions.php:3356
tests/phpunit/includes/GlobalFunctions/wfBaseConvertTest.php:89
tests/phpunit/MediaWikiTestCase.php:131

5) WfBaseConvertTest::testDigitToBase10 with data set #0 ('0', '0', '0', '0', '0', '0', '0')
gmp_init(): Unable to convert variable to GMP - string is not an integer

includes/GlobalFunctions.php:3356
tests/phpunit/includes/GlobalFunctions/wfBaseConvertTest.php:101
tests/phpunit/MediaWikiTestCase.php:131

6) WfBaseConvertTest::testDigitToBase16 with data set #0 ('0', '0', '0', '0', '0', '0', '0')
gmp_init(): Unable to convert variable to GMP - string is not an integer

includes/GlobalFunctions.php:3356
tests/phpunit/includes/GlobalFunctions/wfBaseConvertTest.php:113
tests/phpunit/MediaWikiTestCase.php:131

7) WfBaseConvertTest::testDigitToBase36 with data set #0 ('0', '0', '0', '0', '0', '0', '0')
gmp_init(): Unable to convert variable to GMP - string is not an integer

includes/GlobalFunctions.php:3356
tests/phpunit/includes/GlobalFunctions/wfBaseConvertTest.php:125
tests/phpunit/MediaWikiTestCase.php:131

--


There was 1 failure:

1) IPTCTest::testIPTCParseForcedUTFButInvalid
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    0 => 'ø'
+    0 => ''
 )

So it seems we have regressions in both IPTC and GMP.

Details

Project	Branch	Lines +/-	Subject
mediawiki/core	REL1_25	+1 -1	wfBaseConvert(): Avoid PHP warning when converting zero
mediawiki/core	master	+1 -1	wfBaseConvert(): Avoid PHP warning when converting zero
mediawiki/vagrant	master	+3 -0	Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior
operations/puppet	production	+5 -0	Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior

Customize query in gerrit

Related Objects
Search...

Status	Subtype	Assigned	Task
Resolved	PRODUCTION ERROR	Anomie	T95813 Fix "Warning: Unable to allocate timer" and "Warning: Unable to start limit timer"
Resolved		hashar	T101275 Build a new version of php-luasandbox and hhvm-luasandbox, and deploy to integration hosts
Resolved		hashar	T98876 "mediawiki-phpunit-hhvm" failures on all changes in mediawiki/core due to hhvm upgrade from 3.3.1+dfsg1-1+wm3.1 to 3.6.1+dfsg1-1+wm2
Resolved	PRODUCTION ERROR	Krinkle	T98882 MediaWiki core tests fail with HHVM 3.6.1+dfsg1-1+wm2 : gmp_init(): Unable to convert variable to GMP - string is not an integer

Event Timeline

hashar created this task.May 12 2015, 3:30 PM

hashar raised the priority of this task from to Unbreak Now!.

hashar updated the task description. (Show Details)

hashar added projects: HHVM, MediaWiki-Core-Tests.

hashar added subscribers: Joe, hashar, matmarex and 2 others.

hashar mentioned this in T98876: "mediawiki-phpunit-hhvm" failures on all changes in mediawiki/core due to hhvm upgrade from 3.3.1+dfsg1-1+wm3.1 to 3.6.1+dfsg1-1+wm2.May 12 2015, 3:33 PM

The IPTC failure looks to be the same as T39665: IPTCTest::testIPTCParseForcedUTFButInvalid failure on PHP with buggy glibc (iconv //IGNORE broken)

bd808 mentioned this in T39665: IPTCTest::testIPTCParseForcedUTFButInvalid failure on PHP with buggy glibc (iconv //IGNORE broken).May 12 2015, 3:49 PM

@Smalyshev would you have any time to look at HHVM's iconv implementation and see if backporting your PHP 5.5 patch (< https://github.com/php/php-src/commit/473ec539a1c3d242c8b171dd6a5a98fa17e05c13>) would be possible?

If we can get a working patch filed upstream @Joe can cherry-pick it to our internal HHVM builds even if it hasn't landed yet.

@bd808 I'll take a look though I'm not very familiar with hhvm and setting up build environment for it probably take some time. If we have anything in labs, etc. where I could test (i.e. source/build setup for hhvm) it would help.

In T98882#1280521, @Smalyshev wrote:

@bd808 I'll take a look though I'm not very familiar with hhvm and setting up build environment for it probably take some time. If we have anything in labs, etc. where I could test (i.e. source/build setup for hhvm) it would help.

@Joe can you provide some help to get @Smalyshev bootstrapped into the fun world of HHVM's C++ build chain?

Smalyshev I can surely help you get into the nightt^H^H^H^H^H WONDERFUL experience of building HHVM the debian way. But I advise you NOT to do that.

To build it from source, just follow instructions here:

https://github.com/facebook/hhvm/wiki/Building-and-installing-HHVM-on-Ubuntu-14.04

Please take care of making the fix backportable to the 3.6 branch :)

I think I know what's causing the gmp_init() failures and will upload a fix for that bug.

Change 210653 had a related patch set uploaded (by PleaseStand):
wfBaseConvert(): Avoid PHP warning when converting zero

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

gerritbot added a project: Patch-For-Review.May 13 2015, 7:44 AM

Change 210653 merged by jenkins-bot:
wfBaseConvert(): Avoid PHP warning when converting zero

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

bd808 mentioned this in rMWcfaaff106862: wfBaseConvert(): Avoid PHP warning when converting zero.May 13 2015, 4:01 PM

@bd808 I've made a patch for HHVM: https://github.com/smalyshev/hhvm/commit/33cb291c8de75d7f61a263c18dc27f6dac0c3475 for my tests, seems to fix it. I'll see how I can propose it for hhvm in general.

Also created https://github.com/facebook/hhvm/pull/5425 and https://reviews.facebook.net/D39231

hashar renamed this task from MediaWiki core tests fail with HHVM 3.6.1+dfsg1-1+wm2 to MediaWiki core tests fail with HHVM 3.6.1+dfsg1-1+wm2 : gmp_init(): Unable to convert variable to GMP - string is not an integer.Jun 1 2015, 12:49 PM

hashar lowered the priority of this task from Unbreak Now! to High.

hashar merged a task: T99093: gmp_init(): Unable to convert variable to GMP, string is not an integer.

hashar added a project: Wikimedia-production-error.

hashar set Security to None.

hashar mentioned this in T99093: gmp_init(): Unable to convert variable to GMP, string is not an integer.

hashar moved this task from Untriaged to Jan 2019 / 1.33.wmf13–14 on the Wikimedia-production-error board.

hashar added a subscriber: demon.

@bd808 it turns out hhvm has option hhvm.hack.lang.iconv_ignore_correct=true which despite its name seems to apply to PHP as well as Hack, and with this option the iconv bug does not seem to manifest.

Change 218269 had a related patch set uploaded (by BryanDavis):
Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior

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

Change 218271 had a related patch set uploaded (by BryanDavis):
Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior

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

Change 218271 merged by Ori.livneh:
Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior

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

ori mentioned this in rOPUP15fdfd31380f: Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior.Jun 15 2015, 5:51 AM

Change 218269 merged by jenkins-bot:
Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior

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

bd808 mentioned this in rMWVA528c4581accf: Set hhvm.hack.lang.iconv_ignore_correct for //IGNORE behavior.Jun 15 2015, 3:11 PM

hashar mentioned this in T101275: Build a new version of php-luasandbox and hhvm-luasandbox, and deploy to integration hosts.Jun 16 2015, 11:53 AM

hashar mentioned this in T102616: Upgrade HHVM on CI Trusty slaves from 3.3.1 to 3.6.1.Jun 16 2015, 1:00 PM

Changing priority since we have the workaround of setting hhvm.hack.lang.iconv_ignore_correct = true.

As I understand it, this task is pending @Smalyshev proposed fix on https://reviews.facebook.net/D39231 . So I guess we should keep it open.

hashar added a project: Upstream.Jun 16 2015, 9:35 PM

hashar moved this task from Backlog to Patch proposed upstream on the Upstream board.

hashar moved this task from Jan 2019 / 1.33.wmf13–14 to Mar 2019 / 1.33wmf20-23 on the Wikimedia-production-error board.

@hashar now that I found that setting I'm not even sure whether we need to patch hhvm. It works with this setting, so maybe the patch is not needed after all. For PHP, the patch is necessary, but for hhvm not sure.

@Smalyshev Joelm comment on https://reviews.facebook.net/D39231#637341 seems to indicate its an actual bug in HHVM. 'ignore' settings that have to be set manually always sounds evil to me :-D

Ricordisamoa added a subscriber: Ricordisamoa.Jun 17 2015, 4:41 AM

@hashar you're right, settings like fix.bug = true are ugly, I'll finish the patch.

From an IRC conversation:

I have lowered the task priority yesterday since we have the workaround. It works for CI for the gmp_init() tests and should be suitable for production as well.

I think having the patch completed and upstreamed will save hassle for other peoples. I might be wrong though or underestimating the amount of work needed to finish it. Up to you guys :-)

Change 219516 had a related patch set uploaded (by Legoktm):
wfBaseConvert(): Avoid PHP warning when converting zero

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

Change 219516 merged by jenkins-bot:
wfBaseConvert(): Avoid PHP warning when converting zero

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

Legoktm mentioned this in rMW1d991c668324: wfBaseConvert(): Avoid PHP warning when converting zero.Jun 20 2015, 5:16 AM

https://gerrit.wikimedia.org/r/219516 (branch REL1_25): MW-1.25-release

Legoktm removed a subscriber: • Forrestbot.Jun 29 2015, 5:49 PM

Krinkle closed this task as Resolved.Dec 15 2015, 11:58 PM

Krinkle claimed this task.

demon moved this task from Mar 2019 / 1.33wmf20-23 to Resolved on the Wikimedia-production-error board.Dec 16 2015, 8:39 PM

mmodell changed the subtype of this task from "Task" to "Production Error".Aug 28 2019, 11:12 PM

MediaWiki core tests fail with HHVM 3.6.1+dfsg1-1+wm2 : gmp_init(): Unable to convert variable to GMP - string is not an integerClosed, ResolvedPublicPRODUCTION ERRORActions

Description

Details

Related ObjectsSearch...

Event Timeline

MediaWiki core tests fail with HHVM 3.6.1+dfsg1-1+wm2 : gmp_init(): Unable to convert variable to GMP - string is not an integer
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Related Objects
Search...