[Bug Report] Docker X11 Forwarding error when repo cloned in external drive

[j3soon]

Describe the bug

If the IsaacLab repo is cloned in an external drive, X11 forwarding settings for docker will be ignored due to yq snap package permission errors.

Steps to reproduce

Please try to provide a minimal example to reproduce the bug. Error messages and stack traces are also helpful.

# Assume that /mnt/projects mounts an external drive
cd /mnt/projects
git clone https://github.com/isaac-sim/IsaacLab
cd IsaacLab
# Follow the docker installation guides here: https://isaac-sim.github.io/IsaacLab/source/deployment/docker.html
# Set `__ISAACLAB_X11_FORWARDING_ENABLED: 1` in `docker/.container.yaml`
./docker/container.sh start

Observe the following error message (the script will skip the error and continue starting the container):

[INFO] Using container profile: base
[INFO] Building the docker image and starting the container isaac-lab-base in the background...
[INFO] Required 'yq' package could not be found. Would you like to install it via snap? (y/N)y
[sudo] password for johnson:
yq v4.44.2 from Mike Farah (mikefarah) installed
Error: open /mnt/projects/IsaacLab/docker/.container.yaml: permission denied
[INFO] X11 Forwarding is configured as  in .container.yaml
[INFO] To enable X11 forwarding, set __ISAACLAB_X11_FORWARDING_ENABLED=1 in .container.yaml
[+] Building 1.8s (15/15) FINISHED

Attempting to launch any GUI program inside the container will cause segfault due to not being able to launch the app window.

./docker/container.sh enter
# in the container
/isaac-sim/runapp.sh

System Info

Describe the characteristic of your environment:

• Commit: 05a2f3f
• Isaac Sim Version: 4.0.0-rc.21+4.0.13872.3e3cb0c9.gl
• OS: Ubuntu 22.04
• GPU: Quadro RTX 6000
• CUDA: 12.2
• GPU Driver: 535.183.01

Additional context

I'll open a PR for this.

Checklist

• I have checked that there is no similar issue in the repo (required)
• I have checked that the issue is not in running Isaac Sim itself and is related to the repo

Acceptance Criteria

• (Optional) Merge PR Improves docker X11 forwarding documentation #685
• Merge PR Support docker X11 forwarding when repo cloned in external drives #689

[hhansen-bdai]

Hello @j3soon, thanks for your issue + PRs! We have a PR in the final stages which removes our dependency on yq in favor of the Python Standard Library's configparser module. This should eliminate any issues caused by snap permissions. I will close this issue when that PR is merged.

[RandomOakForest]

Thank you for following up on this, @hhansen-bdai. Could we close this issue now? Thanks.

[j3soon]

Confirmed that yq is no longer used, thanks! Closing this issue.