)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"999be1e15db1511284a826e582c99b6aaf31affe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7758cdcf_5d44febc","updated":"2022-08-16 20:02:27.000000000","message":"https://review.opendev.org/c/openstack/tempest/+/850518","commit_id":"37366af920f72419f3ec41af7359b31eb1f93349"}],"tempest/api/compute/admin/test_volume_swap.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"64ed42f2117519528565f268619a82f04afe210e","unresolved":true,"context_lines":[{"line_number":164,"context_line":"        if not CONF.compute_feature_enabled.volume_multiattach:"},{"line_number":165,"context_line":"            raise cls.skipException(\u0027Volume multi-attach is not available.\u0027)"},{"line_number":166,"context_line":"        if CONF.validation.run_validation:"},{"line_number":167,"context_line":"            raise cls.skipException(\u0027Run validation is not supported yet\u0027)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    @classmethod"},{"line_number":170,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"98af7400_0c31895e","line":167,"updated":"2022-07-06 12:22:07.000000000","message":"Not sure i understand the issue , cant we create two seperate instances in two differnet command instead using count in line #228\n\nWhich storage backend you run for swapping ?","commit_id":"37366af920f72419f3ec41af7359b31eb1f93349"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"b07999ea6e42f6cdee03add6577b3194da55a953","unresolved":true,"context_lines":[{"line_number":164,"context_line":"        if not CONF.compute_feature_enabled.volume_multiattach:"},{"line_number":165,"context_line":"            raise cls.skipException(\u0027Volume multi-attach is not available.\u0027)"},{"line_number":166,"context_line":"        if CONF.validation.run_validation:"},{"line_number":167,"context_line":"            raise cls.skipException(\u0027Run validation is not supported yet\u0027)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    @classmethod"},{"line_number":170,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"f40159eb_678c1530","line":167,"in_reply_to":"58a497ab_e4373e43","updated":"2022-08-16 18:49:55.000000000","message":"Tempest /says/ it doesn\u0027t support validating multiple servers [1], but is that still true?\n\nI tried a local patch that removes that restriction and test_volume_swap_with_multiattach passes.\n\n[1] https://github.com/openstack/tempest/blob/master/tempest/common/compute.py#L214","commit_id":"37366af920f72419f3ec41af7359b31eb1f93349"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"999be1e15db1511284a826e582c99b6aaf31affe","unresolved":true,"context_lines":[{"line_number":164,"context_line":"        if not CONF.compute_feature_enabled.volume_multiattach:"},{"line_number":165,"context_line":"            raise cls.skipException(\u0027Volume multi-attach is not available.\u0027)"},{"line_number":166,"context_line":"        if CONF.validation.run_validation:"},{"line_number":167,"context_line":"            raise cls.skipException(\u0027Run validation is not supported yet\u0027)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    @classmethod"},{"line_number":170,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"e768fe45_6a55a2f5","line":167,"in_reply_to":"58a497ab_e4373e43","updated":"2022-08-16 20:02:27.000000000","message":"https://review.opendev.org/c/openstack/tempest/+/850518","commit_id":"37366af920f72419f3ec41af7359b31eb1f93349"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"4b797bc276997b721f74d28eb310a105d238b02a","unresolved":true,"context_lines":[{"line_number":164,"context_line":"        if not CONF.compute_feature_enabled.volume_multiattach:"},{"line_number":165,"context_line":"            raise cls.skipException(\u0027Volume multi-attach is not available.\u0027)"},{"line_number":166,"context_line":"        if CONF.validation.run_validation:"},{"line_number":167,"context_line":"            raise cls.skipException(\u0027Run validation is not supported yet\u0027)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    @classmethod"},{"line_number":170,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"58a497ab_e4373e43","line":167,"in_reply_to":"98af7400_0c31895e","updated":"2022-07-20 10:46:19.000000000","message":"tempest does not support multiple validation ping but its a limitation from tempest side and not from the features.\nwe can tweak it easly for now or add support to nova .","commit_id":"37366af920f72419f3ec41af7359b31eb1f93349"}]}