Skip to content
This repository has been archived by the owner on Feb 14, 2021. It is now read-only.

Commit

Permalink
Only "notify" if "exec" will run (fixes frankiethekneeman#36)
Browse files Browse the repository at this point in the history
  • Loading branch information
Slashbunny committed Sep 15, 2017
1 parent ac29e40 commit d3933fa
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion manifests/runner.pp
Original file line number Diff line number Diff line change
Expand Up @@ -416,7 +416,9 @@
$kubernetes_opts="${kubernetes_host_opt} ${kubernetes_cert_file_opt} ${kubernetes_key_file_opt} ${kubernetes_ca_file_opt} ${kubernetes_image_opt} ${kubernetes_namespace_opt} ${kubernetes_priviledged_opt} ${kubernetes_cpus_opt} ${kubernetes_memory_opt} ${kubernetes_service_cpus_opt} ${kubernetes_service_memory_opt}"

$opts = "${runner_opts} ${executor_opt} ${docker_opts} ${parallels_vm_opt} ${ssh_opts} ${machine_opts} ${kubernetes_opts}"
notify{"Will run gitlab-ci-multi-runner register --non-interactive ${opts}": }
notify{ "Will run gitlab-ci-multi-runner register --non-interactive ${opts}":
onlyif => "! grep ${node_description} ${::gitlab_ci_multi_runner::toml_file}",
}

# Register a new runner - this is where the magic happens.
# Only if the config.toml file doesn't already contain an entry.
Expand Down

0 comments on commit d3933fa

Please sign in to comment.