-
Notifications
You must be signed in to change notification settings - Fork 69
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
fix(tests): update VSCode runner to use 24.04 #2344
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good if it works 👍
13cd9c3
to
a485f2a
Compare
(cd scripts/docker && docker build -t ubuntu24.04-xvfb -f ubuntu24.04-xvfb.Dockerfile .) | ||
apt list --installed | ||
sudo apt-get update | ||
sudo apt-get install -y curl=7.68.0-1ubuntu2.22 libcurl4=7.68.0-1ubuntu2.22 libcurl4-openssl-dev=7.68.0-1ubuntu2.22 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Were these lines supposed to go into the dockerfile config? Because otherwise this here would be affecting the global scope of the evergreen host, which we'd almost never want to do
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, not meant to be here it's actually leftover from previous evergreen.yml.in
.
Good reminder that evergreen maintains state though, that's one thing I find weird coming from GitHub Actions.
Might be related to mongodb-js/vscode#910
An attempt to fix the test by doing this.
Doesn't seem to work thus far. The error can be re-created locally when running inside the Ubuntu docker. The tests pass on Mac so the issue seems platform-specific.