)]}'
{"manifests/db/sync.pp":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"c093db04b93fc5e7b93fe0bf2abfed66cf367e31","unresolved":false,"context_lines":[{"line_number":4,"context_line":"class cinder::db::sync inherits cinder::params {"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  Package \u003c| title \u003d\u003d \u0027cinder\u0027 |\u003e -\u003e Class[\u0027cinder::db::sync\u0027]"},{"line_number":7,"context_line":"  Cinder_config \u003c||\u003e -\u003e Exec[\u0027cinder-manage db_sync\u0027]"},{"line_number":8,"context_line":"  Cinder_config \u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027cinder-manage db_sync\u0027]"},{"line_number":9,"context_line":"  Exec[\u0027cinder-manage db_sync\u0027] ~\u003e Service \u003c| title \u003d\u003d \u0027cinder-manage db_sync\u0027 |\u003e"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba3cc151_e711bf05","line":7,"updated":"2015-07-07 10:32:08.000000000","message":"please use ~\u003e as it was in original code","commit_id":"07e00835e9bfb43b92d1114e67e4d5481399e0c1"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"4f17d10273fdcd8a9430c5679ba3a8569d7bf961","unresolved":false,"context_lines":[{"line_number":6,"context_line":"  Package \u003c| title \u003d\u003d \u0027cinder\u0027 |\u003e -\u003e Class[\u0027cinder::db::sync\u0027]"},{"line_number":7,"context_line":"  Cinder_config \u003c||\u003e ~\u003e Exec[\u0027cinder-manage db_sync\u0027]"},{"line_number":8,"context_line":"  Cinder_config \u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027cinder-manage db_sync\u0027]"},{"line_number":9,"context_line":"  Exec[\u0027cinder-manage db_sync\u0027] ~\u003e Service \u003c| title \u003d\u003d \u0027cinder-manage db_sync\u0027 |\u003e"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"  exec { \u0027cinder-manage db_sync\u0027:"},{"line_number":12,"context_line":"    command     \u003d\u003e $::cinder::params::db_sync_command,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ba3cc151_3733d563","line":9,"updated":"2015-07-09 11:15:01.000000000","message":"Service \u003c| title \u003d\u003d \u0027cinder-manage db_sync\u0027 |\u003e\n\n... uhm .. what?","commit_id":"75cad782bc1798a2eec7240da068336470b2905d"},{"author":{"_account_id":9060,"name":"Mike Dorman","email":"dorm@dorm.org","username":"gd-mdorman"},"change_message_id":"cde47c786f4a6232201ee5a2b9662c1c930af82a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Class to execute cinder dbsync"},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"class cinder::db::sync inherits cinder::params {"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  Package \u003c| title \u003d\u003d \u0027cinder\u0027 |\u003e -\u003e Class[\u0027cinder::db::sync\u0027]"},{"line_number":7,"context_line":"  Cinder_config \u003c||\u003e ~\u003e Exec[\u0027cinder-manage db_sync\u0027]"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"ba3cc151_c5ddfa41","line":4,"updated":"2015-07-09 19:02:07.000000000","message":"include instead of inherits, as Mathieu as pointed out in other modules.","commit_id":"6d3c56fc9257473ba8ec91963645c7ea69621c65"},{"author":{"_account_id":10540,"name":"Clayton O\u0027Neill","email":"clayton.oneill@charter.com","username":"coneill"},"change_message_id":"7ac1c9b540a39326913a435910c52ec0965e6493","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  include ::cinder::params"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"  Package \u003c| title \u003d\u003d \u0027cinder\u0027 |\u003e -\u003e Class[\u0027cinder::db::sync\u0027]"},{"line_number":9,"context_line":"  Cinder_config \u003c||\u003e ~\u003e Exec[\u0027cinder-manage db_sync\u0027]"},{"line_number":10,"context_line":"  Cinder_config \u003c| title \u003d\u003d \u0027database/connection\u0027 |\u003e ~\u003e Exec[\u0027cinder-manage db_sync\u0027]"},{"line_number":11,"context_line":"  Exec[\u0027cinder-manage db_sync\u0027] -\u003e Service \u003c| title \u003d\u003d \u0027cinder-api\u0027 |\u003e"}],"source_content_type":"text/x-puppet","patch_set":5,"id":"9a41bdd9_d0ef58e7","line":8,"updated":"2015-07-14 20:03:00.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":"71b8ad833518345b16a983094904ff7ec85879c9"}],"spec/classes/cinder_api_spec.rb":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"c4ea89087ac6b6439bcf19113bac837e52f93900","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    it \u0027should stop the service\u0027 do"},{"line_number":183,"context_line":"      is_expected.to contain_service(\u0027cinder-api\u0027).with_ensure(\u0027stopped\u0027)"},{"line_number":184,"context_line":"    end"},{"line_number":185,"context_line":"    it \u0027does not include cinder::db::sync\u0027 do"},{"line_number":186,"context_line":"      is_expected.to contain_class(\u0027cinder::db::sync\u0027)"},{"line_number":187,"context_line":"    end"},{"line_number":188,"context_line":"  end"}],"source_content_type":"text/x-ruby","patch_set":4,"id":"9a41bdd9_80e31433","line":185,"updated":"2015-07-13 12:58:25.000000000","message":"does not?","commit_id":"be44ee66d44304c19df38bb0b1186c776d7733ce"}]}