MLOS icon indicating copy to clipboard operation
MLOS copied to clipboard

Better naming for `config` and `params` in VM and SaaS-related services

Open motus opened this issue 1 year ago • 1 comments

It is very confusing to have two dicts for config and params as arguments to many methods in, e.g., AzureVMService and AzureConfigService. We should be more explicit in naming and explain the difference between these arguments in the docstrings.

motus avatar Nov 06 '23 20:11 motus

One instance of this mismatch: in remote_exec() we take config but in start_host() we take params.

  • [ ] Both should probably be params

bpkroth avatar Dec 01 '23 16:12 bpkroth