)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"37284f87f6d985faf41624265b571cffc5692cc5","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Current template ceilometer.conf.erb is using auth_uri for os_auth_url"},{"line_number":10,"context_line":"which is not a correct approach. This auth_uri is dedicated to be used"},{"line_number":11,"context_line":"by auth_uri under section [keystone_authtoken]. On the other hand, the"},{"line_number":12,"context_line":"passed in identity_endpoint param is not used. Actually, by conversion"},{"line_number":13,"context_line":"with other cookbooks, identity_endpoint should be used for keystone"},{"line_number":14,"context_line":"identity service connection."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_0860e8a3","line":12,"updated":"2015-07-29 15:50:36.000000000","message":"by comparison?","commit_id":"bd2264ed2565587c4859bbe624b946f61d215f3d"}],"spec/common_spec.rb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"6c29afeba2b6d5c0a083f8bc373b3e1904c65ba9","unresolved":false,"context_lines":[{"line_number":352,"context_line":"        end"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"        it \u0027use default value for os_auth_url with api auth version to v3\u0027 do"},{"line_number":355,"context_line":"          node.set[\u0027openstack\u0027][\u0027telemetry\u0027][\u0027api\u0027][\u0027auth\u0027][\u0027version\u0027] \u003d \u0027v3.0\u0027"},{"line_number":356,"context_line":"          expect(chef_run).to render_config_file(file.name)\\"},{"line_number":357,"context_line":"            .with_section_content(\u0027DEFAULT\u0027, %r{^os_auth_url \u003d http://127.0.0.1:5000/v2.0$})"},{"line_number":358,"context_line":"        end"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"3a50d1a3_3ae5e336","line":355,"updated":"2015-07-29 17:08:46.000000000","message":"Humm, I don\u0027t understand this test, the expect is same as above.","commit_id":"c4b64a19b3fe6b1d7c9324b99e1e7dba3f769ddd"},{"author":{"_account_id":9492,"name":"Yi Ming Yin","email":"yinym@cn.ibm.com","username":"yym"},"change_message_id":"c09b4e5b2747cd1cc9703fda2a30279d6af47f8d","unresolved":false,"context_lines":[{"line_number":352,"context_line":"        end"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"        it \u0027use default value for os_auth_url with api auth version to v3\u0027 do"},{"line_number":355,"context_line":"          node.set[\u0027openstack\u0027][\u0027telemetry\u0027][\u0027api\u0027][\u0027auth\u0027][\u0027version\u0027] \u003d \u0027v3.0\u0027"},{"line_number":356,"context_line":"          expect(chef_run).to render_config_file(file.name)\\"},{"line_number":357,"context_line":"            .with_section_content(\u0027DEFAULT\u0027, %r{^os_auth_url \u003d http://127.0.0.1:5000/v2.0$})"},{"line_number":358,"context_line":"        end"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"3a50d1a3_11079682","line":355,"in_reply_to":"3a50d1a3_3ae5e336","updated":"2015-07-30 02:36:45.000000000","message":"This test is for regression purpose. This is because before this patch, the os_auth_url was set to the same as auth_uri. This test is to make sure that os_auth_url does not get impacted by api auth version.","commit_id":"c4b64a19b3fe6b1d7c9324b99e1e7dba3f769ddd"}],"templates/default/ceilometer.conf.erb":[{"author":{"_account_id":7128,"name":"Mark Vanderwiel","email":"vanderwl@us.ibm.com","username":"kramvan"},"change_message_id":"37284f87f6d985faf41624265b571cffc5692cc5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"os_auth_url \u003d \u003c%\u003d @identity_endpoint %\u003e"},{"line_number":3,"context_line":"os_tenant_name \u003d \u003c%\u003d @service_tenant_name %\u003e"},{"line_number":4,"context_line":"os_password \u003d \u003c%\u003d @service_pass %\u003e"},{"line_number":5,"context_line":"os_username \u003d \u003c%\u003d @service_user %\u003e"}],"source_content_type":"application/x-erb","patch_set":1,"id":"3a50d1a3_88e958e8","line":2,"updated":"2015-07-29 15:50:36.000000000","message":"Need a spec test for this","commit_id":"bd2264ed2565587c4859bbe624b946f61d215f3d"}]}
