)]}'
{"devstack/plugin.sh":[{"author":{"_account_id":20523,"name":"Andy Smith","email":"ansmith@redhat.com","username":"ajssmith"},"change_message_id":"6f29d617c1c5828d0c4f65957d7e6e918354f10a","unresolved":false,"context_lines":[{"line_number":23,"context_line":"#"},{"line_number":24,"context_line":"# Environment Configuration"},{"line_number":25,"context_line":"#"},{"line_number":26,"context_line":"# AMQP1_SERVICE - identifies the messaging backend to use.  Should be"},{"line_number":27,"context_line":"#    one of \u0027qpid\u0027 for broker backend, \u0027qpid-dual\u0027 for hybrid router-broker, or"},{"line_number":28,"context_line":"#    \u0027qpid-rpc\u0027 for keeping Rabbit for notifcations."},{"line_number":29,"context_line":"#    @TODO(kgiusti) add qpid-dispatch, etc"}],"source_content_type":"text/x-sh","patch_set":2,"id":"da36d5c6_74461150","line":26,"updated":"2017-02-24 16:00:35.000000000","message":"The qpidd broker is installed by default for the plugin which needs to change when \u0027qpid-rpc\u0027. \n\n\u0027qpid\u0027:\nrpc -\u003e qpidd broker\nnotification -\u003e qpidd broker\n\n\u0027qpid-dual\u0027:\nrpc -\u003e qdrouterd router\nnotification -\u003e qpidd broker\n\n\u0027qpid-rpc\u0027:\nrpc -\u003e qdrouterd router\nnotification -\u003e rabbit broker\n\nSuggestion to change \u0027qpid-rpc\u0027 to \u0027qpid-hybrid\u0027 or some better naming convention  for AMQP1_SERVICE if that makes sense.","commit_id":"aca651f8bd33d247380569bb5b01038bb6b5b385"},{"author":{"_account_id":7385,"name":"Thomas Herve","email":"therve@redhat.com","username":"therve"},"change_message_id":"e853383107641d50915362c86e3c0fdb23642495","unresolved":false,"context_lines":[{"line_number":23,"context_line":"#"},{"line_number":24,"context_line":"# Environment Configuration"},{"line_number":25,"context_line":"#"},{"line_number":26,"context_line":"# AMQP1_SERVICE - identifies the messaging backend to use.  Should be"},{"line_number":27,"context_line":"#    one of \u0027qpid\u0027 for broker backend, \u0027qpid-dual\u0027 for hybrid router-broker, or"},{"line_number":28,"context_line":"#    \u0027qpid-rpc\u0027 for keeping Rabbit for notifcations."},{"line_number":29,"context_line":"#    @TODO(kgiusti) add qpid-dispatch, etc"}],"source_content_type":"text/x-sh","patch_set":2,"id":"da36d5c6_d4825dc7","line":26,"in_reply_to":"da36d5c6_74461150","updated":"2017-02-24 16:10:27.000000000","message":"That makes sense. I tried to address the comments.","commit_id":"aca651f8bd33d247380569bb5b01038bb6b5b385"}],"devstack/settings":[{"author":{"_account_id":20523,"name":"Andy Smith","email":"ansmith@redhat.com","username":"ajssmith"},"change_message_id":"1c2d022476f2b4cefbd37df4c343919f6a3ed572","unresolved":false,"context_lines":[{"line_number":2,"context_line":"AMQP1_SERVICE\u003d${AMQP1_SERVICE:-qpid}"},{"line_number":3,"context_line":"AMQP1_HOST\u003d${AMQP1_HOST:-$SERVICE_HOST}"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"if [ \"$AMQP1_SERVICE\" !\u003d \"qpid-rpc\" ]; then"},{"line_number":6,"context_line":"    disable_service rabbit"},{"line_number":7,"context_line":"fi"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"da36d5c6_293a7799","line":5,"updated":"2017-02-24 19:14:21.000000000","message":"\"qpid-rpc\" should be \"qpid-hybrid\"","commit_id":"97c1ae1ec72a460e4dafddf95325acbca994a09d"}]}