"dry-run: true" still deployed a new helm chart

[jpds]

I have a simple pipeline:

pipeline:
  helm_deploy:
    image: quay.io/ipedrazas/drone-helm
    skip_tls_verify: true
    chart: stable/prometheus
    update_dependencies: false
    release: prometheus
    namespace: production-prometheus
    dry-run: true
    secrets: [ api_server, kubernetes_token ]
    when:
      branch: [master]

Interestingly, when this ran, it actually deployed prometheus, into that namespace.

[minhdanh]

I think this is because the actual env var passed to drone-helm container by Drone is PLUGIN_DRY-RUN, not PLUGIN_DRY_RUN.

[ipedrazas]

Have you tried this?

pipeline:
  helm_deploy:
    image: quay.io/ipedrazas/drone-helm
    skip_tls_verify: true
    chart: stable/prometheus
    update_dependencies: false
    release: prometheus
    namespace: production-prometheus
    dry_run: true
    secrets: [ api_server, kubernetes_token ]
    when:
      branch: [master]

[zakkg3]

We may need to upgrade the docs here then: https://github.com/ipedrazas/drone-helm/blob/master/DOCS.md#advanced-customisations-and-debugging