Replaced hardcoded string with yaml for config map

Author: JainTwinkle

utils/README.md

@@ -25,11 +25,11 @@ Use `--help` option for all available options.
 - [x] Parse configs and dump configs in file to be edited later according to the res
 - [x] If any config value is different than default value then add that config to the --system-config parameter
 - [x] Dump config in a yaml file named system-cm.yaml
-- [ ] Change hardcoded string to dynamic to dump in system-cm.yaml
+- [x] Change hardcoded string to dynamic to dump in system-cm.yaml
+- [x] Extend to more Ray versions
 - [ ] Update format of system config string that works correctly
 - [ ] Segregate internal and external options
 - [ ] Spread code in multiple files
-- [ ] Extend to more Ray versions
 - [ ] Add Try-catch and checks to improve robustness
 - [ ] Add code to also dump Ray operator and cluster yamls
 - [ ] Give a sample cluster Yaml to user to let user edit cluster specific configurations, e.g., cpu, num of workers

utils/ray-res-config.py

@@ -246,17 +246,12 @@ def gen_system_conf(conf_list, verbose):
     if verbose:
         print("Version 1.4.0 specific configs")
     conf_string = get_conf_string(conf_list)
-    # FIXME: this should not be hardcoded and can be made a dict in python to be
-    # loaded as yaml instead of a string
-    sys_conf = """
-    apiVersion: v1
-    data:
-        system_config: '{%s}'
-    kind: ConfigMap
-    metadata:
-        name: system-config-json
-    """ % (conf_string)
-    return yaml.load(sys_conf, yaml.Loader)
+    conf_str = '{%s}' % (conf_string)
+    sys_conf = {'apiVersion': 'v1',
+          'data': {'system_config': conf_str},
+          'kind': 'ConfigMap',
+          'metadata': {'name' : 'system-config-json'}}
+    return sys_conf
 
 # print next steps on how to use generated system-cm.yaml
 # TODO: generalize next steps for different deploy stratagies