Tuesday, 2015-05-12

*** dannywilson has quit IRC00:00
*** dims has joined #openstack-cinder00:00
*** ebalduf has quit IRC00:02
*** esker has joined #openstack-cinder00:03
*** leeantho has quit IRC00:03
*** MentalRay has quit IRC00:03
*** abhiram_moturi has quit IRC00:05
*** abhiram_moturi has joined #openstack-cinder00:06
*** esker has quit IRC00:10
*** esker has joined #openstack-cinder00:10
*** jdurgin has quit IRC00:11
*** esker has quit IRC00:14
*** IanGovett has joined #openstack-cinder00:14
*** abhiram_moturi has quit IRC00:16
*** abhiram_moturi has joined #openstack-cinder00:17
*** abhiram_moturi has quit IRC00:21
*** ebalduf has joined #openstack-cinder00:21
*** abhiram_moturi has joined #openstack-cinder00:21
*** IlyaG has quit IRC00:23
*** heyun has joined #openstack-cinder00:24
*** ebalduf has quit IRC00:26
*** tsekiyama has quit IRC00:27
*** jdurgin has joined #openstack-cinder00:27
*** abhiram_moturi has quit IRC00:32
*** abhiram_moturi has joined #openstack-cinder00:32
*** abhiram_moturi has quit IRC00:35
*** abhiram_moturi has joined #openstack-cinder00:35
*** abhiram_moturi has quit IRC00:37
*** abhiram_moturi has joined #openstack-cinder00:37
*** abhiram_moturi has quit IRC00:39
*** abhiram_moturi has joined #openstack-cinder00:40
*** abhiram_moturi has quit IRC00:42
*** abhiram_moturi has joined #openstack-cinder00:42
*** abhiram_moturi has quit IRC00:46
*** abhiram_moturi has joined #openstack-cinder00:47
*** abhiram_moturi has quit IRC00:49
*** abhiram_moturi has joined #openstack-cinder00:49
*** ociuhandu has quit IRC00:49
*** akerr has joined #openstack-cinder00:50
*** IanGovett has quit IRC00:50
*** akerr_ has joined #openstack-cinder00:51
*** diegows has quit IRC00:53
*** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block | https://wiki.openstack.org/wiki/Cinder"00:54
-openstackstatus- NOTICE: Gerrit has been downgraded to version 2.8 due to the issues observed today. Please report further problems in #openstack-infra.00:54
*** akerr has quit IRC00:55
*** tobe has joined #openstack-cinder00:58
*** takedakn has joined #openstack-cinder00:58
*** abhiram_moturi has quit IRC01:01
*** abhiram_moturi has joined #openstack-cinder01:02
*** abhiram_moturi has quit IRC01:06
*** abhiram_moturi has joined #openstack-cinder01:06
*** IanGovett has joined #openstack-cinder01:07
*** mtanino has quit IRC01:10
*** abhiram_moturi has quit IRC01:13
*** abhiram_moturi has joined #openstack-cinder01:13
openstackgerritJohn Griffith proposed openstack/cinder: Check volume_backend in retype  https://review.openstack.org/18205501:16
*** IlyaG has joined #openstack-cinder01:17
*** IlyaG has quit IRC01:17
*** IlyaG has joined #openstack-cinder01:17
openstackgerritPatrick East proposed openstack/cinder: Refactor PureISCSIDriver into base and iSCSI classes.  https://review.openstack.org/18213601:18
*** takedakn1 has joined #openstack-cinder01:20
*** takedakn has quit IRC01:21
*** IanGovett has quit IRC01:22
*** patrickeast has quit IRC01:22
*** abhiram_moturi has quit IRC01:25
*** abhiram_moturi has joined #openstack-cinder01:25
*** _cjones_ has quit IRC01:28
*** abhiram_moturi has quit IRC01:34
*** abhiram_moturi has joined #openstack-cinder01:34
openstackgerritLiu Xinguo proposed openstack/cinder: Huawei driver fix problems under multipath  https://review.openstack.org/15076701:35
*** Lee1092 has joined #openstack-cinder01:35
*** MentalRay has joined #openstack-cinder01:39
*** abhiram_moturi has quit IRC01:45
*** abhiram_moturi has joined #openstack-cinder01:45
*** Tross1 has joined #openstack-cinder01:46
*** takedakn1 has quit IRC01:47
openstackgerritwanghao proposed openstack/cinder: Notification with volume and snaphost metadata  https://review.openstack.org/18040001:49
*** Tross has quit IRC01:50
*** abhiram_moturi has quit IRC01:52
*** Yogi1 has joined #openstack-cinder01:52
*** abhiram_moturi has joined #openstack-cinder01:52
*** abhiram_moturi has quit IRC01:54
*** primechuck has quit IRC01:54
*** abhiram_moturi has joined #openstack-cinder01:54
*** primechuck has joined #openstack-cinder01:55
*** Yogi1 has quit IRC01:56
*** zhenguo has joined #openstack-cinder01:59
*** abhiram_moturi has quit IRC02:05
*** abhiram_moturi has joined #openstack-cinder02:05
*** Yogi1 has joined #openstack-cinder02:06
*** vilobhmm1 has quit IRC02:10
*** abhiram_moturi has quit IRC02:14
*** abhiram_moturi has joined #openstack-cinder02:15
*** abhiram_moturi has quit IRC02:19
*** abhiram_moturi has joined #openstack-cinder02:19
*** MentalRay has quit IRC02:21
*** Yogi1 has quit IRC02:25
*** abhiram_moturi has quit IRC02:30
*** abhiram_moturi has joined #openstack-cinder02:30
*** bkopilov has quit IRC02:31
*** abhiram_moturi has quit IRC02:39
*** abhiram_moturi has joined #openstack-cinder02:39
*** abhiram_moturi has quit IRC02:43
*** abhiram_moturi has joined #openstack-cinder02:44
*** Apoorva has quit IRC02:44
*** dims has quit IRC02:45
*** BharatK has quit IRC02:48
*** garthb has joined #openstack-cinder02:52
*** abhiram_moturi has quit IRC02:57
*** julim has joined #openstack-cinder02:57
*** abhiram_moturi has joined #openstack-cinder02:58
*** davechen has joined #openstack-cinder03:00
*** MentalRay has joined #openstack-cinder03:06
*** davechen1 has joined #openstack-cinder03:07
*** davechen has quit IRC03:08
*** Apoorva has joined #openstack-cinder03:08
*** Zhongjun has joined #openstack-cinder03:09
*** davechen has joined #openstack-cinder03:12
*** davechen1 has quit IRC03:14
*** Zhongjun has quit IRC03:14
*** Zhongjun has joined #openstack-cinder03:14
*** hflai_ has quit IRC03:17
*** hflai has joined #openstack-cinder03:17
*** abhiram_moturi has quit IRC03:22
*** abhiram_moturi has joined #openstack-cinder03:23
*** abhiram_moturi has quit IRC03:27
*** abhiram_moturi has joined #openstack-cinder03:27
*** Apoorva has quit IRC03:29
*** abhiram_moturi has quit IRC03:33
*** abhiram_moturi has joined #openstack-cinder03:34
*** abhiram_moturi has quit IRC03:36
*** abhiram_moturi has joined #openstack-cinder03:36
*** marcusvrn has joined #openstack-cinder03:38
*** bill_az has quit IRC03:41
*** abhiram_moturi has quit IRC03:45
*** abhiram_moturi has joined #openstack-cinder03:45
*** dalgaaf has quit IRC03:47
*** davechen has left #openstack-cinder03:48
*** IlyaG has quit IRC03:48
*** abhiram_moturi has quit IRC03:51
*** abhiram_moturi has joined #openstack-cinder03:52
*** marcusvrn has quit IRC03:54
*** abhiram_moturi has quit IRC03:58
*** abhiram_moturi has joined #openstack-cinder03:58
*** vmtrooper has joined #openstack-cinder04:00
*** MentalRay has quit IRC04:01
*** haomai___ has quit IRC04:03
*** bill_az has joined #openstack-cinder04:05
*** bkopilov has joined #openstack-cinder04:07
*** abhiram_moturi has quit IRC04:10
*** abhiram_moturi has joined #openstack-cinder04:10
*** akerr_ has quit IRC04:14
openstackgerritwanghao proposed openstack/cinder-specs: Save the real ref when managing volume  https://review.openstack.org/18215004:18
*** nikeshm has joined #openstack-cinder04:22
nikeshmasselin: hi04:22
nikeshmgot the mistake04:22
nikeshmmy network is 172.16.x.x but nodepool is creating vms in 172.2404:23
nikeshm.x.x04:23
*** abhiram_moturi has quit IRC04:27
*** abhiram_moturi has joined #openstack-cinder04:27
*** _cjones_ has joined #openstack-cinder04:28
*** dims has joined #openstack-cinder04:33
*** _cjones_ has quit IRC04:33
*** reed_ has joined #openstack-cinder04:36
*** abhiram_moturi has quit IRC04:38
*** dims has quit IRC04:38
*** abhiram_moturi has joined #openstack-cinder04:38
*** anshul has joined #openstack-cinder04:40
*** jaypipes has quit IRC04:40
*** abhiram_moturi has quit IRC04:43
*** abhiram_moturi has joined #openstack-cinder04:43
*** reed_ has quit IRC04:43
*** abhiram_moturi has quit IRC04:47
*** abhiram_moturi has joined #openstack-cinder04:48
*** abhiram_moturi has quit IRC04:52
*** tobe has quit IRC04:52
*** abhiram_moturi has joined #openstack-cinder04:52
*** 5EXAA60VI has joined #openstack-cinder04:52
*** markvoelker has joined #openstack-cinder04:54
*** lpetrut has joined #openstack-cinder04:55
*** jaypipes has joined #openstack-cinder04:56
*** anuragpalsule has joined #openstack-cinder04:58
*** sks has joined #openstack-cinder05:01
*** abhiram_moturi has quit IRC05:04
*** abhiram_moturi has joined #openstack-cinder05:04
*** tshefi has joined #openstack-cinder05:07
*** dulek has joined #openstack-cinder05:08
*** abhiram_moturi has quit IRC05:10
*** abhiram_moturi has joined #openstack-cinder05:11
*** abhiram_moturi has quit IRC05:13
*** abhiram_moturi has joined #openstack-cinder05:13
*** davechen has joined #openstack-cinder05:15
*** abhiram_moturi has quit IRC05:20
*** abhiram_moturi has joined #openstack-cinder05:20
*** nkrinner has joined #openstack-cinder05:20
*** vmtrooper has quit IRC05:23
*** garthb has quit IRC05:24
*** deepakcs has joined #openstack-cinder05:24
*** abhiram_moturi has quit IRC05:29
*** abhiram_moturi has joined #openstack-cinder05:29
*** abhiram_moturi has quit IRC05:31
*** abhiram_moturi has joined #openstack-cinder05:31
*** abhiram_moturi has quit IRC05:33
*** abhiram_moturi has joined #openstack-cinder05:34
*** abhiram_moturi has quit IRC05:38
*** abhiram_moturi has joined #openstack-cinder05:38
*** abhiram_moturi has quit IRC05:42
*** abhiram_moturi has joined #openstack-cinder05:43
*** nlevinki has joined #openstack-cinder05:43
*** abhiram_moturi has quit IRC05:47
*** abhiram_moturi has joined #openstack-cinder05:47
*** markus_z has joined #openstack-cinder05:51
*** anuragpalsule1 has joined #openstack-cinder05:54
*** anuragpalsule has quit IRC05:54
openstackgerritEdwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem  https://review.openstack.org/15620606:00
*** ankit_ag has joined #openstack-cinder06:00
deepakcsI have a quick Q, can anybody confirm if I can use snapshots as a source of backup in `cinder backup-create` ?06:04
deepakcsLooking at the code, it seems it only support backing up volumes, not snapshots , is that correct ?06:05
*** abhiram_moturi has quit IRC06:07
*** abhiram_moturi has joined #openstack-cinder06:07
*** abhiram_moturi has quit IRC06:07
*** abhiram_moturi has joined #openstack-cinder06:09
*** sgotliv has joined #openstack-cinder06:09
*** abhiram_moturi has quit IRC06:15
*** abhiram_moturi has joined #openstack-cinder06:15
*** markvoelker has quit IRC06:15
*** afazekas_ has joined #openstack-cinder06:16
*** markvoelker has joined #openstack-cinder06:18
*** coolsvap|afk is now known as coolsvap06:18
*** tobe has joined #openstack-cinder06:19
*** tobe has quit IRC06:19
*** coolsvap is now known as coolsvap|afk06:21
*** abhiram_moturi has quit IRC06:24
*** agarciam has quit IRC06:24
*** abhiram_moturi has joined #openstack-cinder06:24
*** coolsvap|afk has quit IRC06:28
*** coolsvap has joined #openstack-cinder06:29
*** bill_az has quit IRC06:29
*** abhiram_moturi has quit IRC06:33
*** abhiram_moturi has joined #openstack-cinder06:33
*** abhiram_moturi has quit IRC06:39
*** abhiram_moturi has joined #openstack-cinder06:39
*** abhiram_moturi has quit IRC06:43
*** abhiram_moturi has joined #openstack-cinder06:44
*** abhiram_moturi has quit IRC06:46
*** abhiram_moturi has joined #openstack-cinder06:46
*** BharatK has joined #openstack-cinder06:47
*** agarciam has joined #openstack-cinder06:54
*** abhiram_moturi has quit IRC06:59
*** abhiram_moturi has joined #openstack-cinder06:59
*** tobe43 has joined #openstack-cinder07:00
*** abhiram_moturi has quit IRC07:06
*** abhiram_moturi has joined #openstack-cinder07:06
wanghaodeepalcs: Yes, it only support backing up volumes now.07:09
*** ronis has joined #openstack-cinder07:13
openstackgerritXinXiaohui proposed openstack/cinder-specs: capacity-headroom  https://review.openstack.org/17038007:15
*** vnagpal has joined #openstack-cinder07:18
deepakcswanghao: do you know why backup up snapshots is not supported in backup-create ?07:18
*** nlevinki_ has joined #openstack-cinder07:20
*** nlevinki has quit IRC07:23
wanghaodeepakcs: don't know why, There is a bp about incremental backup in Kilo :https://blueprints.launchpad.net/cinder/+spec/incremental-backup. In this specs, it want to implement the backup of snapshot to support attached volume.  But in the code of this bp, I can't find this too...07:26
*** abhiram_moturi has quit IRC07:33
*** jistr has joined #openstack-cinder07:33
*** abhiram_moturi has joined #openstack-cinder07:33
*** abhiram_moturi has quit IRC07:35
*** abhiram_moturi has joined #openstack-cinder07:35
*** lpetrut has left #openstack-cinder07:37
*** abhiram_moturi has quit IRC07:40
*** abhiram_moturi has joined #openstack-cinder07:40
DuncanTwanghao: Backup of snap is not yet implemented07:43
DuncanTwanghao: There's a summit session on finalising the details of the design07:43
DuncanTdeepakcs: ^^^^^07:44
wanghaoDuncanT: OK got it.07:44
wanghaoDuncanT: BTW, I update the patch(https://review.openstack.org/#/c/180400/) that you +2 at yesterday, for remove some unnecessary debug log message. So could you help me +2 again today?07:48
DuncanTSure07:48
*** abhiram_moturi has quit IRC07:51
*** abhiram_moturi has joined #openstack-cinder07:51
*** openstackstatus has quit IRC07:52
*** openstack has joined #openstack-cinder07:54
*** abhiram_moturi has quit IRC08:00
*** abhiram_moturi has joined #openstack-cinder08:00
*** daneyon has quit IRC08:01
DuncanTwanghao: Sorry, looks like you're getting ping-ponged on this one08:01
*** abhiram_moturi has quit IRC08:08
*** abhiram_moturi has joined #openstack-cinder08:09
*** Longgeek has joined #openstack-cinder08:09
deepakcsDuncanT: thanks , is there any technical reason/hurdle for not implementing backup of snap in the past, or it was just not picked up ?08:10
*** jordanP has joined #openstack-cinder08:10
DuncanTdeepakcs: technical hurdle - there's no call to attach a snap in any cinder driver08:10
DuncanTdeepakcs: Or do anything other than create a volume from it08:11
deepakcsDuncanT: like volID if we pass snapID to create_backup, then drivers can do what they need to do to get the snap seen as a block device for it to act as a src for backup, no ?08:11
DuncanTdeepakcs: That code would need writing for every driver, which isn't easy with 40+ drivers08:12
DuncanTdeepakcs: And some drivers might not be able to do it at all08:12
deepakcsDuncanT: yeah, hence it should be left to drivers discretion, is what i thought08:12
DuncanTdeepakcs: So the plan is to see if the driver has added support for it, and if not, clone a temporary volume for the backup and delete it at the end08:13
openstackgerritGorka Eguileor proposed openstack/python-cinderclient: Fix v2 list when num volumes > osapi_max_limit  https://review.openstack.org/18187508:13
DuncanTdeepakcs: Since incremental backup was tricky enough, there wasn't time to do backup of snap too08:13
DuncanTdeepakcs: It should get done soon08:14
deepakcsDuncanT: ok, thanks for the info08:14
deepakcsDuncanT: i have one other req... can you pls look at the WIP patches i sent for the volume-snap-impr BPs (nova and cinder parts): https://review.openstack.org/172813 and https://review.openstack.org/17280808:15
DuncanTdeepakcs: they're on my list, but it isn't a code area I look at often so will take a while08:16
DuncanTdeepakcs: I'm in utc+308:16
*** e0ne has joined #openstack-cinder08:17
*** e0ne is now known as e0ne_08:17
deepakcsDuncanT: ok, but these BPs were a result of the security issue you highlighted with processign qcow2 files08:17
deepakcsDuncanT: so i look at you as stakeholder :)08:17
DuncanTI'm aware - easier to spot the issue than assess the fix though. I will be  reviewing them08:17
deepakcsDuncanT: cool08:18
*** xiaohui has quit IRC08:20
*** xiaohui has joined #openstack-cinder08:20
*** Longgeek has quit IRC08:22
*** abhiram_moturi has quit IRC08:26
*** abhiram_moturi has joined #openstack-cinder08:26
*** Longgeek has joined #openstack-cinder08:27
*** Longgeek has quit IRC08:29
*** Longgeek has joined #openstack-cinder08:30
*** anuragpalsule has joined #openstack-cinder08:33
*** anuragpalsule1 has quit IRC08:36
*** nlevinki_ has quit IRC08:41
*** e0ne_ is now known as e0ne08:46
*** e0ne is now known as e0ne_08:46
*** abhiram_moturi has quit IRC08:51
*** abhiram_moturi has joined #openstack-cinder08:52
*** abhiram_moturi has quit IRC08:54
*** abhiram_moturi has joined #openstack-cinder08:54
*** nlevinki_ has joined #openstack-cinder08:55
*** e0ne_ is now known as e0ne08:55
*** abhiram_moturi has quit IRC09:01
*** abhiram_moturi has joined #openstack-cinder09:01
*** abhiram_moturi has quit IRC09:03
*** abhiram_moturi has joined #openstack-cinder09:04
*** abhiram_moturi has quit IRC09:06
*** abhiram_moturi has joined #openstack-cinder09:06
*** abhiram_moturi has quit IRC09:10
*** abhiram_moturi has joined #openstack-cinder09:10
*** abhiram_moturi has quit IRC09:12
*** abhiram_moturi has joined #openstack-cinder09:13
*** anuragpalsule has quit IRC09:15
*** anuragpalsule1 has joined #openstack-cinder09:16
*** aix has joined #openstack-cinder09:16
*** pratyu has joined #openstack-cinder09:18
*** pratyu has quit IRC09:19
*** abhiram_moturi has quit IRC09:23
*** abhiram_moturi has joined #openstack-cinder09:24
*** abhiram_moturi has quit IRC09:28
*** abhiram_moturi has joined #openstack-cinder09:28
*** abhiram_moturi has quit IRC09:30
*** abhiram_moturi has joined #openstack-cinder09:31
*** tobe43 has quit IRC09:31
*** daneyon has joined #openstack-cinder09:31
*** IanGovett has joined #openstack-cinder09:32
*** abhiram_moturi has quit IRC09:35
*** abhiram_moturi has joined #openstack-cinder09:35
*** abhiram_moturi has quit IRC09:39
*** abhiram_moturi has joined #openstack-cinder09:40
wanghaoDuncanT: hi~ I saw your review. well, I will add one message as you said. Wish Mike will approve it too.:)09:40
DuncanTThanks. Hopefully he will09:40
*** abhiram_moturi has quit IRC09:44
*** abhiram_moturi has joined #openstack-cinder09:45
*** davechen has left #openstack-cinder09:45
*** abhiram_moturi has quit IRC09:51
*** aswadr has joined #openstack-cinder09:51
*** abhiram_moturi has joined #openstack-cinder09:51
openstackgerritDeepak C Shetty proposed openstack/cinder-specs: Support storing volume format info  https://review.openstack.org/16539309:52
*** abhiram_moturi has quit IRC09:58
*** abhiram_moturi has joined #openstack-cinder09:58
*** abhiram_moturi has quit IRC10:02
*** abhiram_moturi has joined #openstack-cinder10:03
*** abhiram_moturi has quit IRC10:07
*** abhiram_moturi has joined #openstack-cinder10:07
*** abhiram_moturi has quit IRC10:11
*** abhiram_moturi has joined #openstack-cinder10:11
*** abhiram_moturi has quit IRC10:13
*** abhiram_moturi has joined #openstack-cinder10:14
openstackgerritVictor Stinner proposed openstack/cinder: Replace suds test dependency with suds-jurko  https://review.openstack.org/18013010:15
*** heyun has quit IRC10:19
*** e0ne is now known as e0ne_10:21
*** abhiram_moturi has quit IRC10:25
*** abhiram_moturi has joined #openstack-cinder10:25
*** alonmarx_ has joined #openstack-cinder10:30
*** dims has joined #openstack-cinder10:33
*** alonmarx has quit IRC10:33
*** abhiram_moturi has quit IRC10:38
*** abhiram_moturi has joined #openstack-cinder10:38
*** e0ne_ is now known as e0ne10:39
*** abhiram_moturi has quit IRC10:42
*** abhiram_moturi has joined #openstack-cinder10:43
*** abhiram_moturi has quit IRC10:45
*** abhiram_moturi has joined #openstack-cinder10:45
*** abhiram_moturi has quit IRC10:47
*** abhiram_moturi has joined #openstack-cinder10:48
*** ociuhandu has joined #openstack-cinder10:50
*** coolsvap is now known as coolsvap|afk10:52
*** abhiram_moturi has quit IRC10:56
*** abhiram_moturi has joined #openstack-cinder10:57
*** abhiram_moturi has quit IRC11:01
*** abhiram_moturi has joined #openstack-cinder11:01
*** abhiram_moturi has quit IRC11:03
*** abhiram_moturi has joined #openstack-cinder11:03
*** links has joined #openstack-cinder11:04
*** abhiram_moturi has quit IRC11:06
*** abhiram_moturi has joined #openstack-cinder11:06
*** anshul has quit IRC11:10
*** abhiram_moturi has quit IRC11:14
*** abhiram_moturi has joined #openstack-cinder11:14
*** abhiram_moturi has quit IRC11:19
*** abhiram_moturi has joined #openstack-cinder11:19
openstackgerritwanghao proposed openstack/cinder: Notification with volume and snaphost metadata  https://review.openstack.org/18040011:22
*** anuragpalsule1 has quit IRC11:23
*** timcl has joined #openstack-cinder11:25
*** abhiram_moturi has quit IRC11:29
*** abhiram_moturi has joined #openstack-cinder11:30
*** anshul has joined #openstack-cinder11:30
*** marcusvrn has joined #openstack-cinder11:30
*** marcusvrn has quit IRC11:31
*** marcusvrn has joined #openstack-cinder11:31
*** abhiram_moturi has quit IRC11:34
*** abhiram_moturi has joined #openstack-cinder11:34
*** abhiram_moturi has quit IRC11:36
*** abhiram_moturi has joined #openstack-cinder11:37
*** abhiram_moturi has quit IRC11:39
*** abhiram_moturi has joined #openstack-cinder11:39
*** abhiram_moturi has quit IRC11:43
*** deepakcs has quit IRC11:44
*** abhiram_moturi has joined #openstack-cinder11:44
openstackgerritAbhijeet Malawade proposed openstack/cinder: Remove unused context parameter  https://review.openstack.org/18225011:45
*** jistr is now known as jistr|class11:46
*** jistr|class is now known as jistr11:47
*** jistr is now known as jistr|class11:50
openstackgerritTom Barron proposed openstack/cinder: Use nfs_oversub_ratio when reporting pool capacity  https://review.openstack.org/18019911:51
openstackgerritAnton Arefiev proposed openstack/python-cinderclient: Add findall server side filtering  https://review.openstack.org/18053411:53
openstackgerritAnton Arefiev proposed openstack/python-cinderclient: Find resource refactoring  https://review.openstack.org/18053311:53
*** abhiram_moturi has quit IRC11:55
*** abhiram_moturi has joined #openstack-cinder11:56
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: Add Posix backup driver. Supports simple nas too.  https://review.openstack.org/16364711:56
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS backup driver  https://review.openstack.org/18225411:56
*** jistr|class is now known as jistr11:57
*** Zhongjun has quit IRC11:59
*** e0ne is now known as e0ne_12:00
*** nlevinki_ has quit IRC12:00
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS backup driver  https://review.openstack.org/18225412:01
*** dims_ has joined #openstack-cinder12:03
*** lpetrut has joined #openstack-cinder12:03
*** jseiler has quit IRC12:05
*** lpetrut has quit IRC12:05
*** fthiagogv has joined #openstack-cinder12:05
*** dims has quit IRC12:06
*** bswartz has quit IRC12:12
*** nlevinki_ has joined #openstack-cinder12:13
*** jungleboyj has joined #openstack-cinder12:15
*** aix has quit IRC12:17
*** primechuck has quit IRC12:23
*** abhiram_moturi has quit IRC12:26
*** Longgeek has quit IRC12:26
*** abhiram_moturi has joined #openstack-cinder12:27
*** dulek_ has joined #openstack-cinder12:27
*** Longgeek has joined #openstack-cinder12:27
*** dulek has quit IRC12:27
*** dulek_ has quit IRC12:28
*** dulek_ has joined #openstack-cinder12:28
*** aix has joined #openstack-cinder12:28
*** akerr has joined #openstack-cinder12:29
*** abhiram_moturi has quit IRC12:31
*** abhiram_moturi has joined #openstack-cinder12:31
*** bkopilov has quit IRC12:32
*** e0ne_ is now known as e0ne12:34
*** nlevinki_ has quit IRC12:36
*** abhiram_moturi has quit IRC12:38
*** abhiram_moturi has joined #openstack-cinder12:39
*** dims_ has quit IRC12:40
*** dims has joined #openstack-cinder12:42
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS backup driver  https://review.openstack.org/18225412:44
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: Add Posix backup driver. Supports simple nas too.  https://review.openstack.org/16364712:44
*** nlevinki_ has joined #openstack-cinder12:48
*** merooney has joined #openstack-cinder12:48
*** abhiram_moturi has quit IRC12:50
*** abhiram_moturi has joined #openstack-cinder12:50
*** bswartz has joined #openstack-cinder12:51
*** abhiram_moturi has quit IRC12:57
*** abhiram_moturi has joined #openstack-cinder12:57
*** abhiram_moturi has quit IRC12:59
*** abhiram_moturi has joined #openstack-cinder12:59
openstackgerritwanghao proposed openstack/cinder: Support volume backup of qcow2 format volume  https://review.openstack.org/18227613:00
*** dulek_ has quit IRC13:01
*** dulek_ has joined #openstack-cinder13:02
*** jistr is now known as jistr|mtg13:04
*** abhiram_moturi has quit IRC13:10
*** abhiram_moturi has joined #openstack-cinder13:10
*** primechuck has joined #openstack-cinder13:11
jordanPwanghao, hi. Regarding the patch you just pushed: isn't qemu-nbd concidered insecure ? I though nova replaced it in favor of libguestfs13:13
*** hodos has joined #openstack-cinder13:13
hodoswhy do stable/icehouse and stable/juno cinder branches require oslo.i18n package?13:14
hodosi see a bug also which looks similar https://bugs.launchpad.net/cinder/+bug/144760813:16
openstackLaunchpad bug 1447608 in Cinder " pkg_resources.ContextualVersionConflict: (oslo.i18n 1.3.1 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('oslo.i18n>=1.5.0'), set(['oslo.utils']))" [Undecided,New] - Assigned to Vilobh Meshram (vilobhmm)13:16
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: RemoteFS: Reporting configured reserved_percentage in _update_volume_stats  https://review.openstack.org/18228613:17
openstackgerritPradeep Sathasivam proposed openstack/cinder: Adds friendly zone name support  https://review.openstack.org/18051813:17
*** anuragpalsule has joined #openstack-cinder13:17
*** Yogi1 has joined #openstack-cinder13:19
*** abhiram_moturi has quit IRC13:22
*** abhiram_moturi has joined #openstack-cinder13:23
*** anuragpalsule1 has joined #openstack-cinder13:23
*** timcl has quit IRC13:23
*** anuragpalsule1 has left #openstack-cinder13:23
*** anuragpalsule1 has joined #openstack-cinder13:25
*** anuragpalsule has quit IRC13:25
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add --os-endpoint-type to match other services  https://review.openstack.org/18228713:25
*** abhiram_moturi has quit IRC13:29
*** abhiram_moturi has joined #openstack-cinder13:30
smcginnisjungleboyj: ping13:31
*** zhenguo has quit IRC13:34
*** annegentle has joined #openstack-cinder13:35
*** merooney has quit IRC13:35
*** dustins has joined #openstack-cinder13:37
jungleboyjsmcginnis: Pong13:40
smcginnisjungleboyj: Mornin!13:41
jungleboyjsmcginnis: Morning sir!13:41
*** rushil has joined #openstack-cinder13:41
smcginnisjungleboyj: I had a question about the cinder/openstack directory. Can anything be modified in there, or is that all just synced from oslo?13:42
jungleboyjsmcginnis: All needs to be sync'd from oslo.  Do you need something sync'd over?13:42
*** merooney has joined #openstack-cinder13:43
smcginnisjungleboyj: Just finalizing the i18n logging cleanup series of patches.13:43
smcginnisjungleboyj: There are some things in there that are using % instead of ,13:43
smcginnisBut translation markers are used, so not a big deal.13:43
jungleboyjOh, ok, you would want to propose those fixes to oslo-incubator first.13:43
smcginnisLooks like the final patch to close out that bug can just be to stop ignoring that directory.13:43
smcginnisjungleboyj: Cool, I'll do that.13:43
*** links has quit IRC13:44
smcginnisYay, cinder side is done then. :)13:44
*** merooney has quit IRC13:44
smcginnisjungleboyj: On that note... if you have some time can you take a look at Ankit's patch? https://review.openstack.org/#/c/164697/13:44
smcginnisIt covers the volume directory tree which is a huge PITA since it has so much and has a lot of churn.13:45
smcginnisOne of those things that is never really a good time to push through, but should be done at some point.13:45
smcginnisAnd if we can avoid more rebasing and re-reviewing I think that would be good.13:45
smcginnisjungleboyj: If no time, no worries. Just through I would point it out.13:45
*** mriedem_away is now known as mriedem13:46
jungleboyjsmcginnis: Ok, I will try to make that a priority so we can get that through.13:46
smcginnisjungleboyj: Thanks. Unfortunately a ton of files in there.13:46
jungleboyjYeah.  Not surprised.13:46
smcginnisMight have been better to split it into easier chunks, but oh well.13:47
*** asrangne has joined #openstack-cinder13:47
DuncanTLogging and translations seems to be a never ending series of mechanical patches13:47
*** rushil has quit IRC13:47
smcginnisDuncanT: Yeah. I'm really hoping this is it and it makes it better. Though there will always be some things sneaking in.13:48
smcginnisAt least the hacking check is in place now so we should be good on translation markers.13:48
hodoswhy do stable/icehouse and stable/juno cinder branches require oslo.i18n package?13:48
* smcginnis crosses fingers13:49
hodosdevstack fails cause of it13:49
DuncanTI wonder why e.g. https://review.openstack.org/#/c/164697/14/cinder/volume/drivers/netapp/eseries/host_mapper.py line 87 wasn't caught by the hacking check in the jenkins run?13:50
DuncanTjordanP: Can you add that as a (0 vote) question on the review so that it doesn't get lost, please?13:50
*** aswadr has quit IRC13:50
smcginnisDuncanT: It checks for _Lx() markers but checking for % vs , is a little too difficult.13:50
smcginnisAt least for my checking-fu.13:51
DuncanTsmcginnis: Ah, ok, thanks. I'll see if I can find a hacking check master to write one, the recently added (or at least proposed) AST shouldbe able to do it13:51
smcginnisDuncanT: Still gotta get some time to learn AST. Seems like a lot of cool stuff could be done using that.13:52
*** xek is now known as x3k13:52
DuncanTsmcginnis: Definitely. There are a few libraries built on top of it that make some things easier, particularly automated code transforms13:52
*** vnagpal has quit IRC13:52
*** Zhongjun has joined #openstack-cinder13:53
*** e0ne is now known as e0ne_13:53
DuncanTsmcginnis: Baron/RedBaron was recommended to me at Fosdem13:53
*** markvoelker has quit IRC13:53
*** e0ne_ is now known as e0ne13:55
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add --os-endpoint-type to match other services  https://review.openstack.org/18228713:55
smcginnisDuncanT: Cool, I'll have to check that out. Any helper/wrappers for it should be useful. Thanks!13:55
jordanPDuncanT, I will13:55
openstackgerritSean McGinnis proposed openstack/cinder: Logging not using oslo.i18n guidelines (openstack)  https://review.openstack.org/18229913:56
openstackgerritJohn Griffith proposed openstack/cinder: Check volume_backend in retype  https://review.openstack.org/18205513:56
*** jungleboyj has quit IRC13:57
*** Yogi1 has quit IRC13:59
openstackgerritRick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver  https://review.openstack.org/16965314:01
*** abhiram_moturi has quit IRC14:04
*** abhiram_moturi has joined #openstack-cinder14:04
openstackgerritRick Chen proposed openstack/cinder: Port ProphetStor driver to use new driver model  https://review.openstack.org/17899214:04
*** Yogi1 has joined #openstack-cinder14:07
*** merooney has joined #openstack-cinder14:07
*** thangp has joined #openstack-cinder14:07
*** boichev has quit IRC14:10
*** vmtrooper has joined #openstack-cinder14:12
*** bkopilov has joined #openstack-cinder14:13
*** BharatK has quit IRC14:13
*** rushil has joined #openstack-cinder14:16
*** MentalRay has joined #openstack-cinder14:17
*** cdelatte has joined #openstack-cinder14:19
*** delattec has joined #openstack-cinder14:19
*** annegentle has quit IRC14:21
*** deepakcs has joined #openstack-cinder14:21
*** annegentle has joined #openstack-cinder14:22
*** dulek_ has quit IRC14:25
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/17841014:28
*** breitz has quit IRC14:28
*** lpetrut has joined #openstack-cinder14:28
*** breitz has joined #openstack-cinder14:28
*** jungleboyj has joined #openstack-cinder14:33
*** rmesta has joined #openstack-cinder14:33
*** abhiram_moturi has quit IRC14:37
*** abhiram_moturi has joined #openstack-cinder14:38
*** jistr|mtg is now known as jistr14:39
*** merooney has quit IRC14:41
*** emagana has joined #openstack-cinder14:42
openstackgerritJohn Griffith proposed openstack/cinder: Check type match on create from source/snap  https://review.openstack.org/18232714:42
*** ankit_ag has quit IRC14:44
*** mtanino has joined #openstack-cinder14:45
jgriffiththingee: guitarzan xyang1 smcginnis Reworked without the now defunct dep: https://review.openstack.org/#/c/182327/114:46
jgriffithguitarzan: I changed the name for ya to :)14:46
smcginnisOn my queue.14:46
guitarzanjgriffith: :)14:46
xyang1jgriffith: Ok, will take a look14:48
guitarzanjgriffith: was there any desire to ever ask the driver if retype is possible?14:48
guitarzanI guess matching backend names is close14:48
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add --os-endpoint-type to match other services  https://review.openstack.org/18228714:48
guitarzanI also guess that this is just api "validation" more than anything14:48
*** vmtrooper has quit IRC14:48
jgriffithguitarzan: maybe, but I'm kinda of "of the opinion" that drivers are 'dumb' WRT Cinder14:49
guitarzanI know14:49
jgriffithguitarzan: but some sort of check might be cool, the problem with that though is the rpc boundary14:49
guitarzanyeah, create should stay async14:50
*** merooney has joined #openstack-cinder14:50
guitarzaneither way you get my +1 and I get to remove my api override hacks whenever we get this far :)14:51
jgriffithguitarzan: you and just about every single one of my OpenStack customers14:51
guitarzanhaha14:51
*** e0ne is now known as e0ne_14:51
guitarzanand you still think drivers should be dumb :D14:51
jgriffithguitarzan: I'm kinda surprised how many people are just flat out annoyed with that stupid limitation14:51
jgriffithguitarzan: LMAO14:52
jgriffithwell, "relative" :)14:52
guitarzanpossibly because the limitation is stupid?14:52
jgriffithSo I think they should be "super smart" about their domain :)14:52
guitarzan*cough*independentsnapshots*cough*14:52
jgriffithguitarzan: yes... it is stupid14:52
jgriffithguitarzan: couldn't agree more14:52
jgriffithguitarzan: I'm actually going to consider that again in Liberty FWIW14:52
guitarzanI think we're in a situation where cinder being so opinionated needs to back off a little14:53
guitarzanbut I've been of that opinion forever14:53
jgriffithguitarzan: don't know that I'll get anywhere on it, but at least want to look at it again14:53
jgriffithguitarzan: meh, in some cases I think opinionated is good, and wish we were more-so TBH14:53
jgriffithguitarzan: but only on the things I care about :)14:53
guitarzansure, I just want to sell volumes to people14:53
jgriffithhehe... roger that14:53
jgriffithfunny... me too :)14:54
*** jaypipes has quit IRC14:54
*** reed_ has joined #openstack-cinder14:54
guitarzanhaha14:54
jgriffithjust a different delivery/billing model14:54
*** deepakcs has quit IRC14:54
*** nkrinner has quit IRC14:55
*** vmtrooper has joined #openstack-cinder14:56
*** reed_ has quit IRC14:57
*** e0ne_ is now known as e0ne14:58
*** rwsu has joined #openstack-cinder14:59
*** vmtrooper has quit IRC15:00
*** merooney has quit IRC15:00
*** alexpilotti has joined #openstack-cinder15:01
*** dustins_ has joined #openstack-cinder15:02
*** merooney has joined #openstack-cinder15:03
*** Yogi11 has joined #openstack-cinder15:03
*** anshul has quit IRC15:03
*** rushil_ has joined #openstack-cinder15:04
*** Yogi1 has quit IRC15:04
*** dustins has quit IRC15:05
*** sks has quit IRC15:06
*** rushil has quit IRC15:06
*** ebalduf has joined #openstack-cinder15:07
*** ebalduf has quit IRC15:08
*** ebalduf has joined #openstack-cinder15:08
*** ebalduf has quit IRC15:08
*** hemnafk is now known as hemna15:09
*** ebalduf has joined #openstack-cinder15:10
*** MentalRay has quit IRC15:13
*** ronis has quit IRC15:14
*** aswadr has joined #openstack-cinder15:14
*** patrickeast has joined #openstack-cinder15:14
*** daneyon_ has joined #openstack-cinder15:15
*** afazekas_ has quit IRC15:15
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add --os-endpoint-type to match other services  https://review.openstack.org/18228715:16
hemnamornin15:16
*** crose has joined #openstack-cinder15:16
*** Apoorva has joined #openstack-cinder15:17
*** asrangne has quit IRC15:17
*** juzuluag has joined #openstack-cinder15:18
*** daneyon has quit IRC15:18
*** lpetrut has left #openstack-cinder15:19
openstackgerritAnton Arefiev proposed openstack/python-cinderclient: Fix condition in CheckSizeArgForCreate parser action  https://review.openstack.org/18234415:19
*** nlevinki_ has quit IRC15:19
*** _cjones_ has joined #openstack-cinder15:19
*** aswadr has quit IRC15:20
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS backup driver  https://review.openstack.org/18225415:20
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add --os-endpoint-type to match other services  https://review.openstack.org/18228715:21
*** shakamunyi has joined #openstack-cinder15:22
*** barra204 has joined #openstack-cinder15:22
*** dannywilson has joined #openstack-cinder15:25
*** markus_z has quit IRC15:25
*** krtaylor has quit IRC15:26
*** esker has joined #openstack-cinder15:28
*** jseiler has joined #openstack-cinder15:29
*** _cjones_ has quit IRC15:29
*** _cjones_ has joined #openstack-cinder15:30
*** jdurgin1 has joined #openstack-cinder15:31
*** annegentle has quit IRC15:32
hemnajungleboyj, ping15:32
openstackgerritRushil Chugh proposed openstack/cinder: Avoid LUN ID collisions in NetApp iSCSI drivers  https://review.openstack.org/17923915:32
*** julim has quit IRC15:33
*** IanGovett1 has joined #openstack-cinder15:34
*** IanGovett has quit IRC15:35
*** krtaylor has joined #openstack-cinder15:38
jungleboyjhemna: What up?15:38
*** IanGovett1 has quit IRC15:39
hemnahey, so what's usually the process with those "Updated from global requirements" bot patches15:39
hemnado we just always shove those through ?15:39
hemnare: https://review.openstack.org/#/c/180830/15:39
jgriffithhemna: yes15:40
hemnaok, this is the first one I've seen for os-brick, so I was just curious15:40
jungleboyjhemna: Yeah.15:40
jungleboyjIt is good for a couple people to see if so they are aware of the changes being made.15:40
*** tshefi has quit IRC15:41
*** IanGovett has joined #openstack-cinder15:41
jungleboyjThat is why I don't blindly +2/A it.15:41
jgriffithhemna: FYI, devstack deploy will use current global regardless15:41
hemnathat's what I kind figured, so I didn't ninja it in.15:41
jungleboyjNo Ginsu for you!15:42
jgriffithhemna: also it used to be they'd auto merge after a while anyway so it didn't really matter15:42
jgriffithbut not sure it still does that15:42
*** anuragpalsule1 has quit IRC15:42
hemna:P15:43
hemnaok I'll let someone else +A them then.15:43
hemnathanks for the info.15:43
*** jaypipes has joined #openstack-cinder15:45
*** garthb has joined #openstack-cinder15:46
*** daneyon_ has quit IRC15:47
*** daneyon has joined #openstack-cinder15:48
jungleboyjhemna: NP.  Are we meeting again at some point this week?15:48
*** jistr has quit IRC15:50
*** links has joined #openstack-cinder15:50
thingeehemna: ugh, was hoping it would land. there are other things that conflict with it that are close to merging.15:52
*** agarciam has quit IRC15:54
*** rushil_ has quit IRC15:56
*** MentalRay has joined #openstack-cinder15:56
e0nehemna: hi. did you have a time to take a look on cinder-agent poc?15:57
*** merooney has quit IRC15:57
*** mtreinish has quit IRC15:58
*** merooney has joined #openstack-cinder15:58
*** rushil has joined #openstack-cinder15:58
*** rwsu has quit IRC15:58
*** rwsu has joined #openstack-cinder15:59
*** mtreinish has joined #openstack-cinder16:00
*** dustins_ has quit IRC16:01
*** Yogi12 has joined #openstack-cinder16:02
*** merooney has quit IRC16:03
*** dustins has joined #openstack-cinder16:03
*** dustins has quit IRC16:04
*** Yogi11 has quit IRC16:04
hemnajungleboyj, yah, I was just waiting for slides from jgriffith to finish off the deck.16:04
hemnathingee, the brick removal patch?16:04
thingeeyes16:04
hemnathingee, I issued a recheck...it's almost through check16:05
* jgriffith guesses he'll stop reviewing requirements changes. Apparantly you guys know somethign he doesn't16:05
thingeehemna: yeah saw that16:05
*** dustins has joined #openstack-cinder16:05
thingeehemna: do you have time sometime this week to talk about the capabilities spec?16:05
jungleboyjhemna: Ok.  Was just curious.  This week is looking insane and wanted to make sure I hadn't missed anything.16:06
thingeehemna: I'm going to break things out as suggested with just the terms, but would like to talk about how we want to expose information from the driver to an interface that things like horizon can consume, without clogging the MQ16:06
hemnathingee, yah, I'd love to16:06
thingeewhat does your calendar look like this week?16:06
hemnait's open16:06
jgriffithhemna: where did your g-doc link go?16:06
hemnajgriffith, https://etherpad.openstack.org/p/vancouver_fc_in_openstack16:07
hemnathe dropbox url in there has the latest16:07
jgriffithhemna: thanks16:07
SwansonTo anyone.  Whatever happened to the multiple iscsi target portal support?  Did that make it to nova or brick or whatever magic thing deals with that?16:07
hemnathingee, so today I'm open, tomorrow I have meetings in the mornin until 1, thursday, friday is openm16:08
thingeeSwanson: https://wiki.openstack.org/wiki/ReleaseNotes/Kilo#OpenStack_Block_Storage_.28Cinder.2916:08
thingeehemna: k, would rather do today so I have time to write stuff up and get feedback before the summit16:09
*** Zhongjun has quit IRC16:09
hemnaok16:09
hemnacan do a google hangout with some folks that are interested in it, if you like.16:10
thingeesure, I think jgriffith was16:10
thingeejgriffith: can we brain storm today on capabilities stuff again?16:11
*** openstackgerrit_ has quit IRC16:11
jgriffiththingee: can't do today16:11
Swansonthingee: Thanks!16:11
jgriffiththingee: you guys can go without me if you want though16:12
jgriffiththingee: we're not going to see eye to eye on that I'm pretty sure16:12
jgriffiththingee: so I'd just be in your way :)16:12
thingeejgriffith: ok, I'll ping you with a link to any new spec for feedback.16:12
thingeejgriffith: going to split things up as you suggested earlier.16:12
jgriffiththingee: I'll probably just write my own spec at this point16:12
jgriffithand we can compare/discuss in VAncouver16:12
thingeejgriffith: if you could write something before the summit, that would be great16:13
thingeeyea16:13
jgriffiththingee: will do16:13
*** dustins has quit IRC16:14
thingeethangp, DuncanT: how are the rpc compat. discussions going?16:16
*** abhiram_moturi has quit IRC16:20
*** abhiram_moturi has joined #openstack-cinder16:20
hemnajgriffith, I'm not sure that we aren't on the same page actually.16:21
hemnajgriffith, I kinda thought we were, after all the churning in the meetup, I think we were all kinda saying the same thing, but just had to argue for a while :)16:21
*** ronis has joined #openstack-cinder16:22
jgriffithhemna: maybe16:22
*** abhiram_moturi has quit IRC16:24
*** abhiram_moturi has joined #openstack-cinder16:25
hemnaok it's in the gate16:26
hemnaqueued up16:26
thingeehemna: that's what I thought too the last time we all talked. I guess we'll compare specs and see though16:27
hemnaok cool16:27
hemnaI'd love to move that forward16:27
thingeehemna: how about after lunch? when do you get done?16:27
hemnaI can be available after 1pm PDT16:27
thingeeyay same timezone. 1:30 works for me16:28
hemnaok sounds good.16:28
*** julim has joined #openstack-cinder16:31
*** rushil has quit IRC16:32
*** abhiram_moturi has quit IRC16:34
*** abhiram_moturi has joined #openstack-cinder16:35
*** afazekas_ has joined #openstack-cinder16:35
*** annegentle has joined #openstack-cinder16:36
*** abhiram_moturi has quit IRC16:41
*** annegentle has quit IRC16:41
*** abhiram_moturi has joined #openstack-cinder16:41
*** afazekas_ has quit IRC16:42
*** leeantho has joined #openstack-cinder16:43
*** dustins has joined #openstack-cinder16:43
*** dustins has quit IRC16:43
*** patrickeast has quit IRC16:44
thangpthingee: we got down the high level stuff - https://etherpad.openstack.org/p/duncant-on-versioned-things16:47
thangpthingee: I am working on a prototype right now16:47
*** emagana has quit IRC16:48
*** timcl has joined #openstack-cinder16:48
hemnae0ne, hey man16:49
hemnae0ne, I haven't had any time to look at the agent yet16:50
e0ne:(16:50
thingeeGot a chance to speak about the CI stuff. hope you all feel like it's well represented for the team http://superuser.openstack.org/articles/what-you-need-to-know-about-openstack-cinder16:50
hemnae0ne, I'm trying to get nova os-brick patch passing unit tests.16:50
hemnae0ne,  what's the url16:50
hemnaI'll see if I can poke at it real quick16:50
*** emagana has joined #openstack-cinder16:50
e0nehemna: i've got some feedback from our ironic team. thay pay attantion on the security of it16:51
hemnaok good16:51
*** abhiram_moturi has quit IRC16:54
*** abhiram_moturi has joined #openstack-cinder16:54
*** yuriy_n17 has quit IRC16:55
*** abhiram_moturi has quit IRC17:00
*** abhiram_moturi has joined #openstack-cinder17:01
*** emagana has quit IRC17:02
*** mtreinish has quit IRC17:02
*** esker has quit IRC17:02
*** yrabl has quit IRC17:02
*** ofek-baby_ has quit IRC17:02
*** mtreinish_ has joined #openstack-cinder17:02
*** mtreinish_ is now known as mtreinish17:02
*** emagana has joined #openstack-cinder17:02
*** yrabl has joined #openstack-cinder17:02
*** esker has joined #openstack-cinder17:03
*** marcusvrn has quit IRC17:03
*** tpeoples has quit IRC17:03
*** merooney has joined #openstack-cinder17:03
*** cpallares has quit IRC17:04
*** ofek-baby_ has joined #openstack-cinder17:04
*** cpallares has joined #openstack-cinder17:04
*** BharatK has joined #openstack-cinder17:05
*** juzuluag has quit IRC17:07
*** merooney has quit IRC17:09
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: RemoteFS: Reporting configured reserved_percentage in _update_volume_stats  https://review.openstack.org/18228617:11
*** jordanP has quit IRC17:13
*** merooney has joined #openstack-cinder17:13
*** links has quit IRC17:14
*** abhiram_moturi has quit IRC17:16
*** abhiram_moturi has joined #openstack-cinder17:17
*** rushil has joined #openstack-cinder17:17
*** harlowja has quit IRC17:22
*** harlowja has joined #openstack-cinder17:22
*** marcusvrn has joined #openstack-cinder17:22
*** timcl has quit IRC17:25
*** patrickeast has joined #openstack-cinder17:25
openstackgerritRushil Chugh proposed openstack/cinder: Avoid LUN ID collisions in NetApp iSCSI drivers  https://review.openstack.org/17923917:25
*** merooney_ has joined #openstack-cinder17:26
hemnathingee, https://review.openstack.org/#/c/155552/17:27
*** merooney has quit IRC17:30
SwansonMy log slowly fills/A Moturi joins and quits/Cherry Blossoms Fall17:30
*** abhiram_moturi has quit IRC17:35
*** abhiram_moturi has joined #openstack-cinder17:35
*** annegentle has joined #openstack-cinder17:37
thingeehemna: time to take over the world!17:37
hemnaw00t!17:37
* hemna crosses fingers......17:37
hemnaheh17:38
*** tellesnobrega_ has joined #openstack-cinder17:38
*** tellesnobrega_ has left #openstack-cinder17:38
thingeehehe17:41
*** jdurgin1 has quit IRC17:42
*** timcl has joined #openstack-cinder17:42
*** juzuluag has joined #openstack-cinder17:42
*** annegentle has quit IRC17:42
asselinthingee, nice article17:45
*** vilobhmm11 has joined #openstack-cinder17:46
thingeeasselin: thanks.17:46
*** chlong has quit IRC17:46
*** e0ne is now known as e0ne_17:47
openstackgerritTom Swanson proposed openstack/cinder: Dell SC driver honoring folder name after volume creation.  https://review.openstack.org/17799417:54
openstackgerritTom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls  https://review.openstack.org/18239617:54
*** merooney_ has quit IRC17:55
SwansonAs god is my witness I have no idea if I submitted that correctly.17:55
*** e0ne_ is now known as e0ne17:56
openstackgerritTom Barron proposed openstack/cinder: Use nfs_oversub_ratio when reporting pool capacity  https://review.openstack.org/18019917:58
*** e0ne has quit IRC17:59
*** timcl1 has joined #openstack-cinder17:59
*** chlong has joined #openstack-cinder17:59
*** MentalRay has quit IRC17:59
*** merooney has joined #openstack-cinder18:01
*** timcl has quit IRC18:01
*** fthiagogv has quit IRC18:04
* jungleboyj is crying as I dig into the i18n mega volume patch.18:06
*** BharatK has quit IRC18:09
*** tsekiyama has joined #openstack-cinder18:10
*** abhiram_moturi has quit IRC18:12
*** abhiram_moturi has joined #openstack-cinder18:12
smcginnisjungleboyj: Yep18:14
* jungleboyj can't see your message through the tears smcginnis 18:14
smcginnisHah!18:14
*** rhagarty has quit IRC18:20
*** rhagarty_ has quit IRC18:20
*** abhiram_moturi has quit IRC18:25
*** abhiram_moturi has joined #openstack-cinder18:25
*** abhiram_moturi has quit IRC18:27
*** abhiram_moturi has joined #openstack-cinder18:27
*** rhagarty has joined #openstack-cinder18:28
*** rhagarty_ has joined #openstack-cinder18:29
*** rhagarty has quit IRC18:29
*** rhagarty has joined #openstack-cinder18:29
*** abhiram_moturi has quit IRC18:33
*** abhiram_moturi has joined #openstack-cinder18:34
*** abhiram_moturi has quit IRC18:38
*** abhiram_moturi has joined #openstack-cinder18:38
*** abhiram_moturi has quit IRC18:44
*** madskier has joined #openstack-cinder18:44
*** rushil has quit IRC18:48
*** juzuluag has quit IRC18:48
*** MentalRay has joined #openstack-cinder18:48
*** ebalduf has quit IRC18:48
*** madskier has quit IRC18:49
*** rushil has joined #openstack-cinder18:49
*** Lee1092 has quit IRC18:51
*** e0ne has joined #openstack-cinder18:51
*** afazekas_ has joined #openstack-cinder18:51
*** Rockyg has joined #openstack-cinder18:53
openstackgerritTom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls  https://review.openstack.org/18239618:57
*** afazekas_ has quit IRC19:03
*** timcl1 has quit IRC19:05
*** merooney has quit IRC19:06
*** merooney has joined #openstack-cinder19:07
jungleboyjsmcginnis: Are we supposed to use ',' instead of '%' for LOG.debug ?19:08
smcginnisjungleboyj: Yep, that is for all logging.19:08
SwansonPlease do.  He made me change all mine.19:08
smcginnisAnd to be fair, it's not a requirement, more of a best practice.19:09
hemnajungleboyj, you aren't the only one still confused about the logging bs.19:09
hemnait's a major source of confusion IMHO19:09
smcginnisIt is.19:09
smcginnisDon't preformat with %, use translation markers, don't force conversion (six.text_type) unless absolutely needed.19:10
*** timcl has joined #openstack-cinder19:10
*** rushil has quit IRC19:10
*** ebalduf has joined #openstack-cinder19:13
*** afazekas_ has joined #openstack-cinder19:13
*** afazekas_ has quit IRC19:15
jungleboyjsmcginnis: Just got an explanatio from Jim Carey as he happened to be here.  You are right.19:15
smcginnisjungleboyj: Cool.19:16
*** afazekas_ has joined #openstack-cinder19:16
*** openstackgerrit_ has joined #openstack-cinder19:16
*** ebalduf has quit IRC19:18
*** aix has quit IRC19:18
* jungleboyj should have known that hemna would jump on that. :-)19:18
*** leeantho has quit IRC19:19
jungleboyjReally we only need the % to force the translation and formatting when it is going to be used in more than one place was the explanation I heard.19:19
*** rushil has joined #openstack-cinder19:19
e0nejungleboyj: 2c from me: we also need % for exception messages. e.g.: raise SomeError(_("Volume %s failed") % vol_id)19:22
*** afazekas_ has quit IRC19:22
smcginnise0ne: Right, if it's not something passed in to the logger then it does need to be formatted like you normally would any other string.19:25
*** openstackgerrit_ has quit IRC19:25
openstackgerritTom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls  https://review.openstack.org/18239619:25
*** IlyaG has joined #openstack-cinder19:30
*** annegentle has joined #openstack-cinder19:31
openstackgerritJohn Griffith proposed openstack/python-cinderclient: Add functional post test_hook  https://review.openstack.org/18242619:31
*** merooney_ has joined #openstack-cinder19:33
*** shakamunyi has quit IRC19:35
*** merooney_ has quit IRC19:35
*** barra204 has quit IRC19:36
*** merooney_ has joined #openstack-cinder19:36
*** merooney has quit IRC19:36
*** marcusvrn has quit IRC19:37
*** markvoelker has joined #openstack-cinder19:45
*** turul_ has joined #openstack-cinder19:46
*** marcusvrn has joined #openstack-cinder19:46
*** bkopilov has quit IRC19:46
jungleboyjsmcginnis: I now have a monster and a box of popcorn ... maybe that will get me through.19:48
jungleboyj;-)19:48
*** timcl has quit IRC19:48
*** timcl has joined #openstack-cinder19:49
*** bkopilov has joined #openstack-cinder19:49
*** afazekas has quit IRC19:50
*** shakamunyi has joined #openstack-cinder19:50
*** barra204 has joined #openstack-cinder19:50
* smcginnis envisions jungleboyj jacked up on caffeine and downing fists full of popcorn20:00
jungleboyj:-)  You got it!20:00
jungleboyjAnd head banging to Cosmic Gate and A State of Trance.20:00
jungleboyjSo glad this is all logged.  Oh the fun my kids will have googling me when I am gone.20:02
jungleboyjWTF, I am not the only jungleboyj out there.20:04
jungleboyjI am the only one with the license plate though.20:05
*** lpetrut has joined #openstack-cinder20:06
*** merooney has joined #openstack-cinder20:06
*** lpetrut has quit IRC20:09
*** merooney_ has quit IRC20:09
jungleboyjsmcginnis: Looks good.20:16
jungleboyjthingee: jgriffith DuncanT hemna Anyone else want to look at this i18n volume change or should I just push it through?  https://review.openstack.org/#/c/164697/20:16
*** IlyaG has quit IRC20:16
*** leeantho has joined #openstack-cinder20:17
hemnaLOG.warn20:17
DuncanTjungleboyj: I've lost interest in it after it went from 150 lines to a monstrosity :-(20:17
e0ne~1500 LoC. may be it will be goot time to read it with morning coffee20:18
jungleboyjDuncanT: Good use of monstrosity.20:19
jungleboyjhemna: ?20:19
hemnayuh20:19
hemnaI'll look through it.20:19
DuncanTIt's nearly midnight here, no way I'm touching it now, sorry20:19
hemnaonly 1500+ lines of changes......20:19
jungleboyjLOG.warning(_LW("Anyone else want to look at this i18n volume change or should I just push it through?  https://review.openstack.org/#/c/164697/"))20:19
jungleboyje0ne: Yikes, what time is it for you then?20:20
e0nejungleboyj: it's 11.20pm now20:21
jungleboyjOk, so similar time zone.20:21
e0nejungleboyj: i can't review such big patches so late:(20:21
jungleboyjNo problem.20:21
jungleboyjWhenever you guys have time. Just giving smcginnis a break from begging.20:22
jungleboyjxyang1: Have you seen the new Avengers movie yet.  Your storage makes a nice Cameo in the movie.20:23
e0nejungleboyj: i'm afraid that we'll heve merge hell with this patch:(20:23
jungleboyjYeah, that is why it is better to get it in sooner than later.20:23
xyang1jungleboyj: No.  I'll have to check it out:)20:24
jungleboyjYeah, there is a big VNX rack right in the middle of one of the big action scenes about 2/3 of the way through the movie.20:24
jungleboyjSomehow, despite everything around it being destroyed it goes untouched.20:24
xyang1jungleboyj: I wonder how much they paid for that20:25
jungleboyjI was wondering that too.20:25
hemnaso20:25
hemnais it really necessary to do _LE("Exception: %s") ?20:26
hemnafrom LOG.error(ex) ?20:26
jungleboyjhemna: I would say that is the better way to do it.20:27
e0neOracle ZFSSA CI checked request that was upsated 4 days ago and merged 3 days ago:20:28
e0ne:(20:28
hemnaman I dunno20:28
jungleboyjhemna: I am verbose though.20:28
hemnathat seems excessive to me20:28
*** timcl has quit IRC20:28
hemnalots of changes in our driver common class simply due to that.20:28
e0nehemna: why not LOG.exception?20:28
hemnae0ne, I'm sure this is all really old code at this point.20:29
hemnaback in the wild west days of Cinder20:29
hemna2+ years ago20:29
e0nehemna: :)20:29
thingeehemna: https://etherpad.openstack.org/p/cinder-standard-capabilities https://plus.google.com/hangouts/_/gvfmckslbtj5rpicupee5g5fkqa20:29
hemnae0ne, but you do have a good point20:29
hemnajungleboyj, why not change those to LOG.exception(ex) then ?20:29
jungleboyjxyang1: Wish I could find a picture of it but google is failing me.20:30
*** markvoelker has quit IRC20:30
jungleboyjLet me look.20:30
e0nehemna, jungleboyj: not LOG.exeption(ex) but LOG.expection(_LE("Something went wrong"))20:30
xyang1jungleboyj: :)20:30
e0nelog.exeption will log traceback in any cace20:30
e0nes/cace/case20:31
thingeeif anyone else wants to talk about standard capabilities https://plus.google.com/hangouts/_/gvfmckslbtj5rpicupee5g5fkqa20:31
jungleboyje0ne: They are raising the exception.20:31
jungleboyjhemna: I suppose those could be done as LOG.exception(_LE("Something went wrong"))20:32
jungleboyjsmcginnis: Thoughts? ^^^20:33
e0nejungleboyj: hm.. are we talking about this https://docs.python.org/2/library/logging.html#logging.Logger.exception?20:33
e0newe should not use log.execption when we raise it, should we?20:33
*** logan2 has quit IRC20:33
jungleboyje0ne: That was what I was wondering.20:34
*** vilobhmm11 has quit IRC20:35
tbarronso you guys are saying do LOG.error when it is followed by a raise rather than LOG.exception ?20:37
*** vilobhmm1 has joined #openstack-cinder20:37
tbarronwhy?20:37
*** openstackgerrit has quit IRC20:37
e0nehemna, jungleboyj: should we change to log.exception in the this_patch_with_1500_changed_lines?20:37
*** openstackgerrit has joined #openstack-cinder20:37
*** vilobhmm1 has quit IRC20:38
jungleboyje0ne: I would vote no.  Get this in and then hemna can change it to a preferred method if they have one.20:38
e0netbarron: becaus log.exception will log traceback20:38
tbarrone0ne: I see, ty20:39
xyang1jungleboyj: What is the verdict? It changed log.error to log.exception in many places and then followed by raise20:40
jungleboyjHold on.20:40
*** mriedem has quit IRC20:44
*** merooney has quit IRC20:45
*** Yogi12 has quit IRC20:46
jgriffithscottda: is this: https://bugs.launchpad.net/cinder/+bug/1454407  Why Horizon keeps balking at me now?20:46
openstackLaunchpad bug 1454407 in Cinder "cinderclient no longer honors --endpoint-type" [Undecided,New] - Assigned to Scott DAngelo (scott-dangelo)20:46
jgriffithscottda: just noticed this afternoon, it constantly throws errors doing "any" cinder'ish stuff20:47
*** MentalRay has quit IRC20:47
jungleboyjxyang1: hemna smcginnis Ok, as I am talking this over with Jim/thinking about it I believe the answer is that it is a stylistic choice.20:48
*** MentalRay has joined #openstack-cinder20:48
*** ronis has quit IRC20:48
jungleboyjYou can do LOG.error(_LE("My Exception: %s")) or LOG.exception(_LE("Crap I hit an exception:"))20:49
*** vilobhmm1 has joined #openstack-cinder20:49
*** sgotliv has quit IRC20:49
xyang1jungleboyj: So if nothing is wrong, why change them everywhere20:49
*** vilobhmm1 has quit IRC20:50
*** vilobhmm1 has joined #openstack-cinder20:50
jungleboyjxyang1: I don't think we did change them everywhere.20:50
scottdajgriffith: I'm not sure about Horizon, I just discovered trying to fix another bug...20:50
xyang1jungleboyj: Lots this kind of changes in the vmax driver20:50
scottdahttps://bugs.launchpad.net/python-cinderclient/+bug/145390120:51
openstackLaunchpad bug 1453901 in python-cinderclient "cinderclient --endpoint-type differs from other services os-endpoint-type" [Undecided,In progress] - Assigned to Scott DAngelo (scott-dangelo)20:51
jgriffithscottda: cool, I'll poke around at it later and see if I can figure out what's up20:51
scottdaIf Horizon tries to set the endpoint using '--endpoint-type', then that would explain it.20:51
jgriffithscottda: indeed!20:51
*** mdbooth has quit IRC20:52
*** e0ne has quit IRC20:52
jungleboyjThe advantage to LOG.exception is you will actually get the stack trace which I think is better.20:52
jgriffithscottda: have a shiny +220:52
jungleboyjInstead of just the exception message xyang120:52
scottdaThe bug's been in a while, commit is from Jan 720:52
jgriffithwould be REALLY cool if somebody else would review that sucker20:52
jgriffithscottda: but remember we didn't push the client to pypi until "fairly" recently20:52
scottdaahhhh..right20:53
jgriffithscottda: and my setup I did a pip install latest20:53
scottdaBut the fix you +2'd is not the fix for this bug...this bug is https://bugs.launchpad.net/cinder/+bug/145440720:53
openstackLaunchpad bug 1454407 in Cinder "cinderclient no longer honors --endpoint-type" [Undecided,New] - Assigned to Scott DAngelo (scott-dangelo)20:53
jgriffithscottda: which to me says "cinderclient may be borked"20:53
xyang1jungleboyj: It changed from log.error to log.exception in many places.  If it is not necessary, why change them20:53
*** georg-k has joined #openstack-cinder20:53
*** e0ne has joined #openstack-cinder20:53
scottdaI'm testing a patch now, just needs a tweak for the unit tests20:53
jungleboyjxyang1: Because then you get the traceback leading up to the exception.20:54
jgriffithscottda: yeah, that's where I started when I saw you post that 145440720:54
jgriffithscottda: AFAIC they should both be added, merged and pushed20:54
*** IlyaG has joined #openstack-cinder20:55
xyang1jungleboyj: You were just saying earlier it is a style choice20:55
xyang1jungleboyj: If this is preferred, should we use log.exception from now on20:55
*** Longgeek has quit IRC20:56
jungleboyjxyang1: Well, I am talking to Jim Carey and he noted the traceback advantage.  So, it depends on if you want a traceback in that instance or not.20:56
*** mdbooth has joined #openstack-cinder20:58
smcginnisSorry, was in a meeting and just got back. Looking through the traceback.20:58
smcginnisAgree with jungleboyj's last statement.20:58
smcginnisThe only thing to actually avoid is LOG.exception(ex).20:58
smcginnisThat's incorrect usage IMO.20:58
xyang1smcginnis: ?20:59
smcginnisLOG.exception logs the exception information including the traceback.20:59
openstackgerritAngela Smith proposed openstack/cinder-specs: Spec for FCZM Zone Driver Friendly Zone names  https://review.openstack.org/14438420:59
jungleboyjYeah, that is the wrong way.20:59
smcginnisAnd you're passing in ex to the logger which will convert it to a string.20:59
smcginnisSo you end up logging the exception twice.20:59
xyang1But it raises again, that will double log the trace20:59
smcginnisEither LOG.error(ex) or LOG.exception(_LE('Bad juju'))20:59
smcginnisMy preference being the latter.21:00
smcginnisxyang1: Good point. But do we know where it raised up to and gets catched takes care of logging it?21:00
smcginnisIt might not.21:00
xyang1In lots of places, it does log.exception, then raise21:01
smcginnisTo be clear, this one isn't my patch. Just think it's big and never a good time for something like this so I'd like to see it go through like jungleboyj siad.21:01
smcginnisxyang1: That could be fine.21:01
*** vilobhmm1 has quit IRC21:01
* e0ne needs to take a break to sleep. bb tomorrow21:01
smcginnisxyang1: But if one level up it just get caught and logged there then maybe not.21:01
smcginnisxyang1: My point being we don't know at that level what is happening above is.21:02
*** vilobhmm1 has joined #openstack-cinder21:02
smcginnis/us/21:02
xyang1That will show trace twice, and redundant info21:02
smcginnise0ne: Good night!21:02
xyang1We got those comments sometimes too21:02
*** MentalRay has quit IRC21:02
smcginnisxyang1: If it's being logged.21:02
smcginnisxyang1: Where it's caught.21:03
*** e0ne has quit IRC21:03
*** vilobhmm1 has quit IRC21:03
smcginnisxyang1: I do agree it's a concern. But honestly, I'm getting tired of logging and just want it to go away. :)21:03
*** vilobhmm1 has joined #openstack-cinder21:04
xyang1smcginnis: Agree:)21:04
*** thangp has quit IRC21:05
xyang1smcginnis: I basically think this will create merge nightmare without any gain, it changed from log.error(ex) to log.exception21:06
*** mriedem has joined #openstack-cinder21:07
smcginnisxyang1: This one touches a lot of files, so I agree it will cause a lot of merge problems. I don't think there's any good time to do this though, and aside from the LOG.exception pieces I think this does fix some very valid concerns.21:07
smcginnisxyang1: I'd rather take the hit now and get it done. :/21:08
*** Yogi1 has joined #openstack-cinder21:09
*** annegentle has quit IRC21:10
*** akerr has quit IRC21:11
patrickeastjgriffith: hey, if you have a few min I have some questions about your drivers image cache feature21:13
*** MentalRay has joined #openstack-cinder21:13
openstackgerritxing-yang proposed openstack/cinder: Fix a problem with FAST support in VMAX driver  https://review.openstack.org/17000021:14
*** bswartz has quit IRC21:15
openstackgerritMike Perez proposed openstack/cinder: Remove v1 server support  https://review.openstack.org/14570321:19
jgriffithpatrickeast: sure, what's up?21:19
patrickeastjgriffith: so i’m looking into adding something like that into my driver, and wanted to get your thoughts on whether or not i should make more of a common solution or basically just port what you have into mine21:21
*** jwcroppe_ has joined #openstack-cinder21:21
smcginnispatrickeast, jgriffith: That's on my list to look in to at some point as well.21:21
patrickeastjgriffith: i figured a common solution would give you a chance to change anything you didn’t like about how it was implemented right now21:21
jgriffithpatrickeast: so something "common" would be neat... but there are some reasons I didn't do that to beginwith21:21
smcginnisNot sure if we can make it a generic service, but that would be great if we could.21:21
patrickeasthehe yea my guess was it wouldn’t be easy21:21
jgriffithpatrickeast: if we wanted to make an actual resource that might be a reasonable idea21:22
patrickeastmaybe by common, more like helper methods21:22
smcginnis+121:22
jgriffithbut the only reason it *works* in my case is dedupe, cloning etc21:22
patrickeastyea i dunno how many backends would want to take advantage of it21:22
*** jwcroppe has quit IRC21:22
patrickeastyea for the pure backends its a huge win21:22
jgriffithpatrickeast: right21:22
*** primechuck has quit IRC21:23
jgriffithpatrickeast: the only other idea was an internal volume/snap that was owned by "cinder"21:23
jgriffiththat could then be used for create-from-snap or volume depending on what was best for Vendor-x21:23
jgriffithpatrickeast: so that would do the same thing but make it "official'ish" and create a db resource for it etc21:23
patrickeastjgriffith: so that volume would be created as a side effect of creating from an image? or a new thing like a sharable ‘golden’ volume kind of thing?21:24
jgriffithpatrickeast: actually, I'm happy to look at writing a bp and implementing it21:24
jgriffithpatrickeast: correct21:24
patrickeasthaha alright21:24
jgriffithpatrickeast: oh... wait...21:24
jgriffith:)21:24
*** chlong has quit IRC21:24
jgriffithwhat am I saying21:25
jgriffithLOL21:25
*** MentalRay has quit IRC21:25
patrickeastjgriffith: oh, somewhat related , i think there is a bug in your implementation now… dunno if it actually breaks anything or is just dead code https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/solidfire.py#L61121:27
patrickeastjgriffith: that params variable is just overwritten, right?21:27
jgriffithLOL21:27
jgriffithlemme look21:27
jgriffithwell, it's needed for the delete call21:28
jgriffithso if the image-volume exists, but it's out of date21:28
jgriffithwe need to go in and delete it21:28
jgriffithin order to do that we need to use the params21:29
patrickeastjgriffith: but on line 610 there is a params, and on 611 it is set to something else?21:29
jgriffithhaha21:29
patrickeasti assume its a benign issue21:29
jgriffithoh, yeah, should be params['volumeID'] =21:29
patrickeastjust something i noticed when looking at how it worked21:29
jgriffithpatrickeast: it is but still I should fix it up21:29
*** rushil has quit IRC21:30
*** emagana has quit IRC21:30
jgriffithpatrickeast: the code path is kinda tricky in all of that21:30
jgriffithpatrickeast: I ran into a few weird corner cases21:30
openstackgerritxing-yang proposed openstack/cinder: EMC VMAX Manage/Unmanage Volume  https://review.openstack.org/18246021:30
*** emagana has joined #openstack-cinder21:30
jgriffithprobably should've written them down :)21:30
patrickeastjgriffith: i was going to ask how you handled race conditions and prevented creating two entries in teh cache for the same image, or if thats tackled elsewhere higher up the stack21:31
*** MentalRay has joined #openstack-cinder21:31
patrickeastor maybe it doesn’t really matter21:31
jgriffithpatrickeast: hasn't been an issue, because the create is my "flag"21:32
*** rushil has joined #openstack-cinder21:32
*** Yogi1 has quit IRC21:32
jgriffithpatrickeast: there's certainly a potential there21:33
jgriffithpatrickeast: as it works right now it doesn't "care" if there's a duplicate21:34
jgriffithpatrickeast: I'd rather "not care" than try and make sure it doesn't happen21:35
jgriffithpatrickeast: since it'll do dedupe to *nothing* no harm for the most part21:35
patrickeastjgriffith: gotcha, thats was my assumption, seems like a better way than some locks or something21:35
patrickeasthehe yea exactly21:35
jgriffithpatrickeast: I'm pretty "anti-lock" if you haven't gathered :)21:37
jgriffithpatrickeast: I think they're over-used21:38
patrickeastjgriffith: definitely always better to avoid them if possible21:39
openstackgerritScott DAngelo proposed openstack/python-cinderclient: cinderclient no longer honors --endpoint-type  https://review.openstack.org/18246521:40
patrickeastjgriffith: so for the common cache type thing, you mentioned doing the bp and implementation, if your plate is full with other stuff let me know and i can spend some time on it21:41
openstackgerritxing-yang proposed openstack/cinder: EMC VMAX Manage/Unmanage Volume  https://review.openstack.org/18246021:41
jgriffithpatrickeast: sure21:41
jgriffithpatrickeast: if you were going to do it this week go for it :)21:42
patrickeastjgriffith: hehe this week? i could scrap together a bp and spec np… the impl would probably be a little bit later21:42
jgriffithewww... spec21:43
patrickeastor does it not need a spec?21:43
jgriffithif you ask me no :)21:43
patrickeastwell that makes it easier21:43
jgriffithbut if you ask anybody else probably21:43
patrickeastlol21:43
*** crose has quit IRC21:44
patrickeastso… if i write the code you’ll ninja +2/A? no spec, no problem?21:44
jgriffithpatrickeast: haah... I got in trouble for that21:44
jgriffithjust yesterday in fact21:44
jgriffithand it was only a one line patch with 6 +1's on it21:44
jgriffithso the answer is "no" I won't :)21:44
patrickeastthat seems weird21:44
patrickeastok ok21:44
patrickeastspec it is21:44
jgriffithsad but true21:45
openstackgerritJohn Griffith proposed openstack/cinder: Fix overwrite of params in SF image cache update  https://review.openstack.org/18247121:50
*** annegentle has joined #openstack-cinder21:50
jgriffithpatrickeast: thanks for pointing that out by the way21:50
patrickeastjgriffith: np21:50
*** ociuhandu has quit IRC21:51
thingee"solidfire craft brews" ??21:53
thingeejgriffith: ^21:53
thingeenice21:53
jgriffiththingee: what's that?21:53
jgriffiththingee: oh... cool!21:53
thingeejgriffith: it was in the "OpenStack Summit Vancouver Pre and Preview" email21:53
thingeejgriffith: I'm happy to hear you're all making beer in Colorado too ;)21:54
jgriffithOh doood21:55
jgriffithThere are TONS of breweries here, and one of our lead biz guys runs a popular brewery up the street in his "spare" time21:55
jgriffithas in... "he doesn't sleep, EVER"21:55
thingeejgriffith: I know! fort collins was awesome21:56
* thingee kind of hopes the next midcycle meet up is back in colorado ;)21:56
* thingee and hopes to still get bbq from jgriffith's farm21:56
jgriffiththingee: oh yeah!!  For sure this time!!22:00
*** vilobhmm1 has quit IRC22:00
* jgriffith eyes the pigs.... "who's it gonna be"22:01
*** jungleboyj has quit IRC22:01
jgriffithjust kidding22:01
openstackgerritMitsuhiro Tanino proposed openstack/cinder: LVM: Support efficient data copy using "dd" for create_cloned_volume  https://review.openstack.org/18247322:01
jgriffithmtanino: Nice!22:02
mtaninojgriffith: thanks :)22:02
jgriffithmtanino: honestly I didn't know if that would work with images or not22:02
jgriffithmtanino: that's awesome!22:02
jgriffithmtanino: I think we need to figure out how to push thin as the default in Liberty22:03
jgriffithmtanino: maybe I'll spin up a CI so we get some mileage on it22:03
*** georg-k has quit IRC22:04
mtaninojgriffith: we need to discuss on the sprint?22:04
jgriffithmtanino: sprint, or summit next week?22:04
*** vilobhmm1 has joined #openstack-cinder22:04
jgriffithmtanino: I'd say summit next week personally22:04
*** vilobhmm1 has quit IRC22:05
mtaninojgriffith: I thought the sprint is friday of the summit,22:05
*** vilobhmm1 has joined #openstack-cinder22:05
jgriffithmtanino: OHHHH that sprint!22:05
jgriffithLOL.. yes!22:05
mtaninoyeah!22:05
jgriffithI thought you meant mid-cycle spring22:05
jgriffithsprint22:06
jgriffithyes... Friday for sure!22:06
mtaninoah, I got it.22:06
*** vilobhmm1 has quit IRC22:07
tsekiyamamtanino: jgriffith: for image-to-volume case, qemu-img has -S (sparse_size) option, but it actually doesn't write sparsely to the LVM thin volume..22:10
*** rushil has quit IRC22:10
tsekiyamaso the volume is allocated 100%.22:10
*** vilobhmm11 has joined #openstack-cinder22:10
jgriffithtsekiyama: it's the "dd" that's the problem22:10
jgriffithtsekiyama: because we can't convert directly from the qcow to the disk22:10
jgriffithtsekiyama: remember it does qcow--->raw--->cinder-volume22:10
tsekiyamajgriffith:  yeah in that case it is no problem22:10
tsekiyamajgriffith: I meant qemu-img would be problematic when creating a new volume from Glance image22:11
jgriffithtsekiyama: ahhh22:13
*** jaypipes has quit IRC22:13
*** logan2 has joined #openstack-cinder22:18
tsekiyamajgriffith: I confirmed that "qemu-img convert" is directly writing to the cinder-volume, at least with LVM driver..22:21
*** markvoelker has joined #openstack-cinder22:24
*** Rockyg has quit IRC22:24
jgriffithtsekiyama: ?22:26
*** dims_ has joined #openstack-cinder22:28
jgriffithOh, you're talking about create from image22:28
jgriffithnot clone22:28
jgriffithsorry... I was mixing the two up22:28
tsekiyamajgriffith: right22:28
*** esker has quit IRC22:29
jgriffithtsekiyama: so you're saying that sparse option in qemu-convert still doesn't work22:29
tsekiyamajgriffith: exactly.22:29
jgriffithtsekiyama: ok... sorry I was confused22:29
jgriffithtsekiyama: yeah, I had hoped that had been fixed22:29
jgriffithtsekiyama: always wondered if we could do somethign to get around it... besides caching22:29
jgriffithtsekiyama: but even optimizations there might be good...22:30
jgriffithtsekiyama: like "create the smallest size volume possible" copy the image then extend it22:30
*** dims has quit IRC22:30
jgriffithtsekiyama: so rather than working with say a 10G volume for a base ubuntu image you work with something like 250MB's22:31
jgriffithI think that would sure help22:31
*** MentalRay has quit IRC22:31
tsekiyamajgriffith: ah, that might work, I think22:31
jgriffithtsekiyama: so the trick is you have to have virt-size set on your image properties22:31
jgriffithwhich by default it is not22:32
jgriffithtsekiyama: anyway, I think it would be a worthwhile optimization22:32
jgriffithtsekiyama: that's what I put in the SolidFire driver FWIW22:32
jgriffithmade a significant improvement22:33
tsekiyamajgriffith: Interesting. thank you for the information22:33
jgriffithtsekiyama: sure22:33
jgriffithtsekiyama: let me know how it goes if you decide to work on it22:33
jgriffithtsekiyama: between that and the caching stuff patrickeast brought up I may just offload all of my work for Liberty to other people :)22:34
tsekiyamajgriffith: sure. I'll try looking into solidfire trick22:34
tsekiyama:)22:34
patrickeastjgriffith: as long as you keep working on replication thats ok with me :D22:34
jgriffithtsekiyama: cool, and don't hesitate to ask, it's pretty easy though22:34
jgriffithpatrickeast: speaking of which... I'm convinced you're all trying to just 'spec' me to death22:34
tsekiyamajgriffith: ok, thanks!22:34
jgriffith:)22:34
patrickeasthaha22:35
*** IlyaG has quit IRC22:36
*** IlyaG has joined #openstack-cinder22:38
*** IlyaG has quit IRC22:43
thingeejgriffith: do you still got the cinder logo handy? Product WG is asking for it in there slides at the conference.22:44
thingeeI've been doing a mass search on my damn slow home nas and not finding it22:44
openstackgerritScott DAngelo proposed openstack/python-cinderclient: cinderclient deprecated endpoint_type needs dest=  https://review.openstack.org/18248622:46
*** IlyaG has joined #openstack-cinder22:47
jgriffiththingee: errr... might22:47
jgriffiththingee: it's probably in a backup somewhere22:47
thingeejgriffith: lol that's what I said22:48
thingeeyour backup is as good as mine :P22:48
*** vilobhmm11 has quit IRC22:49
*** vilobhmm1 has joined #openstack-cinder22:49
openstackgerritScott DAngelo proposed openstack/python-cinderclient: cinderclient deprecated endpoint_type needs dest=  https://review.openstack.org/18248622:49
scottdaThat seems like a recipe for a merge issue...22:50
*** primechuck has joined #openstack-cinder22:55
kmartinthingee, is it the openstack logo with the cinder brick behind it22:59
thingeekmartin: it was just the cinder brick logo23:00
kmartinthingee ok, jungleboyj has one in his email signature but it's not that one23:01
*** emagana has quit IRC23:06
*** markvoelker has quit IRC23:08
*** chlong has joined #openstack-cinder23:15
*** esker has joined #openstack-cinder23:16
*** annegentle has quit IRC23:17
*** esker has quit IRC23:22
*** bswartz has joined #openstack-cinder23:33
*** akerr has joined #openstack-cinder23:36
*** akerr_ has joined #openstack-cinder23:38
*** akerr has quit IRC23:41
*** zhenguo has joined #openstack-cinder23:43
*** hodos has quit IRC23:43
*** garthb has quit IRC23:45
*** IlyaG has quit IRC23:55

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