)]}'
{"manifests/agent/auth.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":18,"context_line":"#   Defaults to \u0027ceilometer\u0027."},{"line_number":19,"context_line":"#"},{"line_number":20,"context_line":"# [*auth_password*]"},{"line_number":21,"context_line":"#   (Mandatory) the keystone password for ceilometer services"},{"line_number":22,"context_line":"#"},{"line_number":23,"context_line":"# [*auth_tenant_name*]"},{"line_number":24,"context_line":"#   (Optional) the keystone tenant name for ceilometer services"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_365fac10","line":21,"range":{"start_line":21,"start_character":5,"end_line":21,"end_character":14},"updated":"2016-01-11 10:44:45.000000000","message":"Actually, we usually use \u0027Required\u0027, not a \u0027Mandatory\u0027.","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"}],"manifests/api.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"#   Defaults to \u0027services\u0027."},{"line_number":26,"context_line":"#"},{"line_number":27,"context_line":"# [*keystone_password*]"},{"line_number":28,"context_line":"#   (Mandatory) Password to authenticate with."},{"line_number":29,"context_line":"#"},{"line_number":30,"context_line":"# [*keystone_auth_uri*]"},{"line_number":31,"context_line":"#   (Optional) Public Identity API endpoint."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_d67df06d","line":28,"range":{"start_line":28,"start_character":5,"end_line":28,"end_character":14},"updated":"2016-01-11 10:44:45.000000000","message":"same here","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"}],"manifests/db/mysql.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":6,"context_line":"# \u003d\u003d\u003d Parameters:"},{"line_number":7,"context_line":"#"},{"line_number":8,"context_line":"# [*password*]"},{"line_number":9,"context_line":"#   (Mandatory) password to connect to the database."},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*dbname*]"},{"line_number":12,"context_line":"#   (Optional) name of the database."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_1664e8b6","line":9,"range":{"start_line":9,"start_character":5,"end_line":9,"end_character":14},"updated":"2016-01-11 10:44:45.000000000","message":"and here","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"},{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"b1dd0ab0c4d656638dd96601c346ed85dd59e0b3","unresolved":false,"context_lines":[{"line_number":18,"context_line":"#"},{"line_number":19,"context_line":"# [*host*]"},{"line_number":20,"context_line":"#   (Optional) the default source host user is allowed to connect from."},{"line_number":21,"context_line":"#   Defaults to \u0027localhost\u0027."},{"line_number":22,"context_line":"#"},{"line_number":23,"context_line":"# [*allowed_hosts*]"},{"line_number":24,"context_line":"#   (Optional) other hosts the user is allowd to connect from."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_d1cbafa2","line":21,"range":{"start_line":21,"start_character":17,"end_line":21,"end_character":26},"updated":"2016-01-11 14:19:54.000000000","message":"As I can see, default is \u0027127.0.0.1\u0027","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":1607,"name":"Yu xingchao","email":"yuxcer@gmail.com","username":"yuxcer"},"change_message_id":"cebefe3ef5e640d618f62de2f8bc1c2baba46001","unresolved":false,"context_lines":[{"line_number":18,"context_line":"#"},{"line_number":19,"context_line":"# [*host*]"},{"line_number":20,"context_line":"#   (Optional) the default source host user is allowed to connect from."},{"line_number":21,"context_line":"#   Defaults to \u0027localhost\u0027."},{"line_number":22,"context_line":"#"},{"line_number":23,"context_line":"# [*allowed_hosts*]"},{"line_number":24,"context_line":"#   (Optional) other hosts the user is allowd to connect from."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_6c69c8bd","line":21,"range":{"start_line":21,"start_character":17,"end_line":21,"end_character":26},"in_reply_to":"fa69d971_d1cbafa2","updated":"2016-01-11 14:34:07.000000000","message":"good catch, this inconsistency exist other openstack modules.","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"}],"manifests/init.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"#    Defaults to -1."},{"line_number":26,"context_line":"#"},{"line_number":27,"context_line":"#  [*metering_secret*]"},{"line_number":28,"context_line":"#   (Mandatory)  Secret key for signing messages."},{"line_number":29,"context_line":"#"},{"line_number":30,"context_line":"#  [*notification_topics*]"},{"line_number":31,"context_line":"#    (Optional) AMQP topic used for OpenStack notifications (list value)"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_9670d873","line":28,"range":{"start_line":28,"start_character":5,"end_line":28,"end_character":14},"updated":"2016-01-11 10:44:45.000000000","message":"same here","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"}],"manifests/keystone/auth.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# \u003d\u003d\u003d Parameters:"},{"line_number":6,"context_line":"#"},{"line_number":7,"context_line":"# [*password*]"},{"line_number":8,"context_line":"#   (Mandatory) Password for Ceilometer user."},{"line_number":9,"context_line":"#"},{"line_number":10,"context_line":"# [*email*]"},{"line_number":11,"context_line":"#   (Optional) Email for Ceilometer user."}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_d6273061","line":8,"range":{"start_line":8,"start_character":5,"end_line":8,"end_character":14},"updated":"2016-01-11 10:44:45.000000000","message":"Required","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":53,"context_line":"#   Defaults to \u0027http://127.0.0.1:8777\u0027."},{"line_number":54,"context_line":"#"},{"line_number":55,"context_line":"# [*admin_url*]"},{"line_number":56,"context_line":"#   (Optional) The endpoint\u0027s admin url. "},{"line_number":57,"context_line":"#   This url should *not* contain any trailing \u0027/\u0027."},{"line_number":58,"context_line":"#   Defaults to \u0027http://127.0.0.1:8777\u0027."},{"line_number":59,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_b60b9cd4","line":56,"range":{"start_line":56,"start_character":40,"end_line":56,"end_character":41},"updated":"2016-01-11 10:44:45.000000000","message":"extra space","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":58,"context_line":"#   Defaults to \u0027http://127.0.0.1:8777\u0027."},{"line_number":59,"context_line":"#"},{"line_number":60,"context_line":"# [*internal_url*]"},{"line_number":61,"context_line":"#   (Optional) The endpoint\u0027s internal url. "},{"line_number":62,"context_line":"#   This url should *not* contain any trailing \u0027/\u0027."},{"line_number":63,"context_line":"#   Defaults to \u0027http://127.0.0.1:8777\u0027"},{"line_number":64,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_76fa04bf","line":61,"range":{"start_line":61,"start_character":43,"end_line":61,"end_character":44},"updated":"2016-01-11 10:44:45.000000000","message":"extra space","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"bc90c8574da2f64542c1dd6af4a330d70283fe4b","unresolved":false,"context_lines":[{"line_number":76,"context_line":"#"},{"line_number":77,"context_line":"# [*public_address*]"},{"line_number":78,"context_line":"#   (Optional) DEPRECATED: Use public_url instead."},{"line_number":79,"context_line":"#   Public address for endpoint. "},{"line_number":80,"context_line":"#   Setting this parameter overrides public_url parameter."},{"line_number":81,"context_line":"#   Defaults to \u0027127.0.0.1\u0027."},{"line_number":82,"context_line":"#"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"fa69d971_36000cab","line":79,"updated":"2016-01-11 10:44:45.000000000","message":"extra space","commit_id":"522636e120082ba5f05fdb33293f04e3ae41d613"},{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"b1dd0ab0c4d656638dd96601c346ed85dd59e0b3","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# [*configure_user*]"},{"line_number":23,"context_line":"#   (Optional) Should Ceilometer service user be configured?"},{"line_number":24,"context_line":"#   Defaults to \u0027true\u0027."},{"line_number":25,"context_line":"#"},{"line_number":26,"context_line":"# [*configure_user_role*]"},{"line_number":27,"context_line":"#   (Optional) Should roles be configured on Ceilometer service user?"},{"line_number":28,"context_line":"#   Defaults to \u0027true\u0027."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_31c1eb2f","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":1},"updated":"2016-01-11 14:19:54.000000000","message":"I didn\u0027t see single quotes near true in all another classes. Does it make sense to remove them here?","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":1607,"name":"Yu xingchao","email":"yuxcer@gmail.com","username":"yuxcer"},"change_message_id":"cebefe3ef5e640d618f62de2f8bc1c2baba46001","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# [*configure_user*]"},{"line_number":23,"context_line":"#   (Optional) Should Ceilometer service user be configured?"},{"line_number":24,"context_line":"#   Defaults to \u0027true\u0027."},{"line_number":25,"context_line":"#"},{"line_number":26,"context_line":"# [*configure_user_role*]"},{"line_number":27,"context_line":"#   (Optional) Should roles be configured on Ceilometer service user?"},{"line_number":28,"context_line":"#   Defaults to \u0027true\u0027."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_8cdbdc7b","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":1},"in_reply_to":"fa69d971_31c1eb2f","updated":"2016-01-11 14:34:07.000000000","message":"Done","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"b1dd0ab0c4d656638dd96601c346ed85dd59e0b3","unresolved":false,"context_lines":[{"line_number":60,"context_line":"# [*internal_url*]"},{"line_number":61,"context_line":"#   (Optional) The endpoint\u0027s internal url."},{"line_number":62,"context_line":"#   This url should *not* contain any trailing \u0027/\u0027."},{"line_number":63,"context_line":"#   Defaults to \u0027http://127.0.0.1:8777\u0027"},{"line_number":64,"context_line":"#"},{"line_number":65,"context_line":"# [*port*]"},{"line_number":66,"context_line":"#   (Optional) DEPRECATED: Use public_url, internal_url and admin_url instead."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_916bf71d","line":63,"range":{"start_line":63,"start_character":38,"end_line":63,"end_character":39},"updated":"2016-01-11 14:19:54.000000000","message":"dot is missed","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":1607,"name":"Yu xingchao","email":"yuxcer@gmail.com","username":"yuxcer"},"change_message_id":"cebefe3ef5e640d618f62de2f8bc1c2baba46001","unresolved":false,"context_lines":[{"line_number":60,"context_line":"# [*internal_url*]"},{"line_number":61,"context_line":"#   (Optional) The endpoint\u0027s internal url."},{"line_number":62,"context_line":"#   This url should *not* contain any trailing \u0027/\u0027."},{"line_number":63,"context_line":"#   Defaults to \u0027http://127.0.0.1:8777\u0027"},{"line_number":64,"context_line":"#"},{"line_number":65,"context_line":"# [*port*]"},{"line_number":66,"context_line":"#   (Optional) DEPRECATED: Use public_url, internal_url and admin_url instead."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_2c69908c","line":63,"range":{"start_line":63,"start_character":38,"end_line":63,"end_character":39},"in_reply_to":"fa69d971_916bf71d","updated":"2016-01-11 14:34:07.000000000","message":"Done","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"b1dd0ab0c4d656638dd96601c346ed85dd59e0b3","unresolved":false,"context_lines":[{"line_number":104,"context_line":"#   Setting this parameter overrides admin_url parameter."},{"line_number":105,"context_line":"#   Defaults to \u0027127.0.0.1\u0027."},{"line_number":106,"context_line":"#"},{"line_number":107,"context_line":"# \u003d\u003d\u003d Deprecation notes"},{"line_number":108,"context_line":"#"},{"line_number":109,"context_line":"# If any value is provided for public_protocol, public_address or port parameters,"},{"line_number":110,"context_line":"# public_url will be completely ignored. The same applies for internal and admin parameters."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_51eb3fdc","line":107,"range":{"start_line":107,"start_character":18,"end_line":107,"end_character":23},"updated":"2016-01-11 14:19:54.000000000","message":"do we need to set \u0027:\u0027 here as well?","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":1607,"name":"Yu xingchao","email":"yuxcer@gmail.com","username":"yuxcer"},"change_message_id":"cebefe3ef5e640d618f62de2f8bc1c2baba46001","unresolved":false,"context_lines":[{"line_number":104,"context_line":"#   Setting this parameter overrides admin_url parameter."},{"line_number":105,"context_line":"#   Defaults to \u0027127.0.0.1\u0027."},{"line_number":106,"context_line":"#"},{"line_number":107,"context_line":"# \u003d\u003d\u003d Deprecation notes"},{"line_number":108,"context_line":"#"},{"line_number":109,"context_line":"# If any value is provided for public_protocol, public_address or port parameters,"},{"line_number":110,"context_line":"# public_url will be completely ignored. The same applies for internal and admin parameters."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_0cefec1e","line":107,"range":{"start_line":107,"start_character":18,"end_line":107,"end_character":23},"in_reply_to":"fa69d971_51eb3fdc","updated":"2016-01-11 14:34:07.000000000","message":"Done","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"b1dd0ab0c4d656638dd96601c346ed85dd59e0b3","unresolved":false,"context_lines":[{"line_number":109,"context_line":"# If any value is provided for public_protocol, public_address or port parameters,"},{"line_number":110,"context_line":"# public_url will be completely ignored. The same applies for internal and admin parameters."},{"line_number":111,"context_line":"#"},{"line_number":112,"context_line":"# \u003d\u003d\u003d Examples"},{"line_number":113,"context_line":"#"},{"line_number":114,"context_line":"#  class { \u0027ceilometer::keystone::auth\u0027:"},{"line_number":115,"context_line":"#    public_url   \u003d\u003e \u0027https://10.0.0.10:8777\u0027,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_11c0674f","line":112,"range":{"start_line":112,"start_character":6,"end_line":112,"end_character":14},"updated":"2016-01-11 14:19:54.000000000","message":"same question","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"}],"manifests/logging.pp":[{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"b1dd0ab0c4d656638dd96601c346ed85dd59e0b3","unresolved":false,"context_lines":[{"line_number":6,"context_line":"#"},{"line_number":7,"context_line":"# [*verbose*]"},{"line_number":8,"context_line":"#   (Optional) Should the daemons log verbose messages"},{"line_number":9,"context_line":"#   Defaults to $::os_service_default"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*debug*]"},{"line_number":12,"context_line":"#   (Optional) Should the daemons log debug messages"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_f11273b3","line":9,"range":{"start_line":9,"start_character":19,"end_line":9,"end_character":37},"updated":"2016-01-11 14:19:54.000000000","message":"what is about the dots at the end of the sentences?","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"},{"author":{"_account_id":1607,"name":"Yu xingchao","email":"yuxcer@gmail.com","username":"yuxcer"},"change_message_id":"cebefe3ef5e640d618f62de2f8bc1c2baba46001","unresolved":false,"context_lines":[{"line_number":6,"context_line":"#"},{"line_number":7,"context_line":"# [*verbose*]"},{"line_number":8,"context_line":"#   (Optional) Should the daemons log verbose messages"},{"line_number":9,"context_line":"#   Defaults to $::os_service_default"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*debug*]"},{"line_number":12,"context_line":"#   (Optional) Should the daemons log debug messages"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_8ccc3c8a","line":9,"range":{"start_line":9,"start_character":19,"end_line":9,"end_character":37},"in_reply_to":"fa69d971_f11273b3","updated":"2016-01-11 14:34:07.000000000","message":"Done","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"}],"manifests/wsgi/apache.pp":[{"author":{"_account_id":7732,"name":"IvanBerezovskiy","email":"ivan.berezovskii@gmail.com","username":"iberezovskiy"},"change_message_id":"b1dd0ab0c4d656638dd96601c346ed85dd59e0b3","unresolved":false,"context_lines":[{"line_number":28,"context_line":"#"},{"line_number":29,"context_line":"# [*servername*]"},{"line_number":30,"context_line":"#   (Optional) The servername for the virtualhost."},{"line_number":31,"context_line":"#   Defaults to $::fqdn"},{"line_number":32,"context_line":"#"},{"line_number":33,"context_line":"# [*port*]"},{"line_number":34,"context_line":"#   (Optional) The port."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fa69d971_d15baf84","line":31,"range":{"start_line":31,"start_character":19,"end_line":31,"end_character":23},"updated":"2016-01-11 14:19:54.000000000","message":"dot is missed","commit_id":"f7a61081503917c2ca5fda0e4551ec74f30156a9"}]}