Uploaded image for project: 'Nuxeo ECM Build/Test Environment'
  1. Nuxeo ECM Build/Test Environment
  2. NXBT-2614

Fix hooks timeouts - PR-2652

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Captain Hook
    • Tags:
    • Sprint:
      DevTools-04, DevTools-05

      Description

      Anahide Tchertchian

      https://github.com/nuxeo/nuxeo/pull/2562, did not get a notif on #pull-requests + did not get reviews even if 2 :+1:

      Once again, there were timeouts today. But it is not clear if related to NXBT-2585.
      The docker containers are not frozen.
      The logs look pretty good:

      extract from attached /tmp/hooks_node_1-2562.log
      2018-11-21T13:35:15.320951560Z 2018-11-21 13:35:15,320 INFO [nxtools.hooks.services.config] Loading config file: /opt/conf/nuxeo-tools-hooks.ini
      2018-11-21T13:35:15.322892719Z 2018-11-21 13:35:15,322 INFO [nxtools.hooks.services.database] Connecting to database backend: mongodb://mongo:27017/nxtools-hooks
      2018-11-21T13:35:15.353124499Z 2018-11-21 13:35:15,352 INFO [nxtools.hooks.endpoints.webhook.github_hook] Got a payload https://github.com/nuxeo/nuxeo - pull_request
      2018-11-21T13:35:15.353780545Z 2018-11-21 13:35:15,353 INFO [nxtools.hooks.endpoints.webhook.github_handlers.pull_request] GithubStorePullRequestHandler.handle
      2018-11-21T13:35:15.358868592Z 2018-11-21 13:35:15,358 INFO [nxtools.hooks.entities.db_entities] Pull request nuxeo/nuxeo/pull/2562 saved
      2018-11-21T13:35:17.205426117Z 2018-11-21 13:35:17,205 INFO [nxtools.hooks.endpoints.webhook.github_handlers.pull_request] Review asked for nuxeo/nuxeo/pull/2562/commits/e1b4bd7d4cd4cf27add6c40a99dc0b555debd334
      2018-11-21T13:35:17.206615275Z 2018-11-21 13:35:17,206 INFO [nxtools.hooks.services.github_service] Setting status of nuxeo/nuxeo/commits/e1b4bd7d4cd4cf27add6c40a99dc0b555debd334 to: 0 (of 2) reviews
      2018-11-21T13:35:19.164396387Z 2018-11-21 13:35:19,164 INFO [nxtools.hooks.services.github_service] nuxeo/nuxeo/pull/2562: Checking reviewers
      2018-11-21T13:35:21.011789689Z 2018-11-21 13:35:21,011 WARNING [nxtools.hooks.services.github_service] Hunk #0 no author found
      (...)
      2018-11-21T13:35:35.300263926Z 2018-11-21 13:35:35,300 WARNING [nxtools.hooks.services.github_service] Hunk #4 no author found
      2018-11-21T13:35:35.741631234Z 2018-11-21 13:35:35,741 INFO [nxtools.hooks.services.github_service] Sending slack notification for nuxeo/nuxeo/pull/2562 in *****
      2018-11-21T13:35:36.246767437Z 2018-11-21 13:35:36,246 INFO [nxtools.hooks.services.github_service] Notifying potential reviewers with a comment of nuxeo/nuxeo/pull/2562 on *****
      2018-11-21T13:35:37.391136877Z 2018-11-21 13:35:37,390 INFO [nxtools.hooks.entities.db_entities] Review saved for nuxeo/nuxeo/pull/2562 saved
      2018-11-21T13:35:37.391999205Z 2018-11-21 13:35:37,391 INFO [nxtools.hooks.entities.db_entities] Pull request nuxeo/nuxeo/pull/2562 saved
      

      The logs should be checked:

      • the logged commit is the last of the PR; the first commit is rather expected to identify the branch associated with a PR
        https://github.com/nuxeo/nuxeo/pull/2562/commits
      • 79aceb7d4996d57311f6b3b8b4bb5099b390cd8d
      • dc1288a65a0921d574c2e65691563d4d02c9c95a
      • 81d3f8814c504e11883b8810c02c954a547a7253
      • 79be3aa35fc1a1facaab71e4879a1d6c3aabbaa9
      • e1b4bd7d4cd4cf27add6c40a99dc0b555debd334

      The "hunk" list seems abnormally long.

      There are five modified files with at max 47 contributors:

      $ for file in $(git log origin/8.10..origin/fix-NXP-24329-target-cleanup-8.10 --name-only --pretty=); do echo $file ; git blame -p origin/fix-NXP-24329-target-cleanup-8.10 -- $file |grep author-mail|sort -u  ; done
      nuxeo-distribution/nuxeo-marketplace-jsf-ui/src/main/assemble/assembly.xml
      author-mail <ataillefer@nuxeo.com>
      author-mail <at@nuxeo.com>
      author-mail <fguillaume@nuxeo.com>
      nuxeo-distribution/nuxeo-nxr-jsf-ui/src/main/assemble/assembly.xml
      author-mail <at@nuxeo.com>
      author-mail <fguillaume@nuxeo.com>
      author-mail <jcarsique@nuxeo.com>
      author-mail <slacoin@nuxeo.com>
      nuxeo-distribution/nuxeo-nxr-server/src/main/assemble/assembly.xml
      author-mail <at@nuxeo.com>
      author-mail <fguillaume@nuxeo.com>
      nuxeo-distribution/nuxeo-server-tomcat/src/main/assemble/assembly.xml
      author-mail <ataillefer@nuxeo.com>
      author-mail <at@nuxeo.com>
      author-mail <bdelbosc@nuxeo.com>
      author-mail <bstefanescu@nuxeo.com>
      author-mail <fg@nuxeo.com>
      author-mail <fguillaume@nuxeo.com>
      author-mail <jcarsique@nuxeo.com>
      author-mail <nchapurlat@nuxeo.com>
      author-mail <slacoin@nuxeo.com>
      author-mail <troger@nuxeo.com>
      author-mail <vpasquier@nuxeo.com>
      author-mail <yjulienne@nuxeo.com>
      pom.xml
      author-mail <ajusto@nuxeo.com>
      author-mail <akervern@nuxeo.com>
      author-mail <arnaud.k@gmail.com>
      author-mail <arussel@nuxeo.com>
      author-mail <ataillefer@nuxeo.com>
      author-mail <at@nuxeo.com>
      author-mail <bdelbosc@nuxeo.com>
      author-mail <bs@nuxeo.com>
      author-mail <bstefanescu@nuxeo.com>
      author-mail <damien.metzler@leroymerlin.fr>
      author-mail <dmetzler@gmail.com>
      author-mail <dmetzler@nuxeo.com>
      author-mail <fdavid@nuxeo.com>
      author-mail <fg@nuxeo.com>
      author-mail <fguillaume@nuxeo.com>
      author-mail <gbarata@nuxeo.com>
      author-mail <glefevre@nuxeo.com>
      author-mail <grenard@nuxeo.com>
      author-mail <hbrown@nuxeo.com>
      author-mail <ismith@nuxeo.com>
      author-mail <jcarsique@nuxeo.com>
      author-mail <jenkins@nuxeo.com>
      author-mail <jmo@chalmers.se>
      author-mail <julien.carsique@gmail.com>
      author-mail <kleturc@nuxeo.com>
      author-mail <ldoguin@nuxeo.com>
      author-mail <mcedica@nuxeo.com>
      author-mail <mg@nuxeo.com>
      author-mail <mhilaire@nuxeo.com>
      author-mail <mnixo@nuxeo.com>
      author-mail <mvachette@nuxeo.com>
      author-mail <nchapurlat@nuxeo.com>
      author-mail <nelson.silva@inevo.pt>
      author-mail <nsilva@nuxeo.com>
      author-mail <ogrisel@nuxeo.com>
      author-mail <pgmillon@nuxeo.com>
      author-mail <qlamerand@nuxeo.com>
      author-mail <rdias@nuxeo.com>
      author-mail <sfermigier@nuxeo.com>
      author-mail <slacoin@nuxeo.com>
      author-mail <tdelprat@nuxeo.com>
      author-mail <td@nuxeo.com>
      author-mail <throger@gmail.com>
      author-mail <tmartins@nuxeo.com>
      author-mail <troger@nuxeo.com>
      author-mail <vpasquier@nuxeo.com>
      author-mail <yjulienne@nuxeo.com>
      

        Attachments

          Activity

            People

            • Assignee:
              jcarsique Julien Carsique
              Reporter:
              jcarsique Julien Carsique
              Participants:
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: