)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"df219d1e1381e5cd83c0955a0efff88246b9a1ea","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for CINDER services v1 APIs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FM8E%3D","line":9,"updated":"2014-04-23 11:20:21.000000000","message":"A XML client is also added","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"4b7bca41913b81a0e21b43c9ac869c56fd015c00","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for CINDER services v1 APIs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FMwU%3D","line":9,"in_reply_to":"AAAAXH%2F%2FM1A%3D","updated":"2014-04-23 11:50:47.000000000","message":"Looking at patchset 4 there\u0027s a file tempest/services/volume/xml/admin/volume_services_client.py which seems to be an XML client. The commit message doesn\u0027t mention this XML client","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"0363a4bb5a017703212b4902c503cdfb5f480072","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for CINDER services v1 APIs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FM1A%3D","line":9,"in_reply_to":"AAAAXH%2F%2FM8E%3D","updated":"2014-04-23 11:40:41.000000000","message":"No XML tests are added and only 1 XML client is there.\nI have skipped the XML clients and tests for later.","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"96f823de0792980c09b7d137c08931044fb95a81","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for CINDER services v1 APIs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FFo8%3D","line":9,"in_reply_to":"AAAAXH%2F%2FMwU%3D","updated":"2014-04-24 05:35:22.000000000","message":"Done","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"df219d1e1381e5cd83c0955a0efff88246b9a1ea","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idd0201fd506334c5ff467ac9db9b53cadd1ee45f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FM8M%3D","line":11,"updated":"2014-04-23 11:20:21.000000000","message":"It seems 2 and 3 are missing tests.","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"4b7bca41913b81a0e21b43c9ac869c56fd015c00","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idd0201fd506334c5ff467ac9db9b53cadd1ee45f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FMuY%3D","line":11,"in_reply_to":"AAAAXH%2F%2FM1Y%3D","updated":"2014-04-23 11:50:47.000000000","message":"Hummm ok ! To me the commit message is not clear. You said : \"This patch adds the JSON clients !*and*! tests for below listed cinder service APIs\". But some items of the list are only for the JSON client and not the tests.","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"0363a4bb5a017703212b4902c503cdfb5f480072","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idd0201fd506334c5ff467ac9db9b53cadd1ee45f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FM1Y%3D","line":11,"in_reply_to":"AAAAXH%2F%2FM8M%3D","updated":"2014-04-23 11:40:41.000000000","message":"Their JSON clients are there.\nThey can\u0027t have positive tests as they may change the actual environment and may cause enable/disable of any imp service.\nBut they will be needed for Negative tests.","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"96f823de0792980c09b7d137c08931044fb95a81","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the JSON clients and tests for below listed cinder service APIs -"},{"line_number":10,"context_line":"1. service-list"},{"line_number":11,"context_line":"2. service-enable"},{"line_number":12,"context_line":"3. service-disable"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idd0201fd506334c5ff467ac9db9b53cadd1ee45f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAXH%2F%2FFo4%3D","line":11,"in_reply_to":"AAAAXH%2F%2FMuY%3D","updated":"2014-04-24 05:35:22.000000000","message":"Done","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"}],"tempest/api/volume/admin/test_volume_services.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e41a89b770d5a61331516d62e66b78ee41f71de9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2013 NEC Technologies India Ltd."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":5,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FzTg%3D","line":2,"updated":"2014-04-17 10:57:35.000000000","message":"correct the year.","commit_id":"0f1601b2082e82da8166b41ec56c94752b6a6284"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e41a89b770d5a61331516d62e66b78ee41f71de9","unresolved":false,"context_lines":[{"line_number":20,"context_line":"class VolumesServicesTestJSON(base.BaseVolumeV1AdminTest):"},{"line_number":21,"context_line":"    \"\"\""},{"line_number":22,"context_line":"    Tests Volume Services API."},{"line_number":23,"context_line":"    List and Enable/Disable require admin privileges."},{"line_number":24,"context_line":"    \"\"\""},{"line_number":25,"context_line":"    _interface \u003d \"json\""},{"line_number":26,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FzQw%3D","line":23,"updated":"2014-04-17 10:57:35.000000000","message":"correct the documentation as it is not doing enable/disable right now.","commit_id":"0f1601b2082e82da8166b41ec56c94752b6a6284"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e41a89b770d5a61331516d62e66b78ee41f71de9","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":51,"context_line":"    def test_get_service_by_host_name(self):"},{"line_number":52,"context_line":"        services_on_host \u003d [service for service in self.services if"},{"line_number":53,"context_line":"                            service[\u0027host\u0027] \u003d\u003d self.host_name]"},{"line_number":54,"context_line":"        params \u003d {\u0027host\u0027: self.host_name}"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        resp, services \u003d self.client.list_services(params)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FzRM%3D","line":53,"updated":"2014-04-17 10:57:35.000000000","message":"I think it is the same what list_services does with param filter. Cannot we just check the host name of the services like in previous test.","commit_id":"0f1601b2082e82da8166b41ec56c94752b6a6284"}],"tempest/clients.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"0943722ea52cd35bc4675149591d6ede69ce695a","unresolved":false,"context_lines":[{"line_number":555,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":556,"context_line":"        region \u003d CONF.identity.region"},{"line_number":557,"context_line":"        endpoint_type \u003d CONF.volume.endpoint_type"},{"line_number":558,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":559,"context_line":"        return cinderclient.client.Client(self.CINDERCLIENT_VERSION,"},{"line_number":560,"context_line":"                                          username,"},{"line_number":561,"context_line":"                                          password,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2FR34%3D","side":"PARENT","line":558,"updated":"2014-04-15 15:52:45.000000000","message":"Could you explain this deletion ? If it\u0027s not related to your test, I\u0027d rather see it in another patch","commit_id":"447c3878311431936755917d92b918dfe0b95288"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"adc29eae885cff9d9df55142392d0115774ad406","unresolved":false,"context_lines":[{"line_number":555,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":556,"context_line":"        region \u003d CONF.identity.region"},{"line_number":557,"context_line":"        endpoint_type \u003d CONF.volume.endpoint_type"},{"line_number":558,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":559,"context_line":"        return cinderclient.client.Client(self.CINDERCLIENT_VERSION,"},{"line_number":560,"context_line":"                                          username,"},{"line_number":561,"context_line":"                                          password,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2FRhk%3D","side":"PARENT","line":558,"in_reply_to":"AAAAW3%2F%2FR34%3D","updated":"2014-04-15 16:24:10.000000000","message":"Yup, This is not part of my patch.\nMay be this deletion took place due to simultaneous updation of patches and me having old branch.\nI will add this line in my patch.","commit_id":"447c3878311431936755917d92b918dfe0b95288"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"aca66fea7957ffe0df44302906378759410b6420","unresolved":false,"context_lines":[{"line_number":555,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":556,"context_line":"        region \u003d CONF.identity.region"},{"line_number":557,"context_line":"        endpoint_type \u003d CONF.volume.endpoint_type"},{"line_number":558,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":559,"context_line":"        return cinderclient.client.Client(self.CINDERCLIENT_VERSION,"},{"line_number":560,"context_line":"                                          username,"},{"line_number":561,"context_line":"                                          password,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2F%2FBA%3D","side":"PARENT","line":558,"in_reply_to":"AAAAW3%2F%2FRhk%3D","updated":"2014-04-16 03:27:54.000000000","message":"You should rebase with latest master branch. after that those changes should not reflect in this patch.","commit_id":"447c3878311431936755917d92b918dfe0b95288"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e41a89b770d5a61331516d62e66b78ee41f71de9","unresolved":false,"context_lines":[{"line_number":562,"context_line":"    def _get_volume_client(self, username, password, tenant_name):"},{"line_number":563,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":564,"context_line":"        region \u003d CONF.identity.region"},{"line_number":565,"context_line":"        endpoint_type \u003d CONF.volume.endpoint_type"},{"line_number":566,"context_line":"        return cinderclient.client.Client(self.CINDERCLIENT_VERSION,"},{"line_number":567,"context_line":"                                          username,"},{"line_number":568,"context_line":"                                          password,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FzQc%3D","line":565,"updated":"2014-04-17 10:57:35.000000000","message":"Correct this as per comment. rebase.","commit_id":"0f1601b2082e82da8166b41ec56c94752b6a6284"}],"tempest/services/volume/json/admin/volume_services_client.py":[{"author":{"_account_id":8085,"name":"anju Tiwari","email":"anjutiwari5@gmail.com","username":"Anju"},"change_message_id":"78e8af4b624718a7ed6f3245b35c5d1e10498909","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return resp, body[\u0027services\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def enable_service(self, host_name, binary):"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        Enable service on a host"},{"line_number":43,"context_line":"        host_name: Name of host"},{"line_number":44,"context_line":"        binary: Service binary"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FXsk%3D","line":41,"updated":"2014-04-15 04:51:27.000000000","message":"what is the use of this ??","commit_id":"9d9ae88a30cf4a21a5a3bd7717bf882a7ff6c657"},{"author":{"_account_id":8085,"name":"anju Tiwari","email":"anjutiwari5@gmail.com","username":"Anju"},"change_message_id":"29cd837489e718f401086c78311d2d7d4712dfc2","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return resp, body[\u0027services\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def enable_service(self, host_name, binary):"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        Enable service on a host"},{"line_number":43,"context_line":"        host_name: Name of host"},{"line_number":44,"context_line":"        binary: Service binary"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAXH%2F%2F%2FDc%3D","line":41,"in_reply_to":"AAAAW3%2F%2FXfs%3D","updated":"2014-04-16 03:21:18.000000000","message":"I think you can add these clients only when it is necessary.","commit_id":"9d9ae88a30cf4a21a5a3bd7717bf882a7ff6c657"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"f4dba0b2e7a01c5096022bd12a073713f077a206","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return resp, body[\u0027services\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def enable_service(self, host_name, binary):"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        Enable service on a host"},{"line_number":43,"context_line":"        host_name: Name of host"},{"line_number":44,"context_line":"        binary: Service binary"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FXfs%3D","line":41,"in_reply_to":"AAAAW3%2F%2FXic%3D","updated":"2014-04-15 05:39:43.000000000","message":"These are clients and I guess should be present (as they provide a functionality).","commit_id":"9d9ae88a30cf4a21a5a3bd7717bf882a7ff6c657"},{"author":{"_account_id":8085,"name":"anju Tiwari","email":"anjutiwari5@gmail.com","username":"Anju"},"change_message_id":"8d1116e8adbbd99c098feb9625891ce88023e79b","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return resp, body[\u0027services\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def enable_service(self, host_name, binary):"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        Enable service on a host"},{"line_number":43,"context_line":"        host_name: Name of host"},{"line_number":44,"context_line":"        binary: Service binary"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FXic%3D","line":41,"in_reply_to":"AAAAW3%2F%2FXoM%3D","updated":"2014-04-15 05:30:02.000000000","message":"No , I am asking about client enable_service and disable_service.\n\nI don\u0027t think we should add these two, if we are not using it","commit_id":"9d9ae88a30cf4a21a5a3bd7717bf882a7ff6c657"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"b907a0fc6b12250dcf18242c214894556d919170","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        return resp, body[\u0027services\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def enable_service(self, host_name, binary):"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        Enable service on a host"},{"line_number":43,"context_line":"        host_name: Name of host"},{"line_number":44,"context_line":"        binary: Service binary"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FXoM%3D","line":41,"in_reply_to":"AAAAW3%2F%2FXsk%3D","updated":"2014-04-15 04:53:43.000000000","message":"Use of \"\"\" ?\nmultiline comment","commit_id":"9d9ae88a30cf4a21a5a3bd7717bf882a7ff6c657"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"03ce48d3da96e9d151e88d73954fce372e91f662","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        body \u003d json.loads(body)"},{"line_number":38,"context_line":"        return resp, body[\u0027services\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def enable_service(self, host_name, binary):"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        Enable service on a host"},{"line_number":43,"context_line":"        host_name: Name of host"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FJ3M%3D","line":40,"updated":"2014-04-23 17:01:28.000000000","message":"I get what your going for by having a functionally complete service client by adding these methods. But they\u0027re aren\u0027t being used anywhere yet. We don\u0027t want to commit any code which doesn\u0027t get executed in the gate.\n\nSo either add a test which uses this method or remove it from this commit and add it later when it is used.","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"96f823de0792980c09b7d137c08931044fb95a81","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        body \u003d json.loads(body)"},{"line_number":38,"context_line":"        return resp, body[\u0027services\u0027]"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def enable_service(self, host_name, binary):"},{"line_number":41,"context_line":"        \"\"\""},{"line_number":42,"context_line":"        Enable service on a host"},{"line_number":43,"context_line":"        host_name: Name of host"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FFpI%3D","line":40,"in_reply_to":"AAAAXH%2F%2FJ3M%3D","updated":"2014-04-24 05:35:22.000000000","message":"Done","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"03ce48d3da96e9d151e88d73954fce372e91f662","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        body \u003d json.loads(body)"},{"line_number":49,"context_line":"        return resp, body[\u0027service\u0027]"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def disable_service(self, host_name, binary):"},{"line_number":52,"context_line":"        \"\"\""},{"line_number":53,"context_line":"        Disable service on a host"},{"line_number":54,"context_line":"        host_name: Name of host"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FJ2k%3D","line":51,"updated":"2014-04-23 17:01:28.000000000","message":"Same here","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"96f823de0792980c09b7d137c08931044fb95a81","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        body \u003d json.loads(body)"},{"line_number":49,"context_line":"        return resp, body[\u0027service\u0027]"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def disable_service(self, host_name, binary):"},{"line_number":52,"context_line":"        \"\"\""},{"line_number":53,"context_line":"        Disable service on a host"},{"line_number":54,"context_line":"        host_name: Name of host"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FFpE%3D","line":51,"in_reply_to":"AAAAXH%2F%2FJ2k%3D","updated":"2014-04-24 05:35:22.000000000","message":"Done","commit_id":"fe8c656865b5ad585f8ca0330f2ddbec5d3e8d7f"}]}