*** Dinesh_Bhor has joined #openstack-cinder | 00:34 | |
*** LiangFang has joined #openstack-cinder | 00:36 | |
*** lixiaoy1 has joined #openstack-cinder | 00:39 | |
LiangFang | Thanks Sean | 00:41 |
---|---|---|
*** openstackgerrit has joined #openstack-cinder | 00:46 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Base framework for cinder-status upgrade check https://review.openstack.org/610581 | 00:46 |
*** geguileo_PTO has quit IRC | 00:49 | |
openstackgerrit | Merged openstack/cinder master: SF ensure the correct volume is deleted https://review.openstack.org/584149 | 00:52 |
openstackgerrit | wang yong proposed openstack/cinder master: Cinder volume driver for Inspur AS13000 series https://review.openstack.org/562977 | 00:54 |
openstackgerrit | Merged openstack/os-brick master: Fix a spelling mistake https://review.openstack.org/610450 | 00:56 |
*** TuanDA has joined #openstack-cinder | 01:14 | |
*** Dinesh_Bhor has quit IRC | 01:27 | |
*** Dinesh_Bhor has joined #openstack-cinder | 01:35 | |
*** openstackgerrit has quit IRC | 01:35 | |
*** dillaman has joined #openstack-cinder | 01:52 | |
*** mhen has quit IRC | 01:52 | |
*** jdillaman has quit IRC | 01:52 | |
*** openstackgerrit has joined #openstack-cinder | 02:01 | |
openstackgerrit | zhufl proposed openstack/cinder master: api-ref: fix some inconsistencies in snapshot API https://review.openstack.org/609611 | 02:01 |
*** Dinesh_Bhor has quit IRC | 02:02 | |
openstackgerrit | Dai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver https://review.openstack.org/610645 | 02:06 |
openstackgerrit | Dai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver https://review.openstack.org/610645 | 02:08 |
openstackgerrit | Dai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver https://review.openstack.org/610645 | 02:09 |
openstackgerrit | Dai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver https://review.openstack.org/610645 | 02:11 |
openstackgerrit | zhufl proposed openstack/cinder master: api-ref: fix some inconsistencies in snapshot API https://review.openstack.org/609611 | 02:14 |
*** rosmaita has left #openstack-cinder | 02:51 | |
*** Dinesh_Bhor has joined #openstack-cinder | 02:55 | |
*** dave-mccowan has quit IRC | 03:08 | |
*** Dinesh_Bhor has quit IRC | 03:36 | |
*** Bhujay has joined #openstack-cinder | 03:51 | |
*** bswartz has quit IRC | 03:51 | |
*** Bhujay has quit IRC | 03:52 | |
*** lpetrut has joined #openstack-cinder | 03:52 | |
*** Bhujay has joined #openstack-cinder | 03:52 | |
*** Bhujay has quit IRC | 04:06 | |
*** Dinesh_Bhor has joined #openstack-cinder | 04:30 | |
*** lpetrut has quit IRC | 04:34 | |
*** dpawlik has joined #openstack-cinder | 05:55 | |
*** dpawlik has quit IRC | 06:00 | |
*** Luzi has joined #openstack-cinder | 06:01 | |
*** dpawlik has joined #openstack-cinder | 06:11 | |
*** dpawlik has quit IRC | 06:12 | |
*** dpawlik has joined #openstack-cinder | 06:12 | |
*** Dinesh_Bhor has quit IRC | 06:16 | |
*** edisonxiang has joined #openstack-cinder | 06:44 | |
*** e0ne has joined #openstack-cinder | 06:48 | |
*** tommylikehu has joined #openstack-cinder | 06:54 | |
*** rcernin has quit IRC | 07:01 | |
*** lpetrut has joined #openstack-cinder | 07:20 | |
*** lpetrut has quit IRC | 07:25 | |
*** markvoelker has quit IRC | 07:29 | |
*** markvoelker has joined #openstack-cinder | 07:29 | |
*** Dinesh_Bhor has joined #openstack-cinder | 07:33 | |
*** mvkr has quit IRC | 07:34 | |
*** markvoelker has quit IRC | 07:34 | |
*** edisonxiang has quit IRC | 07:39 | |
*** icey has quit IRC | 07:52 | |
*** icey has joined #openstack-cinder | 07:52 | |
openstackgerrit | David Vallee Delisle proposed openstack/cinder-specs master: cinder-manage volume delete --from-db https://review.openstack.org/610895 | 08:10 |
*** mvkr has joined #openstack-cinder | 08:14 | |
openstackgerrit | Maciej Szwed proposed openstack/cinder master: Adding SPDK volume driver https://review.openstack.org/564229 | 08:21 |
openstackgerrit | Maciej Szwed proposed openstack/cinder master: SPDK drivers documentation https://review.openstack.org/609276 | 08:21 |
*** yikun has quit IRC | 08:22 | |
*** Reepicheep has quit IRC | 08:23 | |
*** markvoelker has joined #openstack-cinder | 08:30 | |
*** Reepicheep has joined #openstack-cinder | 08:30 | |
*** mvkr has quit IRC | 08:35 | |
*** e0ne has quit IRC | 08:37 | |
*** gkadam has joined #openstack-cinder | 08:38 | |
*** gkadam has quit IRC | 08:38 | |
*** e0ne has joined #openstack-cinder | 08:44 | |
*** mvkr has joined #openstack-cinder | 08:48 | |
*** lixiaoy1 has quit IRC | 08:54 | |
*** markvoelker has quit IRC | 09:03 | |
*** tommylikehu has quit IRC | 09:33 | |
*** mvkr has quit IRC | 09:40 | |
*** mvkr has joined #openstack-cinder | 09:41 | |
*** mvpnitesh has joined #openstack-cinder | 09:43 | |
*** mvpnitesh has quit IRC | 09:48 | |
*** imacdonn has quit IRC | 09:54 | |
*** imacdonn has joined #openstack-cinder | 09:55 | |
*** TuanDA has quit IRC | 10:00 | |
*** markvoelker has joined #openstack-cinder | 10:00 | |
*** chrisyang_0660 has quit IRC | 10:04 | |
*** luizbag has joined #openstack-cinder | 10:06 | |
*** erlon has joined #openstack-cinder | 10:07 | |
*** Dinesh_Bhor has quit IRC | 10:20 | |
*** LiangFang has quit IRC | 10:32 | |
*** markvoelker has quit IRC | 10:34 | |
*** brinzhang has joined #openstack-cinder | 10:47 | |
*** ganso has joined #openstack-cinder | 10:51 | |
*** Dinesh_Bhor has joined #openstack-cinder | 10:54 | |
*** Dinesh_Bhor has quit IRC | 11:06 | |
*** e0ne has quit IRC | 11:18 | |
*** markvoelker has joined #openstack-cinder | 11:31 | |
*** e0ne has joined #openstack-cinder | 11:32 | |
*** brinzhang has quit IRC | 11:38 | |
*** abishop has joined #openstack-cinder | 11:38 | |
*** jistr is now known as jistr|afk | 11:47 | |
*** lemko has joined #openstack-cinder | 11:54 | |
*** markvoelker has quit IRC | 12:04 | |
*** e0ne has quit IRC | 12:24 | |
openstackgerrit | Yikun Jiang proposed openstack/cinder master: WIP: volume reimage https://review.openstack.org/606346 | 12:26 |
*** e0ne has joined #openstack-cinder | 12:27 | |
jbernard | smcginnis: i agree, im not aware of any mechanism to store persitent driver metadata per volume | 12:37 |
*** rosmaita has joined #openstack-cinder | 12:46 | |
openstackgerrit | Yikun Jiang proposed openstack/cinder master: Raise the ImageTooBig exception when found it https://review.openstack.org/610950 | 12:54 |
smcginnis | jbernard: We have the provider_info field, but that's a little different I think. | 12:56 |
*** e0ne has quit IRC | 13:12 | |
*** mchlumsky has joined #openstack-cinder | 13:13 | |
*** dpawlik has quit IRC | 13:13 | |
*** e0ne has joined #openstack-cinder | 13:16 | |
*** dpawlik has joined #openstack-cinder | 13:18 | |
*** jistr|afk is now known as jistr | 13:20 | |
*** mriedem has joined #openstack-cinder | 13:23 | |
*** dpawlik has quit IRC | 13:30 | |
*** ociuhandu has joined #openstack-cinder | 13:37 | |
*** ociuhandu has quit IRC | 13:38 | |
*** dpawlik has joined #openstack-cinder | 13:40 | |
*** eharney has joined #openstack-cinder | 13:45 | |
*** lixiaoy1 has joined #openstack-cinder | 13:52 | |
*** e0ne has quit IRC | 13:53 | |
mriedem | smcginnis: let me know if you want any help refactoring the cinder-manage cli as i've suggested here https://review.openstack.org/#/c/610581/ | 13:56 |
mriedem | i could throw that up and you could rebase on top if you don't want to spend the time doing it | 13:57 |
*** Luzi has quit IRC | 14:00 | |
smcginnis | mriedem: If you have the time and inclination to do that, go for it. Then I could get it reviewed faster. | 14:01 |
*** e0ne has joined #openstack-cinder | 14:01 | |
mriedem | ok | 14:01 |
mriedem | what i can do to procrastinate from reviewing specs | 14:01 |
mriedem | *whatever | 14:01 |
smcginnis | I know how that goes. | 14:02 |
*** jcosmao has joined #openstack-cinder | 14:08 | |
*** dpawlik has quit IRC | 14:11 | |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: Move cinder-manage page to cli doc subtree https://review.openstack.org/610989 | 14:14 |
mriedem | smcginnis: ^ | 14:14 |
smcginnis | mriedem: Cool, thanks! | 14:15 |
openstackgerrit | Eric Harney proposed openstack/cinder stable/queens: Fix backup/restore error for ceph rbd backend https://review.openstack.org/610996 | 14:25 |
*** eharney has quit IRC | 14:33 | |
*** dpawlik has joined #openstack-cinder | 14:46 | |
*** eharney has joined #openstack-cinder | 14:48 | |
*** dpawlik has quit IRC | 14:50 | |
*** lixiaoy1 has quit IRC | 14:58 | |
jbernard | smcginnis: in the absence of such a mechanisim, should I consider proposing a generic solution? | 15:48 |
jbernard | smcginnis: i suspect the answer is likely no, but that leaves me wondering what to do with this patch | 15:49 |
jbernard | smcginnis: essentially, i need a db field that i can store key/value pairs | 15:49 |
jbernard | smcginnis: at per-volume granularity | 15:50 |
*** e0ne has quit IRC | 15:50 | |
smcginnis | jbernard: Good question. | 15:53 |
smcginnis | Or questions, rather. | 15:53 |
smcginnis | First thing that comes to mind - etcd is declared as a system service we can expect to have. Key/value storage makes sense there. | 15:54 |
smcginnis | But not really liking the idea of a driver going out and stashing stuff somewhere like that. | 15:54 |
smcginnis | We could add a metadata table, but not a fan of that either. | 15:55 |
smcginnis | jbernard: There's really no way to carry over the data you need as the volume gets retyped? | 15:56 |
jbernard | smcginnis: what is want is 'replication_driver_data' but for things not necessarily replication related | 15:59 |
jbernard | smcginnis: i can overload provider_location... | 15:59 |
jbernard | etcd makes my skin crawl | 15:59 |
jbernard | in this context | 15:59 |
smcginnis | Yeah | 16:00 |
jbernard | smcginnis: side question: is this a blocker for the patch as it exists now? | 16:01 |
smcginnis | jbernard: I'm really hesitant to merge something like that. Can it at least handle retyping from multiattach to non-multiattach? Seems like there are some kinds of retype operations that could still be handled, right? | 16:02 |
jbernard | smcginnis: going /to/ multiattach is easy | 16:03 |
jbernard | smcginnis: going /from/ multiattach means i need to know what flags the volume had before it became a multiattach volume | 16:03 |
jbernard | smcginnis: and therein lies the problem | 16:03 |
jbernard | smcginnis: i know that at retype-time, but i dont have a place to store that to refer later | 16:04 |
smcginnis | And really no way to deduce those flags from the current settings? | 16:04 |
jbernard | smcginnis: let me think about that | 16:05 |
jbernard | smcginnis: (more than i already have) :) | 16:06 |
smcginnis | jbernard: Thanks! :) | 16:13 |
*** lemko has quit IRC | 16:14 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: NFS (chunked) backup: split reads https://review.openstack.org/611079 | 16:30 |
eharney | jbernard: i thought the provider_info fields could be used for this kind of thing | 16:36 |
jbernard | eharney: it's not clear to me, but if they can that would be great | 16:40 |
openstackgerrit | Marc Methot proposed openstack/cinder master: Handling unexpected python error "NoneType object is not iterable" https://review.openstack.org/611081 | 16:40 |
jbernard | eharney: i trying to determine if i can know without creating a volume what flags would be enabled by default for a given pool | 16:40 |
jbernard | eharney: and i would not need such a field in that case, would be ideal | 16:40 |
jbernard | eharney: it would be nice if we had a driver-reserved field for specific but limited use for such cases | 16:41 |
jbernard | eharney: though, i might be the only one to need such a thing | 16:42 |
eharney | jbernard: well, we introduced some of the provider fields for similar needs in the past, but i need to look into how they work again | 16:42 |
jbernard | eharney: it's not entirely clear from what I've read so far | 16:42 |
jbernard | eharney: maybe its exactly what I'm wanting and just havent understood the intent | 16:43 |
jbernard | eharney: all i need is a char sized area for a bitmask | 16:43 |
*** e0ne has joined #openstack-cinder | 17:08 | |
*** whoami-rajat has quit IRC | 17:13 | |
*** mvkr has quit IRC | 17:13 | |
openstackgerrit | Merged openstack/cinder master: Add test-requirements to pylint reqs https://review.openstack.org/610583 | 17:56 |
*** e0ne has quit IRC | 17:59 | |
openstackgerrit | David Vallee Delisle proposed openstack/cinder-specs master: cinder-manage volume delete --from-db https://review.openstack.org/610895 | 18:05 |
*** mvkr has joined #openstack-cinder | 18:14 | |
*** e0ne has joined #openstack-cinder | 18:15 | |
*** e0ne has quit IRC | 18:22 | |
openstackgerrit | David Vallee Delisle proposed openstack/cinder-specs master: cinder-manage object delete --from-db https://review.openstack.org/610895 | 19:02 |
*** dave-mccowan has joined #openstack-cinder | 19:08 | |
openstackgerrit | David Vallee Delisle proposed openstack/cinder-specs master: cinder-manage object delete --from-db https://review.openstack.org/610895 | 19:10 |
openstackgerrit | Eric Harney proposed openstack/cinder master: NFS (chunked) backup: split reads https://review.openstack.org/611079 | 19:26 |
openstackgerrit | Eric Harney proposed openstack/cinder master: Rename BackupNFSSwiftBasedTestCase https://review.openstack.org/611137 | 19:26 |
smcginnis | Guess we know what eharney's working on. | 19:27 |
eharney | heh, it's NFS week for me so far :) | 19:27 |
*** icey has quit IRC | 19:52 | |
*** luizbag has quit IRC | 19:57 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: VMAX: set faked max_oversubscription_ratio https://review.openstack.org/599016 | 20:00 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Restructure Huawei driver https://review.openstack.org/599029 | 20:00 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Update sphinx extension logging https://review.openstack.org/611158 | 20:19 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Huawei: Simplify wait_for_condition calls https://review.openstack.org/599082 | 20:30 |
*** erlon has quit IRC | 20:48 | |
*** mriedem is now known as mriedem_away | 21:02 | |
*** eharney has quit IRC | 21:03 | |
*** mchlumsky has quit IRC | 21:25 | |
openstackgerrit | Merged openstack/os-brick stable/rocky: The validation of iscsi session should be case insensitive https://review.openstack.org/608578 | 21:26 |
*** lbragstad is now known as lbragstad-503 | 21:42 | |
*** MetricT has joined #openstack-cinder | 21:47 | |
*** mriedem_away has quit IRC | 21:55 | |
*** smcginnis is now known as smcginnis_vaca | 21:55 | |
*** MetricT has left #openstack-cinder | 22:05 | |
*** rcernin has joined #openstack-cinder | 22:49 | |
*** ganso has quit IRC | 23:18 | |
*** dave-mccowan has quit IRC | 23:24 | |
*** erlon has joined #openstack-cinder | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!