)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c581f29b494988866a03c0efb11b4295c2f027b0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"``glance-manage db purge_images_table`` to purge the records from images"},{"line_number":12,"context_line":"table."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"DocImpact"},{"line_number":15,"context_line":"SecurityImpact"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ie6641659b54543ed9f96c393d664e52a26bfaf6a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_080f9c2f","line":14,"updated":"2018-07-02 13:13:53.000000000","message":"Do we still need this tag now that docs are in-tree?","commit_id":"cfff1f8f21fcb51c4c815b3c6682da4c302dc001"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"498be7f47da010c8cef2f3863864d0d4769ba460","unresolved":false,"context_lines":[{"line_number":11,"context_line":"``glance-manage db purge_images_table`` to purge the records from images"},{"line_number":12,"context_line":"table."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"DocImpact"},{"line_number":15,"context_line":"SecurityImpact"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ie6641659b54543ed9f96c393d664e52a26bfaf6a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_b8bd8304","line":14,"in_reply_to":"5f7c97a3_080f9c2f","updated":"2018-07-03 09:53:03.000000000","message":"I guess we need to modify information about existing purge command and add information related to purge_images_table command.","commit_id":"cfff1f8f21fcb51c4c815b3c6682da4c302dc001"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d6a4c284d9fd6b280614ce4b5079bb6c0fd6afde","unresolved":false,"context_lines":[{"line_number":11,"context_line":"``glance-manage db purge_images_table`` to purge the records from images"},{"line_number":12,"context_line":"table."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"DocImpact"},{"line_number":15,"context_line":"SecurityImpact"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ie6641659b54543ed9f96c393d664e52a26bfaf6a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_1250e65d","line":14,"in_reply_to":"5f7c97a3_b8bd8304","updated":"2018-07-05 13:23:45.000000000","message":"I think it\u0027s worth using as a reminder when a doc change is required but isn\u0027t included on the current patch.","commit_id":"cfff1f8f21fcb51c4c815b3c6682da4c302dc001"}],"glance/cmd/manage.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d6a4c284d9fd6b280614ce4b5079bb6c0fd6afde","unresolved":false,"context_lines":[{"line_number":348,"context_line":"            sys.exit(_(\"Minimal rows limit is 1.\"))"},{"line_number":349,"context_line":"        ctx \u003d context.get_admin_context(show_deleted\u003dTrue)"},{"line_number":350,"context_line":"        try:"},{"line_number":351,"context_line":"            if purge_images:"},{"line_number":352,"context_line":"                db_api.purge_deleted_rows_from_images(ctx, age_in_days,"},{"line_number":353,"context_line":"                                                      max_rows)"},{"line_number":354,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_12ac6633","line":351,"range":{"start_line":351,"start_character":15,"end_line":351,"end_character":27},"updated":"2018-07-05 13:23:45.000000000","message":"kind of a nit, but maybe name this \u0027purge_images_only\u0027 so that it\u0027s clear that this function is supposed to either (a) purge only the images table, or (b) purge all the other tables.  (I can anticipate someone looking at this and thinking there\u0027s a logic error here.)","commit_id":"cfff1f8f21fcb51c4c815b3c6682da4c302dc001"}]}
