Skip to content

Commit 2b5a41c

Browse files
authored
Port prs 3533, 3579 to main (#3664)
* port prs 3533,3579 to main
1 parent bd4a6ce commit 2b5a41c

File tree

3 files changed

+23
-7
lines changed

3 files changed

+23
-7
lines changed

integration-tests/src/test/java/oracle/weblogic/kubernetes/ItConfigDistributionStrategy.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -825,9 +825,11 @@ private void verifyResourceJDBC1Override(boolean configUpdated) {
825825
for (int i = 1; i <= replicaCount; i++) {
826826
appURI = "dsTest=true&dsName=" + dsName1 + "&" + "serverName=" + managedServerNameBase + i;
827827
String dsConnectionPoolTestUrl = baseUri + appURI;
828-
HttpResponse<String> response = assertDoesNotThrow(() -> OracleHttpClient.get(dsConnectionPoolTestUrl, true));
829-
assertEquals(200, response.statusCode(), "Status code not equals to 200");
830-
assertTrue(response.body().contains("Connection successful"), "Didn't get Connection successful");
828+
testUntil(() -> {
829+
HttpResponse<String> response = assertDoesNotThrow(() -> OracleHttpClient.get(dsConnectionPoolTestUrl, true));
830+
logger.info("Http response status code {0} \n Http response body {1} ", response.statusCode(), response.body());
831+
return response.statusCode() == 200 && response.body().contains("Connection successful");
832+
}, logger, "http response code 200 and message Connection successful");
831833
}
832834
}
833835

integration-tests/src/test/java/oracle/weblogic/kubernetes/actions/impl/Domain.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
import static oracle.weblogic.kubernetes.TestConstants.ADMIN_PASSWORD_DEFAULT;
3434
import static oracle.weblogic.kubernetes.TestConstants.ADMIN_SERVER_NAME_BASE;
3535
import static oracle.weblogic.kubernetes.TestConstants.ADMIN_USERNAME_DEFAULT;
36+
import static oracle.weblogic.kubernetes.TestConstants.OKD;
3637
import static oracle.weblogic.kubernetes.TestConstants.PROJECT_ROOT;
3738
import static oracle.weblogic.kubernetes.TestConstants.RESULTS_ROOT;
3839
import static oracle.weblogic.kubernetes.actions.ActionConstants.RBAC_API_GROUP;
@@ -57,6 +58,7 @@
5758
import static oracle.weblogic.kubernetes.utils.ThreadSafeLogger.getLogger;
5859
import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
5960
import static org.junit.jupiter.api.Assertions.assertFalse;
61+
import static org.junit.jupiter.api.Assertions.assertNotNull;
6062
import static org.junit.jupiter.api.Assertions.assertTrue;
6163

6264
public class Domain {
@@ -457,9 +459,11 @@ public static boolean scaleClusterWithRestApi(String domainUid,
457459
secretName, opServiceAccount, opNamespace, secretToken);
458460

459461
// decode the secret encoded token
460-
String decodedToken = new String(Base64.getDecoder().decode(secretToken));
462+
String decodedToken = OKD ? secretToken : new String(Base64.getDecoder().decode(secretToken));
461463
logger.info("Got decoded token for secret {0} associated with service account {1} in namespace {2}: {3}",
462464
secretName, opServiceAccount, opNamespace, decodedToken);
465+
466+
assertNotNull(decodedToken, "Couldn't get secret, token is null");
463467

464468
// build the curl command to scale the cluster
465469
String command = new StringBuffer()

integration-tests/src/test/java/oracle/weblogic/kubernetes/actions/impl/Secret.java

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
import oracle.weblogic.kubernetes.actions.impl.primitive.Kubernetes;
1818
import oracle.weblogic.kubernetes.logging.LoggingFacade;
1919

20+
import static oracle.weblogic.kubernetes.TestConstants.OKD;
2021
import static oracle.weblogic.kubernetes.utils.CommonTestUtils.testUntil;
2122
import static oracle.weblogic.kubernetes.utils.ThreadSafeLogger.getLogger;
2223

@@ -120,9 +121,18 @@ public static String getSecretEncodedToken(String namespace, String secretName)
120121
for (V1Secret v1Secret : v1Secrets) {
121122
if (v1Secret.getMetadata() != null && v1Secret.getMetadata().getName() != null) {
122123
if (v1Secret.getMetadata().getName().equals(secretName)) {
123-
if (v1Secret.getData() != null) {
124-
byte[] encodedToken = v1Secret.getData().get("token");
125-
return Base64.getEncoder().encodeToString(encodedToken);
124+
if (OKD) {
125+
for (Map.Entry<String, String> annotation : v1Secret.getMetadata().getAnnotations().entrySet()) {
126+
if (annotation.getKey().equals("openshift.io/token-secret.value")) {
127+
return annotation.getValue();
128+
}
129+
}
130+
return null;
131+
} else {
132+
if (v1Secret.getData() != null) {
133+
byte[] encodedToken = v1Secret.getData().get("token");
134+
return Base64.getEncoder().encodeToString(encodedToken);
135+
}
126136
}
127137
}
128138
}

0 commit comments

Comments
 (0)