| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Make the parameters name variable for supporting OEM storages https://review.opendev.org/c/openstack/cinder/+/815461 | 00:10 |
|---|---|---|
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Make the parameters name variable for supporting OEM storages https://review.opendev.org/c/openstack/cinder/+/815461 | 00:11 |
| opendevreview | 王昊 proposed openstack/cinder master: Support to query snapshot filter by updated_at/created_at https://review.opendev.org/c/openstack/cinder/+/733062 | 01:24 |
| opendevreview | Tushar Trambak Gite proposed openstack/python-cinderclient master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/python-cinderclient/+/819259 | 06:04 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVF] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/809518 | 06:39 |
| opendevreview | huangbingyan proposed openstack/cinder master: fix 'huawei dorado v6' create new volume from snapshot error https://review.opendev.org/c/openstack/cinder/+/819790 | 07:24 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: Move nimble driver code to hpe folder https://review.opendev.org/c/openstack/cinder/+/786054 | 07:36 |
| opendevreview | ZhaoYixin proposed openstack/cinder master: Add failure retry during backup restore https://review.opendev.org/c/openstack/cinder/+/820117 | 07:40 |
| opendevreview | huangbingyan proposed openstack/cinder master: fix 'huawei dorado v6' FC initiator can be added to host https://review.opendev.org/c/openstack/cinder/+/819778 | 08:18 |
| opendevreview | Tushar Trambak Gite proposed openstack/python-cinderclient master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/python-cinderclient/+/819259 | 08:19 |
| TusharTgite | hi team , need a review on these patches https://review.opendev.org/c/openstack/cinder-specs/+/819262 , https://review.opendev.org/c/openstack/cinderlib/+/819258 , https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/819263 | 08:22 |
| geguileo | rosmaita: remember I said I could do the quota discussion on the next meeting? Turns out next wednesday is a public holiday here in Spain, so I won't be available :-( | 09:06 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVF]:Fix multiple lsiogrp,lsvdisk calls in Retype. https://review.opendev.org/c/openstack/cinder/+/813532 | 10:52 |
| *** dviroel|out is now known as dviroel | 11:01 | |
| haailani | Please review the change: https://review.opendev.org/c/openstack/cinder/+/813532 | 11:15 |
| haailani | Please review the changes: https://review.opendev.org/c/openstack/cinder/+/811877 | 11:16 |
| opendevreview | Fábio Oliveira proposed openstack/cinder master: NetApp SolidFire: Fix driver startup with OSProfiler https://review.opendev.org/c/openstack/cinder/+/802955 | 11:54 |
| opendevreview | Fábio Oliveira proposed openstack/cinder master: DNM: Fix SolidFire init after setup change https://review.opendev.org/c/openstack/cinder/+/819007 | 11:55 |
| opendevreview | Fábio Oliveira proposed openstack/cinder master: NetApp SolidFire: Fix init after do_setup call https://review.opendev.org/c/openstack/cinder/+/819007 | 12:34 |
| opendevreview | Lucas Morais de oliveira proposed openstack/cinder master: api-ref: include links on backups list and details https://review.opendev.org/c/openstack/cinder/+/820164 | 12:38 |
| rosmaita | geguileo: i forgot you had already mentioned the holiday, let's come up with an alternative, maybe a half-hour video meeting on Tuesday or Thursday or Friday next week, 1400-1430 utc? (I think you said the holidays are Monday and Wednesday) | 13:22 |
| rosmaita | let me know if any of those times are good for you, i can send out a poll if more than 1 will work | 13:22 |
| rosmaita | geguileo: btw, xena cinderlib release must happen by 16 Dec | 13:23 |
| geguileo | rosmaita: I'm ok with Tuesday, Thursday, and Friday | 13:39 |
| rosmaita | geguileo: ack | 13:39 |
| rosmaita | geguileo: when you have a minute, do we care about this stable/train cinderlib patch, or should i just abandon it? https://review.opendev.org/c/openstack/cinderlib/+/771091 | 13:55 |
| opendevreview | Merged openstack/cinder master: Improve no_snapshot_gb_quota description https://review.opendev.org/c/openstack/cinder/+/820014 | 14:00 |
| *** whoami-rajat__ is now known as whoami-rajat | 14:00 | |
| whoami-rajat | rosmaita, i think the default volume type APIs are good (and only) candidate for the project-manager role | 14:31 |
| rosmaita | whoami-rajat: yeah, i think that's right | 14:34 |
| whoami-rajat | cool, taking inspiration from glance for pro-active work :D | 14:35 |
| *** akekane__ is now known as abhishekk | 14:58 | |
| lmcgann | Hello, Im new to some of the technology here and have a quick question. Does cinder require the ceph pools to be replicated like in a RAID1 or can cinder also control a ceph storage which is 4+2 erasure encoded? | 15:27 |
| opendevreview | Bogdan Dobrelya proposed openstack/cinder master: Set service specific rpc control_exchange defaults https://review.opendev.org/c/openstack/cinder/+/820189 | 15:55 |
| *** tosky_ is now known as tosky | 16:17 | |
| opendevreview | Bogdan Dobrelya proposed openstack/cinder master: Set service specific rpc control_exchange defaults https://review.opendev.org/c/openstack/cinder/+/820189 | 16:29 |
| *** priteau is now known as Guest7388 | 16:38 | |
| *** priteau_ is now known as priteau | 16:38 | |
| opendevreview | Bogdan Dobrelya proposed openstack/cinder master: Set service specific rpc control_exchange defaults https://review.opendev.org/c/openstack/cinder/+/820189 | 16:48 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: WIP: Quota: New quota system https://review.opendev.org/c/openstack/cinder/+/819691 | 17:55 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: WIP: Clean quotas on type delete https://review.opendev.org/c/openstack/cinder/+/819692 | 17:55 |
| fabiooliveira | hello, I am in need of some reviews on these patches https://review.opendev.org/c/openstack/cinder/+/802955 and https://review.opendev.org/c/openstack/cinder/+/819007 | 17:57 |
| fabiooliveira | they are required to have SolidFire working after the change to refactor the setup method on drivers base file | 17:57 |
| fabiooliveira | this is the PTG change I am referring to https://review.opendev.org/c/openstack/cinder/+/815659/ | 18:00 |
| fabiooliveira | geguileo rosmaita | 18:01 |
| geguileo | fabiooliveira: is my patch breaking your driver? | 18:02 |
| fabiooliveira | geguileo: kind of, I added a comment talking about it, but after the changes being merged its okay | 18:04 |
| geguileo | fabiooliveira: my patch shouldn't break your driver, if it does it may also break other drivers... | 18:05 |
| geguileo | fabiooliveira: from what I can see the problem is that the driver isn't properly implementing the __getattr__ | 18:09 |
| geguileo | and that's why the getattr call fails with an infinite loop | 18:10 |
| geguileo | fabiooliveira: the method is raising NotImplementedError which is wrong, it should be raising AttributeError | 18:11 |
| geguileo | fabiooliveira: that should be the proper fix | 18:12 |
| fabiooliveira | geguileo: yes, we already have a patch addressed to prevent the infinite loop issue | 18:15 |
| fabiooliveira | > the method is raising NotImplementedError which is wrong, it should be raising AttributeError | 18:19 |
| fabiooliveira | the fix on the second patch is to have a do_setup implemented, old driver base file had the do_setup with a pass inside | 18:19 |
| fabiooliveira | we can change the error being raised also | 18:19 |
| geguileo | fabiooliveira: I'll change your driver in my patch and see if that fixes the issue... In case there are other reasons why my patch fails | 18:23 |
| geguileo | fabiooliveira: since you are paying attention to your driver's result I'll use you as a guinea pig ;-) | 18:23 |
| geguileo | s/use you/use your driver | 18:24 |
| fabiooliveira | ok hahaha | 18:25 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Join do_setup and check_for_setup_error https://review.opendev.org/c/openstack/cinder/+/815659 | 18:33 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Docs: Document clone_image driver method https://review.opendev.org/c/openstack/cinder/+/815660 | 18:33 |
| geguileo | fabiooliveira: let's wait and see if the AttributeError change fixes it | 18:33 |
| * fabiooliveira 👀 | 18:37 | |
| *** dviroel is now known as dviroel|afk | 19:53 | |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: annotate volume/api.py https://review.opendev.org/c/openstack/cinder/+/733622 | 19:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!