Drop `npm` (and possibly `node`) as runtime dependency of `kubeflow/dashboard`

[christian-heusel]

Checks

• I have searched the existing issues.
• My request is related to one of the components in the kubeflow/dashboard repository.

Motivation

Currently the centraldashboard is being served via npm start as part of the Dockerfile:

dashboard/components/centraldashboard/Dockerfile

Lines 19 to 31 in 41dc738

	# Step 2: Packages assets for serving
	FROM node:16.20.2-alpine AS serve
	RUN apk add --no-cache tini
	USER node
	ENV NODE_ENV=production
	WORKDIR /usr/src/app
	COPY --from=build --chown=node:node /centraldashboard .
	EXPOSE 8082
	ENTRYPOINT ["/sbin/tini", "--" , "npm", "start"]

Optimally this dependency could be eliminated in the final part of the image in order to have less attack surface, especially given how old some parts of the components are.

Implementation

The Dockerfile for kubeflow/notebooks (v2) could be a good inspiration for the implementation:

https://github.com/kubeflow/notebooks/blob/notebooks-v2/workspaces/frontend/Dockerfile

Are you willing & able to help?

• I am able to submit a PR!
• I can help test the feature!

[christian-heusel]

/help

[google-oss-prow]

@christian-heusel:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

Details

In response to this:

/help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[adityabavadekar]

Hi @christian-heusel , I'd like to work on this.

Would preferred approach be: nginx for the static files, proxy to the nodejs server (similar to the linked kubeflow/notebooks), or just changing the entrypoint to node dist/server.js directly?

[KUNDAN1334]

/assign