)]}'
{".agents/skills/cinder-code-review/SKILL.md":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"dcf4eb10fa8a88a4d96bb4eca611f3bd728dd639","unresolved":true,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"### Step 2: Invoke Cinder Core Reviewer Persona"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Load the `cinder-core` persona for detailed technical review:"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"~~~"},{"line_number":130,"context_line":"@personas/cinder-core.md"}],"source_content_type":"text/x-markdown","patch_set":4,"id":"381e5b1c_6bb2d678","line":127,"range":{"start_line":127,"start_character":0,"end_line":127,"end_character":61},"updated":"2026-07-15 16:43:13.000000000","message":"this conflicts with Step 2 in cinder-driver-review skill which has a fallback note - If the persona file does not exist, skip persona\ninvocation and proceed with the review using the instructions in this skill.","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"}],".agents/skills/cinder-driver-review/SKILL.md":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":true,"context_lines":[{"line_number":52,"context_line":"```"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"**Decision logic**:"},{"line_number":55,"context_line":"- If ANY file matches `cinder/volume/drivers/*/` → This is a **driver change**, continue with driver review"},{"line_number":56,"context_line":"- If files are in `cinder/api/`, `cinder/scheduler/`, `cinder/backup/` → Defer to `cinder-code-review` skill instead"},{"line_number":57,"context_line":"- If uncertain → Fetch the file list and verify"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"42b6f462_e9f72ae4","line":55,"range":{"start_line":55,"start_character":0,"end_line":55,"end_character":107},"updated":"2026-07-08 15:21:52.000000000","message":". The detection pattern only matches cinder/volume/drivers/*/ (subdirectory required), but several in-tree drivers referenced later in this same skill are flat files directly under cinder/volume/drivers/ — rbd.py, zadara.py, quobyte.py, lvm.py. As written, a patch touching e.g. cinder/volume/drivers/rbd.py would fail the \"is this a driver change\" check and skip driver review entirely. Needs to match both cinder/volume/drivers/\u003cfile\u003e.py and cinder/volume/drivers/\u003cvendor\u003e/**","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d100436d2565f0980a82d88a6d30f73860d2b9ed","unresolved":false,"context_lines":[{"line_number":52,"context_line":"```"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"**Decision logic**:"},{"line_number":55,"context_line":"- If ANY file matches `cinder/volume/drivers/*/` → This is a **driver change**, continue with driver review"},{"line_number":56,"context_line":"- If files are in `cinder/api/`, `cinder/scheduler/`, `cinder/backup/` → Defer to `cinder-code-review` skill instead"},{"line_number":57,"context_line":"- If uncertain → Fetch the file list and verify"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":2,"id":"01150682_7dfb6d19","line":55,"range":{"start_line":55,"start_character":0,"end_line":55,"end_character":107},"in_reply_to":"42b6f462_e9f72ae4","updated":"2026-07-14 11:43:43.000000000","message":"Done","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":true,"context_lines":[{"line_number":85,"context_line":"  tail -n +2 | jq \u0027keys[]\u0027 | grep \"cinder/volume/drivers\""},{"line_number":86,"context_line":"```"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"**Driver to CI mapping** (see https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers):"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"| Driver Path | Required CI System | CI Name Pattern |"},{"line_number":91,"context_line":"|-------------|-------------------|-----------------|"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"522dd08b_5826ecde","line":88,"range":{"start_line":88,"start_character":1,"end_line":88,"end_character":22},"updated":"2026-07-08 15:21:52.000000000","message":"Duplicate CI-mapping tables (\"Driver to CI mapping\" vs. the bottom-of-doc (line 1060) \"Common Third-Party CI Systems\") — they don\u0027t fully agree with each other (columns and rows differ slightly) and will drift over time. Collapse to one table referenced from both places.","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d100436d2565f0980a82d88a6d30f73860d2b9ed","unresolved":false,"context_lines":[{"line_number":85,"context_line":"  tail -n +2 | jq \u0027keys[]\u0027 | grep \"cinder/volume/drivers\""},{"line_number":86,"context_line":"```"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"**Driver to CI mapping** (see https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers):"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"| Driver Path | Required CI System | CI Name Pattern |"},{"line_number":91,"context_line":"|-------------|-------------------|-----------------|"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"58666dcf_9e2a473e","line":88,"range":{"start_line":88,"start_character":1,"end_line":88,"end_character":22},"in_reply_to":"522dd08b_5826ecde","updated":"2026-07-14 11:43:43.000000000","message":"Done","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":true,"context_lines":[{"line_number":90,"context_line":"| Driver Path | Required CI System | CI Name Pattern |"},{"line_number":91,"context_line":"|-------------|-------------------|-----------------|"},{"line_number":92,"context_line":"| `dell_emc/powerflex/` | Dell PowerFlex CI | `Dell Openstack CI` or `powerflex-v4-*` |"},{"line_number":93,"context_line":"| `pure/` | Pure Storage CI | `Pure Storage CI` |"},{"line_number":94,"context_line":"| `netapp/` | NetApp CI | `NetApp CI` |"},{"line_number":95,"context_line":"| `hpe/hpe_3par_*` | HPE 3PAR CI | `HPE 3PAR CI` |"},{"line_number":96,"context_line":"| `solidfire/` | NetApp SolidFire CI | `NetApp SolidFire CI` |"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"60886cfb_53238618","line":93,"range":{"start_line":93,"start_character":12,"end_line":93,"end_character":28},"updated":"2026-07-08 15:21:52.000000000","message":"Also, this is in the process of being rebranded to Everpure, so there needs to be some way to let the skill know when any vendor driver is rebranded and CI names change, or when new drivers are added.","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":true,"context_lines":[{"line_number":90,"context_line":"| Driver Path | Required CI System | CI Name Pattern |"},{"line_number":91,"context_line":"|-------------|-------------------|-----------------|"},{"line_number":92,"context_line":"| `dell_emc/powerflex/` | Dell PowerFlex CI | `Dell Openstack CI` or `powerflex-v4-*` |"},{"line_number":93,"context_line":"| `pure/` | Pure Storage CI | `Pure Storage CI` |"},{"line_number":94,"context_line":"| `netapp/` | NetApp CI | `NetApp CI` |"},{"line_number":95,"context_line":"| `hpe/hpe_3par_*` | HPE 3PAR CI | `HPE 3PAR CI` |"},{"line_number":96,"context_line":"| `solidfire/` | NetApp SolidFire CI | `NetApp SolidFire CI` |"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"b4ed387f_7de46ff4","line":93,"range":{"start_line":93,"start_character":3,"end_line":93,"end_character":9},"updated":"2026-07-08 15:21:52.000000000","message":"Pure drivers are not in a directory - need to properly validate for all drivers","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d100436d2565f0980a82d88a6d30f73860d2b9ed","unresolved":false,"context_lines":[{"line_number":90,"context_line":"| Driver Path | Required CI System | CI Name Pattern |"},{"line_number":91,"context_line":"|-------------|-------------------|-----------------|"},{"line_number":92,"context_line":"| `dell_emc/powerflex/` | Dell PowerFlex CI | `Dell Openstack CI` or `powerflex-v4-*` |"},{"line_number":93,"context_line":"| `pure/` | Pure Storage CI | `Pure Storage CI` |"},{"line_number":94,"context_line":"| `netapp/` | NetApp CI | `NetApp CI` |"},{"line_number":95,"context_line":"| `hpe/hpe_3par_*` | HPE 3PAR CI | `HPE 3PAR CI` |"},{"line_number":96,"context_line":"| `solidfire/` | NetApp SolidFire CI | `NetApp SolidFire CI` |"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"5ecfd543_bc770e63","line":93,"range":{"start_line":93,"start_character":12,"end_line":93,"end_character":28},"in_reply_to":"60886cfb_53238618","updated":"2026-07-14 11:43:43.000000000","message":"Done","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d100436d2565f0980a82d88a6d30f73860d2b9ed","unresolved":false,"context_lines":[{"line_number":90,"context_line":"| Driver Path | Required CI System | CI Name Pattern |"},{"line_number":91,"context_line":"|-------------|-------------------|-----------------|"},{"line_number":92,"context_line":"| `dell_emc/powerflex/` | Dell PowerFlex CI | `Dell Openstack CI` or `powerflex-v4-*` |"},{"line_number":93,"context_line":"| `pure/` | Pure Storage CI | `Pure Storage CI` |"},{"line_number":94,"context_line":"| `netapp/` | NetApp CI | `NetApp CI` |"},{"line_number":95,"context_line":"| `hpe/hpe_3par_*` | HPE 3PAR CI | `HPE 3PAR CI` |"},{"line_number":96,"context_line":"| `solidfire/` | NetApp SolidFire CI | `NetApp SolidFire CI` |"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"78cb9625_6df18a27","line":93,"range":{"start_line":93,"start_character":3,"end_line":93,"end_character":9},"in_reply_to":"b4ed387f_7de46ff4","updated":"2026-07-14 11:43:43.000000000","message":"Done","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":true,"context_lines":[{"line_number":168,"context_line":""},{"line_number":169,"context_line":"**Critical CI Issues** (blocking):"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"- Third-party CI has not reported in 2+ weeks (driver at risk)"},{"line_number":172,"context_line":"- CI reported FAILURE on vendor-specific code changes"},{"line_number":173,"context_line":"- CI tested wrong driver (LVM instead of vendor driver)"},{"line_number":174,"context_line":"- CI logs show driver not loaded/initialized"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"f2948ec6_732a2171","line":171,"range":{"start_line":171,"start_character":2,"end_line":171,"end_character":62},"updated":"2026-07-08 15:21:52.000000000","message":"Step 0b.5 and the review template treat \"no CI report in 14 days\" as a rolling SLA (\"must report within a 2-week span\" / \"at risk of unsupported\"). That\u0027s not the actual policy. Compliance review happens around Milestone 3 of the cycle; if a driver is flagged unsupported there, vendors have until RC1 (two weeks after M3) to fix it before the driver drops from the release. It\u0027s milestone-anchored, not a continuous 2-week clock. As written, this skill would generate a factually incorrect \"CI at risk\" comment on patches mid-cycle. Suggest citing docs.openstack.org/cinder/latest/drivers-all-about.html directly instead of restating the policy from memory.","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d100436d2565f0980a82d88a6d30f73860d2b9ed","unresolved":false,"context_lines":[{"line_number":168,"context_line":""},{"line_number":169,"context_line":"**Critical CI Issues** (blocking):"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"- Third-party CI has not reported in 2+ weeks (driver at risk)"},{"line_number":172,"context_line":"- CI reported FAILURE on vendor-specific code changes"},{"line_number":173,"context_line":"- CI tested wrong driver (LVM instead of vendor driver)"},{"line_number":174,"context_line":"- CI logs show driver not loaded/initialized"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"4e85d2e4_685b1668","line":171,"range":{"start_line":171,"start_character":2,"end_line":171,"end_character":62},"in_reply_to":"f2948ec6_732a2171","updated":"2026-07-14 11:43:43.000000000","message":"Done","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":true,"context_lines":[{"line_number":244,"context_line":"**For all driver reviews**, invoke the Cinder Core persona:"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"```"},{"line_number":247,"context_line":"@personas/cinder-core.md"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"Review this driver change focusing on:"},{"line_number":250,"context_line":"- Driver interface compliance"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"b1d75d41_b8b4235b","line":247,"range":{"start_line":247,"start_character":0,"end_line":247,"end_character":24},"updated":"2026-07-08 15:21:52.000000000","message":"invoked with no stated path relative to this skill file. If those don\u0027t exist yet in this series, the persona-invocation step silently no-ops. Either add a path note or land the persona files in the same patch.","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d100436d2565f0980a82d88a6d30f73860d2b9ed","unresolved":false,"context_lines":[{"line_number":244,"context_line":"**For all driver reviews**, invoke the Cinder Core persona:"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"```"},{"line_number":247,"context_line":"@personas/cinder-core.md"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"Review this driver change focusing on:"},{"line_number":250,"context_line":"- Driver interface compliance"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"92ea3af0_c019b7ec","line":247,"range":{"start_line":247,"start_character":0,"end_line":247,"end_character":24},"in_reply_to":"b1d75d41_b8b4235b","updated":"2026-07-14 11:43:43.000000000","message":"Done","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":true,"context_lines":[{"line_number":257,"context_line":"**For security-sensitive drivers** (encryption, credentials, multi-tenancy):"},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"```"},{"line_number":260,"context_line":"@personas/cinder-coresec.md"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"Security review this driver for:"},{"line_number":263,"context_line":"- Credential handling (backend passwords, API tokens)"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"85163dbf_442870e0","line":260,"range":{"start_line":260,"start_character":0,"end_line":260,"end_character":27},"updated":"2026-07-08 15:21:52.000000000","message":"same as above","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d100436d2565f0980a82d88a6d30f73860d2b9ed","unresolved":false,"context_lines":[{"line_number":257,"context_line":"**For security-sensitive drivers** (encryption, credentials, multi-tenancy):"},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"```"},{"line_number":260,"context_line":"@personas/cinder-coresec.md"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"Security review this driver for:"},{"line_number":263,"context_line":"- Credential handling (backend passwords, API tokens)"}],"source_content_type":"text/x-markdown","patch_set":2,"id":"e5a1ae0a_08efe167","line":260,"range":{"start_line":260,"start_character":0,"end_line":260,"end_character":27},"in_reply_to":"85163dbf_442870e0","updated":"2026-07-14 11:43:43.000000000","message":"Done","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"dcf4eb10fa8a88a4d96bb4eca611f3bd728dd639","unresolved":true,"context_lines":[{"line_number":311,"context_line":"fields and uses constants from `cinder.common.constants` for"},{"line_number":312,"context_line":"`storage_protocol`. For the complete reference template, required/optional"},{"line_number":313,"context_line":"fields, and common mistakes, see the Cinder contributor documentation:"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"https://docs.openstack.org/cinder/latest/contributor/drivers.html#volume-stats-reference-template"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"#### 3.3: Verify Driver Configuration"},{"line_number":318,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"6dfc57b9_ccd3a571","line":315,"range":{"start_line":314,"start_character":0,"end_line":315,"end_character":97},"updated":"2026-07-15 16:43:13.000000000","message":"incorrect link - the anchor should be just ``volume-stats``","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"96082a33539d6100603cb709a467faed70d2f0a7","unresolved":false,"context_lines":[{"line_number":311,"context_line":"fields and uses constants from `cinder.common.constants` for"},{"line_number":312,"context_line":"`storage_protocol`. For the complete reference template, required/optional"},{"line_number":313,"context_line":"fields, and common mistakes, see the Cinder contributor documentation:"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"https://docs.openstack.org/cinder/latest/contributor/drivers.html#volume-stats-reference-template"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"#### 3.3: Verify Driver Configuration"},{"line_number":318,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"201b28bf_0c9853cb","line":315,"range":{"start_line":314,"start_character":0,"end_line":315,"end_character":97},"in_reply_to":"6dfc57b9_ccd3a571","updated":"2026-07-15 18:06:47.000000000","message":"I see the patch for this","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"dcf4eb10fa8a88a4d96bb4eca611f3bd728dd639","unresolved":true,"context_lines":[{"line_number":366,"context_line":"per-protocol return dict examples and common mistakes, see the Cinder"},{"line_number":367,"context_line":"contributor documentation:"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":"https://docs.openstack.org/cinder/latest/contributor/drivers.html#connection-initialization-and-termination"},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"#### 4.3: Create from Snapshot/Clone Handling"},{"line_number":372,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"0204eb21_dadd3f63","line":369,"range":{"start_line":369,"start_character":0,"end_line":369,"end_character":107},"updated":"2026-07-15 16:43:13.000000000","message":"incorrect link - there is no connection-initialization-and-termination anchor. Probably what you want is in ``base-driver-interface`` in the initialize_connection/terminate_connection docstrings","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"96082a33539d6100603cb709a467faed70d2f0a7","unresolved":false,"context_lines":[{"line_number":366,"context_line":"per-protocol return dict examples and common mistakes, see the Cinder"},{"line_number":367,"context_line":"contributor documentation:"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":"https://docs.openstack.org/cinder/latest/contributor/drivers.html#connection-initialization-and-termination"},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"#### 4.3: Create from Snapshot/Clone Handling"},{"line_number":372,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"5fcf2160_2b5700a3","line":369,"range":{"start_line":369,"start_character":0,"end_line":369,"end_character":107},"in_reply_to":"0204eb21_dadd3f63","updated":"2026-07-15 18:06:47.000000000","message":"I see the patch for this","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"dcf4eb10fa8a88a4d96bb4eca611f3bd728dd639","unresolved":true,"context_lines":[{"line_number":478,"context_line":"lock scope decision framework, common patterns, Active-Active checklist,"},{"line_number":479,"context_line":"and existing driver examples, see the Cinder contributor documentation:"},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"https://docs.openstack.org/cinder/latest/contributor/drivers_locking_examples.html#choosing-a-lock-scope"},{"line_number":482,"context_line":""},{"line_number":483,"context_line":"#### 4.7: Cryptography"},{"line_number":484,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"feacc9cd_994261ab","line":481,"range":{"start_line":481,"start_character":0,"end_line":481,"end_character":104},"updated":"2026-07-15 16:43:13.000000000","message":"incorrect link - there is no anchor per the name in the link and more importantly there is no lock-scope in the page anyway","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"96082a33539d6100603cb709a467faed70d2f0a7","unresolved":false,"context_lines":[{"line_number":478,"context_line":"lock scope decision framework, common patterns, Active-Active checklist,"},{"line_number":479,"context_line":"and existing driver examples, see the Cinder contributor documentation:"},{"line_number":480,"context_line":""},{"line_number":481,"context_line":"https://docs.openstack.org/cinder/latest/contributor/drivers_locking_examples.html#choosing-a-lock-scope"},{"line_number":482,"context_line":""},{"line_number":483,"context_line":"#### 4.7: Cryptography"},{"line_number":484,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"d8d182bc_0132f421","line":481,"range":{"start_line":481,"start_character":0,"end_line":481,"end_character":104},"in_reply_to":"feacc9cd_994261ab","updated":"2026-07-15 18:06:47.000000000","message":"I see the patch for this","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"dcf4eb10fa8a88a4d96bb4eca611f3bd728dd639","unresolved":true,"context_lines":[{"line_number":501,"context_line":"constants, and configuration file requirements, see the Cinder contributor"},{"line_number":502,"context_line":"documentation:"},{"line_number":503,"context_line":""},{"line_number":504,"context_line":"https://docs.openstack.org/cinder/latest/contributor/review-patterns.html"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"### Step 6b: Additional Driver Review Checks"},{"line_number":507,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"e92ea333_c7ff4956","line":504,"range":{"start_line":504,"start_character":0,"end_line":504,"end_character":73},"updated":"2026-07-15 16:43:13.000000000","message":"No such page","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"96082a33539d6100603cb709a467faed70d2f0a7","unresolved":false,"context_lines":[{"line_number":501,"context_line":"constants, and configuration file requirements, see the Cinder contributor"},{"line_number":502,"context_line":"documentation:"},{"line_number":503,"context_line":""},{"line_number":504,"context_line":"https://docs.openstack.org/cinder/latest/contributor/review-patterns.html"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"### Step 6b: Additional Driver Review Checks"},{"line_number":507,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"aea43fa8_f99e9c22","line":504,"range":{"start_line":504,"start_character":0,"end_line":504,"end_character":73},"in_reply_to":"e92ea333_c7ff4956","updated":"2026-07-15 18:06:47.000000000","message":"I see the patch for this","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"dcf4eb10fa8a88a4d96bb4eca611f3bd728dd639","unresolved":true,"context_lines":[{"line_number":512,"context_line":"usedforsecurity, and release note completeness, see the Cinder contributor"},{"line_number":513,"context_line":"documentation:"},{"line_number":514,"context_line":""},{"line_number":515,"context_line":"https://docs.openstack.org/cinder/latest/contributor/review-patterns.html#additional-driver-review-patterns"},{"line_number":516,"context_line":""},{"line_number":517,"context_line":"### Step 7: Generate Driver Review Feedback"},{"line_number":518,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"f05df92d_1f9b34b5","line":515,"range":{"start_line":515,"start_character":0,"end_line":515,"end_character":107},"updated":"2026-07-15 16:43:13.000000000","message":"No such page","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"96082a33539d6100603cb709a467faed70d2f0a7","unresolved":false,"context_lines":[{"line_number":512,"context_line":"usedforsecurity, and release note completeness, see the Cinder contributor"},{"line_number":513,"context_line":"documentation:"},{"line_number":514,"context_line":""},{"line_number":515,"context_line":"https://docs.openstack.org/cinder/latest/contributor/review-patterns.html#additional-driver-review-patterns"},{"line_number":516,"context_line":""},{"line_number":517,"context_line":"### Step 7: Generate Driver Review Feedback"},{"line_number":518,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"7287ac90_720dd928","line":515,"range":{"start_line":515,"start_character":0,"end_line":515,"end_character":107},"in_reply_to":"f05df92d_1f9b34b5","updated":"2026-07-15 18:06:47.000000000","message":"OK","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"975c899d5f4b50e4689eae613034199937690752","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7fad0959_3594b406","updated":"2026-07-08 15:21:52.000000000","message":"Reviewed the driver-review skill logic against current Cinder tox/docs. Overall solid coverage of the real review process, but a few things need fixing before this is reliable enough to drive an automated review.","commit_id":"3e9fc0c043ad7a416ff96dd851ccdf2f7bb211de"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"96082a33539d6100603cb709a467faed70d2f0a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"54fee3c9_90a39d19","updated":"2026-07-15 18:06:47.000000000","message":"I see the 2 patches to fix most of these issues. Still one comment outstanding.","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"dcf4eb10fa8a88a4d96bb4eca611f3bd728dd639","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1b4881a7_0b588f03","updated":"2026-07-15 16:43:13.000000000","message":"you needto make some documentation changes - or links to anchors in this skill for ti to work, plus a mismatch between the two skills at step 2","commit_id":"aad25f3a743d32230110f947a9b91f29f97012fc"}]}