)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"59f0bd6201f77d3dcfa137644a198896b9d21d9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7c54ca7d_da16a177","updated":"2022-05-14 01:13:35.000000000","message":"Stealing this from the original author who does not seem to have been active in OpenStack for the past few years.","commit_id":"ac9ae860764966cc140e1f47ced77ac6c4316f0f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"3d3d493b88367cac8357b2c9e9065b34c0ca66f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c96c673a_a1794a39","updated":"2022-05-18 06:58:12.000000000","message":"Worth mentioning other command as well.","commit_id":"ac9ae860764966cc140e1f47ced77ac6c4316f0f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"5c93e7c6eb6ec24801af1a6d58d7afa33dc0c610","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4c31a109_b2463281","updated":"2022-12-15 14:07:00.000000000","message":"Thank you!","commit_id":"b17f770b5fee3d52cece8451b7adb3f944270404"}],"doc/source/cli/glancemanage.rst":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"3d3d493b88367cac8357b2c9e9065b34c0ca66f8","unresolved":true,"context_lines":[{"line_number":82,"context_line":"      metadef_objects, metadef_resource_types, metadef_namespaces and"},{"line_number":83,"context_line":"      metadef_properties."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"``db_purge``"},{"line_number":86,"context_line":"      Purge deleted rows older than a given age from glance db tables."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"      This command interprets the following options when it is invoked:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"0b901287_65f52ae4","line":85,"range":{"start_line":85,"start_character":2,"end_line":85,"end_character":10},"updated":"2022-05-18 06:58:12.000000000","message":"similar to this we have db_purge_images_table command as well to purge deleted rows from images table.","commit_id":"ac9ae860764966cc140e1f47ced77ac6c4316f0f"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"31a568b019cbf92e60fb21ab9e8d01c5eb79646b","unresolved":false,"context_lines":[{"line_number":82,"context_line":"      metadef_objects, metadef_resource_types, metadef_namespaces and"},{"line_number":83,"context_line":"      metadef_properties."},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"``db_purge``"},{"line_number":86,"context_line":"      Purge deleted rows older than a given age from glance db tables."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"      This command interprets the following options when it is invoked:"}],"source_content_type":"text/x-rst","patch_set":7,"id":"ec937096_09088a7f","line":85,"range":{"start_line":85,"start_character":2,"end_line":85,"end_character":10},"in_reply_to":"0b901287_65f52ae4","updated":"2022-07-05 01:46:55.000000000","message":"Done","commit_id":"ac9ae860764966cc140e1f47ced77ac6c4316f0f"}],"doc/source/man/glancemanage.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"fb7246aa1563d736e78efbcd1d0df692503cc74c","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        metadef_properties."},{"line_number":75,"context_line":"  **db_purge [AGE_IN_DAYS] [MAX_ROWS]**"},{"line_number":76,"context_line":"        Purge deleted rows older than a given age from glance db tables."},{"line_number":77,"context_line":"        Please be aware of these related OpenStack Security Notes:"},{"line_number":78,"context_line":"        https://wiki.openstack.org/wiki/OSSN/OSSN-0075."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"OPTIONS"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a461143_93e42cad","line":77,"range":{"start_line":77,"start_character":8,"end_line":77,"end_character":66},"updated":"2017-02-01 22:21:02.000000000","message":"I suggest replacing with:\n\n    This function is useful primarily in test systems.\n    We do not recommend its use in production systems\n    unless you have reviewed OpenStack Security Note\n    `OSSN-0075`_ and understand the risk involved.\n\n    .._`OSSN-0075`: https://wiki.openstack.org/wiki/OSSN/OSSN-0075","commit_id":"1aea81df46008c81c83489318475105d0f84f558"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"edd87baac40deef81f085381cf4e7d6a831aa205","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        metadef_properties."},{"line_number":75,"context_line":"  **db_purge [AGE_IN_DAYS] [MAX_ROWS]**"},{"line_number":76,"context_line":"        Purge deleted rows older than a given age from glance db tables."},{"line_number":77,"context_line":"        Please be aware of these related OpenStack Security Notes:"},{"line_number":78,"context_line":"        https://wiki.openstack.org/wiki/OSSN/OSSN-0075."},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"OPTIONS"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a461143_bf40ff36","line":77,"range":{"start_line":77,"start_character":8,"end_line":77,"end_character":66},"in_reply_to":"3a461143_93e42cad","updated":"2017-02-02 09:24:50.000000000","message":"Done","commit_id":"1aea81df46008c81c83489318475105d0f84f558"},{"author":{"_account_id":22448,"name":"Alexander Bashmakov","email":"abashmak@yahoo.com","username":"abashmak"},"change_message_id":"beb9bfd346fc3ad3101ad1c15982467c93c2f42f","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        db tables including metadef_namespace_resource_types, metadef_tags,"},{"line_number":73,"context_line":"        metadef_objects, metadef_resource_types, metadef_namespaces and"},{"line_number":74,"context_line":"        metadef_properties."},{"line_number":75,"context_line":"  **db_purge [AGE_IN_DAYS] [MAX_ROWS]**"},{"line_number":76,"context_line":"        Purge deleted rows older than a given age from glance db tables."},{"line_number":77,"context_line":"        This function is useful primarily in test systems. We do not recommend"},{"line_number":78,"context_line":"        its use in production systems unless you have reviewed OpenStack"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a461143_2a12f309","line":75,"range":{"start_line":75,"start_character":13,"end_line":75,"end_character":37},"updated":"2017-02-02 19:57:43.000000000","message":"The age_in_days and max_rows are named arguments. I think the title should be changed to: **db_purge [OPTIONS]**\n\nThen, in the description paragraph, above the OSSN note, you can either describe --age_in_days and --max_rows in detail or maybe it\u0027s enough to simply say:\n\n\"refer to ``glance-manage db purge --help`` for details on optional arguments to this command.\"","commit_id":"37e8f461ac0e4fbb61c19ff990db4e9c37b46c5f"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"04194ddcf84b405929be762255a77dd268c03e69","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        db tables including metadef_namespace_resource_types, metadef_tags,"},{"line_number":73,"context_line":"        metadef_objects, metadef_resource_types, metadef_namespaces and"},{"line_number":74,"context_line":"        metadef_properties."},{"line_number":75,"context_line":"  **db_purge [AGE_IN_DAYS] [MAX_ROWS]**"},{"line_number":76,"context_line":"        Purge deleted rows older than a given age from glance db tables."},{"line_number":77,"context_line":"        This function is useful primarily in test systems. We do not recommend"},{"line_number":78,"context_line":"        its use in production systems unless you have reviewed OpenStack"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa31d9ce_be6b38f0","line":75,"range":{"start_line":75,"start_character":13,"end_line":75,"end_character":37},"in_reply_to":"3a461143_2a12f309","updated":"2017-02-14 11:32:35.000000000","message":"Added the command options to the man page. \n\nIn general I think it\u0027s a very bad idea to refer to any help command from a man page. The man page is supposed to be a full command reference if available documenting all options etc.","commit_id":"37e8f461ac0e4fbb61c19ff990db4e9c37b46c5f"},{"author":{"_account_id":22448,"name":"Alexander Bashmakov","email":"abashmak@yahoo.com","username":"abashmak"},"change_message_id":"79c5ec484e2b2294a8bc802d73fd3a84f030652c","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        db tables including metadef_namespace_resource_types, metadef_tags,"},{"line_number":73,"context_line":"        metadef_objects, metadef_resource_types, metadef_namespaces and"},{"line_number":74,"context_line":"        metadef_properties."},{"line_number":75,"context_line":"  **db_purge [AGE_IN_DAYS] [MAX_ROWS]**"},{"line_number":76,"context_line":"        Purge deleted rows older than a given age from glance db tables."},{"line_number":77,"context_line":"        This function is useful primarily in test systems. We do not recommend"},{"line_number":78,"context_line":"        its use in production systems unless you have reviewed OpenStack"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa31d9ce_76f71959","line":75,"range":{"start_line":75,"start_character":13,"end_line":75,"end_character":37},"in_reply_to":"fa31d9ce_be6b38f0","updated":"2017-02-14 16:12:23.000000000","message":"Agreed. Thanks for adding the full description in PS4. Kind of an important nit inline there, otherwise looks good.","commit_id":"37e8f461ac0e4fbb61c19ff990db4e9c37b46c5f"},{"author":{"_account_id":22448,"name":"Alexander Bashmakov","email":"abashmak@yahoo.com","username":"abashmak"},"change_message_id":"9dc01f2eb1b315a6ac8563cbee8cdbc1d0c63491","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        This command interprets the following options when it is invoked:"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        --max_rows      Purge deleted rows older than age in days"},{"line_number":93,"context_line":"        --age_in_days   Limit number of records to delete"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        WARNING: This function is useful primarily in test systems. We do not"},{"line_number":96,"context_line":"        recommend its use in production systems unless you have reviewed"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa31d9ce_d6ce052c","line":93,"range":{"start_line":92,"start_character":0,"end_line":93,"end_character":21},"updated":"2017-02-14 16:11:43.000000000","message":"nit: please include default values if not specified (100 rows, 30 days)","commit_id":"69bfb84e34617d44016658a28820bba01fc18b48"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ff059e79084b7b01d0442d855c2294873b67d541","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        db tables including metadef_namespace_resource_types, metadef_tags,"},{"line_number":85,"context_line":"        metadef_objects, metadef_resource_types, metadef_namespaces and"},{"line_number":86,"context_line":"        metadef_properties."},{"line_number":87,"context_line":"  **db_purge**"},{"line_number":88,"context_line":"        Purge deleted rows older than a given age from glance db tables."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        This command interprets the following options when it is invoked:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa31d9ce_7b212bfc","line":87,"updated":"2017-02-17 15:44:54.000000000","message":"might as well add a blank line before this entry to be consistent with the rest of the file","commit_id":"15517a648ba9cf432f884f5fd46426fb47aea6d6"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"2e59209491356c9dcfac3c23416a9d30422b37d7","unresolved":false,"context_lines":[{"line_number":84,"context_line":"        db tables including metadef_namespace_resource_types, metadef_tags,"},{"line_number":85,"context_line":"        metadef_objects, metadef_resource_types, metadef_namespaces and"},{"line_number":86,"context_line":"        metadef_properties."},{"line_number":87,"context_line":"  **db_purge**"},{"line_number":88,"context_line":"        Purge deleted rows older than a given age from glance db tables."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        This command interprets the following options when it is invoked:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"da36d5c6_754ed3c2","line":87,"in_reply_to":"fa31d9ce_7b212bfc","updated":"2017-02-18 17:12:24.000000000","message":"Done","commit_id":"15517a648ba9cf432f884f5fd46426fb47aea6d6"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ff059e79084b7b01d0442d855c2294873b67d541","unresolved":false,"context_lines":[{"line_number":97,"context_line":"        WARNING: This function is useful primarily in test systems. We do not"},{"line_number":98,"context_line":"        recommend its use in production systems unless you have reviewed"},{"line_number":99,"context_line":"        OpenStack Security Note `OSSN-0075`_ and understand the risk involved."},{"line_number":100,"context_line":"        .. _`OSSN-0075`: https://wiki.openstack.org/wiki/OSSN/OSSN-0075"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"OPTIONS"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fa31d9ce_db1137b1","line":100,"updated":"2017-02-17 15:44:54.000000000","message":"need a blank line between lines 99 and 100 (you can keep the indentation), otherwise the hyperlink isn\u0027t generated correctly","commit_id":"15517a648ba9cf432f884f5fd46426fb47aea6d6"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"2e59209491356c9dcfac3c23416a9d30422b37d7","unresolved":false,"context_lines":[{"line_number":97,"context_line":"        WARNING: This function is useful primarily in test systems. We do not"},{"line_number":98,"context_line":"        recommend its use in production systems unless you have reviewed"},{"line_number":99,"context_line":"        OpenStack Security Note `OSSN-0075`_ and understand the risk involved."},{"line_number":100,"context_line":"        .. _`OSSN-0075`: https://wiki.openstack.org/wiki/OSSN/OSSN-0075"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"OPTIONS"}],"source_content_type":"text/x-rst","patch_set":5,"id":"da36d5c6_954b47b0","line":100,"in_reply_to":"fa31d9ce_db1137b1","updated":"2017-02-18 17:12:24.000000000","message":"Done","commit_id":"15517a648ba9cf432f884f5fd46426fb47aea6d6"}]}
