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

improve message from qserv_testdata

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Qserv
    • Labels:
      None

      Description

      Currently, when I try to run qserv-benchmark but qserv_testdata was not setup, I am getting

      CRITICAL Unable to find tests datasets.
      --
      FOR EUPS USERS :
      Please run :
         eups distrib install qserv_testdata
         setup qserv_testdata
      FOR NON-EUPS USERS :
      Please fill 'testdata_dir' value in ~/.lsst/qserv.conf with the path of the directory containing tests datasets or use --testdata-dir option.

      It is important to note in the section for eups users that this has to be called BEFORE qserv is setup, otherwise it has no effect.

        Attachments

          Activity

          Hide
          jammes Fabrice Jammes added a comment -

          This message is no more useful as integration test code is now in qserv_testdata.
          I've removed it and made a few improvments related to the management of input test data directory in cli tools.

          Show
          jammes Fabrice Jammes added a comment - This message is no more useful as integration test code is now in qserv_testdata. I've removed it and made a few improvments related to the management of input test data directory in cli tools.
          Hide
          jbecla Jacek Becla added a comment -

          What should I review? I can't find any branch called "993".

          Show
          jbecla Jacek Becla added a comment - What should I review? I can't find any branch called "993".
          Hide
          jbecla Jacek Becla added a comment -

          all right, it is in qserv_testdata repo, I forgot to look there!

          Show
          jbecla Jacek Becla added a comment - all right, it is in qserv_testdata repo, I forgot to look there!
          Hide
          jbecla Jacek Becla added a comment -

          Looks fine, go ahead and merge to master.

          Show
          jbecla Jacek Becla added a comment - Looks fine, go ahead and merge to master.

            People

            Assignee:
            jammes Fabrice Jammes
            Reporter:
            fritzm Fritz Mueller
            Reviewers:
            Jacek Becla
            Watchers:
            Fabrice Jammes, Jacek Becla
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.