Adding Environment URL field in Phase model

[krtkvrm]

Added environment_url field in ChallengePhase model, that is the repository url for environment that will be used by RL Worker.
The worker can get the environment_url using api/challenges/phase/environment/<phase-slug>.

[krtkvrm]

@RishabhJain2018 @rishabhbhardwaj @deshraj @fristonio Please review

[yashdusing]

Hey @vkartik97 ! I read your blog and was wondering if you tried this ?
Also, you mentioned of a problem regarding GPU orchestration. Could you please elaborate ?

[krtkvrm]

@yashdusing

Hey @vkartik97 ! I read your blog and was wondering if you tried this ?

Yes, this is already enabled in EC2 P2 but I wanted to achieve sharing of GPU resources among the environment containers in multiple pods.

Also, you mentioned of a problem regarding GPU orchestration. Could you please elaborate ?

GPU Orchestration on P2 is working as expected, the problem is just the sharing extension. I tried it on 1 node minikube, which didn’t give expected behaviour.

[yashdusing]

@vkartik97 I was wondering if this can be used on top of k8s ? It has fractional gpu support and also runs on top of k8s.

[krtkvrm]

@yashdusing Really thanks for researching on the topic.
I did try ray-project at the end of Phase 2, and I ran into similar this issue: [ray] ray misuse gpu in docker container, while trying running tensorflow training on nividia-docker, one of the dependency for Kubernetes GPU Scheduling. I believe that the issue is due to nvidia-docker, but I need to spike more to come to any conclusion.
Again, thanks for working on this :)

[codecov-io]

Codecov Report

Merging #2392 into master will decrease coverage by 0.05%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##           master    #2392      +/-   ##
==========================================
- Coverage   72.74%   72.68%   -0.06%     
==========================================
  Files          82       82              
  Lines        5316     5327      +11     
==========================================
+ Hits         3867     3872       +5     
- Misses       1449     1455       +6

Impacted Files	Coverage Δ
apps/challenges/views.py	100% <ø> (ø)	⬆️
apps/jobs/urls.py	100% <ø> (ø)	⬆️
apps/jobs/views.py	100% <ø> (ø)	⬆️
scripts/workers/remote_submission_worker.py	37.02% <53.84%> (+0.3%)	⬆️

Impacted Files	Coverage Δ
apps/challenges/views.py	100% <ø> (ø)	⬆️
apps/jobs/urls.py	100% <ø> (ø)	⬆️
apps/jobs/views.py	100% <ø> (ø)	⬆️
scripts/workers/remote_submission_worker.py	37.02% <53.84%> (+0.3%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7adb66c...3f412f1. Read the comment docs.

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This pull request has been automatically closed as there is no further activity. Thank you for your contributions.

[RishabhJain2018]

Can you please open up a new PR with the changes in this file?

[krtkvrm]

Done: #2533