Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-5984

Use RO MySQL account in qserv_testdata

    XMLWordPrintable

    Details

    • Type: Story
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: Qserv, qserv_testdata
    • Labels:
      None
    • Story Points:
      4
    • Sprint:
      DB_F20_09, DB_S21_12, DB_F21_06, DB_S22_12, DB_F22_6, DB_S23_6
    • Team:
      Data Access and Database

      Description

      It seems integration tests datasets are now loaded with Loader. So using MySQL root account is no more required in integration tests, an account with SELECT access on test databases (like qsmaster), should be enough.

      Furthermore, all code related to MySQL writes can be removed from

      python/lsst/qserv/tests/sql/cmd.py
      python/lsst/qserv/tests/sql/connection.py
      

        Attachments

          Issue Links

            Activity

            Hide
            fritzm Fritz Mueller added a comment -

            Nate Pease [X]: Could you please take a look and verify that integration tests now run as qsmaster, and not MySQL root?

            Show
            fritzm Fritz Mueller added a comment - Nate Pease [X] : Could you please take a look and verify that integration tests now run as qsmaster , and not MySQL root?
            Hide
            npease Nate Pease [X] (Inactive) added a comment -

            Fritz Mueller I've spent about 20 minutes on this and honestly can't tell. My explorations took me some into the configuration system, trying to understand it was worse than navigating in Boston. I'd like to come back to this in January when we can work on this together.

            Show
            npease Nate Pease [X] (Inactive) added a comment - Fritz Mueller I've spent about 20 minutes on this and honestly can't tell. My explorations took me some into the configuration system, trying to understand it was worse than navigating in Boston. I'd like to come back to this in January when we can work on this together.

              People

              Assignee:
              fritzm Fritz Mueller
              Reporter:
              jammes Fabrice Jammes
              Watchers:
              Andy Salnikov, Fabrice Jammes, Fritz Mueller, Jacek Becla, Nate Pease [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.