ClearML/clearml-agent
1
0
Fork 0
mirror of https://github.com/clearml/clearml-agent synced 2025-02-07 13:26:08 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix template namespace should override default namespace

Browse Source
This commit is contained in:
allegroai 2022-07-22 22:44:32 +03:00
parent e687418194
commit d96b8ff906
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
clearml_agent/glue/k8s.py
View File

@ -572,6 +572,12 @@ class K8sIntegration(Worker):
self, create_clearml_conf, docker_image, docker_args, docker_bash, labels, queue, task_id, template=None self, create_clearml_conf, docker_image, docker_args, docker_bash, labels, queue, task_id, template=None
): ):
template = template or deepcopy(self.template_dict) template = template or deepcopy(self.template_dict)
try:
namespace = template['metadata']['namespace'] or self.namespace
except (KeyError, TypeError, AttributeError):
namespace = self.namespace
template.setdefault('apiVersion', 'v1') template.setdefault('apiVersion', 'v1')
template['kind'] = 'Pod' template['kind'] = 'Pod'
template.setdefault('metadata', {}) template.setdefault('metadata', {})
@ -642,7 +648,7 @@ class K8sIntegration(Worker):
task_id=task_id, task_id=task_id,
docker_image=docker_image, docker_image=docker_image,
queue_id=queue, queue_id=queue,
namespace=self.namespace namespace=namespace
) )
# make sure we provide a list # make sure we provide a list
if isinstance(kubectl_cmd, str): if isinstance(kubectl_cmd, str):