)]}'
{"nova/conf/glance.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"5c0c62f1a1f55246e4622472e0c87125462b0ec5","unresolved":true,"context_lines":[{"line_number":130,"context_line":"        help\u003d\"\"\""},{"line_number":131,"context_line":"Enable Glance image downloads directly via RBD."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Allow compute hosts to download and cache images locally directly from Ceph"},{"line_number":134,"context_line":"rather than downloading via the Glance API."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":".. note:: This option should only be enabled when the compute itself is not"}],"source_content_type":"text/x-python","patch_set":1,"id":"767195e7_16570433","line":133,"range":{"start_line":133,"start_character":6,"end_line":133,"end_character":13},"updated":"2021-04-09 14:47:26.000000000","message":"I guess this is covered by the note below, but you could strengthen this with something like:\n\n\"non-rbd computes\" or \"computes with only local storage\"\n\nAlso \"locally directly\" doesn\u0027t sound very good to my eye ears. What about s/locally/to local storage/ ? That might strengthen the \"this is not a normal ceph arrangement\" statement instead of the above compute comment.","commit_id":"530b6cc6b11482e1741282c62a99605c73333365"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"66458b66a38cefb7b13d59ba5e37ec8328ade18c","unresolved":true,"context_lines":[{"line_number":136,"context_line":".. note:: This option should only be enabled when the compute itself is not"},{"line_number":137,"context_line":"          also using Ceph as a backing store. For example with the libvirt"},{"line_number":138,"context_line":"          driver it should only be enabled when"},{"line_number":139,"context_line":"          :oslo.config:option:`libvirt.images_types` is not set to ``rbd``."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"Related options:"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"31d66dc8_808cec30","line":139,"range":{"start_line":139,"start_character":39,"end_line":139,"end_character":51},"updated":"2021-04-16 11:37:47.000000000","message":"This should be \u0027images_type\u0027. That\u0027s the cause of the build failure","commit_id":"530b6cc6b11482e1741282c62a99605c73333365"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"5b93a94e446cb27c179d94c178f9a121c24a3a53","unresolved":false,"context_lines":[{"line_number":136,"context_line":".. note:: This option should only be enabled when the compute itself is not"},{"line_number":137,"context_line":"          also using Ceph as a backing store. For example with the libvirt"},{"line_number":138,"context_line":"          driver it should only be enabled when"},{"line_number":139,"context_line":"          :oslo.config:option:`libvirt.images_types` is not set to ``rbd``."},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"Related options:"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"6fac33b7_3318975e","line":139,"range":{"start_line":139,"start_character":39,"end_line":139,"end_character":51},"in_reply_to":"31d66dc8_808cec30","updated":"2021-04-16 11:59:44.000000000","message":"💩","commit_id":"530b6cc6b11482e1741282c62a99605c73333365"}]}
