)]}'
{"manifests/db/populate.pp":[{"author":{"_account_id":15881,"name":"Limor Stotland","email":"limor.bortman@nokia.com","username":"LimorStotlamd"},"change_message_id":"43ed5e7fadeb2ec0971c1a0e8d603d00e489f494","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Class to execute \"mistral-db-manage populate\""},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"class mistral::db::populate {"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  include ::mistral::params"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba8a016a_d351f00f","line":4,"updated":"2015-11-22 12:08:45.000000000","message":"why create new file and not use :\npuppet-mistral/manifests/db/sync.pp \nhttps://github.com/openstack/puppet-mistral/blob/master/manifests/db/sync.pp","commit_id":"1fc2a7cd7a4657b8a89f45c88316e2fb0c790242"},{"author":{"_account_id":15881,"name":"Limor Stotland","email":"limor.bortman@nokia.com","username":"LimorStotlamd"},"change_message_id":"a9c1fb8503f49986c2f98fd284691be5932bf9d1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Class to execute \"mistral-db-manage populate\""},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"class mistral::db::populate {"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  include ::mistral::params"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba8a016a_1369b88b","line":4,"in_reply_to":"ba8a016a_b336acb2","updated":"2015-11-22 12:53:39.000000000","message":"I am not sure you are right about the \"running just the populate would always fail without the db_sync\"\nwe use this puppet as it and we don\u0027t have any problem with the populate....","commit_id":"1fc2a7cd7a4657b8a89f45c88316e2fb0c790242"},{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"364113c6466e179be281fb501ef0dbfd9abd42be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Class to execute \"mistral-db-manage populate\""},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"class mistral::db::populate {"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  include ::mistral::params"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba8a016a_b336acb2","line":4,"in_reply_to":"ba8a016a_d351f00f","updated":"2015-11-22 12:49:49.000000000","message":"Hmmm. It was my understanding these were two unique steps. I suppose I thought it entirely possible that someone might want to run a db_sync without running a populate. NOTE: that running just the populate would always fail without the db_sync. It was for this reason, and naming consistency that I thought correcting db_sync to actually call \u0027upgrade head\u0027 would make sense. I realize this is a breaking change... but I would like to consider it for consistency with other OPM modules. Otherwise I fear people will find some of these module names a bit confusing.","commit_id":"1fc2a7cd7a4657b8a89f45c88316e2fb0c790242"}],"manifests/params.pp":[{"author":{"_account_id":15881,"name":"Limor Stotland","email":"limor.bortman@nokia.com","username":"LimorStotlamd"},"change_message_id":"43ed5e7fadeb2ec0971c1a0e8d603d00e489f494","unresolved":false,"context_lines":[{"line_number":9,"context_line":"  $client_package     \u003d \u0027python-mistralclient\u0027"},{"line_number":10,"context_line":"  $log_dir \u003d\u0027/var/log/mistral\u0027"},{"line_number":11,"context_line":"  $db_sync_command \u003d \"mistral-db-manage --config-file\u003d${mistral_conf} upgrade head\""},{"line_number":12,"context_line":"  $db_populate_command \u003d \"mistral-db-manage --config-file\u003d${mistral_conf} populate\""},{"line_number":13,"context_line":"  $update_service_command \u003d \u0027/usr/bin/systemctl daemon-reload\u0027"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"  case $::osfamily {"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba8a016a_1331b8ec","line":12,"updated":"2015-11-22 12:08:45.000000000","message":"your file likes a bit different from the github one...\nlook at the this link : https://github.com/openstack/puppet-mistral/blob/master/manifests/params.pp\n\nwe do have  $dbsync_command \u003d \"/usr/bin/python /usr/bin/mistral-db-manage --config-file\u003d${mistral_conf} populate\"","commit_id":"1fc2a7cd7a4657b8a89f45c88316e2fb0c790242"}]}