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

Port pipe_tasks to Python 3

    Details

    • Story Points:
      3
    • Sprint:
      Alert Production F16 - 9, Alert Production F16 - 10, Alert Production F16 - 11, Alert Production F16 - 11b, Alert Production F16 - 11c
    • Team:
      Alert Production

      Attachments

        Issue Links

          Activity

          Hide
          mrawls Meredith Rawls added a comment -

          I've gotten pipe_tasks and pex_config to pass py2 Jenkins by making a tickets/DM-7292 branch for pex_config and editing pex_config/listField per Tim Jenness' suggestion. However, the full build I ran failed on obs_subaru. Something I didn't touch, pex_config/dictField, is apparently causing a FieldValidationError when obs_subaru is tested. I can't tell if this problem arose indirectly from my small edit to pex_config/listField or not? It's strings all the way down. Here is the relevant Jenkins brain dump. Input appreciated.

          Show
          mrawls Meredith Rawls added a comment - I've gotten pipe_tasks and pex_config to pass py2 Jenkins by making a tickets/ DM-7292 branch for pex_config and editing pex_config/listField per Tim Jenness ' suggestion. However, the full build I ran failed on obs_subaru . Something I didn't touch, pex_config/dictField , is apparently causing a FieldValidationError when obs_subaru is tested. I can't tell if this problem arose indirectly from my small edit to pex_config/listField or not? It's strings all the way down. Here is the relevant Jenkins brain dump. Input appreciated.
          Hide
          tjenness Tim Jenness added a comment -

          My guess is that listField needs the same str translation fix.

          Show
          tjenness Tim Jenness added a comment - My guess is that listField needs the same str translation fix.
          Hide
          mrawls Meredith Rawls added a comment -

          I did put your str translation fix in listField, but it looks to me like dictField is written differently and doesn't explicitly use {{dtype}}s.

          Show
          mrawls Meredith Rawls added a comment - I did put your str translation fix in listField , but it looks to me like dictField is written differently and doesn't explicitly use {{dtype}}s.
          Hide
          mrawls Meredith Rawls added a comment -

          This went much deeper than it needed to... turns out there were still a couple from builtins import str lines in pipe_tasks that shouldn't have been there. They're gone now, and Jenkins is happy on py2 and py3! The tickets/DM-7292 branch of pex_config is identical to master and may be ignored. Nate Lust, please confirm you are OK with me going ahead and merging.

          Show
          mrawls Meredith Rawls added a comment - This went much deeper than it needed to... turns out there were still a couple from builtins import str lines in pipe_tasks that shouldn't have been there. They're gone now, and Jenkins is happy on py2 and py3! The tickets/ DM-7292 branch of pex_config is identical to master and may be ignored. Nate Lust , please confirm you are OK with me going ahead and merging.
          Hide
          mrawls Meredith Rawls added a comment -

          Discussed with Nate in Slack, fixed up a couple last tiny things, all good!

          Show
          mrawls Meredith Rawls added a comment - Discussed with Nate in Slack, fixed up a couple last tiny things, all good!

            People

            • Assignee:
              mrawls Meredith Rawls
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Nate Lust
              Watchers:
              Fred Moolekamp, John Parejko, John Swinbank, Lauren MacArthur, Meredith Rawls, Nate Lust, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: