| opendevreview | Keigo Noha proposed openstack/os-brick stable/train: multipath/iscsi: iSCSI connections are not reinitiated after reboot https://review.opendev.org/c/openstack/os-brick/+/829047 | 01:06 |
|---|---|---|
| opendevreview | Merged openstack/devstack-plugin-ceph master: Disable horizon from basic tempest tests https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/828910 | 03:06 |
| opendevreview | Sam Morrison proposed openstack/cinder master: Restore volumes from host in the same AZ as target volume https://review.opendev.org/c/openstack/cinder/+/816104 | 03:36 |
| opendevreview | Merged openstack/cinder master: [Lightos] standardize connector usage https://review.opendev.org/c/openstack/cinder/+/828085 | 03:46 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Add Target Port Assignment for VSP Driver https://review.opendev.org/c/openstack/cinder/+/828060 | 04:03 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Add port scheduler https://review.opendev.org/c/openstack/cinder/+/828696 | 04:04 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Add port scheduler https://review.opendev.org/c/openstack/cinder/+/828696 | 04:30 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 06:37 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 06:40 |
| opendevreview | Lior Friedman proposed openstack/os-brick master: nvmeof connector check controller already connected https://review.opendev.org/c/openstack/os-brick/+/811718 | 08:04 |
| opendevreview | yuval proposed openstack/os-brick master: nvmeof.py - in case of missing "show-hostnqn" command https://review.opendev.org/c/openstack/os-brick/+/823654 | 08:58 |
| harshailani | Hi reviewers, requesting you to please review the patch: https://review.opendev.org/c/openstack/cinder/+/817351 | 09:24 |
| harshailani | I have addressed the comments from @whoami-rajat | 09:25 |
| opendevreview | yuval proposed openstack/os-brick master: nvmeof.py - in case of missing "show-hostnqn" command https://review.opendev.org/c/openstack/os-brick/+/823654 | 09:38 |
| opendevreview | yuval proposed openstack/os-brick master: nvmeof.py - in case of missing "show-hostnqn" command https://review.opendev.org/c/openstack/os-brick/+/823654 | 09:40 |
| opendevreview | yuval proposed openstack/os-brick master: nvmeof.py - in case of missing "show-hostnqn" command https://review.opendev.org/c/openstack/os-brick/+/823654 | 09:46 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 11:07 |
| *** dviroel|out is now known as dviroel | 11:19 | |
| *** carloss is now known as carloss|afk | 11:20 | |
| opendevreview | yuval proposed openstack/os-brick master: nvmeof.py - in case of missing "show-hostnqn" command https://review.opendev.org/c/openstack/os-brick/+/823654 | 12:27 |
| *** carloss|afk is now known as carloss | 13:25 | |
| harshailani | Hi reviewers, requesting you to please review the patch: https://review.opendev.org/c/openstack/cinder/+/817351 | 13:38 |
| harshailani | I have addressed the comments from @whoami-rajat | 13:38 |
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 13:42 |
| TusharTgite | rosmaita: updated your comment take alook zuul test r passed https://review.opendev.org/c/openstack/cinder/+/773985 | 13:56 |
| rosmaita | TusharTgite: ack | 13:56 |
| opendevreview | yuval proposed openstack/os-brick master: Failure to generate hostnqn in case missing "show-hostnqn" sub-command https://review.opendev.org/c/openstack/os-brick/+/823654 | 14:09 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status https://review.opendev.org/c/openstack/cinder/+/778193 | 14:26 |
| TusharTgite | rosmaita: shall i really abandon this change https://review.opendev.org/c/openstack/cinder/+/799631 or we procees with it | 14:43 |
| TusharTgite | proceed * | 14:44 |
| geguileo | rosmaita: fun times, https://review.opendev.org/c/openstack/os-brick/+/800014 introduced a regression/bug | 14:52 |
| geguileo | rosmaita: eharney whoami-rajat abishop remember how I was mentioning the other day that it was weird that nvme didn't need to disconnect on os-brick and that just removing the export/ACLs was enough? | 14:53 |
| geguileo | turns out it's not enough | 14:53 |
| geguileo | [10790.468902] nvme nvme0: Connect rejected: status 8 (invalid service ID). | 14:53 |
| rosmaita | geguileo: arrgh | 14:53 |
| geguileo | [10790.472841] nvme nvme0: rdma connection establishment failed (-104) | 14:54 |
| geguileo | [10790.476177] nvme nvme0: Failed reconnect attempt 21 | 14:54 |
| geguileo | [10790.477963] nvme nvme0: Reconnecting in 10 seconds... | 14:54 |
| geguileo | the device is leftover | 14:54 |
| geguileo | I believe nvme may end up removing it after a while | 14:54 |
| geguileo | but we need the disconnect call that was removed in that patch | 14:55 |
| geguileo | although an improved version that targets a single device instead of the whole subsystem like we were doing before | 14:55 |
| geguileo | rosmaita: I'm testing it with RDMA, will test with TCP in a bit | 14:57 |
| rosmaita | geguileo: ack | 14:58 |
| geguileo | rosmaita: confirmed, the device gets removed after the kernel tries to reconnect 60 times | 15:01 |
| geguileo | so in 10 minutes | 15:01 |
| rosmaita | looks like we have a "leisure disconnect" ... why is everyone in such a hurry nowadays? Relax with the cinder os-brick leisure disconnect. | 15:05 |
| *** dviroel is now known as dviroel|lunch | 15:37 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for volume os-reset_status https://review.opendev.org/c/openstack/cinder/+/773985 | 16:48 |
| TusharTgite | rosmaita1: updated the changes take a look https://review.opendev.org/c/openstack/cinder/+/773985 | 16:50 |
| rosmaita1 | TusharTgite: ack | 16:50 |
| *** rosmaita1 is now known as rosmaita | 16:50 | |
| *** dviroel|lunch is now known as dviroel | 16:52 | |
| geguileo | rosmaita: oh, it gets worse... until the kernel has given up trying to connect to the disconnected volume we cannot attach it again | 17:05 |
| geguileo | so basically we have to wait 10 minutes before an nvme volume can be reconnected to the same host | 17:05 |
| geguileo | fuuuun | 17:05 |
| rosmaita | yeah, that's not good | 17:06 |
| sreerammounika | Hi reviewers, Please help to review my commit https://review.opendev.org/c/openstack/cinder/+/823015. I have addressed all the comments. | 18:24 |
| *** dviroel is now known as dviroel|out | 21:35 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!