)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"653e40af040461213d5265603fd15bc7a78626d7","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is being set:- object-store-feature-enabled, but it should be:"},{"line_number":11,"context_line":"object-storage-feature-enabled [1]."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1] https://github.com/openstack/tempest/blob/ab3c306b6016cf95b60d64fe2bfe330d9a427719/tempest/config.py#L913"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I052b7ed371d6daee886fa94e88ccca7390cebc38"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_1a7a8b76","line":13,"range":{"start_line":13,"start_character":4,"end_line":13,"end_character":109},"updated":"2018-07-02 07:38:41.000000000","message":"please use git.openstack.org url here.","commit_id":"f41850c33bd61f07020d0978aa796996f6ef8bad"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"abd2d32c14dadd813d58769f1fa2d150c023d8d0","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is being set:- object-store-feature-enabled, but it should be:"},{"line_number":11,"context_line":"object-storage-feature-enabled [1]."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1] https://github.com/openstack/tempest/blob/ab3c306b6016cf95b60d64fe2bfe330d9a427719/tempest/config.py#L913"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I052b7ed371d6daee886fa94e88ccca7390cebc38"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_7ad13f56","line":13,"range":{"start_line":13,"start_character":4,"end_line":13,"end_character":109},"in_reply_to":"5f7c97a3_1a7a8b76","updated":"2018-07-02 07:43:48.000000000","message":"Ok, will fix that.","commit_id":"f41850c33bd61f07020d0978aa796996f6ef8bad"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"76094672096066e41629b6bdb6b6f09bd87c112f","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is being set:- object-store-feature-enabled, but it should be:"},{"line_number":11,"context_line":"object-storage-feature-enabled [1]."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1] https://github.com/openstack/tempest/blob/ab3c306b6016cf95b60d64fe2bfe330d9a427719/tempest/config.py#L913"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I052b7ed371d6daee886fa94e88ccca7390cebc38"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_dad653c5","line":13,"range":{"start_line":13,"start_character":4,"end_line":13,"end_character":109},"in_reply_to":"5f7c97a3_7ad13f56","updated":"2018-07-02 08:13:39.000000000","message":"Done","commit_id":"f41850c33bd61f07020d0978aa796996f6ef8bad"}],"config_tempest/services/services.py":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"596292691032350f1716c14a40815d2ce7e63136","unresolved":false,"context_lines":[{"line_number":240,"context_line":"                supported_versions \u003d []"},{"line_number":241,"context_line":"            else:"},{"line_number":242,"context_line":"                supported_versions \u003d service_object.get_versions()"},{"line_number":243,"context_line":"            if service \u003d\u003d \u0027object-store\u0027:"},{"line_number":244,"context_line":"                service \u003d \u0027object-storage\u0027"},{"line_number":245,"context_line":"            section \u003d service + \u0027-feature-enabled\u0027"},{"line_number":246,"context_line":"            for version in service_info[\u0027supported_versions\u0027]:"},{"line_number":247,"context_line":"                is_supported \u003d any(version in item"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_da65f3e3","line":244,"range":{"start_line":243,"start_character":12,"end_line":244,"end_character":42},"updated":"2018-07-02 07:36:47.000000000","message":"Thanks for the patch,\n We are making the services class stable we donot want to carry workaround here.\n\nDoes changing this params at this place: https://github.com/openstack/python-tempestconf/blob/master/config_tempest/constants.py#L87 does not fixes the issue?","commit_id":"f41850c33bd61f07020d0978aa796996f6ef8bad"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"a7cf962665bc1917c21c3b9ba2ae46466fda017c","unresolved":false,"context_lines":[{"line_number":240,"context_line":"                supported_versions \u003d []"},{"line_number":241,"context_line":"            else:"},{"line_number":242,"context_line":"                supported_versions \u003d service_object.get_versions()"},{"line_number":243,"context_line":"            if service \u003d\u003d \u0027object-store\u0027:"},{"line_number":244,"context_line":"                service \u003d \u0027object-storage\u0027"},{"line_number":245,"context_line":"            section \u003d service + \u0027-feature-enabled\u0027"},{"line_number":246,"context_line":"            for version in service_info[\u0027supported_versions\u0027]:"},{"line_number":247,"context_line":"                is_supported \u003d any(version in item"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_bac0374c","line":244,"range":{"start_line":243,"start_character":12,"end_line":244,"end_character":42},"in_reply_to":"5f7c97a3_5acec3b7","updated":"2018-07-02 07:58:38.000000000","message":"Then please add a FIXME here and in the next line, because we want to get rid of special cases for specific clients from this file.\n\nFirst solution, in line with the current code, is to add a new table in constants which maps exceptions between the service name and the configuration key.\n\nLong term fix, those details should be moved in each Services class.","commit_id":"f41850c33bd61f07020d0978aa796996f6ef8bad"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"76094672096066e41629b6bdb6b6f09bd87c112f","unresolved":false,"context_lines":[{"line_number":240,"context_line":"                supported_versions \u003d []"},{"line_number":241,"context_line":"            else:"},{"line_number":242,"context_line":"                supported_versions \u003d service_object.get_versions()"},{"line_number":243,"context_line":"            if service \u003d\u003d \u0027object-store\u0027:"},{"line_number":244,"context_line":"                service \u003d \u0027object-storage\u0027"},{"line_number":245,"context_line":"            section \u003d service + \u0027-feature-enabled\u0027"},{"line_number":246,"context_line":"            for version in service_info[\u0027supported_versions\u0027]:"},{"line_number":247,"context_line":"                is_supported \u003d any(version in item"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_1acdab2f","line":244,"range":{"start_line":243,"start_character":12,"end_line":244,"end_character":42},"in_reply_to":"5f7c97a3_bac0374c","updated":"2018-07-02 08:13:39.000000000","message":"Done","commit_id":"f41850c33bd61f07020d0978aa796996f6ef8bad"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"abd2d32c14dadd813d58769f1fa2d150c023d8d0","unresolved":false,"context_lines":[{"line_number":240,"context_line":"                supported_versions \u003d []"},{"line_number":241,"context_line":"            else:"},{"line_number":242,"context_line":"                supported_versions \u003d service_object.get_versions()"},{"line_number":243,"context_line":"            if service \u003d\u003d \u0027object-store\u0027:"},{"line_number":244,"context_line":"                service \u003d \u0027object-storage\u0027"},{"line_number":245,"context_line":"            section \u003d service + \u0027-feature-enabled\u0027"},{"line_number":246,"context_line":"            for version in service_info[\u0027supported_versions\u0027]:"},{"line_number":247,"context_line":"                is_supported \u003d any(version in item"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_5acec3b7","line":244,"range":{"start_line":243,"start_character":12,"end_line":244,"end_character":42},"in_reply_to":"5f7c97a3_da65f3e3","updated":"2018-07-02 07:43:48.000000000","message":"no, The keys defined there refer to service, object-store in this case. The issue we faced because tempestconf assumes config param(object-storage) and service name(object-store) to be same","commit_id":"f41850c33bd61f07020d0978aa796996f6ef8bad"}]}