Merge pull request #2700 from oracle/owls_95390_fix_backport_to_3.3

Backport fix for OWLS-95390 - Add validation check to reject domain configuration that uses same serverNamePrefix

Author: robertpatrick

operator/src/main/resources/scripts/introspectDomain.py

@@ -1,4 +1,4 @@
-# Copyright (c) 2018, 2021, Oracle and/or its affiliates.
+# Copyright (c) 2018, 2022, Oracle and/or its affiliates.
 # Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl.
 #
 # ------------
@@ -351,6 +351,7 @@ def validate(self):
     self.validateAdminServer()
     self.validateClusters()
     self.validateServerCustomChannelName()
+    self.validateDynamicClustersDuplicateServerNamePrefix()
     return self.isValid()
 
   def generate(self):
@@ -401,6 +402,15 @@ def validateAdminServer(self):
     if cluster is not None:
       self.addError("The admin server " + self.name(adminServer) + " belongs to the WebLogic cluster " + self.name(cluster) + ", the operator does not support having an admin server participate in a cluster.")
 
+  def validateDynamicClustersDuplicateServerNamePrefix(self):
+    serverNamePrefixes = []
+    for cluster in self.env.getDomain().getClusters():
+      if self.getDynamicServersOrNone(cluster) is not None:
+        if cluster.getDynamicServers().getServerNamePrefix() in serverNamePrefixes:
+          self.addError("The ServerNamePrefix '" + cluster.getDynamicServers().getServerNamePrefix() + "' specified for WebLogic dynamic cluster " + self.name(cluster) + "'s dynamic servers is already in use. The ServerNamePrefix must be unique for each WebLogic dynamic cluster.")
+        else:
+          serverNamePrefixes.append(cluster.getDynamicServers().getServerNamePrefix())
+
   def validateClusters(self):
     for cluster in self.env.getDomain().getClusters():
       self.validateCluster(cluster)