Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-34779

Sporadic test failure in rpl.rpl_semi_sync_cond_var_per_thd

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • 11.4.2
    • 11.4.4
    • Tests
    • None

    Description

      The test case rpl.rpl_semi_sync_cond_var_per_thd fails sporadically like this:

      rpl.rpl_semi_sync_cond_var_per_thd 'row' w17 [ fail ]  Found warnings/errors in server log file!
              Test ended at 2024-08-17 11:31:53
      line
      2024-08-17 11:31:52 6 [Warning] Could not write packet: fd: 2112  state: 74  errno: 2  vio_errno: 32  length: 1160
      2024-08-17 11:31:52 6 [ERROR] mariadbd: Got an error writing communication packets
      ^ Found warnings in /home/buildbot/amd64-ubuntu-2204-debug-ps/build/mysql-test/var/17/log/mysqld.2.err
      ok
      

      It looks like this may just be a bad merge to 11.4. The merge commit is this:

      commit 99b370e0237aeb5cda5055dc582dd87a8d7618fc
      Merge: b86a2f03b6a dfe030fda6e
      Author: Oleksandr Byelkin <sanja@mariadb.com>
      Date:   Tue May 21 19:38:51 2024 +0200
       
          Merge branch '11.2' into 11.4
      

      It changes the test like this, which seems to be removing the suppressions for exactly the messages that cause the test to fail:

      @@ -26,18 +26,26 @@
       
       --connection master
       call mtr.add_suppression("Got an error reading communication packets");
      -call mtr.add_suppression("Got an error writing communication packets");
      -call mtr.add_suppression("Could not read packet");
      -call mtr.add_suppression("Could not write packet");
      +call mtr.add_suppression("Could not read packet:.* vio_errno: 1158");
      +call mtr.add_suppression("Could not write packet:.* vio_errno: 1160");
      

      Note that it removes the "Got an error writing communication packets"; and that it seems to have a typo in the "Could not write packet" regexp, vio_errno: 1160, but the message is actually vio_errno: 32 length: 1160.

      Not sure how that change came into the merge, there is no other related commits in the git history and no explanation in the commit message.

      bnestere, assigning this to you as it is your test case, maybe you can immediately spot if the above analysis is correct and the suppressions just need to be fixed from 11.4?

      Attachments

        Issue Links

          Activity

            People

              bnestere Brandon Nesterenko
              knielsen Kristian Nielsen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Git Integration

                  Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.