| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Improve the function of CLI. https://review.opendev.org/c/openstack/cinder/+/854070 | 02:45 |
|---|---|---|
| opendevreview | Prajakta proposed openstack/cinder stable/yoga: Doc: To update the portset feature details in IBM Spectrum Virtualize user guide https://review.opendev.org/c/openstack/cinder/+/853788 | 04:46 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Improve the function of CLI. https://review.opendev.org/c/openstack/cinder/+/854070 | 05:19 |
| *** tkajinam|off is now known as tkajinam | 05:48 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: nit: correct storwize_portset help text https://review.opendev.org/c/openstack/cinder/+/854092 | 06:26 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Improve the function of CLI. https://review.opendev.org/c/openstack/cinder/+/854070 | 06:37 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Improve the function of CLI. https://review.opendev.org/c/openstack/cinder/+/854070 | 07:20 |
| whoami-rajat | geguileo, hey, any patches you would like to include in os-brick release on Thursday (if it's in a review-able state)? | 08:49 |
| geguileo | whoami-rajat: I have 2, but I need to update them with eharney's suggestion | 08:52 |
| whoami-rajat | geguileo, oh ok, and will you be able to update them by today/tomorrow? since we've less time ... | 08:53 |
| geguileo | whoami-rajat: yeah, I'll finish writing an email and I'll update them | 08:55 |
| whoami-rajat | geguileo, great thanks! | 08:56 |
| whoami-rajat | geguileo, can you take a look at this mypy patch? it is blocking the series from merging https://review.opendev.org/c/openstack/os-brick/+/849021 | 09:05 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: Get system uuid in privsep https://review.opendev.org/c/openstack/os-brick/+/836057 | 09:47 |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver generic volume migration https://review.opendev.org/c/openstack/cinder/+/850320 | 10:15 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: NVMe-oF: read mdstat in Python https://review.opendev.org/c/openstack/os-brick/+/836056 | 10:18 |
| geguileo | whoami-rajat: those are my 2 brick patches I mentioned earlier | 10:19 |
| whoami-rajat | geguileo, ack, will take a look | 10:20 |
| geguileo | whoami-rajat: thanks | 10:31 |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver generic volume migration https://review.opendev.org/c/openstack/cinder/+/850320 | 11:02 |
| opendevreview | Ke Niu proposed openstack/cinderlib master: remove unicode prefix from code https://review.opendev.org/c/openstack/cinderlib/+/854153 | 11:07 |
| *** dviroel is now known as dviroel}rover | 11:26 | |
| *** dviroel}rover is now known as dviroel|rover | 11:26 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/ussuri: Don't destroy existing backup by mistake on import https://review.opendev.org/c/openstack/cinder/+/854133 | 12:58 |
| opendevreview | Merged openstack/os-brick master: mypy: initiator/connectors/rbd.py https://review.opendev.org/c/openstack/os-brick/+/849021 | 14:43 |
| *** dviroel|rover is now known as dviroel|rover|lunch | 14:59 | |
| opendevreview | Eric Harney proposed openstack/os-brick master: Move mypy job to check queue (non-voting) https://review.opendev.org/c/openstack/os-brick/+/849840 | 15:03 |
| whoami-rajat | eharney, reg mypy patches, i guess we will need to rebase all the patches in the chain and that will remove the workflow, i think we can rebase all at once and +W the ones already approved rather than doing this one by one? | 15:11 |
| eharney | whoami-rajat: yeah, was trying to figure out if i need to rebase them all or not, i guess i will | 15:11 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: set no_implicit_optional https://review.opendev.org/c/openstack/os-brick/+/852824 | 15:11 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: os_brick/encryptors/__init__.py https://review.opendev.org/c/openstack/os-brick/+/853457 | 15:12 |
| whoami-rajat | eharney, cool thanks | 15:12 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator/linuxrbd https://review.opendev.org/c/openstack/os-brick/+/853458 | 15:13 |
| eharney | whoami-rajat: done | 15:13 |
| whoami-rajat | eharney, do they need to be rebased as well? https://review.opendev.org/c/openstack/os-brick/+/786733/8 and https://review.opendev.org/c/openstack/os-brick/+/853789/1 ? | 15:14 |
| eharney | whoami-rajat: yes but those hadn't really gotten review yet, so i wasn't worried about them for today | 15:15 |
| eharney | oh, never mind, 786733 did indeed get reviewed | 15:15 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: lvm.py https://review.opendev.org/c/openstack/os-brick/+/786733 | 15:16 |
| whoami-rajat | great | 15:16 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: improve executor coverage https://review.opendev.org/c/openstack/os-brick/+/853789 | 15:16 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: Update format to future __annotations__ https://review.opendev.org/c/openstack/os-brick/+/853778 | 15:21 |
| whoami-rajat | rosmaita, hey, if you're interested in reviewing Gorka's changes (they're not too time consuming), here they are | 15:27 |
| whoami-rajat | 1) https://review.opendev.org/c/openstack/os-brick/+/836057 | 15:27 |
| whoami-rajat | 2) https://review.opendev.org/c/openstack/os-brick/+/836056 | 15:27 |
| rosmaita | whoami-rajat: ack | 15:31 |
| *** dviroel|rover|lunch is now known as dviroel|rover | 16:27 | |
| andrei | hi, Im getting iscsiadm error: iscsiadm config file line 336 do not has value. when volume should be mounted. And I have now idea on how to debug it or investigate in any other way | 16:46 |
| andrei | that is a centos9 | 16:47 |
| andrei | on a latest devstack | 16:47 |
| opendevreview | Luisa proposed openstack/cinder master: NetApp ONTAP: Implemented core functions on REST API https://review.opendev.org/c/openstack/cinder/+/852208 | 16:49 |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver multi-attach feature https://review.opendev.org/c/openstack/cinder/+/850321 | 17:57 |
| opendevreview | Merged openstack/os-brick master: Move mypy job to check queue (non-voting) https://review.opendev.org/c/openstack/os-brick/+/849840 | 18:57 |
| opendevreview | Merged openstack/os-brick master: mypy: set no_implicit_optional https://review.opendev.org/c/openstack/os-brick/+/852824 | 18:57 |
| opendevreview | Merged openstack/os-brick master: mypy: os_brick/encryptors/__init__.py https://review.opendev.org/c/openstack/os-brick/+/853457 | 18:59 |
| opendevreview | Merged openstack/os-brick master: mypy: Update format to future __annotations__ https://review.opendev.org/c/openstack/os-brick/+/853778 | 19:15 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix service stopping on cleanup https://review.opendev.org/c/openstack/cinder/+/854259 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Monkey patch before logs are loaded https://review.opendev.org/c/openstack/cinder/+/854260 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix test runner never finishing https://review.opendev.org/c/openstack/cinder/+/854261 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix inspur unit test https://review.opendev.org/c/openstack/cinder/+/854262 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: fix quobyte breaking other tests https://review.opendev.org/c/openstack/cinder/+/854263 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix 3par https://review.opendev.org/c/openstack/cinder/+/854264 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix IBM XIV https://review.opendev.org/c/openstack/cinder/+/854265 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix zone manager tests https://review.opendev.org/c/openstack/cinder/+/854266 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix Pure test https://review.opendev.org/c/openstack/cinder/+/854267 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix NetApp tests https://review.opendev.org/c/openstack/cinder/+/854268 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix NFS tests https://review.opendev.org/c/openstack/cinder/+/854269 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix cmd tests https://review.opendev.org/c/openstack/cinder/+/854270 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix PowerMax tests https://review.opendev.org/c/openstack/cinder/+/854271 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Fix versions view https://review.opendev.org/c/openstack/cinder/+/854272 | 20:10 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Tests: Randomize tests https://review.opendev.org/c/openstack/cinder/+/854273 | 20:11 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Ceph backup: Close source file https://review.opendev.org/c/openstack/cinder/+/854274 | 20:11 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: RBD: Ensure that everything runs in native threads https://review.opendev.org/c/openstack/cinder/+/854275 | 20:11 |
| geguileo | whoami-rajat: eharney rosmaita abishop ^ that's the unit test stuff I've been whining about | 20:11 |
| eharney | geguileo: looks exciting | 20:11 |
| geguileo | eharney: lol | 20:11 |
| rosmaita | geguileo: looks exhausting | 20:11 |
| geguileo | rosmaita: yeah, it has been | 20:11 |
| geguileo | Anybody having troubles opening the meetings etherpad? https://etherpad.opendev.org/p/cinder-zed-meetings | 20:12 |
| eharney | opens for me | 20:13 |
| rosmaita | me too | 20:13 |
| geguileo | chrome is awesome... it only opens in incognito mode for me now | 20:13 |
| geguileo | eharney: rosmaita thanks | 20:13 |
| andrei | hi, Im getting iscsiadm error: iscsiadm config file line 336 do not has value. when volume should be mounted. And I have now idea on how to debug it or investigate in any other way. that is a centos9. on a latest devstack | 20:13 |
| opendevreview | Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Implemented volume replication functions on REST client https://review.opendev.org/c/openstack/cinder/+/852209 | 20:17 |
| geguileo | andrei: have you looked at /etc/iscsi/iscsid.conf ? | 20:18 |
| andrei | yes, currently trying to turn on debbuging on iscsid | 20:22 |
| andrei | gegulio:^ | 20:22 |
| geguileo | andrei: but you don't do that in that file | 20:27 |
| geguileo | andrei: the complaint if because that file seems to have something wrong | 20:27 |
| andrei | geguileo: right, ave looked through conf file and was not able to figure out anything. there are 337 lines and 336 is: [DEFAULT] | 20:28 |
| andrei | node.session.auth.chap_algs = SHA3-256,SHA256 | 20:28 |
| geguileo | andrei: [DEFAULT] like that? | 20:34 |
| geguileo | If you have a line with "[DEFAULT]" then afaik that is wrong, that isn't a valid format | 20:35 |
| geguileo | so that could be the issue | 20:35 |
| opendevreview | Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Implemented volume migration functions on REST client https://review.opendev.org/c/openstack/cinder/+/852210 | 20:44 |
| opendevreview | Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Implemented revert to snapshot functions on REST client https://review.opendev.org/c/openstack/cinder/+/852211 | 20:44 |
| andrei | geguileo: hm, strange. looks like you are correct | 20:46 |
| andrei | but that is a default iscsi.conf. I have not touched it | 20:47 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: RBD: Improve IOWrapper's close method https://review.opendev.org/c/openstack/os-brick/+/854281 | 21:03 |
| geguileo | andrei: I don't know why it's like that, but you can just remove it | 21:04 |
| geguileo | eharney: ^ that os-brick patch together with https://review.opendev.org/c/openstack/cinder/+/854274/1 fixes the issue where a backup test blocks cinder tests | 21:06 |
| eharney | geguileo: cool | 21:08 |
| andrei | geguileo: yep, for some magic reason everything is working now... many thanks! | 21:21 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: Add volume dependency test coverage https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/854287 | 21:40 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Flattening of child volumes during deletion https://review.opendev.org/c/openstack/cinder/+/835384 | 21:46 |
| *** dviroel|rover is now known as dviroel|out | 22:31 | |
| opendevreview | Merged openstack/os-brick master: NVMe-oF: read mdstat in Python https://review.opendev.org/c/openstack/os-brick/+/836056 | 23:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!