[golemfactory/golem] Nvidia docker (#3045)

maaktweluit requested changes on this pull request.

LGTM!

Only the freeze part should be checked

> @@ -319,6 +319,11 @@ def build_images(cls):

for entry in cls._collect_images():
version = cls._image_version(entry)
+
+ if not cls._image_supported(entry):
+ logger.warning(f’Image {version} is not supported’)

it is better to use multiple arguments when logging, less formatting when the level is disabled

> @@ -118,29 +118,28 @@ def task_given(self, ctd):
return True

def resource_given(self, task_id):
— if task_id in self.task_to_subtask_mapping:
— subtask_id = self.task_to_subtask_mapping[task_id]
— if subtask_id in self.assigned_subtasks:
— subtask = self.assigned_subtasks[subtask_id]
+ subtask_id = self.task_to_subtask_mapping.get(task_id)

nice cleanup!

> @@ -110,7 +110,7 @@ txtorcon==0.20.0
u-msgpack-python==2.5.0
urllib3==1.22
watchdog==0.8.3
-web3==4.2.1
+-e git+https://github.com/mfranciszkiewicz/web3.py@pypiwin32_220#egg=web3

we should put this on our python registry, else it will break on a freeze