search

sciapp / nojava-ipmi-kvm

nojava-ipmi-kvm is a utility to access Java based ipmi kvm consoles without a local java installation.
MIT License
78 stars 26 forks source link

Fix invalid YAML syntax in README #7

Closed ettom closed 3 years ago

ettom commented 3 years ago

The previous example config gave an error:

Traceback (most recent call last):
  File "/usr/bin/nojava-ipmi-kvm", line 33, in <module>
    sys.exit(load_entry_point('nojava-ipmi-kvm==0.9.0', 'console_scripts', 'nojava-ipmi-kvm')())
  File "/usr/lib/python3.8/site-packages/nojava_ipmi_kvm/cli.py", line 130, in main
    host_config = config[args.hostname]
  File "/usr/lib/python3.8/site-packages/nojava_ipmi_kvm/config.py", line 222, in __getitem__
    for k, v in self._config_dict["templates"][raw_host["based_on"]].items():
AttributeError: 'str' object has no attribute 'items'

Key-value pairs are separated by : in YAML.

IngoMeyer441 commented 3 years ago

Thanks, you are right! That was a copy-paste error from the old config format (ini).