)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"2db531cfaf530e0eeb95944c1950faad52c284d6","unresolved":false,"context_lines":[{"line_number":19,"context_line":"So add user/project CONF with admin role at cinder group,"},{"line_number":20,"context_line":"and when determine context is_admin and without token, do"},{"line_number":21,"context_line":"authenticaion with user/project info to call cinder api."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I3c35bba43fee81baebe8261f546c1424ce3a3383"},{"line_number":24,"context_line":"Closes-Bug: #1733736"},{"line_number":25,"context_line":"Closes-Bug: #1734025"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fdfeff1_997b127a","line":22,"updated":"2019-02-20 10:57:39.000000000","message":"so you had two conflicts when backporting (I tested this on my own). You should mention them like eg. https://review.openstack.org/#/c/623359/\n\nAdd a new \u0027Conflicts\u0027 tag here and a NOTE(owner) that will explain the merge conflict issue and how you fixed it.","commit_id":"ab3b98665f170b369dc0012788bcaf18e731b0e3"},{"author":{"_account_id":26286,"name":"huanhongda","email":"hongda.xun@easystack.cn","username":"huanhongda"},"change_message_id":"4db54093a86bd75e16241e4a1492863cf079ea24","unresolved":false,"context_lines":[{"line_number":19,"context_line":"So add user/project CONF with admin role at cinder group,"},{"line_number":20,"context_line":"and when determine context is_admin and without token, do"},{"line_number":21,"context_line":"authenticaion with user/project info to call cinder api."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I3c35bba43fee81baebe8261f546c1424ce3a3383"},{"line_number":24,"context_line":"Closes-Bug: #1733736"},{"line_number":25,"context_line":"Closes-Bug: #1734025"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9fdfeff1_1b9c1e5a","line":22,"in_reply_to":"9fdfeff1_997b127a","updated":"2019-03-02 13:26:18.000000000","message":"Done\nThanks.","commit_id":"ab3b98665f170b369dc0012788bcaf18e731b0e3"}],"releasenotes/notes/config-cinder-admin-creds-b86038a3e87a1021.yaml":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"2db531cfaf530e0eeb95944c1950faad52c284d6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    It is now possible to configure the ``[cinder]`` section of nova.conf to"},{"line_number":5,"context_line":"    allow setting admin-role credentials for scenarios where a user token is"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9fdfeff1_591fea08","line":2,"updated":"2019-02-20 10:57:39.000000000","message":"if you see my original concern on the master change https://review.openstack.org/#/c/522112/9/releasenotes/notes/config-cinder-admin-creds-b86038a3e87a1021.yaml@2 I said it was also a new feature for using admin-only creds.\n\nSo, I\u0027d be afraid of backporting a feature, but here we\u0027re definitely fixing bugs. I think the benefits overcome the risks.","commit_id":"ab3b98665f170b369dc0012788bcaf18e731b0e3"}]}
