| opendevreview | saikrishnagk proposed openstack/cinder master: [SVF] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/809518 | 04:24 |
|---|---|---|
| opendevreview | saikrishnagk proposed openstack/cinder master: [SVF] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/809518 | 04:34 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Add optimization in upload volume to image in rbd -> rbd https://review.opendev.org/c/openstack/cinder/+/809523 | 06:07 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Add optimization in upload volume to image in rbd -> rbd https://review.opendev.org/c/openstack/cinder/+/809523 | 06:37 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Add optimization in upload volume to image in rbd -> rbd https://review.opendev.org/c/openstack/cinder/+/809523 | 06:47 |
| opendevreview | Kazumasa Nomura proposed openstack/cinder master: Update docs for Hitachi driver https://review.opendev.org/c/openstack/cinder/+/809534 | 07:32 |
| opendevreview | Kazumasa Nomura proposed openstack/cinder master: Update docs for Hitachi driver https://review.opendev.org/c/openstack/cinder/+/809534 | 08:03 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-specs master: Add volume reimage spec https://review.opendev.org/c/openstack/cinder-specs/+/809622 | 09:51 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder-specs master: Add volume reimage spec https://review.opendev.org/c/openstack/cinder-specs/+/809622 | 09:57 |
| *** dviroel|out is now known as dviroel | 11:09 | |
| *** bhagyashris_ is now known as bhagyashris | 11:43 | |
| MrClayPole | I've done some more testing with the os_bricks erroring with "No Active sessions" when running "iscsiadm -m session". It seems with "iser_use_multipath" and "iscsi_use_multipath" both set to true in nova.conf, os_bricks only attempts once with "iscsiadm -m session" which then returns "No Active sessions" before iscsid has brought up the iscsi session. With "iser_use_multipath" and "iscsi_use_multipath" both set to false | 12:06 |
| MrClayPole | . The first os_bricks scan fails with the "No Active sessions" error but then tries again after the iscsi session is up and succeeds. Should os_bricks retry when multipath is enabled in nova.conf or am I missing something? | 12:06 |
| rosmaita | MrClayPole: have you filed a bug for this? (I'm not sure it's really a bug, but it would be good to have this written down somewhere so we can look later.) | 12:25 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Call trash operation when plain deletion fails https://review.opendev.org/c/openstack/cinder/+/805887 | 13:11 |
| enriquetaso | I've update the release note ^ | 13:12 |
| rosmaita | enriquetaso: LGTM (sure hope geguileo agrees!) | 13:13 |
| enriquetaso | \ΓΈ/ | 13:13 |
| lbragstad | saw this merged last night https://review.opendev.org/c/openstack/cinder/+/809303 | 13:38 |
| lbragstad | nice work everyone - https://media.giphy.com/media/5xaOcLDE64VMF4LqqrK/giphy.gif | 13:38 |
| lbragstad | i think every openstack service now supports a consistent project-reader and project-member role :) | 13:39 |
| lbragstad | by default | 13:40 |
| enriquetaso | lbragstad++ | 13:44 |
| opendevreview | Merged openstack/cinder stable/stein: Correct group:reset_group_snapshot_status policy https://review.opendev.org/c/openstack/cinder/+/782125 | 13:56 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/rocky: Correct group:reset_group_snapshot_status policy https://review.opendev.org/c/openstack/cinder/+/809657 | 13:59 |
| rosmaita | reminder: festival of XS reviews at 1400 UTC | 14:01 |
| rosmaita | join us in meetpad: https://meetpad.opendev.org/cinder-festival-of-reviews | 14:01 |
| rosmaita | direct link to the etherpad: https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:01 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna ^^ | 14:01 |
| jungleboyj | rosmaita: Will join momentarily. | 14:02 |
| rosmaita | cool | 14:03 |
| jungleboyj | FYI, I am working on stable merges. | 14:11 |
| *** dviroel is now known as dviroel|mtg | 14:21 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder-specs master: Update "Secure RBAC Ready" spec https://review.opendev.org/c/openstack/cinder-specs/+/809741 | 14:23 |
| *** dviroel|mtg is now known as dviroel | 14:39 | |
| opendevreview | Luigi Toscano proposed openstack/cinder master: Add ceph-iscsi CI job https://review.opendev.org/c/openstack/cinder/+/667108 | 14:44 |
| opendevreview | Jay Bryant proposed openstack/cinder master: Fix typo in release note file https://review.opendev.org/c/openstack/cinder/+/809206 | 15:05 |
| opendevreview | Merged openstack/cinder stable/victoria: RBD: use correct stripe unit in clone operation https://review.opendev.org/c/openstack/cinder/+/808475 | 15:59 |
| opendevreview | Merged openstack/cinder stable/victoria: Change 'host' option from HostAddressOpt to StrOpt https://review.opendev.org/c/openstack/cinder/+/808473 | 15:59 |
| opendevreview | Merged openstack/python-cinderclient stable/victoria: Unset tempest.lib timeout in functional tests https://review.opendev.org/c/openstack/python-cinderclient/+/806697 | 16:02 |
| opendevreview | Merged openstack/python-cinderclient stable/victoria: [stable-only] Add missing classes to cinderclient.v3 https://review.opendev.org/c/openstack/python-cinderclient/+/806698 | 16:15 |
| opendevreview | Merged openstack/cinder master: Address new hacking enforcement https://review.opendev.org/c/openstack/cinder/+/757908 | 16:23 |
| opendevreview | OpenStack Release Bot proposed openstack/cinder stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/cinder/+/809826 | 16:24 |
| opendevreview | OpenStack Release Bot proposed openstack/cinder stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/cinder/+/809827 | 16:24 |
| opendevreview | OpenStack Release Bot proposed openstack/cinder master: Update master for stable/xena https://review.opendev.org/c/openstack/cinder/+/809828 | 16:24 |
| opendevreview | OpenStack Release Bot proposed openstack/cinder master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/cinder/+/809829 | 16:24 |
| opendevreview | Venkata krishna Thumu proposed openstack/cinder master: [SVF] Fix rccg and relationship creation issues https://review.opendev.org/c/openstack/cinder/+/809830 | 16:31 |
| opendevreview | Merged openstack/cinder master: Tests: Remove brick_lvm mocks from all unit tests https://review.opendev.org/c/openstack/cinder/+/806005 | 16:34 |
| opendevreview | Merged openstack/cinder master: KumoScale Driver replicated volume missing portals attaches without raid https://review.opendev.org/c/openstack/cinder/+/803450 | 16:35 |
| opendevreview | Merged openstack/cinder master: Mark QNAP driver as unsupported https://review.opendev.org/c/openstack/cinder/+/800830 | 17:08 |
| tosky | rosmaita: out of curiosity, are you thinking about bumping the branching hash again for xena after all those merges? :) | 17:10 |
| rosmaita | tosky: i was not, but guess i could take a look | 17:16 |
| rosmaita | too late | 17:17 |
| tosky | oh, well | 17:29 |
| opendevreview | Merged openstack/cinder master: Add iSCSI IPv6 support to Dell EMC XtremIO driver https://review.opendev.org/c/openstack/cinder/+/781009 | 17:35 |
| opendevreview | Merged openstack/cinder master: PowerMax Driver - Fix for GVG snapshot delete https://review.opendev.org/c/openstack/cinder/+/805228 | 17:35 |
| opendevreview | Merged openstack/cinder stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/cinder/+/809826 | 17:39 |
| opendevreview | Merged openstack/cinder stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/cinder/+/809827 | 17:39 |
| opendevreview | Merged openstack/cinder master: Update master for stable/xena https://review.opendev.org/c/openstack/cinder/+/809828 | 17:40 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Raise minimum oslo.log to 4.5.0 https://review.opendev.org/c/openstack/cinder/+/809891 | 18:26 |
| whoami-rajat | jungleboyj, thanks for the final stable/victoria reviews! | 18:36 |
| whoami-rajat | rosmaita, I've proposed victoria release, can you verify the hashes? https://review.opendev.org/c/openstack/releases/+/809892 | 18:36 |
| jungleboyj | whoami-rajat: No problem. :-) Thanks for getting the list to together. | 18:36 |
| rosmaita | will do | 18:36 |
| whoami-rajat | thanks jungleboyj rosmaita :) | 18:36 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Add deprecation notices to deprecated policy rules https://review.opendev.org/c/openstack/cinder/+/809894 | 19:18 |
| mloza | hello, I have two ceph clusters and I create cinder volumes with different types however, I can only attach first volume type to the instance, the second volume type throws an error | 19:52 |
| mloza | https://paste.opendev.org/raw/809420/ | 19:52 |
| mloza | here's what my cinder.conf looks like https://paste.opendev.org/show/809421/ | 19:54 |
| mloza | cinder secrets https://paste.opendev.org/show/809422/ | 19:56 |
| mloza | is it possible mix two ceph clusters on a compute or do I have setup availability zones? | 19:59 |
| *** dviroel is now known as dviroel|out | 20:31 | |
| opendevreview | Merged openstack/cinder master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/cinder/+/809829 | 21:01 |
| opendevreview | Merged openstack/cinder stable/wallaby: PowerMax Driver - Allow for case mismatch in SGs https://review.opendev.org/c/openstack/cinder/+/809195 | 21:34 |
| opendevreview | Merged openstack/cinder master: Remove unused config imports/objects https://review.opendev.org/c/openstack/cinder/+/805083 | 21:34 |
| pots | rosmaita Hi Brian, could I ask your advice about my driver patch https://review.opendev.org/c/openstack/cinder/+/808837? In order to close the referenced bug, it contains 3 fixes which could be broken into 2 XS and one larger but low-risk change. There's also a merge conflict with https://review.opendev.org/c/openstack/cinder/+/800704 (remove | 21:36 |
| pots | unused code). How should I proceed? | 21:36 |
| pots | rosmaita (All the relevant CIs are green and for extra credit I updated our CI to pass the -autogenerated option) | 21:38 |
| opendevreview | Merged openstack/cinder master: PowerMax Driver - Port load balancing fix https://review.opendev.org/c/openstack/cinder/+/808797 | 21:51 |
| rosmaita | pots: I'm out of the office until Monday morning New York time, but since you updated your CI to pass the autogenerated option, i'll answer now | 22:55 |
| rosmaita | i think it would be better to split up the patch so each one is more focused (and the doc fix only requires one +2) | 22:55 |
| rosmaita | don't worry about the merge conflict yet ... whose worry it is will depend on which patch is merged first | 22:56 |
| rosmaita | have a good weekend! | 22:57 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-specs master: Update "Secure RBAC Ready" spec https://review.opendev.org/c/openstack/cinder-specs/+/809741 | 23:08 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
