Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-55292

Include tideways profiler along with xhprof for PHP7 profiling

    XMLWordPrintable

Details

    • MOODLE_31_STABLE
    • MOODLE_30_STABLE, MOODLE_31_STABLE
    • Hide

      Ensure DEVELOPER_DEBUG is on to report all possible notices.

      • Using an existing XHProf installation
        • Turn on profiling (Site Administration > Development > Profiling)
        • View pages and confirm there is recorded profiling runs.
      • Switch to PHP7 and install the tideways extension
      • Disable all profiling extensions (Both XHProf and Tideways)
        • Ensure profiling (Site Administration > Development > Profiling) is unavailable.
        • View pages and confirm no notices or warnings displayed
      Show
      Ensure DEVELOPER_DEBUG is on to report all possible notices. Using an existing XHProf installation Turn on profiling (Site Administration > Development > Profiling) View pages and confirm there is recorded profiling runs. Switch to PHP7 and install the tideways extension https://github.com/tideways/php-profiler-extension (Installation instructions) Turn on profiling (Site Administration > Development > Profiling) View pages and confirm there is recorded profiling runs. Disable all profiling extensions (Both XHProf and Tideways) Ensure profiling (Site Administration > Development > Profiling) is unavailable. View pages and confirm no notices or warnings displayed

    Description

      • xhprof has not been updated for PHP7
      • uprofiler also does not work for PHP7
      • All other forks have not provided the working stability required for general profiling.

      Adding the extension capability of tideways, it works with PHP7 and PHP5.6 which are the supported Moodle 3.2 PHP versions.

      https://github.com/tideways/php-profiler-extension

      I have used this extension for a few weeks on PHP7 and it has been working very reliably.

      Attachments

        Issue Links

          Activity

            People

              mr-russ Russell Smith
              mr-russ Russell Smith
              Rajesh Taneja Rajesh Taneja
              Andrew Lyons Andrew Lyons
              Damyon Wiese Damyon Wiese
              Matteo Scaramuccia, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              3 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                12/Sep/16