-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an integration test for VNC + websockify #93
Commits on Jun 13, 2024
-
Add an integration test for VNC + websockify
This tests the following components: 1. The VNC server 2. websockify 3. Jupyter Server Proxy 4. The desktop environment itself It does so by: 1. Building the container image 2. Starting a container with that image, with jupyter server running inside. 3. Connecting to the container via websockets, authenticating and exposing the websocket connection as a regular TCP socket on localhost via [websocat](https://github.com/vi/websocat) 4. Taking a screenshot of the remote desktop by connecting to it through `vncdotool`. This requires a tcp connection, hence the double proxying with websocat 5. Verify that the image produced by the screenshot is the same as a reference image we have. This only misses the noVNC + JS from testing, but otherwise represents a massive improvement to the status quo!
Configuration menu - View commit details
-
Copy full SHA for 7aee660 - Browse repository at this point
Copy the full SHA 7aee660View commit details -
Configuration menu - View commit details
-
Copy full SHA for 44137e0 - Browse repository at this point
Copy the full SHA 44137e0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 557135e - Browse repository at this point
Copy the full SHA 557135eView commit details -
Dynamically allocate forwarded port
Prevents clashes between multiple runs Also remove some old commented out code
Configuration menu - View commit details
-
Copy full SHA for fbf2fd1 - Browse repository at this point
Copy the full SHA fbf2fd1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 00808aa - Browse repository at this point
Copy the full SHA 00808aaView commit details -
Configuration menu - View commit details
-
Copy full SHA for af61281 - Browse repository at this point
Copy the full SHA af61281View commit details -
Configuration menu - View commit details
-
Copy full SHA for cbe3e3d - Browse repository at this point
Copy the full SHA cbe3e3dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 66809be - Browse repository at this point
Copy the full SHA 66809beView commit details -
Configuration menu - View commit details
-
Copy full SHA for e8b864a - Browse repository at this point
Copy the full SHA e8b864aView commit details -
Configuration menu - View commit details
-
Copy full SHA for d13d5bd - Browse repository at this point
Copy the full SHA d13d5bdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6e7df3e - Browse repository at this point
Copy the full SHA 6e7df3eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 76b801e - Browse repository at this point
Copy the full SHA 76b801eView commit details -
Configuration menu - View commit details
-
Copy full SHA for dea42a5 - Browse repository at this point
Copy the full SHA dea42a5View commit details -
Don't pass "-it" to docker run
Doesn't actually need to be interactive, we primarily care about sharing stdout
Configuration menu - View commit details
-
Copy full SHA for 3d44feb - Browse repository at this point
Copy the full SHA 3d44febView commit details -
Configuration menu - View commit details
-
Copy full SHA for e8de6d4 - Browse repository at this point
Copy the full SHA e8de6d4View commit details -
Configuration menu - View commit details
-
Copy full SHA for b8710d2 - Browse repository at this point
Copy the full SHA b8710d2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 25bee53 - Browse repository at this point
Copy the full SHA 25bee53View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5054205 - Browse repository at this point
Copy the full SHA 5054205View commit details -
Configuration menu - View commit details
-
Copy full SHA for adec7b2 - Browse repository at this point
Copy the full SHA adec7b2View commit details -
Try increasing shm size on docker run
Maybe this is what github actions needs
Configuration menu - View commit details
-
Copy full SHA for 86c6cdf - Browse repository at this point
Copy the full SHA 86c6cdfView commit details -
Revert "Try increasing shm size on docker run"
It did nothing This reverts commit a33a447.
Configuration menu - View commit details
-
Copy full SHA for 8ea28eb - Browse repository at this point
Copy the full SHA 8ea28ebView commit details -
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
Configuration menu - View commit details
-
Copy full SHA for c285c0b - Browse repository at this point
Copy the full SHA c285c0bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7a59e95 - Browse repository at this point
Copy the full SHA 7a59e95View commit details