Skip to content

Commit 9fa13f3

Browse files
authored
Update comment to mention that webhooks are not supported by helm (#4981)
Update comment to mention that webhooks are not supported by helm The comment is in the helm plugin but only mentions ansible which is confusing. I used the helm/ansible term to be consistent with the rest of the comments in the file. Signed-off-by: Brad Topol <[email protected]>
1 parent 3779341 commit 9fa13f3

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

internal/plugins/helm/v1/init.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -208,15 +208,15 @@ func addInitCustomizations(projectName string) error {
208208
return err
209209
}
210210

211-
// Remove the webhook option for the componentConfig since webhooks are not supported by ansible
211+
// Remove the webhook option for the componentConfig since webhooks are not supported by helm
212212
err = sdkutil.ReplaceInFile(filepath.Join("config", "manager", "controller_manager_config.yaml"),
213213
"webhook:\n port: 9443", "")
214214
if err != nil {
215215
return err
216216
}
217217

218-
// Remove the call to the command as manager. Helm/Ansible has not been exposing this entrypoint
219-
// todo: provide the manager entrypoint for helm/ansible and then remove it
218+
// Remove the call to the command as manager. Helm has not been exposing this entrypoint
219+
// todo: provide the manager entrypoint for helm and then remove it
220220
const command = `command:
221221
- /manager
222222
`

0 commit comments

Comments
 (0)