)]}'
{"ironic_tempest_plugin/tests/scenario/ironic_standalone/test_basic_ops.py":[{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"088c2da741aa9e91f6a166ca013d46de2ff2738d","unresolved":false,"context_lines":[{"line_number":153,"context_line":""},{"line_number":154,"context_line":"    api_microversion \u003d \u00271.31\u0027  # to set the deploy_interface"},{"line_number":155,"context_line":"    driver \u003d \u0027ipmi\u0027"},{"line_number":156,"context_line":"    deploy_interface \u003d \u0027ansible\u0027"},{"line_number":157,"context_line":"    image_ref \u003d CONF.baremetal.whole_disk_image_ref"},{"line_number":158,"context_line":"    wholedisk_image \u003d True"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f91af0f_bab713d9","line":156,"updated":"2017-12-29 14:09:42.000000000","message":"Mentioned this in IRC, but we should add a skip_checks method in this class to check if the ansible interface is enabled (probably via `GET /v1/drivers`). This will make it so this doesn\u0027t fail on pike, and will also help downstream consumers that don\u0027t enable this deploy interface.\n\nWe should actually do that for all of the classes in this file, for the downstream folks, but in a separate patch. :)","commit_id":"34c8bbdce6029f31af066ff0981cab52ad0503a4"},{"author":{"_account_id":9542,"name":"Pavlo Shchelokovskyy","email":"pshchelokovskyy@mirantis.com","username":"pshchelo"},"change_message_id":"b71665f0992eb9ea1235d04906e76fa4cedc9182","unresolved":false,"context_lines":[{"line_number":153,"context_line":""},{"line_number":154,"context_line":"    api_microversion \u003d \u00271.31\u0027  # to set the deploy_interface"},{"line_number":155,"context_line":"    driver \u003d \u0027ipmi\u0027"},{"line_number":156,"context_line":"    deploy_interface \u003d \u0027ansible\u0027"},{"line_number":157,"context_line":"    image_ref \u003d CONF.baremetal.whole_disk_image_ref"},{"line_number":158,"context_line":"    wholedisk_image \u003d True"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f91af0f_3a76a3da","line":156,"in_reply_to":"9f91af0f_bab713d9","updated":"2017-12-29 15:12:36.000000000","message":"there is a config option added to tempest plugin, which is list of enabled deploy interfaces. Devstack (in master) populates it from what ever was passed to it (or default iscsi,direct is used, both in ironic, devstack and tempest plugin).\n\nIt is introduced in a previous patch in this series, and the failing job does not has \u0027ansible\u0027 enabled in tempest config, so I\u0027m puzzled why this test was run at all and not skipped...\n\nWill add couple of logs to see if the cls.skip_checks is even called...","commit_id":"34c8bbdce6029f31af066ff0981cab52ad0503a4"}]}