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

Conda env missing psutil

    XMLWordPrintable

    Details

    • Team:
      Architecture
    • Urgent?:
      No

      Description

      I have just noticed that psutil is not in our new conda env. The python_psutil dependency was removed from the utils table file but nothing replaced it. This means that the file leak testing has been disabled and we didn't notice.

      Please add it back to the env.

        Attachments

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-23835 Update environment with conda-forge third parties and conda compilers

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            bvan Brian Van Klaveren added a comment - - edited

            Does https://github.com/lsst/utils/blob/a8eba2281326a04fcaae180debc70eb5aa62ef68/python/lsst/utils/tests.py need to be modified?

            Should it assume psutil is installed?

            Show
            bvan Brian Van Klaveren added a comment - - edited Does https://github.com/lsst/utils/blob/a8eba2281326a04fcaae180debc70eb5aa62ef68/python/lsst/utils/tests.py need to be modified? Should it assume psutil is installed?
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            I suppose it can assume that. We didn't require it in the early days. I suppose a logging message if it's not installed would have helped but for now I don't object to making it mandatory.

            Show
            tjenness Tim Jenness added a comment - I suppose it can assume that. We didn't require it in the early days. I suppose a logging message if it's not installed would have helped but for now I don't object to making it mandatory.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Looks good to me. Minor comment on the ap_association ticket.

            Show
            tjenness Tim Jenness added a comment - Looks good to me. Minor comment on the ap_association ticket.

              People

              Assignee:
              bvan Brian Van Klaveren
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Tim Jenness
              Watchers:
              Brian Van Klaveren, Gabriele Comoretto [X] (Inactive), Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.