)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2a5cbbb6b36e5b50b3703b0dd7e07e940ced0647","unresolved":false,"context_lines":[{"line_number":10,"context_line":"So this patch removes the copy\u0026paste images_client and make the test"},{"line_number":11,"context_line":"use tempest.lib\u0027s one."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"NOTE: The v1 images_client is still under the development."},{"line_number":14,"context_line":"      So we need more time to switch using tempest.lib\u0027s one."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Depends-On: I3af7b29607b3ccd917fae49ba4d01397dfca7dc1"},{"line_number":17,"context_line":"Change-Id: I8683eea8fe39d3aedde2766d4381ed6e1bff8e17"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3aaa91ec_426b1ff1","line":14,"range":{"start_line":13,"start_character":7,"end_line":14,"end_character":61},"updated":"2016-07-01 06:06:57.000000000","message":"I think its ready now?","commit_id":"1bf1034504f3bdc92ae72adb53753147e8e1a77e"}],"ceilometer/tests/tempest/service/client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2a5cbbb6b36e5b50b3703b0dd7e07e940ced0647","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    compute_params.update(default_params)"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    image_params \u003d {"},{"line_number":131,"context_line":"        \u0027service\u0027: CONF.image.catalog_type,"},{"line_number":132,"context_line":"        \u0027region\u0027: CONF.image.region or CONF.identity.region,"},{"line_number":133,"context_line":"        \u0027endpoint_type\u0027: CONF.image.endpoint_type,"},{"line_number":134,"context_line":"        \u0027build_interval\u0027: CONF.image.build_interval,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3aaa91ec_a2a3e341","line":131,"range":{"start_line":131,"start_character":9,"end_line":131,"end_character":16},"updated":"2016-07-01 06:06:57.000000000","message":"this should be catalog_type as in-\n\nhttps://github.com/openstack/ceilometer/blob/master/ceilometer/tests/tempest/service/images/v1/images_client.py#L34\n\neven very odd :(","commit_id":"1bf1034504f3bdc92ae72adb53753147e8e1a77e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1e901267b6ae92308502c9035f658d5d6056a82c","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    compute_params.update(default_params)"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    image_params \u003d {"},{"line_number":131,"context_line":"        \u0027service\u0027: CONF.image.catalog_type,"},{"line_number":132,"context_line":"        \u0027region\u0027: CONF.image.region or CONF.identity.region,"},{"line_number":133,"context_line":"        \u0027endpoint_type\u0027: CONF.image.endpoint_type,"},{"line_number":134,"context_line":"        \u0027build_interval\u0027: CONF.image.build_interval,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3aaa91ec_e77159ad","line":131,"range":{"start_line":131,"start_character":9,"end_line":131,"end_character":16},"in_reply_to":"3aaa91ec_a2a3e341","updated":"2016-07-01 16:43:54.000000000","message":"No, this is not odd because of I45b7d176defd2de4bbf5bc503533162c16fe0a68","commit_id":"1bf1034504f3bdc92ae72adb53753147e8e1a77e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"52a6ef4cbbed188c5226b1ec275c96ff7e95c9d2","unresolved":false,"context_lines":[{"line_number":128,"context_line":"    compute_params.update(default_params)"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    image_params \u003d {"},{"line_number":131,"context_line":"        \u0027service\u0027: CONF.image.catalog_type,"},{"line_number":132,"context_line":"        \u0027region\u0027: CONF.image.region or CONF.identity.region,"},{"line_number":133,"context_line":"        \u0027endpoint_type\u0027: CONF.image.endpoint_type,"},{"line_number":134,"context_line":"        \u0027build_interval\u0027: CONF.image.build_interval,"}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_b503e90e","line":131,"range":{"start_line":131,"start_character":9,"end_line":131,"end_character":16},"in_reply_to":"3aaa91ec_e77159ad","updated":"2016-07-04 05:42:00.000000000","message":"Yea but here we use ceilometer copy of image client which is expecting \u0027catalog_type\u0027 as 2nd arg not \u0027service\u0027.\n\nhttps://github.com/openstack/ceilometer/blob/master/ceilometer/tests/tempest/service/images/v1/images_client.py#L34\n\nThat\u0027s why it fails - http://logs.openstack.org/59/329259/5/check/gate-ceilometer-dsvm-tempest-plugin-mongodb/4b53aa1/testr_results.html.gz","commit_id":"1bf1034504f3bdc92ae72adb53753147e8e1a77e"}]}