Friday, 2019-02-15

*** whoami-rajat has quit IRC00:00
*** markvoelker has joined #openstack-cinder00:09
*** _alastor_ has quit IRC00:31
*** markvoelker has quit IRC00:42
*** Florian has joined #openstack-cinder00:46
*** lemko has quit IRC00:47
openstackgerritwanghao proposed openstack/cinder master: Support to query volume filter by changes since/before  https://review.openstack.org/35268800:58
*** phuongnh has joined #openstack-cinder01:01
sapd1_smcginnis: Yeah. I'm writing a new weigher. thanks01:19
*** brinzhang has joined #openstack-cinder01:28
*** enriquetaso has quit IRC01:31
*** _alastor_ has joined #openstack-cinder01:33
*** zul has quit IRC01:37
*** _alastor_ has quit IRC01:38
*** markvoelker has joined #openstack-cinder01:39
openstackgerritMerged openstack/cinder master: Remove LOCI publishing from the post pipeline  https://review.openstack.org/63697902:05
*** erlon_ has quit IRC02:07
*** mriedem has quit IRC02:11
*** markvoelker has quit IRC02:12
*** sapd1 has joined #openstack-cinder02:13
openstackgerritMerged openstack/cinder master: Fix HPE3PAR not returning cached stats  https://review.openstack.org/63392902:18
*** KeithMnemonic has quit IRC02:18
*** KeithMnemonic has joined #openstack-cinder02:19
openstackgerritLeopard Ma proposed openstack/cinder master: Rollback the quota_usages table when failed to create a incremental backup without parent backup  https://review.openstack.org/62679002:25
openstackgerritLeopard Ma proposed openstack/cinder master: Rollback the quota_usages table when failed to create a incremental backup  https://review.openstack.org/62679002:32
*** Dinesh_Bhor has joined #openstack-cinder02:39
*** psachin has joined #openstack-cinder02:56
*** markvoelker has joined #openstack-cinder03:09
*** markvoelker has quit IRC03:43
*** dave-mccowan has quit IRC04:03
*** markvoelker has joined #openstack-cinder04:40
*** doubletao has quit IRC04:45
*** rosmaita has left #openstack-cinder04:47
*** markvoelker has quit IRC05:12
*** psachin has quit IRC05:38
*** whoami-rajat has joined #openstack-cinder06:08
*** markvoelker has joined #openstack-cinder06:09
*** e0ne has joined #openstack-cinder06:18
*** tkajinam_ has joined #openstack-cinder06:33
*** tkajinam has quit IRC06:35
*** markvoelker has quit IRC06:43
*** e0ne has quit IRC06:43
*** e0ne has joined #openstack-cinder06:46
*** e0ne has quit IRC06:48
*** psachin has joined #openstack-cinder06:49
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR: Provide an option duing creation of volume from snapshot  https://review.openstack.org/63411906:59
*** Luzi has joined #openstack-cinder07:02
*** lemko has joined #openstack-cinder07:02
*** e0ne has joined #openstack-cinder07:05
*** e0ne has quit IRC07:07
openstackgerritzhangxiaohan proposed openstack/cinder master: Update rbd image discard function to clear with 2**31 -1 bytes limited  https://review.openstack.org/63681807:07
openstackgerritzhangxiaohan proposed openstack/cinder master: Update rbd image discard function to clear with 2**31 -1 bytes limited  https://review.openstack.org/63681807:10
*** tkajinam_ has quit IRC07:14
*** Dinesh_Bhor has quit IRC07:27
*** Dinesh_Bhor has joined #openstack-cinder07:27
*** tkajinam has joined #openstack-cinder07:28
*** tkajinam_ has joined #openstack-cinder07:29
*** tkajinam has quit IRC07:32
*** markvoelker has joined #openstack-cinder07:40
*** bhagyashris has joined #openstack-cinder07:59
*** KeithMnemonic has quit IRC08:08
*** KeithMnemonic has joined #openstack-cinder08:08
*** rchurch has quit IRC08:09
*** rchurch has joined #openstack-cinder08:10
*** markvoelker has quit IRC08:12
*** tkajinam_ has quit IRC08:20
*** yikun has quit IRC08:25
*** erlon_ has joined #openstack-cinder09:06
*** markvoelker has joined #openstack-cinder09:10
*** davidsha has joined #openstack-cinder09:19
*** luizbag has joined #openstack-cinder09:39
*** markvoelker has quit IRC09:42
*** vishakha has joined #openstack-cinder09:46
*** bhagyashris has quit IRC09:47
*** ganso has joined #openstack-cinder09:47
*** erlon_ has quit IRC09:56
*** Dinesh_Bhor has quit IRC09:58
*** erlon has joined #openstack-cinder10:24
*** ociuhandu has joined #openstack-cinder10:24
*** dpawlik has joined #openstack-cinder10:31
*** erlon has quit IRC10:31
dpawlikhi, will be multiattach available in queens for rbd driver (https://review.openstack.org/#/c/595827/) ?10:31
whoami-rajatdpawlik:  IMO, we don't do feature backport so it won't be available for queens.10:33
dpawlikso I can do that only for "downstream" :(10:34
dpawlikthanks whoami-rajat10:34
*** markvoelker has joined #openstack-cinder10:39
*** erlon has joined #openstack-cinder10:52
*** erlon has quit IRC10:52
*** erlon has joined #openstack-cinder10:53
*** erlon has quit IRC10:54
*** erlon has joined #openstack-cinder10:54
*** erlon has quit IRC10:54
*** erlon has joined #openstack-cinder10:55
*** brinzhang has quit IRC11:10
*** markvoelker has quit IRC11:13
*** arne_wiebalck has quit IRC11:20
*** arne_wiebalck_ has joined #openstack-cinder11:20
*** zul has joined #openstack-cinder11:27
*** _alastor_ has joined #openstack-cinder11:45
*** _alastor_ has quit IRC11:50
*** markvoelker has joined #openstack-cinder12:10
erlonwhoami-rajat, hey, have fixed the test: https://review.openstack.org/62057712:12
erlonwhen you have a chance12:12
erlonwhoami-rajat, can you also add these 3 patches to your review queue? https://review.openstack.org/633596, https://review.openstack.org/606959, https://review.openstack.org/63380412:14
*** mma has joined #openstack-cinder12:19
mmaWhen trying to use LocalityInstanceFilter for cinder scheduler I get this error: http://paste.openstack.org/show/745155/    I can't seem to find a way to configure cinder to understand how to do the lookup for the VM against Nova. This is running on a Queens environment and the command is made as an admin user.12:22
*** Luzi has quit IRC12:29
*** lpetrut has joined #openstack-cinder12:40
*** markvoelker has quit IRC12:42
openstackgerritSean McGinnis proposed openstack/cinder master: Limit RBD discard to 32 bit chunks  https://review.openstack.org/63681812:46
*** e0ne has joined #openstack-cinder12:47
*** whoami-rajat has quit IRC12:57
*** lemko has quit IRC13:15
*** thgcorrea has joined #openstack-cinder13:16
*** Florian has quit IRC13:17
*** vishakha has quit IRC13:18
*** dave-mccowan has joined #openstack-cinder13:19
*** phuongnh has quit IRC13:22
*** e0ne has quit IRC13:28
*** e0ne has joined #openstack-cinder13:30
*** enriquetaso has joined #openstack-cinder13:33
*** lyarwood has joined #openstack-cinder13:33
*** mriedem has joined #openstack-cinder13:34
lyarwoodAfternoon/Morning all, I'm working on https://bugs.launchpad.net/nova/+bug/1803961 for Nova where we aren't calling the migrate_volume_completion c-api during the migration of a volume without also retyping the volume.13:35
openstackLaunchpad bug 1803961 in OpenStack Compute (nova) "Nova doesn't call migrate_volume_completion after cinder volume migration" [Undecided,In progress] - Assigned to Matthew Booth (mbooth-9)13:35
lyarwoodreading https://docs.openstack.org/cinder/latest/admin/blockstorage-volume-migration.html suggests that volume migration without retyping is still supported13:35
lyarwoodbut everytime I try this locally the volume ends up with a type so I can't migrate between the available backends13:35
lyarwoodam I missing something here? Do I need to add an additional type that both backends can support?13:36
*** psachin has quit IRC13:36
*** efried is now known as fried_rice13:37
*** markvoelker has joined #openstack-cinder13:39
*** whoami-rajat has joined #openstack-cinder13:41
*** dpawlik has quit IRC13:49
*** dpawlik has joined #openstack-cinder13:50
lyarwoodright, having an additional type without a backend listed works. Does anyone know if you can list multiple backends?13:50
*** sapd1 has quit IRC13:55
whoami-rajaterlon: sure, i'm out right now, will check in some time.13:55
whoami-rajatlyarwood: not sure if I understand correctly, we use 'cinder get-pools' for listing backends. Also can use 'cinder-manage service list'13:58
lyarwoodwhoami-rajat: yeah sorry I was asking if you could list multiple backends as extra specs of a volume type14:03
lyarwoodwhoami-rajat: I think the answer is no14:04
*** dpawlik has quit IRC14:06
*** dpawlik has joined #openstack-cinder14:09
*** eharney has joined #openstack-cinder14:10
smcginnislyarwood: It depends on your extra specsssss.14:11
smcginnisSorry, wireless keyboard battery about to die I think. :)14:12
smcginnisYou might not list any backends.14:12
*** markvoelker has quit IRC14:12
smcginnisSome explicitly tie a volume type to a specific backend by using volume_backend_name. Some just have attributes they want and let the scheduler pick which backend to use that meets those needs.14:13
*** lemko has joined #openstack-cinder14:14
lyarwoodsmcginnis: ack understood, so pure migration is still a valid use case. Appears doing this live against a running instance has been broken for a while now and we don't have tempest test coverage. I'll knock something togther now.14:16
smcginnislyarwood: Been awhile since I tested it myself. Thanks for digging in to that.14:17
lyarwoodnp14:17
*** _alastor_ has joined #openstack-cinder14:19
lyarwoodsmcginnis: one final question if you have a second, any idea why there are no API docs for os-migrate_volume_completion?14:19
smcginnislyarwood: Hmm. Maybe we missed it as an API "extension". Not sure.14:20
lyarwoodnp I can look at adding them as well assuming there's not a valid reason14:20
smcginnisNo reason it shouldn't be in the api-ref that I am aware of.14:20
*** e0ne has quit IRC14:33
*** lpetrut has quit IRC14:39
openstackgerritEric Harney proposed openstack/cinder master: Limit RBD discard to 32 bit chunks  https://review.openstack.org/63681814:39
*** ociuhandu_ has quit IRC14:41
*** ociuhandu_ has joined #openstack-cinder14:42
*** thgcorrea has quit IRC14:51
*** dpawlik has quit IRC14:52
*** dpawlik has joined #openstack-cinder14:53
*** thgcorrea has joined #openstack-cinder15:00
*** dpawlik has quit IRC15:05
*** e0ne has joined #openstack-cinder15:07
*** thgcorrea has quit IRC15:09
*** takamatsu has joined #openstack-cinder15:10
*** markvoelker has joined #openstack-cinder15:10
smcginnisjbernard: Hey Jon. I actually have a glance question for you (potentially could be relevant to cinder too).15:10
smcginnisjbernard: I've seen a couple like this now - https://review.openstack.org/#/c/635230/1/glance_store/_drivers/rbd.py15:11
smcginnisjbernard: Wondering if that is a bug in librados that should be addressed. Seems odd to me that consuming code needs to know to decode the output from that.15:11
smcginnisAh, and actually, we do use that in cinder/volume/drivers/rbd.py too.15:12
smcginnisOh right - https://review.openstack.org/#/c/635195/15:13
*** thgcorrea has joined #openstack-cinder15:14
eharneyit looks like get_fsid() returns bytes... it does seem odd15:16
smcginnisI would think here would be a more approriate place to fix this - https://github.com/ceph/ceph/blob/3e585b0e7101fdc9e0abfa5619f8fa5eb943b27c/src/pybind/rados/rados.pyx#L115415:17
*** dpawlik has joined #openstack-cinder15:17
*** mchlumsky has joined #openstack-cinder15:18
eharneyget_fsid appears to be used for a lot of internal ceph messaging etc, so it's probably intended to be binary data for those cases15:18
*** _alastor_ has quit IRC15:19
*** dpawlik has quit IRC15:22
smcginnisInternally it uses the python library?15:24
eharneyno, just that the python library is mirroring what the internals do15:26
smcginnisH15:27
smcginnisH15:27
smcginnisAAAAAh15:27
* smcginnis looks for new batteries15:27
eharneywe only have to massage it in cinder because we convert it to a string to store it in a db field15:27
eharneylol15:27
smcginnisThis guy has a set of patches all doing the same thing in glace_store, cinder, and nova.15:27
smcginnisJust doing it at the point where it's used rather than fixing it at the source.15:28
*** BlackDex_ is now known as BlackDex15:28
eharneyyeah, your comment there to fix it in the _get_fsid wrapper method is definitely the right way to go15:28
*** jistr is now known as jistr|mtg15:32
*** kmalloc is now known as needscoffee15:32
smcginniseharney: It does seem like a bug in the rbd python bindings though, right? I believe it's a string under python 2 but bytes under python 3. That seems like a compatibility bug, right?15:35
eharneysmcginnis: isn't it just about the fact that in python 2, strings are bytes but in python 3 they aren't?  maybe i missed something15:36
smcginnisYeah, I think that's the issue.15:37
smcginnisSo depending on what python version you are using, you get different results from that library.15:37
smcginnisConsumer code needs to be aware of that and handle it wherever it's used rather than the library providing a consistent result.15:38
smcginnisUnrelated...15:38
smcginniscinder cores, lib freeze is coming up. We need attention on https://review.openstack.org/#/q/project:openstack/os-brick+status:open15:38
*** markvoelker has quit IRC15:42
*** jistr|mtg is now known as jistr16:04
openstackgerritSean McGinnis proposed openstack/python-brick-cinderclient-ext master: Update min tox version to 2.0  https://review.openstack.org/61210616:10
*** takamatsu_ has joined #openstack-cinder16:11
*** takamatsu has quit IRC16:12
openstackgerritSean McGinnis proposed openstack/python-brick-cinderclient-ext master: Update min tox version to 2.0  https://review.openstack.org/61210616:12
*** _alastor_ has joined #openstack-cinder16:14
thgcorreaeharney, thanks for the review, I've replied the comments on https://review.openstack.org/#/c/633596/16:15
*** takamatsu_ has quit IRC16:25
*** lemko has quit IRC16:25
*** takamatsu_ has joined #openstack-cinder16:28
*** TxGirlGeek has joined #openstack-cinder16:28
*** dims has joined #openstack-cinder16:34
*** dims has quit IRC16:38
*** markvoelker has joined #openstack-cinder16:39
*** ociuhandu_ has quit IRC16:41
*** ociuhandu_ has joined #openstack-cinder16:42
*** ociuhandu_ has quit IRC16:44
*** dims has joined #openstack-cinder16:45
*** Florian has joined #openstack-cinder16:48
*** dims has quit IRC16:53
*** dims has joined #openstack-cinder16:55
*** eharney has quit IRC16:59
*** e0ne has quit IRC16:59
*** dpawlik has joined #openstack-cinder17:01
*** davidsha has quit IRC17:02
*** dpawlik has quit IRC17:05
*** markvoelker has quit IRC17:13
*** TxGirlGeek has quit IRC17:21
*** walshh_ has joined #openstack-cinder17:22
walshh_Hi, is anyone available to take a quick look at [16:18] <walshh__> Hi, to the to the infra-root sysadmins.  Could I be added to the new devstack-plugin-vmax-release.  Details in https://review.openstack.org/#/c/635890/  and https://review.openstack.org/#/c/635546.  Thank you!17:24
walshh_wrong message apologies :-)17:27
walshh_Is anyone available to look at https://review.openstack.org/#/c/635546 and https://review.openstack.org/#/c/632112.  Thank you!17:29
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax driver - do an rdf_group check earlier  https://review.openstack.org/63378717:40
smcginniswalshh_: I'll take a look.17:42
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Unit Test Refactoring  https://review.openstack.org/63611817:43
*** TxGirlGeek has joined #openstack-cinder17:43
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Update min tox version to 2.0  https://review.openstack.org/61210617:46
*** eharney has joined #openstack-cinder17:46
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax driver - performance improvements  https://review.openstack.org/63638917:46
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Unmanage Snapshot Improvements  https://review.openstack.org/63664017:48
openstackgerritMerged openstack/python-cinderclient master: Add Python 3.6 classifier to setup.cfg  https://review.openstack.org/62082017:52
*** arne_wiebalck_ has quit IRC17:54
openstackgerritMerged openstack/os-brick master: Don't warn on missing dmidecode  https://review.openstack.org/63097118:02
openstackgerritMerged openstack/cinder stable/ocata: Unity: fail to detach lun when auto zone enabled  https://review.openstack.org/62585518:03
*** ociuhandu_ has joined #openstack-cinder18:06
*** ociuhandu has quit IRC18:09
*** markvoelker has joined #openstack-cinder18:10
*** ociuhandu_ has quit IRC18:10
*** Florian has quit IRC18:14
*** mriedem has quit IRC18:18
*** lemko has joined #openstack-cinder18:22
openstackgerritMerged openstack/os-brick master: Add missing params in NoOpEncryptor  https://review.openstack.org/63176818:22
openstackgerritMerged openstack/os-brick master: Remove time checks from test_custom_execute_timeout_* tests  https://review.openstack.org/62566918:22
*** mriedem has joined #openstack-cinder18:24
*** ociuhandu has joined #openstack-cinder18:24
openstackgerritMerged openstack/os-brick master: iSCSI: log exception if portals not found  https://review.openstack.org/63467518:25
openstackgerritMerged openstack/cinder master: Change a function name of swift backup  https://review.openstack.org/62203918:27
openstackgerritMerged openstack/cinder master: Add check_encryption_provider to volume utils  https://review.openstack.org/63254118:27
*** ociuhandu has quit IRC18:28
*** mchlumsky has quit IRC18:42
*** mchlumsky has joined #openstack-cinder18:43
*** markvoelker has quit IRC18:43
openstackgerritVictoria Martinez de la Cruz proposed openstack/cinder master: Extend remove_version_from_href support  https://review.openstack.org/63599518:44
openstackgerritMerged openstack/python-cinderclient master: Change bash completion dir permissions to 0750  https://review.openstack.org/62301518:47
*** psachin has joined #openstack-cinder18:51
*** thgcorrea has quit IRC18:56
*** erlon has quit IRC18:57
*** TxGirlGeek has quit IRC19:07
*** TxGirlGe_ has joined #openstack-cinder19:07
*** Florian has joined #openstack-cinder19:09
*** luizbag has quit IRC19:11
*** swegener has quit IRC19:15
*** e0ne has joined #openstack-cinder19:19
*** markvoelker has joined #openstack-cinder19:41
*** psachin has quit IRC19:58
-openstackstatus- NOTICE: The StoryBoard service on storyboard.openstack.org is offline momentarily for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-February/002666.html20:02
*** swegener has joined #openstack-cinder20:10
*** TxGirlGe_ has quit IRC20:11
*** markvoelker has quit IRC20:13
*** TxGirlGeek has joined #openstack-cinder20:13
openstackgerritCorey Bryant proposed openstack/python-cinderclient master: add python 3.7 unit test job  https://review.openstack.org/61003620:24
openstackgerritAngela Smith proposed openstack/cinder master: Fix for auth version change in Brcd HTTP  https://review.openstack.org/63729720:26
*** itlinux has joined #openstack-cinder20:30
*** itlinux has quit IRC20:34
openstackgerritMerged openstack/cinder stable/queens: VMAX driver doc - notification of fix on AFA OS upgrade issue  https://review.openstack.org/63207320:37
*** itlinux has joined #openstack-cinder20:38
openstackgerritMerged openstack/cinder master: Add test case: retype volume when driver not initialized  https://review.openstack.org/61898720:41
*** TxGirlGeek has quit IRC20:45
*** TxGirlGeek has joined #openstack-cinder20:45
*** TxGirlGeek has quit IRC20:45
*** e0ne has quit IRC20:51
*** Florian has quit IRC20:56
openstackgerritMerged openstack/cinder master: VNX Driver: delete_hba() instead of remove_hba()  https://review.openstack.org/63606421:06
openstackgerritMerged openstack/cinder master: Add empty check before using zone driver  https://review.openstack.org/63609821:06
*** markvoelker has joined #openstack-cinder21:10
*** ganso has quit IRC21:17
*** mchlumsky has quit IRC21:17
*** jidar has joined #openstack-cinder21:19
openstackgerritDavid Vallee Delisle proposed openstack/cinder stable/rocky: VNX Driver: delete_hba() instead of remove_hba()  https://review.openstack.org/63621221:30
*** whoami-rajat has quit IRC21:30
*** markvoelker has quit IRC21:43
*** itlinux has quit IRC22:12
*** markvoelker has joined #openstack-cinder22:40
*** itlinux has joined #openstack-cinder22:42
*** dave-mccowan has quit IRC22:45
openstackgerritAngela Smith proposed openstack/cinder master: Fix for auth version change in Brcd HTTP  https://review.openstack.org/63729722:51
*** itlinux has quit IRC22:54
*** itlinux has joined #openstack-cinder22:55
openstackgerritMerged openstack/cinder master: Remove 'tgtadm' from rootwrap  https://review.openstack.org/63469823:06
*** lemko has quit IRC23:12
*** markvoelker has quit IRC23:13
*** itlinux has quit IRC23:17
*** takamatsu_ has quit IRC23:29
*** Florian has joined #openstack-cinder23:38

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