)]}'
{"tempest/services/compute/xml/flavors_client.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"b680f9dab8f21a34544e430ef3afd3bdb7b4300d","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                flavor[\u0027links\u0027].append(v)"},{"line_number":47,"context_line":"                continue"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"            if k \u003d\u003d \u0027{%s}ephemeral\u0027 % XMLNS_OS_FLV_EXT_DATA:"},{"line_number":50,"context_line":"                k \u003d \u0027OS-FLV-EXT-DATA:ephemeral\u0027"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"            if k \u003d\u003d \u0027extra_specs\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2F%2BYk%3D","line":49,"updated":"2013-08-19 13:10:41.000000000","message":"If the parser is striping the namespace off, does that mean this line is wrong too?","commit_id":"ccfa65366a09fd273515ca0b1e4f55b452500dda"},{"author":{"_account_id":7532,"name":"Aditi Raveesh","email":"aditi.raveesh@gmail.com","username":"aditiraveesh"},"change_message_id":"dc316e7794a4bd02e52a3b88b66bdd22e830486c","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                flavor[\u0027links\u0027].append(v)"},{"line_number":47,"context_line":"                continue"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"            if k \u003d\u003d \u0027{%s}ephemeral\u0027 % XMLNS_OS_FLV_EXT_DATA:"},{"line_number":50,"context_line":"                k \u003d \u0027OS-FLV-EXT-DATA:ephemeral\u0027"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"            if k \u003d\u003d \u0027extra_specs\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2F7go%3D","line":49,"in_reply_to":"AAAARn%2F%2F%2BYk%3D","updated":"2013-08-20 04:04:58.000000000","message":"The parser strips the namespace off only from child element names, and not from attributes of a node in the xml.\n(The code is in the method xml_to_json)\nIn this case, ephemeral is an attribute of the flavor and not a child element in the xml, so the namespace is retained.","commit_id":"ccfa65366a09fd273515ca0b1e4f55b452500dda"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"b2231b60e02d1f945cda985410430a8f6a8a7788","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                flavor[\u0027links\u0027].append(v)"},{"line_number":47,"context_line":"                continue"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"            if k \u003d\u003d \u0027{%s}ephemeral\u0027 % XMLNS_OS_FLV_EXT_DATA:"},{"line_number":50,"context_line":"                k \u003d \u0027OS-FLV-EXT-DATA:ephemeral\u0027"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"            if k \u003d\u003d \u0027extra_specs\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2FxNQ%3D","line":49,"in_reply_to":"AAAARn%2F%2F64s%3D","updated":"2013-08-22 03:31:30.000000000","message":"ok. would you mind submitting a bug report for this so it doesn\u0027t get lost?","commit_id":"ccfa65366a09fd273515ca0b1e4f55b452500dda"},{"author":{"_account_id":7532,"name":"Aditi Raveesh","email":"aditi.raveesh@gmail.com","username":"aditiraveesh"},"change_message_id":"ed9bbab0b831dc489ef15f5596452b19b80ca0e7","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                flavor[\u0027links\u0027].append(v)"},{"line_number":47,"context_line":"                continue"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"            if k \u003d\u003d \u0027{%s}ephemeral\u0027 % XMLNS_OS_FLV_EXT_DATA:"},{"line_number":50,"context_line":"                k \u003d \u0027OS-FLV-EXT-DATA:ephemeral\u0027"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"            if k \u003d\u003d \u0027extra_specs\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2F64s%3D","line":49,"in_reply_to":"AAAARn%2F%2F66k%3D","updated":"2013-08-20 08:18:48.000000000","message":"We could fix it, but it would be a rather large change, since a lot of tests are dependent on that method. It would be better to do it as a separate commit.\nAlso, we need to check why it was done that way in the first place.","commit_id":"ccfa65366a09fd273515ca0b1e4f55b452500dda"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"a19cda9d2048eb55168cb34e65319263033fca14","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                flavor[\u0027links\u0027].append(v)"},{"line_number":47,"context_line":"                continue"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"            if k \u003d\u003d \u0027{%s}ephemeral\u0027 % XMLNS_OS_FLV_EXT_DATA:"},{"line_number":50,"context_line":"                k \u003d \u0027OS-FLV-EXT-DATA:ephemeral\u0027"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"            if k \u003d\u003d \u0027extra_specs\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2F66k%3D","line":49,"in_reply_to":"AAAARn%2F%2F7go%3D","updated":"2013-08-20 08:10:45.000000000","message":"is it a big job to fix xml_to_json so its consistent? I guess there may be some tests dependent on this behaviour. But this inconsistent stripping of namespaces seems pretty bad to me.","commit_id":"ccfa65366a09fd273515ca0b1e4f55b452500dda"},{"author":{"_account_id":7532,"name":"Aditi Raveesh","email":"aditi.raveesh@gmail.com","username":"aditiraveesh"},"change_message_id":"aa3a74255da5adc2362134c8196fdc57de003e6f","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                flavor[\u0027links\u0027].append(v)"},{"line_number":47,"context_line":"                continue"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"            if k \u003d\u003d \u0027{%s}ephemeral\u0027 % XMLNS_OS_FLV_EXT_DATA:"},{"line_number":50,"context_line":"                k \u003d \u0027OS-FLV-EXT-DATA:ephemeral\u0027"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"            if k \u003d\u003d \u0027extra_specs\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARn%2F%2FxEM%3D","line":49,"in_reply_to":"AAAARn%2F%2FxNQ%3D","updated":"2013-08-22 04:18:49.000000000","message":"It\u0027s submitted as bug 1215263","commit_id":"ccfa65366a09fd273515ca0b1e4f55b452500dda"}]}