| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: support data deduplication and compression https://review.opendev.org/c/openstack/cinder/+/850831 | 01:46 |
|---|---|---|
| opendevreview | Rajat Dhasmana proposed openstack/python-cinderclient stable/yoga: [stable-only] Remove python-cinderclient-functional-py39 job https://review.opendev.org/c/openstack/python-cinderclient/+/874448 | 05:41 |
| Kumar_Kanishka | Hi all. I need a review for this patch https://review.opendev.org/c/openstack/cinder/+/869305 if anyone can spare some time. | 07:26 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: uploading test(5) https://review.opendev.org/c/openstack/cinder/+/869990 | 11:09 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: uploading test(3) https://review.opendev.org/c/openstack/cinder/+/869695 | 11:29 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: support data deduplication and compression https://review.opendev.org/c/openstack/cinder/+/850831 | 11:56 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: WIP RBD: Fix Restore Full Backups https://review.opendev.org/c/openstack/cinder/+/750782 | 12:08 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: DNM: Add CI Job For lvm+ceph With c-bak Enabled https://review.opendev.org/c/openstack/cinder/+/784815 | 12:09 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Replace empty mock.MagicMock() call with spec https://review.opendev.org/c/openstack/cinder/+/874569 | 12:16 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Fix typo'd unit test name https://review.opendev.org/c/openstack/cinder/+/865035 | 12:43 |
| rosmaita | whoami-rajat: left a comment for you on https://review.opendev.org/c/openstack/python-cinderclient/+/874448 | 13:15 |
| mnaser | is anyone here from the dell team which maintains the powerstore driver? | 15:11 |
| mnaser | REST Response: 422 with data {"messages":[{"code":"0xE0A040010003","severity":"Error","message_l10n":"The clone cannot be created because the max number of clones: 32, in a family has been reached.","arguments":["32"]}]} | 15:12 |
| mnaser | it looks like.. you cant have more than 32 clones for an image...? this is very crippling but i cant find any docs | 15:12 |
| *** JasonF is now known as JayF | 15:15 | |
| rosmaita | mnaser: i think Tony Saad is powerstore maintainer, not sure what his nick is ... maybe happystacker knows? | 15:49 |
| mnaser | rosmaita: yeah cause I feel like otherwise this should be documented. I’m trying to work with the customer and maybe have them try to reach out to dell but yeah. | 15:54 |
| rosmaita | mnaser: i wonder whether they're hitting this: https://review.opendev.org/c/openstack/cinder/+/867753 | 15:57 |
| rosmaita | merged kind of recently | 15:57 |
| mnaser | rosmaita: kinda? I did see a fallback happen into image cache but the problem was I think that cinder was trying to download the image from glance and they were building multiple volumes at once | 15:59 |
| mnaser | And so glance gave an internal server error cause I think it was overwhelmed since it was trying to download a bunch of 20G volumes in parallel | 16:00 |
| ganso | rosmaita, whoami-rajat: hi! patch is ready for approvals :) https://review.opendev.org/c/openstack/cinder/+/873145 | 16:02 |
| whoami-rajat | ganso, done, thanks for the update | 16:04 |
| opendevreview | Eric Harney proposed openstack/cinder master: Ceph backup: Remove leftover unicode comment https://review.opendev.org/c/openstack/cinder/+/874660 | 16:28 |
| opendevreview | Rajat Dhasmana proposed openstack/python-cinderclient stable/yoga: [stable-only] Skip backup test in functional py39 job https://review.opendev.org/c/openstack/python-cinderclient/+/874448 | 18:24 |
| whoami-rajat | rosmaita, what do you think about this? ^ | 18:24 |
| rosmaita | looking | 18:25 |
| whoami-rajat | unfortunately, I don't have c-bak installed in my system and can't actually test where gate is failing i.e. creating a backup | 18:25 |
| rosmaita | that's ok, we have coverage in the py3.6 tests | 18:26 |
| rosmaita | whoami-rajat: LGTM | 18:27 |
| whoami-rajat | rosmaita, apart from the backup tests, everything passed in my ubuntu focal env, not sure why your patch is failing with those keystone errors | 18:28 |
| whoami-rajat | also i needed to source the openrc file first | 18:28 |
| rosmaita | interesting | 18:28 |
| whoami-rajat | but i think that is done in gate as well, else other tests would fail | 18:28 |
| rosmaita | right | 18:28 |
| whoami-rajat | yeah, not sure what's going on | 18:31 |
| whoami-rajat | geguileo, hey, can you take a look at my comment here? https://review.opendev.org/c/openstack/cinder/+/839064 | 18:31 |
| geguileo | whoami-rajat: looking | 18:32 |
| whoami-rajat | i assume you first went with a boolean config option but changed it later to integer (1 and 2) but forgot to update it here | 18:32 |
| geguileo | whoami-rajat: good catch!! | 18:32 |
| geguileo | whoami-rajat: yes, Pete suggested the change and I though it was a good idea | 18:33 |
| geguileo | s/though/thought | 18:33 |
| whoami-rajat | geguileo, oh ok, in a previous patch in the series, I suggested a boolean option :P | 18:34 |
| whoami-rajat | but it was after the merge and I'm fine either way | 18:34 |
| geguileo | whoami-rajat: really? r:-??? | 18:34 |
| geguileo | I no longer forget | 18:34 |
| geguileo | s/forget/remember | 18:34 |
| whoami-rajat | https://review.opendev.org/c/openstack/cinder/+/836072/10/cinder/volume/driver.py#267 | 18:34 |
| whoami-rajat | it's recent but don't worry about it | 18:34 |
| geguileo | In my defense I pushed this almost 1 year ago... | 18:35 |
| geguileo | lol | 18:35 |
| whoami-rajat | :D | 18:36 |
| whoami-rajat | the series look good, I mean the code and detailed test coverage | 18:36 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: LVM nvmet: Add support for multiple ip addresses https://review.opendev.org/c/openstack/cinder/+/839064 | 18:40 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Add missing extend_target driver method https://review.opendev.org/c/openstack/cinder/+/843493 | 18:40 |
| geguileo | whoami-rajat: ^ done | 18:40 |
| geguileo | whoami-rajat: thanks for the ping | 18:40 |
| geguileo | and the review | 18:40 |
| opendevreview | Gorka Eguileor proposed openstack/os-brick master: WIP: Support non SAM LUN addressing https://review.opendev.org/c/openstack/os-brick/+/874689 | 19:23 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Pure: Report SAM-2 addressing mode for LUNs https://review.opendev.org/c/openstack/cinder/+/874690 | 19:23 |
| geguileo | simondodsley: ^ If we are lucky those should work for iSCSI and FC | 19:24 |
| geguileo | simondodsley: I'll await your validation | 19:24 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: Fix NetApp NFS driver to never spawn a native thread again https://review.opendev.org/c/openstack/cinder/+/853678 | 20:37 |
| mnaser | i filed https://bugs.launchpad.net/cinder/+bug/2008024 for the dell folks | 21:36 |
| opendevreview | Kumar Kanishka proposed openstack/cinder master: [SVf] Invalid Exception raised due to invalid portset value https://review.opendev.org/c/openstack/cinder/+/874633 | 22:32 |
| opendevreview | Kumar Kanishka proposed openstack/cinder master: [SVf] Invalid Exception raised due to invalid portset value https://review.opendev.org/c/openstack/cinder/+/874633 | 22:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!