)]}'
{"manifests/db/sync.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"4a4e5ac5c8e2dbe011c0f7e6bba7961d67aa5894","unresolved":false,"context_lines":[{"line_number":23,"context_line":"  Glance_cache_config\u003c||\u003e ~\u003e Exec[\u0027glance-manage db_sync\u0027]"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  exec { \u0027glance-manage db_sync\u0027:"},{"line_number":26,"context_line":"    command     \u003d\u003e \"glance-manage --config-file\u003d/etc/glance/glance-registry.conf ${extra_params} db_sync\","},{"line_number":27,"context_line":"    path        \u003d\u003e \u0027/usr/bin\u0027,"},{"line_number":28,"context_line":"    user        \u003d\u003e \u0027glance\u0027,"},{"line_number":29,"context_line":"    refreshonly \u003d\u003e true,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"fa80f949_684b9cb1","line":26,"updated":"2015-11-03 15:05:43.000000000","message":"i\u0027m not sure about hardcoding config path here. But it seems, that we don\u0027t have another way","commit_id":"b56c3cb8ecf4115508d6bfab6726d6468e648e27"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"0758ed04293067f917071682b2b867461fe72ed0","unresolved":false,"context_lines":[{"line_number":23,"context_line":"  Glance_cache_config\u003c||\u003e ~\u003e Exec[\u0027glance-manage db_sync\u0027]"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  exec { \u0027glance-manage db_sync\u0027:"},{"line_number":26,"context_line":"    command     \u003d\u003e \"glance-manage --config-file\u003d/etc/glance/glance-registry.conf ${extra_params} db_sync\","},{"line_number":27,"context_line":"    path        \u003d\u003e \u0027/usr/bin\u0027,"},{"line_number":28,"context_line":"    user        \u003d\u003e \u0027glance\u0027,"},{"line_number":29,"context_line":"    refreshonly \u003d\u003e true,"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"fa80f949_c36dc9cd","line":26,"in_reply_to":"fa80f949_684b9cb1","updated":"2015-11-03 15:12:47.000000000","message":"I could just take that out entirely and leave it to the user to include it in extra params, I just left it in there because it was already included in $::glance::params::db_sync_command and I didn\u0027t want to change that. I suppose that it might make more sense to leave it to extra_params now that it exists though.","commit_id":"b56c3cb8ecf4115508d6bfab6726d6468e648e27"},{"author":{"_account_id":10540,"name":"Clayton O\u0027Neill","email":"clayton.oneill@charter.com","username":"coneill"},"change_message_id":"47f4c9f9ab9f5b427eadd43749397f1a535a65ee","unresolved":false,"context_lines":[{"line_number":10,"context_line":"#   Defaults to \u0027--config-file /etc/glance/glance-registry.conf\u0027"},{"line_number":11,"context_line":"#"},{"line_number":12,"context_line":"class glance::db::sync("},{"line_number":13,"context_line":"  $extra_params \u003d \u0027--config-file /etc/glance/glance-registry.conf\u0027,"},{"line_number":14,"context_line":") {"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"  include ::glance::params"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"da85f559_1abdd8b6","line":13,"updated":"2015-11-11 14:55:25.000000000","message":"I think this should default to an empty string.  Reading from the \"correct\" config file should be the default and this will just break distributions or deployers that choose to put the config file in a non-default location.","commit_id":"bd25fa8de83d51073ec8921fa20fe94314a45bac"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"b86cdfccd227f40772c14d23e50b14ea6aa1da5e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"#   Defaults to \u0027--config-file /etc/glance/glance-registry.conf\u0027"},{"line_number":11,"context_line":"#"},{"line_number":12,"context_line":"class glance::db::sync("},{"line_number":13,"context_line":"  $extra_params \u003d \u0027--config-file /etc/glance/glance-registry.conf\u0027,"},{"line_number":14,"context_line":") {"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"  include ::glance::params"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"da85f559_d56f23fa","line":13,"in_reply_to":"da85f559_1abdd8b6","updated":"2015-11-11 15:16:23.000000000","message":"Done","commit_id":"bd25fa8de83d51073ec8921fa20fe94314a45bac"}]}