)]}'
{"gate/post_test_hook.sh":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fac721f52b8be750d0da10fc4f937d9f8cacb2e6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    # NOTE(danms): Run this a few times to make sure that we end"},{"line_number":8,"context_line":"    # up with nothing more to archive"},{"line_number":9,"context_line":"    for i in `seq 30`; do"},{"line_number":10,"context_line":"        $MANAGE $conf db archive_deleted_rows --verbose --max_rows 1000"},{"line_number":11,"context_line":"        RET\u003d$?"},{"line_number":12,"context_line":"        if [[ $RET -gt 1 ]]; then"},{"line_number":13,"context_line":"            echo Archiving failed with result $RET"}],"source_content_type":"text/x-sh","patch_set":3,"id":"df7087c5_f6ecd0f4","line":10,"updated":"2018-03-07 16:31:13.000000000","message":"This didn\u0027t work properly:\n\nhttp://logs.openstack.org/94/550194/3/check/nova-next/167b301/logs/devstack-gate-post_test_hook.txt.gz#_2018-03-07_16_20_22_945\n\n/usr/local/bin/nova-manage --config-file db archive_deleted_rows --verbose --max_rows 1000","commit_id":"2bce7de3aebfcfe4f80f693553e64d290f06d79c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a51ab903ac63a28ab3eaffbc0283904b9d38d017","unresolved":false,"context_lines":[{"line_number":22,"context_line":"BASE\u003d${BASE:-/opt/stack}"},{"line_number":23,"context_line":"source ${BASE}/new/devstack/lib/nova"},{"line_number":24,"context_line":"cell_conf\u003d$(conductor_conf 1)"},{"line_number":25,"context_line":"conf\u003d\"--config-file $NOVA_CONF --config-file $cell_conf\""},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"archive_deleted_rows $conf"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"df7087c5_3666c8ef","line":25,"updated":"2018-03-07 16:28:42.000000000","message":"A comment here would be good to explain why we\u0027re passing in two config files (once is to cleanup the cell1 database, another is needed to access the API database to remove instance mappings). I\u0027m not sure how oslo.config is going to handle the conflicting [database]/connection entries (one for cell0 and one for cell1). Also, this can eventually go away once we have an --all-cells option on archive_deleted_rows, and at that point we can remove the conf stuff since it will just use NOVA_CONF by default and we should be good.","commit_id":"2bce7de3aebfcfe4f80f693553e64d290f06d79c"}]}