You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What about using from collections import defaultdict with resources: Optional[Dict[str, ResourcesConfig]] = defaultdict(lambda: ResourcesConfig(cpu="500m", memory="1024Mi"))
it would also simplify (or even make not needed) the resources_for function.
It could simplify it a bit but it currently does some more work. It think it makes sense to rethink a bit resources configuration to enable changing all of the available params (including docker images if one wishes to) #129
We may need to have a fresh look at this as we have grouping nodes as a feature to consider as well.
What about using
from collections import defaultdict
withresources: Optional[Dict[str, ResourcesConfig]] = defaultdict(lambda: ResourcesConfig(cpu="500m", memory="1024Mi"))
it would also simplify (or even make not needed) the
resources_for
function.Originally posted by @marrrcin in #11 (comment)
The text was updated successfully, but these errors were encountered: