Bladeren bron

AMBARI-10313. RU: Replacement text not seen in pre-req check (dlysnichenko)

Lisnichenko Dmitro 10 jaren geleden
bovenliggende
commit
aa6ef089ba

+ 4 - 6
ambari-server/src/main/java/org/apache/ambari/server/checks/AbstractCheckDescriptor.java

@@ -126,13 +126,14 @@ public abstract class AbstractCheckDescriptor {
       PrerequisiteCheck prerequisiteCheck, PrereqCheckRequest request) {
       PrerequisiteCheck prerequisiteCheck, PrereqCheckRequest request) {
     String fail = m_description.getFail(key);
     String fail = m_description.getFail(key);
 
 
-    if (-1 != fail.indexOf("{{version}}") && null != request.getRepositoryVersion()) {
+    if (fail.contains("{{version}}") && null != request.getRepositoryVersion()) {
       fail = fail.replace("{{version}}", request.getRepositoryVersion());
       fail = fail.replace("{{version}}", request.getRepositoryVersion());
     }
     }
 
 
-    if (-1 != fail.indexOf("{{fails}}")) {
+    if (fail.contains("{{fails}}")) {
       List<String> names = prerequisiteCheck.getFailedOn();
       List<String> names = prerequisiteCheck.getFailedOn();
 
 
+      // If Type=PrereqCheckType.HOST, names list is already populated
       if (getDescription().getType() == PrereqCheckType.SERVICE) {
       if (getDescription().getType() == PrereqCheckType.SERVICE) {
         Clusters clusters = clustersProvider.get();
         Clusters clusters = clustersProvider.get();
         AmbariMetaInfo metaInfo = ambariMetaInfo.get();
         AmbariMetaInfo metaInfo = ambariMetaInfo.get();
@@ -155,12 +156,9 @@ public abstract class AbstractCheckDescriptor {
         } catch (Exception e) {
         } catch (Exception e) {
           LOG.warn("Could not load service info map");
           LOG.warn("Could not load service info map");
         }
         }
-
-        fail = fail.replace("{{fails}}", formatEntityList(names));
-
       }
       }
 
 
-
+      fail = fail.replace("{{fails}}", formatEntityList(names));
     }
     }
 
 
     return fail;
     return fail;