Wednesday, 2015-12-09

hemnano problemo :)00:00
*** jerrygb has quit IRC00:01
*** zhangjn has joined #openstack-cinder00:05
*** zhangjn has quit IRC00:06
*** p0rtal_ has joined #openstack-cinder00:06
*** zhangjn has joined #openstack-cinder00:06
*** p0rtal has quit IRC00:09
*** diogogmt has quit IRC00:09
*** lcurtis has quit IRC00:10
*** Apoorva has joined #openstack-cinder00:12
*** salv-orl_ has joined #openstack-cinder00:18
*** akshai has quit IRC00:18
*** bardia has quit IRC00:18
openstackgerritMerged openstack/python-cinderclient: Pass proxy environment variable to tox
*** bardia has joined #openstack-cinder00:19
openstackgerritMerged openstack/python-cinderclient: Fix comma location in comment
openstackgerritAnthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support
openstackgerritMerged openstack/python-cinderclient: Add optional argument to list subcommand
*** pck_ has joined #openstack-cinder00:20
*** hemna has quit IRC00:20
*** salv-orlando has quit IRC00:20
*** markstur has quit IRC00:20
*** hemna has joined #openstack-cinder00:20
*** markstur has joined #openstack-cinder00:20
*** pck has quit IRC00:22
*** zul has quit IRC00:22
*** jungleboyj has joined #openstack-cinder00:22
*** mordred has quit IRC00:23
*** zul has joined #openstack-cinder00:23
*** mordred has joined #openstack-cinder00:23
*** EinstCra_ has quit IRC00:24
*** zhangjn has quit IRC00:24
*** mc_nair_ has quit IRC00:26
*** chlong has joined #openstack-cinder00:28
*** takedakn has quit IRC00:30
*** mkoderer has quit IRC00:30
*** belmoreira has quit IRC00:30
*** alonma has joined #openstack-cinder00:33
mtaninoxyang: Hi, Do you think this patch is right fix?
mtaninoxyang: I think the fix conflicts your snapshot backup proposal.00:36
*** mkoderer has joined #openstack-cinder00:37
*** alonma has quit IRC00:38
hemnamtanino, xyang, what does the comment "when changing data00:39
hemnaof a volume" mean ?00:39
*** alonma has joined #openstack-cinder00:39
mtaninohemna: xyang I guess that means "restore".00:42
hemnaI guess.  the commit message should be clarified00:43
*** alonma has quit IRC00:44
mtaninoAs snapshot is dependent on a volume, we should check whther the original volume has snapshots or not when we restore a backup.00:44
hemnayah that's better :)00:44
mtaninoI imagine.00:44
hemnaok I'm out for the night...00:44
*** hemna is now known as hemnafk00:44
mtaninobye :)00:44
tbarronmtanino, lixiaoy1, xyang : so the idea here is that b/c snapshots are just deltas with the original volume, overwriting the original volume by an over-write restore would break that relationship, right?00:46
tbarronmtanino: you are however thinking that forbidding a restore in this circumstance conflicts with xyang's proposal to backup snapshots themselves?00:48
*** openstackstatus has quit IRC00:50
*** openstack has joined #openstack-cinder00:52
mtaninotbarron: But adding the check from that patch, user need to delete snapshot before restoring.00:52
tbarronmtanino, xyang, lixiaoy1: I havent' really thought through the implications of incremental backup and snapshot backup.00:52
mtaninotbarron: Ah, ok.00:52
tbarronmtanino, lixiaoy1, xyang: xyang has her hand in both of these areas, so I will wait to see what she says.  Again, mtanino, good that you brought up the issue in any case.00:54
*** zhangjn has joined #openstack-cinder00:55
mtaninotbarron: yes, I agree. Waiting xyang's idea.00:55
*** ianbrown_ has quit IRC01:03
*** EinstCrazy has joined #openstack-cinder01:03
*** zhangjn has quit IRC01:04
tbarronsomewhat related, I've recently had the (perhaps crazy) idea that we should not insist that an original volume cannot be deleted or overwritten if it has snapshots.01:05
tbarronit is true that some drivers implement snapshots as deltas to an original.01:06
*** zhangjn has joined #openstack-cinder01:06
tbarronbut others do read-only clones.01:07
tbarronor some might do a dumb copy.01:07
tbarronso core code could allow the delete (from DB, etc.) and those drivers that do a delta from an original could keep the backing entity for the origianl01:07
tbarronaround on their backend if they need to.01:08
tbarronwhy impose their way of implementation on the core code?01:08
*** shihanzhang has joined #openstack-cinder01:15
lixiaoy1tbarron: mtanino: I agree that we should not insist that an original volume cannot be deleted or overwritten. it depends on whether it is full snapshot or incremental snapshot in backend01:17
mtaninotbarron: lixiaoy1 As for the delete case, I'm not sure the background why we can't delete it.01:18
lixiaoy1tbarron:mtanino: if it is incremental snapshot, we can't delete it01:18
mtaninolixiaoy1: tbarron yes, but driver knows it.01:19
*** shz has quit IRC01:19
lixiaoy1tbarron:mtanino: Iam not sure if it allows to delete volumes when it has snapshots, whether any problems happen01:20
*** diogogmt has joined #openstack-cinder01:21
mtaninolixiaoy1: tbarron So all drivers already depends on the current restriction, we can't change the policy.01:21
mtaninofor delete case.01:22
*** krtaylor has quit IRC01:22
lixiaoy1tbarron:mtanino: no01:22
lixiaoy1tbarron:mtanino: I am not sure why at first we have such restriction01:22
mtaninolixiaoy1: tbarron ah, yes, me too01:23
lixiaoy1tbarron:mtanino: meanwhile, I don't know how snapshots are used in cinder, we only have snapshot create, delete01:23
mtaninolixiaoy1: tbarron We can create a volume from snapshot01:23
*** zhenguo has joined #openstack-cinder01:24
lixiaoy1tbarron:mtanino: oh, yeah01:24
mtaninolixiaoy1: tbarron It's main use case of snapshot , and we can backup snapshot in the future.01:25
lixiaoy1tbarron:mtanino: let me check whether cinder copies all the metadata of a volume to its snapshot when creating snapshot, if yes, seems we can delete volumes if full snapshot01:25
lixiaoy1tbarron:mtanino: very appreciate your help on this patch01:26
mtaninolixiaoy1: sure01:26
*** garthb has quit IRC01:32
lixiaoy1tbarron:mtanino: I think not copy all the volume metadata during creating snapshot.01:33
lixiaoy1tbarron:mtanino: this is in create_snapshot_in_db in volume.manager01:34
lixiaoy1mtanino:as a result if deleting the volume, such metadata info lost01:35
lixiaoy1mtanino: meanwhile same issue for extending volumes I think01:36
*** krtaylor has joined #openstack-cinder01:36
*** zhangjn_ has joined #openstack-cinder01:36
mtaninolixiaoy1: Why is it issue?01:38
lixiaoy1mtanino: which one? metadata or extend?01:38
mtaninolixiaoy1: about metadata01:39
lixiaoy1mtanino: when creating a volume from snapshot, we need to recover all the metadata01:39
*** zhangjn has quit IRC01:40
*** Lee1092 has joined #openstack-cinder01:40
lixiaoy1mtanino: if a snapshot is full snapshot, and we delete the volume, we can't recover all the metadata except we save metadata in snapshot01:40
*** alonma has joined #openstack-cinder01:41
mtaninolixiaoy1: hmm, actually I don't have much knowledge around metadata, but it might be a design or problem.01:45
lixiaoy1mtanino: I am not sure :)01:45
*** alonma has quit IRC01:45
mtaninolixiaoy1: Using volume clone(create --source-volid), it might copy full metadata01:46
lixiaoy1mtanino: yes01:46
*** esp has joined #openstack-cinder01:46
*** jamielennox is now known as jamielennox|away01:47
lixiaoy1mtanino: when creating a snapshot from a volume, it doesn't copy full metadata. as a result the snapshot depends on volumes. this is current mechanism01:47
*** xyang has quit IRC01:48
lixiaoy1anyone can talk about snapshot usages in some drivers?01:48
tbarronlixiaoy1, mtanino : so you are saying the dependency of snapshot on original volume isn't due to backend implementations where it is a delta that depends on the original, but01:49
tbarronrather that the core code has chosen not to copy full metadata, right?01:50
lixiaoy1mtanino: metadata is one aspect, another point is whether it is incremental or not in backend.01:52
openstackgerritDave McCowan proposed openstack/cinder: Check context before returning cached value
lixiaoy1mtanino: if we change to use driver interface to decide whether depends or not, metadata needs to change01:53
openstackgerritMerged openstack/cinder: XtremIO: fix missing multiattach flag
*** leeantho has quit IRC01:54
tbarronlixiaoy1, mtanino : yeah, but if xyang's proposal to backup from snapshot goes through (and I like it) then the base for incrmental could itself be a snapshot, righ?01:54
lixiaoy1tbarron: yes01:54
tbarronincremental on base dependencies are one dimension, snap on original is another.01:55
tbarronI think the first is legitimate but am questioning the second.01:55
tbarronmaybe snapshots should be from a core perspective be considered read-only clones/copies.01:56
tbarronif backends optimize them by doing deltas to an original, then that is their own concern, not core code's.01:56
tbarronlixiaoy1: because in general the core code doesn't have to keep an original in existence just becaue it has a snapshot.01:57
tbarronlixiaoy1: keeping it in existence because it has incremental backups is different.01:57
tbarronlixiaoy1: of course I said the last one wrong, keeping the base backup in existence because it has incremental is differnt.01:58
tbarronlixiaoy1: I might be confused, it's late here :-)01:58
lixiaoy1tbarron: I am also confused about snapshot although it is morning here. I begin to notice my patch may be wrong02:00
lixiaoy1tbarron: as during incremental snapshot, if the volume data are changed, the original data(dela) will be copied to snapshot if incremental02:01
tbarronlixiaoy1: I think your patch is consistent with current practice.  mtanino has however raised good issues that lead me to wonder about the current practice.02:01
tbarronlixiaoy1: some backends might be safe if the original changes when they have done an incremental snapshot; some might not be and need to keep a shadow copy of the original around.02:02
*** chlong has quit IRC02:03
*** xyang has joined #openstack-cinder02:03
tbarronlixiaoy1: my thinking at the moment is that is the backend's concern, not the core code's.02:03
lixiaoy1tbarron: but restoring the volume seems like to ios to the volume. we allow to write to the volume02:03
lixiaoy1tbarron: so do you agree to use driver interface to check depencies like mtanino proposed?02:04
tbarronlixiaoy1: you are right, so I guess everyone has to handle that in some way or another on their backend.02:04
tbarronlixiaoy1: well, it is late :-), but I am more inclined to say backends should take care of any dependencies they have themselves, rather than02:05
tbarronthe core code having to care.02:05
mtaninolixiaoy1: tbarron As for the backup case, I think above comment might be right solution.02:06
lixiaoy1tbarron: thanks, then goodbye?02:06
tbarronCore code should be able to treat the backend as if it was doing a dumb point-in-time copy when doing a snapshot.02:06
mtaninonn, I might be confused. please ignore it.02:07
tbarronlixiaoy1: oh, I'll hang out a bit, but I'll shut up and listen :-)02:08
lixiaoy1mtanino:  tbarron: I would like to send an email to discuss the problem, ok?02:10
tbarronlixiaoy1, mtanino: violating my promise :-), OK by me.02:11
lixiaoy1to check any other potential problems02:11
mtaninolixiaoy1: tbarron OK or good topic to discuss on the openstack-dev ML.02:12
*** vgridnev has quit IRC02:12
tbarronmtanino, lixiaoy1 good topic for ML I think.02:13
lixiaoy1thanks, I will send an email02:13
mtaninoso, good night!02:13
lixiaoy1good night, you two02:13
tbarronlixiaoy1, mtanino: good "night"02:14
*** Apoorva has quit IRC02:15
*** Apoorva has joined #openstack-cinder02:16
*** mtanino has quit IRC02:17
openstackgerritWilson Liu proposed openstack/cinder: Rename Huawei drivers
*** dave-mccowan has quit IRC02:20
openstackgerritMerged openstack/cinder: 3PAR: Implement v2 replication (managed)
*** p0rtal has joined #openstack-cinder02:33
*** p0rtal_ has quit IRC02:37
openstackgerritMerged openstack/cinder: Volume driver for Coho Data storage solutions
*** alonma has joined #openstack-cinder02:41
openstackgerritMerged openstack/cinder: Dynamically Pull Out Option Sections
*** alonma has quit IRC02:45
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
*** jamielennox|away is now known as jamielennox02:50
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements
openstackgerritwanghao proposed openstack/cinder-specs: Get the replica volume ref for DR
*** diogogmt has quit IRC02:53
*** jerrygb has joined #openstack-cinder02:57
*** gouthamr has joined #openstack-cinder02:58
*** jerrygb has quit IRC03:02
*** jdurgin1 has quit IRC03:07
*** gouthamr_ has joined #openstack-cinder03:08
*** gouthamr has quit IRC03:11
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror)
*** Apoorva has quit IRC03:14
*** chlong has joined #openstack-cinder03:16
*** jerrygb has joined #openstack-cinder03:18
*** links has joined #openstack-cinder03:24
*** mtanino has joined #openstack-cinder03:31
*** bardia has quit IRC03:31
itzdiliphow should be the multi pool is handled in driver?  preconfigured with set of pools or creating on fly?03:34
lixiaoy1itzdilip: as I know, some in conf file, some in extra_specs03:40
lixiaoy1itzdilip: seems preconfigured03:41
*** alonma has joined #openstack-cinder03:41
*** alonma has quit IRC03:46
*** jerrygb has quit IRC03:48
*** jerrygb has joined #openstack-cinder03:48
*** chlong has quit IRC03:49
*** alonma has joined #openstack-cinder03:51
*** alejandrito has quit IRC03:52
*** jerrygb has quit IRC03:52
*** chlong has joined #openstack-cinder03:53
*** [1]Thelo has joined #openstack-cinder03:55
*** alonma has quit IRC03:56
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror)
*** alonma has joined #openstack-cinder03:57
*** [2]Thelo has quit IRC03:57
*** zhangjn_ has quit IRC03:58
*** xyang has quit IRC03:58
*** changbl has joined #openstack-cinder04:00
*** thambirajan has joined #openstack-cinder04:00
*** alonma has quit IRC04:02
*** links has quit IRC04:03
*** jerrygb has joined #openstack-cinder04:03
*** mtanino has quit IRC04:03
*** bardia has joined #openstack-cinder04:03
*** alonma has joined #openstack-cinder04:04
*** mattymo has quit IRC04:05
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror)
*** alonma has quit IRC04:08
*** mattymo has joined #openstack-cinder04:11
*** gouthamr_ has quit IRC04:12
*** dims has joined #openstack-cinder04:13
*** dims has quit IRC04:15
*** dims has joined #openstack-cinder04:18
*** bardia has quit IRC04:19
*** changbl has quit IRC04:21
openstackgerritxing-yang proposed openstack/python-cinderclient: CLI for backup snapshots
*** alonma has joined #openstack-cinder04:23
*** alonma has quit IRC04:27
*** klkumar has joined #openstack-cinder04:28
*** akshai has joined #openstack-cinder04:28
*** zhangjn has joined #openstack-cinder04:29
*** dims has quit IRC04:29
*** alonma has joined #openstack-cinder04:32
*** changbl has joined #openstack-cinder04:35
*** gouthamr has joined #openstack-cinder04:36
*** alonma has quit IRC04:37
*** links has joined #openstack-cinder04:38
*** gouthamr has quit IRC04:48
*** jerrygb has quit IRC04:49
*** jerrygb has joined #openstack-cinder04:51
*** jerrygb has quit IRC04:51
*** jerrygb has joined #openstack-cinder04:51
*** zhangjn has quit IRC04:52
*** boris-42_ has quit IRC05:03
*** p0rtal has quit IRC05:05
*** bardia has joined #openstack-cinder05:06
*** shausy has joined #openstack-cinder05:06
*** smoriya_ has joined #openstack-cinder05:09
*** smoriya has quit IRC05:10
*** klkumar has quit IRC05:11
*** sheel has joined #openstack-cinder05:14
*** klkumar has joined #openstack-cinder05:15
*** zhangjn has joined #openstack-cinder05:15
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror)
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO)
*** harlowja_at_home has joined #openstack-cinder05:19
*** dims has joined #openstack-cinder05:20
*** dilip has joined #openstack-cinder05:24
*** dilip has quit IRC05:24
*** dilip has joined #openstack-cinder05:25
*** itzdilip has quit IRC05:26
*** klkumar has quit IRC05:26
*** dilip has quit IRC05:26
*** dilip has joined #openstack-cinder05:26
*** dilip has quit IRC05:26
*** dilip has joined #openstack-cinder05:27
*** dilip has quit IRC05:27
*** dilip has joined #openstack-cinder05:27
*** zhangjn has quit IRC05:28
*** zhangjn has joined #openstack-cinder05:28
*** dilip has quit IRC05:30
*** dilip has joined #openstack-cinder05:31
*** klkumar has joined #openstack-cinder05:32
*** martyturner has joined #openstack-cinder05:36
*** martyturner has quit IRC05:41
*** vgridnev has joined #openstack-cinder05:42
*** shausy has quit IRC05:44
*** shausy has joined #openstack-cinder05:44
*** alonma has joined #openstack-cinder05:46
*** alonma has quit IRC05:50
openstackgerritMerged openstack/cinder: Update migrate_volume API to use versionedobjects
*** alonma has joined #openstack-cinder05:52
*** alonma has quit IRC05:57
*** salv-orl_ has quit IRC06:00
*** salv-orlando has joined #openstack-cinder06:00
*** alonma has joined #openstack-cinder06:02
*** alonma has quit IRC06:06
*** jerrygb has quit IRC06:17
*** jerrygb has joined #openstack-cinder06:17
*** salv-orl_ has joined #openstack-cinder06:17
*** alonma has joined #openstack-cinder06:17
*** salv-orlando has quit IRC06:20
*** dilip has quit IRC06:20
*** vinayp has joined #openstack-cinder06:21
*** dilip has joined #openstack-cinder06:21
*** dilip has quit IRC06:21
*** jerrygb has quit IRC06:21
*** itzdilip has joined #openstack-cinder06:22
*** alonma has quit IRC06:22
*** alonma has joined #openstack-cinder06:23
*** logan- has quit IRC06:24
*** sgotliv_ has joined #openstack-cinder06:26
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata
*** ebalduf has quit IRC06:28
*** alonma has quit IRC06:28
*** alonma has joined #openstack-cinder06:30
*** jaypipes has joined #openstack-cinder06:31
*** alonma has quit IRC06:34
*** ChubYann has quit IRC06:37
*** lpetrut has joined #openstack-cinder06:37
*** chlong has quit IRC06:40
openstackgerritRui Chen proposed openstack/python-cinderclient: Fix Resource.__eq__ mismatch semantics of object equal
*** lpetrut has quit IRC06:48
openstackgerritJohn Griffith proposed openstack/cinder: Add cluster Pairing setup to SolidFire driver
*** shausy has quit IRC06:50
*** shausy has joined #openstack-cinder06:50
*** nkrinner has joined #openstack-cinder06:56
openstackgerritJohn Griffith proposed openstack/cinder: Add replicate by volume-type to SolidFire driver
openstackgerritJohn Griffith proposed openstack/cinder: Add cluster Pairing setup to SolidFire driver
*** jerrygb has joined #openstack-cinder07:00
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
openstackgerritJohn Griffith proposed openstack/cinder: Replace use of mox with mock in test_solidfire
*** jerrygb has quit IRC07:06
*** harlowja_at_home has quit IRC07:06
*** bardia has quit IRC07:08
*** rcernin has joined #openstack-cinder07:20
*** boris-42_ has joined #openstack-cinder07:31
*** sgotliv_ has quit IRC07:34
*** zhangjn has quit IRC07:36
*** itzdilip has quit IRC07:37
*** alexschm has joined #openstack-cinder07:38
*** itzdilip has joined #openstack-cinder07:38
*** klkumar has quit IRC07:41
*** alonma has joined #openstack-cinder07:42
*** itzdilip has quit IRC07:42
*** klkumar has joined #openstack-cinder07:44
*** itzdilip has joined #openstack-cinder07:45
*** itzdilip has quit IRC07:45
*** itzdilip has joined #openstack-cinder07:46
*** nkrinner has quit IRC07:46
*** dilip has joined #openstack-cinder07:46
*** nkrinner has joined #openstack-cinder07:46
*** alonma has quit IRC07:46
*** itzdilip has quit IRC07:48
*** rhagarty_ has joined #openstack-cinder07:48
*** rhagarty has quit IRC07:49
*** e0ne has joined #openstack-cinder07:51
*** alonma has joined #openstack-cinder07:52
*** yhayashi has quit IRC07:52
*** dilip has quit IRC07:53
*** dilip has joined #openstack-cinder07:53
*** nkrinner has quit IRC07:54
*** nkrinner has joined #openstack-cinder07:55
*** alonma has quit IRC07:57
openstackgerritwanghao proposed openstack/cinder: Get the replica volume ref for DR
*** alonma has joined #openstack-cinder07:59
*** dilip has quit IRC07:59
openstackgerritwanghao proposed openstack/cinder: Get the replica volume ref for DR
*** zhangjn has joined #openstack-cinder08:00
*** alonma has quit IRC08:03
*** anshul has joined #openstack-cinder08:04
openstackgerritwanghao proposed openstack/cinder-specs: Get the replica volume ref for DR
*** alonma has joined #openstack-cinder08:05
*** alonma has quit IRC08:05
*** egonzalez has joined #openstack-cinder08:06
*** alonma has joined #openstack-cinder08:06
*** isaacb has joined #openstack-cinder08:06
*** dilip has joined #openstack-cinder08:13
*** liverpooler has joined #openstack-cinder08:15
*** belmoreira has joined #openstack-cinder08:15
*** klkumar has quit IRC08:17
*** vgridnev has quit IRC08:23
*** markus_z has joined #openstack-cinder08:23
*** smoriya_ has quit IRC08:31
*** salv-orl_ has quit IRC08:58
*** jordanP has joined #openstack-cinder08:58
*** jerrygb has joined #openstack-cinder09:02
*** openstackstatus has quit IRC09:05
*** openstack has joined #openstack-cinder09:07 [freenode-info] channel flooding and no channel staff around to help? Please check with freenode support:
*** jerrygb has quit IRC09:08
*** alonma has joined #openstack-cinder09:09
*** ociuhandu has quit IRC09:14
*** e0ne has quit IRC09:15
*** dilip has quit IRC09:16
*** dilip has joined #openstack-cinder09:17
*** sgotliv_ has joined #openstack-cinder09:18
*** chenying has joined #openstack-cinder09:19
*** skraynev has quit IRC09:20
*** chenying_ has quit IRC09:21
*** ndipanov has joined #openstack-cinder09:27
*** shausy has quit IRC09:28
*** skraynev has joined #openstack-cinder09:28
*** shausy has joined #openstack-cinder09:29
*** alonma has quit IRC09:31
*** vgridnev has joined #openstack-cinder09:32
*** openstackgerrit has quit IRC09:32
*** openstackgerrit has joined #openstack-cinder09:33
*** alonma has joined #openstack-cinder09:33
*** shausy has quit IRC09:42
*** ildikov has quit IRC09:42
*** shausy has joined #openstack-cinder09:42
*** zhangjn has quit IRC09:45
*** skraynev has quit IRC09:48
*** skraynev has joined #openstack-cinder09:49
*** dilip has quit IRC09:50
*** dilip has joined #openstack-cinder09:50
*** dilip has quit IRC09:50
*** itzdilip has joined #openstack-cinder09:51
*** zhangjn has joined #openstack-cinder09:57
*** salv-orlando has joined #openstack-cinder09:59
*** openstackgerrit has quit IRC10:02
*** gcb has quit IRC10:02
*** zhenguo has quit IRC10:02
*** openstackgerrit has joined #openstack-cinder10:03
*** salv-orlando has quit IRC10:03
*** ildikov has joined #openstack-cinder10:09
*** e0ne has joined #openstack-cinder10:10
*** jamielennox is now known as jamielennox|away10:13
*** sileht has quit IRC10:16
openstackgerritYuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver
*** sileht has joined #openstack-cinder10:19
*** sileht has quit IRC10:19
*** sileht has joined #openstack-cinder10:20
*** aix has joined #openstack-cinder10:20
*** rushiagr_away is now known as rushiagr10:21
*** dims has quit IRC10:25
*** alexschm has quit IRC10:26
*** alexschm has joined #openstack-cinder10:29
*** ianbrown has joined #openstack-cinder10:30
*** alonma has quit IRC10:31
*** alonma has joined #openstack-cinder10:31
*** alexschm_ has joined #openstack-cinder10:31
*** cbader has quit IRC10:32
*** cbader has joined #openstack-cinder10:33
*** alexschm has quit IRC10:34
*** alonma has quit IRC10:35
*** aix has quit IRC10:37
*** alonma has joined #openstack-cinder10:37
*** jistr has joined #openstack-cinder10:39
*** alonma has quit IRC10:41
*** eduardo___ has joined #openstack-cinder10:44
*** alonma has joined #openstack-cinder10:59
*** alonma has quit IRC11:04
*** jerrygb has joined #openstack-cinder11:04
*** kmartin has quit IRC11:05
*** kmartin has joined #openstack-cinder11:05
*** alonma has joined #openstack-cinder11:06
*** jerrygb has quit IRC11:09
*** salv-orlando has joined #openstack-cinder11:10
*** alonma has quit IRC11:10
*** alonma has joined #openstack-cinder11:10
*** lprice has joined #openstack-cinder11:13
*** lprice1 has quit IRC11:13
*** aix has joined #openstack-cinder11:14
*** dilip has joined #openstack-cinder11:14
openstackgerritYuriy Nesenenko proposed openstack/cinder: Small refactoring in
*** itzdilip has quit IRC11:15
*** adrianofr has joined #openstack-cinder11:16
*** dilip has quit IRC11:16
openstackgerritYuriy Nesenenko proposed openstack/cinder: Small refactoring in
*** dilip has joined #openstack-cinder11:17
*** zhangjn has quit IRC11:23
*** EinstCrazy has quit IRC11:24
*** dilip has quit IRC11:25
*** abhi has quit IRC11:26
*** links has quit IRC11:26
*** klkumar has joined #openstack-cinder11:28
*** angela-s has quit IRC11:29
*** dims has joined #openstack-cinder11:37
*** dilip has joined #openstack-cinder11:39
*** rushiagr is now known as rushiagr_away11:48
*** ndipanov has quit IRC11:48
*** ndipanov has joined #openstack-cinder11:49
*** kmartin has quit IRC11:49
*** rushiagr_away is now known as rushiagr11:49
*** zhangjn has joined #openstack-cinder11:49
*** EinstCrazy has joined #openstack-cinder11:51
*** dilip has quit IRC11:52
*** rushiagr is now known as rushiagr_away11:52
*** cdelatte has quit IRC12:05
*** takedakn has joined #openstack-cinder12:11
*** Lee1092 has quit IRC12:11
*** links has joined #openstack-cinder12:16
*** salv-orl_ has joined #openstack-cinder12:17
*** salv-orlando has quit IRC12:20
*** smoriya has joined #openstack-cinder12:24
*** houming has joined #openstack-cinder12:31
*** rushiagr_away is now known as rushiagr12:32
*** gcb has joined #openstack-cinder12:33
*** alejandrito has joined #openstack-cinder12:40
*** chlong has joined #openstack-cinder12:42
*** boris-42_ has quit IRC12:43
*** dilip has joined #openstack-cinder12:43
*** salv-orl_ has quit IRC12:44
openstackgerritKazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver
*** dilip has quit IRC12:46
*** cdelatte has joined #openstack-cinder12:48
*** takedakn1 has joined #openstack-cinder12:48
*** salv-orlando has joined #openstack-cinder12:49
*** takedakn has quit IRC12:49
*** takedakn1 is now known as takedakn12:49
*** jerrygb has joined #openstack-cinder12:53
*** nikeshm has joined #openstack-cinder12:55
nikeshmbp "" was registered and implemented in liberty, can we use same bp for mitaka for this patch
nikeshmits already got one +2, so thought better to arise this12:57
e0nenikeshm: it's partially implemented in Liberty, so it seems to be OK to re-use it instead of creating a new one12:58
nikeshme0ne : hmm, ok , thanks12:59
*** takedakn has quit IRC12:59
*** dilip has joined #openstack-cinder13:00
*** zhipeng has joined #openstack-cinder13:00
*** dilip has quit IRC13:01
*** julim has joined #openstack-cinder13:01
*** Lee1092 has joined #openstack-cinder13:06
*** ildikov has quit IRC13:07
*** asselin has joined #openstack-cinder13:19
*** ildikov has joined #openstack-cinder13:22
*** chenying has quit IRC13:29
*** chenying has joined #openstack-cinder13:30
*** edmondsw has joined #openstack-cinder13:36
*** martyturner has joined #openstack-cinder13:36
*** takedakn has joined #openstack-cinder13:37
*** gouthamr has joined #openstack-cinder13:40
*** akerr has joined #openstack-cinder13:40
*** zhonghua-lee has quit IRC13:40
*** zhonghua-lee has joined #openstack-cinder13:41
*** gouthamr_ has joined #openstack-cinder13:41
*** diogogmt has joined #openstack-cinder13:41
*** zhipeng has quit IRC13:42
*** dilip has joined #openstack-cinder13:42
*** eharney has joined #openstack-cinder13:44
*** gouthamr has quit IRC13:45
*** timcl has joined #openstack-cinder13:49
*** dilip has quit IRC13:52
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova
*** itzdilip has joined #openstack-cinder13:52
*** Thelo has joined #openstack-cinder13:55
*** [1]Thelo has quit IRC13:57
*** logan- has joined #openstack-cinder14:01
openstackgerritErik Johannes proposed openstack/cinder: Implement CHAP Authentication for E-Series Driver
*** sgotliv_ has quit IRC14:03
*** martyturner has quit IRC14:03
*** gcb has quit IRC14:06
*** martyturner has joined #openstack-cinder14:08
*** jerrygb_ has joined #openstack-cinder14:08
*** houming has quit IRC14:09
*** haomaiwang has joined #openstack-cinder14:10
*** jerrygb has quit IRC14:11
openstackgerritYuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver
*** houming has joined #openstack-cinder14:17
*** sgotliv_ has joined #openstack-cinder14:18
*** takedakn has quit IRC14:19
*** breitz has quit IRC14:19
*** breitz has joined #openstack-cinder14:19
*** merooney has joined #openstack-cinder14:21
*** diogogmt has quit IRC14:24
*** mriedem_away is now known as mriedem14:25
*** diogogmt has joined #openstack-cinder14:26
*** zhipeng has joined #openstack-cinder14:27
*** jgregor has joined #openstack-cinder14:27
*** shausy has quit IRC14:28
*** ildikov has quit IRC14:28
*** ildikov has joined #openstack-cinder14:30
*** diablo_rojo has joined #openstack-cinder14:31
*** mtanino has joined #openstack-cinder14:31
*** ildikov has quit IRC14:35
*** timwilliams has joined #openstack-cinder14:35
*** timwilliams has quit IRC14:37
*** bardia has joined #openstack-cinder14:38
*** diogogmt has quit IRC14:38
*** dave-mccowan has joined #openstack-cinder14:38
*** akshai_ has joined #openstack-cinder14:40
*** zhipeng has quit IRC14:42
*** akshai has quit IRC14:44
*** mc_nair has joined #openstack-cinder14:45
*** lpabon has joined #openstack-cinder14:45
openstackgerritYuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver
openstackgerritSean McGinnis proposed openstack/python-cinderclient: DO NOT MERGE: This is only a test
anteayasmcginnis: I see that14:47
smcginnisanteaya: OK, nevermind. Appears to be working now. Sorry for the distraction. :)14:47
anteayano worries14:48
anteayaglad I had to do nothing to make you happy14:48
anteayagood start to the day14:48
smcginnisanteaya: It definitely wasn't the other day. I'll keep an eye on it. :)14:48
smcginnisanteaya: Yeah, good easy one for the morning. :D14:48
anteayaokay if you see it disappear to let me know14:48
anteayanow keep in mind freenode has had a bad time lately14:48
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver
anteayaif openstackgerrit isn't in channel it can't report anything14:48
smcginnisanteaya: Yeah, it's quite possible it was around the time of the ddos.14:49
anteayathat might explain the silence then14:49
smcginnisanteaya: It was odd though that we were getting notifications for cinder and os-brick, just not client.14:49
smcginnisOh well.14:49
anteayawhile we get grumpy when it happens fungi has to play server roulette to get our bots back in action14:50
smcginnisanteaya: Not fun, I'm sure!14:50
*** ntpttr has joined #openstack-cinder14:51
anteayanot so much14:52
anteayaI don't do anything14:52
anteayabut fungi does14:52
*** merooney has quit IRC14:59
*** ndipanov has quit IRC15:02
*** eharney has quit IRC15:02
e0nejgriffith: hi. FYI, I've updated spec to implement cinderclient extension instead of python-brickclient15:02
*** ndipanov has joined #openstack-cinder15:02
*** dustins|gone is now known as dustins15:03
*** flaper87 has joined #openstack-cinder15:03
*** eharney has joined #openstack-cinder15:03
*** flaper87 has left #openstack-cinder15:05
*** merooney has joined #openstack-cinder15:05
*** xyang1 has joined #openstack-cinder15:06
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver
*** wanghao has joined #openstack-cinder15:09
jgriffithe0ne: cool, I'll have a look.  Tahnks!15:09
*** cdelatte has quit IRC15:09
*** cdelatte has joined #openstack-cinder15:10
e0nejgriffith: great, thanks! there is also a draft implementation for it15:10
*** mdenny has joined #openstack-cinder15:11
*** gouthamr_ has quit IRC15:11
*** timcl1 has joined #openstack-cinder15:11
smcginniswanghao: Hey15:11
wanghaosmcginnis: hi, I have updated this spec: get replica volume ref15:12
smcginniswanghao: Already open in a tab. ;)15:12
*** delattec has joined #openstack-cinder15:12
smcginniswanghao: I'm hoping to review it again today.15:12
smcginnisJust have a few things to get through first.15:12
jgriffithsmcginnis: what... are you psychic!15:12
wanghaosmcginnis: yeah, great, thanks15:12
smcginnisjgriffith: ;)15:13
wanghaosmcginnis: jgriffith I was shocked a little :)15:14
*** cdelatte has quit IRC15:15
*** timcl has quit IRC15:15
jgriffithwanghao: :)15:15
wanghaothingee: hi~, are you here?15:16
smcginnisStill pretty early for him. He's US west coast.15:17
smcginnisHe's pretty good about seeing pings while he's gone though.15:17
wanghaosmcginnis:okay, got it.15:18
*** dilip has joined #openstack-cinder15:19
*** itzdilip has quit IRC15:19
*** alonma has quit IRC15:19
wanghaosmcginnis:Just wanna to talked about his suggestion in this spec. I have a little confusion about it.15:20
*** alonma has joined #openstack-cinder15:20
*** ildikov has joined #openstack-cinder15:20
*** ebalduf has joined #openstack-cinder15:22
*** noqa_v_qoovnie has quit IRC15:22
*** alonma has quit IRC15:24
*** diogogmt has joined #openstack-cinder15:25
*** alonma has joined #openstack-cinder15:26
*** alonma has quit IRC15:26
*** dilip has quit IRC15:27
*** alonma has joined #openstack-cinder15:27
jgriffithwanghao: could I ask a few questions about this?15:28
openstackgerritNate Potter proposed openstack/cinder: Move retype quota checks to API
*** ociuhandu has joined #openstack-cinder15:30
wanghaojgriffith: sure15:31
openstackgerritRyan McNair proposed openstack/cinder: Fix non-migration swap with error
*** houming has quit IRC15:31
jgriffithwanghao: I posted some comments in the review (well.. quesitons more than comments)15:32
jgriffithwanghao: can you hep me understand the use case for this a bit better?  More specifically what problem you're trying to solve.15:33
jgriffithwanghao: I'm afraid I didn't quite figure it out in the spec, and the code ruins the assumption that I did have15:33
*** anshul has quit IRC15:33
jgriffithwanghao: because you're adding a volume_ref field, but I'm confused because it's a string and I don't know what it is supposed to "be" or what you intend to use it for15:34
jgriffithwanghao: my apologies for not "getting it" :)15:34
*** dilip has joined #openstack-cinder15:34
*** gouthamr has joined #openstack-cinder15:35
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova
*** timcl1 has quit IRC15:37
wanghaojgriffith: okay, simplely to say, if there are two independent cloud sites, we create replication volume in site 1, and drivers build replica volume in site 2's storage.  well, when disaster happen, site 1 is down, and then we want to find our data back and continue to use it in site 2.15:39
*** belmoreira has quit IRC15:39
smcginniswanghao: So to validate my high level understanding so far:15:40
openstackgerritJohn Griffith proposed openstack/cinder: Add replicate by volume-type to SolidFire driver
wanghaojgriffith: but how?  So we want to manage the replica volume in site 2.15:40
smcginniswanghao: This is for DR. If the primary goes down, we need to be able to feed the replication target information into the secondary side's Cinder service.15:40
openstackgerritNate Potter proposed openstack/cinder: Move snapshot and volume quota checks to API
smcginniswanghao: So something like a manage existing, but passing in details so the driver knows it's managing what was a replication target?15:40
openstackgerritJohn Griffith proposed openstack/cinder: Add cluster Pairing setup to SolidFire driver
jgriffithwanghao: ok, so I kinda get what you're trying to accomplish, but I'm not following how this will help you accomplish it15:43
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova
jgriffithwanghao: I'm also not sure how I feel about explicitly putting entries in the db to point to references in "another" cloud15:43
jgriffithwanghao: that can be a VERY slippery slope15:43
*** thangp has joined #openstack-cinder15:43
smcginnisYeah, not sure we need to muck with the DB here.15:44
jgriffithwanghao: my point is I believe this can be solved other ways maybe, using mechanisms we already have in place15:44
*** baumann has joined #openstack-cinder15:44
*** rs007 has joined #openstack-cinder15:44
jgriffithwanghao: say like maybe admin-metadata... or even an explicit type15:44
*** rs007_ has joined #openstack-cinder15:44
smcginnisProbably just a get_repl_metadata() type call on the source side, then a manage_existing(repl_metadata) on the dest.15:44
*** rs007 has quit IRC15:44
* smcginnis is brainstorming without having reread any recent details of what the spec actually calls for15:45
jgriffithwanghao: because in the case of a cloud2cloud DR setup and admin can/should have some special things in place IMHO.  One of those things could be a "replicated-volume" type15:45
e0nejgriffith, smcginnis: thanks for the review!15:45
jgriffithwanghao: that in combination with some metadata perhaps would accomplish the same thing here no?15:45
wanghaojgriffith:smcginnis  em... at first I want to put this into replication_driver_data15:46
openstackgerritNate Potter proposed openstack/cinder: Move consistency group quota checks to API
jgriffithwanghao: so let's back up a second... can you provide me an example of "what" that column would be?15:50
*** alexschm_ has quit IRC15:51
wanghaojgriffith: In my mind, it will just be a string column, that drivers put the replica volume ref into it when they build the replication in source side.15:52
*** zhangjn has quit IRC15:52
wanghaojgriffith: Like replication_driver_data15:53
jgriffithwanghao: what do you mean by "replica_volume_ref" though?15:53
jgriffithwanghao: what would that actually "be"... a provider id?  a cinder-uuid?15:54
*** liverpooler has quit IRC15:55
wanghaojgriffith: emm, more like provider id, you can use it to manage this volume in another openstack system.15:55
*** sheel has quit IRC15:55
wanghaojgriffith: Like if we want to manage a LVM volume, we need to know it's lv name.15:56
jgriffithwanghao: sure, so we could probably use existing rep data for that don't you think?15:56
jgriffithwanghao: or even tack something on to the provider_id field15:57
jgriffithwanghao: the name "volume_ref" is kinda misleading by the way15:57
jgriffithwanghao: so for example provider_id: rep_target_id: "foo"15:57
*** haomaiwang has quit IRC15:57
jgriffithor something like that15:57
*** zhipeng has joined #openstack-cinder15:58
jgriffithor back to using the replication_driver_data column15:58
smcginnisMeeting time15:58
jgriffithit seems to me like that would fit and kinda makes more sense15:58
*** haomaiwang has joined #openstack-cinder15:58
*** jseiler has joined #openstack-cinder15:58
jgriffithok... meeting!!!  Yay!!!15:58
wanghaojgriffith: yes, exactly15:58
*** haomaiwang has quit IRC15:59
*** erlon has joined #openstack-cinder15:59
*** haomaiwa_ has joined #openstack-cinder15:59
*** haomaiwa_ has quit IRC16:00
*** rhedlind has joined #openstack-cinder16:00
*** haomaiwang has joined #openstack-cinder16:00
*** pots has joined #openstack-cinder16:00
wanghaojgriffith: but I feel it's a little hard to handle it in replication_driver_data, we need to handle the string and get the 'provider_id:rep_target_id:"foo" ' from a long string.16:00
*** haomaiwang has quit IRC16:01
*** haomaiwa_ has joined #openstack-cinder16:01
*** pck_ is now known as pck16:02
yuriy_n17join  #openstack-meeting16:02
*** yuriy_n17 has quit IRC16:03
*** zz_john5223 is now known as john522316:03
*** kmartin has joined #openstack-cinder16:07
*** nkrinner has quit IRC16:08
*** jerrygb_ has quit IRC16:18
*** kfarr has joined #openstack-cinder16:18
*** diogogmt has quit IRC16:22
openstackgerritYuriy Nesenenko proposed openstack/cinder: Small refactoring in
*** vgridnev has quit IRC16:23
*** alonma has quit IRC16:23
openstackgerritYuriy Nesenenko proposed openstack/cinder: Add synchronization in Block Device driver
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver
*** harlowja_at_home has joined #openstack-cinder16:27
*** hemnafk is now known as hemna16:27
openstackgerritMikhail Khodos proposed openstack/cinder: --amend
*** martyturner has quit IRC16:30
*** alonma has joined #openstack-cinder16:30
*** crose has joined #openstack-cinder16:31
*** martyturner has joined #openstack-cinder16:33
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova
*** crose has quit IRC16:33
*** alonma has quit IRC16:35
*** alonma has joined #openstack-cinder16:36
*** haomaiwa_ has quit IRC16:40
*** haomaiwang has joined #openstack-cinder16:40
*** alonma has quit IRC16:40
*** alonma has joined #openstack-cinder16:42
*** ctina_ has joined #openstack-cinder16:43
*** p0rtal has joined #openstack-cinder16:44
*** lpabon has quit IRC16:46
*** alonma has quit IRC16:47
*** markstur has quit IRC16:48
*** alonma has joined #openstack-cinder16:49
*** markstur has joined #openstack-cinder16:50
*** earlephilhower has joined #openstack-cinder16:50
*** klkumar has quit IRC16:51
*** alonma has quit IRC16:53
*** rushiagr is now known as rushiagr_away16:53
*** rushiagr_away is now known as rushiagr16:54
winston-de0ne: so you mean you didn't add that to cinderclient?16:59
openstackgerritNate Potter proposed openstack/cinder: Move retype quota checks to API
openstackgerritNate Potter proposed openstack/cinder: Move retype quota checks to API
e0newinston-d: I've got a PoC implementation with cinderclient and get-conector API17:00
e0newinston-d: I'm going to implement attach/detach too. it was already done for brickclient PoC17:00
*** martyturner has quit IRC17:00
*** haomaiwang has quit IRC17:01
winston-dso brickclient is a new project?17:01
*** links has quit IRC17:01
*** haomaiwang has joined #openstack-cinder17:01
e0newinston-d: we decided to not introduce new project and move all to cinderclient17:02
e0newinston-d: so I need to move code from my brickclient repo to cinderclient17:02
DuncanTsmcginnis: Need to understand the implications - if it can be used to access other tenant data or something then it is a security bug17:02
winston-dok, sounds good17:02
wanghaosmcginnis: So you and John didn't agree to add a new column to DB, right?17:02
smcginniswanghao: No, I think we were in agreement, but jgriffith may have more input on that.17:03
*** merooney has quit IRC17:03
smcginnisDuncanT: Can you explain that a little more?17:03
DuncanTe0ne: How do you intend to avoid confusion by having anything that says 'attach' in cinder client?17:03
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver
e0neDuncanT: I would like to have something like local-attach17:04
*** jgregor has quit IRC17:04
*** jgregor has joined #openstack-cinder17:05
e0neDuncanT: TBH, I've implemented it only for CLI, I'm not sure yet how to provide python API as a cinderclient extension17:05
DuncanTe0ne: I hate it. People *will* get confused17:05
jgriffithDuncanT: e0ne the local-attach seemed reasonable to me :(17:05
e0neDuncanT: it should be something like cinderclient.brick.local_attach17:05
scottdae0ne: Will there also be local-terminate_connection or will that be wrapped in local-attach?17:05
DuncanTe0ne: Somebody will file a bug trying to use it inside a guest or something17:05
e0nescottda: yes17:05
scottdaok, cool. I put a comment to that effect in the spec.17:06
*** alonma has joined #openstack-cinder17:06
e0nescottda: I've added it to the spec17:06
*** yuriy_n17_ has joined #openstack-cinder17:06
*** martyturner has joined #openstack-cinder17:06
*** yuriy_n17_ has quit IRC17:07
e0neDuncanT: what do you mean? attach inside a guest is one of expected use cases17:07
DuncanTsmcginess: ldev is in the volume_metadata - looks like it might be the provider_id - if changing it in the metadata allows you to change what backend volume your cidner vol points to, then that's a security bug17:07
DuncanTe0ne: For normal clouds, the guest can't access the cidner storage network17:07
winston-de0ne: well, yes and no. For ironic, yes, but not for VM (especially VM with overlay network)17:07
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Add validation for volume_type at copy_volume_to_image()
jgriffithDuncanT: e0ne I'm still unclear why you wouldn't explicity disable it by default17:07
e0neDuncanT: good point, agree17:07
*** diogogmt has joined #openstack-cinder17:08
e0nejgriffith: it will17:08
jgriffithbut ok17:08
e0nejgriffith: I won't add os-brick as a mandatory dependency17:08
jgriffithso disable by default, have a help that says what it's for and be done I thougt17:08
*** thangp has quit IRC17:08
e0nejgriffith: so, by default, it won't work17:08
jgriffithe0ne: well, I meant a step further like something in policy that just completely disables it unless set explicitly17:09
jgriffithbut it sounds like there's concerns with the naming itself etc....17:09
e0nejgriffith: I remember that point in our conversation17:09
wanghaosmcginnis:jgriffith  It's more easier to handle the string by adding a new column IMHO, but I'm fine if we want to use some existing column to store this replica volume provider id.17:09
smcginnisDuncanT: Ah, yeah. Definitely badness.17:09
smcginnisI would consider that a security bug.17:09
jgriffithseems like there's a group that wants a "new" client and a group (like me) that wants it in existing client17:09
hemnaildikov, ping17:10
e0nejgriffith: need to do move investigation what is the best way to implement it17:10
jgriffithif we can't agree then I guess the "new client" group wins because it's silly to not do this17:10
e0nejgriffith: and than't why my spec is not merged yet:(17:10
jgriffithwanghao: so I would prefer using an existing column if possible17:10
*** alonma has quit IRC17:10
jgriffithwanghao: if nothing else, maybe we could try that approach... if we decide we need to later we can revisit a new column?17:11
winston-de0ne: do you have spec review link handy?17:11
*** jungleboyj has quit IRC17:11
DuncanTjgriffith: I can't disable something in a client17:11
jgriffithDuncanT: I think you can actually17:11
*** rcernin has quit IRC17:11
e0neDuncanT: we can to not install os-brick17:11
DuncanTjgriffith: apt-get isntall python-cinderclient; cinder help; see something about attach. Try it. Get error. File bug.17:11
winston-de0ne: thx17:12
hemnae0ne, so the client patch doesn't include a requirements.txt update to add os-brick17:12
jgriffithDuncanT: well, I'm saying I think I can hide the command altogether unless it's "turned on"17:12
e0neDuncanT: it won't be an error. There will be a user-friendly message how to use it and what lilitations are17:12
hemnae0ne, is that intentional ?17:12
wanghaojgriffith: emm, it's fine to me. I will try it.17:12
jgriffithDuncanT: I'll see if I can code something up.  Maybe I'm wrong, but I think I've done that before17:13
DuncanTjgriffith: Ok17:13
e0nehemna: yes, because it's an extension17:13
jgriffithDuncanT: if I'm mistaken then fair enough17:13
*** diogogmt_ has joined #openstack-cinder17:13
hemnae0ne, so how does a user of the client know they have to install it ?17:13
*** diogogmt has quit IRC17:13
*** diogogmt_ is now known as diogogmt17:13
hemnarun it, it fails17:13
DuncanTjgriffith: Sure, I might be missing something too17:13
e0nehemna: cinder --help?17:13
jgriffithhemna: ok... this is kinda cra-cra17:14
hemnaI'm just asking17:14
hemnatrying to understand the use case17:14
e0nehemna: if we want to add it as a mandatory requirement - we can discuss it later, it's not a big deal for us and package maintainers17:14
wanghaoWell, it's too later in my TZ, so bye everyone~17:14
jgriffithhemna: sorry... what I mean is....  bolting on os-brick as a req for cinderclient for one specific set of use cases17:14
winston-dwanghao: come on, it's only 1:14 am17:15
hemnae0ne, cinder --help poops out a boatload of stuff, I'm sure it would easily be missed17:15
e0nehemna: it requires 'sudo permissions', so I think it's ok that the feature is disabled be default17:15
jgriffithhemna: let me see if I can hide things first and come back to this discussion17:15
wanghaowinston-d: haha,17:15
hemnajgriffith, I'm not advocating adding it to requirements.txt17:15
winston-dwanghao: :) gdnite17:15
hemnaI'm just trying to figure out what an end user's experience will be like17:15
*** vgridnev has joined #openstack-cinder17:15
jgriffithhemna: ahh... ok.  Wasn't sure17:15
hemnaand see if it makes sense and what we can do to make it better17:15
hemnaif at all17:15
hemnacinder --help is almost useless IMHO17:16
jgriffithhemna: cool, yeah that's awesome17:16
wanghaowinston-d: I need to get up early in 6:00 AM....17:16
e0nehemna: - line #1917:16
wanghaowinston-d: Good night too.17:16
hemnae0ne, so we do a try import somewhere else and dump out that error message ?17:17
jgriffithBTW... we have thing slike backup-create in their now :(17:17
e0nehemna: yes17:17
e0nehemna: - line #2917:17
jgriffithwhich has the same problem people are concerned about WRT confusion if things aren't enabled17:17
hemnaah yah I just saw that17:17
hemnaok I'll pull the patch and see what it says17:17
hemnamaybe include the actual command to install it ?17:18
*** leeantho has joined #openstack-cinder17:18
e0nehemna: it's WIP code now. I just try to implement an extension17:18
hemnano worries17:18
e0nehemna: create better UX is on my plan17:18
hemnaI'm not trying to criticize, I just want to understand the work flow and use case for end users.17:18
*** jistr has quit IRC17:19
e0nehemna: got it, any feedback is welcome17:19
hemnae0ne, thanks for taking this on :)17:19
hemnamy plate is way over full right now17:19
*** ianbrown has quit IRC17:20
*** Apoorva has joined #openstack-cinder17:21
*** fthiagogv has joined #openstack-cinder17:21
*** electrocucaracha has joined #openstack-cinder17:21
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta driver
*** ianbrown has joined #openstack-cinder17:23
DuncanTsmcginnis: Looks like it isn't a security bug the ldev is shoved in the metadata but never used from there AFAICT - they put it in the provider_location as well, and use it from there17:23
e0nehemna: thanks for help.17:23
*** fthiagogv has quit IRC17:23
hemnaok cool17:23
*** fthiagogv has joined #openstack-cinder17:23
hemnawe can craft a better message later before the patch lands, but that's a good start thanks.17:24
electrocucarachahi there, We were playing around with horizon and we noticed that displaying bootable volume options is not working for non-admin users, from horizon source code perspective it's using same source code for all users. I'm wondering if this is a bug of Cinder server17:24
*** dims has quit IRC17:24
e0nehemna, DuncanT: so, are you OK to make it inside cinderclient?17:25
*** smoriya has quit IRC17:25
*** jaypipes has quit IRC17:25
hemnae0ne, maybe explain what the command line call looks like for issuing the attach17:25
hemnaand detach17:26
DuncanTe0ne: I don't understand what is being proposed at the moment.17:26
e0nehemna: - line #6417:26
e0neDuncanT: please, take a look on spec ^^17:26
hemnae0ne, cool thanks17:27
DuncanTe0ne: No, I hate it. I am entirely certain that somebody will try to run that from inside a guest and complain when it doesn't work17:27
DuncanTe0ne: I think it adds unnecessary confusion for a niche usecase17:28
*** esker has joined #openstack-cinder17:28
*** isaacb has quit IRC17:28
DuncanTe0ne: I think the cinder dev team underestimate how easily confused and damn hard work normal tenants can be17:28
*** jordanP has quit IRC17:29
DuncanTe0ne: We keep talking about needing smarter admins, who in real life often aren't there, tenants really do sit there poking likely looking buttons with no understanding of what they're doing. I've got a bug tracking system full of tickets to that effect17:29
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient: WIP. Add Python and CLI interfaces for os-brick as an extention
*** salv-orlando has quit IRC17:30
*** thangp has joined #openstack-cinder17:30
winston-dDuncanT: as an operator, I share your pain.17:30
e0neDuncanT: we can make it hidden by default17:30
DuncanTe0ne: IMO bare metal attach has no place in the 'cinder' command line17:31
hemnaDuncanT, the same can be said for a new client.17:31
e0neDuncanT: I understand you, I'm asked about cinderclient.attach method almost once per month17:31
e0nehemna: +117:31
hemnathey'll pull down said brickclient and try running attach from inside the vm17:31
hemnaand it'll fail17:31
hemnaif they went through the trouble of pulling down cinderclient inside their guest, they'd do the same thing for brickclient17:32
DuncanThemna: At least there you can say 'don't use brickclient, use cinder/nova, brick is for something different'17:32
ildikovhemna: hi, sorry I was on a call17:32
hemnausers are stupid, they'll do dumb things17:32
hemnathey shouldn't pull down cinderclient inside the guest either17:32
hemnait's really the same thing IMHO17:32
DuncanThemna: 'cinder-client' looks like the right thing to be using to do stuff with cinder volumes.... 'brick' doesn't17:32
hemnaDuncanT, which is the argument for putting the local-attach in cinderclient :P17:33
DuncanThemna: I pull down cinder-client in guests all the time, e.g. guests crontab their own backups17:33
hemnaildikov, quick question.   I want to test out the multiattach for nova side, so I can try and reproduce the bug
openstackLaunchpad bug 1471278 in Cinder "target removal on detaching volume with multi-attach flag" [Undecided,New] - Assigned to Walt Boring (walter-boring)17:33
DuncanThemna: I'm happy to make life harder for bare metal users to make it easier for 'normal' users17:33
hemnaildikov, which nova patch should I pull to get all of it ?17:33
hemnaDuncanT, I'm just saying it's the same thing really.17:34
hemnait'll fail just the same either way inside the guest.17:34
DuncanThemna: It is far easier to tell somebody 'brick-client is the wrong thing' than 'cinder local-attach is wrong, but cinder <everything else>' is right17:35
*** egonzalez has quit IRC17:35
ildikovhemna: theoretically this one:
hemnaDuncanT,  I do see your point and understand.  it's sucky either way really.17:35
hemnaildikov, ok I'll try that today and see how it goes :)17:36
hemnaI'd like to get the lvm driver fixed, since it is our reference driver.17:36
ildikovhemna: although I have to admit that the tempest tests are still not green for it and I don't know what the exact issue is at the moment... :S17:36
*** bardia has quit IRC17:36
DuncanThemna: yup17:36
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova
hemnaildikov, who needs tempest anyway :)17:36
ildikovhemna: I took over the work and did a quick rebase, but then many things went wrong17:37
ildikovhemna: lol :)17:37
*** p0rtal has quit IRC17:37
hemnaildikov, thanks for the info.  I'll try and get it working today.   the patch looks like it won't induce merge conflicts, so hopefully the real code works :)17:37
ildikovhemna: I tried to set up tempest locally, but I think I will go for Devstack instead to see for myself what goes wrong and where17:38
hemnaok off to a beating....bbiab.17:38
winston-dif we have brick-client, what use cases would it serve? local-attach/detach for one.17:38
*** hemna is now known as hemnafk17:38
ildikovhemnafk: thanks for playing wiht it, let me know if you found any issues17:38
*** sgotliv_ has quit IRC17:39
e0newinston-d: yes, only local attach/detach17:39
*** gouthamr has quit IRC17:40
winston-dnothing else?17:40
e0newinston-d: only these two and get-connector17:41
e0newinston-d: nothing else in the nearlies future17:41
e0neDuncanT: each option (cinderclient or brickclient) will not work for everybody. we'll have confused users in any case17:42
*** zhipeng has quit IRC17:42
*** timcl has joined #openstack-cinder17:43
*** changbl has quit IRC17:43
*** merooney has joined #openstack-cinder17:44
*** gouthamr has joined #openstack-cinder17:45
*** zhipeng has joined #openstack-cinder17:47
*** ndipanov has quit IRC17:47
*** salv-orlando has joined #openstack-cinder17:49
DuncanTe0ne: I think I'm going to have a much easier job telling cloud users not to touch brick-client than trying to tell them that only some bits of cinder-client apply to them17:50
*** sgotliv_ has joined #openstack-cinder17:51
*** salv-orlando has quit IRC17:52
*** martyturner has quit IRC17:55
*** lpetrut has quit IRC17:56
e0neDuncanT: the 3rd option: make it as a extension to cinderclient and install like 'pip install python-cinderclient && pip install python-cinderclient-brick'17:56
e0neDuncanT: we just need to extend our cinderclient extensions API a bit for such case17:56
e0neDuncanT: and your users won't see any local-attachment stuff for cinderclient w/o installing additional packages17:57
e0neDuncanT: is it good for you?17:57
DuncanTe0ne: That sounds reasonable, yes17:58
* e0ne changing the spec17:59
*** merooney has quit IRC17:59
*** hemnafk is now known as hemna17:59
*** markus_z has quit IRC17:59
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova
e0neDuncanT: please, review ^^18:00
*** haomaiwang has quit IRC18:01
*** haomaiwang has joined #openstack-cinder18:01
e0nehemna: ^^18:01
*** changbl has joined #openstack-cinder18:02
hemnaso python-cinderclient-brick ?18:03
hemnawhich is a post install extension to cinderclient ?18:03
*** merooney has joined #openstack-cinder18:03
*** jkremer has joined #openstack-cinder18:04
*** jkremer has quit IRC18:04
*** rushiagr is now known as rushiagr_away18:05
*** jkremer has joined #openstack-cinder18:05
hemnaildikov, ping18:06
jgriffithDuncanT: e0ne here's a quick and dirty example:
openstackgerritMerged openstack/cinder: Python 3: fix a lot of tests
*** alonma has joined #openstack-cinder18:06
jgriffithDuncanT: e0ne so that will get us "part" way there... that hides the commands unless --local is set18:06
smcginnishemna: python-cinderclient-onlyuse-ifyouknow-whatyourdoing18:06
jgriffithDuncanT: e0ne Then on the Cinder API side you could enforce a policy to enable/disable it18:07
*** rushiagr_away is now known as rushiagr18:07
jgriffithDuncanT: e0ne in which case if a user enabled it in the client, calls it and they're not supposed to it will respond with "policy doesn't allow"18:07
DuncanTjgriffith: That's no worse than the two-part install I guess18:07
jgriffithDuncanT: e0ne I think there's much more that can be done to improve it18:08
hemnaI like the idea of a pip install cinderclient-brick extension18:08
jgriffithhemna: that works too.. this is just "how" cinderclient would pull it in18:08
hemnayah cool18:08
*** fthiagogv has quit IRC18:08
*** eduardo___ has quit IRC18:08
*** fthiagogv has joined #openstack-cinder18:09
*** sgotliv_ has quit IRC18:09
* DuncanT is ok with that idea, eitehr way18:09
*** timcl has left #openstack-cinder18:09
ildikovhemna: pong18:09
hemnaildikov, so I'm getting a failure on a 2nd attach call18:10
ildikovthe first is ok?18:10
hemnan-api is raising that18:10
hemnafirst attach is fine18:10
*** alonma has quit IRC18:10
hemna2nd attach fails due to 'in-use' set.   that shouldn't happen if the volume is multiattachable18:11
*** rs007_ has quit IRC18:11
ildikovhmm, that's not good18:12
ildikovwhen we stopped the work on the patches during summer the attach worked :(18:12
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
*** gouthamr has quit IRC18:12
ildikovor my colleague lied...18:13
hemnaman pastebin is less than ideal for reading dumps18:13
hemnaanyway the volume dict has the "multiattach": true in it.18:13
e0nejgriffith, DuncanT, hemna, smcginnis: team, please, lets make a decision and go forward with it. we are trying to get it landed since Vancouver summit :(18:13
smcginnise0ne: +118:14
jgriffithe0ne: my vote is just implement it in a new module like I have in the example18:14
jgriffithe0ne: but I'll defer to the HP folks18:14
jgriffithe0ne: if we agree I can clean up that code and submit at least the foundation for it18:15
e0neI'm ready to implement any solution which community will accept18:15
hemnahonestly, I don't really care either way, just as long as users have the ability to do it.18:15
e0nehemna: +118:15
*** fthiagogv has quit IRC18:15
*** fthiagogv has joined #openstack-cinder18:15
jgriffithsmcginnis: hemna DuncanT e0ne ok... so does that mean "go forward with the concept proposed"?18:16
*** lcurtis has joined #openstack-cinder18:16
*** martyturner has joined #openstack-cinder18:16
smcginnisjgriffith: If DuncanT doesn't totally hate it anymore.18:16
* e0ne can't +1 on own spec but18:16
smcginnise0ne: Sure you can. :)18:16
jgriffithe0ne: :)18:16
smcginnisI see folks doing it all the time. Hah!18:16
e0neset cheatmode=On18:17
jgriffithe0ne: I'm talking voting on actually using the cinderclient hack I just proposed in conjunction with policy18:17
*** alonma has joined #openstack-cinder18:17
smcginnisDuncanT: That works for you?18:17
e0ne+1 on any. both options looks good18:17
smcginnise0ne: Yeah, the problem is I can see valid arguments on both sides.18:17
DuncanTAs long as if I do 'pip install python-cinderclient; cinder help' and don't see anything about attach without doing some other step, I'm passibly happy18:18
smcginnisI don't want that to stall things though.18:18
jgriffithwell now I'm confused but ok... whatevers; I'm going to move along now18:18
*** markvoelker has quit IRC18:18
*** fthiagogv has quit IRC18:18
DuncanTSo I'm +1 on either proposal18:18
smcginnisDuncanT: Love it - passibly happy. :)18:18
hemnaildikov, crap.  I just touched the nova/volume/ to add a debug log output and tried to bounce n-api, n-cpu and they both puked on startup18:18
*** fthiagogv has joined #openstack-cinder18:18
*** rushiagr is now known as rushiagr_away18:18
DuncanTjgriffith's or just the split install18:18
hemnaildikov, IncompatibleObjectVersion Version 1.18 of BlockDeviceMappingList is not supported18:18
jgriffithDuncanT: ahhh18:18
ildikovhemna: bah :(18:18
*** bardia has joined #openstack-cinder18:18
jgriffithok, well there's some options whatever works best for e0ne and everyone else18:19
hemnaildikov, *sigh*  trying to debug this to figure it out.  nova is always uber HARD18:19
ildikovhemna: the version is bumped in the second patch in the chain18:19
hemnaildikov, guess I'll restack and see what happens.18:19
ildikovhemna: it's the obj: ... one IIRC18:20
e0ne'pip install cinderclient-brick' is a good compromise to make everyone happy, IMO18:20
hemnaI'm an idiot.18:20
hemnaheh.  I stacked, then pulled the patch.18:20
hemnaok this should work, restacking.18:20
e0nejgriffith: I like your proposal too but we need to confirm that DuncanT is OK for it too18:20
*** abhi has joined #openstack-cinder18:20
DuncanTe0ne: YEs18:21
e0neDuncanT: to be clear: separate install (pip install cinderclient-brick) or John's proposal (cinder --local)18:21
*** markvoelker has joined #openstack-cinder18:21
*** alonma has quit IRC18:22
DuncanTe0ne: I'm fine with either. I guess John's proposal is easier18:22
* e0ne modifies spec again18:22
jgriffithe0ne: and I don't care much either... just so long as the pip install is just installing extensions and not a full-blown client18:22
jgriffithe0ne: they each have pros/cons neither a clear winner from what I can see18:23
hemnajgriffith, +118:23
e0nejgriffith: +118:23
DuncanTI'm heading off, but I'll happily +1 either version in a spec18:23
jgriffithI wouldn't be surprised if folks like Dean Troyer or guitarzan might even already have cases where they do this sort of thing.. but I dunno18:24
hemnajgriffith, might be worth asking guitarzan about this as well18:24
guitarzanjgriffith: a cinderclient extension? some of the guys use one for this (detach) purpose right now18:26
* guitarzan didn't read all the backlog18:26
e0ne is updated with '-L/--local' option18:26
guitarzanfor the record, we use a separate package for the extension18:27
guitarzan(I guess that's obvious)18:27
jgriffithguitarzan: yeah, we were talking between a seperate installable package for the extensions or just backing them directly in18:27
jgriffithguitarzan: sounds to me like the external package that gets imported is the "norm"18:28
jgriffithguitarzan: e0ne hemna You could use a combination of the approaches as well...18:28
jgriffiththat way nobody would need "two" clients on their system to keep from getting confused :)18:28
e0nejgriffith: I don't see any banefits on getting both options together18:29
e0ne'pip install cinderclient-some-extension' should enable it by default.18:29
e0neor we'll have a lot of questions like: I installed an extension, but it dosn't work18:30
jgriffithe0ne: KO18:30
jgriffithdo the pip install extensions thing then18:30
hemnae0ne, +118:30
hemnajgriffith, +118:31
jgriffithjust PLEASE for the love of all that is Beer, DO NOT CREATE A NEW CLIENT :)18:31
* e0ne rolls back a spec18:31
*** kfarr has quit IRC18:31
hemnasmcginnis, ping18:32
bswartzjgriffith: I read your ML post about snapshots and extend -- do you happen to know if there's tempest test coverage for that particular behavior (create from snapshots when volume was extended after snapshot)?18:32
e0nethe current state of propose to install python-cinder-brick as an extension18:32
*** subhadeep_de has joined #openstack-cinder18:33
*** adrianofr has quit IRC18:33
*** adrianofr has joined #openstack-cinder18:33
*** p0rtal has joined #openstack-cinder18:33
smcginnishemna: sup18:33
smcginnisbswartz: DuncanT was going to look in to adding some tests.18:34
*** angela-s has joined #openstack-cinder18:34
bswartzsmcginnis: +100018:34
hemnaso this bug:
openstackLaunchpad bug 1449492 in os-brick "Cinder not working with IPv6 ISCSI" [Medium,In progress] - Assigned to Lukas Bezdicka (social-b)18:34
hemnait's not a bug in cinder, nor is it a bug in Nova.18:34
hemnabrick is really doing the work18:34
hemnadoes it make sense to remove cinder and nova from that bug Affects list?18:34
smcginnishemna: A bug in brick?18:35
smcginnishemna: Yeah, I think so.18:35
hemnaI'm trying to triage the os-brick bugs and make sure they are up to date.18:35
smcginnishemna: Awesome!18:35
hemnaok so, in general, is it ok to remove cinder/nova affects for things like this18:35
smcginnisYeah, I'd say remove it from cinder then if it's all inside brick.18:35
hemnafolks that file the bugs don't know, and that's ok.18:35
smcginnisYeah, I think so.18:35
smcginnishemna: Yeah, agreed. Up to us I think to track down where the bug actually is.18:36
hemnaok cool.  just wanted to make sure.18:36
smcginnishemna: End users don't always know.18:36
smcginnishemna: Thanks!18:36
hemnacan't remove nova for some reason on that bug18:36
hemnamaybe because there was a patch against nova originally18:37
*** subhadeep_de has quit IRC18:37
smcginnishemna: Hmm, maybe.18:37
smcginnisOr maybe someone from nova needs to do it?18:38
hemnayah could be.18:38
hemnathe os-brick patch is up and has 1 +218:38
hemnaI'll just mark the nova side fixed once it's in.18:38
smcginnisCool, that should work too.18:38
e0neneed to head home. will be back later tonight18:39
smcginnise0ne: Thanks for sticking with this spec.18:40
* e0ne hopes that spec will be approved today18:40
*** jungleboyj has joined #openstack-cinder18:40
*** e0ne has quit IRC18:41
*** aix has quit IRC18:43
*** lpetrut has joined #openstack-cinder18:45
hemnacan you ssh into the cirros booted vms ?18:45
hemnawhat are the credentials ?18:46
smcginnishemna: I thought so. It's been a while since I've used them.18:46
guitarzanhemna: it's something silly18:46
hemnaI tried ssh cirros@<ip>  no joy18:47
guitarzanthere is a password18:47
guitarzangoogle is failing me18:48
guitarzanthat's it18:48
hemnawhich user ?18:48
hemnaPermission denied, please try again.18:49
guitarzanwith the smiley?18:49
angela-syah, i wasn't trying to be cute18:49
hemnaok I missed that18:49
smcginnisHah, this could be a weird conversation for someone not familiar.18:50
angela-swhen isn't the conversation weird on here?18:50
hemnaildikov, ok so I restacked, and the volume is attached to 2 vms, but after the 2nd attach the volume is still in 'attaching' state.18:51
smcginnisangela-s: Touche!18:51
ildikovhemna: hmm, that's interesting18:51
hemnatrying to track it down18:52
ildikovhemna: I mean that state should be in-use after the first one, right?18:52
ildikovI'm just checking the code further, but to be honest the translations are putting me to hell18:53
ildikovthe api code is duplicated in the sense of existing in both Cinder and Nova18:53
hemnaildikov, it is after the 1st attach18:53
hemnaand the 2nd nova volume-attach call works18:53
hemnanova finds the volume and gives it to the 2nd vm18:54
ildikovand in Nova there is code for untranslate18:54
hemnabut that's it.  the volume should get moved from attaching back to in-use18:54
ildikovso the last step seems to be missing18:55
hemnaI'm trying to find out if cinder was called (attach) after nova is done18:55
ildikovbut that should not depend on which attach you're in as far as I saw from the code18:56
ildikov... I mean it does not really touch the mechanism itself it just adds an 'or' to an 'if' to use the volume if it's 'in-use' too18:57
ildikovI couldn't even find whether it allows the re-use only if multiattach is true or always18:58
hemnacheck_attach is the only place nova really looks at that flag18:59
ildikovunderstanding Nova code is a full time job...18:59
*** david-ly_ has joined #openstack-cinder19:00
hemnahehe for sure19:00
*** haomaiwang has quit IRC19:01
ildikovhemna: yeah, right check_attach looks like the one19:01
*** haomaiwang has joined #openstack-cinder19:01
*** david-ly_ is now known as david-lyle_19:03
*** david-lyle has quit IRC19:03
*** electrocucaracha has quit IRC19:04
*** david-lyle_ is now known as david-lyle19:04
*** salv-orlando has joined #openstack-cinder19:05
hemnaildikov, strange19:07
*** akerr is now known as akerr_away19:07
hemnaI did another test, and nova never seems to call cinder after the 2nd attach completes19:07
hemnathere should have been another os-attach call to the c-api19:09
*** Apoorva has quit IRC19:09
*** zhipeng has quit IRC19:10
*** geguileo has quit IRC19:10
hemnayup, nova never calls attach after the 2nd one completes19:10
*** zhipeng has joined #openstack-cinder19:11
*** electrocucaracha has joined #openstack-cinder19:13
ildikovhemna: this patch modifies the compute part:
ildikovI cannot see anything that would affect calling out to Cinder19:15
*** e0ne has joined #openstack-cinder19:15
hemnaildikov, I think I found it.....19:16
hemnait's only going to call cinder attach if the volume attach_status was detached19:17
hemnaduring the 2nd attach it will be 'attached'19:17
*** alonma has joined #openstack-cinder19:17
hemnaso it won't call volume_api.attach19:17
hemnathus leaving the volume in attaching state19:18
hemnathat's like 308 in the updated patchset19:18
openstackgerritNate Potter proposed openstack/cinder: Move retype quota checks to API
hemnabut that check is the same as line 305 on github19:18
ildikovhemna: resolving merge conflicts on a totally unknown code :(19:20
ildikovhemna: sorry for wasting your time on this19:20
hemnano worries19:20
hemnaI'm glad I can help19:20
ildikovhemna: I triple checked it several times and there are still tiny things like this :.(19:21
ildikovby tiny I mean the size19:21
hemnayah that's one of the issues in nova.  it's so amazingly complex19:21
hemnaone small thing can break it all19:21
ildikovwill upload a new version in a few mins19:22
*** alonma has quit IRC19:22
hemnaok thanks!!19:22
*** Apoorva has joined #openstack-cinder19:24
openstackgerritApoorva Deshpande proposed openstack/cinder: Retype functionality in Tintri driver
*** sgotliv_ has joined #openstack-cinder19:26
*** wanghao has quit IRC19:27
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)
ildikovhemna: I removed the check, new version is up19:27
hemnaok thanks.19:29
hemnaildikov, gerrit says merge conflict19:30
ildikovI shouldn't have mentioned the joy of rebasing this code :)19:31
*** p0rtal_ has joined #openstack-cinder19:32
*** subhadeep_de has joined #openstack-cinder19:32
hemnaafter 2nd attach status = 'in-use'19:33
e0nehemna: did you start any activities related to force detach?19:34
hemnae0ne, no19:34
hemnae0ne, it's on my list, but unfortunately nova is closed19:35
e0neok, so I can start work on spec19:35
hemnaso, we are toast19:35
e0nenova closed for new specs in mitaka?19:35
ildikovhemna: awesome, thanks :)19:35
*** salv-orlando has quit IRC19:36
ildikovhemna: the merge conflict was in a test, so the code is not worse than it was19:36
*** p0rtal has quit IRC19:36
ildikove0ne: yeap, they closed last week19:36
hemnait seems like you get maybe 4 weeks to add nova-specs after the summit19:37
hemnaand that's it.19:37
e0neanyway, we can start with cinder side19:37
hemnae0ne, sure.19:37
hemnaunused code though, which sux19:37
e0neand propose for N release in nova19:37
ildikove0ne: you can still add one to the backlog folder19:37
e0nehemna: +119:38
ildikovand then ping guys as soon as possible close to the end of Mitaka or very early N19:38
e0neildikov: thanks for the advise19:38
ildikove0ne: np, I wish I would know any better option19:39
ildikovhemna: quick question19:39
ildikovdo you happen to have n-api output too from your tests?19:40
*** diablo_rojo has quit IRC19:40
ildikovI have a few failing tempest test cases, where the 'serverId' field in the attachments seems to be null19:42
hemnajgriffith, ping19:42
hemnaTarget 2,3,4 have multiple attaches to the same host19:44
*** _cjones_ has joined #openstack-cinder19:45
openstackgerritYuriy Nesenenko proposed openstack/cinder: Small refactoring in
ildikovhemna: tnx19:45
hemnaildikov, anything else ?19:45
ildikovhemna: not for now, I will try to set up an env too to be able to test manually19:46
*** briancurtin has quit IRC19:46
hemnaok cool.19:46
hemnathanks so much for the help19:46
ildikovhemna: waiting for the gate and even the local test runs takes ages19:46
*** merooney has quit IRC19:46
* ildikov thanks19:46
hemnayah :(19:46
hemnait's a painful process for sure19:46
*** martyturner has quit IRC19:46
*** briancurtin has joined #openstack-cinder19:47
ildikovhemna: is there anything with the Devstack env I should keep an eye on, or just spin up an env and it should work?19:47
hemnaI didn't do anything special19:47
hemnaI just fired up another vagrant and pulled the nova patchset and stacked.19:48
hemnajust remember to create the lvm volume with the --allow-multiattach flag19:48
hemnacinder create --allow-multiattach --name test 119:48
hemnaafter that and the updated nova patch, it worked.19:48
hemnathe detach bug is still there with lvm fwiw19:48
hemnawhich is what I'm trying to figure out how to fix w/o disabling multiattach for it.19:49
ildikovhemna: ok, thanks for the advice and also for checking that bug19:49
*** diablo_rojo has joined #openstack-cinder19:49
*** martyturner has joined #openstack-cinder19:50
*** electrocucaracha has quit IRC19:52
*** harlowja_at_home has quit IRC19:54
*** john5223 is now known as zz_john522319:56
*** fthiagogv has quit IRC19:57
*** david-lyle has quit IRC19:58
*** ociuhandu has quit IRC19:58
*** david-lyle has joined #openstack-cinder19:58
*** ChubYann has joined #openstack-cinder19:59
*** akerr_away is now known as akerr19:59
*** akerr is now known as akerr_away19:59
*** akerr_away is now known as akerr19:59
hemnanova never sends the host_name along in the attach call to cinder19:59
hemnaif it did19:59
*** mrnobody has joined #openstack-cinder20:00
*** merooney has joined #openstack-cinder20:00
*** haomaiwang has quit IRC20:01
hemnawe could test the attachments table20:01
hemnaand track it that way20:01
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Add validation for volume_type of volume object
*** haomaiwang has joined #openstack-cinder20:01
e0nehemna: so host_name is not used anyware:(20:04
hemnait's easy to add20:05
hemnaI hacked ildikov's patch and going to see if that gets it in the table20:05
hemnathe lvm driver can use it to see if it has any more attachments to that host20:05
hemnaand then use that to decide to remove the target or not.20:05
*** hemna is now known as hemnalunch20:06
scottdae0ne: We are working to get this Nova force-detach solution through:
scottdae0ne: This spec for a new API died after many months and much pain:
*** salv-orlando has joined #openstack-cinder20:10
e0nescottda: thanks! I'll take a look on both patches tomorrow20:10
scottdae0ne: There's an ML thread about the contentious issues:
scottdae0ne: And it is to be discussed at Nova meeting this week; December 10th 2015 1400 UTC20:11
e0nescottda: good, I'll be there20:12
scottdaLet me know if you want to discuss, I've a long history here :)20:12
*** alonma has joined #openstack-cinder20:12
e0nescottda: I have to read spec, patch and all comments before20:13
scottdaYou could start with the ML thread, might save some time....20:13
scottdaRead the patch for an idea of where this is likely to go...20:14
scottdaShort summary of spec for API change is: folks thought Cinder API should be made more robust so this doesn't happen (I discussed doing this in the future releases at Tokyo summit)...20:15
scottdaand people didn't want to change the Nova API with a --force option for volume detach20:15
scottdaBut there's a lot of back and forth20:15
*** vgridnev has quit IRC20:17
*** xyang1 has quit IRC20:17
*** alonma has quit IRC20:17
jgriffithhemnalunch: rigth, each with it's own target/nexus20:17
*** xyang1 has joined #openstack-cinder20:17
e0nescottda: I need to read more to be know what nova need20:18
*** xyang has joined #openstack-cinder20:18
e0nescottda: I don't want to start it at 10pm20:18
jgriffithhemnalunch: oh.. wait, what?20:19
scottdae0ne: Cool. The use case of cinder-attach-without-nova is interesting because we might find misbehaving client of that API will need force detach as well. It would be nice if we could do all the Cinder-side force-detach without any help from the consumer.20:20
*** alonma has joined #openstack-cinder20:20
e0nescottda: yea, that's why I'm interesting on it20:20
*** p0rtal has joined #openstack-cinder20:21
scottdaCinder has force-detach API already, but it needs a connector passed in.20:22
e0nescottda: we discussed at summit, that we want to save this info in the DB20:22
scottdayes. hemnalunch was going to work on that, but he might need some help.20:23
scottdaI hope to have some time for this after the new year.20:23
*** p0rtal_ has quit IRC20:24
*** alonma has quit IRC20:24
*** merooney has quit IRC20:26
*** alonma has joined #openstack-cinder20:26
*** mrnobody has quit IRC20:29
*** alonma has quit IRC20:30
*** Lee1092 has quit IRC20:31
*** alonma has joined #openstack-cinder20:32
*** esker has quit IRC20:33
*** alonma has quit IRC20:36
*** alonma has joined #openstack-cinder20:38
openstackgerritSimon Dodsley proposed openstack/cinder: Update Pure REST API supported version numbers in Pure driver
*** belmoreira has joined #openstack-cinder20:41
*** alonma has quit IRC20:42
*** alonma has joined #openstack-cinder20:44
*** alonma has quit IRC20:48
*** xyang has quit IRC20:48
*** dims has joined #openstack-cinder20:49
*** alonma has joined #openstack-cinder20:50
*** merooney has joined #openstack-cinder20:53
*** alonma has quit IRC20:55
*** krtaylor has quit IRC20:56
*** Apoorva has quit IRC20:57
*** dims has quit IRC20:58
*** electrocucaracha has joined #openstack-cinder20:58
*** jseiler has quit IRC20:58
*** baumann1 has joined #openstack-cinder21:00
*** dims has joined #openstack-cinder21:00
*** haomaiwang has quit IRC21:01
*** dims has quit IRC21:01
*** 17WAA61QI has joined #openstack-cinder21:01
*** baumann has quit IRC21:02
*** xyang has joined #openstack-cinder21:04
*** akerr has quit IRC21:05
*** merooney has quit IRC21:05
*** raildo is now known as raildo-afk21:06
*** p0rtal_ has joined #openstack-cinder21:06
*** hemnalunch is now known as hemna21:08
openstackgerritJohn Griffith proposed openstack/cinder: Add cluster Pairing setup to SolidFire driver
*** ctina_ has quit IRC21:09
*** lpetrut has quit IRC21:09
hemnascottda, so yah the connector needs to get saved, but nova has to change to pass it at attach_volume time.21:10
*** p0rtal has quit IRC21:10
scottdaoh yeah, that's right. Crap21:10
jgriffithxyang: ok, I *think* I caught all the things that got lost in the rebases :(21:10
scottdahemna: IF that counts as a Nova API change, it needs a spec. Which means not until N release21:11
scottdaWe could still do force-detach-all in Mitaka. Unless I've forgotten something else...21:11
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)
hemnascottda, well the nova api itself isn't changing21:13
hemnabut we need to update cinderclient to accept the connector and get nova to pass the connector at attach time.21:13
scottdayeah, true. Might be able to get this in Nova then.21:13
e0nein such case, we need to release cinderclient very fast21:14
scottdaWe can still go forward with cinderclient and cinder server changes.21:14
scottdae0ne: I thought you were going to sleep sometime :)21:14
smcginnisWe should be able to get an update out within a couple days once we know it's ready.21:14
smcginnisscottda: Yeah, no kidding. e0ne's pulling an all nighter!21:15
smcginnisNo sleep until you get your spec approved! :P21:15
scottdaHow about the cinder changes. Does that need a spec? If so....21:15
e0newhat is 'to sleep'?21:15
hemnascottda, I think so.  we need a new db column in the volume_attachment table21:15
smcginnisCinder changes for multiattach?21:16
hemnato store the connector21:16
hemnaand then an API change to accept it, etc21:16
smcginnishemna: Oh, yeah. We probably should have a spec so it's at least documented.21:16
scottdasmcginnis: cinder changes to store the volume connector at attach time, so we can use if for a 'force-detach' from cinder-only, without nova help21:16
*** erlon has quit IRC21:16
scottdaamongst other reasons21:16
smcginnisWas originally thinking it would be just a bug fix, but that's probably significant enough we should do a spec.21:16
hemnaso passing host_name along with the attach call fails21:17
hemnacinder replies with 40021:17
scottdahemna: I *think* I can work on this .....need to have my annual 5 minute conversation with my manager next week.21:17
e0newe've got one issue whith this21:17
e0newe can connector info for iscsi, nfs21:17
e0nebut we can't do it in he same way for rbd21:18
hemnarbd doesn't add anything to the connector21:18
hemnaas well as the other vendor connectors21:18
e0newhat about other protocols? I didn't check them21:18
e0neok, sounds good21:18
hemnathe reply to that is the initiator connector dict21:19
*** eharney has quit IRC21:19
*** thangp has quit IRC21:24
*** belmoreira has quit IRC21:26
*** lpetrut has joined #openstack-cinder21:27
e0newill try to sleep. it sound like a good thing:). bb tomorrow21:32
*** lpetrut has quit IRC21:32
*** p0rtal_ has quit IRC21:32
*** e0ne has quit IRC21:32
*** ianbrown has quit IRC21:33
*** p0rtal has joined #openstack-cinder21:33
*** chris_morrell has quit IRC21:34
*** chris_morrell has joined #openstack-cinder21:35
*** p0rtal has quit IRC21:37
*** Apoorva has joined #openstack-cinder21:39
*** p0rtal has joined #openstack-cinder21:44
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)
*** zz_john5223 is now known as john522321:52
openstackgerritAlex O'Rourke proposed openstack/cinder: LeftHand: Implement v2 replication (unmanaged)
*** nikeshm has quit IRC21:54
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
openstackgerritJohn Griffith proposed openstack/cinder: Add replicate by volume-type to SolidFire driver
*** 17WAA61QI has quit IRC22:01
*** garthb has joined #openstack-cinder22:01
*** haomaiwang has joined #openstack-cinder22:01
smcginnishemna: It's not talk like a pirate day, is it? :P22:02
hemnait should be22:02
smcginnishemna: Missed it. It was Sept 19th:
*** edmondsw has quit IRC22:04
*** electrocucaracha has quit IRC22:05
jgriffithsmcginnis: ye are too late22:07
jgriffithsmcginnis: tis talketh liketh Shakespeare day22:08
smcginnisSurely you jest. It doth on current daylight breaks?22:08
smcginnis(Or something like that!)22:09
jgriffithsmcginnis: well play'd sir!22:09
hemnaso... it seems the sqlalchemy Volume object doesn't have the volume attachments22:11
hemnaso when a driver gets called w/ the Volume model object the attachments aren't in there.22:12
hemnaIt looks like the consistencygroup var is being created in the Volume object22:12
hemnabut I'm not sure how to get the attachments added to it automatically22:13
openstackgerritNate Potter proposed openstack/cinder: Show qos_specs_id based on policy
hemna^^ I presume that adds the consistency group object in the Volume object ?22:13
DuncanTTo call or not to call, whether 'tis nobler in the code to accept anonymous attachments, or let them go unattached, and in doing so take battle against all previous versions?22:13
smcginnisNow that is well played! ^^22:14
smcginnishemna: Isn't dulek our sqlalchemy expert? :)22:14
hemnaIf I can get that list of attachments added to the volume object, I think I have a solution for lvm22:15
*** crose has joined #openstack-cinder22:15
hemnawhich would be helpful for any driver that does the same thing that lvm does (1 export per host)22:15
openstackgerritJohn Griffith proposed openstack/cinder: Add snapshot ID's to volume detail view
smcginnishemna: Mine does.22:16
jgriffithDuncanT: BRAVO' FRIKIN'O22:16
smcginnisFYI: grenade tests are probably going to fail until this makes it through:
jgriffithDuncanT: err... "BravFrikin'O"22:17
jgriffithsmcginnis: and my work here is done for the day... off to the bar22:17
smcginnisHehe, yep22:18
mc_nairhemna: I think you should be able to access the volume_attachments using volume.volume_attachment and it will lazy load it from the DB once you do that (
hemnamc_nair, hrmm22:18
hemnaI can try at least.....22:18
openstackgerritAngela Smith proposed openstack/cinder: Adds HTTPS southbound connector for Brocade FC Zone Driver
openstackgerritNate Potter proposed openstack/cinder: Show qos_specs_id based on policy
mc_nairhemna: think that won't actually get loaded til you access it22:19
hemnawhat do you know22:20
hemnamc_nair, that worked22:20
mc_nairhemna: uh oh- it worked and you're still hrmm-ing?22:20
openstackgerritSimon Dodsley proposed openstack/cinder: Update Pure REST API supported version numbers in Pure driver
hemnamc_nair, it worked!22:21
hemnaI'm continuing my hatched job22:21
*** dave-mccowan has quit IRC22:21
mc_nairhemna: haha cool - just wanted to make sure something wasn't still off with this in particular.  Good luck22:22
mc_nairhemna: btw, scrolled up and saw you saying you're plates pretty full, if there's some coding pieces you could use a hand with lemme know and I'd be happy to help out with them22:23
DuncanTCarving from the carcass a choice cut, a pound of fair code to wet the mouth and sate the appetites of the clammering hordes who scream for bloody and multiple attachments?22:23
*** crose has quit IRC22:24
jgriffithDuncanT: I must say I'm rather impressed... don't know what the hell you just said, but just the same :)22:24
smcginnisjgriffith: Is that any different than normal?22:24
hemnamc_nair, ok thanks man22:24
jgriffithsmcginnis: that was my next line :)22:24
*** ebalduf has quit IRC22:25
*** alejandrito has quit IRC22:26
mc_nairDuncanT: I'm sold.  You have a past life as an Elizabethan salesman?22:27
DuncanTjgriffith: If a fool's capering is taken as an oracle by the wise who are ignored by the fool, then is not the fool wise and the wise fools? What we taketh in our sup is by our own hand to mouth poured and so no man may claim his fate in another's hands but that he choice that place that it was put and so guided it himself?22:27
hemnaok so terminate_connection has the connector, but the lvm driver nukes the target during remove_export22:27
hemnaso close...yet so far22:27
DuncanTmc_nair: Ex theatre tech who'd bum beer between shows for such performances... I'm out of practice22:28
scottdahemna: Won't we want to put logic in (the manager) to only call remove_export upon the last attachment going away during call to detach()22:30
scottdaElse the individual drivers will have to deal with it.22:30
mc_nairDuncanT: haha nice. Beer... the great motivator22:30
hemnacould be22:30
jgriffithscottda: but that only works if you're attached on same host22:31
hemnaI'm just hacking around at the moment22:31
jgriffithscottda: otherwise you should call it on the node22:31
jgriffithscottda: and by the way, multi-attach to the same node should just share the same connection anyway22:31
scottdajgriffith: Is that true for all drivers?22:31
jgriffithscottda: I suppose that they can do whatever they want, but I don't see a reason why not22:32
jgriffithscottda: make management easier that's for sure22:32
scottdaI though 3par did it one way and Lefthand the other hemna ?22:32
*** adrianofr has quit IRC22:33
jgriffithscottda: my point was just that before rewriting everything for a specific backend might be good to look a little more at things22:33
jgriffithanyway... meeting22:33
scottdajgriffith: I agree. It would suck to be specific to the back end.22:33
*** david-lyle has quit IRC22:34
scottdaBut if some drivers have 1 connection/host for multiple attaches, and some have 1 connection /attach that's a problem22:35
hemnait looks like that would 'work'22:35
hemnawould have to migrate that logic into the manager22:35
*** baumann1 has quit IRC22:35
hemnainstead of the hack I did in the lvm.py22:35
scottdaship it22:36
hemna3par does a new export for every volume, even on the same host22:36
hemnalefthand acts like lvm22:36
scottdaDoes that logic look at the host field?22:36
hemna1 export per host only22:36
hemnamy hack to lvm.py22:36
scottdai.e. can it be made generic for both cases (LH and 3par style)22:36
hemnascottda, ^^22:36
hemnalet me rework that hack and move the code into the manager from lvm.py22:37
scottdalooks good22:38
hemnathe good thing is, that the nova multiattach code hasn't landed yet22:38
scottdaright, we can still influence that22:38
hemnaand this required me changing nova code to pass the host22:38
*** alonma has joined #openstack-cinder22:38
*** briancurtin has quit IRC22:38
scottdaWell, we might as well work to get whatever changes we want into Nova. Host, connector, kitchen sink22:39
*** david-ly_ has joined #openstack-cinder22:39
*** dilip has quit IRC22:39
hemnawho calls manager.remove_export22:39
*** patrickeast has quit IRC22:40
hemnaah yes our friend detach_volume22:40
guitarzanwait, did hemna just say it was good that nova's code hasn't landed?22:40
hemnahehe :)22:41
guitarzanit's like bizarro day :)22:41
hemnapirate bizarro22:41
*** alonma has quit IRC22:43
*** alonma has joined #openstack-cinder22:45
*** david-ly_ is now known as david-lyle22:45
*** jgregor has quit IRC22:47
*** jgregor has joined #openstack-cinder22:47
*** jgregor has quit IRC22:48
*** alonma has quit IRC22:49
*** merooney has joined #openstack-cinder22:49
*** diablo_rojo has quit IRC22:50
*** mriedem is now known as mriedem_away22:51
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Add validation for volume_type of volume object
*** jungleboyj has quit IRC22:51
*** harlowja has quit IRC22:51
*** alonma has joined #openstack-cinder22:51
*** harlowja has joined #openstack-cinder22:51
hemnait only calls remove_export if there are no more attachments on that host.22:55
hemnaterminate_connection gets called either way22:55
*** alonma has quit IRC22:55
*** esker has joined #openstack-cinder22:56
hemnaI guess I should make a formal patch out of this and push it up for revew22:57
scottdaAnd a bug I reckon22:57
*** esker has quit IRC22:57
openstackLaunchpad bug 1471278 in Cinder "target removal on detaching volume with multi-attach flag" [Undecided,New] - Assigned to Walt Boring (walter-boring)22:58
hemnathat's what started all of this22:58
scottdaOH, yeah, that's an oldy but a goody22:58
*** esker has joined #openstack-cinder22:58
*** zhipeng has quit IRC22:58
scottdahemna: Have you been reviewing Illdikov's patches? They seem to keep failing build, so I haven't delved into them....22:59
hemnaand the best part is that the lvm driver isn't changed22:59
hemnascottda, tempest has been giving her fits22:59
*** boris-42_ has joined #openstack-cinder22:59
scottdaand none of the drivers need to change, yes?22:59
hemnascottda, correct22:59
scottdaso that is very cool.22:59
hemnathis even works with the 3PARism22:59
*** akshai_ has quit IRC22:59
hemnasince it seems to be the bastard child of the lot23:00
scottdaOK, I'll start reviewing those Nova patches. We for sure need the host passed in. But perhaps we can get the connector info as well.23:00
*** haomaiwang has quit IRC23:01
*** haomaiwang has joined #openstack-cinder23:01
*** akshai has joined #openstack-cinder23:01
hemnaI suppose it wouldn't be ideal to put the connector change in with the multiattach effort23:01
hemnabut sure would be nice to add23:01
hemnano changes to attach in there23:02
*** vgridnev has joined #openstack-cinder23:03
scottdaBut this is only for force-detach, so not critical23:04
scottdaJust a dream...23:04
*** dave-mccowan has joined #openstack-cinder23:07
hemnaI think that could be next23:08
hemnajust need a new column in the attachment table to track the connector per attachment.23:08
*** dustins has quit IRC23:09
*** martyturner has quit IRC23:13
*** dustins has joined #openstack-cinder23:14
*** alonma has joined #openstack-cinder23:19
*** vgridnev has quit IRC23:19
*** xyang has quit IRC23:22
*** alonma has quit IRC23:24
*** xyang1 has quit IRC23:25
*** briancurtin has joined #openstack-cinder23:29
*** dilip has joined #openstack-cinder23:32
*** haomaiwang has quit IRC23:32
*** haomaiwang has joined #openstack-cinder23:32
*** ntpttr has left #openstack-cinder23:34
*** ianbrown has joined #openstack-cinder23:36
*** ianbrown has quit IRC23:36
*** haomaiwang has quit IRC23:37
*** rcernin has joined #openstack-cinder23:38
*** takedakn has joined #openstack-cinder23:41
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - get_short_host_name not called in find_device_number
*** sgotliv_ has quit IRC23:48
*** patrickeast has joined #openstack-cinder23:51
*** ianbrown has joined #openstack-cinder23:55
*** [1]Thelo has joined #openstack-cinder23:57
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: WIP don't call remove_export with attachments left
*** Thelo has quit IRC23:59
*** [1]Thelo is now known as Thelo23:59

Generated by 2.14.0 by Marius Gedminas - find it at!