Skip to content

Commit ff98527

Browse files
authored
Merge pull request #283 from gianlucam76/ignore-empty-section
Fix issue deploying helm charts
2 parents e68841c + 333fd61 commit ff98527

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

controllers/handlers_helm.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1448,13 +1448,14 @@ func collectHelmContent(manifest string, logger logr.Logger) ([]*unstructured.Un
14481448

14491449
elements := strings.Split(manifest, separator)
14501450
for i := range elements {
1451-
if elements[i] == "" {
1451+
section := removeCommentsAndEmptyLines(elements[i])
1452+
if section == "" {
14521453
continue
14531454
}
14541455

1455-
policy, err := utils.GetUnstructured([]byte(elements[i]))
1456+
policy, err := utils.GetUnstructured([]byte(section))
14561457
if err != nil {
1457-
logger.Error(err, fmt.Sprintf("failed to get policy from Data %.100s", elements[i]))
1458+
logger.Error(err, fmt.Sprintf("failed to get policy from Data %.100s", section))
14581459
return nil, err
14591460
}
14601461

0 commit comments

Comments
 (0)