Skip to content

Commit b8c036e

Browse files
committed
Fix Aerospike Enterprise
1 parent 1d00cdd commit b8c036e

File tree

5 files changed

+9
-8
lines changed

5 files changed

+9
-8
lines changed

embedded-aerospike-enterprise/src/main/java/com/playtika/testcontainers/aerospike/enterprise/AerospikeEnterpriseConfigurer.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ public class AerospikeEnterpriseConfigurer {
1515
private final AerospikeProperties aerospikeProperties;
1616
private final AerospikeEnterpriseProperties enterpriseProperties;
1717

18-
public void configure(GenericContainer<?> aerospikeContainer) throws IOException, InterruptedException {
18+
public void configure(GenericContainer<?> aerospikeContainer) throws IOException, InterruptedException {
1919
if (aerospikeProperties.getFeatureKey() == null || aerospikeProperties.getFeatureKey().isBlank()) {
2020
log.warn("Evaluation feature key file not provided by 'embedded.aerospike.featureKey' property. " +
2121
"Pay attention to license details: https://github.com/aerospike/aerospike-server.docker/blob/master/enterprise/ENTERPRISE_LICENSE");
@@ -32,9 +32,9 @@ private void setupDisallowExpunge(GenericContainer<?> aerospikeContainer) throws
3232
String namespace = aerospikeProperties.getNamespace();
3333
Container.ExecResult result = aerospikeContainer.execInContainer("asadm", "-e",
3434
String.format("enable; manage config namespace %s param disallow-expunge to true", namespace));
35-
if (result.getStderr().length() > 0) {
35+
if (result.getExitCode() != 0) {
3636
throw new IllegalStateException("Failed to set up 'disallow-expunge' to true: " + result.getStderr());
3737
}
38-
log.info("Set up 'disallow-expunge' to true: {}", result.getStdout());
38+
log.info("Success setting up 'disallow-expunge' to true");
3939
}
4040
}

embedded-aerospike-enterprise/src/main/java/com/playtika/testcontainers/aerospike/enterprise/EnterpriseAerospikeTestOperationsAutoConfiguration.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,9 @@ public class EnterpriseAerospikeTestOperationsAutoConfiguration {
2121
@Bean
2222
@ConditionalOnProperty(value = "embedded.aerospike.time-travel.enabled", havingValue = "true", matchIfMissing = true)
2323
public ExpiredDocumentsCleaner expiredDocumentsCleaner(IAerospikeClient client,
24+
AerospikeEnterpriseProperties aerospikeEnterpriseProperties,
2425
AerospikeProperties properties) {
25-
return new AerospikeExpiredDocumentsCleaner(client, properties.getNamespace(), true);
26+
return new AerospikeExpiredDocumentsCleaner(client, properties.getNamespace(), aerospikeEnterpriseProperties.isDurableDeletes());
2627
}
2728

2829
}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
embedded.aerospike.dockerImage=aerospike/aerospike-server-enterprise:6.3.0.16
1+
embedded.aerospike.dockerImage=aerospike/aerospike-server-enterprise:7.0.0.3

embedded-aerospike-enterprise/src/test/java/com/playtika/testcontainers/aerospike/enterprise/ValidateEnterpriseAerospikeBootstrapConfigurationTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,13 +20,13 @@ void failOnNonEnterpriseImage() {
2020

2121
@Test
2222
void failOnUnsuitableEnterpriseImageVersion() {
23-
contextRunner.withPropertyValues("embedded.aerospike.dockerImage=aerospike/aerospike-server-enterprise:6.1.0.16_1")
23+
contextRunner.withPropertyValues("embedded.aerospike.dockerImage=aerospike/aerospike-server-enterprise:6.1.0.16")
2424
.run(context -> assertThat(context).hasFailed());
2525
}
2626

2727
@Test
2828
void skipValidation() {
29-
contextRunner.withPropertyValues("embedded.aerospike.dockerImage=aerospike-server:6.1.0.16_1",
29+
contextRunner.withPropertyValues("embedded.aerospike.dockerImage=aerospike-server:6.1.0.16",
3030
"embedded.aerospike.enabled=false")
3131
.run(context -> assertThat(context).hasNotFailed());
3232

embedded-aerospike-enterprise/src/test/resources/log4j2.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
</Console>
77
</Appenders>
88
<Loggers>
9-
<Logger name="com.playtika.testcontainer" level="info"/>
9+
<Logger name="com.playtika.testcontainers" level="info"/>
1010
<Logger name="org.springframework" level="error"/>
1111
<Root level="error">
1212
<AppenderRef ref="Console"/>

0 commit comments

Comments
 (0)