Uploaded image for project: 'Tools (JBoss Tools)'
  1. Tools (JBoss Tools)
  2. JBIDE-9277

AS adapter does not enable polling when start/shutdown commands are disabled

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 3.3.0.M2
    • Fix Version/s: 3.3.0.M3
    • Component/s: server
    • Labels:
      None
    • Steps to Reproduce:
      Hide

      EXEC: create server (AS7 for instance)
      EXEC: double click on it in 'Servers' view
      EXEC: Check 'Do not execute startup and shutdown commands'
      EXEC: Start server from 'Servers' view
      ASSERT: Server state should be 'Starting' since we have not started server from CLI before
      currently: Server is in 'Started' state, which is not true (and that is probably because HTTP poller is disabled)

      Show
      EXEC: create server (AS7 for instance) EXEC: double click on it in 'Servers' view EXEC: Check 'Do not execute startup and shutdown commands' EXEC: Start server from 'Servers' view ASSERT: Server state should be 'Starting' since we have not started server from CLI before currently: Server is in 'Started' state, which is not true (and that is probably because HTTP poller is disabled)
    • Release Notes Text:
      Hide
      Externally managed servers will not be launched or polled.
      The new text reads "Server is externally managed. Assume server is started."
      An "externally-managed" server has it's state set to 'started' automatically.
      An "externally-managed" server cannot be stopped or restarted.
      Show
      Externally managed servers will not be launched or polled. The new text reads "Server is externally managed. Assume server is started." An "externally-managed" server has it's state set to 'started' automatically. An "externally-managed" server cannot be stopped or restarted.

      Description

      See my comment

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            maxandersen Max Rydahl Andersen added a comment -

            >> If I before said that deploy only should always be in started mode then I'm stupid - I >>only remember deploy only mode having the problem of not being able to be started - it >>just sat there and you had to manually invoke publish.

            >This is not true. The deploy only server had an automatic-success start method, and you >demanded, for your demos, that this was unacceptable and that it must start in "started" >mode at all times since it's only purpose is to deploy. You said it hurt your demos to >have to explain why you create a deploy only server, and then "start" something that >doesn't actually "start". You then directed me to open a jira and change it. This was the >jira: https://issues.jboss.org/browse/JBIDE-8671

            Funny, when I read JBIDE-8671 I still read it as it should allow a Start/Stop mode and go into starting.

            But anyway - Deploy only server was a different beast without the nice "assumed started" feature we now got. We should align these things.

            >And honestly your point back then was completely valid. If a deploy-only server's only >job is to deploy, then it should begin in state 'started' and always be deploying, and if >the user does not want to deploy, then he should disable automatic publishing for that >server. But, as a deploy-only server is most-often used to deploy, it should default to >automatic publishing, and it can only do that when started.

            I disagree now that I have had a running server running on the directory deploy only server adapters goes against.

            >I'm still not exactly sure what you want, or if what you want is possible.

            It worked fine before this fix. The server adapter started deploying when I press Start, it stops deploying when I press stop. Easy and simple to explain.

            >You basically want two separate functions: having start/stop "connect or disconnect", or >having start/stop "actually start or stop" a server. However having a start/stop who's >only job is to "connect or disconnect" basically just means publish or not publish... >which is the exact same thing as turning off automatic publishing for that server... no?

            Turning off automatic publishing is cumbersome and doesn't work on a "running" server.

            And yes, I would prefer if WTP had some other "state" it can represent but it doesn't so we'll have to live within those limitations.

            For me the start/stop is start/stop of the server adapter; not the exact server. What start/stop does is then up to the server adapter to interpret.

            >If I'm wrong I'd love to hear more thoughts on it, but for now I'm still baffled as to >what exactly you're looking for.

            What was done before - that the AS server adapter is in "stopped/not-started" mode by default - that start button starts the server adapter (aka. automatic publishing and if managed server then start the server); and stop button stopts the server adapter (aka. automatic publishing stops and the if managed server then we gracefully attempt to stop the server)

            Show
            maxandersen Max Rydahl Andersen added a comment - >> If I before said that deploy only should always be in started mode then I'm stupid - I >>only remember deploy only mode having the problem of not being able to be started - it >>just sat there and you had to manually invoke publish. >This is not true. The deploy only server had an automatic-success start method, and you >demanded, for your demos, that this was unacceptable and that it must start in "started" >mode at all times since it's only purpose is to deploy. You said it hurt your demos to >have to explain why you create a deploy only server, and then "start" something that >doesn't actually "start". You then directed me to open a jira and change it. This was the >jira: https://issues.jboss.org/browse/JBIDE-8671 Funny, when I read JBIDE-8671 I still read it as it should allow a Start/Stop mode and go into starting. But anyway - Deploy only server was a different beast without the nice "assumed started" feature we now got. We should align these things. >And honestly your point back then was completely valid. If a deploy-only server's only >job is to deploy, then it should begin in state 'started' and always be deploying, and if >the user does not want to deploy, then he should disable automatic publishing for that >server. But, as a deploy-only server is most-often used to deploy, it should default to >automatic publishing, and it can only do that when started. I disagree now that I have had a running server running on the directory deploy only server adapters goes against. >I'm still not exactly sure what you want, or if what you want is possible. It worked fine before this fix. The server adapter started deploying when I press Start, it stops deploying when I press stop. Easy and simple to explain. >You basically want two separate functions: having start/stop "connect or disconnect", or >having start/stop "actually start or stop" a server. However having a start/stop who's >only job is to "connect or disconnect" basically just means publish or not publish... >which is the exact same thing as turning off automatic publishing for that server... no? Turning off automatic publishing is cumbersome and doesn't work on a "running" server. And yes, I would prefer if WTP had some other "state" it can represent but it doesn't so we'll have to live within those limitations. For me the start/stop is start/stop of the server adapter; not the exact server. What start/stop does is then up to the server adapter to interpret. >If I'm wrong I'd love to hear more thoughts on it, but for now I'm still baffled as to >what exactly you're looking for. What was done before - that the AS server adapter is in "stopped/not-started" mode by default - that start button starts the server adapter (aka. automatic publishing and if managed server then start the server); and stop button stopts the server adapter (aka. automatic publishing stops and the if managed server then we gracefully attempt to stop the server)
            Hide
            rob.stryker Rob Stryker added a comment -

            The committed patch has been reverted, and behaviour has been restored to what it was previously.

            Max has indicated he prefers the start / stop are quick and automatic when in this mode. This indicates to me that polling should NOT be attempted. If this is the case, it will need to be documented.

            Max: Please verify this is the expected behaviour and I can tag this as docmentation required, and close the issue out as "won't fix"

            Show
            rob.stryker Rob Stryker added a comment - The committed patch has been reverted, and behaviour has been restored to what it was previously. Max has indicated he prefers the start / stop are quick and automatic when in this mode. This indicates to me that polling should NOT be attempted. If this is the case, it will need to be documented. Max: Please verify this is the expected behaviour and I can tag this as docmentation required, and close the issue out as "won't fix"
            Hide
            maxandersen Max Rydahl Andersen added a comment -

            If the poller ui gets greyed out/disabled when using the "assume started mode" then this works for me.

            Show
            maxandersen Max Rydahl Andersen added a comment - If the poller ui gets greyed out/disabled when using the "assume started mode" then this works for me.
            Hide
            rob.stryker Rob Stryker added a comment -

            Greying out the pollers when server is ignoring launches =]

            Show
            rob.stryker Rob Stryker added a comment - Greying out the pollers when server is ignoring launches =]
            Hide
            lzoubek Libor Zoubek added a comment -

            verified on JBossAS Tools 2.3.0.v20110825-1600-H670-M3

            Show
            lzoubek Libor Zoubek added a comment - verified on JBossAS Tools 2.3.0.v20110825-1600-H670-M3

              People

              • Assignee:
                rob.stryker Rob Stryker
                Reporter:
                lzoubek Libor Zoubek
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development