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

mariabackup SST fail if data-directory have lost+found directory

Details

    Description

      Mariabackup SST fail if database directory contains lost+found 

       rm: cannot remove ‘/data/lib/mariadb/lost+found’: Permission denied

      scripts/wsrep_sst_mariabackup.sh does not exclude lost+found directory while performing SST

      https://github.com/MariaDB/server/blob/93ac7ae70ff000353538f732899b421a3f2ea7ce/scripts/wsrep_sst_mariabackup.sh

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. MDEV-8064 ignore-db-dir=lost+found not working

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Stalled

          Bug - A problem which impairs or prevents the functions of the product. MDEV-10303 rsync_sst_rsync: ignore lost+found during database directory transfer

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. MDEV-10487 Galera SST using rsync does not filter out lost+found

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. MDEV-21295 SST should not remove ignore-db-dir directories

          • Major - Major loss of function.
          • Open

          Activity

            Ascending order - Click to sort in descending order
            pramod.mahto@mariadb.com Pramod Mahto created issue -
            pramod.mahto@mariadb.com Pramod Mahto made changes -
            Field Original Value New Value
            Priority Blocker [ 1 ] Major [ 3 ]
            jplindst Jan Lindström (Inactive) made changes -
            Assignee Julius Goryavsky [ sysprg ]
            jplindst Jan Lindström (Inactive) made changes -
            Fix Version/s 10.2 [ 14601 ]
            Fix Version/s 10.3 [ 22126 ]
            Fix Version/s 10.4 [ 22408 ]
            Fix Version/s 10.5 [ 23123 ]
            jplindst Jan Lindström (Inactive) added a comment -

            Some idea can be see https://github.com/mariadb/server/commit/44e3046d3b09a21e21295979d6ddad9f332ebadd

            jplindst Jan Lindström (Inactive) added a comment - Some idea can be see https://github.com/mariadb/server/commit/44e3046d3b09a21e21295979d6ddad9f332ebadd
            marko Marko Mäkelä made changes -
            Component/s Galera SST [ 10121 ]
            marko Marko Mäkelä added a comment -

            I think that the mariabackup --databases-exclude option should be applicable. It probably expects a regular expression as a parameter, and the + likely is a regular expression meta-character that will need escaping. You should also test that the argument will only match a full string, in case someone does CREATE DATABASE `#mysql50#not_lost+found`;.

            marko Marko Mäkelä added a comment - I think that the mariabackup --databases-exclude option should be applicable. It probably expects a regular expression as a parameter, and the + likely is a regular expression meta-character that will need escaping. You should also test that the argument will only match a full string, in case someone does CREATE DATABASE `#mysql50#not_lost+found`; .
            GeoffMontee Geoff Montee (Inactive) made changes -
            GeoffMontee Geoff Montee (Inactive) made changes -
            GeoffMontee Geoff Montee (Inactive) made changes -
            GeoffMontee Geoff Montee (Inactive) made changes -
            sysprg Julius Goryavsky made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            sysprg Julius Goryavsky added a comment -

            Fixed, patch here: https://github.com/MariaDB/server/commit/909cd1a6ba52b28a705190d7576a15cd38ebea9e

            sysprg Julius Goryavsky added a comment - Fixed, patch here: https://github.com/MariaDB/server/commit/909cd1a6ba52b28a705190d7576a15cd38ebea9e
            sysprg Julius Goryavsky made changes -
            Assignee Julius Goryavsky [ sysprg ] Jan Lindström [ jplindst ]
            Status In Progress [ 3 ] In Review [ 10002 ]
            jplindst Jan Lindström (Inactive) added a comment -

            Please add mtr test as described on git comment.

            jplindst Jan Lindström (Inactive) added a comment - Please add mtr test as described on git comment.
            jplindst Jan Lindström (Inactive) made changes -
            Assignee Jan Lindström [ jplindst ] Julius Goryavsky [ sysprg ]
            Status In Review [ 10002 ] Stalled [ 10000 ]
            sysprg Julius Goryavsky made changes -
            Status Stalled [ 10000 ] In Progress [ 3 ]
            sysprg Julius Goryavsky added a comment -

            Added a test that checks the correct operation in the presence of databases, which can potentially cause problems when checking names. Also patch has been rebased to 10.2 branch (latest revision):

            https://github.com/MariaDB/server/commit/e1875eeb251aa83a3dc8ab148228d2de42671699

            sysprg Julius Goryavsky added a comment - Added a test that checks the correct operation in the presence of databases, which can potentially cause problems when checking names. Also patch has been rebased to 10.2 branch (latest revision): https://github.com/MariaDB/server/commit/e1875eeb251aa83a3dc8ab148228d2de42671699
            sysprg Julius Goryavsky made changes -
            Assignee Julius Goryavsky [ sysprg ] Jan Lindström [ jplindst ]
            Status In Progress [ 3 ] In Review [ 10002 ]
            jplindst Jan Lindström (Inactive) added a comment -

            ok to push.

            jplindst Jan Lindström (Inactive) added a comment - ok to push.
            jplindst Jan Lindström (Inactive) made changes -
            Assignee Jan Lindström [ jplindst ] Julius Goryavsky [ sysprg ]
            Status In Review [ 10002 ] Stalled [ 10000 ]
            sysprg Julius Goryavsky added a comment -

            merged with head revision, https://github.com/MariaDB/server/commit/888010d9dd5409c48caf01fb0f465c98caa232c7

            sysprg Julius Goryavsky added a comment - merged with head revision, https://github.com/MariaDB/server/commit/888010d9dd5409c48caf01fb0f465c98caa232c7
            sysprg Julius Goryavsky made changes -
            issue.field.resolutiondate 2020-10-20 10:44:55.0 2020-10-20 10:44:55.208
            sysprg Julius Goryavsky made changes -
            Fix Version/s 10.2.35 [ 25022 ]
            Fix Version/s 10.3.26 [ 25021 ]
            Fix Version/s 10.4.16 [ 25020 ]
            Fix Version/s 10.5.7 [ 25019 ]
            Fix Version/s 10.6.0 [ 24431 ]
            Fix Version/s 10.2 [ 14601 ]
            Fix Version/s 10.3 [ 22126 ]
            Fix Version/s 10.4 [ 22408 ]
            Fix Version/s 10.5 [ 23123 ]
            Resolution Fixed [ 1 ]
            Status Stalled [ 10000 ] Closed [ 6 ]
            serg Sergei Golubchik made changes -
            Workflow MariaDB v3 [ 104887 ] MariaDB v4 [ 157441 ]
            mariadb-jira-automation Jira Automation (IT) made changes -
            Zendesk Related Tickets 161198 110083

            People

              sysprg Julius Goryavsky
              pramod.mahto@mariadb.com Pramod Mahto
              Votes:
              1 Vote for this issue
              Watchers:
              6 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.