Tuesday, 2018-10-16

*** Dinesh_Bhor has joined #openstack-cinder00:34
*** LiangFang has joined #openstack-cinder00:36
*** lixiaoy1 has joined #openstack-cinder00:39
LiangFangThanks Sean00:41
*** openstackgerrit has joined #openstack-cinder00:46
openstackgerritSean McGinnis proposed openstack/cinder master: Base framework for cinder-status upgrade check  https://review.openstack.org/61058100:46
*** geguileo_PTO has quit IRC00:49
openstackgerritMerged openstack/cinder master: SF ensure the correct volume is deleted  https://review.openstack.org/58414900:52
openstackgerritwang yong proposed openstack/cinder master: Cinder volume driver for Inspur AS13000 series  https://review.openstack.org/56297700:54
openstackgerritMerged openstack/os-brick master: Fix a spelling mistake  https://review.openstack.org/61045000:56
*** TuanDA has joined #openstack-cinder01:14
*** Dinesh_Bhor has quit IRC01:27
*** Dinesh_Bhor has joined #openstack-cinder01:35
*** openstackgerrit has quit IRC01:35
*** dillaman has joined #openstack-cinder01:52
*** mhen has quit IRC01:52
*** jdillaman has quit IRC01:52
*** openstackgerrit has joined #openstack-cinder02:01
openstackgerritzhufl proposed openstack/cinder master: api-ref: fix some inconsistencies in snapshot API  https://review.openstack.org/60961102:01
*** Dinesh_Bhor has quit IRC02:02
openstackgerritDai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver  https://review.openstack.org/61064502:06
openstackgerritDai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver  https://review.openstack.org/61064502:08
openstackgerritDai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver  https://review.openstack.org/61064502:09
openstackgerritDai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver  https://review.openstack.org/61064502:11
openstackgerritzhufl proposed openstack/cinder master: api-ref: fix some inconsistencies in snapshot API  https://review.openstack.org/60961102:14
*** rosmaita has left #openstack-cinder02:51
*** Dinesh_Bhor has joined #openstack-cinder02:55
*** dave-mccowan has quit IRC03:08
*** Dinesh_Bhor has quit IRC03:36
*** Bhujay has joined #openstack-cinder03:51
*** bswartz has quit IRC03:51
*** Bhujay has quit IRC03:52
*** lpetrut has joined #openstack-cinder03:52
*** Bhujay has joined #openstack-cinder03:52
*** Bhujay has quit IRC04:06
*** Dinesh_Bhor has joined #openstack-cinder04:30
*** lpetrut has quit IRC04:34
*** dpawlik has joined #openstack-cinder05:55
*** dpawlik has quit IRC06:00
*** Luzi has joined #openstack-cinder06:01
*** dpawlik has joined #openstack-cinder06:11
*** dpawlik has quit IRC06:12
*** dpawlik has joined #openstack-cinder06:12
*** Dinesh_Bhor has quit IRC06:16
*** edisonxiang has joined #openstack-cinder06:44
*** e0ne has joined #openstack-cinder06:48
*** tommylikehu has joined #openstack-cinder06:54
*** rcernin has quit IRC07:01
*** lpetrut has joined #openstack-cinder07:20
*** lpetrut has quit IRC07:25
*** markvoelker has quit IRC07:29
*** markvoelker has joined #openstack-cinder07:29
*** Dinesh_Bhor has joined #openstack-cinder07:33
*** mvkr has quit IRC07:34
*** markvoelker has quit IRC07:34
*** edisonxiang has quit IRC07:39
*** icey has quit IRC07:52
*** icey has joined #openstack-cinder07:52
openstackgerritDavid Vallee Delisle proposed openstack/cinder-specs master: cinder-manage volume delete --from-db  https://review.openstack.org/61089508:10
*** mvkr has joined #openstack-cinder08:14
openstackgerritMaciej Szwed proposed openstack/cinder master: Adding SPDK volume driver  https://review.openstack.org/56422908:21
openstackgerritMaciej Szwed proposed openstack/cinder master: SPDK drivers documentation  https://review.openstack.org/60927608:21
*** yikun has quit IRC08:22
*** Reepicheep has quit IRC08:23
*** markvoelker has joined #openstack-cinder08:30
*** Reepicheep has joined #openstack-cinder08:30
*** mvkr has quit IRC08:35
*** e0ne has quit IRC08:37
*** gkadam has joined #openstack-cinder08:38
*** gkadam has quit IRC08:38
*** e0ne has joined #openstack-cinder08:44
*** mvkr has joined #openstack-cinder08:48
*** lixiaoy1 has quit IRC08:54
*** markvoelker has quit IRC09:03
*** tommylikehu has quit IRC09:33
*** mvkr has quit IRC09:40
*** mvkr has joined #openstack-cinder09:41
*** mvpnitesh has joined #openstack-cinder09:43
*** mvpnitesh has quit IRC09:48
*** imacdonn has quit IRC09:54
*** imacdonn has joined #openstack-cinder09:55
*** TuanDA has quit IRC10:00
*** markvoelker has joined #openstack-cinder10:00
*** chrisyang_0660 has quit IRC10:04
*** luizbag has joined #openstack-cinder10:06
*** erlon has joined #openstack-cinder10:07
*** Dinesh_Bhor has quit IRC10:20
*** LiangFang has quit IRC10:32
*** markvoelker has quit IRC10:34
*** brinzhang has joined #openstack-cinder10:47
*** ganso has joined #openstack-cinder10:51
*** Dinesh_Bhor has joined #openstack-cinder10:54
*** Dinesh_Bhor has quit IRC11:06
*** e0ne has quit IRC11:18
*** markvoelker has joined #openstack-cinder11:31
*** e0ne has joined #openstack-cinder11:32
*** brinzhang has quit IRC11:38
*** abishop has joined #openstack-cinder11:38
*** jistr is now known as jistr|afk11:47
*** lemko has joined #openstack-cinder11:54
*** markvoelker has quit IRC12:04
*** e0ne has quit IRC12:24
openstackgerritYikun Jiang proposed openstack/cinder master: WIP: volume reimage  https://review.openstack.org/60634612:26
*** e0ne has joined #openstack-cinder12:27
jbernardsmcginnis: i agree, im not aware of any mechanism to store persitent driver metadata per volume12:37
*** rosmaita has joined #openstack-cinder12:46
openstackgerritYikun Jiang proposed openstack/cinder master: Raise the ImageTooBig exception when found it  https://review.openstack.org/61095012:54
smcginnisjbernard: We have the provider_info field, but that's a little different I think.12:56
*** e0ne has quit IRC13:12
*** mchlumsky has joined #openstack-cinder13:13
*** dpawlik has quit IRC13:13
*** e0ne has joined #openstack-cinder13:16
*** dpawlik has joined #openstack-cinder13:18
*** jistr|afk is now known as jistr13:20
*** mriedem has joined #openstack-cinder13:23
*** dpawlik has quit IRC13:30
*** ociuhandu has joined #openstack-cinder13:37
*** ociuhandu has quit IRC13:38
*** dpawlik has joined #openstack-cinder13:40
*** eharney has joined #openstack-cinder13:45
*** lixiaoy1 has joined #openstack-cinder13:52
*** e0ne has quit IRC13:53
mriedemsmcginnis: 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
mriedemi could throw that up and you could rebase on top if you don't want to spend the time doing it13:57
*** Luzi has quit IRC14:00
smcginnismriedem: 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-cinder14:01
mriedemok14:01
mriedemwhat i can do to procrastinate from reviewing specs14:01
mriedem*whatever14:01
smcginnisI know how that goes.14:02
*** jcosmao has joined #openstack-cinder14:08
*** dpawlik has quit IRC14:11
openstackgerritMatt Riedemann proposed openstack/cinder master: Move cinder-manage page to cli doc subtree  https://review.openstack.org/61098914:14
mriedemsmcginnis: ^14:14
smcginnismriedem: Cool, thanks!14:15
openstackgerritEric Harney proposed openstack/cinder stable/queens: Fix backup/restore error for ceph rbd backend  https://review.openstack.org/61099614:25
*** eharney has quit IRC14:33
*** dpawlik has joined #openstack-cinder14:46
*** eharney has joined #openstack-cinder14:48
*** dpawlik has quit IRC14:50
*** lixiaoy1 has quit IRC14:58
jbernardsmcginnis: in the absence of such a mechanisim, should I consider proposing a generic solution?15:48
jbernardsmcginnis: i suspect the answer is likely no, but that leaves me wondering what to do with this patch15:49
jbernardsmcginnis: essentially, i need a db field that i can store key/value pairs15:49
jbernardsmcginnis: at per-volume granularity15:50
*** e0ne has quit IRC15:50
smcginnisjbernard: Good question.15:53
smcginnisOr questions, rather.15:53
smcginnisFirst 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
smcginnisBut not really liking the idea of a driver going out and stashing stuff somewhere like that.15:54
smcginnisWe could add a metadata table, but not a fan of that either.15:55
smcginnisjbernard: There's really no way to carry over the data you need as the volume gets retyped?15:56
jbernardsmcginnis: what is want is 'replication_driver_data' but for things not necessarily replication related15:59
jbernardsmcginnis: i can overload provider_location...15:59
jbernardetcd makes my skin crawl15:59
jbernardin this context15:59
smcginnisYeah16:00
jbernardsmcginnis: side question: is this a blocker for the patch as it exists now?16:01
smcginnisjbernard: 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
jbernardsmcginnis: going /to/ multiattach is easy16:03
jbernardsmcginnis: going /from/ multiattach means i need to know what flags the volume had before it became a multiattach volume16:03
jbernardsmcginnis: and therein lies the problem16:03
jbernardsmcginnis: i know that at retype-time, but i dont have a place to store that to refer later16:04
smcginnisAnd really no way to deduce those flags from the current settings?16:04
jbernardsmcginnis: let me think about that16:05
jbernardsmcginnis: (more than i already have) :)16:06
smcginnisjbernard: Thanks! :)16:13
*** lemko has quit IRC16:14
openstackgerritEric Harney proposed openstack/cinder master: NFS (chunked) backup: split reads  https://review.openstack.org/61107916:30
eharneyjbernard: i thought the provider_info fields could be used for this kind of thing16:36
jbernardeharney: it's not clear to me, but if they can that would be great16:40
openstackgerritMarc Methot proposed openstack/cinder master: Handling unexpected python error "NoneType object is not iterable"  https://review.openstack.org/61108116:40
jbernardeharney: i trying to determine if i can know without creating a volume what flags would be enabled by default for a given pool16:40
jbernardeharney: and i would not need such a field in that case, would be ideal16:40
jbernardeharney: it would be nice if we had a driver-reserved field for specific but limited use for such cases16:41
jbernardeharney: though, i might be the only one to need such a thing16:42
eharneyjbernard: well, we introduced some of the provider fields for similar needs in the past, but i need to look into how they work again16:42
jbernardeharney: it's not entirely clear from what I've read so far16:42
jbernardeharney: maybe its exactly what I'm wanting and just havent understood the intent16:43
jbernardeharney: all i need is a char sized area for a bitmask16:43
*** e0ne has joined #openstack-cinder17:08
*** whoami-rajat has quit IRC17:13
*** mvkr has quit IRC17:13
openstackgerritMerged openstack/cinder master: Add test-requirements to pylint reqs  https://review.openstack.org/61058317:56
*** e0ne has quit IRC17:59
openstackgerritDavid Vallee Delisle proposed openstack/cinder-specs master: cinder-manage volume delete --from-db  https://review.openstack.org/61089518:05
*** mvkr has joined #openstack-cinder18:14
*** e0ne has joined #openstack-cinder18:15
*** e0ne has quit IRC18:22
openstackgerritDavid Vallee Delisle proposed openstack/cinder-specs master: cinder-manage object delete --from-db  https://review.openstack.org/61089519:02
*** dave-mccowan has joined #openstack-cinder19:08
openstackgerritDavid Vallee Delisle proposed openstack/cinder-specs master: cinder-manage object delete --from-db  https://review.openstack.org/61089519:10
openstackgerritEric Harney proposed openstack/cinder master: NFS (chunked) backup: split reads  https://review.openstack.org/61107919:26
openstackgerritEric Harney proposed openstack/cinder master: Rename BackupNFSSwiftBasedTestCase  https://review.openstack.org/61113719:26
smcginnisGuess we know what eharney's working on.19:27
eharneyheh, it's NFS week for me so far :)19:27
*** icey has quit IRC19:52
*** luizbag has quit IRC19:57
openstackgerritSean McGinnis proposed openstack/cinder master: VMAX: set faked max_oversubscription_ratio  https://review.openstack.org/59901620:00
openstackgerritSean McGinnis proposed openstack/cinder master: Restructure Huawei driver  https://review.openstack.org/59902920:00
openstackgerritSean McGinnis proposed openstack/cinder master: Update sphinx extension logging  https://review.openstack.org/61115820:19
openstackgerritSean McGinnis proposed openstack/cinder master: Huawei: Simplify wait_for_condition calls  https://review.openstack.org/59908220:30
*** erlon has quit IRC20:48
*** mriedem is now known as mriedem_away21:02
*** eharney has quit IRC21:03
*** mchlumsky has quit IRC21:25
openstackgerritMerged openstack/os-brick stable/rocky: The validation of iscsi session should be case insensitive  https://review.openstack.org/60857821:26
*** lbragstad is now known as lbragstad-50321:42
*** MetricT has joined #openstack-cinder21:47
*** mriedem_away has quit IRC21:55
*** smcginnis is now known as smcginnis_vaca21:55
*** MetricT has left #openstack-cinder22:05
*** rcernin has joined #openstack-cinder22:49
*** ganso has quit IRC23:18
*** dave-mccowan has quit IRC23:24
*** erlon has joined #openstack-cinder23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!