ICEpush
  1. ICEpush
  2. PUSH-372

Add startup logging for ICEpush configuration parameters

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1, EE-3.3.0.GA_P04
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEpush
    • Assignee Priority:
      P2

      Description

      It would be useful to have ICEpush log it's configuration parameters in the same manner that ICEfaces does at application startup.

        Activity

        Ken Fyten created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Fix Version/s 4.1 [ 11570 ]
        Fix Version/s EE-3.3.0.GA_P04 [ 12271 ]
        Assignee Priority P2 [ 10011 ]
        Ken Fyten made changes -
        Assignee Mircea Toma [ mircea.toma ] Ken Fyten [ ken.fyten ]
        Hide
        Ken Fyten added a comment -

        Completed on IF 4 trunk.

        Output now appears as follows in the server log at init of ICEpush lib:

        29-Oct-2015 16:50:59.808 INFO [http-nio-8080-exec-9] org.icepush.ConfigurationServer.<init> ICEpush Configuration: 
        org.icepush.cloudPushIdTimeout = 1800000 [default]
        org.icepush.contextPath: null [default]
        org.icepush.disableRemoteHostLookup: false [default]
        org.icepush.emptyResponseRetries = 3 [default]
        org.icepush.groupTimeout = 120000 [default]
        org.icepush.networkErrorRetryTimeouts: 1 1 1 2 2 3 [default]
        org.icepush.heartbeatTimeout = 15000 [default]
        org.icepush.notificationQueueSize = 1000 [default]
        org.icepush.notificationStormLoopInterval = 275 [default]
        org.icepush.notificationStormMaximumRequests = 25 [default]
        org.icepush.notificationStormBackOffInterval = -1 [default]
        org.icepush.pushIdTimeout = 120000 [default]
        org.icepush.serverErrorRetryTimeouts: 1000 2000 4000 [default]
        org.icepush.useAsyncContext: true [default] 

        With defaults overridden:

        29-Oct-2015 16:46:52.580 INFO [http-nio-8080-exec-4] org.icepush.ConfigurationServer.<init> ICEpush Configuration: 
        org.icepush.cloudPushIdTimeout = 1000000
        org.icepush.contextPath: /pushpath
        org.icepush.disableRemoteHostLookup: true
        org.icepush.emptyResponseRetries = 2
        org.icepush.groupTimeout = 100000
        org.icepush.networkErrorRetryTimeouts: 1 2 3
        org.icepush.heartbeatTimeout = 20000
        org.icepush.notificationQueueSize = 500
        org.icepush.notificationStormLoopInterval = 500
        org.icepush.notificationStormMaximumRequests = 20
        org.icepush.notificationStormBackOffInterval = 500
        org.icepush.pushIdTimeout = 90000
        org.icepush.serverErrorRetryTimeouts: 5000 10000
        org.icepush.useAsyncContext: false 
        Show
        Ken Fyten added a comment - Completed on IF 4 trunk. Output now appears as follows in the server log at init of ICEpush lib: 29-Oct-2015 16:50:59.808 INFO [http-nio-8080-exec-9] org.icepush.ConfigurationServer.<init> ICEpush Configuration: org.icepush.cloudPushIdTimeout = 1800000 [ default ] org.icepush.contextPath: null [ default ] org.icepush.disableRemoteHostLookup: false [ default ] org.icepush.emptyResponseRetries = 3 [ default ] org.icepush.groupTimeout = 120000 [ default ] org.icepush.networkErrorRetryTimeouts: 1 1 1 2 2 3 [ default ] org.icepush.heartbeatTimeout = 15000 [ default ] org.icepush.notificationQueueSize = 1000 [ default ] org.icepush.notificationStormLoopInterval = 275 [ default ] org.icepush.notificationStormMaximumRequests = 25 [ default ] org.icepush.notificationStormBackOffInterval = -1 [ default ] org.icepush.pushIdTimeout = 120000 [ default ] org.icepush.serverErrorRetryTimeouts: 1000 2000 4000 [ default ] org.icepush.useAsyncContext: true [ default ] With defaults overridden: 29-Oct-2015 16:46:52.580 INFO [http-nio-8080-exec-4] org.icepush.ConfigurationServer.<init> ICEpush Configuration: org.icepush.cloudPushIdTimeout = 1000000 org.icepush.contextPath: /pushpath org.icepush.disableRemoteHostLookup: true org.icepush.emptyResponseRetries = 2 org.icepush.groupTimeout = 100000 org.icepush.networkErrorRetryTimeouts: 1 2 3 org.icepush.heartbeatTimeout = 20000 org.icepush.notificationQueueSize = 500 org.icepush.notificationStormLoopInterval = 500 org.icepush.notificationStormMaximumRequests = 20 org.icepush.notificationStormBackOffInterval = 500 org.icepush.pushIdTimeout = 90000 org.icepush.serverErrorRetryTimeouts: 5000 10000 org.icepush.useAsyncContext: false
        Hide
        Ken Fyten added a comment -

        For future testing convenience, the following configuration parameters can be added to a web.xml to override the ICEpush defaults:

            <context-param>
                <param-name>org.icepush.cloudPushIdTimeout</param-name>
                <param-value>1000000</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.contextPath</param-name>
                <param-value>pushpath</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.disableRemoteHostLookup</param-name>
                <param-value>true</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.emptyResponseRetries</param-name>
                <param-value>2</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.groupTimeout</param-name>
                <param-value>100000</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.networkErrorRetryTimeouts</param-name>
                <param-value>1 2 3</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.heartbeatTimeout</param-name>
                <param-value>20000</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.notificationQueueSize</param-name>
                <param-value>500</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.notificationStormLoopInterval</param-name>
                <param-value>500</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.notificationStormMaximumRequests</param-name>
                <param-value>20</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.notificationStormBackOffInterval</param-name>
                <param-value>500</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.pushIdTimeout</param-name>
                <param-value>90000</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.serverErrorRetryTimeouts</param-name>
                <param-value>5000 10000</param-value>
            </context-param>
            <context-param>
                <param-name>org.icepush.useAsyncContext</param-name>
                <param-value>false</param-value>
            </context-param> 
        Show
        Ken Fyten added a comment - For future testing convenience, the following configuration parameters can be added to a web.xml to override the ICEpush defaults: <context-param> <param-name>org.icepush.cloudPushIdTimeout</param-name> <param-value>1000000</param-value> </context-param> <context-param> <param-name>org.icepush.contextPath</param-name> <param-value>pushpath</param-value> </context-param> <context-param> <param-name>org.icepush.disableRemoteHostLookup</param-name> <param-value> true </param-value> </context-param> <context-param> <param-name>org.icepush.emptyResponseRetries</param-name> <param-value>2</param-value> </context-param> <context-param> <param-name>org.icepush.groupTimeout</param-name> <param-value>100000</param-value> </context-param> <context-param> <param-name>org.icepush.networkErrorRetryTimeouts</param-name> <param-value>1 2 3</param-value> </context-param> <context-param> <param-name>org.icepush.heartbeatTimeout</param-name> <param-value>20000</param-value> </context-param> <context-param> <param-name>org.icepush.notificationQueueSize</param-name> <param-value>500</param-value> </context-param> <context-param> <param-name>org.icepush.notificationStormLoopInterval</param-name> <param-value>500</param-value> </context-param> <context-param> <param-name>org.icepush.notificationStormMaximumRequests</param-name> <param-value>20</param-value> </context-param> <context-param> <param-name>org.icepush.notificationStormBackOffInterval</param-name> <param-value>500</param-value> </context-param> <context-param> <param-name>org.icepush.pushIdTimeout</param-name> <param-value>90000</param-value> </context-param> <context-param> <param-name>org.icepush.serverErrorRetryTimeouts</param-name> <param-value>5000 10000</param-value> </context-param> <context-param> <param-name>org.icepush.useAsyncContext</param-name> <param-value> false </param-value> </context-param>
        Ken Fyten made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #46154 Thu Oct 29 16:55:20 MDT 2015 ken.fyten PUSH-372 - Log ICEpush config. params at startup.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/PushStormDetectionServer.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/Configuration.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/ConfigurationServer.java
        Hide
        Ken Fyten added a comment -

        Committed to icefaces 3.3 maintenance branch:

        [10:56:30] - svn commit -m "PUSH-372 - Log ICEpush config. params at startup." 1 packet
        [10:56:32] - Sending        /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/Configuration.java
        [10:56:32] - Sending        /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java
        [10:56:32] - Sending        /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/PushStormDetectionServer.java
        [10:56:32] - Sending        /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/servlet/ConfigurationServlet.java
        [10:56:36] - Committed revision 46155. 
        Show
        Ken Fyten added a comment - Committed to icefaces 3.3 maintenance branch: [10:56:30] - svn commit -m "PUSH-372 - Log ICEpush config. params at startup." 1 packet [10:56:32] - Sending /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/Configuration.java [10:56:32] - Sending /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/LocalPushGroupManager.java [10:56:32] - Sending /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/PushStormDetectionServer.java [10:56:32] - Sending /Code/svn/icepush-core-3.3-maint/icepush/core/src/main/java/org/icepush/servlet/ConfigurationServlet.java [10:56:36] - Committed revision 46155.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #46157 Fri Oct 30 13:50:48 MDT 2015 ken.fyten PUSH-372 - Log ICEpush config. params at startup.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/Configuration.java
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/ConfigurationServer.java
        Hide
        Ken Fyten added a comment -

        Added ICEpush EE configuration parameter logging as well to both the icepush-ee-3.3-maintenance branch and the icepush-ee-trunk:

        [13:44:06] - svn commit -m "PUSH-372 - Log ICEpush EE config. params at startup." 1 packet
        [13:44:07] - Sending        /Code/svn/icepush-ee-maintenance/icepush-ee-3.3.0.GA-maintenance/icepush-ee/core-ee/src/main/java/com/icesoft/icepush/AdjustHeartbeatServer.java
        [13:44:08] - Sending        /Code/svn/icepush-ee-maintenance/icepush-ee-3.3.0.GA-maintenance/icepush-ee/core-ee/src/main/java/com/icesoft/icepush/BrowserBoundServlet.java
        [13:44:10] - Committed revision 43401. 
        [13:55:41] - svn commit -m "PUSH-372 - Log ICEpush EE config. params at startup." 1 packet
        [13:55:42] - Sending        /Code/svn/icepush-ee-trunk/icepush-ee/core-ee/src/main/java/com/icesoft/icepush/BrowserBoundServlet.java
        [13:55:43] - Committed revision 43402. 

        This logging appears as follows when ICEpush EE is initialized:

        30-Oct-2015 13:41:11.265 INFO [http-nio-8080-exec-1] com.icesoft.icepush.BrowserBoundServlet.createBlockingConnectionServer ICEpush EE Configuration: 
        com.icesoft.icepush.minHeartbeatInterval = 5000
        com.icesoft.icepush.maxHeartbeatInterval = 50000  
        Show
        Ken Fyten added a comment - Added ICEpush EE configuration parameter logging as well to both the icepush-ee-3.3-maintenance branch and the icepush-ee-trunk: [13:44:06] - svn commit -m "PUSH-372 - Log ICEpush EE config. params at startup." 1 packet [13:44:07] - Sending /Code/svn/icepush-ee-maintenance/icepush-ee-3.3.0.GA-maintenance/icepush-ee/core-ee/src/main/java/com/icesoft/icepush/AdjustHeartbeatServer.java [13:44:08] - Sending /Code/svn/icepush-ee-maintenance/icepush-ee-3.3.0.GA-maintenance/icepush-ee/core-ee/src/main/java/com/icesoft/icepush/BrowserBoundServlet.java [13:44:10] - Committed revision 43401. [13:55:41] - svn commit -m "PUSH-372 - Log ICEpush EE config. params at startup." 1 packet [13:55:42] - Sending /Code/svn/icepush-ee-trunk/icepush-ee/core-ee/src/main/java/com/icesoft/icepush/BrowserBoundServlet.java [13:55:43] - Committed revision 43402. This logging appears as follows when ICEpush EE is initialized: 30-Oct-2015 13:41:11.265 INFO [http-nio-8080-exec-1] com.icesoft.icepush.BrowserBoundServlet.createBlockingConnectionServer ICEpush EE Configuration: com.icesoft.icepush.minHeartbeatInterval = 5000 com.icesoft.icepush.maxHeartbeatInterval = 50000
        Ken Fyten made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #46901 Fri Dec 18 16:37:37 MST 2015 ken.fyten PUSH-372 - Made config logging only happen once per application session, instead of once per user-session.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/ConfigurationServer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #46902 Fri Dec 18 16:39:27 MST 2015 ken.fyten PUSH-372 - Made config logging only happen once per application session, instead of once per user-session.
        Files Changed
        Commit graph MODIFY /icepush/tags/icepush-core-4.1.0/icepush/core/src/main/java/org/icepush/ConfigurationServer.java
        Hide
        Carmen Cristurean added a comment -

        Verified ICEfaces-4.1.0 Jenkins Build# 4 / IE11, FF41, Chrome47 using auction and showcase > ProgressBar >Push.

        Show
        Carmen Cristurean added a comment - Verified ICEfaces-4.1.0 Jenkins Build# 4 / IE11, FF41, Chrome47 using auction and showcase > ProgressBar >Push.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Ken Fyten
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: