Testing looks good. I'm surprised the pep8 tests are passing due to excessive line lengths in doc/source/devref/filter_scheduler.rst and etc/nova/nova.conf.sample. Perhaps such files aren't covered in the tests? Not sure the best way to handle wrapping long lines in those files, so any advice is appreciated.
I'll still need to cook up a patch to remove permitted_instance_types. It is late here, so will take care of it tomorrow.
Ok, I took a first stab at a patch
https:/ /review. openstack. org/#/c/ 11454/
Testing looks good. I'm surprised the pep8 tests are passing due to excessive line lengths in doc/source/ devref/ filter_ scheduler. rst and etc/nova/ nova.conf. sample. Perhaps such files aren't covered in the tests? Not sure the best way to handle wrapping long lines in those files, so any advice is appreciated.
I'll still need to cook up a patch to remove permitted_ instance_ types. It is late here, so will take care of it tomorrow.