)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"c6523732426a5d63489cfabfc0f8b2996232ae4d","unresolved":false,"context_lines":[{"line_number":14,"context_line":"We rename it to \u0027NoBacking\u0027, which describes it accurately. We also add"},{"line_number":15,"context_line":"doctext describing what it does."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"DocImpact"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"The config option libvirt.images_type gets an additional value:"},{"line_number":20,"context_line":"\u0027nobacking\u0027. The effect of this is identical to setting the value"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"5a18252c_e5313dee","line":17,"updated":"2016-04-12 15:31:03.000000000","message":"I was under impression we no longer use DocImpact, but rather simply add reno notes to commits instead.","commit_id":"d93a69ad58ddad492f19bcc400b8f055b81ba149"}],"nova/conf/libvirt.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"b5b3c3fd38700a04c0dc5f5941d5f633b2d34503","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    cfg.StrOpt(\u0027images_type\u0027,"},{"line_number":222,"context_line":"               default\u003d\u0027default\u0027,"},{"line_number":223,"context_line":"               choices\u003d(\u0027raw\u0027, \u0027flat\u0027, \u0027qcow2\u0027, \u0027lvm\u0027, \u0027rbd\u0027, \u0027ploop\u0027,"},{"line_number":224,"context_line":"                        \u0027default\u0027),"},{"line_number":225,"context_line":"               help\u003d\u0027VM Images format. If default is specified, then\u0027"},{"line_number":226,"context_line":"                    \u0027 use_cow_images flag is used instead of this one.\u0027),"},{"line_number":227,"context_line":"    cfg.StrOpt(\u0027images_volume_group\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"dab17558_3842910f","line":224,"updated":"2016-05-17 14:04:05.000000000","message":"Do you want to indicate in the help text that raw is deprecated in favor of flat?","commit_id":"cff4d78a4a7ddbb18cb875c66b3c56f04a6caea3"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"231db081f25eed082d2435c20f5701282a6d8f96","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    cfg.StrOpt(\u0027images_type\u0027,"},{"line_number":222,"context_line":"               default\u003d\u0027default\u0027,"},{"line_number":223,"context_line":"               choices\u003d(\u0027raw\u0027, \u0027flat\u0027, \u0027qcow2\u0027, \u0027lvm\u0027, \u0027rbd\u0027, \u0027ploop\u0027,"},{"line_number":224,"context_line":"                        \u0027default\u0027),"},{"line_number":225,"context_line":"               help\u003d\u0027VM Images format. If default is specified, then\u0027"},{"line_number":226,"context_line":"                    \u0027 use_cow_images flag is used instead of this one.\u0027),"},{"line_number":227,"context_line":"    cfg.StrOpt(\u0027images_volume_group\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"bab6814e_c8999e70","line":224,"in_reply_to":"bab6814e_26ba2e24","updated":"2016-05-18 20:11:21.000000000","message":"In triaging the docs bug that this patch generated, the closest I can find to something outside the config sample reference was this:\n\nhttp://docs.openstack.org/mitaka/config-reference/compute/hypervisor-kvm.html#configure-compute-backing-storage\n\nWhich only mentions raw, qcow2 and lvm, it doesn\u0027t even mention rbd or ploop.","commit_id":"cff4d78a4a7ddbb18cb875c66b3c56f04a6caea3"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"6ee1de0c5c0128097a538abcccf28de43bf49057","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    cfg.StrOpt(\u0027images_type\u0027,"},{"line_number":222,"context_line":"               default\u003d\u0027default\u0027,"},{"line_number":223,"context_line":"               choices\u003d(\u0027raw\u0027, \u0027flat\u0027, \u0027qcow2\u0027, \u0027lvm\u0027, \u0027rbd\u0027, \u0027ploop\u0027,"},{"line_number":224,"context_line":"                        \u0027default\u0027),"},{"line_number":225,"context_line":"               help\u003d\u0027VM Images format. If default is specified, then\u0027"},{"line_number":226,"context_line":"                    \u0027 use_cow_images flag is used instead of this one.\u0027),"},{"line_number":227,"context_line":"    cfg.StrOpt(\u0027images_volume_group\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"bab6814e_26ba2e24","line":224,"in_reply_to":"bab6814e_4bf676d8","updated":"2016-05-18 14:35:34.000000000","message":"Yeah, but most of our newer augmented config help text blurbs have been including the various options and what they mean in much more detail. So, it\u0027s appropriate to add here.","commit_id":"cff4d78a4a7ddbb18cb875c66b3c56f04a6caea3"},{"author":{"_account_id":9555,"name":"Matthew Booth","email":"mbooth@redhat.com","username":"MatthewBooth"},"change_message_id":"72598ab6c1a35f98cd8bf7226d4d2b4e73e30752","unresolved":false,"context_lines":[{"line_number":221,"context_line":"    cfg.StrOpt(\u0027images_type\u0027,"},{"line_number":222,"context_line":"               default\u003d\u0027default\u0027,"},{"line_number":223,"context_line":"               choices\u003d(\u0027raw\u0027, \u0027flat\u0027, \u0027qcow2\u0027, \u0027lvm\u0027, \u0027rbd\u0027, \u0027ploop\u0027,"},{"line_number":224,"context_line":"                        \u0027default\u0027),"},{"line_number":225,"context_line":"               help\u003d\u0027VM Images format. If default is specified, then\u0027"},{"line_number":226,"context_line":"                    \u0027 use_cow_images flag is used instead of this one.\u0027),"},{"line_number":227,"context_line":"    cfg.StrOpt(\u0027images_volume_group\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"bab6814e_4bf676d8","line":224,"in_reply_to":"dab17558_3842910f","updated":"2016-05-18 08:29:57.000000000","message":"There\u0027s probably a doc somewhere that needs to be updated, but I\u0027m not sure where. This help text didn\u0027t mention raw anyway, so I haven\u0027t touched it.\n\nI\u0027ll follow this up.","commit_id":"cff4d78a4a7ddbb18cb875c66b3c56f04a6caea3"}],"nova/virt/libvirt/imagebackend.py":[{"author":{"_account_id":20223,"name":"zhaolihui","email":"zhaolh@awcloud.com","username":"zhaolihui"},"change_message_id":"c44e1d4ec4a4c4d359f9b99f42fe5bb7820560f4","unresolved":false,"context_lines":[{"line_number":526,"context_line":"                         \u0027error: %(error)s\u0027),"},{"line_number":527,"context_line":"                      {\u0027path\u0027: self.path,"},{"line_number":528,"context_line":"                       \u0027error\u0027: e})"},{"line_number":529,"context_line":"            return \u0027raw\u0027"},{"line_number":530,"context_line":""},{"line_number":531,"context_line":"    def _supports_encryption(self):"},{"line_number":532,"context_line":"        # NOTE(dgenin): Kernel, ramdisk and disk.config are fetched using"}],"source_content_type":"text/x-python","patch_set":3,"id":"bae84128_8e28ea87","line":529,"range":{"start_line":529,"start_character":20,"end_line":529,"end_character":23},"updated":"2016-02-23 10:38:39.000000000","message":"If this \u0027raw\u0027 should be \u0027Unbacked\u0027? But the key point isn\u0027t here, the point is that although you rename raw to unbacked the function is the same","commit_id":"1f8a60096202973edbc27a2c8bcc191d65e60a37"},{"author":{"_account_id":9555,"name":"Matthew Booth","email":"mbooth@redhat.com","username":"MatthewBooth"},"change_message_id":"c6c4848d3d12976f52120953671b0ee7f505d8fb","unresolved":false,"context_lines":[{"line_number":526,"context_line":"                         \u0027error: %(error)s\u0027),"},{"line_number":527,"context_line":"                      {\u0027path\u0027: self.path,"},{"line_number":528,"context_line":"                       \u0027error\u0027: e})"},{"line_number":529,"context_line":"            return \u0027raw\u0027"},{"line_number":530,"context_line":""},{"line_number":531,"context_line":"    def _supports_encryption(self):"},{"line_number":532,"context_line":"        # NOTE(dgenin): Kernel, ramdisk and disk.config are fetched using"}],"source_content_type":"text/x-python","patch_set":3,"id":"bae84128_49b9b214","line":529,"range":{"start_line":529,"start_character":20,"end_line":529,"end_character":23},"in_reply_to":"bae84128_8e28ea87","updated":"2016-02-23 14:05:08.000000000","message":"Zhaolihui, I referred to this specifically in the commit message. Please read the commit message.","commit_id":"1f8a60096202973edbc27a2c8bcc191d65e60a37"}]}