insights-core icon indicating copy to clipboard operation
insights-core copied to clipboard

Add parser for HammerComputeResourceList

Open pipopopo opened this issue 4 years ago • 2 comments

Attempt to create a parser for HammerComputeResourceList

# py.test -svk hammer_compute_resource_list | grep hammer_compute_resource_list

/tmp/git/insights-core/insights/combiners/nfs_exports.py:73: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3,and in 3.9 it will stop working
  if isinstance(nfsexportsd, collections.Iterable):
insights/parsers/tests/test_hammer_compute_resource_list.py::test_hammer_compute_resource_list PASSED
insights/parsers/tests/test_hammer_compute_resource_list.py::test_no_data PASSED

pipopopo avatar Mar 28 '20 12:03 pipopopo

Can one of the admins verify this patch?

jenkins-qa-bot avatar Mar 28 '20 12:03 jenkins-qa-bot

@pipopopo assuming that you have setup your environment as recommended in CONTRIBUTING.md and the Wiki, you can run both pytest and flake8 in the root of your repo to ensure that you have no errors. You can also run make clean html_debug in the docs subdir to verify that your docstrings are correct and you can review your generated docs in docs/_build/html/index.html. If all of these are satisfactory then the CI tests are more likely to pass when you commit your changes. More info on documentation requirements and testing are available in the Core docs.

bfahr avatar Mar 30 '20 21:03 bfahr