)]}'
{"octavia/api/v1/types/l7policy.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"ac3067b6ad5c47638e4072c983fcc9117bdc2046","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    description \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":28,"context_line":"    enabled \u003d wtypes.wsattr(bool)"},{"line_number":29,"context_line":"    action \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":30,"context_line":"    redirect_pool_id \u003d wtypes.wsattr(wtypes.UuidType())"},{"line_number":31,"context_line":"    redirect_url \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":32,"context_line":"    position \u003d wtypes.wsattr(wtypes.IntegerType())"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_b4bdcd04","line":30,"updated":"2016-03-04 21:41:55.000000000","message":"I remember there was a reason we were staying away from UuidType for some things? Something about matching Neutron-LBaaS\u0027 validations which are just String based? Do you know if that applies here?","commit_id":"00216535b24ccee25e643c367471ee4ac9134332"},{"author":{"_account_id":6951,"name":"Brandon Logan","email":"brandon.logan@rackspace.com","username":"brandon-logan"},"change_message_id":"8b710f64a59b98386c5077d144fa3fb0a21c69c5","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    description \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":28,"context_line":"    enabled \u003d wtypes.wsattr(bool)"},{"line_number":29,"context_line":"    action \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":30,"context_line":"    redirect_pool_id \u003d wtypes.wsattr(wtypes.UuidType())"},{"line_number":31,"context_line":"    redirect_url \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":32,"context_line":"    position \u003d wtypes.wsattr(wtypes.IntegerType())"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_6f408e36","line":30,"in_reply_to":"7af24918_34dc1d46","updated":"2016-03-04 22:07:45.000000000","message":"the restriction adam is talking about was only for project_id/tenant_id.  neutron (and as such neutron-lbaas) does not validate that a tenant_id is a uuid.","commit_id":"00216535b24ccee25e643c367471ee4ac9134332"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"b01c2110e555f310da84d9bf1911401ddc14248e","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    description \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":28,"context_line":"    enabled \u003d wtypes.wsattr(bool)"},{"line_number":29,"context_line":"    action \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":30,"context_line":"    redirect_pool_id \u003d wtypes.wsattr(wtypes.UuidType())"},{"line_number":31,"context_line":"    redirect_url \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":32,"context_line":"    position \u003d wtypes.wsattr(wtypes.IntegerType())"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_ef993ed2","line":30,"in_reply_to":"7af24918_6f408e36","updated":"2016-03-04 22:08:42.000000000","message":"Ok, then good to go. Thanks, just wanted to verify.","commit_id":"00216535b24ccee25e643c367471ee4ac9134332"},{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"e469f44172d1d2615ad768c046ce11fea1f5443d","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    description \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":28,"context_line":"    enabled \u003d wtypes.wsattr(bool)"},{"line_number":29,"context_line":"    action \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":30,"context_line":"    redirect_pool_id \u003d wtypes.wsattr(wtypes.UuidType())"},{"line_number":31,"context_line":"    redirect_url \u003d wtypes.wsattr(wtypes.StringType())"},{"line_number":32,"context_line":"    position \u003d wtypes.wsattr(wtypes.IntegerType())"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7af24918_34dc1d46","line":30,"in_reply_to":"7af24918_b4bdcd04","updated":"2016-03-04 21:46:52.000000000","message":"I don\u0027t think so. Or rather, I don\u0027t think we need to worry about this. Neutron-LBaaS talks to us using a driver we wrote and will only refer to the redirect_pool_id using a UUID. Plus, the APIs are already different enough (ie. we\u0027re nested, N-LBaaS is not, and there are other subtle differences) that we probably don\u0027t need to worry about this. I don\u0027t see us needing to accept anything other than a UUID or None value (which works) in this field.","commit_id":"00216535b24ccee25e643c367471ee4ac9134332"}]}
