opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from zonemanager module https://review.opendev.org/c/openstack/cinder/+/907372 | 02:40 |
---|---|---|
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Fujitsu drivers https://review.opendev.org/c/openstack/cinder/+/907373 | 02:43 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from HPE drivers https://review.opendev.org/c/openstack/cinder/+/907374 | 02:44 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Veritas drivers https://review.opendev.org/c/openstack/cinder/+/907375 | 02:47 |
opendevreview | Pete Zaitcev proposed openstack/cinder master: tests: stop trying to cache database schema https://review.opendev.org/c/openstack/cinder/+/907376 | 03:04 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from zonemanager module https://review.opendev.org/c/openstack/cinder/+/907372 | 04:02 |
opendevreview | Pete Zaitcev proposed openstack/cinder master: Speed up starting cinder-backup https://review.opendev.org/c/openstack/cinder/+/657543 | 04:47 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from HPE drivers https://review.opendev.org/c/openstack/cinder/+/907374 | 05:57 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Infortrend driver https://review.opendev.org/c/openstack/cinder/+/907383 | 07:06 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from qnap driver tests https://review.opendev.org/c/openstack/cinder/+/907384 | 07:08 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from StorPool driver https://review.opendev.org/c/openstack/cinder/+/907385 | 07:13 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from SandStone driver https://review.opendev.org/c/openstack/cinder/+/907386 | 07:16 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Zadara driver https://review.opendev.org/c/openstack/cinder/+/907387 | 07:19 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from STX drivers https://review.opendev.org/c/openstack/cinder/+/907388 | 07:24 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Synology driver https://review.opendev.org/c/openstack/cinder/+/907389 | 07:27 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from ProphetStor driver https://review.opendev.org/c/openstack/cinder/+/907390 | 07:29 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from netapp drivers (1/2) https://review.opendev.org/c/openstack/cinder/+/907391 | 07:50 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from netapp drivers https://review.opendev.org/c/openstack/cinder/+/907391 | 07:57 |
opendevreview | XuQi proposed openstack/cinder master: Fujitsu driver: Improve volume deletion https://review.opendev.org/c/openstack/cinder/+/898199 | 08:29 |
opendevreview | Yian Zong proposed openstack/cinder master: Recognize Dell PowerMax Unisphere 10.x (x>0) https://review.opendev.org/c/openstack/cinder/+/900118 | 08:44 |
*** geguileo is now known as Guest1194 | 09:36 | |
*** akrog is now known as geguileo | 10:24 | |
*** elodilles is now known as elodilles_afk | 10:33 | |
opendevreview | Yian Zong proposed openstack/cinder master: Dell PowerStore driver: Add Cinder active-active support https://review.opendev.org/c/openstack/cinder/+/907133 | 10:43 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from FusionStorage driver https://review.opendev.org/c/openstack/cinder/+/907417 | 12:05 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Hedvig driver https://review.opendev.org/c/openstack/cinder/+/907418 | 12:06 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from test codes https://review.opendev.org/c/openstack/cinder/+/907384 | 12:09 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Insspur driver https://review.opendev.org/c/openstack/cinder/+/907419 | 12:13 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Nexenta drivers https://review.opendev.org/c/openstack/cinder/+/907420 | 12:17 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from VMWare drivers https://review.opendev.org/c/openstack/cinder/+/907421 | 12:19 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Huawei drivers https://review.opendev.org/c/openstack/cinder/+/907422 | 12:22 |
tkajinam | ^^^ I think I covered all of the drivers/modules with six still used. | 12:23 |
opendevreview | Takashi Kajinami proposed openstack/cinder master: Remove six from Inspur driver https://review.opendev.org/c/openstack/cinder/+/907419 | 12:34 |
rosmaita | tkajinam: i think you are duplicating these patches: https://review.opendev.org/q/topic:%22remove-six%22+owner:cyril@redhat.com | 12:49 |
rosmaita | I like your per-driver approach better, though | 12:49 |
rosmaita | but you might want to make cyril a co-author | 12:49 |
Luzi | rosmaita? i have a question about the extra_specs of volume types, do you have some time? | 13:45 |
rosmaita | Luzi: sure | 13:45 |
Luzi | there is an extra_spec for replication. I know that does not cover something like ceph automatically having multiple copies of a volume. What is is exactly used for? | 13:47 |
rosmaita | Luzi: volume replication is an optional feature that depends on backend capabilities reported by the driver. So if an operator defines a volume-type that enables replication (by adding it to the volume-type extra_specs), the scheduler can make volume-create requests for that type land on a backend that supports replication. | 13:52 |
Luzi | yeah i got to that point. But what is or should be the difference between using this feature in openstack vs using ceph native replication? | 13:54 |
Luzi | I am currently looking into the discoverability of some aspects of volume types, and it would be nice to also be able to discover such replication that only happens in the backend (as well as whether this type will create encrypted volumes) | 14:04 |
Luzi | but i wonder if the extra-specs and how the currently work might be the right place to achieve this | 14:04 |
rosmaita | Luzi: well, discoverability of backend capabilities has been a work in progress for a long time | 14:12 |
rosmaita | not all extra_specs are user visible because they can give away backend information | 14:12 |
Luzi | yeah I know | 14:12 |
rosmaita | but luckily replication_enabled is user visible since xena | 14:13 |
Luzi | for example all the encryption parameters are hidden from users... But would it be okay to add an extra_spec like "encryption_enabled" that just gives a boolean information for the user? As soon as an encryption type is added or removed? | 14:14 |
rosmaita | i thought there was a way for a user to know if the volume type has an encryption type ... probably should ask eharney | 14:22 |
*** elodilles_afk is now known as elodilles | 15:21 | |
opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: Add ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/900348 | 15:26 |
opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: Add ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/900348 | 15:27 |
opendevreview | Yian Zong proposed openstack/cinder master: Recognize Dell PowerMax Unisphere 10.x (x>0) https://review.opendev.org/c/openstack/cinder/+/900118 | 16:10 |
tkajinam | rosmaita, I can update the commit message if needed. | 16:10 |
tkajinam | rosmaita, I found these, and tried rebasing it first but noticed per-driver approach, which was already partially merged, caused the current conflict and there was already a conversation saying that approach is better, so I decided to create these new ones. | 16:10 |
tkajinam | eg. https://review.opendev.org/c/openstack/cinder/+/851042 | 16:10 |
opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: Add ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/900348 | 18:41 |
opendevreview | Maksim Malchuk proposed openstack/cinder master: Fix 'cinder-backup' service when Swift with TLS enabled https://review.opendev.org/c/openstack/cinder/+/907494 | 20:22 |
opendevreview | Bryan Neumann proposed openstack/cinder master: Dell EMC: PowerMax - Configurable SRDF snapshots https://review.opendev.org/c/openstack/cinder/+/899051 | 20:23 |
opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: Add ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/900348 | 21:05 |
opendevreview | Bryan Neumann proposed openstack/cinder master: Dell EMC: PowerMax - Configurable SRDF snapshots https://review.opendev.org/c/openstack/cinder/+/899051 | 21:26 |
*** tosky_ is now known as tosky | 23:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!