)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16883,"name":"huyang","email":"huyang1@huawei.com","username":"huyang"},"change_message_id":"34e161e7746171c8fa45c7d814d9d34903990684","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vilobh Meshram \u003cvilobhmm@yahoo-inc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-31 17:46:25 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Instance un-usable if instance snapshot fails due to in-sufficient storage space"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Protecting data is one of the most important aspect for better User"},{"line_number":10,"context_line":"experience. Not every private cloud deployment have the facility to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_bd0524c3","line":7,"updated":"2015-08-04 12:24:59.000000000","message":"The first line should be limited to 50 characters. see https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"0db0e6acb18ab6c9311f933e8cfbcbb118f929e8"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"828b3a81182d2cd847ff521e393a6a881d53e179","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vilobh Meshram \u003cvilobhmm@yahoo-inc.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-31 17:46:25 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Instance un-usable if instance snapshot fails due to in-sufficient storage space"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Protecting data is one of the most important aspect for better User"},{"line_number":10,"context_line":"experience. Not every private cloud deployment have the facility to"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1a4dcd0f_ebedfd25","line":7,"in_reply_to":"1a4dcd0f_bd0524c3","updated":"2015-08-05 02:56:03.000000000","message":"Done","commit_id":"0db0e6acb18ab6c9311f933e8cfbcbb118f929e8"}],"specs/mitaka/approved/instance-snapshot-vm-unusable-insufficient-storage-space.rst":[{"author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"change_message_id":"0bc4c894c91e56840139610d9a419d740a4a0dea","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"If step #1 fails, lets say, due to in-sufficient storage space, Nova"},{"line_number":24,"context_line":"fails to create the snapshot. As a result of which the instance"},{"line_number":25,"context_line":"goes in an un-usable state and can;t be recovered. For step #2 a"},{"line_number":26,"context_line":"seperate spec will be filed which will take care of glance quota"},{"line_number":27,"context_line":"enhancements as not every private cloud deployment would have"},{"line_number":28,"context_line":"infinite storage for image/snapshot storage and having a quota"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_24722b7d","line":25,"updated":"2015-07-31 23:01:13.000000000","message":"\u0027can;t\u0027 -\u003e \u0027can\u0027t\u0027","commit_id":"88a37ebcae3d716196c6549b5233d92d0a327614"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"a561076ae8beaed584b34bda3f1f5f29376199b0","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"If step #1 fails, lets say, due to in-sufficient storage space, Nova"},{"line_number":24,"context_line":"fails to create the snapshot. As a result of which the instance"},{"line_number":25,"context_line":"goes in an un-usable state and can;t be recovered. For step #2 a"},{"line_number":26,"context_line":"seperate spec will be filed which will take care of glance quota"},{"line_number":27,"context_line":"enhancements as not every private cloud deployment would have"},{"line_number":28,"context_line":"infinite storage for image/snapshot storage and having a quota"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_4db51b7c","line":25,"in_reply_to":"3a50d1a3_24722b7d","updated":"2015-08-01 00:45:22.000000000","message":"Done","commit_id":"88a37ebcae3d716196c6549b5233d92d0a327614"},{"author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"change_message_id":"0bc4c894c91e56840139610d9a419d740a4a0dea","unresolved":false,"context_lines":[{"line_number":87,"context_line":"   the disk utility methods exposed are centered around"},{"line_number":88,"context_line":"   CONF.instance_path and the parameters reported by the resource"},{"line_number":89,"context_line":"   tracker are more centered around the CONF.instance_path. But the"},{"line_number":90,"context_line":"   CONF.snapshots_directory need not be on the same paitition,"},{"line_number":91,"context_line":"   under same mount point or same disk."},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_6425935a","line":90,"updated":"2015-07-31 23:01:13.000000000","message":"paitition -\u003e \u0027partition\u0027","commit_id":"88a37ebcae3d716196c6549b5233d92d0a327614"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"a561076ae8beaed584b34bda3f1f5f29376199b0","unresolved":false,"context_lines":[{"line_number":87,"context_line":"   the disk utility methods exposed are centered around"},{"line_number":88,"context_line":"   CONF.instance_path and the parameters reported by the resource"},{"line_number":89,"context_line":"   tracker are more centered around the CONF.instance_path. But the"},{"line_number":90,"context_line":"   CONF.snapshots_directory need not be on the same paitition,"},{"line_number":91,"context_line":"   under same mount point or same disk."},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_6db81776","line":90,"in_reply_to":"3a50d1a3_6425935a","updated":"2015-08-01 00:45:22.000000000","message":"Done","commit_id":"88a37ebcae3d716196c6549b5233d92d0a327614"},{"author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"change_message_id":"0bc4c894c91e56840139610d9a419d740a4a0dea","unresolved":false,"context_lines":[{"line_number":97,"context_line":"usage and availability details, which eventually gets populated"},{"line_number":98,"context_line":"to the Nova database via the Nova Conductor layer. But the"},{"line_number":99,"context_line":"calculations of Resource Tracker revolve around CONF.instance_path"},{"line_number":100,"context_line":"and those values can;t be relied upon if CONF.instance_path and"},{"line_number":101,"context_line":"CONF.snapshots_directory are not under same mount point, different"},{"line_number":102,"context_line":"disks, different parititions etc."},{"line_number":103,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_a4d2bb32","line":100,"updated":"2015-07-31 23:01:13.000000000","message":"can;t -\u003e can\u0027t","commit_id":"88a37ebcae3d716196c6549b5233d92d0a327614"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"a561076ae8beaed584b34bda3f1f5f29376199b0","unresolved":false,"context_lines":[{"line_number":97,"context_line":"usage and availability details, which eventually gets populated"},{"line_number":98,"context_line":"to the Nova database via the Nova Conductor layer. But the"},{"line_number":99,"context_line":"calculations of Resource Tracker revolve around CONF.instance_path"},{"line_number":100,"context_line":"and those values can;t be relied upon if CONF.instance_path and"},{"line_number":101,"context_line":"CONF.snapshots_directory are not under same mount point, different"},{"line_number":102,"context_line":"disks, different parititions etc."},{"line_number":103,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_0db0738a","line":100,"in_reply_to":"3a50d1a3_a4d2bb32","updated":"2015-08-01 00:45:22.000000000","message":"Done","commit_id":"88a37ebcae3d716196c6549b5233d92d0a327614"},{"author":{"_account_id":3189,"name":"Chet Burgess","email":"cfb@metacloud.com","username":"cfb-n"},"change_message_id":"7a6759fc46b3fac76ec1470d94405ef87df2c82a","unresolved":false,"context_lines":[{"line_number":79,"context_line":"   more centered around the CONF.instance_path. But the"},{"line_number":80,"context_line":"   CONF.snapshots_directory need not be on the same partition, under same mount"},{"line_number":81,"context_line":"   point or same disk."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"Alternatives"},{"line_number":85,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa80f949_4a37a2b0","line":82,"updated":"2015-11-03 18:19:20.000000000","message":"I think a key component of this is how the required space is calculated. The total size a snapshot will take isn\u0027t always easy to determine. File formats like QCOW2 can allow the files to grow larger then the virtual size.\n\nHow do you plan on determine the \"required space\"? What happens when the actual required space is larger?\n\nI think the facility will have to be a best effort minimum. As in at best it can validate that the minimum amount of space is available, but I\u0027m not sure it can guarantee a maximum.","commit_id":"0b9223f054123a3b1ddae100921864d0ee326270"},{"author":{"_account_id":3189,"name":"Chet Burgess","email":"cfb@metacloud.com","username":"cfb-n"},"change_message_id":"7a6759fc46b3fac76ec1470d94405ef87df2c82a","unresolved":false,"context_lines":[{"line_number":90,"context_line":"around CONF.instance_path and those values can\u0027t be relied upon if"},{"line_number":91,"context_line":"CONF.instance_path and CONF.snapshots_directory are not under same mount point,"},{"line_number":92,"context_line":"different disks, different parititions etc."},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"Data model impact"},{"line_number":95,"context_line":"-----------------"},{"line_number":96,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"fa80f949_aa696e8d","line":93,"updated":"2015-11-03 18:19:20.000000000","message":"While it would require more work I really like the idea of incorporating this into resource tracker. Resource tracker is at least something that is more centralized that Operators can write tooling around monitoring.","commit_id":"0b9223f054123a3b1ddae100921864d0ee326270"}]}