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

MTMount OSS telemetry topic causes Java too many parameters error

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Epic Link:
    • Sprint:
      TSSW Sprint - Nov 25 - Dec 6, TSSW Sprint - Dec 9 - Dec 20, TSSW Sprint - Dec 23 - Jan 3, TSSW Sprint - Jan 06 - Jan 17, TSSW Sprint - Jan 21 - Feb 01
    • Team:
      Telescope and Site

      Description

      This is a ticket to track the Java "too many parameters" error in the MtMount OSS telemetry topic.  This issue will probably go away when the vendor updates the topics.  QA needs the ticket to properly shutdown the tests that are affected.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          rbovill Rob Bovill added a comment -

          This is still an issue, so I'm making this comment to ensure this isn't forgotten.

          Show
          rbovill Rob Bovill added a comment - This is still an issue, so I'm making this comment to ensure this isn't forgotten.
          Hide
          Permalink
          aheyer Andrew Heyer [X] (Inactive) added a comment -

          Thanks Rob, I will look into this within the next few days. 

          Show
          aheyer Andrew Heyer [X] (Inactive) added a comment - Thanks Rob, I will look into this within the next few days. 
          Hide
          Permalink
          aheyer Andrew Heyer [X] (Inactive) added a comment -

          Hi Rob, PR created here https://github.com/lsst-ts/ts_xml/pull/22

          It seems that the only test that wasn't passing was a unit field Amp. I also rebased and removed MTMount commands being skipped. 

          I recall there being many more issues when I initially looked at it, now I think I was mistaken?

          Show
          aheyer Andrew Heyer [X] (Inactive) added a comment - Hi Rob, PR created here https://github.com/lsst-ts/ts_xml/pull/22 It seems that the only test that wasn't passing was a unit field Amp. I also rebased and removed MTMount commands being skipped.  I recall there being many more issues when I initially looked at it, now I think I was mistaken?
          Hide
          Permalink
          rbovill Rob Bovill added a comment -

          Please fix the <Unit> fields in the Telemetry XML. It would also be good to update ts_xml/tests/test_Units.py to catch this condition.

          Show
          rbovill Rob Bovill added a comment - Please fix the <Unit> fields in the Telemetry XML. It would also be good to update ts_xml/tests/test_Units.py to catch this condition.
          Hide
          Permalink
          aheyer Andrew Heyer [X] (Inactive) added a comment -

          added a test to catch for blank spaces and unit attributes have been fixed.

          Show
          aheyer Andrew Heyer [X] (Inactive) added a comment - added a test to catch for blank spaces and unit attributes have been fixed.
          Hide
          Permalink
          rbovill Rob Bovill added a comment -

          MTMount XML changes approved. Tests now passing. Updates to Unit tests are also good. See https://github.com/lsst-ts/ts_xml/pull/224 for details.

          Show
          rbovill Rob Bovill added a comment - MTMount XML changes approved. Tests now passing. Updates to Unit tests are also good. See https://github.com/lsst-ts/ts_xml/pull/224 for details.

            People

            Assignee:
            aheyer Andrew Heyer [X] (Inactive)
            Reporter:
            rbovill Rob Bovill
            Reviewers:
            Rob Bovill
            Watchers:
            Andrew Heyer [X] (Inactive), Rob Bovill
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.