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

cat test timeFuncs.py requires lsst10 connection and credentials

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: cat
    • Labels:
      None
    • Team:
      Data Access and Database

      Description

      This test should only run if appropriate credentials are in place and perhaps only if the host is at NCSA.

        Attachments

          Activity

          Hide
          ktl Kian-Tat Lim added a comment - - edited

          Implemented check in tickets/DM-1637, 5d9492. Buildbot run was 2317, b528.

          Show
          ktl Kian-Tat Lim added a comment - - edited Implemented check in tickets/ DM-1637 , 5d9492. Buildbot run was 2317, b528.
          Hide
          jbecla Jacek Becla added a comment -

          I added 2 minor comments on github. Thanks for implementing it.

          Show
          jbecla Jacek Becla added a comment - I added 2 minor comments on github. Thanks for implementing it.
          Hide
          jbecla Jacek Becla added a comment -

          I do wonder though.. why do we want to restrict it to ncsa machines only? E.g., I often tend to run tests from slac machines or laptop at home, and I connect to lsst10 using credential files. Is that not recommended?

          Show
          jbecla Jacek Becla added a comment - I do wonder though.. why do we want to restrict it to ncsa machines only? E.g., I often tend to run tests from slac machines or laptop at home, and I connect to lsst10 using credential files. Is that not recommended?
          Hide
          ktl Kian-Tat Lim added a comment -

          The point is that this test is testing code that runs in the database instance. As long as it is tested by buildbot in one place, it doesn't need to be tested by everyone else on that same exact database server.

          Show
          ktl Kian-Tat Lim added a comment - The point is that this test is testing code that runs in the database instance. As long as it is tested by buildbot in one place, it doesn't need to be tested by everyone else on that same exact database server.
          Hide
          ktl Kian-Tat Lim added a comment -

          Merged to master in 5d09534.

          Show
          ktl Kian-Tat Lim added a comment - Merged to master in 5d09534.

            People

            • Assignee:
              fritzm Fritz Mueller
              Reporter:
              ktl Kian-Tat Lim
              Reviewers:
              Jacek Becla
              Watchers:
              Jacek Becla, John Swinbank, Kian-Tat Lim
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: