)]}'
{"cinder/volume/drivers/dell_emc/powermax/common.py":[{"author":{"_account_id":30165,"name":"Simon O\u0027Donovan","email":"simon.odonovan@dell.com","username":"odonos12"},"change_message_id":"f5994664b47a2d6de68f5d4cc0ab1ccb06266f93","unresolved":false,"context_lines":[{"line_number":5329,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":5330,"context_line":"                message\u003dexception_message)"},{"line_number":5331,"context_line":""},{"line_number":5332,"context_line":"    def update_metadata("},{"line_number":5333,"context_line":"            self, model_update, existing_metadata, object_metadata):"},{"line_number":5334,"context_line":"        \"\"\"Update volume metadata in model_update."},{"line_number":5335,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"7faddb67_b97760d1","line":5332,"updated":"2019-09-04 15:15:36.000000000","message":"In this are we updating a model_update with existing metadata (existing_metadata) with new metadata (object_metadata)? If so there is a potential that the existing metadata update will overwrite newer data if they both contain the same keys as it is called afterwards.","commit_id":"3508d01a7ef3af91f9a55c78ca9a8089dd111835"},{"author":{"_account_id":23601,"name":"michael","display_name":"michael-mcaleer","email":"Michael.Mcaleer@dell.com","username":"michael-mcaleer"},"change_message_id":"302a3eef3f365395a61a4dcd96b62b3794e7f0b4","unresolved":false,"context_lines":[{"line_number":5329,"context_line":"            raise exception.VolumeBackendAPIException("},{"line_number":5330,"context_line":"                message\u003dexception_message)"},{"line_number":5331,"context_line":""},{"line_number":5332,"context_line":"    def update_metadata("},{"line_number":5333,"context_line":"            self, model_update, existing_metadata, object_metadata):"},{"line_number":5334,"context_line":"        \"\"\"Update volume metadata in model_update."},{"line_number":5335,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"5faad753_e111e105","line":5332,"in_reply_to":"7faddb67_b97760d1","updated":"2019-09-12 15:20:24.000000000","message":"This was included so we don\u0027t overwrite any existing metadata that is set, we were failing tempest tests because of the oversight. Existing metadata would fall under user set metadata, not values we are setting in our driver so the users settings remain unaffected.","commit_id":"3508d01a7ef3af91f9a55c78ca9a8089dd111835"}],"releasenotes/notes/powermax-vol-metadata-acd2555818d25b72.yaml":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"02aa175e98009e15c328c4625f58ce71d9a1bd1b","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    All volumes and snapshots created using the PowerMax for Cinder driver now"},{"line_number":5,"context_line":"    have additional metadata included pertaining to the details of the asset on"},{"line_number":6,"context_line":"    the backend storage array."}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5faad753_c96f31fa","line":6,"updated":"2019-09-12 12:56:35.000000000","message":"Not sure if this is worth having a release note. What does the end user need to know with this? This seems like an internal detail.","commit_id":"3508d01a7ef3af91f9a55c78ca9a8089dd111835"},{"author":{"_account_id":23601,"name":"michael","display_name":"michael-mcaleer","email":"Michael.Mcaleer@dell.com","username":"michael-mcaleer"},"change_message_id":"302a3eef3f365395a61a4dcd96b62b3794e7f0b4","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    All volumes and snapshots created using the PowerMax for Cinder driver now"},{"line_number":5,"context_line":"    have additional metadata included pertaining to the details of the asset on"},{"line_number":6,"context_line":"    the backend storage array."}],"source_content_type":"text/x-yaml","patch_set":12,"id":"5faad753_a1e5290b","line":6,"in_reply_to":"5faad753_c96f31fa","updated":"2019-09-12 15:20:24.000000000","message":"We were asked to include it so it would help with identifying Cinder assets on the storage backend where there are large environments. It seemed like a reasonable request and having detailed volume information about what exists on the backend serves to improve user experience in my opinion.","commit_id":"3508d01a7ef3af91f9a55c78ca9a8089dd111835"}]}