Tuesday, 2019-12-03

« Monday, 2019-12-02 Index Wednesday, 2019-12-04 »
*** dpawlik has joined #openstack-cinder00:23
*** dpawlik has quit IRC00:28
*** ociuhandu has joined #openstack-cinder01:31
*** ociuhandu has quit IRC01:35
*** awalende has joined #openstack-cinder02:00
*** yikun has joined #openstack-cinder02:01
*** awalende has quit IRC02:04
*** Xuchu has joined #openstack-cinder02:12
*** davee_ has quit IRC02:16
*** davee_ has joined #openstack-cinder02:17
*** Xuchu has quit IRC02:31
*** yaawang has quit IRC02:36
*** yaawang has joined #openstack-cinder02:36
*** Xuchu has joined #openstack-cinder02:46
*** n-saito has joined #openstack-cinder02:49
*** Liang__ has joined #openstack-cinder03:38
*** jbernard has quit IRC03:45
*** jbernard has joined #openstack-cinder03:46
*** Xuchu has quit IRC03:48
*** yaawang has quit IRC04:01
*** yaawang has joined #openstack-cinder04:02
*** bhagyashris has joined #openstack-cinder04:09
*** udesale has joined #openstack-cinder04:13
*** baojg has quit IRC04:17
*** baojg has joined #openstack-cinder04:19
*** dpawlik has joined #openstack-cinder04:25
*** dpawlik has quit IRC04:29
*** Xuchu has joined #openstack-cinder04:43
*** tkajinam has quit IRC05:02
*** udesale has quit IRC05:15
*** udesale has joined #openstack-cinder05:16
*** tkajinam has joined #openstack-cinder05:33
openstackgerritAbhishek Kekane proposed openstack/cinder master: Support multiple stores of Glance  https://review.opendev.org/66167605:41
*** udesale has quit IRC05:42
*** udesale has joined #openstack-cinder05:51
*** bhagyashris has quit IRC05:57
*** bhagyashris has joined #openstack-cinder05:58
*** ircuser-1 has joined #openstack-cinder06:19
*** jkulik has joined #openstack-cinder06:19
*** zhanglong has joined #openstack-cinder06:22
*** sapd1_x has joined #openstack-cinder06:23
*** dpawlik has joined #openstack-cinder06:26
*** dpawlik has quit IRC06:30
*** yaawang has quit IRC06:47
*** yaawang has joined #openstack-cinder06:48
*** dpawlik has joined #openstack-cinder07:00
*** tkajinam_ has joined #openstack-cinder07:18
*** tkajinam_ has quit IRC07:19
openstackgerritRajat Dhasmana proposed openstack/cinder master: Fix: Purge volume record when attachment exists  https://review.opendev.org/69667607:20
*** tkajinam_ has joined #openstack-cinder07:20
*** tkajinam has quit IRC07:21
openstackgerritRajat Dhasmana proposed openstack/cinder master: Fix: Purge volume record when attachment exists  https://review.opendev.org/69667607:23
*** sapd1_x has quit IRC07:23
*** Kuirong has joined #openstack-cinder07:24
openstackgerritRajat Dhasmana proposed openstack/cinder master: Fix: Purge volume record when attachment exists  https://review.opendev.org/69667607:26
*** whoami-rajat has joined #openstack-cinder07:26
*** rosmaita has quit IRC07:44
*** yaawang has quit IRC07:46
*** yaawang has joined #openstack-cinder07:48
*** CobHead has quit IRC07:50
*** jc__ has joined #openstack-cinder07:51
*** sahid has joined #openstack-cinder07:51
*** trident has quit IRC07:51
*** jc_ has quit IRC07:51
*** johanssone has quit IRC07:51
*** trident has joined #openstack-cinder07:51
*** johanssone has joined #openstack-cinder07:52
*** rosmaita has joined #openstack-cinder07:57
*** awalende has joined #openstack-cinder08:14
*** tesseract has joined #openstack-cinder08:16
*** Roamer` has joined #openstack-cinder08:26
*** n-saito has quit IRC08:28
*** tosky has joined #openstack-cinder08:29
lyarwoodgeguileo: https://review.opendev.org/#/q/topic:remove-iscsi-encryption-workaround - morning, I've added you to the os-brick changes removing your iscsi encryption workaround btw, reviews welcome if you have time this week :)08:35
lyarwoodhttps://review.opendev.org/#/c/695564/ - also looking for a second core review on this change if anyone else has time today.08:39
*** udesale has quit IRC08:43
*** udesale has joined #openstack-cinder08:44
awalendeDid anyone had issues deleting a barbican encrypted volume with an admin account? I'll get an "No Secret found" exception :( The regular owner of the volume is still able to delete it.08:46
lyarwoodawalende: that sounds like policy on the barbican side is blocking admin accounts from modifying user secrets, that's actually a sane thing to do. Cinder should however ignore that and just nuke the volume anyway if policy allows it so I'd write that up as a bug.08:54
awalendeIs the cinder issue tracker on launchpad? I'd like to  file an issue for this09:01
*** martinkennelly has joined #openstack-cinder09:03
*** e0ne has joined #openstack-cinder09:08
*** tkajinam_ has quit IRC09:20
*** dpawlik has quit IRC09:34
*** dasp has quit IRC09:42
*** dasp has joined #openstack-cinder09:43
*** yaawang has quit IRC09:45
*** yaawang has joined #openstack-cinder09:46
openstackgerritRajat Dhasmana proposed openstack/cinder master: Fix: Purge volume record when attachment exists  https://review.opendev.org/69667609:56
*** davidsha has joined #openstack-cinder09:59
*** dpawlik has joined #openstack-cinder10:02
*** Liang__ has quit IRC10:05
*** rcernin has quit IRC10:06
*** dpawlik has quit IRC10:07
*** pcaruana has joined #openstack-cinder10:08
*** dpawlik has joined #openstack-cinder10:29
*** lpetrut has joined #openstack-cinder10:34
*** Xuchu has quit IRC10:42
*** udesale has quit IRC10:44
*** dpawlik has quit IRC10:46
geguileolyarwood: reviewed the os-brick patches10:47
geguileolyarwood: thank you very much for working on this10:49
*** udesale has joined #openstack-cinder10:58
*** zhanglong has quit IRC10:59
*** ociuhandu has joined #openstack-cinder11:02
*** dpawlik has joined #openstack-cinder11:02
*** dpawlik has quit IRC11:06
*** dpawlik has joined #openstack-cinder11:10
*** ociuhandu has quit IRC11:12
*** laurent\ has quit IRC11:30
*** laurent\ has joined #openstack-cinder11:31
*** dpawlik has quit IRC11:32
*** vishalmanchanda has joined #openstack-cinder11:34
*** dpawlik has joined #openstack-cinder11:36
*** awalende_ has joined #openstack-cinder11:37
*** bhagyashris has quit IRC11:40
*** trident has quit IRC11:40
*** awalende has quit IRC11:41
*** trident has joined #openstack-cinder11:42
*** ociuhandu has joined #openstack-cinder11:50
*** lpetrut has quit IRC11:53
*** ociuhandu has quit IRC12:26
*** ociuhandu has joined #openstack-cinder12:28
*** udesale has quit IRC12:46
*** udesale has joined #openstack-cinder12:47
*** enriquetaso has joined #openstack-cinder12:52
*** dasp has quit IRC13:12
*** dasp has joined #openstack-cinder13:12
*** ociuhandu has quit IRC13:26
*** ociuhandu has joined #openstack-cinder13:27
*** ociuhandu has quit IRC13:33
*** lpetrut has joined #openstack-cinder13:40
*** eharney has quit IRC13:58
*** dave-mccowan has joined #openstack-cinder14:02
*** senrique_ has joined #openstack-cinder14:05
*** enriquetaso has quit IRC14:05
*** Kuirong has quit IRC14:14
*** Xuchu has joined #openstack-cinder14:14
*** Kuirong has joined #openstack-cinder14:17
*** Xuchu has quit IRC14:18
*** whoami-rajat has quit IRC14:21
*** bhagyashris has joined #openstack-cinder14:23
*** trident has quit IRC14:24
*** trident has joined #openstack-cinder14:25
*** bhagyashris has quit IRC14:29
*** pcaruana has quit IRC14:33
*** dave-mccowan has quit IRC14:42
*** Kuirong has quit IRC14:53
*** ociuhandu has joined #openstack-cinder14:58
*** Lucas_Gray has joined #openstack-cinder15:01
*** martinkennelly has quit IRC15:02
openstackgerritMerged openstack/os-brick master: iscsi: Add _get_device_link retry when waiting for /dev/disk/by-id/ to populate  https://review.opendev.org/69556415:03
*** pcaruana has joined #openstack-cinder15:04
*** eharney has joined #openstack-cinder15:08
*** udesale has quit IRC15:10
*** udesale has joined #openstack-cinder15:10
*** bhagyashris has joined #openstack-cinder15:14
openstackgerritEric Harney proposed openstack/cinder stable/rocky: Increase cpu limit for image conversion  https://review.opendev.org/69711015:14
*** bhagyashris has quit IRC15:19
*** bhagyashris has joined #openstack-cinder15:25
*** awalende_ has quit IRC15:25
*** awalende has joined #openstack-cinder15:26
*** awalende has quit IRC15:30
*** bhagyashris has quit IRC15:35
*** tesseract has quit IRC15:37
openstackgerritLee Yarwood proposed openstack/os-brick stable/train: iscsi: Add _get_device_link retry when waiting for /dev/disk/by-id/ to populate  https://review.opendev.org/69711515:39
openstackgerritLee Yarwood proposed openstack/os-brick stable/stein: iscsi: Add _get_device_link retry when waiting for /dev/disk/by-id/ to populate  https://review.opendev.org/69711615:39
openstackgerritLee Yarwood proposed openstack/os-brick stable/rocky: iscsi: Add _get_device_link retry when waiting for /dev/disk/by-id/ to populate  https://review.opendev.org/69711715:39
openstackgerritLee Yarwood proposed openstack/os-brick stable/queens: iscsi: Add _get_device_link retry when waiting for /dev/disk/by-id/ to populate  https://review.opendev.org/69711815:40
*** ociuhandu has quit IRC15:42
*** ociuhandu has joined #openstack-cinder15:43
*** tesseract has joined #openstack-cinder15:48
*** ociuhandu has quit IRC15:48
geguileorosmaita: you already reviewed rocky's version of this trivial backport... https://review.opendev.org/#/c/696384/115:49
*** ociuhandu has joined #openstack-cinder15:49
geguileoeharney: ^ same for you   XD15:49
*** jmlowe has quit IRC15:52
rosmaitageguileo: that was a good fix to get backported16:02
rosmaitageguileo: i still owe you a review on https://review.opendev.org/#/c/696375/, will get to that a bit later today16:03
geguileorosmaita: thanks! :-)16:04
*** jmlowe has joined #openstack-cinder16:09
hemna_mep16:14
*** udesale has quit IRC16:17
*** dave-mccowan has joined #openstack-cinder16:19
hemna_geguileo so how is it that config loading works for cinder, but not cinderlib ?16:21
geguileohemna_: because cinder loads from files and command line16:21
geguileohemna_: but cinderlib must load from parameters16:21
hemna_from parameters?   meaning python strings ?16:21
geguileoand cannot read from the command line (that belongs to the program using cinderlib)16:21
geguileoyou initialize a backend calling cinderlib.Backend with the configuration16:22
geguileoexample for LVM16:22
geguileolvm = cl.Backend(volume_driver='cinder.volume.drivers.lvm.LVMVolumeDriver',16:22
geguileo                 volume_group='cinder-volumes',16:22
geguileo                 target_protocol='iscsi',16:22
geguileo                 target_helper='lioadm',16:22
geguileo                 volume_backend_name='lvm_iscsi')16:22
geguileoSo cinderlib needs to set those in oslo.config16:23
geguileobecause we don't want to start creating temporary configuration files that we have to clean up afterwards16:23
geguileoand we would still need to tell oslo config not to read from the command line16:23
hemna_and there is no helper in cinderlib to load from a cinder.conf ?16:23
geguileothere is a tool that converts a cinder.conf file to something that can be used with cinderlib16:24
geguileothe functional tests use it16:24
hemna_url?16:24
geguileoit's cinder-cfg-to-cinderlib-code16:24
geguileohemna_: https://github.com/openstack/cinderlib/blob/master/setup.cfg#L4316:25
geguileothe code is https://github.com/openstack/cinderlib/blob/master/cinderlib/cmd/cinder_cfg_to_python.py16:27
*** dpawlik has quit IRC16:27
geguileothat one converts oslo config options into Python elements (list, dictionaries, etc)16:27
geguileobut then when cinderlib create de StringIO object to simulate cinder.conf file we need to do the reverse16:27
hemna_I guess I'm trying to understand the need for the conversion, since we instantiate the same drivers16:28
geguileohemna_: oh, right16:28
geguileohemna_: that's because drivers do weird stuff16:28
hemna_and afaik the drivers want a config object16:28
geguileoand some actually add configuration options dynamically and reload the file16:29
hemna_ugh wtf16:29
geguileoyup16:29
geguileolot's of fun16:29
geguileoI used to just pass an oslo configuration object with the parameters set by cinderlib...16:29
geguileobut NetApp did some weird stuff16:30
geguileolet me check if I can find it again16:30
*** davidsha has quit IRC16:33
geguileohemna_: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/dataontap/utils/utils.py#L5016:34
*** lpetrut has quit IRC16:38
*** ociuhandu has quit IRC16:41
openstackgerritGorka Eguileor proposed openstack/os-brick stable/rocky: Fix FC scan too broad  https://review.opendev.org/69713316:46
geguileoeharney: rosmaita Apparently I'm dumb as a rock... And somehow forgot to backport the patch to Rocky ^ (though I could have sworn I did)16:46
eharneygeguileo: it's ok, you're still a pretty sharp rock16:47
geguileorofl rofl16:47
rosmaitawhat eharney said16:47
rosmaitageguileo: glad you caught that16:50
geguileome too16:50
rosmaitai would've sworn that i checked rocky16:50
rosmaitaguess that was a different patch16:50
geguileoyeah, you -1 the other patch in queens that was missing the rocky counterpart16:51
geguileoand I though the rocky patch had already merged...16:51
smcginnishttps://review.opendev.org/#/c/696384/1 hasn't landed in rocky yet either.16:58
*** tesseract has quit IRC17:03
*** dcdamien has joined #openstack-cinder17:03
*** sahid has quit IRC17:14
*** Lucas_Gray has quit IRC17:15
*** senrique_ has quit IRC17:20
*** senrique_ has joined #openstack-cinder17:20
*** e0ne has quit IRC17:35
*** dpawlik has joined #openstack-cinder17:37
*** dpawlik has quit IRC17:41
openstackgerritJon Bernard proposed openstack/cinder master: RBD: catch argument exceptions when configuring multiattach  https://review.opendev.org/69437917:58
*** dcdamien has quit IRC18:00
*** ociuhandu has joined #openstack-cinder18:04
*** ociuhandu has quit IRC18:05
*** ociuhandu has joined #openstack-cinder18:07
*** ociuhandu has quit IRC18:12
*** jmlowe has quit IRC18:15
*** jmlowe has joined #openstack-cinder18:16
*** whoami-rajat has joined #openstack-cinder18:17
*** ociuhandu has joined #openstack-cinder18:18
*** vishalmanchanda has quit IRC18:21
*** vishalmanchanda has joined #openstack-cinder18:21
openstackgerritMerged openstack/cinder master: PowerMax Driver - retype attached replication fix  https://review.opendev.org/68642918:21
openstackgerritMerged openstack/os-brick stable/queens: Handle None value 'inititator_target_map'  https://review.opendev.org/69638418:23
*** jmlowe has quit IRC18:24
*** eharney has quit IRC18:24
*** senrique_ has quit IRC18:26
*** awalende has joined #openstack-cinder18:27
*** awalende has quit IRC18:31
*** ociuhandu has quit IRC18:39
*** jmlowe has joined #openstack-cinder18:39
*** ociuhandu has joined #openstack-cinder18:40
*** ociuhandu has quit IRC18:44
*** gmann is now known as gmann_afk19:00
openstackgerritRajat Dhasmana proposed openstack/cinder master: Fix: DBReferenceError while purging volume records  https://review.opendev.org/69667619:13
*** senrique_ has joined #openstack-cinder19:16
*** awalende has joined #openstack-cinder19:20
*** awalende has quit IRC19:24
*** whoami-rajat has quit IRC19:25
*** e0ne has joined #openstack-cinder19:26
*** senrique__ has joined #openstack-cinder19:30
*** senrique_ has quit IRC19:33
*** martinkennelly has joined #openstack-cinder19:34
*** eharney has joined #openstack-cinder19:36
*** dpawlik has joined #openstack-cinder19:37
*** dpawlik has quit IRC19:42
openstackgerritMerged openstack/os-brick stable/rocky: Handle None value 'inititator_target_map'  https://review.opendev.org/69652119:52
openstackgerritMerged openstack/cinder stable/stein: Increase cpu limit for image conversion  https://review.opendev.org/69361019:56
openstackgerritMerged openstack/cinder master: Removed unnecessary lines  https://review.opendev.org/69662719:56
openstackgerritMerged openstack/cinderlib master: Fix LVM extend volume  https://review.opendev.org/69452419:56
openstackgerritMerged openstack/cinderlib master: Fix cloning metadata storage  https://review.opendev.org/69010119:56
openstackgerritMerged openstack/cinder master: Tests: Isolate rbd deferred deletion tests  https://review.opendev.org/69524619:56
openstackgerritEric Harney proposed openstack/cinder stable/train: Tests: Isolate rbd deferred deletion tests  https://review.opendev.org/69717319:58
*** ociuhandu has joined #openstack-cinder20:03
*** ociuhandu has quit IRC20:08
*** gmann_afk is now known as gmann20:25
*** e0ne has quit IRC20:30
*** senrique_ has joined #openstack-cinder20:32
*** senrique__ has quit IRC20:35
*** vesper11 has quit IRC20:49
*** vesper11 has joined #openstack-cinder20:51
*** e0ne has joined #openstack-cinder21:09
*** e0ne has quit IRC21:11
*** e0ne has joined #openstack-cinder21:15
*** e0ne has quit IRC21:18
*** martinkennelly has quit IRC21:34
*** awalende has joined #openstack-cinder22:00
*** awalende has quit IRC22:04
*** pcaruana has quit IRC22:06
openstackgerritSofia Enriquez proposed openstack/cinder master: Allow removing NFS snapshots in error status  https://review.opendev.org/67913822:26
*** senrique_ has quit IRC22:35
*** tkajinam has joined #openstack-cinder22:56
*** rcernin has joined #openstack-cinder22:57
*** ociuhandu has joined #openstack-cinder23:30
*** ociuhandu has quit IRC23:36
*** dpawlik has joined #openstack-cinder23:39
*** dpawlik has quit IRC23:43
« Monday, 2019-12-02 Index Wednesday, 2019-12-04 »

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