runner: --idle-timeout not passed to terraform

[casperdcl]

cml runner --idle-timeout=-1 ...
info: Terraform apply...
info: {..., "idleTimeout":300, ...}

happens on e.g. --cloud=gcp and --cloud=aws but does not happen on-prem (local) where terraform isn't used

[DavidGOrtega]

I remember this hitting me testing a few weeks ago, however I finally determined it was correct... or almost. Happens that -1 is not accepted in cloud instances

${
    typeof idleTimeout !== 'undefined' && idleTimeout >= 0
      ? `idle_timeout = ${idleTimeout}`
      : ''
  }

Where you trying -1 @casperdcl ?

[casperdcl]

Not sure I follow the question. cml runner --idle-timeout=-1 --cloud=aws results in idle_timeout = 300 being passed to terraform.

bin/cml/runner.js::runLocal is fine.

bin/cml/runner.js::runCloud has a bug.

[0x2b3bfa0]

Nota bene: -1 is accepted everywhere since #700