)]}'
{"heat/doc/resources.py":[{"author":{"_account_id":7193,"name":"Thomas Spatzier","email":"thomas.spatzier@icloud.com","username":"thomasspatzier"},"change_message_id":"8cce061185796a9998cb1e0633ec81669bb55707","unresolved":false,"context_lines":[{"line_number":171,"context_line":"            definition.append(para)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        if prop.update_allowed:"},{"line_number":174,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Can be updated without replacement\u0027))"},{"line_number":175,"context_line":"            definition.append(para)"},{"line_number":176,"context_line":"        else:"},{"line_number":177,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Updates cause replacement\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F6%2F0%3D","line":174,"updated":"2013-10-25 07:11:21.000000000","message":"Very minor thing: the other descriptions are punctuated, so for consistency the same should be done here.","commit_id":"c3817184fe6f54ead5f62d739f6ed0a17f88026c"},{"author":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"change_message_id":"7c2df001a91fe35f9bd242d13162e2c098c874aa","unresolved":false,"context_lines":[{"line_number":171,"context_line":"            definition.append(para)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        if prop.update_allowed:"},{"line_number":174,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Can be updated without replacement\u0027))"},{"line_number":175,"context_line":"            definition.append(para)"},{"line_number":176,"context_line":"        else:"},{"line_number":177,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Updates cause replacement\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F3uU%3D","line":174,"in_reply_to":"AAAATn%2F%2F30M%3D","updated":"2013-10-25 21:37:18.000000000","message":"Clint, I think that is going to get messy. I don\u0027t think this is a problem really. If you look at the generated output (pasted below) they appear together and make sense when reading. Note it does say \"Can be ...\", and right below is the policy. Is that not clear enough?\n\n\nflavor : String\nThe ID or name of the flavor to boot onto.\n\nCan be updated without replacement\n\nRequired property.\n\nflavor_update_policy : String\nPolicy on how to apply a flavor update; either by requesting a server resize or by replacing the entire server.\n\nCan be updated without replacement\n\nOptional property, defaults to “RESIZE”.\n\nAllowed values: RESIZE, REPLACE","commit_id":"c3817184fe6f54ead5f62d739f6ed0a17f88026c"},{"author":{"_account_id":4715,"name":"Angus Salkeld","email":"asalkeld@redhat.com","username":"asalkeld"},"change_message_id":"ceaf72c3e8ea27f9560992fa45868519977fb194","unresolved":false,"context_lines":[{"line_number":171,"context_line":"            definition.append(para)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        if prop.update_allowed:"},{"line_number":174,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Can be updated without replacement\u0027))"},{"line_number":175,"context_line":"            definition.append(para)"},{"line_number":176,"context_line":"        else:"},{"line_number":177,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Updates cause replacement\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F6nU%3D","line":174,"in_reply_to":"AAAATn%2F%2F6%2BU%3D","updated":"2013-10-25 09:31:37.000000000","message":"Err, so what do you want me to do? This sounds like a corner case. All I am doing\nis generically printing out \"does/does not/ need replacing based on a boolean. ... confused.","commit_id":"c3817184fe6f54ead5f62d739f6ed0a17f88026c"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"69a7d0da12f0c2ade0a986fd66d7c27553cb4a22","unresolved":false,"context_lines":[{"line_number":171,"context_line":"            definition.append(para)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        if prop.update_allowed:"},{"line_number":174,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Can be updated without replacement\u0027))"},{"line_number":175,"context_line":"            definition.append(para)"},{"line_number":176,"context_line":"        else:"},{"line_number":177,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Updates cause replacement\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F6%2BU%3D","line":174,"in_reply_to":"AAAATn%2F%2F6%2F0%3D","updated":"2013-10-25 07:16:58.000000000","message":"We may need to add notes for those rare cases where we do update_allowed but still might end up triggering a replacement. Not sure where those are, but I feel like I\u0027ve seen that before.","commit_id":"c3817184fe6f54ead5f62d739f6ed0a17f88026c"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"480325514cddb527be36ab15604cdc9a7a64b545","unresolved":false,"context_lines":[{"line_number":171,"context_line":"            definition.append(para)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"        if prop.update_allowed:"},{"line_number":174,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Can be updated without replacement\u0027))"},{"line_number":175,"context_line":"            definition.append(para)"},{"line_number":176,"context_line":"        else:"},{"line_number":177,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Updates cause replacement\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F30M%3D","line":174,"in_reply_to":"AAAATn%2F%2F6nU%3D","updated":"2013-10-25 20:57:27.000000000","message":"The corner case is in OS::Nova::Server, image and flavor have policies which control whether changing them will cause a replacement. So we need a way to declare in the schema that it _may_ be replaced. The boolean UpdateAllowed, unfortunately, won\u0027t allow this. What about a simple list of \"Replacement optional\" properties that can be added to OS::Nova::Server (and anywhere else we find that does this). It would just control documentation.","commit_id":"c3817184fe6f54ead5f62d739f6ed0a17f88026c"},{"author":{"_account_id":7193,"name":"Thomas Spatzier","email":"thomas.spatzier@icloud.com","username":"thomasspatzier"},"change_message_id":"8cce061185796a9998cb1e0633ec81669bb55707","unresolved":false,"context_lines":[{"line_number":174,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Can be updated without replacement\u0027))"},{"line_number":175,"context_line":"            definition.append(para)"},{"line_number":176,"context_line":"        else:"},{"line_number":177,"context_line":"            para \u003d nodes.paragraph(\u0027\u0027, _(\u0027Updates cause replacement\u0027))"},{"line_number":178,"context_line":"            definition.append(para)"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"        if prop.required:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F6%2Fs%3D","line":177,"updated":"2013-10-25 07:11:21.000000000","message":"same as above","commit_id":"c3817184fe6f54ead5f62d739f6ed0a17f88026c"}]}