Bump node version to 18 in build image (#940)

* bump node version to 18 in build image

* update comment

internal/web/ui/README.md

@@ -3,4 +3,4 @@
 ## Prerequisites
 
 * Yarn >= v1.22
-* Node.js >= v16
+* Node.js >= v18

tools/build-image/Dockerfile

@@ -59,7 +59,7 @@ RUN go install sigs.k8s.io/controller-tools/cmd/controller-gen@$CONTROLLER_GEN_V
 FROM rfratto/viceroy:v0.4.0
 
 # Install NodeJS LTS. This is needed because the most recent version of NodeJS
-# from official Debian packages is v12, and we need LTS version v16.
+# from official Debian packages is v12, and we need version v18.
 #
 # This must be done before installing other dependencies, otherwise nodesource
 # will fail on installing NodeJS for all platforms instead of just our host
@@ -68,7 +68,7 @@ FROM rfratto/viceroy:v0.4.0
 
 RUN  apt-get update && apt-get install -qy ca-certificates curl gnupg && mkdir -p /etc/apt/keyrings  \
  &&  curl -fsSL https://deb.nodesource.com/gpgkey/nodesource-repo.gpg.key | gpg --dearmor -o /etc/apt/keyrings/nodesource.gpg \
- &&  echo "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_16.x nodistro main" | tee /etc/apt/sources.list.d/nodesource.list \
+ &&  echo "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_18.x nodistro main" | tee /etc/apt/sources.list.d/nodesource.list \
  &&  apt-get update && apt-get install -qy nodejs \
  &&  rm -rf /var/lib/apt/lists/*