)]}'
{"deliverables/rocky/manila.yaml":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"87127a363b4428a428437cf93239113bfc89470a","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        repo: openstack/manila"},{"line_number":31,"context_line":"    version: 7.1.0"},{"line_number":32,"context_line":"  - projects:"},{"line_number":33,"context_line":"      - hash: c9a0388e030abce950d737d8de68472a3fb6f3f8"},{"line_number":34,"context_line":"        repo: openstack/manila"},{"line_number":35,"context_line":"    version: 7.2.0"},{"line_number":36,"context_line":"repository-settings:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5fc1f717_66fc2fb5","line":33,"range":{"start_line":33,"start_character":14,"end_line":33,"end_character":54},"updated":"2019-03-25 04:06:01.000000000","message":"I strongly suggest you make this: ce370a1892ed5e1820d55008acb2e87abc7b9db8 and then we can take time to discuss If180f1a2ea8c307c8a40f2a17b35796357306c03 without holding up the release.\n\nWhat is the rationale for backporting these deprcations to pike?  It certainly isn\u0027t described in the bug.\n\nIn terms of the VMAX diver the code was added in https://review.openstack.org/#/c/404859/ (2017-01-25) which was after https://review.openstack.org/#/c/415079/ (2017-01-10) so I guess it was a case or reviews passing in the ether? \n and the code has always been misaligned. The driver exposes vmax_ethernet_ports but then using emc_interface_ports.  Also at least the Unity driver lists emc_interface_ports rather than unity_ethernet_ports as the way to configure the driver[1] or did I misunderstand the state.\n\n \n\n[1] https://docs.openstack.org/manila/latest/admin/emc_unity_driver.html#configurations or did I misunderstand the state.","commit_id":"ce84caede3c7e77ce7e57bf16b79b684a4e964b5"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"e2edc2642341b27db3eada335ee1af9747904d42","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        repo: openstack/manila"},{"line_number":31,"context_line":"    version: 7.1.0"},{"line_number":32,"context_line":"  - projects:"},{"line_number":33,"context_line":"      - hash: c9a0388e030abce950d737d8de68472a3fb6f3f8"},{"line_number":34,"context_line":"        repo: openstack/manila"},{"line_number":35,"context_line":"    version: 7.2.0"},{"line_number":36,"context_line":"repository-settings:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5fc1f717_8b011b09","line":33,"range":{"start_line":33,"start_character":14,"end_line":33,"end_character":54},"in_reply_to":"5fc1f717_66fc2fb5","updated":"2019-03-25 10:10:07.000000000","message":"OK let\u0027s defer the discussion of If180f1a2ea8c307c8a40f2a17b35796357306c03 in the interests of catching stable/rocky releases up with stable/queens.\n\nBut using ce370a1892ed5e1820d55008acb2e87abc7b9db8 will skip I04b1fe88808596aa8c05429b964190d654587f9a which is a good bug fix that is now included in stable/queens.  Can we use its stable/rocky SHA here?","commit_id":"ce84caede3c7e77ce7e57bf16b79b684a4e964b5"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"0d5a5d5b3e00528252a4db450ae7826c8c6f3e7e","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        repo: openstack/manila"},{"line_number":31,"context_line":"    version: 7.1.0"},{"line_number":32,"context_line":"  - projects:"},{"line_number":33,"context_line":"      - hash: c9a0388e030abce950d737d8de68472a3fb6f3f8"},{"line_number":34,"context_line":"        repo: openstack/manila"},{"line_number":35,"context_line":"    version: 7.2.0"},{"line_number":36,"context_line":"repository-settings:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5fc1f717_8df59859","line":33,"range":{"start_line":33,"start_character":14,"end_line":33,"end_character":54},"in_reply_to":"5fc1f717_8b011b09","updated":"2019-03-26 00:28:31.000000000","message":"Sorry my bad:  Looking at the top of the git DAG we have:\n\n * c9a0388e (HEAD -\u003e stable/rocky, origin/stable/rocky, gerrit/stable/rocky) VMAX manila - deprecate old tags correctly\n *   5180e545 Merge \"Fix driver filter to not check share_backend_name\" into stable/rocky\n |\\  \n | * ce370a18 Fix driver filter to not check share_backend_name\n * |   a1ab550b Merge \"Destroy type quotas when a share type is deleted\" into stable/rocky\n |\\ \\  \n | * | aae7bc72 Destroy type quotas when a share type is deleted\n | |/  \n * |   278fdf75 Merge \"Fix tls-proxy issues with the devstack plugin\" into stable/rocky\n\nSo using 5180e545417465c883f693c1f8c993bd303e966f will only skip the VMAX change","commit_id":"ce84caede3c7e77ce7e57bf16b79b684a4e964b5"}]}