Tuesday, 2020-08-18

*** zhanglong has joined #openstack-cinder00:04
openstackgerritRajat Dhasmana proposed openstack/cinder stable/rocky: RBD: Cleanup temporary file during exception  https://review.opendev.org/74661900:12
openstackgerritRajat Dhasmana proposed openstack/cinder master: Modify default/delete volume type logic  https://review.opendev.org/74149800:16
*** zhanglong has quit IRC00:18
*** zhanglong has joined #openstack-cinder00:20
*** brinzhang_ has joined #openstack-cinder00:24
*** brinzhang0 has quit IRC00:27
openstackgerritAlan Bishop proposed openstack/os-brick stable/train: ScaleIO: Connection info backward compatibility  https://review.opendev.org/74662100:45
openstackgerritAlan Bishop proposed openstack/os-brick stable/train: Add release note for scaleio connector upgrade  https://review.opendev.org/74662200:45
*** spatel has joined #openstack-cinder00:50
*** n-saito has joined #openstack-cinder01:20
*** sapd1 has joined #openstack-cinder01:51
*** zhanglong has quit IRC01:59
*** zhanglong has joined #openstack-cinder02:00
*** brinzhang0 has joined #openstack-cinder02:14
*** brinzhang_ has quit IRC02:17
openstackgerritMerged openstack/cinder master: Fix dfs-sdk package name in extra reqs  https://review.opendev.org/72633602:36
*** zhanglong has quit IRC02:46
*** whoami-rajat__ has quit IRC02:48
*** zhanglong has joined #openstack-cinder02:51
*** sapd1 has quit IRC02:52
*** psachin has joined #openstack-cinder03:37
*** zhanglong has quit IRC03:37
openstackgerritAlan Bishop proposed openstack/os-brick stable/train: ScaleIO: Connection info backward compatibility  https://review.opendev.org/74662103:38
openstackgerritAlan Bishop proposed openstack/os-brick stable/train: Add release note for scaleio connector upgrade  https://review.opendev.org/74662203:38
*** dave-mccowan has quit IRC03:47
openstackgerritMerged openstack/cinder stable/ussuri: Creating snapshot on NFS backend fails  https://review.opendev.org/74258104:03
*** laurent\ has quit IRC04:05
*** sapd1 has joined #openstack-cinder04:17
*** sfernand has quit IRC04:22
*** rishabhhpe has joined #openstack-cinder04:34
*** vishalmanchanda has joined #openstack-cinder04:47
*** zhanglong has joined #openstack-cinder04:50
*** ajitha has joined #openstack-cinder04:51
*** strigazi has quit IRC04:52
*** strigazi has joined #openstack-cinder04:54
*** ajitha has quit IRC05:02
*** spatel has quit IRC05:08
*** abdysn has joined #openstack-cinder05:11
*** strigazi has quit IRC05:12
*** m75abrams has joined #openstack-cinder05:21
*** sapd1 has quit IRC05:28
*** ociuhandu has joined #openstack-cinder05:43
*** ociuhandu has quit IRC05:47
*** zhanglong has quit IRC06:43
*** zhanglong has joined #openstack-cinder06:45
*** laurent\ has joined #openstack-cinder06:58
*** rcernin has quit IRC07:02
*** sapd1 has joined #openstack-cinder07:03
*** zhanglong has quit IRC07:03
*** rcernin has joined #openstack-cinder07:03
*** zhanglong has joined #openstack-cinder07:07
*** zhanglong has quit IRC07:15
*** zhanglong has joined #openstack-cinder07:18
*** lpetrut has joined #openstack-cinder07:23
*** e0ne has joined #openstack-cinder07:26
*** whoami-rajat__ has joined #openstack-cinder07:30
*** sapd1 has quit IRC07:38
*** tosky has joined #openstack-cinder07:40
openstackgerritLuigi Toscano proposed openstack/python-cinderclient master: zuul functional job: drop the custom playbooks  https://review.opendev.org/74623707:44
*** rosmaita has quit IRC07:49
*** ociuhandu has joined #openstack-cinder08:06
*** ociuhandu has quit IRC08:11
*** Lucas_Gray has joined #openstack-cinder08:13
*** sapd1 has joined #openstack-cinder08:23
*** benfelin has joined #openstack-cinder08:27
openstackgerritHui Jiang proposed openstack/cinder master: [DNM] just for test, trigger CI.  https://review.opendev.org/74134908:28
*** rcernin has quit IRC08:34
*** zhanglong has quit IRC08:34
*** zhanglong has joined #openstack-cinder08:34
*** martinkennelly has joined #openstack-cinder08:35
*** priteau has joined #openstack-cinder08:35
*** ociuhandu has joined #openstack-cinder08:37
*** zhanglong has quit IRC08:38
*** zhanglong has joined #openstack-cinder08:40
*** zhanglong has quit IRC08:51
*** aarora06 has joined #openstack-cinder08:54
*** zhanglong has joined #openstack-cinder08:55
*** rishabhhpe has quit IRC08:58
*** rishabhhpe has joined #openstack-cinder08:58
*** rishabhhpe has quit IRC09:08
*** sapd1 has quit IRC09:09
*** rishabhhpe has joined #openstack-cinder09:10
*** benfelin has quit IRC09:33
*** brinzhang_ has joined #openstack-cinder09:34
*** rishabhhpe has quit IRC09:36
*** brinzhang0 has quit IRC09:37
*** rishabhhpe has joined #openstack-cinder09:37
*** martinkennelly has quit IRC09:38
openstackgerritLuigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix AttributeError when raising exception  https://review.opendev.org/74653509:44
openstackgerritLuigi Toscano proposed openstack/cinder stable/ussuri: Brocade: Fix lookup UnboundLocalError  https://review.opendev.org/74653609:44
*** benfelin has joined #openstack-cinder09:47
*** Luzi has joined #openstack-cinder09:53
*** tkajinam has quit IRC09:58
*** zhanglong has quit IRC10:05
*** chhagarw has joined #openstack-cinder10:14
*** sapd1 has joined #openstack-cinder10:14
*** martinkennelly has joined #openstack-cinder10:15
*** ociuhandu has quit IRC10:17
*** ociuhandu has joined #openstack-cinder10:18
*** aarora06 has quit IRC10:25
*** whoami-rajat__ has quit IRC10:38
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver -  Replacing generations with snap_ids  https://review.opendev.org/74043310:53
*** manoj_kumar_kata has joined #openstack-cinder11:26
manoj_kumar_katahttps://review.opendev.org/#/c/745576/11:31
*** Lucas_Gray has quit IRC11:32
manoj_kumar_kata@sean mcginnis: could you please review this change for ibm storwize driver11:32
*** brinzhang0 has joined #openstack-cinder11:34
*** brinzhang_ has quit IRC11:36
*** dave-mccowan has joined #openstack-cinder12:03
*** rosmaita has joined #openstack-cinder12:04
*** rosmaita has quit IRC12:04
smcginnismanoj_kumar_kata: There doesn't appear to be third party CI results on that patch yet. Best to wait to ask for reviews until that happens.12:07
*** enriquetaso has joined #openstack-cinder12:08
*** rosmaita has joined #openstack-cinder12:13
*** zzzeek has quit IRC12:19
*** amar7ibm has joined #openstack-cinder12:22
*** zzzeek has joined #openstack-cinder12:23
*** amar7ibm49 has joined #openstack-cinder12:25
*** amar7ibm has quit IRC12:25
*** amar7ibm49 has quit IRC12:25
*** amar7ibm has joined #openstack-cinder12:26
*** rishabhhpe has quit IRC12:26
*** rishabhhpe has joined #openstack-cinder12:27
*** manoj_kumar_kata has quit IRC12:31
*** Lucas_Gray has joined #openstack-cinder12:42
*** enriquetaso has quit IRC12:50
*** rishabhhpe has quit IRC12:52
*** rishabhhpe has joined #openstack-cinder12:52
openstackgerritLuigi Toscano proposed openstack/python-cinderclient master: Bump pylint to 2.3.0 (same version used by cinder)  https://review.opendev.org/74670512:53
*** psachin has quit IRC12:57
*** whoami-rajat__ has joined #openstack-cinder12:59
openstackgerritRajat Dhasmana proposed openstack/cinder stable/rocky: RBD: Cleanup temporary file during exception  https://review.opendev.org/74661913:00
*** sfernand has joined #openstack-cinder13:04
*** priteau has quit IRC13:06
*** manoj_kumar_kata has joined #openstack-cinder13:11
*** enriquetaso has joined #openstack-cinder13:13
*** brinzhang_ has joined #openstack-cinder13:13
*** brinzhang0 has quit IRC13:17
*** manoj_kumar_kata has quit IRC13:17
*** manoj_kumar_kata has joined #openstack-cinder13:17
*** sapd1 has quit IRC13:20
*** amar7ibm has quit IRC13:30
*** Luzi has quit IRC13:34
*** vishakha has joined #openstack-cinder13:43
*** rishabhhpe has quit IRC13:46
*** tkajinam has joined #openstack-cinder13:50
*** jv_ has quit IRC13:54
*** abdysn has quit IRC14:04
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver -  Replacing generations with snap_ids  https://review.opendev.org/74043314:09
*** rosmaita has quit IRC14:18
*** brinzhang0 has joined #openstack-cinder14:19
*** davee_ has joined #openstack-cinder14:20
*** brinzhang_ has quit IRC14:21
devfazhi, is there a way to "flatten" volumes created from a snapshot (ceph backend)?14:21
*** rosmaita has joined #openstack-cinder14:22
*** davee_ has quit IRC14:22
devfazif im reading the code correctly, then openstack is not saving any information about the "im based on a snapshot"-state, so doing the flatten with ceph-tools should be ok? Is someone able to confirm this?14:30
eharneydevfaz: there is an "rbd_flatten_volume_from_snapshot" option for the rbd driver which does this14:33
*** mvorwerk has joined #openstack-cinder14:34
devfazeharney: yes, but i have an existing volume, this setting is used for newly created ones :(14:36
*** rosmaita has quit IRC14:47
*** lpetrut has quit IRC14:48
*** rosmaita has joined #openstack-cinder14:51
openstackgerritLuigi Toscano proposed openstack/python-cinderclient master: zuul functional job: drop the custom playbooks  https://review.opendev.org/74623715:04
*** priteau has joined #openstack-cinder15:04
*** vishalmanchanda has quit IRC15:06
*** sapd1 has joined #openstack-cinder15:12
*** lpetrut has joined #openstack-cinder15:23
*** mvorwerk has quit IRC15:24
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver -  Replacing generations with snap_ids  https://review.opendev.org/74043315:40
*** brinzhang_ has joined #openstack-cinder15:48
*** brinzhang0 has quit IRC15:51
*** m75abrams has quit IRC16:01
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - Force add rep group volume  https://review.opendev.org/74675816:20
*** brinzhang0 has joined #openstack-cinder16:39
*** brinzhang_ has quit IRC16:42
*** Lucas_Gray has quit IRC16:43
*** ajitha has joined #openstack-cinder16:44
*** whoami-rajat__ has quit IRC16:48
*** sapd1 has quit IRC16:54
*** ociuhandu has quit IRC17:03
openstackgerritAndrei Perepiolkin proposed openstack/cinder master: [WIP] Add Cinder driver for Open-E JovianDSS data storage  https://review.opendev.org/72690617:12
*** ociuhandu has joined #openstack-cinder17:17
*** ociuhandu has quit IRC17:22
*** lpetrut has quit IRC17:29
*** manoj_kumar_kata has quit IRC18:12
*** _erlon_ has joined #openstack-cinder18:17
*** jopdorp has quit IRC19:01
*** arne_wiebalck has quit IRC19:01
*** rajinir has quit IRC19:01
*** ajitha has quit IRC19:01
*** jopdorp has joined #openstack-cinder19:04
*** rajinir has joined #openstack-cinder19:08
*** arne_wiebalck has joined #openstack-cinder19:09
hemnaanyone know what the volume_type_projects table is used for ?19:20
hemnaI'm seeing some strange data in it for the deleted column19:20
hemnahttp://paste.openstack.org/show/796923/19:20
hemnawould seem to break this: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L45419:21
smcginnishemna: What does that break?19:33
hemnaI dunno, but looking at the code, the values should be 0 or 1 afaik19:33
smcginnisWell, 0 or non-0.19:33
smcginnisNot sure why the deleted value would be incremental, but...19:34
hemnamatches the id ?19:34
smcginnisCould be what it's doing.19:34
hemnahrmm, production db looks like it matches the id column19:35
hemnavolume_types has a deleted column as well, but it's a 0 or a 1 only19:36
smcginnisProbably just coincidence that they both increment sequentially.19:36
smcginnisOriginal patch where it was added: https://review.opendev.org/#/c/114395/19:36
hemnalooks like other tables just have a 0 or 1 for deleted column19:37
hemnalooks like that's the only place that uses soft_delete()19:40
smcginnisIt came from nova, and I know they were handling deletes of things differently than we do.19:41
*** mvorwerk has joined #openstack-cinder19:41
openstackgerritAjitha Robert proposed openstack/cinder master: Add multiattach in Nimble driver  https://review.opendev.org/74012519:43
*** priteau has quit IRC19:44
openstackgerritAndrei Perepiolkin proposed openstack/cinder master: [WIP] Add Cinder driver for Open-E JovianDSS data storage  https://review.opendev.org/72690619:44
*** brinzhang_ has joined #openstack-cinder19:44
hemnaI think I might have found the problem19:46
hemnaNotice:19:47
hemnahttps://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L4719:47
hemnavs19:47
*** brinzhang0 has quit IRC19:47
hemnahttps://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L44619:47
hemnaoopsie19:47
hemna:(19:48
hemnaheh19:51
hemnahttps://bugs.launchpad.net/cinder/+bug/149674719:51
openstackLaunchpad bug 1496747 in Cinder "Removing project access to volume type will not work after 128+ access rule" [Medium,Fix released] - Assigned to Michal Dulko (michal-dulko-f)19:51
*** vishakha has quit IRC19:52
hemnathat 'fix' just extended the range to int sizes, instead of tinyint19:52
hemnaso, that fix was wrong to do19:54
hemnaall other tables that have a deleted column are tinyint19:54
hemnabut they act as a boolean, because the model declares it as a boolean19:54
hemnainstead of an int.19:54
*** ociuhandu has joined #openstack-cinder19:58
smcginnisWe should remove the boolean constraint and have a migration that changes anything > 1 to be 1.20:01
hemnayah it needs to be changed back to tinyint, then data migrated, then redeclared as Boolean in code20:01
smcginnisI wonder if the soft_delete needs that to match the ID to be able to do some sort of "delete anything older than X" type operation.20:02
*** ociuhandu has quit IRC20:02
smcginnisBut since we never actually implemented soft delete management in cinder, not sure if it really matters.20:02
hemnaI can't find where that soft_delete is defined20:02
hemnahttps://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L44-L4520:03
*** martinkennelly has quit IRC20:04
*** martinkennelly has joined #openstack-cinder20:04
smcginnisLooks like it is some sort of "move to trash" / "restore from trash" type mechanism.20:07
smcginnisLike "oops, I didn't mean to delete all my instances, I better get those back quick"20:07
*** e0ne has quit IRC20:21
*** brinzhang0 has joined #openstack-cinder20:25
*** e0ne has joined #openstack-cinder20:26
*** brinzhang_ has quit IRC20:28
*** e0ne has quit IRC20:33
*** martinkennelly has quit IRC20:57
*** mvorwerk has quit IRC20:59
*** Lucas_Gray has joined #openstack-cinder21:03
*** _erlon_ has quit IRC21:12
*** JaySee has quit IRC21:17
*** rcernin has joined #openstack-cinder21:18
*** JaySee has joined #openstack-cinder21:18
*** rcernin has quit IRC21:35
*** chhagarw has quit IRC21:39
*** enriquetaso has quit IRC21:39
*** rcernin has joined #openstack-cinder21:52
*** brinzhang_ has joined #openstack-cinder21:54
*** brinzhang0 has quit IRC21:56
*** rcernin has quit IRC21:57
*** rcernin has joined #openstack-cinder22:08
*** rcernin has quit IRC22:09
*** brinzhang0 has joined #openstack-cinder22:14
*** brinzhang_ has quit IRC22:17
*** tosky has quit IRC22:18
*** rcernin has joined #openstack-cinder22:33
*** lxkong has quit IRC22:42
*** patrickeast has quit IRC22:42
*** sri_ has quit IRC22:42
*** dviroel has quit IRC22:43
*** knikolla has quit IRC22:43
*** sri_ has joined #openstack-cinder22:51
*** patrickeast has joined #openstack-cinder22:54
*** dviroel has joined #openstack-cinder22:55
*** lxkong has joined #openstack-cinder22:55
*** knikolla has joined #openstack-cinder22:55
openstackgerritMerged openstack/python-cinderclient stable/train: Fix typo: dow -> down  https://review.opendev.org/74485822:56
*** Lucas_Gray has quit IRC23:05
openstackgerritLucio Seki proposed openstack/cinder master: NetApp SolidFire: Enable driver IPv6 api request  https://review.opendev.org/74681323:08
*** benfelin has quit IRC23:11
*** brinzhang_ has joined #openstack-cinder23:44
*** brinzhang0 has quit IRC23:47

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!