)]}'
{"jenkins/scripts/propose_update.sh":[{"author":{"_account_id":9410,"name":"Yanis Guenane","email":"yguenane@redhat.com","username":"yguenane"},"change_message_id":"cbada58f59d47c104ad8f615b711fb9209a1225a","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        bash -xe tools/sync-projects.sh $1"},{"line_number":39,"context_line":"    }"},{"line_number":40,"context_line":"elif [ \"$OWN_PROJECT\" \u003d\u003d \"puppet-modulesync-configs\" ] ; then"},{"line_number":41,"context_line":"    INITIAL_COMMIT_MSG\u003d\"Updated from puppet-modulesync-configs\""},{"line_number":42,"context_line":"    TOPIC\u003d\"openstack/puppet-modulesync-configs\""},{"line_number":43,"context_line":"    # Pretty ugly, but this should be moved to a local script."},{"line_number":44,"context_line":"    PROJECTS\u003d$(cat managed_modules.yml | cut -d \u0027-\u0027 -f 2,3 | grep puppet | sed \u0027s/ //g\u0027 | sed \u0027s/^/openstack\\//g\u0027)"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1a4dcd0f_b0bf79a9","line":41,"updated":"2015-08-10 16:20:01.000000000","message":"An idea: Retrieve current commit message from openstack/puppet-modulesync-configs and amend the first line with [msync] this way when reading other project git history we still know what it is about. What do you think ?","commit_id":"688bcec5d98f4e40a2fc024b2b8e2e8e743adc17"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"14ec84527fd05aea7ea2e7fdd5483b6a6784236d","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        bash -xe tools/sync-projects.sh $1"},{"line_number":39,"context_line":"    }"},{"line_number":40,"context_line":"elif [ \"$OWN_PROJECT\" \u003d\u003d \"puppet-modulesync-configs\" ] ; then"},{"line_number":41,"context_line":"    INITIAL_COMMIT_MSG\u003d\"Updated from puppet-modulesync-configs\""},{"line_number":42,"context_line":"    TOPIC\u003d\"openstack/puppet-modulesync-configs\""},{"line_number":43,"context_line":"    # Pretty ugly, but this should be moved to a local script."},{"line_number":44,"context_line":"    PROJECTS\u003d$(cat managed_modules.yml | cut -d \u0027-\u0027 -f 2,3 | grep puppet | sed \u0027s/ //g\u0027 | sed \u0027s/^/openstack\\//g\u0027)"}],"source_content_type":"text/x-sh","patch_set":1,"id":"1a4dcd0f_981d1664","line":41,"in_reply_to":"1a4dcd0f_b0bf79a9","updated":"2015-08-13 14:24:28.000000000","message":"Ya, this would have to be a follow up patch. But we might be able to do something.  Currently, the script simply appends the if a review is currently open.  So, we\u0027d have to add some magic to commit messages to do this.\n\nIf there is no open reviews, it is not an issue.","commit_id":"688bcec5d98f4e40a2fc024b2b8e2e8e743adc17"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"b4226867a3e2e737b51b775a689470ca48e73ea6","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    gem install bundler"},{"line_number":48,"context_line":"    ${GEM_HOME}/bin/bundle install"},{"line_number":49,"context_line":"    function update {"},{"line_number":50,"context_line":"        bash -xe tools/sync-projects.sh $1"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":"elif [ \"$OWN_PROJECT\" \u003d\u003d \"requirements-constraints\" ] ; then"},{"line_number":53,"context_line":"    INITIAL_COMMIT_MSG\u003d\"Updated from generate-constraints\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a4dcd0f_a297b3ea","line":50,"updated":"2015-08-12 18:00:45.000000000","message":"There is no script by this name in the openstack/puppet-modulesync-configs repo.","commit_id":"e9bf034b665d4e2d56b31c8bfd15cb73315c4bf0"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"03dea73c99c9e6ac56234ff88a5085e7d0256c71","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    gem install bundler"},{"line_number":48,"context_line":"    ${GEM_HOME}/bin/bundle install"},{"line_number":49,"context_line":"    function update {"},{"line_number":50,"context_line":"        bash -xe tools/sync-projects.sh $1"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":"elif [ \"$OWN_PROJECT\" \u003d\u003d \"requirements-constraints\" ] ; then"},{"line_number":53,"context_line":"    INITIAL_COMMIT_MSG\u003d\"Updated from generate-constraints\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a4dcd0f_a0b1ebe1","line":50,"in_reply_to":"1a4dcd0f_6558d58a","updated":"2015-08-12 18:32:13.000000000","message":"In that case I\u0027ve moved my -1 for this to the child change which would otherwise start running this before it can possibly work.","commit_id":"e9bf034b665d4e2d56b31c8bfd15cb73315c4bf0"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"7c6ab934e029b10673a34b768a429681b0982107","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    gem install bundler"},{"line_number":48,"context_line":"    ${GEM_HOME}/bin/bundle install"},{"line_number":49,"context_line":"    function update {"},{"line_number":50,"context_line":"        bash -xe tools/sync-projects.sh $1"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":"elif [ \"$OWN_PROJECT\" \u003d\u003d \"requirements-constraints\" ] ; then"},{"line_number":53,"context_line":"    INITIAL_COMMIT_MSG\u003d\"Updated from generate-constraints\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"1a4dcd0f_6558d58a","line":50,"in_reply_to":"1a4dcd0f_a297b3ea","updated":"2015-08-12 18:25:05.000000000","message":"I have the Depends-On header the opposite way: https://review.openstack.org/#/c/210528/","commit_id":"e9bf034b665d4e2d56b31c8bfd15cb73315c4bf0"}]}