)]}'
{"specs/kilo/non-glance-image-refs.rst":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"8b630ae4a6d358df8915ebcdcda9d27d2490391d","unresolved":false,"context_lines":[{"line_number":35,"context_line":"* If it starts with \u0027http://\u0027 then it will be downloaded by http image service."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"* If it starts with \u0027file://\u0027 then it is referencing local file system and"},{"line_number":38,"context_line":"  will be downloaded by corresponding image service."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Because of kernel and ramdisk uuids are currently got from image_source"},{"line_number":41,"context_line":"image properties returned by glance, so links for those should be put into"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_d38711d6","line":38,"updated":"2014-11-19 10:57:38.000000000","message":"This is a tricky one btw. On what file system it is, provided we have many conductors?","commit_id":"092c78dde09f6f0f334aa6cfdd8242f66158606e"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"74f33dad7147aa7d6cb752fc3c2a91b1dbb622a3","unresolved":false,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"* If it starts with \u0027file://\u0027 then it is referencing local file system and"},{"line_number":38,"context_line":"  will be downloaded by corresponding image service."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Because of kernel and ramdisk uuids are currently got from image_source"},{"line_number":41,"context_line":"image properties returned by glance, so links for those should be put into"},{"line_number":42,"context_line":"instance_info dictionary."}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_9ce93fb9","line":39,"updated":"2014-11-18 13:42:56.000000000","message":"Love the idea!\n\nI can see in the future we even extending it to tftp://, ssh:// etc... It reminds me the way git allow us to use different protocols to transfer data :)","commit_id":"092c78dde09f6f0f334aa6cfdd8242f66158606e"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"7a83489f97004bb5189c068ea63fa7b246adf904","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Proposed change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"The proposal is to create new base image service for downloading images"},{"line_number":27,"context_line":"provided URL for them; also to add some common protocol support, i.e."},{"line_number":28,"context_line":"downloading from remote HTTP servers and using images available in local file"},{"line_number":29,"context_line":"system."}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a890539_1f85ae06","line":26,"updated":"2014-11-21 23:05:59.000000000","message":"\"new base image service\" -- I think you mean \"new image service base class\"","commit_id":"bd16cfb966e548eb6a699bb4371436dde0660612"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"7a83489f97004bb5189c068ea63fa7b246adf904","unresolved":false,"context_lines":[{"line_number":37,"context_line":"  HTTP image service."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"* If it starts with \u0027file://\u0027 then it is referencing local file system and"},{"line_number":40,"context_line":"  will be copied by image service for current conductor\u0027s local files."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Alternatives"},{"line_number":43,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a890539_3f7d0a1d","line":40,"updated":"2014-11-21 23:05:59.000000000","message":"I\u0027m not sure this makes sense -- if it\u0027s a local file, why / where from / where to is it copied?","commit_id":"bd16cfb966e548eb6a699bb4371436dde0660612"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"7a83489f97004bb5189c068ea63fa7b246adf904","unresolved":false,"context_lines":[{"line_number":42,"context_line":"Alternatives"},{"line_number":43,"context_line":"------------"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"None"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Data model impact"},{"line_number":48,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a890539_5f7ad624","line":45,"updated":"2014-11-21 23:05:59.000000000","message":"Continue having a hard-dependency on Glance.","commit_id":"bd16cfb966e548eb6a699bb4371436dde0660612"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"7a83489f97004bb5189c068ea63fa7b246adf904","unresolved":false,"context_lines":[{"line_number":96,"context_line":"Because of kernel and ramdisk uuids are currently got from image_source"},{"line_number":97,"context_line":"image properties returned by Glance, links for those should be put into"},{"line_number":98,"context_line":"instance_info dictionary if Glance is not used."},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"Developer impact"},{"line_number":101,"context_line":"----------------"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a890539_7f73124e","line":99,"updated":"2014-11-21 23:05:59.000000000","message":"As in my other comment, how do you propose we handle other image metadata?","commit_id":"bd16cfb966e548eb6a699bb4371436dde0660612"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"58c0a25c4548e7112dccd6c95f6703b9e88c5d87","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"Because of kernel and ramdisk UUIDs are currently got from image_source"},{"line_number":99,"context_line":"image properties returned by Glance, links for those should be put into"},{"line_number":100,"context_line":"instance_info dictionary if Glance is not used."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Developer impact"},{"line_number":103,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_19668abb","line":100,"updated":"2014-11-24 15:19:16.000000000","message":"I think it would be good to note any other instance_info changes that we know may need to be manually enterred, and add a note that deployers and other developers need to keep this in mind going forward.\n\nEg, something like this:\n\n  Since Ironic may be used without Glance, developers can\u0027t make the assumption that\n  Glance image metadata is the only source for such information. Deployers must be\n  capable of supplying ironic with all required metadata programatically, and such\n  requirements must be documented.\n\n  For instance, an Image may be assumed to be a complete disk image (with local bootloader)\n  if node.instance_info contains an \"image_source\" reference without both the \"kernel\"\n  and \"ramdisk\" references.","commit_id":"09c0edb2f91258dc213028fcf577a64ed05e3d17"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"e5c2a0c8dc41420e3cf0faaedff1310e802b5563","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"Because of kernel and ramdisk UUIDs are currently got from image_source"},{"line_number":99,"context_line":"image properties returned by Glance, links for those should be put into"},{"line_number":100,"context_line":"instance_info dictionary if Glance is not used."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Developer impact"},{"line_number":103,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_e633b65f","line":100,"in_reply_to":"5a890539_19668abb","updated":"2014-11-25 14:07:03.000000000","message":"Thanks for your suggestion, done.","commit_id":"09c0edb2f91258dc213028fcf577a64ed05e3d17"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"58c0a25c4548e7112dccd6c95f6703b9e88c5d87","unresolved":false,"context_lines":[{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Possibility of specifying URLs for images and protocols supported should be"},{"line_number":157,"context_line":"added to documentation."},{"line_number":158,"context_line":""},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"References"},{"line_number":161,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_3906e6ce","line":158,"updated":"2014-11-24 15:19:16.000000000","message":"Add\u0027l doc impact mentioned above -- operators who choose not to run Glance\nmay need to supply instance image metadata manually, if using drivers that require such.","commit_id":"09c0edb2f91258dc213028fcf577a64ed05e3d17"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"e5c2a0c8dc41420e3cf0faaedff1310e802b5563","unresolved":false,"context_lines":[{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Possibility of specifying URLs for images and protocols supported should be"},{"line_number":157,"context_line":"added to documentation."},{"line_number":158,"context_line":""},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"References"},{"line_number":161,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_26413ea9","line":158,"in_reply_to":"5a890539_3906e6ce","updated":"2014-11-25 14:07:03.000000000","message":"Done","commit_id":"09c0edb2f91258dc213028fcf577a64ed05e3d17"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"3d45161fd1c6e2a7bd6839e3ee75cc5f052e81f4","unresolved":false,"context_lines":[{"line_number":90,"context_line":"Performance Impact"},{"line_number":91,"context_line":"------------------"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"None"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Other deployer impact"},{"line_number":96,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_d5f53e85","line":93,"updated":"2014-11-27 08:45:47.000000000","message":"nit: Getting it from the local fs vs downloading it will affect performance","commit_id":"e5efa77b4ccdb92b4307d66a821b07b89c3749e2"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"3d45161fd1c6e2a7bd6839e3ee75cc5f052e81f4","unresolved":false,"context_lines":[{"line_number":134,"context_line":"  local file system."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"* Implement a class that will return image service based on protocol defined"},{"line_number":137,"context_line":"  in URL."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5a890539_3544c2b6","line":137,"updated":"2014-11-27 08:45:47.000000000","message":"factory class","commit_id":"e5efa77b4ccdb92b4307d66a821b07b89c3749e2"}]}