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

Reject the Following Connection Request in Rotator Controller

    XMLWordPrintable

Details

    Description

      Allow the controller to be able to reject all the following connection request. By doing this, once there are no connections from TCP/IP clients anymore, the controller software will execute to end. The wrapper service in PXI controller will rerun the control software. The benefit of this is to clean all the drive related information contains the faults.

      This task will need to update the EUI and controller code.

      Attachments

        Issue Links

          Activity

            ttsai Te-Wei Tsai added a comment -

            Since I will unify the code of GUI and DDS (which means both of them allow the multiple connections), the reboot is fine to me if the CSC user does not care about the CSC will enter the Fault state after the EUI user decides to reboot the system. To be honest, I think the main arguments here is the design logic instead of the risks in the above comments.

            ttsai Te-Wei Tsai added a comment - Since I will unify the code of GUI and DDS (which means both of them allow the multiple connections), the reboot is fine to me if the CSC user does not care about the CSC will enter the Fault state after the EUI user decides to reboot the system. To be honest, I think the main arguments here is the design logic instead of the risks in the above comments.
            ttsai Te-Wei Tsai added a comment -

            After the discussion with Russell, we agreed to change the implementation to disconnect the CSC connection directly.

            ttsai Te-Wei Tsai added a comment - After the discussion with Russell, we agreed to change the implementation to disconnect the CSC connection directly.
            ttsai Te-Wei Tsai added a comment -

            The GUI will prompt a new window to confirm the reboot of controller:

            ttsai Te-Wei Tsai added a comment - The GUI will prompt a new window to confirm the reboot of controller:
            ttsai Te-Wei Tsai added a comment -

            Please see the update of PR. Thanks!

            ttsai Te-Wei Tsai added a comment - Please see the update of PR. Thanks!

            Reviewed on github. Looks very nice. I'm impressed with how simple you were able to make it.

            rowen Russell Owen added a comment - Reviewed on github. Looks very nice. I'm impressed with how simple you were able to make it.

            People

              ttsai Te-Wei Tsai
              ttsai Te-Wei Tsai
              Russell Owen
              Russell Owen, Te-Wei Tsai
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.