)]}'
{"manifests/db/sync.pp":[{"author":{"_account_id":10540,"name":"Clayton O\u0027Neill","email":"clayton.oneill@charter.com","username":"coneill"},"change_message_id":"a417df4e1fe63568e2eb79df314b81ded4d65d6d","unresolved":false,"context_lines":[{"line_number":6,"context_line":"  include ::heat::params"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"  Package \u003c| title \u003d\u003d \u0027heat-common\u0027 |\u003e -\u003e Class[\u0027heat::db::sync\u0027]"},{"line_number":9,"context_line":"  Package \u003c| title \u003d\u003d \u0027heat-engine\u0027 |\u003e -\u003e Class[\u0027heat::db::sync\u0027]"},{"line_number":10,"context_line":"  Heat_config\u003c||\u003e -\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":11,"context_line":"  Heat_config\u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":12,"context_line":"  Exec[\u0027heat-dbsync\u0027] ~\u003e Service \u003c| title \u003d\u003d \u0027heat-engine\u0027 |\u003e"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"9a41bdd9_b03e548a","line":9,"updated":"2015-07-14 19:59:06.000000000","message":"Package upgrades need to notify the exec so that db migrations can be run on upgrade. This is a change of semantics, however really all package upgrades should trigger a db sync, since any of them may inadvertently pull in the common package which is what actually contains the migrations.","commit_id":"33a6c87bf2bb45c8007a1dc599429da4a00a6427"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"84022306d8715a366ee8a82c90a4c3e00c9d52f3","unresolved":false,"context_lines":[{"line_number":6,"context_line":"  include ::heat::params"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"  Package \u003c| tag \u003d\u003d \u0027heat-package\u0027 |\u003e ~\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":9,"context_line":"  Exec[\u0027heat-dbsync\u0027] ~\u003e Service \u003c| tag \u003d\u003d \u0027heat-service\u0027 |\u003e"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  Heat_config\u003c||\u003e -\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":12,"context_line":"  Heat_config\u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027heat-dbsync\u0027]"}],"source_content_type":"text/x-puppet","patch_set":10,"id":"1a4dcd0f_59f2ebd3","line":9,"updated":"2015-08-04 14:11:32.000000000","message":"The same as for another projects: we shouldn\u0027t run exec after new heat package is installed. It can be run after all packages.","commit_id":"b0bc5d9a482ffda8c0e49aa70cf229c33bb171bb"},{"author":{"_account_id":9410,"name":"Yanis Guenane","email":"yguenane@redhat.com","username":"yguenane"},"change_message_id":"00cf3c326aa86adb4ff4c2dd42b0bbf1b4392723","unresolved":false,"context_lines":[{"line_number":6,"context_line":"  include ::heat::params"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"  Package \u003c| tag \u003d\u003d \u0027heat-package\u0027 |\u003e ~\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":9,"context_line":"  Exec[\u0027heat-dbsync\u0027] ~\u003e Service \u003c| tag \u003d\u003d \u0027heat-service\u0027 |\u003e"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  Heat_config\u003c||\u003e -\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":12,"context_line":"  Heat_config\u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027heat-dbsync\u0027]"}],"source_content_type":"text/x-puppet","patch_set":10,"id":"1a4dcd0f_bfa5cf47","line":9,"in_reply_to":"1a4dcd0f_59f2ebd3","updated":"2015-08-05 14:33:02.000000000","message":"Same comment as here: https://review.openstack.org/#/c/196683/6/manifests/db/sync.pp,cm","commit_id":"b0bc5d9a482ffda8c0e49aa70cf229c33bb171bb"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"84022306d8715a366ee8a82c90a4c3e00c9d52f3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"  Exec[\u0027heat-dbsync\u0027] ~\u003e Service \u003c| tag \u003d\u003d \u0027heat-service\u0027 |\u003e"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  Heat_config\u003c||\u003e -\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":12,"context_line":"  Heat_config\u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  exec { \u0027heat-dbsync\u0027:"},{"line_number":15,"context_line":"    command     \u003d\u003e $::heat::params::dbsync_command,"}],"source_content_type":"text/x-puppet","patch_set":10,"id":"1a4dcd0f_79ffa7aa","line":12,"updated":"2015-08-04 14:11:32.000000000","message":"The same as for another projects: i think we can run exec after line \u0027database/connection\u0027 was added.","commit_id":"b0bc5d9a482ffda8c0e49aa70cf229c33bb171bb"},{"author":{"_account_id":9410,"name":"Yanis Guenane","email":"yguenane@redhat.com","username":"yguenane"},"change_message_id":"00cf3c326aa86adb4ff4c2dd42b0bbf1b4392723","unresolved":false,"context_lines":[{"line_number":9,"context_line":"  Exec[\u0027heat-dbsync\u0027] ~\u003e Service \u003c| tag \u003d\u003d \u0027heat-service\u0027 |\u003e"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  Heat_config\u003c||\u003e -\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":12,"context_line":"  Heat_config\u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027heat-dbsync\u0027]"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"  exec { \u0027heat-dbsync\u0027:"},{"line_number":15,"context_line":"    command     \u003d\u003e $::heat::params::dbsync_command,"}],"source_content_type":"text/x-puppet","patch_set":10,"id":"1a4dcd0f_7f6ae7ef","line":12,"in_reply_to":"1a4dcd0f_79ffa7aa","updated":"2015-08-05 14:33:02.000000000","message":"Same comment as here: https://review.openstack.org/#/c/196683/6/manifests/db/sync.pp,cm","commit_id":"b0bc5d9a482ffda8c0e49aa70cf229c33bb171bb"}]}