)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"8b3b380ac07cc71f8e6105f8676e4e92a65b59cb","unresolved":true,"context_lines":[{"line_number":11,"context_line":"Accordingly updated code and documentation."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ie2542fc4b21050c4f14aea67ea488d9f9eeaae79"},{"line_number":14,"context_line":"(cherry picked from commit 6cabe11c9207376f905db3a0bcadf6d43e2c2af1)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"097b1c89_e61366b6","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":68},"updated":"2022-07-14 11:52:33.000000000","message":"we should mention the chain of all past backport commits and not only of the parent.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"b890c7c4fcc29f6f98e57887fd7844454ae8a2c1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Accordingly updated code and documentation."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ie2542fc4b21050c4f14aea67ea488d9f9eeaae79"},{"line_number":14,"context_line":"(cherry picked from commit 6cabe11c9207376f905db3a0bcadf6d43e2c2af1)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"78d731f9_8d90395b","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":68},"in_reply_to":"097b1c89_e61366b6","updated":"2022-07-14 11:55:00.000000000","message":"Ack","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"8fc4c2b23b3a3e969ba67f0b69c224625f367640","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9d7791b0_f44116a4","updated":"2022-01-26 14:44:00.000000000","message":"After mid-cycle discussion removing my -1","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"f4966299bebe9c8c6011ccd0477a3779b951ca43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ab8fc7a6_ff797029","updated":"2021-12-10 07:13:07.000000000","message":"As per community guidelines, only one backport at a time is recommended.\nSo, lets wait for victoria backport to get completed.\n","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":8108,"name":"Nobuto Murata","email":"nobuto.murata@canonical.com","username":"nobuto-m"},"change_message_id":"32bf2487862064104ff50f98ba3b462af44bdfd0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cd265ffb_76c1d70b","updated":"2022-02-02 00:58:40.000000000","message":"FYI, we\u0027ve opened a downstream bug as Ubuntu packaging of Ussuri:\nhttps://bugs.launchpad.net/ubuntu/+source/cinder/+bug/1959712","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"cf4cd66c442eafef94a8ad704302b46fbd077464","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4b292dce_88e5b174","updated":"2022-07-06 12:31:25.000000000","message":"Hi Brian,\nIt would be great to know your feedback. Thanks.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9348788eba4233999c8314c1c8d742a263ce597c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"35c58645_f407288d","updated":"2022-06-30 10:28:45.000000000","message":"Hi reviewers,\nThis patch was discussed in midcycle meeting [Jan 26th].\nKindly refer comments from this patch on Jan 26th.\n\nThe HPE Storage CI has now passed on this patch.\nZuul had passed already.\n\nAppreciate your feedback.\n","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"8b3b380ac07cc71f8e6105f8676e4e92a65b59cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2b430f51_1be04505","updated":"2022-07-14 11:52:33.000000000","message":"I don\u0027t recall the whole discussion but based on recent comments, this doesn\u0027t look like a feature (as i thought it was at first glance). Merged in victoria, looks good for ussuri.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"fc1f6ce0d282abf4beaaa1d88c973b24beb8f0ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5391e4a1_3f4f14cd","updated":"2022-01-14 14:33:50.000000000","message":"I have two concerns about this patch:\n\n(1) It could be considered to set a precedent for other drivers to backport features.  The project team has not thought that is a good idea in the past, and I\u0027m pretty sure we don\u0027t want to do it in general.  If we decide to allow this backport, we need to be very clear about why it\u0027s being allowed so that other vendors can set expectations accordingly.\n\n(2) From one perspective, this looks like a low risk patch because it is a very small change and all the work is being done on the backend, but the thing to worry about is whether there are other changes present in branch W that support proper functioning; if these other changes aren\u0027t present in branch W-1, then we will be introducing a regression.  So we need to be careful, and it would be good to have some evidence documenting that this change won\u0027t introduce a problem.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":32266,"name":"Venkata krishna Thumu","display_name":"VenkataKrishna","email":"venkata.krishna.reddy@ibm.com","username":"venkatakrishnathumu","status":"Active"},"change_message_id":"143a8d3f1b57e0eabd466a77f17631feb8844fb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dbf83a46_973fcbee","updated":"2021-12-14 16:14:29.000000000","message":"Make sure CI run for this patch","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d1bc0504ffa586751ab1e0ef575ce4c295916c16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d9f0f9f8_6c9ea51d","updated":"2022-07-08 13:46:28.000000000","message":"Sorry, I completely lost track of this patch.  We decided at the (yoga!) midcycle that it is OK to backport.  Raghavendra\u0027s comments indicate that it\u0027s been tested locally to work with stable/ussuri, so I think this is all set to go.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"2c5fb930776c570e17d3a53d2b3a34e8c86e9b0c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ec80d6b6_1ae2532e","updated":"2022-01-27 06:23:48.000000000","message":"Thank you Brian \u0026 Simon.\n","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"b890c7c4fcc29f6f98e57887fd7844454ae8a2c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"df091ce9_81344ba2","updated":"2022-07-14 11:55:00.000000000","message":"Thank you Rajat","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"42e3fdade5c3c645ec79c8722f0e29dd407362c3","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8201fa45_4f8c1df1","updated":"2022-01-17 06:55:18.000000000","message":"Thanks Brian and Simon for review.\nReplies inline.\n\n\u003e Brian Rosmaita:\n\u003e I have two concerns about this patch:\n\n\u003e (1) It could be considered to set a precedent for other drivers to backport features.  The project\n\u003e team has not thought that is a good idea in the past, and I\u0027m pretty sure we don\u0027t want to do it in \n\u003e general.  If we decide to allow this backport, we need to be very clear about why it\u0027s being allowed\n\u003e so that other vendors can set expectations accordingly.\n\nRaghavendra:\n[1] From code perspective, this is a small change.\nHowever, from customer perspective this is big ask.\nAs per link: https://ubuntu.com/about/release-cycle,\nUbuntu 20.04 and Ussuri have standard support till 2025 and extended support till 2030.\nOther releases (V, W \u0026 X) have just 1 or 2 years support.\nSo, it would be great if this change is allowed in Ussuri.\n\nBrian:\n\u003e (2) From one perspective, this looks like a low risk patch because it is a very small change and all \n\u003e the work is being done on the backend, but the thing to worry about is whether there are other \n\u003e changes present in branch W that support proper functioning; if these other changes aren\u0027t \n\u003e present in branch W-1, then we will be introducing a regression.  So we need to be careful, and it \n\u003e would be good to have some evidence documenting that this change won\u0027t introduce a problem.\n\nRaghavendra:\n[2] This change is independent of only one other (HPE 3par) change present in Ussuri.\nhttps://review.opendev.org/c/openstack/cinder/+/713803\nFor Victoria, we did not submit any change.\nWe will surely perform testing from our side to ensure there is no regression because of this patch.\nWe will also ensure HPE 3par CI is passed on this patch.\n\nSimon:\n\u003e You say this is for Train, so you may be referencing RHOSP16 as the deployment. Red Hat will \n\u003e possibly downstream backports if you raise a Bugzilla.\n\nRaghavendra:\nHPE team will check with RH team.\n\n\nRegards,\nRaghavendra Tilay.\n\n","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"44ba2dee6fd925aa576637fbde2f7e9238059bde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b09d71c0_9eb58201","updated":"2022-01-26 22:59:27.000000000","message":"The tl;dr of today\u0027s midcycle discussion is that we decided that this change is NOT a feature, so we do not have to worry about this particular backport setting a precedent.  This is not a feature backport.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"ddcf125a3e7db4b54c36bbbeba508799f47a1493","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5065112e_2e4e6d59","updated":"2022-01-12 12:43:01.000000000","message":"The victoria backport has been merged.\nRequest reviewers to share feedback. Thanks.\n","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f0e8ba41f1753ba2407309d1d71d2ca8a030e290","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1be9195e_0660327a","updated":"2022-01-14 13:32:53.000000000","message":"This looks suspiciously like a feature, so I\u0027m surprised it was backported to victoria without discussion.  It doesn\u0027t look like a big change, but I really don\u0027t want to set a precedent of feature backports.  We need to have a record somewhere about why this backport is OK to refer to when this comes up in the future.\n\nI\u0027ll put it on the agenda for the next Cinder meeting:\n  https://etherpad.opendev.org/p/cinder-yoga-meetings\n\nIt\u0027s not too late to revert the victoria backport if necessary.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"1871d731ac550b2e113f2b35c150b292b15f13dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0ef4ff4b_ed58daee","updated":"2022-02-01 07:00:06.000000000","message":"run-HPE Storage CI","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":22248,"name":"HPE AlletraMP iSCSI CI","email":"hp.cinder.blr.ci@groups.ext.hpe.com","username":"hp-storage-blr-ci","tags":["SERVICE_USER"]},"change_message_id":"8c362e15882e9c923a16b5ce7f2d6c7dfbb9ef7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"14ea8811_2a2a1afa","updated":"2022-06-30 07:30:14.000000000","message":"run-HPE Storage CI","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"2bd614b524811f294961c0e34a3339efba772be6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"17661320_dfc5c68f","updated":"2022-01-31 10:37:10.000000000","message":"run-HPE Storage CI","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"d504df9b6c77be7000f79824e9266cb2f691dc02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20f1c8f0_93f2880b","updated":"2022-01-31 12:05:32.000000000","message":"run-HPE Storage CI","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"da16c8f51a45b641936489ce3ca04ffa5e947b96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9f0d9472_aadfdba4","updated":"2022-02-01 04:47:55.000000000","message":"run-HPE Storage CI","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"1ea811366d06f0d32e0df2ea1c0769b8cef828ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b6238dda_83bf124a","updated":"2022-06-30 05:08:09.000000000","message":"run-HPE Storage CI\n","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"c8aaa1b8c5acac67a208a7057ecc4bd368ec5b8f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9102f126_0c60e23b","updated":"2022-06-14 07:06:01.000000000","message":"run-Nimble Storage Alletra 6k CI\n","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"1d5d255c875d0e2ee0dde74d8492c96d3cba0b4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9717444c_34bf7900","in_reply_to":"1be9195e_0660327a","updated":"2022-01-14 13:56:08.000000000","message":"We\u0027ll need it in train for a major user.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"5d87d7f160c025fa4c9e6353ffe8a23beb93e5a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"deea85f3_d3d1ca9b","in_reply_to":"3342b642_a828b2c3","updated":"2022-01-17 10:45:39.000000000","message":"Hi Simon,\n\nLet me elaborate.\nThere are two different customers:\n[1] One customer asking for Ussuri backport; because of its extended support till 2030.\n[2] RH asking for Train backport; for RHOSP16.\n\nFor time being, we will concentrate only on Ussuri backport.\n\nThanks,\nRaghavendra.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"6ca9839e6723ae9b71863c7ed5782162324e28ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"84cf8e5b_c38ae618","in_reply_to":"5391e4a1_3f4f14cd","updated":"2022-01-14 16:41:03.000000000","message":"From a vendors standpoint, we have never backported features and if a major customer needs a feature we create a patch for them that they then apply themselves to implement the feature - this is assuming it is an easy patch and doesn\u0027t require any core code to be patched. That would be a hard \"upgrade please\" to the customer.\nYou say this is for Train, so you may be referencing RHOSP16 as the deployment. Red Hat will possibly downstream backports if you raise a Bugzilla.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"232293161dbf165a2ec2a4e515a2e25f33951444","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3342b642_a828b2c3","in_reply_to":"8201fa45_4f8c1df1","updated":"2022-01-17 08:05:25.000000000","message":"The Victoria version of this backport was merged so I guess the cores thought that it was OK to backport this as a fix rather than a feature backport (I tend to disagree with that view though..), however the problem IMHO is that Ussuri went into Extended Maintenance last year and so there should be no further backports, feature or otherwise.\nThis has nothing to do with Ubuntu\u0027s support status, especially as you do infer that this is for a RH deployment as another reviewer said this would be needed for Train and you have said you will check with RH about a downstream patch.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"40a199b4afe17225eb9f4745aa727ece05b26d2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8b232a95_e5b1a39f","in_reply_to":"deea85f3_d3d1ca9b","updated":"2022-01-17 13:36:09.000000000","message":"OK - so that 2030 date support is for Ubunut - NOT Ussuri.\nUssuri is EM, so shouldn\u0027t be having patches applied to it, unless they are security related.\n\nI\u0027ll lesave this to Brian as the PTL to make the call as he has a -2 on the patch.\n\nWhy can\u0027t HPE issue the customer with a patch, in the same way other vendors do for EM and Unsupported versions of OpenStack?\n\nBy your wording I imagine that this customer is using Canonicals OpenStack. I would suggest you discuss a downstream backport with Canonical, however would this also not subsequently require Charm backports fro Ussuri? I suspect you won\u0027t get those through easily.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"}],"releasenotes/notes/hpe-3par-primera-add-iscsi-5af339643dfa0928.yaml":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"f0e8ba41f1753ba2407309d1d71d2ca8a030e290","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    HPE 3PAR Driver: Add support of iSCSI driver for Primera 4.2"},{"line_number":5,"context_line":"    or higher versions."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"8dbd55df_e785a7d3","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":8},"updated":"2022-01-14 13:32:53.000000000","message":"We don\u0027t normally backport features.","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"},{"author":{"_account_id":7130,"name":"David Hill","email":"davidchill@hotmail.com","username":"dhill"},"change_message_id":"1d5d255c875d0e2ee0dde74d8492c96d3cba0b4b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    HPE 3PAR Driver: Add support of iSCSI driver for Primera 4.2"},{"line_number":5,"context_line":"    or higher versions."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"751314ff_b3c9d7f7","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":8},"in_reply_to":"8dbd55df_e785a7d3","updated":"2022-01-14 13:56:08.000000000","message":"Ack","commit_id":"c0e8eda0dce9f63488db59f9d2c71f538c75cd51"}]}
