Jetty

Implement integration tests for Deployment Manager JMX features

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 7.4.0
  • Fix Version/s: 7.4.2
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    1

Description

This enhancement implement integration tests to verify that Deployment Manager's JMX integration is working correctly.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael Gorovoy added a comment -

Implementation of the DeployMgrJmxTest

Show
Michael Gorovoy added a comment - Implementation of the DeployMgrJmxTest
Michael Gorovoy made changes -
Field Original Value New Value
Attachment jetty-1352.diff [ 54398 ]
Hide
Permalink
Michael Gorovoy added a comment -

Jesse, please review the patch when you get a chance.

Show
Michael Gorovoy added a comment - Jesse, please review the patch when you get a chance.
Michael Gorovoy made changes -
Assignee Michael Gorovoy [ mgorovoy ] Jesse McConnell [ jesse ]
Hide
Permalink
Michael Gorovoy added a comment -

Minor comment edit.

Show
Michael Gorovoy added a comment - Minor comment edit.
Michael Gorovoy made changes -
Attachment jetty-1352-1.diff [ 54432 ]
Michael Gorovoy made changes -
Attachment jetty-1352.diff [ 54398 ]
Hide
Permalink
Jesse McConnell added a comment -

pls merge with the JETTY-1353 test, will close that issue as both this patch and that one are now applied

Show
Jesse McConnell added a comment - pls merge with the JETTY-1353 test, will close that issue as both this patch and that one are now applied
Jesse McConnell made changes -
Assignee Jesse McConnell [ jesse ] Michael Gorovoy [ mgorovoy ]
Hide
Permalink
Michael Gorovoy added a comment - - edited

Potentially it is better to keep the two tests separate, because the JmxTest harness expects that the application is always remains in its original deployed state. Perhaps we could rename the Deployment Manager test to something generic, and use it for JMX tests that require that Jetty is restarted before each test runs.

Show
Michael Gorovoy added a comment - - edited Potentially it is better to keep the two tests separate, because the JmxTest harness expects that the application is always remains in its original deployed state. Perhaps we could rename the Deployment Manager test to something generic, and use it for JMX tests that require that Jetty is restarted before each test runs.
Jesse McConnell made changes -
Affects Version/s 7.4.0 [ 17302 ]
Affects Version/s 7.3.2 [ 17237 ]
Hide
Permalink
Michael Gorovoy added a comment -

Done.

Show
Michael Gorovoy added a comment - Done.
Michael Gorovoy made changes -
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 7.4.2 [ 17381 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Michael Gorovoy
    Reporter:
    Michael Gorovoy
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: