Friday, 2015-07-17

*** salv-orlando has joined #openstack-cinder00:01
*** dannywilson has quit IRC00:02
*** openstack has joined #openstack-cinder00:04
*** IanGovett has joined #openstack-cinder00:04
*** markvoelker has quit IRC00:04
*** vokt has quit IRC00:04
*** salv-orlando has quit IRC00:06
*** timcl has quit IRC00:09
*** tsekiyam_ has joined #openstack-cinder00:11
*** boris-42 has quit IRC00:12
*** akerr is now known as akerr_away00:12
*** sghanekar_ has joined #openstack-cinder00:13
*** vilobhmm has quit IRC00:14
*** vilobhmm has joined #openstack-cinder00:14
*** gouthamr_ has joined #openstack-cinder00:14
*** tsekiyama has quit IRC00:15
*** mtanino has quit IRC00:15
*** tsekiyam_ has quit IRC00:16
*** gouthamr has quit IRC00:17
*** rushil has joined #openstack-cinder00:17
*** dalgaaf has quit IRC00:18
*** heyun has joined #openstack-cinder00:23
openstackgerritAnthony Lee proposed openstack/cinder: Fix 3PAR driver handling of existing VLUNs
*** garthb has quit IRC00:29
openstackgerritAnthony Lee proposed openstack/cinder: Fix 3PAR driver handling of existing VLUNs
openstackgerritAnthony Lee proposed openstack/cinder: Fix 3PAR driver handling of existing VLUNs
*** barra204 has quit IRC00:39
*** akerr_away is now known as akerr00:39
*** leeantho has quit IRC00:39
*** gouthamr_ has quit IRC00:40
*** gouthamr has joined #openstack-cinder00:41
*** cdelatte has quit IRC00:41
*** laughterwym has joined #openstack-cinder00:45
*** diogogmt has quit IRC00:46
*** akerr has quit IRC00:48
*** diogogmt has joined #openstack-cinder00:49
*** topshare has joined #openstack-cinder00:51
*** earlephilhower has quit IRC00:55
*** skylerberg has joined #openstack-cinder00:59
*** gouthamr_ has joined #openstack-cinder01:03
*** gouthamr has quit IRC01:04
*** liuxg has joined #openstack-cinder01:04
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor
*** jasondotstar has quit IRC01:08
*** lixiaoy1 has joined #openstack-cinder01:09
*** IanGovett has quit IRC01:12
*** _cjones_ has quit IRC01:12
*** hemna has joined #openstack-cinder01:14
*** gouthamr_ has quit IRC01:18
*** Apoorva has quit IRC01:20
*** hemna has quit IRC01:21
*** sghanekar_ has quit IRC01:26
*** hemna has joined #openstack-cinder01:27
*** breitz has quit IRC01:30
*** breitz has joined #openstack-cinder01:30
*** Lee1092 has joined #openstack-cinder01:31
*** afazekas has quit IRC01:34
*** patrickeast has quit IRC01:38
openstackgerritLisaLi proposed openstack/python-cinderclient: Show backup and volume info in backup_restore
*** jdandrea has quit IRC01:46
*** liuxg has quit IRC01:48
*** tobe_ has joined #openstack-cinder01:49
*** vilobhmm has quit IRC01:57
*** HoloIRCUser2 has joined #openstack-cinder01:57
*** HoloIRCUser5 has joined #openstack-cinder02:02
*** HoloIRCUser2 has quit IRC02:02
*** HoloIRCUser2 has joined #openstack-cinder02:03
*** HoloIRCUser5 has quit IRC02:03
*** HoloIRCUser2 is now known as liuxg02:03
openstackgerritThang Pham proposed openstack/cinder: WIP: Continue conversion to volume object
*** jungleboyj has joined #openstack-cinder02:13
*** piyanai has joined #openstack-cinder02:19
*** dims_ has quit IRC02:31
*** zhenguo has joined #openstack-cinder02:32
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: improve create and delete operation
*** chlong-afk has quit IRC02:37
*** chlong has joined #openstack-cinder02:42
*** lcurtis has joined #openstack-cinder02:46
openstackgerritTom Jose Kalapura proposed openstack/python-cinderclient: Add response message when volume delete
openstackgerritVincent Hou proposed openstack/cinder: Volume status management during migration
*** hakimo has joined #openstack-cinder02:53
*** hakimo_ has quit IRC02:55
*** piyanai has quit IRC02:55
*** hakimo_ has joined #openstack-cinder03:10
*** hakimo has quit IRC03:10
openstackgerritYaguang Tang proposed openstack/cinder: Remove simple scheduler which is deprecated since Juno
*** boris-42 has joined #openstack-cinder03:14
*** julim has quit IRC03:18
*** topshare has quit IRC03:27
*** hakimo has joined #openstack-cinder03:27
*** hakimo_ has quit IRC03:27
*** lcurtis has quit IRC03:31
*** dims_ has joined #openstack-cinder03:32
*** topshare has joined #openstack-cinder03:32
openstackgerritSeiji Aguchi proposed openstack/cinder: Remove resource lock operation for HBSD
*** dims_ has quit IRC03:37
*** rushil has quit IRC03:38
*** annegentle has joined #openstack-cinder03:38
*** hakimo_ has joined #openstack-cinder03:44
*** hakimo has quit IRC03:44
openstackgerritLei Li proposed openstack/cinder: Adjust to right exception for chunkeddriver
*** mriedem1 has quit IRC03:46
*** HoloIRCUser1 has joined #openstack-cinder03:50
*** liuxg has quit IRC03:51
*** HoloIRCUser1 has quit IRC03:54
*** coolsvap|away is now known as coolsvap03:54
*** xyang has quit IRC03:55
*** hemna has quit IRC03:58
openstackgerritLiu Xinguo proposed openstack/cinder: Enhance FC zone support for Huawei driver
*** erlon has quit IRC04:00
*** hakimo has joined #openstack-cinder04:01
*** hakimo_ has quit IRC04:01
openstackgerritLiu Xinguo proposed openstack/cinder: Add hypermetro support for huawei volume driver
*** hemna has joined #openstack-cinder04:08
*** hemna has quit IRC04:11
*** annegentle has quit IRC04:11
*** hemna has joined #openstack-cinder04:12
*** hakimo_ has joined #openstack-cinder04:18
*** hakimo has quit IRC04:18
*** HoloIRCUser2 has joined #openstack-cinder04:22
*** abhinavsgsits has joined #openstack-cinder04:23
*** HoloIRCUser3 has joined #openstack-cinder04:23
*** abhinavsgsits is now known as aagrawal04:24
*** lixiaoy1 has quit IRC04:26
*** HoloIRCUser2 has quit IRC04:27
*** HoloIRCUser3 is now known as liuxg04:28
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: improve snapshot and clone operation
*** harlowja_at_home has joined #openstack-cinder04:35
*** skylerberg has quit IRC04:38
openstackgerritwanghao proposed openstack/cinder: Support volume backup of qcow2 format volume
*** haomaiw__ has joined #openstack-cinder04:42
*** zhenguo has quit IRC04:43
*** haomaiwa_ has quit IRC04:44
*** hemna has joined #openstack-cinder04:48
*** hemna has quit IRC04:49
*** hemna has joined #openstack-cinder04:52
*** tobe_ has quit IRC04:53
*** aagrawal has quit IRC04:58
*** shyama has joined #openstack-cinder04:58
*** aagrawal has joined #openstack-cinder04:58
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication
*** liuxg has quit IRC04:59
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication
*** dannywilson has joined #openstack-cinder05:06
*** lixiaoy1 has joined #openstack-cinder05:09
openstackgerritLiu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver
*** boris-42_ has joined #openstack-cinder05:20
*** kaisers2 has joined #openstack-cinder05:21
*** ameade_ has joined #openstack-cinder05:21
*** jungleboyj_ has joined #openstack-cinder05:22
*** boris-42 has quit IRC05:23
*** kaisers1 has quit IRC05:23
*** jbernard has quit IRC05:23
*** ameade has quit IRC05:23
*** ondergetekende has quit IRC05:23
*** jungleboyj has quit IRC05:23
*** ameade_ is now known as ameade05:24
*** ondergetekende has joined #openstack-cinder05:24
*** boris-42_ is now known as boris-4205:24
*** jbernard has joined #openstack-cinder05:24
*** heyun has quit IRC05:25
*** heyun has joined #openstack-cinder05:28
*** lpetrut has joined #openstack-cinder05:28
*** topshare has quit IRC05:28
*** vilobhmm has joined #openstack-cinder05:29
*** lpetrut has quit IRC05:34
*** dannywilson has quit IRC05:35
*** dannywilson has joined #openstack-cinder05:36
*** s0nea has quit IRC05:37
*** BharatK has joined #openstack-cinder05:38
*** s0nea has joined #openstack-cinder05:40
*** markvoelker has joined #openstack-cinder05:41
BharatKthingee, Need your blessing for the patch
*** markvoelker_ has joined #openstack-cinder05:44
*** markvoelker has quit IRC05:45
*** tobe_ has joined #openstack-cinder05:46
*** topshare has joined #openstack-cinder05:46
*** heyun has quit IRC05:48
*** topshare has quit IRC05:48
*** heyun has joined #openstack-cinder05:50
*** annashen has joined #openstack-cinder05:51
*** topshare has joined #openstack-cinder05:54
*** deepakcs has joined #openstack-cinder05:56
*** dims_ has joined #openstack-cinder06:00
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: improve snapshot and clone operation
*** dims_ has quit IRC06:06
*** ig0r_ has joined #openstack-cinder06:08
*** harlowja_at_home has quit IRC06:09
*** agrawala has joined #openstack-cinder06:13
*** ronis has joined #openstack-cinder06:13
openstackgerritGaurang Tapase proposed openstack/cinder: Adding NFS support to the GPFS Driver
*** aagrawal has quit IRC06:15
*** tobe_ has quit IRC06:16
*** dannywil_ has joined #openstack-cinder06:24
*** dannywil_ has quit IRC06:25
*** dannywil_ has joined #openstack-cinder06:26
*** agrawala is now known as aagrawal06:26
*** dannywilson has quit IRC06:27
*** gaurangt has joined #openstack-cinder06:30
*** tobe_ has joined #openstack-cinder06:31
*** ronis has quit IRC06:35
*** aagrawal has quit IRC06:36
*** aagrawal has joined #openstack-cinder06:36
*** hemna has quit IRC06:36
*** ndipanov has quit IRC06:42
*** ndipanov has joined #openstack-cinder06:43
*** ig0r_ has quit IRC06:45
*** ig0r_ has joined #openstack-cinder06:51
*** belmoreira has joined #openstack-cinder06:51
openstackgerritLei Li proposed openstack/cinder: Adjust to right exception for chunkeddriver
*** s0nea has quit IRC07:00
*** topshare has quit IRC07:02
*** s0nea has joined #openstack-cinder07:02
*** sgordon has quit IRC07:03
openstackgerritVilobh Meshram proposed openstack/cinder: ConsistencyGroup Object
*** afazekas has joined #openstack-cinder07:06
*** topshare has joined #openstack-cinder07:06
*** vilobhmm has quit IRC07:07
*** markus_z has joined #openstack-cinder07:10
*** boris-42 has quit IRC07:12
*** aagrawal has quit IRC07:12
*** aagrawal has joined #openstack-cinder07:12
*** aagrawal has quit IRC07:19
*** aagrawal has joined #openstack-cinder07:19
*** heyun1 has joined #openstack-cinder07:25
*** sgordon has joined #openstack-cinder07:25
*** heyun has quit IRC07:26
*** abehl has joined #openstack-cinder07:27
*** ronis has joined #openstack-cinder07:32
*** jordanP has joined #openstack-cinder07:38
*** salv-orlando has joined #openstack-cinder07:39
*** abhishekk has joined #openstack-cinder07:42
*** dannywil_ has quit IRC07:49
*** dims_ has joined #openstack-cinder07:49
*** coolsvap is now known as coolsvap|afk07:51
openstackgerritLi Yingjun proposed openstack/cinder: Set default policy for "volume:get"
openstackgerritLei Zhang proposed openstack/cinder: Make concurrent backups for same volume impossible
*** dims_ has quit IRC07:56
*** kaisers2 has quit IRC07:59
*** annashen has quit IRC08:05
*** HoloIRCUser3 has joined #openstack-cinder08:07
*** HoloIRCUser3 has quit IRC08:11
*** ronis_ has joined #openstack-cinder08:12
*** ronis has quit IRC08:12
*** jistr has joined #openstack-cinder08:15
*** chenying has joined #openstack-cinder08:16
*** salv-orlando has quit IRC08:17
*** p_deore has joined #openstack-cinder08:21
*** laughterwym has quit IRC08:22
*** aagrawal has quit IRC08:26
*** aagrawal has joined #openstack-cinder08:27
*** agarciam has quit IRC08:33
openstackgerritPranali Deore proposed openstack/cinder: Validate name and description string
*** markvoelker_ has quit IRC08:42
*** lpetrut has joined #openstack-cinder08:42
*** lpetrut has quit IRC08:43
*** ociuhandu has joined #openstack-cinder08:43
*** aagrawal has quit IRC08:51
*** aagrawal has joined #openstack-cinder08:51
*** coolsvap|afk is now known as coolsvap08:54
*** markus_z has quit IRC08:56
*** aagrawal has quit IRC08:56
*** aagrawal has joined #openstack-cinder08:56
*** ociuhandu has quit IRC08:56
*** markvoelker has joined #openstack-cinder08:57
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication
*** sgotliv has quit IRC08:57
*** lpetrut has joined #openstack-cinder09:00
*** lpetrut has quit IRC09:00
openstackgerritwanghao proposed openstack/cinder: Support volume backup of qcow2 format volume
*** e0ne has joined #openstack-cinder09:01
*** markvoelker has quit IRC09:02
theanalystCan any of the cores please review the bp:
theanalystDuncanT: ^^09:04
*** deepakcs has quit IRC09:04
*** aix has joined #openstack-cinder09:07
*** Maike has joined #openstack-cinder09:08
*** lpetrut has joined #openstack-cinder09:10
*** markvoelker has joined #openstack-cinder09:12
*** markus_z has joined #openstack-cinder09:13
*** dims_ has joined #openstack-cinder09:15
*** lpetrut has quit IRC09:16
*** markvoelker has quit IRC09:17
*** agarciam has joined #openstack-cinder09:18
*** dims_ has quit IRC09:20
openstackgerritTaoBai proposed openstack/cinder: ==============WIP============== IBM Storwize Volume-replication V2 implementation
openstackgerritTaoBai proposed openstack/cinder: ==============WIP==============
openstackgerritTaoBai proposed openstack/cinder: ==============WIP==============
*** tobe_ has quit IRC09:25
*** markvoelker has joined #openstack-cinder09:26
*** topshare has quit IRC09:29
*** markvoelker has quit IRC09:31
*** rhagarty has quit IRC09:40
*** rhagarty_ has quit IRC09:40
*** sseverson has quit IRC09:40
*** cbader has quit IRC09:40
*** asselin_ has joined #openstack-cinder09:40
*** sseverson has joined #openstack-cinder09:40
*** asselin has quit IRC09:40
*** cbader has joined #openstack-cinder09:40
*** rhagarty has joined #openstack-cinder09:40
*** rhagarty_ has joined #openstack-cinder09:40
*** markvoelker has joined #openstack-cinder09:41
*** markvoelker has quit IRC09:45
*** lixiaoy1 has quit IRC09:50
openstackgerritAbhishek Lekshmanan proposed openstack/cinder: Backups: allow name to be specified during restore
openstackgerritJordan Pittier proposed openstack/cinder: Rework the Scality SOFS driver to use the RemoteFS class
*** piyanai has joined #openstack-cinder09:55
*** markvoelker has joined #openstack-cinder09:55
*** ankit has joined #openstack-cinder09:57
*** ankit has quit IRC09:58
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: improve image operations
*** ankit has joined #openstack-cinder09:59
*** ankit has quit IRC09:59
*** markvoelker has quit IRC10:00
*** ankit_ag has joined #openstack-cinder10:00
*** chenying has quit IRC10:03
*** dims_ has joined #openstack-cinder10:05
*** deepakcs has joined #openstack-cinder10:07
*** markvoelker has joined #openstack-cinder10:07
*** heyun1 has quit IRC10:10
*** markvoelker has quit IRC10:12
*** IanGovett has joined #openstack-cinder10:15
*** mdenny has quit IRC10:20
*** jasondotstar has joined #openstack-cinder10:21
*** markvoelker has joined #openstack-cinder10:22
*** openstackgerrit has quit IRC10:31
*** markvoelker has quit IRC10:32
*** openstackgerrit has joined #openstack-cinder10:32
openstackgerritYuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side
*** yuriy_n17 has joined #openstack-cinder10:35
*** markvoelker has joined #openstack-cinder10:37
*** aix has quit IRC10:41
*** markvoelker has quit IRC10:42
*** gaurangt has left #openstack-cinder10:45
*** aix has joined #openstack-cinder10:45
*** markvoelker has joined #openstack-cinder10:51
*** markvoelker has quit IRC10:56
openstackgerritDaniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object
*** vlaza has joined #openstack-cinder11:03
*** s0nea has quit IRC11:04
*** dims_ has quit IRC11:06
*** markvoelker has joined #openstack-cinder11:06
*** cdelatte has joined #openstack-cinder11:07
*** s0nea has joined #openstack-cinder11:07
*** HoloIRCUser3 has joined #openstack-cinder11:08
*** kcaj has joined #openstack-cinder11:09
*** markvoelker has quit IRC11:11
*** coolsvap is now known as coolsvap|away11:14
*** piyanai has quit IRC11:19
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
*** markvoelker has joined #openstack-cinder11:19
*** ganso_ has joined #openstack-cinder11:19
*** piyanai has joined #openstack-cinder11:20
*** piyanai has quit IRC11:22
*** markvoelker has quit IRC11:24
*** lcurtis has joined #openstack-cinder11:26
*** markvoelker has joined #openstack-cinder11:32
*** jordanP has quit IRC11:35
*** jordan__ has joined #openstack-cinder11:35
*** gouthamr has joined #openstack-cinder11:42
*** gouthamr has quit IRC11:42
openstackgerritDaniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object
*** gouthamr has joined #openstack-cinder11:43
openstackgerritDaniel Tadrzak proposed openstack/cinder: CGSnapshot Object
*** markvoelker has quit IRC11:44
*** HoloIRCUser2 has joined #openstack-cinder11:52
*** lcurtis has quit IRC11:53
*** HoloIRCUser3 has quit IRC11:54
*** markvoelker has joined #openstack-cinder11:55
*** HoloIRCUser3 has joined #openstack-cinder11:57
*** HoloIRCUser2 has quit IRC11:57
*** markus_z has quit IRC11:57
*** markvoelker has quit IRC12:00
*** nkrinner_afk is now known as nkrinner12:05
*** gouthamr has quit IRC12:06
*** jasondotstar has quit IRC12:06
*** porrua has joined #openstack-cinder12:08
*** markvoelker has joined #openstack-cinder12:09
openstackgerritDaniel Tadrzak proposed openstack/cinder: CGSnapshot Object
*** kjelly_ has joined #openstack-cinder12:13
*** markvoelker has quit IRC12:13
*** markvoelker has joined #openstack-cinder12:16
*** markvoelker has quit IRC12:21
*** jasondotstar has joined #openstack-cinder12:21
*** asselin has joined #openstack-cinder12:23
*** edmondsw has joined #openstack-cinder12:23
*** markvoelker has joined #openstack-cinder12:24
*** dims_ has joined #openstack-cinder12:25
openstackgerritDaniel Tadrzak proposed openstack/cinder: Cleanup for SnapshotObject
*** lpetrut has joined #openstack-cinder12:28
*** BharatK has quit IRC12:30
*** aagrawal has quit IRC12:32
*** markvoelker has quit IRC12:32
*** aagrawal has joined #openstack-cinder12:32
*** deepakcs has quit IRC12:34
*** akerr has joined #openstack-cinder12:35
*** ankit_ag has quit IRC12:35
*** ankit_ag has joined #openstack-cinder12:35
*** kjnelson has joined #openstack-cinder12:36
*** coolsvap|away is now known as coolsvap12:37
*** porrua has left #openstack-cinder12:38
*** markvoelker has joined #openstack-cinder12:39
*** piyanai has joined #openstack-cinder12:40
*** piyanai has quit IRC12:40
*** eharney has quit IRC12:41
*** markvoelker has quit IRC12:43
*** HoloIRCUser7 has joined #openstack-cinder12:44
*** lpetrut has quit IRC12:46
*** abhishekk has quit IRC12:46
*** HoloIRCUser3 has quit IRC12:47
*** piyanai has joined #openstack-cinder12:51
*** eharney has joined #openstack-cinder12:51
*** julim has joined #openstack-cinder12:52
*** julim has quit IRC12:53
*** julim has joined #openstack-cinder12:53
*** markvoelker has joined #openstack-cinder12:53
*** dims_ has quit IRC12:54
*** Tross1 has joined #openstack-cinder12:54
*** Tross has quit IRC12:55
*** nkrinner has quit IRC12:55
*** piyanai has quit IRC12:55
*** markvoelker has quit IRC12:58
*** ankit_ag has quit IRC12:59
*** dustins has joined #openstack-cinder12:59
*** aagrawal has quit IRC13:00
*** piyanai has joined #openstack-cinder13:00
*** aagrawal has joined #openstack-cinder13:00
*** piyanai has quit IRC13:00
*** dalgaaf has joined #openstack-cinder13:03
*** Maike has quit IRC13:05
*** strictlyb has quit IRC13:05
*** markvoelker has joined #openstack-cinder13:05
*** piyanai has joined #openstack-cinder13:06
*** markvoelker_ has joined #openstack-cinder13:07
*** markvoelker has quit IRC13:09
*** bnemec has joined #openstack-cinder13:11
*** jasondotstar has quit IRC13:11
*** p_deore has quit IRC13:13
*** xyang1 has joined #openstack-cinder13:18
*** mriedem has joined #openstack-cinder13:24
mriedemthingee: does os-brick 0.3.0 come out today?13:25
openstackgerritStefan Amann proposed openstack/os-brick: Perform port_rescan on s390x platforms
*** primechuck has joined #openstack-cinder13:26
openstackgerritAnton Arefiev proposed openstack/cinder: Add drivers list generator
*** dansmith is now known as superdan13:30
*** kambiz has quit IRC13:31
*** BharatK has joined #openstack-cinder13:32
BharatKDuncanT, Modified the patch as per your comments please review it.13:33
*** crose has joined #openstack-cinder13:34
*** markvoelker_ has quit IRC13:34
*** Yogi1 has joined #openstack-cinder13:35
*** wyatt88 has quit IRC13:35
*** gouthamr has joined #openstack-cinder13:36
mriedemis cinder have unit test failures with mock 1.1.4 this morning?13:37
mriedemironic and nova are13:37
*** merooney has joined #openstack-cinder13:37
openstackgerritAnton Arefiev proposed openstack/cinder: Add drivers list generator
mriedemwondering about capping mock in g-r13:37
*** anteaya has quit IRC13:38
*** jungleboyj_ has quit IRC13:40
*** kambiz has joined #openstack-cinder13:41
openstackgerritMichal Dulko proposed openstack/cinder-specs: Add Cinder Volume A/A HA spec
*** anteaya has joined #openstack-cinder13:42
*** hakimo has joined #openstack-cinder13:43
*** hakimo_ has quit IRC13:43
*** dims_ has joined #openstack-cinder13:45
*** diogogmt has quit IRC13:47
*** sgotliv has joined #openstack-cinder13:49
*** markvoelker has joined #openstack-cinder13:49
*** markvoelker has quit IRC13:54
*** aagrawal has quit IRC13:56
*** jdandrea has joined #openstack-cinder13:56
*** sgotliv has quit IRC13:57
*** merooney has quit IRC13:59
*** BharatK has left #openstack-cinder13:59
*** BharatK has quit IRC13:59
*** markvoelker has joined #openstack-cinder13:59
aarefievsmcginnis: hi, thanks for review drivers list generator13:59
aarefievsmcginnis: could you also take a look at  , please. it blocks bug fix
smcginnisaarefiev: Sure, I'll get them in my queue. Thanks for your work on that.14:02
*** aarefiev_phone has joined #openstack-cinder14:04
*** julim_ has joined #openstack-cinder14:05
*** eharney has quit IRC14:06
*** mdenny has joined #openstack-cinder14:06
*** markvoelker has quit IRC14:07
*** julim has quit IRC14:08
*** crose has quit IRC14:09
*** hakimo has quit IRC14:11
*** hakimo_ has joined #openstack-cinder14:11
*** abehl has quit IRC14:12
*** Tross1 has quit IRC14:14
*** markvoelker has joined #openstack-cinder14:14
*** Yogi1 has quit IRC14:15
*** markvoelker has quit IRC14:18
*** merooney has joined #openstack-cinder14:20
*** Yogi1 has joined #openstack-cinder14:21
e0nethingee, jgriffith: hi. could you please merge we've got broken unit tests for python-cinderclient in stable/kilo14:23
*** mylu has joined #openstack-cinder14:24
*** vlaza has quit IRC14:26
*** erlon has joined #openstack-cinder14:26
jgriffithe0ne: won't help... without a client release because the tests in gate don't use git, they use pip14:27
jgriffithe0ne: well, it'll help.. but you know what I mean14:28
jgriffithe0ne: We had planned to fix these via requirements update and pinning14:28
e0nejgriffith: thanks.14:28
jgriffithe0ne: which I thought was "done"14:28
*** markvoelker has joined #openstack-cinder14:28
e0nejgriffith: oh... it was my second question14:29
jgriffithe0ne: The update finally hit cinder, but not minderclient it would seem14:29
jgriffithe0ne: I tried to submit it last friday, everybody let it sit, then conveniently said on Tuesday when the automated job hit "Why are you doing this, it's already here"  sigh14:30
e0nejgriffith: i'm wondering how it works:
*** akshai has joined #openstack-cinder14:30
e0nejgriffith: it does'nt update requirements to the latest stable!14:30
*** breitz has quit IRC14:30
jgriffithe0ne: we just need that to land and you should be good14:30
e0nejgriffith: we'd hit this issue today:(14:30
openstackLaunchpad bug 1475654 in python-cinderclient "incorrect verstion for hacking package on stable/kilo" [Undecided,New] - Assigned to Yuriy Nesenenko (ynesenenko)14:30
jgriffithDuncanT: don't yell at me for pushing that through14:31
*** eharney has joined #openstack-cinder14:32
e0nejgriffith: now, i'm confused more14:32
*** topshare has joined #openstack-cinder14:32
jgriffithe0ne: how come?14:32
*** topshare has quit IRC14:32
jgriffithe0ne: that patch from proposal bot will fix the issue14:32
e0nejgriffith: please, look at hacking version14:33
*** markvoelker has quit IRC14:33
e0nefrom the my bug's description: python-cinderclient should follow global requirements but no we have hacking>=0.10.0,<0.11 in [1] and hacking>=0.8.0,<0.9 in [2] in stable/kilo14:33
*** xyang has joined #openstack-cinder14:33
*** topshare has joined #openstack-cinder14:33
jgriffithe0ne: not sure I follow the concern?14:34
*** breitz has joined #openstack-cinder14:34
jgriffithe0ne: in stable kilo:
jgriffithignore master for a second14:34
jgriffiththe global req update just fixes the syntax on hacking, doesn't change versions14:35
e0nejgriffith: but global reqs have other version:
jgriffithe0ne: ahh14:35
jgriffithe0ne: well that update just hasn't made it's way through to us yet14:36
e0nejgriffith: and my patch fixes unit tests!14:36
e0nejgriffith: it's the second problem with stable/kilo:(14:36
*** piyanai has quit IRC14:36
e0nejgriffith: e.g.
jgriffithe0ne: right, but what I was getting at is I think the update of mock is going to fix the unit tests also14:37
*** tsekiyama has joined #openstack-cinder14:37
e0nejgriffith: oh.. now i understood it14:37
jgriffithe0ne: and it doesn't fix them anyway according to gerrit :)14:37
*** ronis_ has quit IRC14:38
jgriffithe0ne: wait.. never mind... you've got all kinds of crazy going on14:38
e0nejgriffith: :)14:38
jgriffithe0ne: my point was just that once this lands:, the issues you are addressing in your patch that changes up unit tests should be addressed14:39
e0nejgriffith: now i'm agree with you14:39
jgriffithe0ne: "resolved", "fixed", "no longer fail"14:39
*** harlowja_at_home has joined #openstack-cinder14:39
jgriffithe0ne: I was working on these last Friday14:39
jgriffithe0ne: there was a collective agreement among projects and the oslo folks that the best course for stable was to just pin the reqs, rather than update tests14:40
e0nejgriffith: it's a dependency hell14:40
jgriffithe0ne: it is!!!14:40
e0nejgriffith: so... what do you thinks about hacking version?14:41
jgriffithe0ne: I think everybody in OpenStack became "library happy" over the last couple years, and it's actually not a good thing14:41
jgriffithe0ne: so I'm not sure about that... it looks like Robert's update to that just landed on the 15'th14:41
jgriffithe0ne: oh.. wait, it's in the queue I think14:42
jgriffithhmm.. not that one :)14:42
e0nejgriffith: hacking (0.10.0) was landed on January, 7th14:43
e0neit's a crazy deps friday!14:43
*** mgarza_ has joined #openstack-cinder14:43
*** lcurtis has joined #openstack-cinder14:43
*** krtaylor has quit IRC14:45
e0nejgriffith: fyi. fix hacking
jgriffithe0ne: hmmm14:45
e0nejgriffith: the same issue is in ironic too14:45
jgriffithe0ne: frankly minderclient has become so F'd up I don't even know what to think about it or do any more14:45
jgriffithe0ne: cinderclient even14:45
*** hemna has joined #openstack-cinder14:46
e0nejgriffith: maybe we need to fix it in global reqs14:48
*** aarefiev_phone has quit IRC14:49
jgriffithe0ne: ?14:49
e0nejgriffith: i mean set older version of hacking for stable/kilo14:49
*** mtanino has joined #openstack-cinder14:49
*** markvoelker has joined #openstack-cinder14:50
*** hemna has quit IRC14:52
jgriffithe0ne: do you have contact with Yuriy?14:53
jgriffithe0ne: it's not quite right, but I don't want to wait until tomorrow to see an update and get things rolling14:53
jgriffithe0ne: happy to update myself, but don't want to do that if he's around and able to work on it himself14:54
*** piyanai has joined #openstack-cinder14:54
aarefievsmcginnis: thank you14:55
*** cbader_ has joined #openstack-cinder14:56
*** markvoelker has quit IRC14:57
*** markvoelker_ has joined #openstack-cinder14:57
*** markvoelker_ has quit IRC14:57
*** markvoelker has joined #openstack-cinder14:57
*** cbader_ has quit IRC14:57
*** krtaylor has joined #openstack-cinder14:57
openstackgerritSzymon Wróblewski proposed openstack/cinder: Tooz locks
*** jaypipes has joined #openstack-cinder15:01
xyangjgriffith, e0ne, DuncanT, smcginnis, tbarron: question regarding
jgriffithxyang: what's up?15:01
xyangI want to introduce a new status 'backing-up-in-use' for backing up an attached volume, any objections?15:01
xyangthe reason if that I need to know the original status when cleaning up failed backups if services restarts15:02
tbarronxyang: this is for the cleanup, right?15:02
jgriffithxyang: I suppose not, but it would've been cool if we did sub-states15:02
jgriffithxyang: I'm certainly not asking you to do that by the way :)15:02
geguileoxyang: I'm not too happy about it15:02
tbarronxyang: I think the cleanup is already unsafe if more than one backup process is running15:02
xyangjgriffith: we don't have that yet, right?  seems to be abandoned15:02
jgriffithxyang: yeah :(15:02
tbarronxyang: I'm not sure about how we should be doing cleanup15:03
tbarronxyang: the current way just looks for leftover junk and assumes that no-one else is running15:03
xyangthere is existing code that does clean up and assumes the orig status is available15:03
geguileotbarron: Because right now that's true15:03
xyangnow we allow in-use volume  to be backed up15:03
tbarronxyang: so I'm not objecting to what you are proposing.  I jst don't know the best way forward yet.15:03
geguileotbarron: We don't support Active-Active15:03
xyangI need to know the original status15:03
tbarrongeguileo: I understand.15:04
xyangit is not saved any where if the service restarts15:04
geguileoxyang: Why not store it in the temporary snapshot metadata?15:04
tbarrongeguileo: xyang: I'm just wondering about the best way forward.15:04
xyangit could fail before the temp snap is created15:04
geguileotbarron: Ok, I'm not happy about that new state   XD15:04
xyanggeguileo: if you have a better idea, I'm happy to learn it15:05
geguileoxyang: That way you don't need a new state15:05
jgriffithxyang: geguileo what if we just add a "previous-state" column to the DB15:05
tbarrongeguileo: I just don't want to say "yeah that's cool" in off-tne-cuff IRC15:05
jgriffithand don't mess with the craziness in the states so much15:05
geguileoxyang: Mmm, I just said my not so good idea15:05
xyangjgriffith: that was another way15:05
tbarronxyang: ^^^ not trying to hold you up, but I think it needs more pondering15:05
xyangjgriffith: do we want to introduce another column15:05
geguileoxyang: Why not store it in the temporary snapshot metadata?15:05
jgriffithtbarron: no pondering... and answer at this point15:05
tbarronjgriffith: :-)15:06
xyanggeguileo: the temp snap may not exist yet15:06
jgriffiths/and answer/just an answer/15:06
xyangif the temp snap exists, I know for sure that is for in-use15:06
jgriffithgeguileo: xyang the "previous state" idea has other uses, I'd prefer it I think15:06
geguileoxyang: You should create the temp snap in the API before you change the volume status15:06
xyanggeguileo: I don't think so15:07
xyangthat should be created later15:07
geguileoxyang: I mean the DB entry15:07
geguileoxyang: Not the actual snapshot15:07
xyangit could be either a temp snap or volume15:07
xyangdepends on the driver15:07
xyangfor LVM it is temp snap15:07
xyangfor, it is a volume15:07
jgriffithgeguileo: xyang yeah, so that's one thing for me... I'd use volume-meta and do the same thing, but to be clear some won't use snapshots for this15:08
xyangfor LVM we can read the local path of a snapshot15:08
xyangbut not for other drivers15:08
geguileoxyang: I'd have to think about it, but I'm not too keen on adding that new state you mentioned15:09
xyangeither a new state for in-use, or a new column,15:09
xyangsince we don't know whether it is a temp volume or temp snapshot in api, I don't think we can create that early15:10
geguileoNew column makes more sense to me, but I think we should give it a little thought15:10
geguileoAs not to make it too specific15:10
xyangif there's more use for the new column, I'm all for it15:11
geguileoAnd maybe give it more versability for other uses/cases15:11
geguileoI think we should give it a thought of possible uses15:11
geguileoI'll add it to my "I have to think about this" list  :-)15:11
jgriffithgeguileo: what's specific about "previous_state" ?15:12
jgriffithgeguileo: I don't know how you get much "less" specific than that?15:12
tbarronplaying my hand, I want to re-think the way we cleanup failed backups anyways since it hampers backup scale out15:12
jgriffithgeguileo: I don't know what XD means :)15:13
* jgriffith is lame15:13
xyangmay be we just need to pick a name that everyone likes15:13
xyangvoting time:)15:13
geguileoIt's a smily for laughing15:13
geguileoLike lol15:13
tbarronso I'm reluctant to drive new state or even new column for that problem alone as currently implemented15:13
jgriffithgeguileo: Ohhh!!  :)15:13
jgriffithgeguileo: thanks!!15:13
jgriffithNow I have a new shorthand I can use... I like it15:13
xyangtbarron: I don't want to go that route either15:13
xyangtbarron: just don't know what is the alternative15:14
tbarronthis is hilarious15:14
e0nejgriffith: was away. thanks for review15:14
xyangthat's why I brought it up here15:14
geguileoxyang: What I meant was that the new column proposal is ok15:14
tbarronxyang: me either, but I want to get you smart folks thinking about it15:14
geguileoBut maybe we want a list of states in there15:14
tbarrongeguileo: +115:14
jgriffithxyang: tbarron how about I just submit a patch to add the column right now... then xyang can rebase against it :)15:14
geguileoOr maybe something different15:14
jgriffitherrr... XD15:14
geguileoThat's why I mean we should think about it15:15
jgriffithgeguileo: so I've proposed that before FWIW15:15
tbarronjgriffith: it may well be the right thing to do, but if so, for more than just that one problem15:15
jgriffithgeguileo: it didn't go over well, and frankly that's how we ended up with tflow15:15
xyangjgriffith: fine with me.  what does your "errr" mean?15:15
jgriffithgeguileo: I've always though a simple queue impl would be MUCH better for state management than anything we do today15:15
jgriffithbut that's a whole different bike-shedding event XD15:15
geguileojgriffith: A queue of jobs you mean?15:16
jgriffithgeguileo: That was one part of the proposal... and a stack of states as an object progresses through it's jobs15:16
geguileoxyang: I think we can go ahead with the new column for now and if I suddenly think that we are missing something I'll ping you15:17
jgriffithbut I digress... xyang is trying to finish here patch so I don't want to derail15:17
jgriffithgeguileo: we can change it to a list as well later I think without much trouble15:17
jgriffithgeguileo: if that proves to have benefit (which I think it might)15:17
geguileojgriffith: The list thing was just an example15:17
jgriffithgeguileo: well, I know... but I liked it because it's something I've played with before15:18
jgriffithgeguileo: I wrote a prototype that stored state changes for the cycle of an operation in order in DB.. then I could do unwinding, recovery etc15:18
jgriffithgeguileo: but it was less interesting that I thought it would be because even though I had all the info on the transition states, the actual unwinding and retry stuff was such a PITA it wasn't worth it15:19
geguileojgriffith: That's sounds like an interesting idea, and it could also serve as traceability of which user did what15:19
jgriffithI'm of the opinion that you should instead 'fail fast' and 'start over'15:19
jgriffithgeguileo: yeah, I didn't think of the other more useful bits until you guys started talking about it here15:20
geguileojgriffith: Yep, it's usually better to just start from scratch when you fail15:20
*** dannywilson has joined #openstack-cinder15:20
geguileojgriffith: Than actually trying to figure out in which specific state you are and what you need to do to fix it15:20
jgriffithgeguileo: especially in a Cloud... but sadly most people developing against OpenStack these days don't seem to have a "cloudy" mentality.15:20
geguileoAnyway, we digress, the column idea seem fine for now.   :-)15:21
xyangjgriffith: so are you going to add the column or I add the column in my patch?15:21
jgriffithgeguileo: I'm also a little tired of hearing that "things always fail" because in reality, most OpenStack deployments I've worked with are MUCH more robust than people would lead you to believe15:21
jgriffithxyang: geguileo I'll let you two decide and delegate to me if you'd like :)15:21
tbarronI'd like to see the column patch and understand it better15:22
*** diogogmt has joined #openstack-cinder15:22
xyangtbarron: you want to see a separate patch for column?15:22
geguileojgriffith: That's true, but we still have more failures than I would like to have  :-)15:22
tbarronxyang: that is up to you and john15:22
geguileoBy the way, anybody knows what is going on with the gates?15:22
tbarronxyang: jgriffith: I just want to see code to understand better15:23
jgriffithgeguileo: indeed, but most aren't doing anything to address them :(15:23
xyangtbarron: jgriffith says it is up to geguileo and me:)  So I'll add it to my patch15:23
jgriffithgeguileo: don't take that wrong... it's certainly not directed at you... and YES it's noted the improvements you've added15:23
jgriffithxyang: cool!15:23
*** coolsvap is now known as coolsvap|away15:23
tbarronxyang: ++15:23
geguileoxyang: Ok15:23
jgriffithgeguileo: you've found and fixed a good deal of issues15:24
geguileojgriffith: Don't worry, I took your statement as it is, a generalization which in some part also relates to me  :-)15:24
*** pots_ has joined #openstack-cinder15:24
xyangname for the column again?  orig_status, previous_status?15:24
geguileoxyang: +1 to previous_status15:24
jgriffithgeguileo: an to myself as well15:25
xyangany other votes?15:25
geguileojgriffith: xyang Any idea what is going on with the 2 broken gates?  (python27 and python34)15:25
jgriffithgeguileo: example?15:26
jgriffithgeguileo: afraid I'm not as involved in monitoring things as of late15:26
geguileojgriffith: I've reviewed now like 5 patches with the same gate issue15:26
geguileojgriffith: Oh, I'm not monitoring per se15:26
xyangwhat is the error15:26
e0nejgriffith: please, post a link to review request15:27
jgriffithgeguileo: well... now, but you've "noticed" them, I haven't really looked much :(15:27
geguileoThose are the 2 erros I've seen repeatedly in the last 5 reviews I've done15:27
*** jaypipes is now known as blockedpipes15:27
jgriffithawww rap15:27
jgriffithcrap even15:27
geguileoAnyway, if you are asking me about the error it means you don't know about it15:28
*** annashen has joined #openstack-cinder15:28
geguileoAnd probably nobody is working on it yet15:28
*** kjelly_ has quit IRC15:28
jgriffithgeguileo: Oh.. I might know... just a sec15:28
openstackgerritJohn Griffith proposed openstack/cinder: Backups: allow name to be specified during restore
Swansonxyang: can you take a look at this one again?  I coded it in a way that works but perhaps that is an unexpected use case?15:29
jgriffithgeguileo: see what that does for us15:29
geguileojgriffith: Ok, I'll monitor that one and see how it goes15:29
geguileojgriffith: Thanks15:30
*** pots has joined #openstack-cinder15:30
xyangSwanson: sure15:30
jgriffithgeguileo: I suspect they just need rebase.  Given it was started in early June, I've noticed most folks don't rebase as they go along.  Just a hunch, without looking closely at the logs and such15:30
openstackgerritKyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient
*** daneyon has joined #openstack-cinder15:31
jgriffithgeguileo: but given I've merged at least 3 changes in the last couple weeks that break the world seems like a good place to start :)15:31
geguileojgriffith: I think some of the patches had been rebased15:31
geguileojgriffith: XD15:31
jgriffithgeguileo: yeah, but I like to do that first... and at least eliminate it15:31
geguileojgriffith: Yes, and let's hope it's just that15:31
*** jdurgin has joined #openstack-cinder15:31
jgriffithgeguileo: indeed, those are nice ones to deal with :)15:32
*** dannywilson has quit IRC15:34
*** annashen has quit IRC15:35
xyangSwanson: so I can see how you get cgsnapshot_id in snapshot, but it is still different from snapshot_id.  you always has a snapshot_id15:36
Swansonxyang: yes, but if the snapshot was created via a cgsnapshot that snapshot id is meaningless to me.  The only way I can find the snapshot in question is to look for the cgsnapshot id.15:37
*** Tross has joined #openstack-cinder15:38
*** mylu has quit IRC15:38
xyangSwanson: so you are not associating snapshot_id with anything in your backend if it is part of a cgsnapshot?15:38
*** mylu has joined #openstack-cinder15:39
xyangSwanson: how do you know which snapshot to delete then?  you still need to do that15:39
xyangSwanson: do you mean that is only an entry in cinder db15:39
Swansonxyang:  Via cgsnapshot_delete or snapshot_delete?15:39
*** earlephilhower has joined #openstack-cinder15:40
xyangSwanson: when you do cgsnapshot_delete, you still need to call snapshot_delete for each one,  so sounds like you are saying that is just db entry15:40
Swansonxyang: When I create a consistency group snapshot on my backend I only have 1 id to work with.  It is associated as a group AND with each volume.15:40
xyangSwanson: so you won't be able to create_volume_from_snapshot if that snapshot is part of a cgsnapshot?15:41
openstackgerritSzymon Wróblewski proposed openstack/cinder: POC: Tooz locks demo
Swansonxyang: The  cgsnapshot is wiped out as a group and then I roll through the db marking them deleted.15:41
theanalystjgriffith: hi, you commented on saying patchset 5 was rebased .. what does it exactly mean in the context?15:41
*** sgotliv has joined #openstack-cinder15:42
Swansonxyang: I can create_volume_from_snapshot if I have the cgsnapshot_id.15:42
*** yuriy_n17 has quit IRC15:42
*** mylu has quit IRC15:42
smcginnisSwanson: But then which snapshot?15:42
jgriffiththeanalyst: that's an auto generated comment, noting that "I" triggered a rebase against it15:42
xyangSwanson: I don't see how that works, you don't have an independent entry in you backend15:42
*** sgotliv has quit IRC15:42
jgriffiththeanalyst: rebasing means updating the patch with things that have changed in the master branch since you started working on it15:43
theanalystjgriffith: ah ok!15:43
*** sgotliv has joined #openstack-cinder15:43
xyangSwanson: seems to me you need to block that case.15:43
jgriffiththeanalyst: it pulls in all of the changes "around" you while preserving your work15:43
*** belmoreira has quit IRC15:43
Swansonxyang: I have the source volume and the snapshot id.  I have everything I need.15:43
theanalystjgriffith: yep I know what a git-rebase means.. use it too often.. I thought that comment is again some sort of keyword like recheck etc :)15:43
jgriffiththeanalyst: ahh.. ok :)15:43
xyangSwanson: but if your cgsnapshot goes away, what happens to that volume?15:44
Swansonxyang: Depends on how they delete it.  Either I delete the snapshots in the db or I delete the volumes.15:44
smcginnisSwanson: So you're saying with the cg ID you can get the set of snaps, then from the source vol you can determine which on is the desired snap.15:44
jgriffiththeanalyst: didn't know if you've rebased that one lately, but given last update was on the 1st I knew it needed the requirements update at least, so I went ahead and did a rebase on it.15:45
Swansonxyang: Actually the reverse.  I look for the volume and then look for the snapshot labeled with the cgsnapshot id.15:45
*** mylu has joined #openstack-cinder15:45
theanalystjgriffith: np, while we're at it if you find the time, could you please review the bp associated iwth the patch15:45
tbarrontheanalyst: well played :)15:46
* jgriffith always says yes, but lately never actually does it :(15:46
theanalysttbarron: heh15:46
Swansonsmcginnis: see comment inappropriately marked xyang ^^15:46
*** mylu has quit IRC15:47
smcginnisSwanson: Got it. At any rate you are able to isolate the snap you need. xyang. what is the concern with delete?15:47
theanalystsorry I was supposed to fish out the actual link
tbarrontheanalyst: I think your patch is actually useful (can't say the same for lots of the additional knobs patches I see)15:47
*** mylu has joined #openstack-cinder15:48
theanalysttbarron: thank you15:48
*** garthb has joined #openstack-cinder15:48
*** chlong has quit IRC15:49
openstackgerritj_king proposed openstack/cinder: Add get_trust_context method to BackupManager
*** garthb_ has joined #openstack-cinder15:49
*** garthb_ has quit IRC15:49
jgriffiththeanalyst: so I've been asked to NOT approve or target BP's15:50
jgriffiththeanalyst: but FWIW I don't have any objection to your BP15:50
theanalysttbarron: I know you had some comments on my other force-backup patch.. which xing yang is currently implementing15:50
jgriffiththeanalyst: you'll have to work with thingee on approval/targeting15:51
theanalystjgriffith: ah ok15:51
*** earlephilhower has quit IRC15:51
theanalystI'm kind of a bit new to the openstack ecosystem, but can this BP still be in target for the next release or have I missed the deadline already15:52
theanalystjgriffith: ^^15:52
jgriffiththeanalyst: I'm not aware of any deadlines for this type of work.  You should be fine.15:53
theanalystjgriffith: ty15:54
tbarrontheanalyst: on your force-backup stuff, my point was only that it should be co-ordinated with xing's initiative for non-disruptive backup15:54
tbarrontheanalyst: seemed to me there was overlap15:55
*** earlephilhower has joined #openstack-cinder15:55
xyangsmcginnis: sorry, I'll get to you later.  in a meeting now15:55
*** hemnafk is now known as hemna15:55
theanalysttbarron: yeah just caught up with that.. seems xing's patch does account for force flag as well.15:56
smcginnisxyang: No problem, thanks!15:56
*** afazekas has quit IRC15:58
*** breitz has quit IRC15:59
Swansonxyang: thanks.15:59
*** vilobhmm has joined #openstack-cinder16:00
*** jordan__ has quit IRC16:00
*** leeantho has joined #openstack-cinder16:01
*** Apoorva has joined #openstack-cinder16:06
openstackgerritAbhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes.
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
*** jdurgin has quit IRC16:12
*** vokt has joined #openstack-cinder16:14
*** annashen has joined #openstack-cinder16:16
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements
*** harlowja_at_home has quit IRC16:18
anteayathingee: hey there16:23
thingeeanteaya: hi16:23
*** _cjones_ has joined #openstack-cinder16:23
anteayathingee: so I believe it is erlon's system that is currently disabled that he is trying to get enabled16:23
anteayathingee: have you an opinion?16:24
thingeeanteaya: yeah I'm fine with it. We'll monitoring it today and give notice if there's still problems.16:24
thingeeanteaya: looks like some of the issues were addressed in the mailing list16:24
anteayathingee: very good, thank you16:24
thingeeanteaya: thank you16:24
*** asselin__ has joined #openstack-cinder16:26
mtaninothingee: thank you!16:26
*** asselin has quit IRC16:27
theanalystthingee: could you please take a look at the bp
openstackgerritSean McGinnis proposed openstack/cinder: Add abandon script to clean up orphaned reviews
smcginnisthingee: ^^ Would love your thoughts on that.16:30
openstackgerritTomoki Sekiyama proposed openstack/cinder: Efficient image download and upload for Glance Cinder backend
*** jistr has quit IRC16:31
*** ashuk has joined #openstack-cinder16:32
*** asselin__ has quit IRC16:34
*** annashen has quit IRC16:36
*** aagrawal has joined #openstack-cinder16:36
*** dims_ has quit IRC16:37
*** piyanai has quit IRC16:38
*** Yogi1 has quit IRC16:38
*** annashen has joined #openstack-cinder16:40
*** sgotliv has quit IRC16:41
*** HoloIRCUser4 has joined #openstack-cinder16:43
*** HoloIRCUser7 has quit IRC16:43
thingeejgriffith: you mind addressing the comments here and I'll push it through?
*** HoloIRCUser4 has quit IRC16:44
potshi, new 3rd party tester here.  can anyone tell me if rasselin's os-ext-testing scripts are still the best/easiest way to get started?  i'm having all kinds of problems getting those scripts to work.16:46
potsin particular the script doesn't seem to be installing jenkins and i'm not sure how it's supposed to get installed.16:47
aarefievthingee: hi, have a min? I have question about
thingeeaarefiev: about to take off..I'll bbl16:47
* thingee take off on a plane16:47
*** s0nea has quit IRC16:48
*** rushil has joined #openstack-cinder16:48
*** s0nea has joined #openstack-cinder16:50
*** Apoorva has quit IRC16:52
earlephilhowerpots:  I set up our CI maybe 1.5 months ago, but haven't had to go back to rasselin's original archive since.   What exactly is failing on install_slave?16:54
* earlephilhower checks my setup notes...16:54
*** mylu has quit IRC16:54
earlephilhowerpots, are you trying to use static build hosts?  That's a no-no...never works AFACIT16:54
potsi'm trying to install on a ubuntu cloud image.  the main problem is that it's not installing jenkins and i can't find where it's actually supposed to be installed.16:55
potsi'm new to puppet also which doesn't help.16:55
earlephilhowerJoin the club.  Puppet, ansible, zuul, and 10 other tools needed...Jenkins installed on the master, not part of install_slace (that's only for static slaves).16:56
earlephilhowerPuppet does the jenkins install on the master only.  Dynamic slaves get injected by nodepool via SSH PSK16:57
potsi ran across a bunch of minor issues also: references to files in /etc/project-config that seemed like they should be to /root/project-config, cgconfig.conf was broken for ubuntu, puppet was not set to start, etc.16:57
*** sghanekar_ has joined #openstack-cinder16:57
potsdo i need to use nodepool?  i'd like to keep it as simple as possible.16:57
*** piyanai has joined #openstack-cinder16:58
*** mylu has joined #openstack-cinder16:58
earlephilhowerNodepool is what spins up a new clean VM for jenkins after each build.  W/o it you'll have to make static slaves work.  It wasn't working before.16:58
earlephilhowerNodepool is quite simple, actually.  layout.yaml, give it an OS account to spin up VMs, and it takes care of the rest...16:59
earlephilhowerIt also makes dynamic slaves, from scratch, starting w/the ubuntu cloud VM image16:59
potswas hoping not to spin up VMs to run the slave, i'm running all this under VMware Workstation while I wait for hardware16:59
earlephilhowerSo never need to look at install_slave16:59
earlephilhowerCan you set up an ubu openstack cloud under VMW workstation?  Things would then be much easier, I think.17:00
potsmaybe, i'm just afraid it'll break at the last step.17:01
*** vilobhmm has quit IRC17:01
potsinstall_slave seems to be setting up nodepool, so i had assumed that nodepool could just manage static that not the case?17:02
*** Tross has quit IRC17:02
earlephilhowerI'll have to check on that, because I don't remember seeing install_slave doing any of that....AFAIK nodepool runs on jenkins host only and really is only useful for creating updated/cached golden images daily and spinning up/down clean VMs for each job17:03
earlephilhowerLet me check github, see if things changed in past 6 wks...17:03
*** jungleboyj_ has joined #openstack-cinder17:03
*** patrickeast has joined #openstack-cinder17:04
*** jungleboyj_ is now known as jungleboyj17:04
earlephilhowerCrud...maybe I don't know what I'm talking about anymore... "NEW 7/1/2015: This repo is being migrated to use project-config and puppet-openstackci"17:04
tsekiyamapatrickeast: I've updated the glance cinder backend patch to address your comments. Please check it if you have a chance.
*** ronis has joined #openstack-cinder17:05
patrickeasttsekiyama: awesome, will do17:05
tsekiyamapatrickeast: thanks!17:05
potsi'm not sure what that means--i'm drinking from a firehose trying to come up to speed17:05
earlephilhowerIt gets better the second time around.  Unfortunately I've not looked at project-config,  But for your sanity I'd beg, borrow, or steal an OS cloud and try to get dynamic nodes up.  The script may not have been touched to work w/the OS infra based changes they're doing.17:08
*** mylu has quit IRC17:10
earlephilhowerHigh level is you've got a jenkins VM as the master controller.  Same VM runs Zuul to hook in the OS GIT review system.  When a new patch comes in, Zuul tells Jenkins which connectes to a running VM and does the tempest.  When completed, the VM is shutdown by Jenkins/Nodepool(also running on Jenkins master) and another one is spun up.17:10
earlephilhowerThere are assumptions that the VM JEnkins runs on is "clean" and the std. scritps modify system files which cause things to break with static servers.  You can fix, of course, with the source.17:11
earlephilhowerBut it seems like a real PITA compared to going with the flow.17:12
earlephilhowerDo you have a Master VM running yet, and what services are up?17:12
patrickeastpots: oo setting up a ci system?17:12
patrickeastdefinitely avoid static slaves if you can17:12
*** topshare has quit IRC17:12
potsyes, i'm trying.  i was hoping to use static slaves so i could integrate with our internal CI resources.17:13
asselin_pots nodepool doesn't manage static slaves, it manages single-use slaves17:13
potshi asselin, do you concur that static slaves don't work with the current scripts?17:13
asselin_pots yes, statis slaves don't work, unless you e.g. revert to a snapshot after each test17:14
asselin_i remember someone doing something like that17:14
*** piyanai has quit IRC17:14
*** Apoorva has joined #openstack-cinder17:14
*** mylu has joined #openstack-cinder17:15
patrickeastyou can clean them up to mostly work (i did that for a few months) but its not perfect17:15
patrickeasti had some scripts to uninstall a bunch of stuff, clean out various configs and files left behind, etc17:16
potsramy, do you think that your current os-ext-testing instructions will still work if i use nodepool & single-use slaves?17:16
patrickeasti still found myself having to manually intervene maybe once a week to get it working again17:16
patrickeastpots: i dont think anyone has tried them in a while17:16
asselin_pots I never got static slaves working. I don't recommend them at all for cinder dsvm testing17:17
*** annashen has quit IRC17:17
*** thangp has joined #openstack-cinder17:18
asselin_they're really to do less intrusive taskss, such as publish to pypi, or run unit tests that are contained within a sandbox virtualenv17:18
patrickeastasselin_: might be useful to remove that info from or put a big warning label on it17:18
potsso i should bring up an openstack cloud to run the slaves?  or can i just use kvm on the master node?17:19
asselin_patrickeast, good idea17:19
asselin_pots, setup a small cloud17:19
potsi assume nodepool is openstack-specific?17:19
patrickeastpots: you would need an openstack cloud, but a very simple one, you can do an all-in-one kind of thing17:19
*** mylu has quit IRC17:19
asselin_pots, you can even use devstack17:20
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: FC discover existing devices for removal
patrickeastpots: i use RDO to setup my nodes, its relatively easy to get going and handles restarts and stuff better than devstack, but you can’t beat the simplicity of devstack17:20
*** mylu has joined #openstack-cinder17:21
*** piyanai has joined #openstack-cinder17:22
potsok, i'll see if i can get that going.  thanks all for your help!17:23
*** daneyon has quit IRC17:23
earlephilhowerGood luck, pots.  You'll need it. :)17:23
thingeeaarefiev: back, looking17:24
potsthere goes the weekend :(17:24
patrickeastpots: also, if no one has pointed you to them yet, there is help available for this
patrickeasta couple times a week there are meetings full of other folks who have set them up and can answer questions17:25
potsyep, it's in my calendar.  i'll have a lot of questions on Monday...17:25
asselin_pots, also a small warning, there's lots of refactoring going on on my repo coupled with upstream changes. If you run into issues, ping me or others around.17:26
*** harlowja has quit IRC17:26
*** harlowja has joined #openstack-cinder17:26
potswill do--thank you all for your help!17:26
asselin_pots and take a look at these changes, especially recent merges:,n,z17:26
*** ashuk has quit IRC17:26
*** annashen has joined #openstack-cinder17:27
*** mylu has quit IRC17:27
*** mylu has joined #openstack-cinder17:27
*** piyanai has quit IRC17:27
*** mylu has quit IRC17:28
thingeeaarefiev: responded17:30
*** mylu has joined #openstack-cinder17:30
*** Tross has joined #openstack-cinder17:30
earlephilhowerthingee, hemna:  Howdy.  Silly question, but I see you updated os-brick changelog to 0.3.0 but didn't add a tag in git.  Any ETA for the new tag?17:32
hemnaearlephilhower, I don't publish os-brick.  only the PTL can.17:32
thingeeearlephilhower: I'm working it out right now with dhellmann17:33
earlephilhowerCool, thanks.17:33
* earlephilhower reads up on what PTL is...not the televangelists I imagine... :)17:33
*** Yogi1 has joined #openstack-cinder17:33
thingeeearlephilhower: I put in a request earlier this week to the release folks, but this sort of thing is not something I can just do anymore unfortunately17:33
openstackgerritDoug Hellmann proposed openstack/os-brick: Use pbr's automatically generated changelog
earlephilhowerthingee, no worries.  Appreciate the info.17:34
*** jdurgin has joined #openstack-cinder17:36
thingeeearlephilhower: at this point it'll probably be monday since we don't like to do releases on friday's17:39
earlephilhowerthingee, With the new tag, would the g-r also be updated as part of the PTL's process?17:41
*** e0ne has quit IRC17:44
aarefievthingee: about your answer, you are looking at problem from backend side, I'll try to see if problem in messaging, when service status is up(backend is fine), but services cant communicate, and it is difficult to see it17:44
*** boris-42 has joined #openstack-cinder17:44
*** vilobhmm has joined #openstack-cinder17:46
*** mylu has quit IRC17:47
thingeeaarefiev: today services communicate through cinder-manager service list17:47
thingeeaarefiev: I might be missing what you're trying to accomplish with this17:48
*** shyama_ has joined #openstack-cinder17:49
jungleboyjAnyone else seeing unit tests fail in test_image_utils.TestFetch.test_defaults ?17:49
jungleboyjI am seeing on our internal code and on Master which has me baffled.17:50
*** shyama has quit IRC17:50
*** shyama_ is now known as shyama17:50
aarefievthingee: I try to say that it is possible that backend is fine, but messaging has problems, and I'd like to see it with this cmd17:50
*** Tross has quit IRC17:50
thingeeaarefiev: can you define messaging here?17:51
thingeecurrently it's a face smiling or a frown face17:51
aarefievthingee: I mean rpc17:51
aarefieve.g. rabbitmq17:51
thingeeaarefiev: you want to know from cinder if rabbit is running correctly?17:52
aarefievthingee: I want to now if problem in it, if no continue looking the problem17:53
thingeeaarefiev: if there is a problem with rabbit, there's going to be a problem with start cinder-scheduler or cinder-vol17:53
smcginnisjungleboyj: Yeah, something is up. Not sure what though. Haven't had time to dig in.17:54
aarefievthingee: I saw cases when connection to rpc is fine, but messages can be received to services17:55
thingeeaarefiev: and why is that?17:56
*** mylu has joined #openstack-cinder17:56
aarefievthingee: there a lot of problems with rabbit, i don't know them, I just want to find out root cause of problem in cinder17:57
aarefievthingee: as I can see some of them is already fixed, but not all17:58
thingeeaarefiev: I would like to understand that better first. Otherwise, there's not really a way to identify the problem to communicate it.17:58
jungleboyjsmcginnis: Ok, so not just me seeing it.  That was the first thing I wanted to.17:59
thingeeaarefiev: regardless, based on the vague information given here, it seems like this would a problem for all than just Cinder.17:59
aarefiev thingee: I just want identify problem in cinder or backend or in rpc17:59
*** piyanai has joined #openstack-cinder18:00
*** mylu has quit IRC18:01
thingeeaarefiev: I definitely welcome fixing issues of cinder + rabbit. But 1) I'm not even aware of this, yet. 2) The information you're giving me is not complete for me to understand what the problem is exactly. 3) your proposal is about message on this problem, but it seems like we don't know what the problem is exactly to be able to identify and communicate it.18:01
*** mylu has joined #openstack-cinder18:02
*** jungleboyj has quit IRC18:03
aarefiev thingee: but if problem in rabbit, I spent a lot of time to identify this, so I've add this tool to cinder manage, btw it could not 'one' problem18:03
aarefiev thingee: I don't want to fix it, I try to make troubleshooting easy18:04
thingeeaarefiev: how are you going to identify the problem if you don't know what it is?18:05
aarefievthingee: I mean to see among cinder, rabbit18:06
*** haomaiw__ has quit IRC18:06
*** shyama has quit IRC18:06
thingeeaarefiev: I'm sorry but that didn't make sense to me for my question.18:06
*** crose has joined #openstack-cinder18:07
aarefievthingee: if problem in rabbit has appeared, now it is difficult to see it18:07
aarefiev thingee: service is in status up18:08
aarefiev thingee: we can't see e.g. which volumes don't report stats18:09
*** vincent_hou has joined #openstack-cinder18:09
aarefiev thingee: or problem on node with one of scheduler18:09
*** dannywilson has joined #openstack-cinder18:10
*** ronis has quit IRC18:11
*** vincent_hou has quit IRC18:14
*** piyanai has quit IRC18:14
aarefievthingee: sorry, it's late, I have to go out, I'll try contact with you next week to continue discussion, ofcourse  if you have time; thank you for your time18:14
*** aarefiev has quit IRC18:16
*** jungleboyj has joined #openstack-cinder18:19
*** haomaiwang has joined #openstack-cinder18:23
*** delattec has joined #openstack-cinder18:24
*** raunak has joined #openstack-cinder18:25
*** cdelatte has quit IRC18:28
*** haomaiwang has quit IRC18:28
*** piyanai has joined #openstack-cinder18:28
*** e0ne has joined #openstack-cinder18:29
*** haomaiwa_ has joined #openstack-cinder18:31
*** markvoelker has quit IRC18:32
*** ronis has joined #openstack-cinder18:33
*** haomaiwa_ has quit IRC18:36
*** mdenny has quit IRC18:38
thingeesmcginnis: I think it's related to a dependency change somehow. I have an outdated venv that passes with the latest cinder18:39
thingeejungleboyj: ^18:39
*** lpetrut has joined #openstack-cinder18:40
*** haomaiwang has joined #openstack-cinder18:41
* thingee is that guy on the plane that will use wifi for getting new deps of cinder to run unit tests18:45
*** delattec has quit IRC18:46
thingeeugh about to land. I'll check on it again when I can18:46
thingeesmcginnis, jungleboyj thanks for the heads up18:46
smcginnisthingee: Safe travels.18:47
jungleboyjthingee: Thank you.  Safe trip.18:47
patrickeasttsekiyama: put a few more comments on the review18:51
patrickeasttsekiyama: if you haven’t started looking at it maybe today i’ll try and put some code in to handle creating the raw image-volume for any image format coming in18:52
patrickeastrework whats in my cache patchset to do that18:52
tsekiyamapatrickeast: Thank you, I'll check your comments18:52
*** haomaiwang has quit IRC18:53
*** delattec has joined #openstack-cinder18:54
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Graceful shutdown WSGI/RPC server
*** raunak has quit IRC18:55
*** haomaiwang has joined #openstack-cinder18:57
xyangsmcginnis, Swanson: I added comments in that patch.  I think it is fine, just need some clarification because it looks like you are mixing the two ids18:58
*** mylu has quit IRC18:58
*** mylu has joined #openstack-cinder18:58
Swansonxyang: Thanks!  Adding comments to the patch now.18:59
xyangSwanson: sounds good18:59
*** lpetrut has quit IRC19:00
*** chenying has joined #openstack-cinder19:00
*** mylu has quit IRC19:02
*** mdenny has joined #openstack-cinder19:02
jgriffithmgagne: you around?19:02
mgagnejgriffith: I am19:02
jgriffithmgagne: wanted to pick your brain on this bug:
openstackLaunchpad bug 1475285 in Cinder "Non-admin user can perform 'extra-specs-list'" [High,Confirmed]19:02
jgriffithmgagne: Looks like it was part of the private types work:
*** mylu has joined #openstack-cinder19:03
mgagnejgriffith: what can I do to help?19:03
jgriffithmgagne: but regardless, I was going to just add  a filter for non-admin and extra-specs...19:04
jgriffithmgagne: hmmm.. wait, that won't work anyway :(19:04
mgagnejgriffith: ok let me read the whole thing19:04
jgriffithmgagne: anyhow.. my question became... what a bout a mix of types?19:04
jgriffithmgagne: so an admin in one project who is not an admin in another requests extra-specs...19:04
jgriffithmgagne: the way we run today, we don't have a way of filtering that out, unless we check each type individually I suppose19:05
jgriffithmgagne: in a nut-shell the issue is that a non-admin can now do "cinder extra-specs-list"19:05
mgagnejgriffith: and you wonder if he should be able to list extra-specs from a private type?19:06
jgriffithmgagne: part of the issue IMO is that we are kinda lazy with extra_specs_list and just pull the info out of the type-list19:06
jgriffithmgagne: correct19:06
mgagnejgriffith: hmmm I haven't thought of that19:06
openstackgerritTom Barron proposed openstack/cinder-specs: Scaling backup service blueprint spec
jgriffithmgagne: so I can 'fix' this bug, but it sort of introduces another odd case that's questionable, so I thought I'd consult with you :)19:07
jgriffithmgagne: although I suppose it's fine if somebody is an "Admin" in any project then they have the ability to "see" extra-specs-list19:08
mgagnejgriffith: I think the reason why I haven't "thought" of that is that Nova doesn't allow you to list extra-specs of all flavors. there is no "flavor-key-list" command19:08
jgriffithmgagne: hehe :)  that's very true19:08
mgagneotherwise there should already condition in place to handle such case in db.api19:09
*** ndipanov has quit IRC19:09
mgagnecould it be reused for extra-specs-list? but then: what if an admin wishes to list ALL extra-specs, irregardless of ownership19:09
jgriffithhmm... good point19:10
mgagneso now we are back to that --all-tenants thing19:10
*** haomaiwang has quit IRC19:11
jgriffithmgagne: I think I can just compare the context to the returned type in volume_types:get_all19:12
mgagnein fact, in nova, it's --all to list private flavors. I don't see a flavor in cinder v2 cli19:12
*** haomaiwang has joined #openstack-cinder19:12
mgagneI thought I implemented it but meh19:12
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues
mgagneit got removed19:13
mgagnedue to bug #146210419:14
openstackbug 1462104 in Cinder "access-type-add doesn't take a volume type name" [Low,Fix released] - Assigned to Ed Balduf (ebalduf)19:14
jgriffithmgagne: sigh19:15
jgriffithmgagne: indded19:15
*** mylu has quit IRC19:17
jgriffithmgagne: well, thanks for the help :)19:17
jgriffithmgagne: I'll see what I can do to fix both issues19:17
mgagnethis could have been fixed by updating default value of is_public in list() while keeping support for --all19:17
*** dims_ has joined #openstack-cinder19:18
mgagneand using vtypes = cs.volume_types.list(is_public=True) when --all is not used19:18
mgagneor not19:18
*** haomaiwang has quit IRC19:18
mgagneI'm saying silly things19:18
jgriffithmgagne: :)19:19
SwansonWhat is the record for most items in 'check'?19:19
openstackgerritMitsuhiro Tanino proposed openstack/cinder: LVM: Support efficient data copy for LVM driver
*** afazekas has joined #openstack-cinder19:19
mgagneno, it fact, it would have worked since it would have kept the old behavior while fixing VolumeTypeAccessManager19:20
mgagneok, back to nova cells hell :-/19:20
jgriffithmgagne: oh dear.. sorry to hear that :(19:20
jgriffithK, I'll get lunch and then try that out, thanks for the help mgagne19:21
mgagneyw, bon appétit ;)19:21
*** mylu has joined #openstack-cinder19:21
*** dustins has quit IRC19:22
*** chenying has quit IRC19:22
*** mylu has quit IRC19:23
*** merooney has quit IRC19:23
*** edmondsw has quit IRC19:24
*** mylu has joined #openstack-cinder19:25
*** IlyaG has joined #openstack-cinder19:26
openstackgerritMitsuhiro Tanino proposed openstack/cinder: LVM: Support efficient data copy for LVM driver
*** mylu has quit IRC19:28
*** mylu has joined #openstack-cinder19:29
*** erlon has quit IRC19:30
*** dims__ has joined #openstack-cinder19:30
*** haomaiwang has joined #openstack-cinder19:32
*** dims_ has quit IRC19:34
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: FC discover existing devices for removal
*** akerr has quit IRC19:34
*** lpetrut has joined #openstack-cinder19:35
*** haomaiwang has quit IRC19:36
*** haomaiwa_ has joined #openstack-cinder19:37
*** mylu has quit IRC19:39
*** mylu has joined #openstack-cinder19:40
*** haomaiwang has joined #openstack-cinder19:40
*** haomaiwa_ has quit IRC19:41
*** nikeshm has joined #openstack-cinder19:43
*** mylu has quit IRC19:45
*** Lee1092 has quit IRC19:45
rhe00_thingee: ping19:47
*** haomaiwang has quit IRC19:47
*** ig0r_ has quit IRC19:47
*** mylu has joined #openstack-cinder19:51
*** haomaiwa_ has joined #openstack-cinder19:52
*** piyanai has quit IRC19:53
*** haomaiwa_ has quit IRC19:57
*** haomaiwang has joined #openstack-cinder20:03
openstackgerritJohn Griffith proposed openstack/cinder: Remove logging statements from migrations
*** akerr has joined #openstack-cinder20:03
*** haomaiwa_ has joined #openstack-cinder20:06
*** akerr_ has joined #openstack-cinder20:07
*** haomaiwang has quit IRC20:08
*** Yogi1 has quit IRC20:09
*** piyanai has joined #openstack-cinder20:10
*** akerr has quit IRC20:10
thingeerhe00_: hi20:12
*** delattec has quit IRC20:12
thingeemriedem: release will be monday20:13
thingeesmcginnis: yeah blowing away my venv allowed it to be replicated20:14
thingeemriedem: apparently people like to have fun on fridays and not have to fix things :)20:14
mriedemthingee: fwiw i don't think would have passed jenkins anyway20:14
mriedemthingee: we've been trying to fix nova's gate all day....20:14
lifelessand this is why ci pins are important :)20:15
*** haomaiwa_ has quit IRC20:16
*** jasondotstar has joined #openstack-cinder20:16
*** lpetrut has quit IRC20:18
thingeelifeless: I finally updated some of my openstack/requirement bump patches. dhellmann got me up to speed with upper constraints20:20
*** thangp has quit IRC20:21
*** haomaiwang has joined #openstack-cinder20:21
lifelessthingee: there's a patch there with full docs20:22
*** Apoorva_ has joined #openstack-cinder20:24
*** Apoorva has quit IRC20:24
*** raunak has joined #openstack-cinder20:26
*** haomaiwang has quit IRC20:26
*** haomaiwang has joined #openstack-cinder20:26
*** kjnelson has left #openstack-cinder20:27
*** dims__ has quit IRC20:34
*** Apoorva_ has quit IRC20:34
*** haomaiwang has quit IRC20:34
*** haomaiwang has joined #openstack-cinder20:35
*** piyanai has quit IRC20:36
*** jungleboyj has quit IRC20:40
*** mylu has quit IRC20:42
*** mylu has joined #openstack-cinder20:42
*** julim_ has quit IRC20:43
*** e0ne has quit IRC20:43
*** piyanai has joined #openstack-cinder20:44
rhe00_thingee: our CI is back up and stable (if I stop experimenting with the scripts to make it faster). However I am currently 90+ changes behind. What would you like to see before the change to add back the driver is accepted?20:44
openstackgerritMatthew Edmonds proposed openstack/cinder: move oslo.vmware to test-requirements
*** piyanai has quit IRC20:46
*** Apoorva has joined #openstack-cinder20:47
*** piyanai has joined #openstack-cinder20:48
*** haomaiwang has quit IRC20:48
*** haomaiwang has joined #openstack-cinder20:51
*** crose has quit IRC20:51
*** piyanai has quit IRC20:51
*** bnemec has quit IRC20:54
*** jwcroppe_ has joined #openstack-cinder20:54
*** jwcroppe has quit IRC20:56
*** haomaiwang has quit IRC20:56
*** eharney has quit IRC20:57
*** mylu has quit IRC20:57
openstackgerritMathieu Gagné proposed openstack/cinder: Add volume_attachment to volume usage notifications
*** edmondsw has joined #openstack-cinder20:59
*** Apoorva has quit IRC20:59
*** dims_ has joined #openstack-cinder21:00
*** Apoorva has joined #openstack-cinder21:00
*** alwaysatthenoc has joined #openstack-cinder21:01
*** jungleboyj has joined #openstack-cinder21:04
*** jasondotstar has quit IRC21:05
*** akerr_ is now known as akerr_away21:08
*** akerr_away is now known as akerr_21:08
*** akshai has quit IRC21:10
*** haomaiwang has joined #openstack-cinder21:11
*** gouthamr has quit IRC21:12
*** sghanekar_ has quit IRC21:14
*** jwcroppe has joined #openstack-cinder21:14
nikeshmlooks like gate is failing on gate-cinder-python27 and gate-cinder-python3421:15
*** jwcroppe_ has quit IRC21:16
*** sgotliv has joined #openstack-cinder21:17
*** jwcroppe_ has joined #openstack-cinder21:17
*** haomaiwang has quit IRC21:17
*** jwcroppe has quit IRC21:19
*** haomaiwa_ has joined #openstack-cinder21:21
*** jwcroppe has joined #openstack-cinder21:23
*** cbader has quit IRC21:23
*** raunak has quit IRC21:23
*** skylerberg has joined #openstack-cinder21:23
*** jwcroppe_ has quit IRC21:25
*** primechuck has quit IRC21:25
*** rushil has quit IRC21:26
*** akerr_ has quit IRC21:27
*** haomaiwang has joined #openstack-cinder21:27
*** jwcroppe_ has joined #openstack-cinder21:28
*** haomaiwa_ has quit IRC21:29
*** blockedpipes has quit IRC21:29
*** jwcroppe has quit IRC21:30
*** jasondotstar has joined #openstack-cinder21:31
*** haomaiwang has quit IRC21:32
*** sghanekar_ has joined #openstack-cinder21:32
*** ronis has quit IRC21:34
*** jwcroppe has joined #openstack-cinder21:35
*** jwcroppe_ has quit IRC21:37
*** divab has quit IRC21:38
*** alwaysatthenoc has quit IRC21:38
*** haomaiw__ has joined #openstack-cinder21:38
*** divab has joined #openstack-cinder21:39
*** divab has quit IRC21:40
*** divab has joined #openstack-cinder21:40
*** jwcroppe_ has joined #openstack-cinder21:40
*** jwcroppe has quit IRC21:42
*** jwcroppe has joined #openstack-cinder21:43
edmondswjungleboyj, see my response in
*** jwcroppe_ has quit IRC21:45
jungleboyjedmondsw: So, when the operator wants to use that driver they have to go install the requirement themselves?  I don't think that is the way we have been operating in the past.21:46
edmondswyes, they would need to go install that21:46
edmondswI think we've been trying to do this, albeit perhaps inconsistently21:47
*** haomaiw__ has quit IRC21:48
patrickeastthat matches up with some of the other drivers (like mine) that require extra libs21:48
jungleboyjpatrickeast: You are requiring the users to install additional packages?21:48
edmondswif you want an example from another project:
patrickeastjungleboyj: so someone who wants to use our driver needs to ‘pip install purestorage’ on their c-vol host before it will work21:50
jungleboyjOk.  Then I am still -1 on the patch.21:50
*** haomaiwa_ has joined #openstack-cinder21:50
jungleboyjUpdated the patch.  If you want to move this out the requirements file then we need to appropriately catch when the library isn't installed.21:52
edmondswpatrickeast, do we do that for pure?21:52
jungleboyjThat is what he just pasted above.21:52
patrickeastnope its all open source and available on pypi21:52
jungleboyjOther drivers do do that if they are expecting additonal code installed.21:52
patrickeastjust that we don’t want to force it on people not using our driver21:52
patrickeastand don’t want to duplicate the code in cinder21:53
edmondswok, sure... tx guys21:53
jungleboyjedmondsw: That makes more sense.  Then I would be more comfortable with it.21:54
edmondswyep, I'll make that change21:54
*** mriedem has quit IRC21:54
asselin_patrickeast, jungleboyj  It's better not adding it to the requirements21:56
asselin_we used to do that for 3par and lefthand, and it took a while to get them out21:56
jungleboyjYeah, I remember that.21:58
*** ganso_ has quit IRC21:58
*** raunak has joined #openstack-cinder21:59
openstackgerritTom Barron proposed openstack/cinder-specs: Scaling backup service blueprint spec
*** jungleboyj has quit IRC22:02
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues
*** piyanai has joined #openstack-cinder22:04
*** IlyaG has quit IRC22:07
openstackgerritMatthew Edmonds proposed openstack/cinder: move oslo.vmware to test-requirements
*** dalgaaf has quit IRC22:18
*** haomaiw__ has joined #openstack-cinder22:21
*** haomaiwa_ has quit IRC22:22
*** piyanai has quit IRC22:26
*** edmondsw has quit IRC22:28
*** asselin has joined #openstack-cinder22:30
*** IanGovett has quit IRC22:34
*** asselin has quit IRC22:35
*** sgotliv has quit IRC22:39
*** haomaiwa_ has joined #openstack-cinder22:42
*** haomaiw__ has quit IRC22:43
*** jasondotstar has quit IRC22:44
*** haomaiw__ has joined #openstack-cinder22:45
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: remove the iscsi rescan during disconnect
*** haomaiwa_ has quit IRC22:46
*** mdenny has quit IRC22:47
*** mgarza_ has quit IRC22:49
*** mylu has joined #openstack-cinder22:54
*** mriedem has joined #openstack-cinder22:57
*** haomaiw__ has quit IRC22:59
*** haomaiwa_ has joined #openstack-cinder22:59
jgriffithpatrickeast: asselin_ +1... I've always been a bit opposed to EVERYTHING in the world going into requirements file23:04
jgriffithhemna: surely remembers my whining about it :)23:04
*** haomaiwa_ has quit IRC23:06
*** annashen has quit IRC23:06
openstackgerritxing-yang proposed openstack/cinder: Non-disruptive backup
*** xyang has quit IRC23:07
jgriffithxyang1: Whhhaat?  You added a "prev state column"?23:08
jgriffithxyang1: Just kidding :)23:08
xyang1jgriffith: :)23:08
*** annashen has joined #openstack-cinder23:08
xyang1jgriffith: I added to volumes instead of backups23:09
xyang1jgriffith: Because it is volume status23:09
jgriffithxyang1: That where I would've put them as well23:09
jgriffithxyang1: or put "it"23:09
*** garthb has quit IRC23:09
jgriffithxyang1: logging in the migration file :(23:09
xyang1jgriffith: I wondered about it:)23:10
xyang1jgriffith: I thought jungleboyj_ will give -1 if i don't have it23:10
jgriffithxyang1: I'm sure he probably would23:10
*** mylu has quit IRC23:11
xyang1jgriffith: I can remove that and the try catch23:11
jgriffithxyang1: if something else comes up I would love that personally23:11
xyang1jgriffith: And reference your patch:)23:11
xyang1jgriffith: I can remove that, no problem23:12
jgriffithxyang1: so how would ....23:12
jgriffithxyang1: so in "def create_backup"23:13
*** haomaiwa_ has joined #openstack-cinder23:13
xyang1jgriffith: manager?23:13
jgriffithxyang1: yeah, I'll just comment in the review23:13
jgriffithbesides, as I walk through it I might be wrong :)23:14
xyang1jgriffith: I don't see your comments23:14
jgriffithxyang1: I'm not done yet :)23:14
xyang1jgriffith: Ok:)23:14
patrickeastxyang1: hey, for those temporary volumes/snapshots you could use the internal tenant :D23:14
xyang1patrickeast: Yes, is yours merged yet?23:15
patrickeastno :(23:15
* patrickeast finds link23:15
*** haomai___ has joined #openstack-cinder23:17
xyang1patrickeast: I'll take a look.  I have a couple of dependent patches myself, like to sort out other concerns first23:18
*** haomaiwa_ has quit IRC23:18
*** haomai___ has quit IRC23:23
*** haomaiwa_ has joined #openstack-cinder23:25
jgriffithxyang1: I made a number of comments23:27
jgriffithxyang1: overall I'm positive so gave it a +123:27
jgriffithxyang1: there are some things around safety in getting the status from the object, and exactly what we should set it to in case of failure.23:28
jgriffithxyang1: logs of course I already mentioned :)  And a long winded random babble of nonsense that may or may not mean anything to anybody23:28
xyang1jgriffith: Thanks! Let me read your comments:)23:30
*** hemna is now known as hemnafk23:31
*** haomaiw__ has joined #openstack-cinder23:32
xyang1jgriffith: The rollback part, it was like that before23:32
*** nikeshm has quit IRC23:32
*** lcurtis has quit IRC23:33
*** haomaiwa_ has quit IRC23:33
xyang1jgriffith: It sets back to available before, so I just added in-use for attached volumes23:33
*** sghanekar_ has quit IRC23:33
xyang1jgriffith: I guess this is because we didn't change the volume itself, just trying to create a backup from it23:34
openstackgerritTom Barron proposed openstack/cinder: POC for scalable backup
jgriffithxyang1: Oh... well I meant the new setting of "previous_state" to None23:35
jgriffithxyang1: Oh.. I see your point23:35
xyang1jgriffith: Oh, what should we set it?23:35
jgriffithxyang1: well.. I dunno :)23:36
jgriffithxyang1: but if we tried to do a backup of the volume and it failed, might be worth noting that... no?23:36
xyang1jgriffith: Trying to set it back to original state23:36
jgriffithxyang1: wel.. you're not though23:36
xyang1jgriffith: The backup itself is error23:36
jgriffithxyang1: so one problem with the previous_state stuff is that its just setting it to "None"23:36
jgriffithxyang1: it's not even setting ti to what it "was" at the beginning of the operation23:37
jgriffithsee what I mean?23:37
jgriffithpatrickeast: I had a though on your patch there23:37
xyang1jgriffith: In the beginning it is None, I set it to availabe or inuse before backup23:37
jgriffithpatrickeast: any interest/value in considering auto-creation of the internal tenant?23:38
*** sghanekar_ has joined #openstack-cinder23:38
xyang1jgriffith: So i can set status back to previous_status after backup23:38
xyang1jgriffith: So the previous state didn't have any t23:38
xyang1Anthing if we just created a volume23:39
jgriffithxyang1: You do "previous_state = volume['previous_state']" ....23:39
xyang1jgriffith: Let me check23:39
jgriffithxyang1: then you don't touch it until the exception block.. where you set it to None in the db.volume_update23:39
jgriffithxyang1: line #344 in backup.manager23:40
xyang1jgriffith: I see what you are saying:)23:41
*** vilobhmm has quit IRC23:41
xyang1jgriffith: I think I don't need that line23:41
patrickeastjgriffith: i was thinking about that, we could probably do it since it doesn’t really matter what the fields are as long as they are a string23:41
xyang1jgriffith: Hold on23:41
jgriffithpatrickeast: which part?23:42
jgriffithpatrickeast: the auto-generate thing?23:42
*** haomaiw__ has quit IRC23:42
jgriffithpatrickeast: what I'm saying is you don't even need it in the config23:42
patrickeastjgriffith: yep23:42
jgriffithpatrickeast: you just create one.. store it in the DB23:42
patrickeastjgriffith: ohh interesting23:42
jgriffithadmins can go to keystone and say "what's the internal-cinder tenant ID"23:42
jgriffithpatrickeast: even expose an admin call in Cinder to get it if you want23:43
xyang1jgriffith: Line 343 set status back to original status, line 344, previous status is no longer needed, so set to none23:43
jgriffithpatrickeast: but I don't see much reason for exposing "Internal cinder" details to anybody23:43
jgriffithxyang1: :)23:43
patrickeastjgriffith: so right now the internal tenant doesn’t actually have to be a keystone project or need a real user23:43
patrickeastjgriffith: i kind of like that functionality23:43
xyang1jgriffith: Sorry it is messy:)23:43
jgriffithbut what I'm saying is ; is it not needed?  Is there any reason we should not track what it's been through?23:43
jgriffithpatrickeast: just a though23:44
patrickeastjgriffith: yea i mean… i like both23:44
jgriffithpatrickeast: or even add it to setup/config in the Cinder install guide23:44
patrickeastjust not sure how to get there23:44
jgriffithpatrickeast: that's the EZ part :)23:44
jgriffithsetup/config... or an admin call out to Keystone (the second option is harder)23:44
patrickeastjgriffith: so one thing i was wondering is why can’t it just be the cinder service tenant?23:45
jgriffithpatrickeast: I think.. but doable23:45
patrickeastwe already have to create one for the service to use, right?23:45
jgriffithpatrickeast: I'd have to look at that... that actually might work23:45
jgriffithI just can remember if policy let's that tenant "do" anything23:45
jgriffithworth looking at23:45
patrickeastbut thats the nice part of it with how it works now… it doesn’t actually have to do anything other than fill in db columns in cinder23:46
xyang1jgriffith: Set volume['status'] to previous_status which is available or in use, volume['previous_status'] is no longer needed after backup is complete, so I set it to None23:46
patrickeastwe don’t need to be able to do anything outside of that23:46
*** haomaiwa_ has joined #openstack-cinder23:46
*** alwaysatthenoc has joined #openstack-cinder23:49
patrickeastjgriffith: so if we go down the auto create route, are you thinking we add a new table in the db to keep track of it? or is there somewhere already we can stash it?23:50
*** raunak has quit IRC23:50

Generated by 2.14.0 by Marius Gedminas - find it at!