)]}'
{"README.md":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f57195002eef0c997665133316654f7de254faaf","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Requirements"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Chef 12 or higher required."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Cookbooks"},{"line_number":14,"context_line":"---------"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1af94dfe_957e2617","line":11,"updated":"2016-03-18 08:12:05.000000000","message":"Do we really need Chef 12 now? Even if we do, I think this should be a separate patch and be changed consistently in all cookbooks.","commit_id":"f4a86b73ec47cade38a63ce8afb6caccac79a836"},{"author":{"_account_id":14790,"name":"Samuel Cassiba","email":"samuel@cassi.ba","username":"scassiba"},"change_message_id":"e56e137cdae1f0fa21ff670b858cd9d9578e6321","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Requirements"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Chef 12 or higher required."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Cookbooks"},{"line_number":14,"context_line":"---------"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1af94dfe_e20a3b25","line":11,"in_reply_to":"1af94dfe_957e2617","updated":"2016-03-18 15:03:42.000000000","message":"Newer ChefDKs ship with Chef 12 out of the box. While it can likely work with Chef 11, not all cookbooks are guaranteed to do so.","commit_id":"f4a86b73ec47cade38a63ce8afb6caccac79a836"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f57195002eef0c997665133316654f7de254faaf","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"api-ec2"},{"line_number":29,"context_line":"----"},{"line_number":30,"context_line":"- This is obsoleted by https://github.com/cloudscaling/cookbook-openstack-ec2"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"api-metadata"},{"line_number":33,"context_line":"----"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1af94dfe_95d3c603","line":30,"updated":"2016-03-18 08:12:05.000000000","message":"Does this also change from running the deprecated nova-api-ec2 (which is already missing in Ubuntu Xenial) to the new ec2-api project? Then this comment could be a bit more explicit.","commit_id":"f4a86b73ec47cade38a63ce8afb6caccac79a836"},{"author":{"_account_id":14790,"name":"Samuel Cassiba","email":"samuel@cassi.ba","username":"scassiba"},"change_message_id":"e56e137cdae1f0fa21ff670b858cd9d9578e6321","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"api-ec2"},{"line_number":29,"context_line":"----"},{"line_number":30,"context_line":"- This is obsoleted by https://github.com/cloudscaling/cookbook-openstack-ec2"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"api-metadata"},{"line_number":33,"context_line":"----"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1af94dfe_8561c1ab","line":30,"in_reply_to":"1af94dfe_95d3c603","updated":"2016-03-18 15:03:42.000000000","message":"In previous meetings, we discussed the possibility of including the ec2 cookbook in the main OpenStack cookbooks, with support from Cloudscaling.\n\nJan and I also discussed removing the api-ec2 parts for Mitaka in favor of supporting the ec2 cookbook.","commit_id":"f4a86b73ec47cade38a63ce8afb6caccac79a836"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f57195002eef0c997665133316654f7de254faaf","unresolved":false,"context_lines":[{"line_number":334,"context_line":"API Configuration Attributes"},{"line_number":335,"context_line":"----------------------------"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"* `openstack[\"compute\"][\"enabled_apis\"]` - Which apis have been enabled in nova compute, only for osapi_compute. For metadata, include the api-metadata recipe."},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"Notification Attributes"},{"line_number":340,"context_line":"-----------------------"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1af94dfe_3538f23c","line":337,"updated":"2016-03-18 08:12:05.000000000","message":"Do we need to keep this attribute at all? Is there any reason to run nova without osapi_compute enabled? Or can we just drop this option?","commit_id":"f4a86b73ec47cade38a63ce8afb6caccac79a836"},{"author":{"_account_id":14790,"name":"Samuel Cassiba","email":"samuel@cassi.ba","username":"scassiba"},"change_message_id":"e56e137cdae1f0fa21ff670b858cd9d9578e6321","unresolved":false,"context_lines":[{"line_number":334,"context_line":"API Configuration Attributes"},{"line_number":335,"context_line":"----------------------------"},{"line_number":336,"context_line":""},{"line_number":337,"context_line":"* `openstack[\"compute\"][\"enabled_apis\"]` - Which apis have been enabled in nova compute, only for osapi_compute. For metadata, include the api-metadata recipe."},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"Notification Attributes"},{"line_number":340,"context_line":"-----------------------"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1af94dfe_25c38d58","line":337,"in_reply_to":"1af94dfe_3538f23c","updated":"2016-03-18 15:03:42.000000000","message":"It may be worthwhile to keep around for in the future if a new api is added to compute.","commit_id":"f4a86b73ec47cade38a63ce8afb6caccac79a836"}],"spec/identity_registration_spec.rb":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f57195002eef0c997665133316654f7de254faaf","unresolved":false,"context_lines":[{"line_number":153,"context_line":"      end"},{"line_number":154,"context_line":"    end"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    describe \"when \u0027ec2\u0027 is not in the list of enabled_apis\" do"},{"line_number":157,"context_line":"      before do"},{"line_number":158,"context_line":"        node.set[\u0027openstack\u0027][\u0027compute\u0027][\u0027conf\u0027][\u0027DEFAULT\u0027][\u0027enabled_apis\u0027] \u003d \u0027osapi_compute\u0027"},{"line_number":159,"context_line":"      end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"1af94dfe_9504664a","side":"PARENT","line":156,"updated":"2016-03-18 08:12:05.000000000","message":"Looks like this block can also be deleted","commit_id":"9b46ac6d13961b4c9191f8232080ebe0f42e157f"},{"author":{"_account_id":14790,"name":"Samuel Cassiba","email":"samuel@cassi.ba","username":"scassiba"},"change_message_id":"e56e137cdae1f0fa21ff670b858cd9d9578e6321","unresolved":false,"context_lines":[{"line_number":153,"context_line":"      end"},{"line_number":154,"context_line":"    end"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    describe \"when \u0027ec2\u0027 is not in the list of enabled_apis\" do"},{"line_number":157,"context_line":"      before do"},{"line_number":158,"context_line":"        node.set[\u0027openstack\u0027][\u0027compute\u0027][\u0027conf\u0027][\u0027DEFAULT\u0027][\u0027enabled_apis\u0027] \u003d \u0027osapi_compute\u0027"},{"line_number":159,"context_line":"      end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"1af94dfe_e5cfe553","side":"PARENT","line":156,"in_reply_to":"1af94dfe_9504664a","updated":"2016-03-18 15:03:42.000000000","message":"I wanted to keep the negative tests in place. This could be reworked with the rest of the ec2 things removed.","commit_id":"9b46ac6d13961b4c9191f8232080ebe0f42e157f"}]}