Skip to content

Commit

Permalink
remove todo and fixme
Browse files Browse the repository at this point in the history
  • Loading branch information
JaeAeich committed Oct 14, 2024
1 parent c6e3964 commit 5607e62
Showing 1 changed file with 1 addition and 20 deletions.
21 changes: 1 addition & 20 deletions tesk/k8s/converter/converter.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,18 +94,6 @@ def from_tes_task_to_k8s_job(self, task: TesTask) -> V1Job:
taskmaster_job.metadata.annotations[
self.tesk_k8s_constants.annotation_constants.ANN_TESTASK_NAME_KEY
] = task.name
# taskmaster_job.metadata.labels[self.constants.label_userid_key] = user[
# "username"
# ]

# if task.tags and "GROUP_NAME" in task.tags:
# taskmaster_job.metadata.labels[self.constants.label_userid_key] = task[
# "tags"
# ]["GROUP_NAME"]
# elif user["is_member"]:
# taskmaster_job.metadata.labels[self.constants.label_groupname_key] = user[
# "any_group"
# ]

json_input = json.dumps(
task.dict(),
Expand Down Expand Up @@ -170,23 +158,16 @@ def from_tes_task_to_k8s_config_map(
if taskmaster_config_map.metadata.annotations is None:
taskmaster_config_map.metadata.annotations = {}

# FIXME: What if the task name is None?
task_name = task.name or "task-name-not-set"
task_name = task.name or "String"

taskmaster_config_map.metadata.annotations[
self.tesk_k8s_constants.annotation_constants.ANN_TESTASK_NAME_KEY
] = task_name

# taskmaster_config_map.metadata.labels[self.constants.label_userid_key]
# = user["username"]

if task.tags and "GROUP_NAME" in task.tags:
taskmaster_config_map.metadata.labels[
self.tesk_k8s_constants.label_constants.LABEL_GROUPNAME_KEY
] = task.tags["GROUP_NAME"]
# elif user["is_member"]:
# taskmaster_config_map.metadata.labels[self.constants.label_groupname_key]
# = user["any_group"]

assert taskmaster_config_map.metadata.name is not None
assert task.resources is not None
Expand Down

0 comments on commit 5607e62

Please sign in to comment.