fix: use correct uid/gid when cloning git repos

[olevski]

This was hardcoded for the old sessions so we just re-used the same hardcoded values from v1 sessions. But for v2 we should take this info from the environment.

closes #629

/deploy

[coveralls]

Pull Request Test Coverage Report for Build 13180787014

Details

• 0 of 4 (0.0%) changed or added relevant lines in 2 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage decreased (-0.01%) to 86.558%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
components/renku_data_services/notebooks/blueprints.py	0	1	0.0%
components/renku_data_services/notebooks/core_sessions.py	0	3	0.0%

Files with Coverage Reduction	New Missed Lines	%
components/renku_data_services/base_api/error_handler.py	1	89.47%

Totals
Change from base Build 13108744249:	-0.01%
Covered Lines:	15384
Relevant Lines:	17773

---

💛 - Coveralls

[leafty]

Thanks! Let's deploy this to check that it works as intended.

[RenkuBot]

You can access the deployment of this PR at https://renku-ci-ds-635.dev.renku.ch

[leafty]

The container to fix is git-clone. The git-proxy container should not be updated (it does not touch the file system).

[olevski]

Well jupyter still does not work when you use non 1001 uid and gid. I see this error in the logs:

File "/opt/conda/lib/python3.11/pathlib.py", line 1116, in mkdir                                                                                                   │
│     os.mkdir(self, mode)                                                                                                                                             │
│ PermissionError: [Errno 13] Permission denied: '/home/jovyan/.local/share/jupyter/runtime'

But that is not our fault. At least we are doing the right thing and respecting the stuff the user setup. Some images will only work with specific uid/gid.

[olevski]

So I tested with vscode and that works with gid/uid 1001. See this project for example: https://renku-ci-ds-635.dev.renku.ch/v2/projects/tasko.olevski/test1

[leafty]

LGTM Thanks 👍

[RenkuBot]

Tearing down the temporary RenkuLab deplyoment for this PR.