)]}'
{"extraconfig/tasks/major_upgrade_ceph_mon.sh":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"ba9afbacec84c4044a339166aa4ae06e47f2e126","unresolved":false,"context_lines":[{"line_number":18,"context_line":"fi"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"CEPH_STATUS\u003d$(ceph health | awk \u0027{print $1}\u0027)"},{"line_number":21,"context_line":"if [ ${CEPH_STATUS} \u003d HEALTH_ERR ]; then"},{"line_number":22,"context_line":"    echo ERROR: Ceph cluster status is HEALTH_ERR, cannot be upgraded"},{"line_number":23,"context_line":"    exit 1"},{"line_number":24,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_3f3015e8","line":21,"updated":"2016-09-15 21:24:44.000000000","message":"https://review.openstack.org/#/c/359410/2/extraconfig/tasks/major_upgrade_ceph_mon.sh","commit_id":"bf687680499d3874b43c5442bcbe4404ea5f0e3e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b2de3e6aea9f50f352e98b464bb9f8df4ceba566","unresolved":false,"context_lines":[{"line_number":54,"context_line":"elif [[ \"$UPDATED_VERSION\" \u003d~ ^10\\.2.* ]]; then"},{"line_number":55,"context_line":"    # RPM could own some of these but we can\u0027t take risks on the pre-existing files"},{"line_number":56,"context_line":"    for d in /var/lib/ceph/mon /var/log/ceph /var/run/ceph /etc/ceph; do"},{"line_number":57,"context_line":"        chown -R ceph:ceph $d || true"},{"line_number":58,"context_line":"    done"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    # Replay udev events with newer rules"}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_6f7c92b5","line":57,"updated":"2016-09-16 11:03:24.000000000","message":"just a thought here... do we at least want to know if this happens, e.g. if the file doesn\u0027t exist? or we dont care if it does/not.","commit_id":"bf687680499d3874b43c5442bcbe4404ea5f0e3e"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"6836fd5a63d7a35c025a9f9cd19772b47e290423","unresolved":false,"context_lines":[{"line_number":54,"context_line":"elif [[ \"$UPDATED_VERSION\" \u003d~ ^10\\.2.* ]]; then"},{"line_number":55,"context_line":"    # RPM could own some of these but we can\u0027t take risks on the pre-existing files"},{"line_number":56,"context_line":"    for d in /var/lib/ceph/mon /var/log/ceph /var/run/ceph /etc/ceph; do"},{"line_number":57,"context_line":"        chown -R ceph:ceph $d || true"},{"line_number":58,"context_line":"    done"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    # Replay udev events with newer rules"}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_92eae50d","line":57,"in_reply_to":"7a8ec9b2_6f7c92b5","updated":"2016-09-16 11:13:11.000000000","message":"let\u0027s echo it at least, thanks!","commit_id":"bf687680499d3874b43c5442bcbe4404ea5f0e3e"}],"extraconfig/tasks/major_upgrade_ceph_storage.sh":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"94442fcc034efc164401f9b33a7b71215a5f1dca","unresolved":false,"context_lines":[{"line_number":63,"context_line":"elif [[ \"$UPDATED_VERSION\" \u003d~ ^10\\.2.* ]]; then"},{"line_number":64,"context_line":"    # RPM could own some of these but we can\u0027t take risks on the pre-existing files"},{"line_number":65,"context_line":"    for d in /var/lib/ceph/osd /var/log/ceph /var/run/ceph /etc/ceph; do"},{"line_number":66,"context_line":"        chown -R ceph:ceph $d || true"},{"line_number":67,"context_line":"    done"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"    # Replay udev events with newer rules"}],"source_content_type":"text/x-sh","patch_set":1,"id":"7a8ec9b2_4f2d16a7","line":66,"updated":"2016-09-16 11:03:52.000000000","message":"same questino, do we care if it isn\u0027t successful?","commit_id":"bf687680499d3874b43c5442bcbe4404ea5f0e3e"}]}