| opendevreview | Ade Lee proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 03:29 |
|---|---|---|
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 03:30 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group os-reset_status https://review.opendev.org/c/openstack/cinder/+/804735 | 03:30 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 03:30 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for group-snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804757 | 03:30 |
| opendevreview | 王昊 proposed openstack/cinder master: Support multiple filters together for volume time comparison filter https://review.opendev.org/c/openstack/cinder/+/740146 | 07:34 |
| opendevreview | 王昊 proposed openstack/cinder master: Support to query snapshot filter by updated_at/created_at https://review.opendev.org/c/openstack/cinder/+/733062 | 07:43 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Fix: Online migration for volume_use_quota_online_data_migration https://review.opendev.org/c/openstack/cinder/+/807083 | 09:10 |
| snadge | zenkuro, whoami-rajat.. thanks for the follow up, i have noted that advice if i need further troubleshooting into the problem.. the other sysadmin noticed that the systems which wern't working had a bunch of iscsi attachments which were stale, so they were removed and the affected systems restarted.. they now seem to be creating volumes which are detected as expected | 09:25 |
| snadge | it was an odd problem, it would create the volumes then say that they are not there, they just wouldn't attach properly | 09:25 |
| venkatakrishnat | Hi Cores, it would be great if you take some time to review this patch https://review.opendev.org/c/openstack/cinder/+/776307 which is to fix IBM SVC HyperSwap snapshot clone issues. | 10:16 |
| opendevreview | Kazumasa Nomura proposed openstack/cinder master: Hitachi: Add generic volume groups https://review.opendev.org/c/openstack/cinder/+/782910 | 10:16 |
| opendevreview | katari manoj kumar proposed openstack/cinder master: [SVF]:HyperSwap volume service status update https://review.opendev.org/c/openstack/cinder/+/791281 | 11:08 |
| opendevreview | katari manoj kumar proposed openstack/cinder master: [SVF]:HyperSwap volume service status update https://review.opendev.org/c/openstack/cinder/+/791281 | 11:09 |
| opendevreview | Giulio Fidente proposed openstack/devstack-plugin-ceph master: Use Ceph Pacific for wallaby tempest jobs https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/807706 | 11:24 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Implement project personas for volume API https://review.opendev.org/c/openstack/cinder/+/760199 | 11:42 |
| rosmaita | sfernand: looks like the NetApp CI cinder-cDOT-FCP job could use some attention | 13:16 |
| zenkuro | rosmaita: abandonning -> done. | 13:17 |
| rosmaita | zenkuro: ty | 13:18 |
| opendevreview | Alan Bishop proposed openstack/cinder master: WIP: Implement Xena project personas for volume types https://review.opendev.org/c/openstack/cinder/+/760197 | 13:23 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for snapshot metadata https://review.opendev.org/c/openstack/cinder/+/760153 | 13:36 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Implement project personas for volume API https://review.opendev.org/c/openstack/cinder/+/760199 | 13:36 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for volume transfers https://review.opendev.org/c/openstack/cinder/+/760194 | 13:47 |
| opendevreview | Giulio Fidente proposed openstack/devstack-plugin-ceph master: Use Ceph Pacific for wallaby tempest jobs https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/807706 | 13:51 |
| venkatakrishnat | Hi jungleboyj, rosmaita: Could you please have a look at this patch https://review.opendev.org/c/openstack/cinder/+/788782 which got one CR +2 from Rajat after addressing all the recent comments from reviewers. | 14:16 |
| opendevreview | Merged openstack/cinder master: Docs: Discourage using naked rechecks https://review.opendev.org/c/openstack/cinder/+/806989 | 14:34 |
| geguileo | ganso: your should look into the HA A/A specs, it is explained there what the problem is with using just the host | 14:39 |
| geguileo | ganso: long story short, DON'T DO IT, EVER | 14:40 |
| geguileo | whoami-rajat: I just added a comment on https://review.opendev.org/c/openstack/cinder/+/807083 | 14:40 |
| geguileo | whoami-rajat: my code was garbage, and the patch needs more changes | 14:40 |
| geguileo | whoami-rajat: because the problem is that the volumes are not getting loaded with the volume_admin_metadata table, so that's why the attribute admin_metadata is missing | 14:41 |
| * geguileo facepalms. Hard. With a chair. | 14:42 | |
| raghavendrat | hi geguileo: and jungleboyj: this was reviewed few days back: | 14:50 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/805796 | 14:50 |
| raghavendrat | HPE 3PAR: Add HPE Alletra 9k related information | 14:50 |
| whoami-rajat | geguileo, ack, will fix it from the ovo side | 14:51 |
| raghavendrat | It would be great if you can re-vote. Thanks. | 14:51 |
| geguileo | raghavendrat: done | 14:51 |
| geguileo | whoami-rajat: no, no, no, no OVO involvement | 14:51 |
| geguileo | that was my head not working as it should | 14:51 |
| geguileo | the code is happening at the ORM level in the SQLA api module | 14:52 |
| geguileo | what we need is to properly do the joined load of the other table and to properly look for the temporary key | 14:52 |
| whoami-rajat | oh sorry, read the wrong comment | 14:52 |
| geguileo | the volume_admin at the ORM level could be None, I think | 14:52 |
| geguileo | whoami-rajat: no, my bad for saying stupid stuff in the comment | 14:53 |
| raghavendrat | Thank you Gorka | 14:53 |
| whoami-rajat | geguileo, ack, will do, thanks for the review | 14:54 |
| geguileo | whoami-rajat: thank you for fixing my code | 14:54 |
| whoami-rajat | np :) | 14:55 |
| venkatakrishnat | Hi Core-reviewers, Can someone please review this patch https://review.opendev.org/c/openstack/cinder/+/784140 which is to fix Mirror volume extend related issues for IBM SVC driver and must-go to coming release. | 15:00 |
| opendevreview | Giulio Fidente proposed openstack/devstack-plugin-ceph master: Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/807706 | 15:06 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement secure RBAC for snapshot actions https://review.opendev.org/c/openstack/cinder/+/760152 | 15:17 |
| whoami-rajat | geguileo, hey, I've one query, if we do the joined load in non-admin case, wouldn't that expose the admin_metadata part to all places calling it? https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/api.py#L1865 | 15:22 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Update project personas for default_types https://review.opendev.org/c/openstack/cinder/+/806991 | 15:29 |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph stable/stein: Update .gitreview and TOX_CONSTRAINTS_FILE for stable/stein https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/807773 | 16:45 |
| manoj_katari | Hi core team, could you please review this IBM SVF plugin change for fixing issues during attach-detach operations : https://review.opendev.org/c/openstack/cinder/+/772623 | 17:06 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: policy.py https://review.opendev.org/c/openstack/cinder/+/775008 | 17:18 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove unused config imports/objects https://review.opendev.org/c/openstack/cinder/+/805083 | 17:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: annotate volume/api.py https://review.opendev.org/c/openstack/cinder/+/733622 | 17:36 |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph stable/stein: Fix the setup of the stable/stein branch https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/807773 | 17:53 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator https://review.opendev.org/c/openstack/os-brick/+/786732 | 18:05 |
| opendevreview | Merged openstack/cinder master: [SVF]:HyperSwap volume service status update https://review.opendev.org/c/openstack/cinder/+/791281 | 18:06 |
| opendevreview | Merged openstack/cinder master: [SVF] Update rccg details for mirror volumes https://review.opendev.org/c/openstack/cinder/+/788782 | 18:06 |
| opendevreview | Merged openstack/cinder master: HPE 3PAR: Add HPE Alletra 9k related information https://review.opendev.org/c/openstack/cinder/+/805796 | 18:06 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: utils.py https://review.opendev.org/c/openstack/os-brick/+/786731 | 18:19 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator https://review.opendev.org/c/openstack/os-brick/+/786732 | 18:19 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: lvm.py https://review.opendev.org/c/openstack/os-brick/+/786733 | 18:19 |
| opendevreview | Eric Harney proposed openstack/cinder master: Correct type annotations for utils.retry() https://review.opendev.org/c/openstack/cinder/+/807789 | 18:21 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Update project personas for default_types https://review.opendev.org/c/openstack/cinder/+/806991 | 18:21 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: backup https://review.opendev.org/c/openstack/cinder/+/787195 | 18:25 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for the limits API https://review.opendev.org/c/openstack/cinder/+/759987 | 19:07 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for volume transfers https://review.opendev.org/c/openstack/cinder/+/760194 | 19:10 |
| opendevreview | Alan Bishop proposed openstack/cinder master: WIP: Implement Xena project personas for volume access API https://review.opendev.org/c/openstack/cinder/+/760184 | 19:21 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Implement project personas for volume groups https://review.opendev.org/c/openstack/cinder/+/759962 | 21:25 |
| opendevreview | Luigi Toscano proposed openstack/devstack-plugin-ceph stable/stein: Fix the setup of the stable/stein branch https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/807773 | 22:22 |
| opendevreview | Luigi Toscano proposed openstack/cinder master: Native multibackend-matrix Zuul v3 job https://review.opendev.org/c/openstack/cinder/+/751579 | 23:06 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Implement project personas for group snapshots https://review.opendev.org/c/openstack/cinder/+/759961 | 23:33 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Implement project personas for group snapshots https://review.opendev.org/c/openstack/cinder/+/759961 | 23:35 |
| opendevreview | Merged openstack/cinder master: Remove 'enable_v3_api' option https://review.opendev.org/c/openstack/cinder/+/803533 | 23:49 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!