Skip to content

Commit 80770a5

Browse files
authored
fix(cli): parse security_opt as list not tuple (#3587)
1 parent 310b7de commit 80770a5

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

renku/core/session/session.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,11 @@ def session_start(
142142
"""
143143
from renku.domain_model.project_context import project_context
144144

145+
# NOTE: The Docker client in Python requires the parameters below to be a list and will fail with a tuple.
146+
# Click will convert parameters with the flag "many" set to True to tuples.
147+
kwargs["security_opt"] = list(kwargs.get("security_opt", []))
148+
kwargs["device_cgroup_rules"] = list(kwargs.get("device_cgroup_rules", []))
149+
145150
pinned_image = get_value("interactive", "image")
146151
if pinned_image and image_name is None:
147152
image_name = pinned_image

0 commit comments

Comments
 (0)