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

DDS packages need some extra env vars to run tests

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: templates
    • Labels:
      None

      Description

      Packages that use OpenSplice DDS need some environment variables that scons does not normally see. Please consider adding the following to tests/SConscript for T&S packages:

      for name in ("OSPL_URI", "OPENSPLICE_LOC", "ADLINK_LICENSE"):
          val = os.environ.get(name)
          if val is not None:
              env.AppendENVPath(name, val)
      

      I consider this moderate priority since I keep getting bitten by it (pytest works and everything seems fine, but scons fails).

        Attachments

          Activity

          Hide
          jsick Jonathan Sick added a comment -

          Sounds good. I'll need to add a specific "T&S" axis to the package flavours listed on Slack, but I think that's workable.

          Show
          jsick Jonathan Sick added a comment - Sounds good. I'll need to add a specific "T&S" axis to the package flavours listed on Slack, but I think that's workable.
          Hide
          jsick Jonathan Sick added a comment -
          Show
          jsick Jonathan Sick added a comment - Hi Russell Owen , does https://github.com/lsst/templates/pull/57 meet your specifications? See specifically the example SConscript file: https://github.com/lsst/templates/pull/57/files#diff-e3bde17e98b4a3e8d849a6ea910c7300
          Hide
          rowen Russell Owen added a comment -

          Looks great! One minor suggestion on github. Thank you very much for adding this.

          Show
          rowen Russell Owen added a comment - Looks great! One minor suggestion on github. Thank you very much for adding this.
          Hide
          jsick Jonathan Sick added a comment -

          The new template is available from @sqrbot-jr on Slack now. Select "DDS Python" from the Flavor menu to use it, for example.

          Show
          jsick Jonathan Sick added a comment - The new template is available from @sqrbot-jr on Slack now. Select "DDS Python" from the Flavor menu to use it, for example.

            People

            • Assignee:
              jsick Jonathan Sick
              Reporter:
              rowen Russell Owen
              Reviewers:
              Russell Owen
              Watchers:
              Jonathan Sick, Russell Owen
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel