[DB-14827] Bug Fix: Migration Complexity Explanation empty in case of…

… Oracle (#2171) - updated html template to check for empty value before adding Migration Complexity Explanation heading.
yugabyte · Jan 17, 2025 · e44dc10 · e44dc10
1 parent 2a705b2
commit e44dc10
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/yb-voyager/cmd/assessMigrationCommand.go b/yb-voyager/cmd/assessMigrationCommand.go
@@ -1607,7 +1607,7 @@ func generateAssessmentReportJson(reportDir string) error {
 	if err != nil {
 		return fmt.Errorf("unable to build migration complexity explanation for json report: %w", err)
 	}
-	log.Info(assessmentReport.MigrationComplexityExplanation)
+	log.Infof("migration complexity explanation: %q", assessmentReport.MigrationComplexityExplanation)
 
 	strReport, err := json.MarshalIndent(assessmentReport, "", "\t")
 	if err != nil {

diff --git a/yb-voyager/cmd/templates/migration_assessment_report.template b/yb-voyager/cmd/templates/migration_assessment_report.template
@@ -272,7 +272,7 @@
             {{ end }}
         {{end}}
 
-        {{if ne .MigrationComplexity "NOT AVAILABLE"}}
+        {{if and (ne .MigrationComplexity "NOT AVAILABLE") (ne (len .MigrationComplexityExplanation) 0)}}
             <h2>Migration Complexity Explanation</h2>
             <p>{{ .MigrationComplexityExplanation }}</p>
         {{end}}