Skip to content

Commit f55d45a

Browse files
authored
[improve][ci] Move replication tests to new group Broker Group 5 in Pulsar CI (#24917)
1 parent d20eb2e commit f55d45a

25 files changed

Lines changed: 31 additions & 23 deletions

.github/workflows/pulsar-ci.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -318,6 +318,8 @@ jobs:
318318
group: BROKER_GROUP_3
319319
- name: Brokers - Broker Group 4
320320
group: BROKER_GROUP_4
321+
- name: Brokers - Broker Group 5
322+
group: BROKER_GROUP_5
321323
- name: Brokers - Client Api
322324
group: BROKER_CLIENT_API
323325
- name: Brokers - Client Impl

build/run_unit_group.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ function test_group_broker_group_1() {
8282
}
8383

8484
function test_group_broker_group_2() {
85-
mvn_test -pl pulsar-broker -Dgroups='schema,utils,functions-worker,broker-io,broker-discovery,broker-compaction,broker-naming,broker-replication,websocket,other'
85+
mvn_test -pl pulsar-broker -Dgroups='schema,utils,functions-worker,broker-io,broker-discovery,broker-compaction,broker-naming,websocket,other'
8686
}
8787

8888
function test_group_broker_group_3() {
@@ -95,6 +95,10 @@ function test_group_broker_group_4() {
9595
mvn_test -pl pulsar-broker -Dgroups='cluster-migration'
9696
}
9797

98+
function test_group_broker_group_5() {
99+
mvn_test -pl pulsar-broker -Dgroups='broker-replication'
100+
}
101+
98102
function test_group_broker_client_api() {
99103
mvn_test -pl pulsar-broker -Dgroups='broker-api'
100104
}

pulsar-broker/src/test/java/org/apache/pulsar/broker/SameAuthParamsLookupAutoClusterFailoverTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242
import org.testng.annotations.DataProvider;
4343
import org.testng.annotations.Test;
4444

45+
@Test(groups = "broker-replication")
4546
public class SameAuthParamsLookupAutoClusterFailoverTest extends OneWayReplicatorTestBase {
4647

4748
public void setup() throws Exception {

pulsar-broker/src/test/java/org/apache/pulsar/broker/service/AbstractReplicatorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@
5151
import org.testng.Assert;
5252
import org.testng.annotations.Test;
5353

54-
@Test(groups = "broker")
54+
@Test(groups = "broker-replication")
5555
public class AbstractReplicatorTest {
5656

5757
@Test

pulsar-broker/src/test/java/org/apache/pulsar/broker/service/DisabledCreateTopicToRemoteClusterForReplicationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
import org.testng.annotations.Test;
4545

4646
@Slf4j
47-
@Test(groups = "broker")
47+
@Test(groups = "broker-replication")
4848
public class DisabledCreateTopicToRemoteClusterForReplicationTest extends OneWayReplicatorTestBase {
4949

5050
@Override

pulsar-broker/src/test/java/org/apache/pulsar/broker/service/OneWayReplicatorDeduplicationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@
9191
import org.testng.annotations.Test;
9292

9393
@Slf4j
94-
@Test(groups = "broker")
94+
@Test(groups = "broker-replication")
9595
public class OneWayReplicatorDeduplicationTest extends OneWayReplicatorTestBase {
9696

9797
static final ObjectMapper JACKSON = new ObjectMapper();

pulsar-broker/src/test/java/org/apache/pulsar/broker/service/OneWayReplicatorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@
124124
import org.testng.annotations.Test;
125125

126126
@Slf4j
127-
@Test(groups = "broker")
127+
@Test(groups = "broker-replication")
128128
public class OneWayReplicatorTest extends OneWayReplicatorTestBase {
129129

130130
@Override

pulsar-broker/src/test/java/org/apache/pulsar/broker/service/OneWayReplicatorUsingGlobalPartitionedTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@
4949
import org.testng.annotations.Test;
5050

5151
@Slf4j
52-
@Test(groups = "broker")
52+
@Test(groups = "broker-replication")
5353
public class OneWayReplicatorUsingGlobalPartitionedTest extends OneWayReplicatorTest {
5454

5555
@Override

pulsar-broker/src/test/java/org/apache/pulsar/broker/service/OneWayReplicatorUsingGlobalZKTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252
import org.testng.annotations.Test;
5353

5454
@Slf4j
55-
@Test(groups = "broker")
55+
@Test(groups = "broker-replication")
5656
public class OneWayReplicatorUsingGlobalZKTest extends OneWayReplicatorTest {
5757

5858
@Override

pulsar-broker/src/test/java/org/apache/pulsar/broker/service/PeerReplicatorTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
import org.testng.annotations.DataProvider;
4545
import org.testng.annotations.Test;
4646

47-
@Test(groups = "broker")
47+
@Test(groups = "broker-replication")
4848
public class PeerReplicatorTest extends ReplicatorTestBase {
4949

5050
@Override
@@ -187,7 +187,7 @@ public void testGetPeerClusters() throws Exception {
187187
*
188188
* @throws Exception
189189
*/
190-
@Test(groups = "broker")
190+
@Test
191191
public void testPeerClusterInReplicationClusterListChange() throws Exception {
192192

193193
// clean up peer-clusters

0 commit comments

Comments
 (0)