Github user ahgittin commented on a diff in the pull request:
https://github.com/apache/brooklyn-library/pull/126#discussion_r142702860
--- Diff:
qa/src/test/java/org/apache/brooklyn/qa/load/AbstractLoadTest.java ---
@@ -161,8 +161,8 @@ protected ManagementContext setUpPlatform() {
// Create management node
persistenceDir = Files.createTempDir();
launcher = BrooklynLauncher.newInstance()
- .persistMode(PersistMode.CLEAN)
- .highAvailabilityMode(HighAvailabilityMode.MASTER)
+ .persistMode(doPersistence() ? PersistMode.CLEAN :
PersistMode.DISABLED)
+ .highAvailabilityMode(doPersistence() ?
HighAvailabilityMode.MASTER : HighAvailabilityMode.DISABLED)
--- End diff --
due to how tests are structured and annotated methods run by the framework,
methods for this sort of thing are easier to work with i find. happy if you
want to spike an alternative with fields but i doubt it's better.
---
