Skip to content

Added tests to use script scaleCluster.sh/ClusterStatus.sh #2350

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 7, 2021
Merged

Added tests to use script scaleCluster.sh/ClusterStatus.sh #2350

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a29c07f
Added tests to use script scaleCluster.sh/ClusterStatus.sh
Apr 30, 2021
9e3476a
Changes based on the comments
May 5, 2021
5f69ec7
Added more desriptions
May 6, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
110 changes: 109 additions & 1 deletion integration-tests/src/test/java/oracle/weblogic/kubernetes/ItServerStartPolicy.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
import static oracle.weblogic.kubernetes.utils.CommonPatchTestUtils.patchServerStartPolicy;
import static oracle.weblogic.kubernetes.utils.CommonTestUtils.checkClusterReplicaCountMatches;
import static oracle.weblogic.kubernetes.utils.CommonTestUtils.checkPodDeleted;
import static oracle.weblogic.kubernetes.utils.CommonTestUtils.checkPodDoesNotExist;
import static oracle.weblogic.kubernetes.utils.CommonTestUtils.checkPodInitializing;
import static oracle.weblogic.kubernetes.utils.CommonTestUtils.checkPodReadyAndServiceExists;
import static oracle.weblogic.kubernetes.utils.CommonTestUtils.createConfigMapAndVerify;
Expand All @@ -89,7 +90,6 @@
* managed server. The replica count is set to 1 and serverStartPolicy is set
* to IF_NEEDED at managed server level.
*/

@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
@DisplayName("ServerStartPolicy attribute in different levels in a MII domain")
@IntegrationTest
Expand All @@ -104,6 +104,8 @@ class ItServerStartPolicy {
public static final String START_CLUSTER_SCRIPT = "startCluster.sh";
public static final String STOP_DOMAIN_SCRIPT = "stopDomain.sh";
public static final String START_DOMAIN_SCRIPT = "startDomain.sh";
public static final String SCALE_CLUSTER_SCRIPT = "scaleCluster.sh";
public static final String STATUS_CLUSTER_SCRIPT = "clusterStatus.sh";
public static final String managedServerNamePrefix = "managed-server";
public static final String CLUSTER_1 = "cluster-1";
public static final String CLUSTER_2 = "cluster-2";
Expand Down Expand Up @@ -1130,6 +1132,107 @@ public void testRestartingMSWithExplicitServerStartStateWhileVaryingReplicaCount
logger.info("managed server " + serverName + " restarted successfully.");
}

/**
* Scale the configured cluster using the sample script scaleCluster.sh script
* Verify that server(s) in the configured cluster are scaled up and in RUNNING state.
* Verify that server(s) in the dynamic cluster are not affected.
* Restore the env using the sample script stopServer.sh.
*/
@Order(19)
@Test
@DisplayName("Scale the configured cluster with scaleCluster.sh script")
public void testConfigClusterScale() {
int newReplicaCount = 2;
String configServerName = "config-cluster-server" + newReplicaCount;
String configServerPodName = domainUid + "-" + configServerName;
String dynamicServerPodName = domainUid + "-managed-server" + newReplicaCount;

// use clusterStatus.sh to make sure the server-to-be-test doesn't exist
String regex = ".*" + CLUSTER_1 + "(\\s+)1(\\s+)5(\\s+)1(\\s+)1(\\s+)1";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What call returns the formatted data given above? I can see how the regex matches this, but this seems really brittle.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see it's from clusterStatus.sh... Okay, I guess that's what we have to test.

scalingClusters(CLUSTER_1, dynamicServerPodName, replicaCount, regex, false);
regex = ".*" + CLUSTER_2 + "(\\s+)0(\\s+)2(\\s+)1(\\s+)1(\\s+)1";
scalingClusters(CLUSTER_2, configServerPodName, replicaCount, regex, false);

// use scaleCluster.sh to scale a dynamic cluster and
// use clusterStatus.sh to verify scaling results
regex = ".*" + CLUSTER_2 + "(\\s+)0(\\s+)2(\\s+)2(\\s+)2(\\s+)2";
scalingClusters(CLUSTER_2, configServerPodName, newReplicaCount, regex, true);

// check managed server from other cluster are not affected
logger.info("Check dynamic managed server pods are not affected");
assertDoesNotThrow(() -> assertTrue(checkClusterReplicaCountMatches(CLUSTER_1,
domainUid, domainNamespace, replicaCount)));
checkPodDoesNotExist(dynamicServerPodName, domainUid, domainNamespace);

// use clusterStatus.sh to restore test env
regex = ".*" + CLUSTER_2 + "(\\s+)0(\\s+)2(\\s+)1(\\s+)1(\\s+)1";
scalingClusters(CLUSTER_2, configServerPodName, replicaCount, regex, false);
}

