Skip to content

Conversation

dblnz
Copy link
Contributor

@dblnz dblnz commented Nov 7, 2024

This adds devcontainer config to be used with codespaces and Visual Studio

@dblnz dblnz marked this pull request as draft November 7, 2024 16:05
@dblnz dblnz added the chore label Nov 7, 2024
@danbugs
Copy link
Contributor

danbugs commented Nov 7, 2024

Is this possible now? I tried doing this about 2 years ago and, iirc, it wasn't possible to access /dev/kvm.

@dblnz
Copy link
Contributor Author

dblnz commented Nov 7, 2024

Is this possible now? I tried doing this about 2 years ago and, iirc, it wasn't possible to access /dev/kvm.

I ran some tests locally and it seems to be working.

@danbugs
Copy link
Contributor

danbugs commented Nov 7, 2024

Is this possible now? I tried doing this about 2 years ago and, iirc, it wasn't possible to access /dev/kvm.

I ran some tests locally and it seems to be working.

Awesome! Might just not work if people try to use it from the browser? We can add a note on that after testing, though.

@dblnz
Copy link
Contributor Author

dblnz commented Nov 7, 2024

Is this possible now? I tried doing this about 2 years ago and, iirc, it wasn't possible to access /dev/kvm.

I ran some tests locally and it seems to be working.

Awesome! Might just not work if people try to use it from the browser? We can add a note on that after testing, though.

I ran a couple of times today in the browser, but the issue was it built the container image too slowly and I believe it got terminated due to timeout.
Now I am looking into how to make the container smaller so that it is built faster

@devigned
Copy link
Contributor

devigned commented Nov 8, 2024

@dblnz have you considered building a base image and then using that rather than base Debian?

@dblnz
Copy link
Contributor Author

dblnz commented Nov 8, 2024

@dblnz have you considered building a base image and then using that rather than base Debian?

Yes, and that is the current plan, after I test locally and see everything works, then I'll push the image so it can be used with devcontainers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants