When running with DCI_TESTGROUPS="--all" the actual writing of tests fails because of:

Fatal error: Allowed memory size of 268435456 bytes exhausted (tried to allocate 4194304 bytes) in /var/www/core/scripts/run-tests.sh on line 1142

Guess we have to increase the PHP memory_limit from currently 256MB to 384MB or even more.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bzrudi71’s picture

Just startet new test with php memory_limit adjusted to 384MB, let's see... However, even if it works, I'm not to happy with it. run_tests.sh has some optimization potential :-)

ricardoamaro’s picture

The actual running testbot has:

~/repos/drupaltestbot-puppet$ cat modules/testing_bot/files/etc/php5/apache2/php5.4/php.ini | grep memory
; Maximum amount of memory a script may consume (128MB)
; http://php.net/memory-limit
memory_limit = 320M

I'll add this requirement into the new testbot and remake the bootable testbot

ricardoamaro’s picture

Component: Containers » Code
Assigned: Unassigned » ricardoamaro

ricardoamaro’s picture

Status: Active » Fixed
ricardoamaro’s picture

Please reopen if you see that the issue presists

bzrudi71’s picture

Status: Fixed » Needs work

I'm sorry ricardo but...

Fatal error: Allowed memory size of 335544320 bytes exhausted (tried to allocate 81 bytes) in /var/www/core/scripts/run-tests.sh on line 1142

ricardoamaro’s picture

really? o_O

Can you try and rebuild your containers with a new checkout from drupalci_testbot project?

See http://coreresults.drupal-pt.org/ wich is running the latest tests. ;)

ricardoamaro’s picture

Ok, this is still happening.
http://coreresults.drupal-pt.org/node/37

Sutharsan’s picture

I investigated the relation between the chunk size and the memory usage. By making a small adjustment we can significantly reduce the memory usage. In the below figure you can find the relation between chunk size, memory usage and execution time. To be on the safe site I suggest to use a chunk size of 350. Test results may vary depending on hardware and server configuration.

The test script can be found at: https://gist.github.com/Sutharsan/f265a1b8536bb686dd39

Sutharsan’s picture

Status: Needs work » Needs review
FileSize
539 bytes

Changing the chunk size to 350 items.

ricardoamaro’s picture

Testing it manually now

bzrudi71’s picture

Very interesting results! Thank's a lot @Sutharsan

ricardoamaro’s picture

This is tested locally 2 times and RTBC

ricardoamaro’s picture

Status: Needs review » Reviewed & tested by the community
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x, along with a small documentation piece to explain what this is for, since this is the second time in 4 days we had to touch it. :D

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.