)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"c8fd6405f0aa1aa93913ff8a98b4d3f2b7e85739","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Merge trove state retrieve fields into one place"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"State description and state are always with same value, see"},{"line_number":10,"context_line":"https://github.com/openstack/trove/blob/master/trove/extensions/mgmt/instances/models.py#L217-L218"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Id86f513c329946f3816a852e478f57a5dec21779"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9f91af0f_c605c112","line":10,"updated":"2018-01-09 14:31:23.000000000","message":"lol. why did they add 2 fields for the same value? it\u0027s not like one is deprecated either.","commit_id":"9efed3124d5eca6b26f0036c1254b6f8831a4a1d"}],"ceilometer/pipeline/data/event_definitions.yaml":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"c8fd6405f0aa1aa93913ff8a98b4d3f2b7e85739","unresolved":false,"context_lines":[{"line_number":510,"context_line":"- event_type: trove.*"},{"line_number":511,"context_line":"  traits: \u0026trove_base_traits"},{"line_number":512,"context_line":"    state:"},{"line_number":513,"context_line":"      fields: [\u0027payload.state_description\u0027, \u0027payload.state\u0027]"},{"line_number":514,"context_line":"    instance_type:"},{"line_number":515,"context_line":"      fields: payload.instance_type"},{"line_number":516,"context_line":"    user_id:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9f91af0f_e619e579","line":513,"updated":"2018-01-09 14:31:23.000000000","message":"i would just remove state_description. they are same value and they will both always be there.","commit_id":"9efed3124d5eca6b26f0036c1254b6f8831a4a1d"},{"author":{"_account_id":22752,"name":"Hanxi Liu","email":"apolloliuhx@gmail.com","username":"Hanxi_Liu"},"change_message_id":"e42f43edcc68c0c20c69bf0281823281103d4dda","unresolved":false,"context_lines":[{"line_number":510,"context_line":"- event_type: trove.*"},{"line_number":511,"context_line":"  traits: \u0026trove_base_traits"},{"line_number":512,"context_line":"    state:"},{"line_number":513,"context_line":"      fields: [\u0027payload.state_description\u0027, \u0027payload.state\u0027]"},{"line_number":514,"context_line":"    instance_type:"},{"line_number":515,"context_line":"      fields: payload.instance_type"},{"line_number":516,"context_line":"    user_id:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f96bb07_49ae44e8","line":513,"in_reply_to":"9f91af0f_e15a2281","updated":"2018-01-11 12:32:17.000000000","message":"curious why the same status info is published twice using two fields.","commit_id":"9efed3124d5eca6b26f0036c1254b6f8831a4a1d"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"f7497354a1689cef2842eb389932f2963ba27964","unresolved":false,"context_lines":[{"line_number":510,"context_line":"- event_type: trove.*"},{"line_number":511,"context_line":"  traits: \u0026trove_base_traits"},{"line_number":512,"context_line":"    state:"},{"line_number":513,"context_line":"      fields: [\u0027payload.state_description\u0027, \u0027payload.state\u0027]"},{"line_number":514,"context_line":"    instance_type:"},{"line_number":515,"context_line":"      fields: payload.instance_type"},{"line_number":516,"context_line":"    user_id:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f96bb07_e25ba22b","line":513,"in_reply_to":"9f91af0f_e15a2281","updated":"2018-01-11 16:39:13.000000000","message":"sure. that works. or we can just add state_description back if they do fix it. personally, i\u0027d rather not add something without reason. just waste space in db.","commit_id":"9efed3124d5eca6b26f0036c1254b6f8831a4a1d"},{"author":{"_account_id":23037,"name":"Huachao Mao","email":"mao.huachao@99cloud.net","username":"huachaomao"},"change_message_id":"6bc30adbb533f0526329969be0a14bed0ecf070d","unresolved":false,"context_lines":[{"line_number":510,"context_line":"- event_type: trove.*"},{"line_number":511,"context_line":"  traits: \u0026trove_base_traits"},{"line_number":512,"context_line":"    state:"},{"line_number":513,"context_line":"      fields: [\u0027payload.state_description\u0027, \u0027payload.state\u0027]"},{"line_number":514,"context_line":"    instance_type:"},{"line_number":515,"context_line":"      fields: payload.instance_type"},{"line_number":516,"context_line":"    user_id:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9f91af0f_e15a2281","line":513,"in_reply_to":"9f91af0f_e619e579","updated":"2018-01-11 01:35:33.000000000","message":"How we just keep both fields, but rename the first state to state_description, so that if trove make state description and state unrelated in the future, we can get both fields as expected without code update here.","commit_id":"9efed3124d5eca6b26f0036c1254b6f8831a4a1d"}]}