)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6cb48f0a74ec8aca6f254b5544e19821bd9851e9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     damini \u003cdamini.chopra45@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-01-30 13:03:00 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Fix of Shelved and unshelved"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is for Tempest Test in which Instance created by"},{"line_number":10,"context_line":"non-admin user which changes status to \u0027ACTIVE\u0027,shelved by"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f93b717_42179046","line":7,"range":{"start_line":7,"start_character":5,"end_line":7,"end_character":33},"updated":"2018-01-30 08:21:02.000000000","message":"i think this should be testing shelve, unshelve. something like:\n\n[WIP]Add tests for shelve/unshelve operation with different tenant","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":27616,"name":"Damini Chopra","email":"damini.chopra45@gmail.com","username":"damini"},"change_message_id":"968ccaff5e2399c3fe19888008fa5b956f5e1fd4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     damini \u003cdamini.chopra45@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-01-30 13:03:00 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Fix of Shelved and unshelved"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is for Tempest Test in which Instance created by"},{"line_number":10,"context_line":"non-admin user which changes status to \u0027ACTIVE\u0027,shelved by"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f93b717_b96d6acd","line":7,"range":{"start_line":7,"start_character":5,"end_line":7,"end_character":33},"in_reply_to":"5f93b717_42179046","updated":"2018-02-06 10:26:42.000000000","message":"Done","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6cb48f0a74ec8aca6f254b5544e19821bd9851e9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch is for Tempest Test in which Instance created by"},{"line_number":10,"context_line":"non-admin user which changes status to \u0027ACTIVE\u0027,shelved by"},{"line_number":11,"context_line":"admin user and changes its status to \u0027SHELVED_OFFLOADED\u0027 and"},{"line_number":12,"context_line":"then unshelved by non-admin user changes its status to \u0027ACTIVE\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"closes-bug : #1675791"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f93b717_4230f0e8","line":12,"range":{"start_line":12,"start_character":18,"end_line":12,"end_character":32},"updated":"2018-01-30 08:21:02.000000000","message":"non-admin user(owner)","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":27616,"name":"Damini Chopra","email":"damini.chopra45@gmail.com","username":"damini"},"change_message_id":"968ccaff5e2399c3fe19888008fa5b956f5e1fd4","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch is for Tempest Test in which Instance created by"},{"line_number":10,"context_line":"non-admin user which changes status to \u0027ACTIVE\u0027,shelved by"},{"line_number":11,"context_line":"admin user and changes its status to \u0027SHELVED_OFFLOADED\u0027 and"},{"line_number":12,"context_line":"then unshelved by non-admin user changes its status to \u0027ACTIVE\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"closes-bug : #1675791"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f93b717_b9a24afb","line":12,"range":{"start_line":12,"start_character":18,"end_line":12,"end_character":32},"in_reply_to":"5f93b717_4230f0e8","updated":"2018-02-06 10:26:42.000000000","message":"Done","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6cb48f0a74ec8aca6f254b5544e19821bd9851e9","unresolved":false,"context_lines":[{"line_number":11,"context_line":"admin user and changes its status to \u0027SHELVED_OFFLOADED\u0027 and"},{"line_number":12,"context_line":"then unshelved by non-admin user changes its status to \u0027ACTIVE\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"closes-bug : #1675791"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: If061f53ed831f104c9b0236da4868ecabab71856"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f93b717_823fd8b2","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":21},"updated":"2018-01-30 08:21:02.000000000","message":"Related-Bug: #1675791\n\nas it is not fixing the bug, it is just add test for that","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":27616,"name":"Damini Chopra","email":"damini.chopra45@gmail.com","username":"damini"},"change_message_id":"968ccaff5e2399c3fe19888008fa5b956f5e1fd4","unresolved":false,"context_lines":[{"line_number":11,"context_line":"admin user and changes its status to \u0027SHELVED_OFFLOADED\u0027 and"},{"line_number":12,"context_line":"then unshelved by non-admin user changes its status to \u0027ACTIVE\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"closes-bug : #1675791"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: If061f53ed831f104c9b0236da4868ecabab71856"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f93b717_f900d203","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":21},"in_reply_to":"5f93b717_823fd8b2","updated":"2018-02-06 10:26:42.000000000","message":"Done","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":11278,"name":"Anusha Ramineni","email":"anusha.ramineni@india.nec.com","username":"anusha08"},"change_message_id":"f8ed1400ef62948952a865bb3f1e739dd39f178c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Add Tests for shelve/unshelve operation with different tenant"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is for Tempest Test in which Instance created by"},{"line_number":10,"context_line":"non-admin user which changes status to \u0027ACTIVE\u0027,shelved by"},{"line_number":11,"context_line":"admin user and changes its status to \u0027FORCE_SHELVED_OFFLOAD\u0027 and"},{"line_number":12,"context_line":"then unshelved by owner changes its status to \u0027ACTIVE\u0027"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5f93b717_681fbd07","line":9,"range":{"start_line":9,"start_character":10,"end_line":9,"end_character":31},"updated":"2018-02-01 16:35:45.000000000","message":"is to add tempest test","commit_id":"4478929a0b2a5756e38b0df9833ffb5a483e3c43"},{"author":{"_account_id":11278,"name":"Anusha Ramineni","email":"anusha.ramineni@india.nec.com","username":"anusha08"},"change_message_id":"f8ed1400ef62948952a865bb3f1e739dd39f178c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[WIP]Add Tests for shelve/unshelve operation with different tenant"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch is for Tempest Test in which Instance created by"},{"line_number":10,"context_line":"non-admin user which changes status to \u0027ACTIVE\u0027,shelved by"},{"line_number":11,"context_line":"admin user and changes its status to \u0027FORCE_SHELVED_OFFLOAD\u0027 and"},{"line_number":12,"context_line":"then unshelved by owner changes its status to \u0027ACTIVE\u0027"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5f93b717_a8de253b","line":10,"range":{"start_line":10,"start_character":21,"end_line":10,"end_character":47},"updated":"2018-02-01 16:35:45.000000000","message":"mentioning status wont be required, as its by default.","commit_id":"4478929a0b2a5756e38b0df9833ffb5a483e3c43"},{"author":{"_account_id":11278,"name":"Anusha Ramineni","email":"anusha.ramineni@india.nec.com","username":"anusha08"},"change_message_id":"f8ed1400ef62948952a865bb3f1e739dd39f178c","unresolved":false,"context_lines":[{"line_number":10,"context_line":"non-admin user which changes status to \u0027ACTIVE\u0027,shelved by"},{"line_number":11,"context_line":"admin user and changes its status to \u0027FORCE_SHELVED_OFFLOAD\u0027 and"},{"line_number":12,"context_line":"then unshelved by owner changes its status to \u0027ACTIVE\u0027"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: If061f53ed831f104c9b0236da4868ecabab71856"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5f93b717_a8b705f5","line":13,"updated":"2018-02-01 16:35:45.000000000","message":"Related-Bug: #1675791 is still missing from previous comment","commit_id":"4478929a0b2a5756e38b0df9833ffb5a483e3c43"}],"tempest/api/compute/admin/test_servers.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6cb48f0a74ec8aca6f254b5544e19821bd9851e9","unresolved":false,"context_lines":[{"line_number":199,"context_line":"                                wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"    @decorators.idempotent_id(\u0027ca9639bd-ca25-41a9-a45d-21749161c32e\u0027)"},{"line_number":202,"context_line":"    def test_shelve_unshelve(self):"},{"line_number":203,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":204,"context_line":"        self.client.shelve_server(server[\u0027id\u0027])"},{"line_number":205,"context_line":"        waiters.wait_for_server_status(self.client,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f93b717_02a2e8fd","line":202,"range":{"start_line":202,"start_character":8,"end_line":202,"end_character":28},"updated":"2018-01-30 08:21:02.000000000","message":"how about \"test_shelve_unshelve_server_with_different_tenant\"","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6cb48f0a74ec8aca6f254b5544e19821bd9851e9","unresolved":false,"context_lines":[{"line_number":201,"context_line":"    @decorators.idempotent_id(\u0027ca9639bd-ca25-41a9-a45d-21749161c32e\u0027)"},{"line_number":202,"context_line":"    def test_shelve_unshelve(self):"},{"line_number":203,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":204,"context_line":"        self.client.shelve_server(server[\u0027id\u0027])"},{"line_number":205,"context_line":"        waiters.wait_for_server_status(self.client,"},{"line_number":206,"context_line":"                                       server[\u0027id\u0027],"},{"line_number":207,"context_line":"                                       \u0027SHELVED_OFFLOADED\u0027,"},{"line_number":208,"context_line":"                                       )"},{"line_number":209,"context_line":"        self.non_admin_client.unshelve_server(server[\u0027id\u0027])"},{"line_number":210,"context_line":"        waiters.wait_for_server_status(self.client,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f93b717_02ce682d","line":207,"range":{"start_line":204,"start_character":0,"end_line":207,"end_character":59},"updated":"2018-01-30 08:21:02.000000000","message":"like this -\nreplace line L204 - L208 with \n https://github.com/openstack/tempest/blob/c961a656ccdc0f1242b4ff3237a16d4a7cdf4e07/tempest/scenario/test_shelve_instance.py#L46","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6cb48f0a74ec8aca6f254b5544e19821bd9851e9","unresolved":false,"context_lines":[{"line_number":204,"context_line":"        self.client.shelve_server(server[\u0027id\u0027])"},{"line_number":205,"context_line":"        waiters.wait_for_server_status(self.client,"},{"line_number":206,"context_line":"                                       server[\u0027id\u0027],"},{"line_number":207,"context_line":"                                       \u0027SHELVED_OFFLOADED\u0027,"},{"line_number":208,"context_line":"                                       )"},{"line_number":209,"context_line":"        self.non_admin_client.unshelve_server(server[\u0027id\u0027])"},{"line_number":210,"context_line":"        waiters.wait_for_server_status(self.client,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f93b717_c25e4075","line":207,"range":{"start_line":207,"start_character":40,"end_line":207,"end_character":58},"updated":"2018-01-30 08:21:02.000000000","message":"thi actually depend on CONF.compute.shelved_offload_time. you can use shelve method from tempest.commom","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6cb48f0a74ec8aca6f254b5544e19821bd9851e9","unresolved":false,"context_lines":[{"line_number":210,"context_line":"        waiters.wait_for_server_status(self.client,"},{"line_number":211,"context_line":"                                       server[\u0027id\u0027],"},{"line_number":212,"context_line":"                                       \u0027ACTIVE\u0027,"},{"line_number":213,"context_line":"                                       )"},{"line_number":214,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5f93b717_42df90d5","line":213,"range":{"start_line":213,"start_character":39,"end_line":213,"end_character":40},"updated":"2018-01-30 08:21:02.000000000","message":"you can close it in L212 only","commit_id":"b1f3b3acb44cebc7f9263813b88a95ae3f068815"},{"author":{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},"change_message_id":"799d56d5d8b177b7c09c6279129fe1aea870e0cd","unresolved":false,"context_lines":[{"line_number":200,"context_line":"                                wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":201,"context_line":""},{"line_number":202,"context_line":"    @decorators.idempotent_id(\u0027ca9639bd-ca25-41a9-a45d-21749161c32e\u0027)"},{"line_number":203,"context_line":"    def test_shelve_unshelve_server_with_different_tenant(self):"},{"line_number":204,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":205,"context_line":"        compute.shelve_server(self.servers_client, server[\u0027id\u0027],"},{"line_number":206,"context_line":"                              force_shelve_offload\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5f93b717_de2d5d95","line":203,"range":{"start_line":203,"start_character":3,"end_line":203,"end_character":4},"updated":"2018-02-02 06:10:53.000000000","message":"Also if you can add comments in the code, it will make easy to understand code. \n\n# create server with non-admin user (owner)\nserver \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)\n# shelve server with admin\ncompute.shelve_server(self.servers_client, server[\u0027id\u0027],\n                      force_shelve_offload\u003dTrue)\n# unshelve server with owner\nself.non_admin_client.unshelve_server(server[\u0027id\u0027])\nwaiters.wait_for_server_status(self.client,\n                               server[\u0027id\u0027],\n                               \u0027ACTIVE\u0027)","commit_id":"4478929a0b2a5756e38b0df9833ffb5a483e3c43"},{"author":{"_account_id":11278,"name":"Anusha Ramineni","email":"anusha.ramineni@india.nec.com","username":"anusha08"},"change_message_id":"f8ed1400ef62948952a865bb3f1e739dd39f178c","unresolved":false,"context_lines":[{"line_number":202,"context_line":"    @decorators.idempotent_id(\u0027ca9639bd-ca25-41a9-a45d-21749161c32e\u0027)"},{"line_number":203,"context_line":"    def test_shelve_unshelve_server_with_different_tenant(self):"},{"line_number":204,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":205,"context_line":"        compute.shelve_server(self.servers_client, server[\u0027id\u0027],"},{"line_number":206,"context_line":"                              force_shelve_offload\u003dTrue)"},{"line_number":207,"context_line":"        self.non_admin_client.unshelve_server(server[\u0027id\u0027])"},{"line_number":208,"context_line":"        waiters.wait_for_server_status(self.client,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5f93b717_e8bd0db7","line":205,"range":{"start_line":205,"start_character":30,"end_line":205,"end_character":50},"updated":"2018-02-01 16:35:45.000000000","message":"is this admin client?","commit_id":"4478929a0b2a5756e38b0df9833ffb5a483e3c43"},{"author":{"_account_id":19457,"name":"aditi sharma","email":"adi.sky17@gmail.com","username":"aditi"},"change_message_id":"799d56d5d8b177b7c09c6279129fe1aea870e0cd","unresolved":false,"context_lines":[{"line_number":202,"context_line":"    @decorators.idempotent_id(\u0027ca9639bd-ca25-41a9-a45d-21749161c32e\u0027)"},{"line_number":203,"context_line":"    def test_shelve_unshelve_server_with_different_tenant(self):"},{"line_number":204,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":205,"context_line":"        compute.shelve_server(self.servers_client, server[\u0027id\u0027],"},{"line_number":206,"context_line":"                              force_shelve_offload\u003dTrue)"},{"line_number":207,"context_line":"        self.non_admin_client.unshelve_server(server[\u0027id\u0027])"},{"line_number":208,"context_line":"        waiters.wait_for_server_status(self.client,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5f93b717_630b5229","line":205,"range":{"start_line":205,"start_character":30,"end_line":205,"end_character":50},"in_reply_to":"5f93b717_e8bd0db7","updated":"2018-02-02 06:10:53.000000000","message":"This seems to be non-admin client\nhttps://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L67","commit_id":"4478929a0b2a5756e38b0df9833ffb5a483e3c43"}]}
