Merge "Validate that node_version is set"

2024-09-19 20:31:36 +00:00 · 2024-09-19 20:31:36 +00:00 · 2dd43ed7f1
commit 2dd43ed7f1
parent 7e54b91d61 e188ce71ca
2 changed files with 6 additions and 1 deletions
--- a/roles/ensure-nodejs/README.rst
+++ b/roles/ensure-nodejs/README.rst
@ -3,4 +3,5 @@ Install NodeJS from nodesource
 **Role Variables**

 .. zuul:rolevar:: node_version
-   :default: 6
+
+   Required. What version of Node to install.
--- a/roles/ensure-nodejs/tasks/main.yaml
+++ b/roles/ensure-nodejs/tasks/main.yaml
@ -1,3 +1,7 @@
+- name: Ensure we know what version of node to install
+  assert:
+    that: node_version is defined
+
 - name: Update apt cache
  apt:
    update_cache: yes