Basic Sandboxed Coder

[allenwang28]

follow up for #188

Enroot install instructions are here. Commands:

# Debian-based distributions
arch=$(dpkg --print-architecture)
curl -fSsL -O https://github.com/NVIDIA/enroot/releases/download/v4.0.0/enroot-hardened_4.0.0-1_${arch}.deb
curl -fSsL -O https://github.com/NVIDIA/enroot/releases/download/v4.0.0/enroot-hardened+caps_4.0.0-1_${arch}.deb # optional
sudo apt install -y ./*.deb

# RHEL-based distributions
arch=$(uname -m)
sudo dnf install -y epel-release # required on some distributions
sudo dnf install -y https://github.com/NVIDIA/enroot/releases/download/v4.0.0/enroot-hardened-4.0.0-1.el8.${arch}.rpm
sudo dnf install -y https://github.com/NVIDIA/enroot/releases/download/v4.0.0/enroot-hardened+caps-4.0.0-1.el8.${arch}.rpm # optional

[allenwang28]

Quick updates:

• I removed the vLLM example for now because I didn't like the way it looked. @Jack-Khuu had a PR that had something like this: Add a prompt wrapper + Switching to Instruct model allenwang28/forge#1
• Added a unit test and an integration test

[Jack-Khuu]

Legit

Non-code ask:

• Install instructions (README or just put in the PR description, until it's ready for the core install file)

[allenwang28]

Install instructions (README or just put in the PR description, until it's ready for the core install file)

good call, added to the PR description

[joecummings]

Why is this in here?

[allenwang28]

for some reason a bunch of unit tests were failing due to wrong cleanup. Not sure why I saw this in this PR specifically, but we should have this line anyways. If we want to be really clean I can add this in a separate PR

[joecummings]

Couple nits, but looks good!