)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"88b099ad692b9e07fc0d13b645bc2a99d77600fa","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Joshua White \u003cjoshua.l.white@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-22 14:41:08 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Compute test \"test_create_delete_image\" fails"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Test fails when tempest_roles\u003dadmin in tempest.conf"},{"line_number":10,"context_line":"but passes when tempest_roles\u003dmember.  This is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa0719c6_ceef71e7","line":7,"updated":"2016-03-23 20:45:04.000000000","message":"Please change the title to reflect the change.","commit_id":"2d3e32c46190d76b5818252cddfcfba98811b9ed"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"9aa0ab3d54495ed950b0bcde518599eae5dbc20b","unresolved":false,"context_lines":[{"line_number":10,"context_line":"but passes when tempest_roles\u003dmember.  This is"},{"line_number":11,"context_line":"because when checking for delete the admin role will"},{"line_number":12,"context_line":"return a value whether the image is active or deleted"},{"line_number":13,"context_line":"which sends the test in an endless loop until it fails do to time out."},{"line_number":14,"context_line":"Change addresses bug by checking status of returned message."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I4e6937c992398cfc7db5391cfabb101aaefdb7e6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa0719c6_86392044","line":13,"range":{"start_line":13,"start_character":55,"end_line":13,"end_character":57},"updated":"2016-03-23 15:56:19.000000000","message":"typo \"due\"","commit_id":"2d3e32c46190d76b5818252cddfcfba98811b9ed"}],"tempest/lib/services/compute/images_client.py":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"2733154da7c25308982183d01c01ff4e885b218e","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":134,"context_line":"        # When user has admin role a 200 will return even for deleted images"},{"line_number":135,"context_line":"        # which will lead to a return value of False. In this case"},{"line_number":136,"context_line":"        # the status of the image needs to be verified by checking whether"},{"line_number":137,"context_line":"        # it is ACTIVE or DELETED."},{"line_number":138,"context_line":"        try:"},{"line_number":139,"context_line":"            if \u0027image\u0027 in self.show_image(id):"},{"line_number":140,"context_line":"                if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa0719c6_3b35d1f6","line":137,"range":{"start_line":134,"start_character":8,"end_line":137,"end_character":33},"updated":"2016-03-22 22:32:02.000000000","message":"Remove comments, add them on the details of the fix","commit_id":"d79b95bf42e048d1d223af3bef763c2ec55324a5"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"2733154da7c25308982183d01c01ff4e885b218e","unresolved":false,"context_lines":[{"line_number":136,"context_line":"        # the status of the image needs to be verified by checking whether"},{"line_number":137,"context_line":"        # it is ACTIVE or DELETED."},{"line_number":138,"context_line":"        try:"},{"line_number":139,"context_line":"            if \u0027image\u0027 in self.show_image(id):"},{"line_number":140,"context_line":"                if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":141,"context_line":"                    return True"},{"line_number":142,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa0719c6_8110eaa9","line":139,"range":{"start_line":139,"start_character":12,"end_line":139,"end_character":46},"updated":"2016-03-22 22:32:02.000000000","message":"remove the if...\nif action didn\u0027t throw the error then the added if is enough","commit_id":"d79b95bf42e048d1d223af3bef763c2ec55324a5"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"2733154da7c25308982183d01c01ff4e885b218e","unresolved":false,"context_lines":[{"line_number":139,"context_line":"            if \u0027image\u0027 in self.show_image(id):"},{"line_number":140,"context_line":"                if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":141,"context_line":"                    return True"},{"line_number":142,"context_line":"            else:"},{"line_number":143,"context_line":"                self.show_image(id)"},{"line_number":144,"context_line":"        except lib_exc.NotFound:"},{"line_number":145,"context_line":"            return True"},{"line_number":146,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa0719c6_261f7460","line":143,"range":{"start_line":142,"start_character":12,"end_line":143,"end_character":35},"updated":"2016-03-22 22:32:02.000000000","message":"No need for this else... if id does not exists and it is non admin, previous self.show_image will throw the error","commit_id":"d79b95bf42e048d1d223af3bef763c2ec55324a5"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"9aa0ab3d54495ed950b0bcde518599eae5dbc20b","unresolved":false,"context_lines":[{"line_number":133,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":134,"context_line":"        # When user has admin role a 200 will return even for deleted images"},{"line_number":135,"context_line":"        # which will lead to a return value of False. In this case"},{"line_number":136,"context_line":"        # the status of the image needs to be verified by checking whether"},{"line_number":137,"context_line":"        # it is ACTIVE or DELETED."},{"line_number":138,"context_line":"        try:"},{"line_number":139,"context_line":"            if \u0027image\u0027 in self.show_image(id):"},{"line_number":140,"context_line":"                if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa0719c6_ec009368","line":137,"range":{"start_line":136,"start_character":67,"end_line":137,"end_character":25},"updated":"2016-03-23 15:56:19.000000000","message":"it doesn\u0027t necessarily needs to check for the Active state, all it needs is to check for anything different than Deleted.","commit_id":"2d3e32c46190d76b5818252cddfcfba98811b9ed"},{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"88b099ad692b9e07fc0d13b645bc2a99d77600fa","unresolved":false,"context_lines":[{"line_number":133,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":134,"context_line":"        # When user has admin role a 200 will return even for deleted images"},{"line_number":135,"context_line":"        # which will lead to a return value of False. In this case"},{"line_number":136,"context_line":"        # the status of the image needs to be verified by checking whether"},{"line_number":137,"context_line":"        # it is ACTIVE or DELETED."},{"line_number":138,"context_line":"        try:"},{"line_number":139,"context_line":"            if \u0027image\u0027 in self.show_image(id):"},{"line_number":140,"context_line":"                if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa0719c6_436b3af1","line":137,"range":{"start_line":136,"start_character":67,"end_line":137,"end_character":25},"in_reply_to":"fa0719c6_ec009368","updated":"2016-03-23 20:45:04.000000000","message":"I think it\u0027s better to use doc string here.","commit_id":"2d3e32c46190d76b5818252cddfcfba98811b9ed"},{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"88b099ad692b9e07fc0d13b645bc2a99d77600fa","unresolved":false,"context_lines":[{"line_number":140,"context_line":"                if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":141,"context_line":"                    return True"},{"line_number":142,"context_line":"            else:"},{"line_number":143,"context_line":"                self.show_image(id)"},{"line_number":144,"context_line":"        except lib_exc.NotFound:"},{"line_number":145,"context_line":"            return True"},{"line_number":146,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa0719c6_ae15dde5","line":143,"updated":"2016-03-23 20:45:04.000000000","message":"Why we need to call show_image() twice? \u0027image\u0027 in self.show_image(id) is a subset of self.show_image(id).","commit_id":"2d3e32c46190d76b5818252cddfcfba98811b9ed"},{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"f2d690184866b32105e08475e0f1bf69e420561e","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":134,"context_line":"        # Added status check for user with admin role"},{"line_number":135,"context_line":"        try:"},{"line_number":136,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":137,"context_line":"                return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a1b2928_846deb06","line":134,"updated":"2016-04-11 15:54:09.000000000","message":"Doesn\u0027t this implicitly add a requirement that any test uses this function now also requires admin privileges? Many tests don\u0027t require admin users, so this may have an unexpected impact","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":97,"name":"Daryl Walleck","email":"daryl.walleck@rackspace.com","username":"dwalleck"},"change_message_id":"9b61da654cd6dadad06e65c6c7f95f59d5570b7d","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":134,"context_line":"        # Added status check for user with admin role"},{"line_number":135,"context_line":"        try:"},{"line_number":136,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":137,"context_line":"                return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a1b2928_b7965b92","line":134,"in_reply_to":"7a1b2928_846deb06","updated":"2016-04-11 15:56:26.000000000","message":"Sorry, misread this change. Carry on :)","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"12f0e73c4c72e4492110b39286c4d9d6e2922bd2","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":134,"context_line":"        # Added status check for user with admin role"},{"line_number":135,"context_line":"        try:"},{"line_number":136,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":137,"context_line":"                return True"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a122d0e_f631c70c","line":134,"updated":"2016-04-18 15:39:42.000000000","message":"I think this comment should be before \u0027if\u0027 and it should have a few words about admin role and image status like you describe that in the commit message.","commit_id":"95dcef21f730922f5bbf0bc34b1d84e2aa844209"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"73423b710f32b4027da0f91741ab79f2d05b106a","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":134,"context_line":"        # Added status check for user with admin role"},{"line_number":135,"context_line":"        try:"},{"line_number":136,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":137,"context_line":"                return True"}],"source_content_type":"text/x-python","patch_set":7,"id":"1a122d0e_9bf3549e","line":134,"in_reply_to":"1a122d0e_f631c70c","updated":"2016-04-18 18:06:25.000000000","message":"I agree. This is not trivial, and a couple of words like \"trying to get info on a deleted image doesn\"t raise a 404 when performed with the \u0027admin\u0027 role.\"","commit_id":"95dcef21f730922f5bbf0bc34b1d84e2aa844209"}],"tempest/services/image/v1/json/images_client.py":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"1e1335ab66859073f0fd4c9e045cf633c6161277","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":218,"context_line":"                return True"},{"line_number":219,"context_line":"        except lib_exc.NotFound:"},{"line_number":220,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_1b9323a5","line":217,"range":{"start_line":217,"start_character":15,"end_line":217,"end_character":34},"updated":"2016-04-11 22:11:34.000000000","message":"Is \"get_image_meta\" method retrieving the same as \"show_image\"?","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":19587,"name":"Joshua White","email":"joshua.l.white@intel.com","username":"jwhite"},"change_message_id":"27538486ad3186affa071b0c4e2427e9dd3fd3c5","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":218,"context_line":"                return True"},{"line_number":219,"context_line":"        except lib_exc.NotFound:"},{"line_number":220,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_5ec1092e","line":217,"range":{"start_line":217,"start_character":15,"end_line":217,"end_character":34},"in_reply_to":"5a18252c_1b9323a5","updated":"2016-04-11 23:11:43.000000000","message":"Nice catch Luz.  I verified, and it is returning the same info. Actually, using show_image here returns 404 with admin.  All and all it is the same. They are both sending a request for the same url.  Only get_image_meta uses HEAD (and grabs header info, meta data) and show_image uses GET and grabs the whole message.","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"c15cd300fe4c4050bde0dd1d4f372c49921b97c3","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":218,"context_line":"                return True"},{"line_number":219,"context_line":"        except lib_exc.NotFound:"},{"line_number":220,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_faaf06e1","line":217,"range":{"start_line":217,"start_character":15,"end_line":217,"end_character":34},"in_reply_to":"5a18252c_1e5aec83","updated":"2016-04-12 23:26:09.000000000","message":"In admin role, Glance v1 api show _image() returns 404 according to http://logs.openstack.org/39/304339/4/check/gate-tempest-dsvm-full/4230478/logs/testr_results.html.gz","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"bb4808f4c55c938931c023bd3654314dc84a58df","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":218,"context_line":"                return True"},{"line_number":219,"context_line":"        except lib_exc.NotFound:"},{"line_number":220,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_1e5aec83","line":217,"range":{"start_line":217,"start_character":15,"end_line":217,"end_character":34},"in_reply_to":"5a18252c_5ec1092e","updated":"2016-04-12 04:53:33.000000000","message":"Yea but HEAD is preferred here(as it is fast)as we just want to check status not all other info.\n\nShow_image from glance v1 API will always return 403/404 if not active so if check is not valid if you use show_image.\n\nSo get_image_meta is fine to use here.\n\nif self.get_image_meta(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":19587,"name":"Joshua White","email":"joshua.l.white@intel.com","username":"jwhite"},"change_message_id":"6d0108412806053d4343e7fa9dfff7e6df940915","unresolved":false,"context_lines":[{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        # Added status check for user with admin role"},{"line_number":216,"context_line":"        try:"},{"line_number":217,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":218,"context_line":"                return True"},{"line_number":219,"context_line":"        except lib_exc.NotFound:"},{"line_number":220,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_94490bba","line":217,"range":{"start_line":217,"start_character":15,"end_line":217,"end_character":34},"in_reply_to":"5a18252c_faaf06e1","updated":"2016-04-14 21:02:56.000000000","message":"Thanks everyone. Yes that is true show_image(id) will return 404.  So Ghanshyam solution will work.  I will go and make the change.","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"7ff2a6f93c3d522a06e534d525d84a5b76c3323d","unresolved":false,"context_lines":[{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":215,"context_line":"        try:"},{"line_number":216,"context_line":"            self.get_image_meta(id)[\u0027status\u0027] \u003d \u0027deleted\u0027"},{"line_number":217,"context_line":"        except lib_exc.NotFound:"},{"line_number":218,"context_line":"            return True"},{"line_number":219,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a18252c_a7adee9c","line":216,"range":{"start_line":216,"start_character":12,"end_line":216,"end_character":57},"updated":"2016-04-15 01:55:36.000000000","message":"if self.get_image_meta(id)[\u0027status\u0027] \u003d\u003d \u0027deleted\u0027:\n  return True","commit_id":"6f88a3f7134ecf5afaa1ce06249cee6009620bba"}],"tempest/services/image/v2/json/images_client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"bb4808f4c55c938931c023bd3654314dc84a58df","unresolved":false,"context_lines":[{"line_number":107,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":108,"context_line":"        # Added status check for user with admin role"},{"line_number":109,"context_line":"        try:"},{"line_number":110,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":111,"context_line":"                return True"},{"line_number":112,"context_line":"        except lib_exc.NotFound:"},{"line_number":113,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_0140cd55","line":110,"range":{"start_line":110,"start_character":12,"end_line":110,"end_character":67},"updated":"2016-04-12 04:53:33.000000000","message":"Did you get error in v2 API case also?\n\nThere is no get image metadata in v2 API and show_image always return 404 either admin o non-admin. \nI do not think this issue valid for glance v2 API ?","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"},{"author":{"_account_id":19587,"name":"Joshua White","email":"joshua.l.white@intel.com","username":"jwhite"},"change_message_id":"6d0108412806053d4343e7fa9dfff7e6df940915","unresolved":false,"context_lines":[{"line_number":107,"context_line":"    def is_resource_deleted(self, id):"},{"line_number":108,"context_line":"        # Added status check for user with admin role"},{"line_number":109,"context_line":"        try:"},{"line_number":110,"context_line":"            if self.show_image(id)[\u0027image\u0027][\u0027status\u0027] \u003d\u003d \u0027DELETED\u0027:"},{"line_number":111,"context_line":"                return True"},{"line_number":112,"context_line":"        except lib_exc.NotFound:"},{"line_number":113,"context_line":"            return True"}],"source_content_type":"text/x-python","patch_set":5,"id":"5a18252c_d4c263e8","line":110,"range":{"start_line":110,"start_character":12,"end_line":110,"end_character":67},"in_reply_to":"5a18252c_0140cd55","updated":"2016-04-14 21:02:56.000000000","message":"thanks Ghanshyam, I verified that this issue is not affecting v2.  Tried to be proactive thinking the change was needed :/.  I will change it back.  Actually the v2 test I ran never call this function.  So it wouldn\u0027t have been an issue, anyway.","commit_id":"6c51b3bcf12927111c7e99f743ea750906c84a01"}]}