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

init script for mariadb galera cluster

    Details

    • Type: Task
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Fix Version/s: 10.1
    • Component/s: None
    • Labels:

      Description

      Having a look into the init script from mariadb-galera-server-5.5 I see some things that are not good. Some of them also fit to mariadb in general.

      • kill -9 (See Bug #870310)
      • killall mysqld (Do I really want to kill all running mysqld?)

      Things that are not good for Galera or are missing for a better Galera support.

      • Relying on a credentials file (/etc/mysql/debian.cnf). This is always bad, because it is inconsistent. Restarting/Status-Check a server will break, because the credentials in the file does not match the ones in the database. I wonder why not use any socket-auth.
      • Missing Bootstrap for Galera. PXC ships with a bootstrap option. May be worth to think about it.
      • The --wsrep-recover Option is ignored. This will most likely prevent a SST after a crash, as it can be used to get the last committed seqno

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              elenst Elena Stepanova added a comment -

              Bug #870310 was (somewhat) fixed as MDEV-703. Now the timeout is 10 min, and I'd say if it didn't shutdown properly in this time, something is most likely wrong, and it will have to be killed anyway. The reason for doing it in the script is the upgrade i think, which would hung if the server hangs on shutdown.

              killall is still present though.

              For the Galera issues, Jan, could you please take a look and decide whether it should be done on our side or we should reassign it to Seppo? And if it's to be done on our side, please decide between you and Holyfoot who would be doing it.
              As far as I know, the script gets merges from Codership, but at some point it was touched by Holyfoot as well, as a part of MDEV-507.

              Thanks.

              Show
              elenst Elena Stepanova added a comment - Bug #870310 was (somewhat) fixed as MDEV-703 . Now the timeout is 10 min, and I'd say if it didn't shutdown properly in this time, something is most likely wrong, and it will have to be killed anyway. The reason for doing it in the script is the upgrade i think, which would hung if the server hangs on shutdown. killall is still present though. For the Galera issues, Jan, could you please take a look and decide whether it should be done on our side or we should reassign it to Seppo? And if it's to be done on our side, please decide between you and Holyfoot who would be doing it. As far as I know, the script gets merges from Codership, but at some point it was touched by Holyfoot as well, as a part of MDEV-507 . Thanks.
              Hide
              erkules erkan yanar added a comment -

              Forget the wsrep-recover thing. It is in mysqld_safe and not in the init script.

              Show
              erkules erkan yanar added a comment - Forget the wsrep-recover thing. It is in mysqld_safe and not in the init script.
              Hide
              danblack Daniel Black added a comment -

              ref kill/killall - perhaps check its not in a DONOR state.

              bootstrap - see MDEV-4253

              credentials - see MDEV-5500 however socket-auth sounds like a much better option

              Show
              danblack Daniel Black added a comment - ref kill/killall - perhaps check its not in a DONOR state. bootstrap - see MDEV-4253 credentials - see MDEV-5500 however socket-auth sounds like a much better option

                People

                • Assignee:
                  nirbhay_c Nirbhay Choubey
                  Reporter:
                  erkules erkan yanar
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated: