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

Create integration test case using data duplicator

    XMLWordPrintable

    Details

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

      Description

      Integration tests should provide a new test case which use sph-duplicate in partition package.

        Attachments

          Issue Links

            Activity

            Hide
            jbecla Jacek Becla added a comment -

            FYI, you need to rebase your qserv branch

            git checkout master
            git pull
            git checkout u/vaikunth/DM-1570
            git rebase master

            it rebases cleanly

            Show
            jbecla Jacek Becla added a comment - FYI, you need to rebase your qserv branch git checkout master git pull git checkout u/vaikunth/DM-1570 git rebase master it rebases cleanly
            Hide
            jbecla Jacek Becla added a comment - - edited

            regarding changes in qserv module:

            • please remove trailing spaces in code you committed. I saw it in admin/python/lsst/qserv/admin/dataDuplicator.py, didn't check elsewhere.
            • it'd be good to squash commits (do you know how to do that? If not, let's not mess around with it now)
            • update year in copyright lines for files you touched
            Show
            jbecla Jacek Becla added a comment - - edited regarding changes in qserv module: please remove trailing spaces in code you committed. I saw it in admin/python/lsst/qserv/admin/dataDuplicator.py, didn't check elsewhere. it'd be good to squash commits (do you know how to do that? If not, let's not mess around with it now) update year in copyright lines for files you touched
            Hide
            jbecla Jacek Becla added a comment -

            Looks like you are good to go now, K-T tweaked gitolite, so go ahead and do the merging.

            Show
            jbecla Jacek Becla added a comment - Looks like you are good to go now, K-T tweaked gitolite, so go ahead and do the merging.
            Hide
            jbecla Jacek Becla added a comment -

            all done, I updated headers, removed trailing spaces, squashed, and pushed things to the master (both qserv and qserv_testdata).

            Nice work Vaikunth! Thanks

            Show
            jbecla Jacek Becla added a comment - all done, I updated headers, removed trailing spaces, squashed, and pushed things to the master (both qserv and qserv_testdata). Nice work Vaikunth! Thanks
            Hide
            vaikunth Vaikunth Thukral added a comment -

            Great, thanks Jacek!

            Show
            vaikunth Vaikunth Thukral added a comment - Great, thanks Jacek!

              People

              Assignee:
              vaikunth Vaikunth Thukral
              Reporter:
              jammes Fabrice Jammes
              Reviewers:
              Jacek Becla
              Watchers:
              Fabrice Jammes, Jacek Becla, Serge Monkewitz, Vaikunth Thukral
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.