Closed ecksun closed 3 years ago
@ecksun Thanks for reporting this. Do you think we should make this as an optional parameter?
@ecksun Thanks for reporting this. Do you think we should make this as an optional parameter?
probably?
I'm not super familiar with helm and helm plugins so there might be something I haven't understood correctly or that I have missed. But consider my current task:
- name: Install local Helm plugin for Google Cloud Storage repository support
community.kubernetes.helm_plugin:
plugin_path: https://github.com/hayorov/helm-gcs
release_namespace: NOT_REALLY_REQUIRED
state: present
resolved_by_pr #358
SUMMARY
release_namespace is required in the helm_plugin task.
ISSUE TYPE
COMPONENT NAME
community.kubernetes.helm_plugin
ANSIBLE VERSION
CONFIGURATION
OS / ENVIRONMENT
STEPS TO REPRODUCE
Looking at the souce this seems to affect even master.
I'm assuming that in many cases it make sense to have the argument required (perhaps it was more relevant with helm2?), but I don't think it does in this case.
EXPECTED RESULTS
The task should run without having to specify namespace, as the plugin is only local and doesn't deal with any namespaces.
ACTUAL RESULTS