/**
* Scale the dynamic cluster using the sample script scaleCluster.sh script
* Verify that server(s) in the dynamic cluster are scaled up and in RUNNING state.
* Verify that server(s) in the configured cluster are not affected.
* Restore the env using the sample script stopServer.sh.
*/
@Order(20)
@Test
@DisplayName("Scale the dynamic cluster with scaleCluster.sh script")
public void testDynamicClusterScale() {
int newReplicaCount = 2;
String dynamicServerName = "managed-server" + newReplicaCount;
String dynamicServerPodName = domainUid + "-" + dynamicServerName;
String configServerPodName = domainUid + "-config-cluster-server" + newReplicaCount;

// use clusterStatus.sh to make sure the server-to-be-test doesn't exist
String regex = ".*" + CLUSTER_1 + "(\\s+)1(\\s+)5(\\s+)1(\\s+)1(\\s+)1";
scalingClusters(CLUSTER_1, dynamicServerPodName, replicaCount, regex, false);
regex = ".*" + CLUSTER_2 + "(\\s+)0(\\s+)2(\\s+)1(\\s+)1(\\s+)1";
scalingClusters(CLUSTER_2, configServerPodName, replicaCount, regex, false);

// use scaleCluster.sh to scale a dynamic cluster and
// use clusterStatus.sh to verify scaling results
regex = ".*" + CLUSTER_1 + "(\\s+)1(\\s+)5(\\s+)2(\\s+)2(\\s+)2";
scalingClusters(CLUSTER_1, dynamicServerPodName, newReplicaCount, regex, true);

// check managed server from other cluster are not affected
logger.info("Check configured managed server pods are not affected");
assertDoesNotThrow(() -> assertTrue(checkClusterReplicaCountMatches(CLUSTER_2,
domainUid, domainNamespace, replicaCount)));
checkPodDoesNotExist(configServerPodName, domainUid, domainNamespace);

// use clusterStatus.sh to restore test env
regex = ".*" + CLUSTER_1 + "(\\s+)1(\\s+)5(\\s+)1(\\s+)1(\\s+)1";
scalingClusters(CLUSTER_1, dynamicServerPodName, replicaCount, regex, false);
}

private void scalingClusters(String clusterName, String serverPodName, int replicaNum,
String regex, boolean checkPodExist) {
// use scaleCluster.sh to scale a given cluster
logger.info("Scale cluster {0} using the script scaleCluster.sh", clusterName);
String result = assertDoesNotThrow(() ->
executeLifecycleScript(SCALE_CLUSTER_SCRIPT, CLUSTER_LIFECYCLE, clusterName, " -r " + replicaNum, false),
String.format("Failed to run %s", SCALE_CLUSTER_SCRIPT));

if (checkPodExist) {
checkPodReadyAndServiceExists(serverPodName, domainUid, domainNamespace);
} else {
checkPodDoesNotExist(serverPodName, domainUid, domainNamespace);
}

// verify that scaleCluster.sh does scale to a required replica number
assertDoesNotThrow(() -> assertTrue(checkClusterReplicaCountMatches(clusterName,
domainUid, domainNamespace, replicaNum)));

// use clusterStatus.sh to verify scaling results
result = assertDoesNotThrow(() ->
executeLifecycleScript(STATUS_CLUSTER_SCRIPT, CLUSTER_LIFECYCLE, clusterName),
String.format("Failed to run %s", STATUS_CLUSTER_SCRIPT));

assertTrue(verifyExecuteResult(result, regex), "The script should scale the given cluster: " + clusterName);
logger.info("The cluster {0} scaled successfully.", clusterName);
}

private static void createDomainSecret(String secretName, String username, String password, String domNamespace) {
Map<String, String> secretMap = new HashMap<>();
secretMap.put("username", username);
Expand Down Expand Up @@ -1284,6 +1387,7 @@ private String executeLifecycleScript(String script,
boolean checkResult,
String... args) {
String domainName = (args.length == 0) ? domainUid : args[0];

CommandParams params;
String commonParameters = " -d " + domainName + " -n " + domainNamespace;
params = new CommandParams().defaults();
Expand All @@ -1292,6 +1396,10 @@ private String executeLifecycleScript(String script,
+ Paths.get(domainLifecycleSamplePath.toString(), "/" + script).toString()
+ commonParameters + " -s " + entityName + " " + extraParams);
} else if (scriptType.equals(CLUSTER_LIFECYCLE)) {
if (extraParams.contains("-r")) {
commonParameters += " " + extraParams;
}

params.command("sh "
+ Paths.get(domainLifecycleSamplePath.toString(), "/" + script).toString()
+ commonParameters + " -c " + entityName);
Expand Down