Skip to main content

Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

    •  
      request #15077 Pull Request processing fails on huge Git merge
    Actions
    Infos
    #15077
    Guilhem Bonnefille (CS) (gbonnefille)
    2020-07-17 11:42
    2020-07-06 17:27
    16341
    Details
    Pull Request processing fails on huge Git merge
    When a merge between a trunk branch and a feature branch introduce a HUGE diff, the pull request processing fails.

    On FPM log we can see a memory error:

    [06-Jul-2020 10:04:11] WARNING: [pool tuleap] child 18853 said into stderr: "NOTICE: PHP message: PHP Fatal error: Allowed memory size of 268435456 bytes exhausted (tried to allocate 134217736 bytes) in /usr/share/tuleap/src/common/System/Command.class.php on line 33"


    Looking at the code, the PullRequestMerger->detectMergeConflict call a mergeTree action and read result in memory.

    As the expected feature is to detect a conflict marker, perhaps the git command can be processed by a `grep --quiet` command and only focus on the return code.
    Pull Request
    11.15
    Empty
    • [ ] enhancement
    • [ ] internal improvement
    Empty
    Stage
    Thomas Gerbet (tgerbet)
    Closed
    2020-07-17
    Attachments
    Artifact links

    Releases

    Fixed in

    Artifact ID Project Tracker Summary Status Last Update Date Submitted By Assigned to
    rel #14817 Tuleap Releases 11.17 Delivered 2020-23-07 10:57 Manuel Vacelet (vaceletm)
    Empty
    References
    Referencing request #15077

    Artifact Tracker v5

    request #15082 FastForward not detected when checking mergeability

    Git commit

    tuleap/tuleap/stable

    Merge commit 'refs/changes/44/19544/1' of ssh://gerrit.tuleap.net:29418/tuleap into HEAD 0b50e9e68b
    User avatar Yannis ROSSETTO (rossettoy) , 2020-07-17 11:41
    request #15077: Mergeability status detection crashes when the merge is massive 72d9d8fdd6
    User avatar Thomas Gerbet (tgerbet) , 2020-07-16 13:48
    Detect the manual merge of a PR before updating the merge statuses 1805fb4ebf
    User avatar Thomas Gerbet (tgerbet) , 2020-07-16 14:36
    Display settings

    Follow-ups

    User avatar
    Yannis ROSSETTO (rossettoy)2020-07-17 11:42
    Integrated into Tuleap 11.16.99.164
    • Status changed from Under review to Closed
    • Connected artifacts
    • Close date set to 2020-07-17
    User avatar
    Guilhem Bonnefille (CS) (gbonnefille)2020-07-07 10:28
    Please, note that adding traces in execInPath (before and after call) was very useful.
    Perhaps can you plan to add a logger here and trace these calls as debug.
    User avatar
    Thomas Gerbet (tgerbet)2020-07-06 17:37
    Great catch and thanks for the detailed report 👍 ! I'm guessing the debugging was painful.


    I will take a look to see what can be done, hopefully for this release cycle (11.17).
    • Status changed from New to Verified
    • Assigned to changed from None to Thomas Gerbet (tgerbet)