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

Provide the T2SA vendor with new code to exercise the system

    XMLWordPrintable

    Details

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

      Description

      Scott Sandwith requested new code to exercise our connection to their T2SA alignment system. They have old code provided by Colin, but will want something new if they are to make any changes to the API.

        Attachments

        1. cmdlineutil_3.zip
          11 kB
        2. cmdlineutil_4.zip
          11 kB
        3. cmdlineutil.zip
          19 kB

          Activity

          Hide
          rowen Russell Owen added a comment - - edited

          Code sent to Scott and Dave and attached to this ticket. (Code updated 2022-08-08 to remove an import of ts_tcpip).

          Show
          rowen Russell Owen added a comment - - edited Code sent to Scott and Dave and attached to this ticket. (Code updated 2022-08-08 to remove an import of ts_tcpip).
          Hide
          rowen Russell Owen added a comment -

          Please try out the code with the real T2SA and let me know if it works.

          Show
          rowen Russell Owen added a comment - Please try out the code with the real T2SA and let me know if it works.
          Hide
          dmills Dave Mills added a comment - - edited

          Looks mostly good to me, connected right away and prints responses ok.
          Did an exit and tried to reconnect, but T2SA had crashed on exit command

          Repeated the crash on exit so at least Scott can debug it now

          measure M1M3 returned a
          WARNING:root Took 17.96 seconds to read b'ACK-300 EMP
          and never printed the measurement result

          Show
          dmills Dave Mills added a comment - - edited Looks mostly good to me, connected right away and prints responses ok. Did an exit and tried to reconnect, but T2SA had crashed on exit command Repeated the crash on exit so at least Scott can debug it now measure M1M3 returned a WARNING:root Took 17.96 seconds to read b'ACK-300 EMP and never printed the measurement result
          Hide
          rowen Russell Owen added a comment - - edited

          Interesting.

          Regarding "but T2S had crashed on exit command": the exit command just makes this test code close the connection and quit. It does not send anything to the T2SA. So I wonder if the T2SA had just not realized that the connection was lost, and thus was not allowing a new connection? Trying to read a closed socket should immediately return a useful error, so the T2SA ought to be able to handle that quickly and reliably.

          Show
          rowen Russell Owen added a comment - - edited Interesting. Regarding "but T2S had crashed on exit command": the exit command just makes this test code close the connection and quit. It does not send anything to the T2SA. So I wonder if the T2SA had just not realized that the connection was lost, and thus was not allowing a new connection? Trying to read a closed socket should immediately return a useful error, so the T2SA ought to be able to handle that quickly and reliably.
          Hide
          rowen Russell Owen added a comment -

          Version 3 and 4 (attached) fix an incompatibility with Windows. Version 4 exits more cleanly.

          Show
          rowen Russell Owen added a comment - Version 3 and 4 (attached) fix an incompatibility with Windows. Version 4 exits more cleanly.

            People

            Assignee:
            rowen Russell Owen
            Reporter:
            rowen Russell Owen
            Reviewers:
            Dave Mills
            Watchers:
            Dave Mills, Russell Owen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.