Monday, 2015-03-02

*** delatte has joined #openstack-cinder00:07
*** delattec has quit IRC00:10
*** ebalduf has joined #openstack-cinder00:17
*** ebalduf has quit IRC00:21
*** dannywilson has joined #openstack-cinder00:46
*** ho has joined #openstack-cinder00:48
*** dannywilson has quit IRC00:51
*** Lee1092 has joined #openstack-cinder01:00
*** vilobhmm1 has joined #openstack-cinder01:01
openstackgerritThang Pham proposed openstack/cinder: Raise exception for invalid mock assert calls  https://review.openstack.org/16020801:02
*** vilobhmm has quit IRC01:05
*** davechen has joined #openstack-cinder01:07
openstackgerritThang Pham proposed openstack/cinder: Snapshot and volume objects  https://review.openstack.org/13356601:13
openstackgerritThang Pham proposed openstack/cinder: Cinder objects base  https://review.openstack.org/13187301:13
openstackgerritThang Pham proposed openstack/cinder: WIP: Switch to oslo_versionedobjects  https://review.openstack.org/16020901:13
*** ebalduf has joined #openstack-cinder01:18
*** mriedem has quit IRC01:18
*** vilobhmm1 has quit IRC01:22
*** ebalduf has quit IRC01:22
*** akerr_away is now known as akerr01:30
*** scottda_ has joined #openstack-cinder01:38
*** scottda_ has quit IRC01:45
*** kaufer has joined #openstack-cinder01:49
*** scottda_ has joined #openstack-cinder02:04
*** davechen_ has joined #openstack-cinder02:05
*** diegows has quit IRC02:07
*** davechen has quit IRC02:08
*** scottda_ has quit IRC02:10
*** scottda_ has joined #openstack-cinder02:11
openstackgerritwuyuting proposed openstack/python-cinderclient: Make cinderclient support snapshot-force-delete  https://review.openstack.org/15314802:15
*** ebalduf has joined #openstack-cinder02:18
*** scottda_ has quit IRC02:21
*** kaufer has quit IRC02:22
*** ebalduf has quit IRC02:23
*** setuid has quit IRC02:27
*** setuid has joined #openstack-cinder02:28
*** kaufer has joined #openstack-cinder02:31
*** patri962 has joined #openstack-cinder02:38
*** patri674 has joined #openstack-cinder02:38
*** patri824 has joined #openstack-cinder02:38
*** patri659 has joined #openstack-cinder02:38
*** patrickeast___ has quit IRC02:38
*** patrickeast has quit IRC02:38
*** patrickeast__ has quit IRC02:38
*** patrickeast_ has quit IRC02:39
*** patri28 has quit IRC02:39
*** patrickeast has joined #openstack-cinder02:41
*** Yogi1 has joined #openstack-cinder02:42
*** junhongl has joined #openstack-cinder02:42
*** DaveChen has joined #openstack-cinder02:51
*** davechen_ has quit IRC02:51
*** davechen_ has joined #openstack-cinder02:59
*** jseiler_ has quit IRC03:00
*** jseiler_ has joined #openstack-cinder03:00
*** DaveChen has quit IRC03:02
*** _cjones_ has joined #openstack-cinder03:14
*** ebalduf has joined #openstack-cinder03:19
*** ebalduf has quit IRC03:23
*** Yogi1 has quit IRC03:46
*** _cjones_ has quit IRC03:51
openstackgerritxing-yang proposed openstack/cinder: Create Consistency Group from CG Snapshot API  https://review.openstack.org/14595203:53
*** david-lyle_afk has quit IRC04:02
*** pradipm has joined #openstack-cinder04:05
*** akerr has quit IRC04:10
*** akerr has joined #openstack-cinder04:11
*** ebalduf has joined #openstack-cinder04:20
*** ebalduf has quit IRC04:24
*** patrickeast_ has joined #openstack-cinder04:27
*** patrickeast has quit IRC04:27
*** patrickeast_ is now known as patrickeast04:27
*** Mandell has joined #openstack-cinder04:27
*** patri659 has quit IRC04:28
*** patri962 has quit IRC04:29
*** patri824 has quit IRC04:29
*** patri674 has quit IRC04:29
*** patrickeast_ has joined #openstack-cinder04:30
*** patrickeast__ has joined #openstack-cinder04:30
*** patri484 has joined #openstack-cinder04:30
*** patri533 has joined #openstack-cinder04:30
*** kaufer has quit IRC04:34
*** BharatK has joined #openstack-cinder04:34
*** coolsvap_ is now known as coolsvap04:43
*** fifieldt has joined #openstack-cinder04:44
*** jwcroppe has joined #openstack-cinder04:49
*** kaisers has joined #openstack-cinder04:51
openstackgerritYAMADA Hideki proposed openstack/cinder: Fix missing clone_image API support for sheepdog driver.  https://review.openstack.org/15985605:07
*** ebalduf has joined #openstack-cinder05:21
*** avishay has joined #openstack-cinder05:21
*** ankit_ag has joined #openstack-cinder05:21
*** patri338 has joined #openstack-cinder05:23
*** patri781 has joined #openstack-cinder05:23
*** patri996 has joined #openstack-cinder05:23
*** patri952 has joined #openstack-cinder05:23
*** patrickeast has quit IRC05:23
*** patrickeast_ has quit IRC05:24
*** patri484 has quit IRC05:24
*** patri533 has quit IRC05:24
*** patrickeast__ has quit IRC05:25
*** ebalduf has quit IRC05:26
*** patrickeast has joined #openstack-cinder05:27
*** BharatK has quit IRC05:39
*** anuragpalsule has joined #openstack-cinder05:44
*** akerr is now known as akerr_away05:50
*** BharatK has joined #openstack-cinder06:01
*** jcru has joined #openstack-cinder06:10
*** deepakcs has joined #openstack-cinder06:12
*** ebalduf has joined #openstack-cinder06:21
*** ebalduf has quit IRC06:26
*** nkrinner has joined #openstack-cinder06:27
*** anshul has joined #openstack-cinder06:27
*** tshefi has joined #openstack-cinder06:35
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex  https://review.openstack.org/15240106:40
*** jcru has quit IRC06:46
openstackgerritwanghao proposed openstack/cinder: Fix the unicode encode error when create volume  https://review.openstack.org/15970706:47
*** _cjones_ has joined #openstack-cinder06:51
openstackgerritwanghao proposed openstack/cinder: Add config option to override url for versions  https://review.openstack.org/15937406:52
*** _cjones_ has quit IRC06:56
openstackgerritrakesh mishra proposed openstack/cinder: Implement making volume snapshots public  https://review.openstack.org/15937206:57
*** abhijeetm has joined #openstack-cinder07:05
*** sgotliv has quit IRC07:11
*** takedakn has joined #openstack-cinder07:13
*** ebalduf has joined #openstack-cinder07:22
*** ebalduf has quit IRC07:27
openstackgerritAbhijeet Malawade proposed openstack/cinder: Get volume from db again before updating it's status  https://review.openstack.org/14440907:32
*** Maike has joined #openstack-cinder07:35
*** Ilja has joined #openstack-cinder07:41
*** Miouge has joined #openstack-cinder07:45
*** jcru has joined #openstack-cinder07:47
*** jcru has quit IRC07:52
*** thingee has joined #openstack-cinder07:52
thingeeDuncanT: can you please sign off on https://review.openstack.org/#/c/131873/ and https://review.openstack.org/#/c/133566/07:53
thingeewe really need to get these merged for gate testing07:54
thingeeDuncanT: xing has already mentioned she's fine with them as well07:54
*** chlong has quit IRC08:02
DuncanTThingee: I'll be back at my desk in an hour or so, will look at them then08:07
thingeeDuncanT: thank you!08:08
thingeeDuncanT: I've already started going through, but march 1st deadline is now past. http://lists.openstack.org/pipermail/openstack-dev/2015-February/056964.html08:18
*** nellysmitt has joined #openstack-cinder08:18
thingeecode not ready for bps is not getting in08:18
*** afazekas has joined #openstack-cinder08:18
*** Guest78669 is now known as d0ugal08:20
*** d0ugal has joined #openstack-cinder08:21
openstackgerritLi Yingjun proposed openstack/python-cinderclient: Fixes quota-class-update commands  https://review.openstack.org/16025308:21
*** ebalduf has joined #openstack-cinder08:23
*** sgotliv has joined #openstack-cinder08:24
*** ebalduf has quit IRC08:27
*** ronis has joined #openstack-cinder08:36
*** nshaikh has joined #openstack-cinder08:37
*** sgotliv has quit IRC08:38
*** sgotliv has joined #openstack-cinder08:38
*** pradipm has quit IRC08:42
*** ronis has quit IRC08:53
*** pwehrle has joined #openstack-cinder08:53
*** bkopilov has quit IRC08:59
*** bkopilov has joined #openstack-cinder08:59
thingeeDuncanT: ooo it looks jgriffith gave a +2 on the last one :)09:04
thingeethis makes me very happy09:04
thingeeok I'm going to get some sleep. catch ya'll later09:05
*** thingee has quit IRC09:05
*** pradipm has joined #openstack-cinder09:08
*** jistr has joined #openstack-cinder09:12
*** dulek has joined #openstack-cinder09:13
*** alecv has joined #openstack-cinder09:18
*** pradipm has quit IRC09:21
*** pradipm has joined #openstack-cinder09:23
*** yuriy_n17 has joined #openstack-cinder09:23
*** ebalduf has joined #openstack-cinder09:24
openstackgerritYuriy Nesenenko proposed openstack/cinder: Change datetime.now() to datetime.utcnow()  https://review.openstack.org/15985409:26
*** ebalduf has quit IRC09:28
*** ronis has joined #openstack-cinder09:34
*** takedakn has quit IRC09:39
*** ronis has quit IRC09:40
*** Maike has quit IRC09:43
*** Maike has joined #openstack-cinder09:43
*** e0ne has joined #openstack-cinder09:49
openstackgerritDave Chen proposed openstack/cinder: Role based properties protection  https://review.openstack.org/16009009:54
*** davechen_ has quit IRC09:55
*** Miouge has quit IRC10:00
*** Miouge has joined #openstack-cinder10:04
*** jordanP has joined #openstack-cinder10:05
*** bkopilov has quit IRC10:11
*** Miouge has quit IRC10:12
*** Miouge has joined #openstack-cinder10:14
*** bkopilov has joined #openstack-cinder10:16
*** tshefi has quit IRC10:20
*** TobiasE has joined #openstack-cinder10:21
*** Mandell has quit IRC10:22
*** tshefi has joined #openstack-cinder10:23
*** ebalduf has joined #openstack-cinder10:25
*** lpetrut has joined #openstack-cinder10:27
*** marcusvrn has joined #openstack-cinder10:28
*** lpetrut has quit IRC10:28
*** ebalduf has quit IRC10:29
*** bkopilov has quit IRC10:29
*** bkopilov has joined #openstack-cinder10:30
*** marcusvrn1 has quit IRC10:32
*** akerr_away has quit IRC10:40
*** akerr has joined #openstack-cinder10:41
*** aix has joined #openstack-cinder10:41
*** bkopilov has quit IRC10:46
*** TobiasE has quit IRC10:49
*** ronis has joined #openstack-cinder10:53
*** EmilienM is now known as EmilienM|afk10:54
openstackgerritDave Chen proposed openstack/cinder: Role based properties protection  https://review.openstack.org/16009010:58
openstackgerritDave Chen proposed openstack/cinder: Policies based properties protection  https://review.openstack.org/16014511:01
*** Ilja has quit IRC11:01
*** Ilja has joined #openstack-cinder11:02
*** sgotliv has quit IRC11:14
openstackgerritRonen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend  https://review.openstack.org/15396911:20
*** lpetrut has joined #openstack-cinder11:24
*** ho has quit IRC11:25
*** ebalduf has joined #openstack-cinder11:25
*** sgotliv has joined #openstack-cinder11:27
openstackgerritMarc Koderer proposed openstack/cinder: Port rbd driver to use new driver model  https://review.openstack.org/13866111:29
*** ebalduf has quit IRC11:30
openstackgerritMarc Koderer proposed openstack/cinder: Adapt volume manager to new driver structure  https://review.openstack.org/13899911:31
*** tbarron has joined #openstack-cinder11:31
*** kallebe has joined #openstack-cinder11:35
*** ronis has quit IRC11:35
*** Miouge has quit IRC11:38
*** TobiasE has joined #openstack-cinder11:41
*** sgotliv_ has joined #openstack-cinder11:45
*** sgotliv has quit IRC11:48
*** avishay has quit IRC11:59
*** avishay_ has joined #openstack-cinder11:59
*** e0ne is now known as e0ne_12:03
*** coolsvap is now known as coolsvap_12:06
*** pradipm has quit IRC12:09
*** avishay_ has quit IRC12:09
*** avishay__ has joined #openstack-cinder12:10
*** ronis has joined #openstack-cinder12:10
*** pradipm has joined #openstack-cinder12:11
*** takedakn has joined #openstack-cinder12:12
*** e0ne_ has quit IRC12:13
*** coolsvap_ is now known as coolsvap12:15
*** pradipm has quit IRC12:17
*** pradipm has joined #openstack-cinder12:17
*** ronis has quit IRC12:20
*** pradipm has quit IRC12:20
*** pradipm has joined #openstack-cinder12:20
*** lpetrut has quit IRC12:20
*** nshaikh has quit IRC12:23
*** Miouge has joined #openstack-cinder12:24
*** BharatK has quit IRC12:24
*** ebalduf has joined #openstack-cinder12:26
*** e0ne has joined #openstack-cinder12:27
*** abhijeetm has quit IRC12:30
*** ebalduf has quit IRC12:30
*** IanGovett has joined #openstack-cinder12:33
*** abhijeetm has joined #openstack-cinder12:37
*** diegows has joined #openstack-cinder12:41
*** bkopilov has joined #openstack-cinder12:43
*** Miouge has quit IRC12:44
*** delattec has joined #openstack-cinder12:46
*** sgotliv_ has quit IRC12:48
*** delatte has quit IRC12:49
*** ndipanov has joined #openstack-cinder12:49
*** sgotliv has joined #openstack-cinder12:51
*** Ilja has quit IRC12:51
*** akerr has quit IRC12:52
*** ganso_ has joined #openstack-cinder12:52
*** deepakcs has quit IRC12:57
*** dulek_ has joined #openstack-cinder12:57
*** dulek has quit IRC12:57
openstackgerritAbhijeet Malawade proposed openstack/cinder: Get volume from db again before updating it's status  https://review.openstack.org/14440912:59
*** ronis has joined #openstack-cinder13:03
*** tbarron has quit IRC13:04
*** kaisers has quit IRC13:04
*** tbarron has joined #openstack-cinder13:04
*** kaisers has joined #openstack-cinder13:07
*** markvoelker has joined #openstack-cinder13:08
*** esker has quit IRC13:10
openstackgerritRonen Mesonzhnik proposed openstack/cinder: Provide backup service that uses ProtecTIER as the backend  https://review.openstack.org/15396913:11
*** avishay__ has quit IRC13:14
*** setuid has quit IRC13:16
*** setuid has joined #openstack-cinder13:17
*** thrawn01 has quit IRC13:20
*** e0ne is now known as e0ne_13:20
*** e0ne_ is now known as e0ne13:24
*** bswartz has quit IRC13:25
*** ebalduf has joined #openstack-cinder13:27
*** thrawn01 has joined #openstack-cinder13:31
*** frankm has quit IRC13:31
*** ebalduf has quit IRC13:31
*** timcl has joined #openstack-cinder13:32
*** akerr has joined #openstack-cinder13:35
*** tbarron has quit IRC13:37
*** thrawn01 has quit IRC13:41
*** Ilja has joined #openstack-cinder13:41
*** marcusvrn1 has joined #openstack-cinder13:46
*** marcusvrn has quit IRC13:48
*** julim has joined #openstack-cinder13:49
*** marcusvrn has joined #openstack-cinder13:50
openstackgerritMarc Koderer proposed openstack/cinder: Add script to list support matrix  https://review.openstack.org/16034613:50
*** thrawn01 has joined #openstack-cinder13:51
*** marcusvrn1 has quit IRC13:53
*** thrawn01 has quit IRC13:57
*** thrawn01 has joined #openstack-cinder13:59
*** nshaikh has joined #openstack-cinder13:59
*** bswartz has joined #openstack-cinder14:00
*** anuragpalsule has quit IRC14:04
*** afazekas has quit IRC14:06
*** dustins has joined #openstack-cinder14:09
*** mriedem has joined #openstack-cinder14:10
*** mriedem has quit IRC14:11
*** mriedem has joined #openstack-cinder14:11
*** Yogi1 has joined #openstack-cinder14:12
*** dulek_ has quit IRC14:14
*** nshaikh has quit IRC14:14
*** dulek has joined #openstack-cinder14:14
*** kaufer has joined #openstack-cinder14:15
*** abhijeetm has quit IRC14:23
*** afazekas has joined #openstack-cinder14:23
*** ebalduf has joined #openstack-cinder14:28
*** ebalduf has quit IRC14:32
*** xek has joined #openstack-cinder14:33
*** takedakn has quit IRC14:36
*** jungleboyj has quit IRC14:37
*** afazekas has quit IRC14:41
*** nikesh_vedams has joined #openstack-cinder14:46
nikesh_vedamshi14:46
e0nehi14:46
*** eharney has joined #openstack-cinder14:49
nikesh_vedamshttp://paste.openstack.org/show/185118/ http://paste.openstack.org/show/185123/14:49
*** aviramb has joined #openstack-cinder14:51
*** dulek has quit IRC14:52
*** thangp has joined #openstack-cinder14:55
nikesh_vedamsi wanted to provide host-assited volume migration. i wrote code for that.14:56
nikesh_vedamsBut since volume migration works for backends with same volume type. I want to do retype in my cinder driver.14:57
nikesh_vedamsI donot want to retype like this "cinder retype --migration-policy on-demand dothill_vol dothill_realstor2" becuase in this case my migrate_volume function will not be called. So need some help to write a retype function14:57
*** dulek has joined #openstack-cinder14:58
nikesh_vedamsi am repeating this question in IRC because i am confused so please pardon me14:59
DuncanTnikesh_vedams: Create two types that both point to your backend15:03
DuncanTnikesh_vedams: "cinder retype <vol id> <type 2>" should call your driver15:03
*** timcl1 has joined #openstack-cinder15:03
*** timcl has quit IRC15:07
*** ebalduf has joined #openstack-cinder15:10
*** sigmavirus24_awa is now known as sigmavirus2415:11
nikesh_vedamsDuncanT:  http://paste.openstack.org/show/185118/ http://paste.openstack.org/show/185123/15:12
DuncanTnikesh_vedams: I don't know that driver assisted retype works for retype between two different backends. Create two types on the same backend and try that15:13
*** mtanino has joined #openstack-cinder15:14
*** tbarron has joined #openstack-cinder15:17
*** timcl1 has quit IRC15:18
*** afazekas has joined #openstack-cinder15:19
*** rushil has joined #openstack-cinder15:21
*** asselin_ has joined #openstack-cinder15:22
*** timcl has joined #openstack-cinder15:23
*** jungleboyj has joined #openstack-cinder15:23
*** kallebe has quit IRC15:25
*** jcru has joined #openstack-cinder15:26
*** e0ne is now known as e0ne_15:28
*** e0ne_ is now known as e0ne15:28
*** jcru has quit IRC15:28
*** jcru has joined #openstack-cinder15:29
openstackgerritLi Yingjun proposed openstack/python-cinderclient: Fixes quota-class-update commands  https://review.openstack.org/16025315:29
*** setuid has left #openstack-cinder15:29
*** annegentle has joined #openstack-cinder15:31
*** pradipm has quit IRC15:31
*** beekneemech is now known as bnemec15:32
*** rmesta has joined #openstack-cinder15:36
openstackgerritYuriy Nesenenko proposed openstack/cinder: Change datetime.now() to datetime.utcnow()  https://review.openstack.org/15985415:36
*** dulek has quit IRC15:38
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator  https://review.openstack.org/15744115:51
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator  https://review.openstack.org/15744115:53
*** dustins_ has joined #openstack-cinder15:54
*** kmartin has joined #openstack-cinder15:57
*** kallebe has joined #openstack-cinder15:57
*** dustins has quit IRC15:57
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support  https://review.openstack.org/8584715:57
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection  https://review.openstack.org/15965015:59
*** anshul has quit IRC15:59
*** anuragpalsule has joined #openstack-cinder16:01
*** lpetrut has joined #openstack-cinder16:02
openstackgerritYuriy Nesenenko proposed openstack/cinder: Change datetime.now() to datetime.utcnow()  https://review.openstack.org/15985416:03
*** coolsvap is now known as coolsvap_16:05
*** hemnafk is now known as hemna16:06
*** anuragpalsule has left #openstack-cinder16:06
hemnafloop16:06
*** annegentle has quit IRC16:10
*** ronis has quit IRC16:13
*** dannywilson has joined #openstack-cinder16:15
*** nkrinner has quit IRC16:16
*** anuragpalsule has joined #openstack-cinder16:16
*** deepakcs has joined #openstack-cinder16:17
*** kaisers has quit IRC16:17
*** kallebe has left #openstack-cinder16:18
*** kaisers has joined #openstack-cinder16:20
*** tsekiyama has joined #openstack-cinder16:23
*** patrickeast has quit IRC16:25
*** patri952 has quit IRC16:25
*** patri781 has quit IRC16:25
*** patri338 has quit IRC16:25
*** patri996 has quit IRC16:25
*** julim has quit IRC16:26
*** mwichmann has joined #openstack-cinder16:28
*** boris-42 has quit IRC16:32
*** marcusvrn has quit IRC16:32
*** marcusvrn has joined #openstack-cinder16:33
*** ankit_ag has quit IRC16:34
*** Adriano_ has joined #openstack-cinder16:34
openstackgerritJulien Danjou proposed openstack/cinder: tests: replace mox by mox3, clean out mox usage  https://review.openstack.org/14747616:37
openstackgerritJulien Danjou proposed openstack/cinder: tests: remove useless variable  https://review.openstack.org/16040816:37
*** annegentle has joined #openstack-cinder16:37
*** marcusvrn has quit IRC16:37
*** marcusvrn has joined #openstack-cinder16:38
mkodererDuncanT: I wrote a little script that prints out the driver support matrix16:42
mkodererDuncanT: https://review.openstack.org/#/c/160346/116:42
mkodererjgriffith: ^^ you might have a look at this too16:42
mkodererjust to get an idea what we could do with ABC...16:43
*** deepakcs has quit IRC16:44
e0nemkoderer: hi. interesting... do you have a sample output?16:44
*** sgotliv has quit IRC16:46
*** jdurgin has joined #openstack-cinder16:46
*** jdurgin has quit IRC16:46
*** _cjones_ has joined #openstack-cinder16:47
mkoderere0ne: http://paste.openstack.org/show/185237/16:48
mkoderere0ne: a bit raw but a first step..16:48
*** akerr has quit IRC16:48
*** akerr has joined #openstack-cinder16:51
*** diegows has quit IRC16:51
nikesh_vedamsDuncanT: call is going in my retype function16:53
openstackgerritJean-Daniel Bonnetot proposed openstack/cinder-specs: Add the possibility for admin to list volume-backend-names declared in cinder.conf.  https://review.openstack.org/16041616:54
openstackgerritGrzegorz Grasza (xek) proposed openstack/cinder: Service object  https://review.openstack.org/16041716:55
*** dustins_ has quit IRC16:55
*** e0ne is now known as e0ne_16:55
*** e0ne_ is now known as e0ne16:56
openstackgerritGrzegorz Grasza (xek) proposed openstack/cinder: Service object  https://review.openstack.org/16041716:56
e0nemkoderer: thanks16:56
openstackgerritYuriy Nesenenko proposed openstack/cinder: Change datetime.now() to datetime.utcnow()  https://review.openstack.org/15985416:57
*** asselin__ has joined #openstack-cinder16:58
*** jaypipes has joined #openstack-cinder17:00
*** jordanP has quit IRC17:02
*** asselin_ has quit IRC17:03
*** jordanP has joined #openstack-cinder17:03
*** pwehrle has quit IRC17:03
Swansonmkoderer: what is that showing exactly?17:03
*** diegows has joined #openstack-cinder17:05
*** alecv has quit IRC17:05
mkodererSwanson: it shows that ceph rbd gives you transfer functionaity, can be extended, is cloneable etc etc17:05
mkodererSwanson: it's automatically discovery by the inheritance of the volume driver17:06
*** annegentle has quit IRC17:07
*** rwsu has joined #openstack-cinder17:09
*** julim has joined #openstack-cinder17:10
Swansonmkoderer: Thanks!17:12
*** e0ne has quit IRC17:14
*** jistr has quit IRC17:21
*** yuriy_n17 has quit IRC17:23
openstackgerritJean-Daniel Bonnetot proposed openstack/cinder-specs: Add the possibility for admin to list volume-backend-names declared in cinder.conf.  https://review.openstack.org/16041617:25
*** patrickeast has joined #openstack-cinder17:26
*** adam_g_out is now known as adam_g17:27
openstackgerritGorka Eguileor proposed openstack/cinder: Make lio iSCSI changes persistent to avoid lost  https://review.openstack.org/16042817:27
*** anuragpalsule has left #openstack-cinder17:27
openstackgerritJean-Daniel Bonnetot proposed openstack/cinder-specs: Add the possibility for admin to list volume-backend-names declared in cinder.conf.  https://review.openstack.org/16041617:28
*** leeantho has joined #openstack-cinder17:30
*** ronis has joined #openstack-cinder17:31
*** annashen has joined #openstack-cinder17:31
*** asselin__ has quit IRC17:32
DuncanTmkoderer: I hadn't gotten round to trying to write that, Good job, thanks!17:35
openstackgerritBob Callaway proposed openstack/cinder: Add Manage/Unmanage support to NetApp NFS drivers  https://review.openstack.org/15886417:36
jgriffithmkoderer: nice!17:37
*** afazekas is now known as _afezekas|pub17:37
*** alexpilotti has joined #openstack-cinder17:39
jgriffithmkoderer: so that's pretty interesting/cool17:39
*** patrickeast_ has joined #openstack-cinder17:44
*** patrickeast__ has joined #openstack-cinder17:44
*** patri136 has joined #openstack-cinder17:44
*** patri505 has joined #openstack-cinder17:44
nikesh_vedamsHi'17:45
nikesh_vedamsmorning17:45
*** Apoorva has joined #openstack-cinder17:47
*** leeantho has quit IRC17:49
nikesh_vedamsjgriffith: morning17:50
*** timcl has quit IRC17:52
*** avishay__ has joined #openstack-cinder17:53
*** leeantho has joined #openstack-cinder17:54
*** avishay__ is now known as avishay17:57
nikesh_vedamsavishay: hi18:01
avishaynikesh_vedams: hi18:02
*** jogo has joined #openstack-cinder18:04
nikesh_vedamsavishay: http://paste.openstack.org/show/185123/   http://paste.openstack.org/show/185118/   i want to add retype and call is going in  my retype function18:04
nikesh_vedamsavishay: now only thing i have to change volume-type of volume18:05
*** jogo has left #openstack-cinder18:05
*** boris-42 has joined #openstack-cinder18:05
nikesh_vedamsavishay: because i only have volume_backend_name as a extra-specs18:05
*** Mandell has joined #openstack-cinder18:06
nikesh_vedamsnikesh_vedams: my array supports pools and vdisks18:06
nikesh_vedamsavishay : my array supports pools and vdisks18:06
*** annegentle has joined #openstack-cinder18:07
nikesh_vedamsavishay : so creating a volume type for each pool and vdisk as a backend and same driver is taking care to interact with pools and vdisks18:08
nikesh_vedamsavishay : so user can create a volume type 1 to create a volume on poolA,volume type2 to create a volume on vdisk1 and so on18:09
*** dustins has joined #openstack-cinder18:09
*** vilobhmm has joined #openstack-cinder18:10
*** _afezekas|pub has quit IRC18:10
nikesh_vedamsavishay: so now if user want to migrate so suppose from vdisk1 to vdisk2 having different volume types,i want to do retype18:10
avishaynikesh_vedams: correct18:11
nikesh_vedamsavishay:so in retype function can i change volume type18:11
nikesh_vedamsavishay: i donot have any extra-specs18:12
avishaynikesh_vedams: that is exactly what retype means18:12
avishaynikesh_vedams: you have the volume_backend_name extra spec, no?18:12
nikesh_vedamsavishay: is some syntax is available how can i do that,means do i have to update volume_stats18:13
nikesh_vedamsyes18:13
*** Maike has quit IRC18:13
avishaynikesh_vedams: you are asking how to run the retype command?18:13
nikesh_vedamshmm in retype function how can i changed volume type18:14
nikesh_vedamsdef retype(self, context, volume, new_type, diff, host):18:15
nikesh_vedamssome hints18:15
avishaynikesh_vedams: retype changes the volume type to new_type18:16
avishaynikesh_vedams: i don't understand the question18:16
nikesh_vedamsavishay :ok i want to write my own retype function,18:17
nikesh_vedamsavishay: so thinking what i can write so that it will change the volume type18:18
avishaynikesh_vedams: so you have the volume which has the original type, you have new_type, the diff between the two, and the host where the volume should be placed18:18
nikesh_vedamsavishay:yes18:18
avishaynikesh_vedams: based on that, the driver needs to change whatever properties are requested, including the host if it is different18:19
avishaynikesh_vedams: the ibm storwize driver implements it, and probably others too - you can use for reference18:19
*** lpetrut has quit IRC18:20
*** harlowja_away is now known as harlowja_18:21
*** thingee has joined #openstack-cinder18:23
nikesh_vedamsavishay:ok thanks18:23
avishaynikesh_vedams: do you have a question?18:24
*** jordanP has quit IRC18:25
*** gugl has joined #openstack-cinder18:26
nikesh_vedamsavishay : hmm let me analyze the other drivers like you said and will get back latter18:27
avishaynikesh_vedams: ok good luck18:27
nikesh_vedamsthankjs18:27
nikesh_vedamsthanks18:28
*** frakt has quit IRC18:28
*** frakt has joined #openstack-cinder18:29
*** jdurgin has joined #openstack-cinder18:35
openstackgerritAnthony Lee proposed openstack/cinder: Adding manage/unmanage support for LeftHand driver  https://review.openstack.org/15512718:38
*** aix has quit IRC18:41
openstackgerritSean McGinnis proposed openstack/cinder: Remove use of contextlib.nested  https://review.openstack.org/15873218:45
kauferthingee: Can you remove your -2 on https://review.openstack.org/#/c/145913/ ... the cross project spec and the server-side work have both been merged18:49
openstackgerritharsh mishra proposed openstack/cinder: Fix for  inconsistent cinder-services state change  https://review.openstack.org/16010418:49
*** bkopilov has quit IRC18:53
*** bkopilov has joined #openstack-cinder18:53
thingeekaufer: done!18:56
*** timcl has joined #openstack-cinder18:56
kauferthingee: Thanks!18:56
*** avishay has quit IRC18:59
jungleboyjkaufer: Yay!18:59
*** asmith_brcd has joined #openstack-cinder19:00
*** devlaps has joined #openstack-cinder19:01
SwansonIf anyone with the ability to +2 this could take a look that would be super.  https://review.openstack.org/#/c/159932/  Thanks!19:01
Swanson(If anyone with the ability to +1 that wants to take a look that would be super, too.)19:01
kauferjungleboyj: Awesome, so that BP is done. I do have a question on the REST API doc for this. Is that something that I update or is there some other team that creates the doc like this: http://developer.openstack.org/api-ref-blockstorage-v2.html19:02
thingeekaufer: it would be nice if you updated it :) https://github.com/openstack/api-site19:04
thingeejgriffith: can you reply back to https://review.openstack.org/#/c/135002/ please.19:04
kauferthingee: Yes, I'd like to ... once I figure out how, I'll dig into that repo, thanks19:05
smcginnisjungleboyj: Afternoon! Not sure if you saw - you were getting paged here:19:05
smcginnisjungleboyj: https://review.openstack.org/#/c/154739/19:05
*** crose has joined #openstack-cinder19:05
thingeekaufer: oh yeah, oxygenxml editor is recommended for that sort of thing. You can ping the #openstack-doc team for help with getting setup.19:06
kauferthingee: Will do, thanks!19:07
jungleboyjsmcginnis: I didn't.19:08
jungleboyjsmcginnis: I will look at that as soon as I am done booking the trip to vancouver.  :-)19:10
smcginnisjungleboyj: Yay! ;)19:10
*** annegentle has quit IRC19:10
jungleboyjsmcginnis: Sounds like several of us will be at the Pinnacle Hotel Vancouver Harbourfront.19:10
thingeejungleboyj: is it cheap?19:11
thingeejungleboyj: still haven't booked anything and on the usual budget19:11
jungleboyj:-(19:11
*** frakt has quit IRC19:13
*** frakt has joined #openstack-cinder19:14
thingeejgriffith, DuncanT: thanks for getting the cinder object stuff through!19:14
* thingee watches zuul anxiously19:14
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Update volume type name for volume type API  https://review.openstack.org/14090619:14
openstackgerritWalter A. Boring IV (hemna) proposed openstack/python-cinderclient: Update to change name for volume type client  https://review.openstack.org/14118719:15
smcginnisjungleboyj: Sorry, stepped away.19:15
smcginnisjungleboyj: I think I booked the Shangri-la.19:15
smcginnisjungleboyj: But maybe I should see about changing that.19:16
mtaninothingee: hi,19:17
mtaninothingee: About this IET work https://review.openstack.org/#/c/158829/ , is it better to open a BP for traking purpose?19:18
*** lpetrut has joined #openstack-cinder19:19
smcginnisjungleboyj: Doh! Pinnacle is sold out. Guess I'm staying where I was.19:19
*** ebalduf has quit IRC19:19
*** shakamunyi has joined #openstack-cinder19:20
*** Mandell has quit IRC19:21
thingeemtanino: We're not accepting anymore bps for k-3.19:21
thingeemtanino: http://lists.openstack.org/pipermail/openstack-dev/2015-January/055719.html19:21
thingeemtanino: and http://lists.openstack.org/pipermail/openstack-dev/2015-February/056964.html19:21
mtaninothingee: Yes. I understand. That driver is originally implemented at Juno and after refactoring SCSI target, that was removed.19:22
*** diemt has joined #openstack-cinder19:23
mtaninothingee: So now I try to implement that driver again based on the bug from jgriffith.19:23
mtaninothingee: In this case, do we need to implement it until Liverty?19:23
mtaninoliberty.19:23
thingeemtanino: yes, we're full for k-3 see https://launchpad.net/cinder/+milestone/kilo-319:23
*** tbarron has quit IRC19:24
openstackgerritThang Pham proposed openstack/cinder: VMware: Add vmware:adapter_type extra spec  https://review.openstack.org/15926019:24
*** tbarron has joined #openstack-cinder19:24
thingeemtanino: I've been really loud about this in the irc meetings and mailing list. I invite you to join the cinder community in announcing what you want to propose in milestones.19:25
thingeewith the process that has been communicated multiple times.19:26
*** crose has quit IRC19:26
*** annegentle has joined #openstack-cinder19:27
mtaninothingee: I understand current k-3 status. My only concern is some people feel this as a regression.19:27
thingeemtanino: that's fair19:27
thingeemtanino: if jgriffith and another core wants to sponsor it, that's fine. I also want to see unit tests with this.19:28
mtaninothingee: Anish already proposed unit test.19:28
mtaninolet me check a URL.19:28
mtaninohttps://review.openstack.org/#/c/158925/19:28
thingeemtanino: please don't do separate patches for unit tests in the future.19:29
thingeeIdeally I don't want to see code getting merged unless the unit tests are being merged with it.19:29
mtaninothingee: I understand. I will ask anish and merge one patch.19:29
*** devlaps has quit IRC19:29
thingeeIf you want to work with someone else, consider co-author in the commit message.19:29
mtaninothingee: I understand.19:30
thingeemtanino: thanks for your work on this. Let me know when you have someone else to sponsor this and we'll get things targeted.19:30
mtaninothingee: so, let me ask jgriffith about this. If I will get sponser, do I still need BP?19:31
mtaninoor Bug?19:31
thingeemtanino: yes19:31
mtaninothingee: need a BP.19:31
thingeewe won't be merging new features that don't have a targeted approved bp.19:31
mtaninothingee: that's right.19:32
*** e0ne has joined #openstack-cinder19:32
*** e0ne is now known as e0ne_19:32
*** e0ne_ is now known as e0ne19:32
mtaninothingee: thank you for taking time. I will try this to get merged in Kilo.19:33
*** sgotliv has joined #openstack-cinder19:34
*** jordandh has quit IRC19:35
e0nejgriffith: hi John19:37
thingeejungleboyj: you mind verifying this? https://bugs.launchpad.net/neutron/+bug/140973319:37
openstackLaunchpad bug 1409733 in Cinder "adopt namespace-less oslo imports" [High,In progress] - Assigned to Anish Bhatt (anish7)19:37
openstackgerritKallebe Monteiro proposed openstack/python-cinderclient: Fix outdated URLs and some minor fixes  https://review.openstack.org/16046919:37
thingeeanish: ^19:37
*** jordandh has joined #openstack-cinder19:38
jungleboyjthingee: Verify the patch from Anish?19:40
thingeejungleboyj: yeah, are we done with this migration to oslo_19:40
jungleboyjI approved his patch.  I have a couple more to deal with yet.19:41
jungleboyjThat was on the list for this week.19:41
thingeejungleboyj: ok!19:41
jungleboyjthingee: :-)  So much Oslo fun!19:41
thingeejungleboyj: thanks again for keeping up with all the oslo work :)19:41
thingeejungleboyj: I really appreciate it19:41
jungleboyjthingee: Welcome.  I know, thanks for helping approve the patches.19:41
thingeejungleboyj: yeaaa trying to making that painless for you :)19:42
jungleboyj:-)  Thank you my friend!19:43
openstackgerritPatrick East proposed openstack/cinder: PureISCSIDriver consistency group updates.  https://review.openstack.org/15763719:43
*** ebalduf has joined #openstack-cinder19:44
*** lpetrut has quit IRC19:46
jungleboyjthingee: On a related note, talked to Oslo team today about config-generator .19:47
openstackgerritGorka Eguileor proposed openstack/cinder: Make lio iSCSI changes persistent to avoid lost  https://review.openstack.org/16042819:48
jungleboyjthingee: Thus far, other projects have more consolidated config files.  Nova and Neutron have not yet tackled this and thought my approach made the most sense for the larger projects.19:49
jungleboyjthingee: So, I am going to blaze the trail there and they will follow.  Doug noted that using the multiple entry points approach only made sense if you wanted to create individual config files for each of the drivers which doesn't make sense.19:50
*** Mandell has joined #openstack-cinder19:53
*** gmurphy has joined #openstack-cinder19:55
*** gmurphy has left #openstack-cinder19:55
*** rushil has quit IRC19:55
jgriffithe0ne: hey19:56
*** vilobhmm2 has joined #openstack-cinder19:58
*** e0ne is now known as e0ne_19:59
*** xyang1 has joined #openstack-cinder19:59
*** bswartz has quit IRC19:59
*** vilobhmm has quit IRC20:00
*** e0ne has joined #openstack-cinder20:00
*** sigmavirus24 has left #openstack-cinder20:01
e0nejgriffith: i feels more lucky today... i would like to ask you about yours -120:02
*** patrickeast__ has quit IRC20:04
*** patri505 has quit IRC20:04
*** patri136 has quit IRC20:04
*** patrickeast_ has quit IRC20:04
*** patrickeast has quit IRC20:04
*** sandywalsh_ has joined #openstack-cinder20:05
*** nellysmi_ has joined #openstack-cinder20:05
*** patrickeast has joined #openstack-cinder20:07
*** fifieldt has quit IRC20:07
*** sandywalsh has quit IRC20:07
*** patrickeast_ has joined #openstack-cinder20:08
*** patrickeast__ has joined #openstack-cinder20:08
*** patri875 has joined #openstack-cinder20:08
*** patri928 has joined #openstack-cinder20:08
*** nellysmitt has quit IRC20:09
*** EmilienM|afk is now known as EmilienM20:10
jgriffithe0ne: haha... which one?20:11
e0nejgriffith: hm... did i get second? :)20:11
jgriffithe0ne: LOL.. no, but if you have a link it's faster than me searching :)20:12
e0nejgriffith: https://review.openstack.org/#/c/131182/20:12
jgriffithe0ne: too late, found it ;)20:12
*** karimb has joined #openstack-cinder20:12
*** bswartz has joined #openstack-cinder20:12
e0neit's not my day too:(20:12
mtaninojgriffith: hi,20:13
jgriffithe0ne: good enough for me, thanks for looking at it20:13
jgriffithmtanino: hey20:13
mtaninojgriffith: I talked thingee about IET work and he recommends to open BP for introducing a feature.20:13
jgriffithmtanino: hmmm.. I thought it was a bug?20:13
*** _afezekas|pub has joined #openstack-cinder20:13
e0nejgriffith: np. i'll continue to work on it once i've got merged other rally scenario20:13
jgriffiththingee: ^^20:14
jgriffithmtanino: thingee bug because up until Kilo we had an IET option, now we don't20:14
jungleboyjsmcginnis: mtanino https://review.openstack.org/#/c/154739/ is +A20:14
jgriffithmtanino: thingee the other "correct" option IMO would be add it to the bp I used for the seperation20:14
jungleboyjsmcginnis: Check through the Summit's website.  There are still rooms available that way.20:14
mtaninojgriffith: Which one?20:15
smcginnisjungleboyj: Ah, good point. Would be nice to be a little closer.20:15
jungleboyjsmcginnis: they must have blocked the whole hotel or something because IBM's travel tool said it was Sold Out as well.20:15
jgriffithmtanino: thingee so it's really not a new feature I don't think.20:15
mtaninojgriffith: yes. I agree this is not a new feature. just adding a feature to keep compatibility.20:15
smcginnisjungleboyj: Yeah, I was looking at Dell's travel tool. Didn't even think to look at the conference page.20:15
jgriffithmtanino: lemme find bp... just a sec20:16
mtaninojungleboyj: thank you for the review :)20:16
jungleboyjsmcginnis: Thank kmartin ,  he is the smart one.20:16
mtaninojgriffith: sure :)20:16
smcginnisjungleboyj: ;)20:16
jungleboyjsmcginnis: He keeps hemna and I on track.  :-)20:16
jgriffithmtanino: LOL... I submitted that as a bug :)20:16
jungleboyjmtanino: You are welcome.  Thank you for fixing that!20:16
mtaninojgriffith: Yes. I think so20:16
jgriffithPartial-Bug: #132913920:16
openstackbug 1329139 in Cinder "code duplication in iscsi helper(s)" [Undecided,In progress] https://launchpad.net/bugs/1329139 - Assigned to John Griffith (john-griffith)20:16
hemnae0ne, posted a comment on your shadow table patch20:16
mtaninojgriffith: I couldn't find BP lol20:17
jgriffithmtanino: http://goo.gl/CmMIK620:17
jgriffithmtanino: I specifically call out IET in the commit and marked this as partil because of it20:17
jgriffithmtanino: so IMHO you should be able to associate it as "closes" that same bug20:18
e0nehemna: thanks20:18
jgriffithmtanino: but we'll have to make sure thingee is ok with that20:18
mtaninojgriffith: hmmm20:18
mtaninojgriffith: yes.20:18
hemnalunch....20:18
jgriffithmtanino: FYI https://bugs.launchpad.net/cinder/+bug/132913920:19
openstackLaunchpad bug 1329139 in Cinder "code duplication in iscsi helper(s)" [Undecided,In progress] - Assigned to John Griffith (john-griffith)20:19
*** thingee has quit IRC20:19
jgriffithmtanino: still in progress20:19
*** bswartz has quit IRC20:19
mtaninojgriffith: so closes-bug: #1329139 is fine?20:19
mtaninojgriffith: we also need close this bug. https://bugs.launchpad.net/cinder/+bug/140991820:20
openstackLaunchpad bug 1409918 in Cinder "iet target driver not implemented in new iscsi target driver model" [High,In progress] - Assigned to Anish Bhatt (anish7)20:20
jgriffithmtanino: IMO yes20:20
jgriffithmtanino: hehe.. yes!!!20:21
mtaninojgriffith: fyi, I just opened BP https://blueprints.launchpad.net/cinder/+spec/implement-iet-target-driver 10 minutes ago.20:23
jgriffithmtanino: sigh... :)20:24
jgriffithmtanino: well.. I guess the commit message can close the bug and the BP at the same time20:24
jgriffithmtanino: but honestly I don't think a bp to return a broken implementation is appropriate20:24
*** fifieldt has joined #openstack-cinder20:25
smcginnisjungleboyj: Switched hotels - thanks!20:25
jungleboyjsmcginnis: Yay!20:25
mtaninojgriffith: oops, thingee is not online.20:26
jgriffithhmmm20:26
jgriffithwell, then it's up to me and another core or two to decide20:26
jgriffithjungleboyj: e0ne hemna thoughts regarding IET?20:26
mtaninojgriffith: OK. please make a decision.20:27
jgriffithjungleboyj: e0ne hemna if you need a recap I'm happy to provide20:27
jgriffithjungleboyj: e0ne hemna or I'll just make the call myself and annoy people :)20:27
jgriffitheharney: ^^20:27
mtaninoIET: https://review.openstack.org/#/c/158829/20:27
jungleboyjjgriffith: Let me look back at the discussion.20:28
* eharney reading around20:28
*** _afezekas|pub has quit IRC20:29
mtaninotalk with thingee. http://paste.openstack.org/show/185438/20:30
e0nemtanino, jgriffith: what about CI for this driver?20:30
jgriffithe0ne: that's N/A IMO20:30
jgriffithe0ne: IET has been in place as ref impl since the beginning20:30
eharneymy opinion is that since we broke it and it was there before, we should stick it back in and a bug is fine20:32
*** lpabon has joined #openstack-cinder20:32
jgriffitheharney: ditto.. thanks20:33
e0nejgriffith: if it was working before, it would be good ti fix it. but i can't vote for it becase i don20:35
e0ne*because i didn't use it and can't try20:35
jgriffithe0ne: fair20:35
jgriffithe0ne: but, does that mean you can't vote on third party drivers either?20:36
e0neanyway, mtanino, please add unit tests to the same patch20:36
mtaninojgriffith: I have to merge my patch and anish's test patch.20:36
jgriffithe0ne: ie you can't vote on HP, SolidFire, IBM patches :)20:36
e0nejgriffith: you catched me20:37
mtaninoe0ne: currently unit test is proposed at separate patch. I will merge those into one patch.20:37
jgriffithe0ne: :)20:37
jgriffithe0ne: sorry... I couldn't resist20:37
e0nemtanino: yes, that is what i'm talking about20:37
jgriffithe0ne: ahh... ok, that's more than fair20:37
mtaninoe0ne: thingee gave me that comment and I agreed it.20:37
*** Yogi1 has quit IRC20:38
e0nejgriffith: i mean can review and +/-1 for it. i try to not +2/+A pathses which i'm not sure for 100% for it and i can't test it on my env20:38
jgriffithe0ne: cool20:39
jgriffithe0ne: good policy20:39
e0nejgriffith: thanks20:39
*** ndipanov is now known as ndipanov_gone20:40
e0nejgriffith: TBH, sometime it's too thin side bitween bug and feature20:43
openstackgerritErlon R. Cruz proposed openstack/cinder: Adds pool aware scheduling for HNAS drivers  https://review.openstack.org/14213320:44
openstackgerritErlon R. Cruz proposed openstack/cinder: Adds SSH communication to HNAS drivers  https://review.openstack.org/14290420:44
jgriffithe0ne: yeah, thats it definitely true20:45
jungleboyjjgriffith: I agree with eharney20:47
*** xyang has joined #openstack-cinder20:48
jungleboyjmtanino: Would be good to get the unit tests in with the patch for fixing IET.20:49
*** eharney has quit IRC20:51
*** kmartin has quit IRC20:53
*** asmith_brcd_ has joined #openstack-cinder20:55
*** asmith_brcd_ has quit IRC20:56
*** lpabon has quit IRC21:05
*** annegentle has quit IRC21:05
jgriffithjungleboyj: thanks21:06
*** shakamunyi has quit IRC21:07
jungleboyjjgriffith: Welcome.21:08
*** med_ has quit IRC21:08
*** shakamunyi has joined #openstack-cinder21:08
*** kmartin has joined #openstack-cinder21:09
hemnaIET ?21:11
hemnajgriffith, sorry, was at lunch21:12
jgriffithhemna: no worries21:12
jgriffithhemna: yeah, the IET iscsi driver21:12
jgriffithhemna: just a different target21:12
jgriffiththe original :)21:12
hemnaoh I don't think I've looked at it in a bit21:13
*** diegows has quit IRC21:13
jgriffithhemna: probably not because it's been missing for about 3 months :)21:13
hemnaheh21:14
hemnadoh21:14
hemnajgriffith, thoughts on my last point on the shadow tables review ?  https://review.openstack.org/#/c/131182/21:14
hemnabasically any existing tables w/ shadows will require 2x the work to update in a new migration21:15
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator  https://review.openstack.org/15744121:16
jgriffithhemna: yeah, I just assumed that was a 'cost'21:17
jgriffithhemna: no good?21:17
e0nehemna: i'll be very happy if you know how to make it "less expensive"21:18
hemnayah I guess just file it under...bummer21:19
mtaninojungleboyj: yes. Unit test is here https://review.openstack.org/#/c/158925/ and I have to merge those two for one patch.21:19
hemnaso I'm trying to figure out exactly what I'll have to do wrt to the multiattach patch21:19
hemnathat this falls under21:19
hemnacolumns in the volumes table go away and move to a new table21:19
jgriffithhemna: the good thing is it's something that we'll have to all do eventually :)21:19
hemnaso, do I just drop columns from the shadow table?21:20
hemnado I have to migrate data from the existing volumes shadow table to a new volume_attachments table?!21:20
hemnathat part sux21:20
*** jungleboyj has quit IRC21:20
e0nehemna: imo, you don't need to drop shadow columns21:20
hemnahrmm21:21
jgriffithhemna: you just duplicate the migration for the shadow table21:21
hemnawhat will happen the next time the rows are migrated to the shadow table then ?21:21
jgriffithsame same21:21
e0nehemna: are you talking about 'undelete'?21:21
jgriffiththy're "updated"21:21
jgriffithhemna: ohh... wait21:22
jgriffithhemna: you're not talking a DB migration, your talking abou the data migration piece?21:22
hemnaso here is my existing migration21:22
hemnahttps://review.openstack.org/#/c/85847/37/cinder/db/sqlalchemy/migrate_repo/versions/037_add_volume_attachment.py21:22
hemnaI'm concerned about the data21:22
*** Adriano_ has quit IRC21:22
hemnawell and a little bit about the columns21:22
hemnathat patch drops 4 columns from the volumes table21:23
hemnaand migrates data to a new table21:23
jgriffithhemna: you duplicate that same migration for the shadow table21:23
jgriffithhemna: you have to do it the same way... the timing might be a trick21:23
jgriffithbut I don't think so... since we don't do live migrations curently that I'm aware of21:23
hemnadang ok21:24
*** chlong has joined #openstack-cinder21:24
jgriffithhemna: let's look at Nova and see if there's an example...21:24
hemnaand just hope that nothing fails in the shadow data migration21:24
*** SergeyLukjanov has quit IRC21:24
jgriffithhemna: not sure I follow that TBH21:24
jgriffithhemna: you mean if a shadow migration runs when your'e doing a db migration?21:24
hemnawell there were a bunch of concerns in that patch for the data migration piece.21:24
jgriffithI dont' think we allow that, or that it's any different than a regular db migration21:24
hemnaand I'm a bit concerned those same things will be against the shadow table data migration21:25
hemnaand dealing with failures, etc there.21:25
hemnaseems like a bit of a mess21:25
jgriffithhemna: yes, for sure they will be21:25
jgriffithhemna: well... maybe the data migration of the patch is the mess, no?21:25
hemnax221:25
hemnaheh21:25
jgriffithhemna: so I'm not sure how else you do shadow tables21:25
hemnawell yah, there is that21:25
jgriffithhemna: unless you think maybe we shouldn't do them at all?21:25
hemnaI have no opinion on that just yet21:26
jgriffithhehe... "just yet"21:26
jgriffith:)21:26
hemnaI'm just trying to understand what we'll have to do if this lands21:26
hemnawhat the process will be, requirements, things to look for in reviews, etc21:26
jgriffithhemna: stand on one foot and sing the German National Anthem in Spanish21:26
hemnabecause it's definitely a change21:26
hemna:)21:26
hemnaI guess there is a use case for operators wanting to keep deleted data around?21:27
e0nehemna: from the spec: "Shadow tables could have blob field to store some "deleted" data and to not impose restrictions on database schema changes."21:27
jgriffithhemna: http://goo.gl/JcXj1S21:28
jgriffithhemna: and http://goo.gl/zNGYHv21:28
*** annegentle has joined #openstack-cinder21:29
hemnaheh new columns are easy :)21:29
jgriffith:)21:29
jgriffithas opposed to?21:29
hemnamoving existing ones w/ existing data, a la my multiattach patch21:29
jgriffithhemna: well.. you don't want to "move"21:30
hemnaok so it is what it is21:30
jgriffithhemna: you want to do a multi-part migration21:30
jgriffithhemna: which reminds me... I meant to ping you about that21:30
hemna:)21:30
jgriffithhemna: so here's teh trick21:30
jgriffithhemna: one migration creates the new columns and puts the data in them21:30
jgriffithhemna: a follow up migration drops the old column21:31
e0nehemna: you don't need t move any data to shadow tables during the migrations becase shadow stores deleted data21:31
jgriffithhemna: or you can put the copy/drop in the second21:31
jgriffithhemna: that's how I did the nova-volume migration... lemme see if I can find it21:31
jgriffithhemna: http://goo.gl/owkPLw21:32
*** enterprisedc has joined #openstack-cinder21:32
*** ronis_ has joined #openstack-cinder21:32
jgriffithhemna: and the follow up 9021:32
jgriffithhemna: does that help?21:32
jgriffithhemna: so create a temporary mapping...21:33
*** ronis has quit IRC21:33
jgriffithhemna: then do the swap work and drop21:34
jgriffithhemna: it's certainly different than what you're doing, but the same pattern should work (I think)21:34
hemnaI'm kinda doing that in the downgrade sql script21:34
hemnahttps://review.openstack.org/#/c/85847/37/cinder/db/sqlalchemy/migrate_repo/versions/037_sqlite_downgrade.sql21:34
jgriffithhemna: heeh.. well, cuz with sql you have to on FK's no matter what21:35
hemnayah21:35
hemnapita21:35
jgriffithI mean sqlite21:35
*** eharney has joined #openstack-cinder21:36
*** bswartz has joined #openstack-cinder21:37
hemnaok well, I think multiattach is dead for k anyway21:38
jgriffithhemna: why?21:38
hemnaI honestly just wanted to learn what the process was for shadow tables where columns were going away and data was being migrated21:39
jgriffithhemna: so back up... why is multi-attach dead?21:39
jgriffithhemna: because shadow tables are hard?21:39
hemnaheh no21:39
hemna-2 on the client by Duncan21:39
jgriffithhemna: meh... that's ok21:40
*** med_ has joined #openstack-cinder21:40
jgriffithhemna: I mean.. he's right it's not going to 'work' without Nova21:40
*** med_ has quit IRC21:41
*** med_ has joined #openstack-cinder21:41
jgriffithhemna: and as such no reason to release a client with it21:41
jgriffithhemna: but I don't know why that should mean you can't keep the code going in Cinder itself?21:41
jgriffithhemna: that was kinda my whole point about making sure it's 100% back compatable21:41
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support  https://review.openstack.org/8584721:41
jgriffithhemna: so you can continue working/developing it outside of the K boundaries so to speak21:41
hemnaoh yah I know it won't work w/ Nova :)21:42
hemna(rebased)21:42
jgriffithhemna: well... so not having a client impl is fine IMO21:42
jgriffithhemna: make that a WIP until nova is ready21:42
jgriffithhemna: no reason to "punt" so to speak21:42
hemnaok21:42
jgriffithunless you just don't want to work on it any more21:42
jgriffithhemna: totally your call21:42
hemnayah this is my 3rd try on this21:42
hemnaI'll keep at it for now21:43
jgriffithhemna: but I think there's value in merging then doing finishing touches and clean up in L21:43
hemnaok I'm fine with that21:43
jgriffithhemna: ahh... yes, I see21:43
jgriffithhemna: that sucks21:43
hemnayah it's been a long 1.5 years on this21:43
hemna:(21:43
hemnamostly due to nova <--> cinder stuff21:43
jgriffithhemna: well... on the bright side it's WAY further ahead this time than any of the others21:43
hemnathis is true :)21:43
jgriffithhemna: in fact did we ever even get an actual ready review up until this time around?21:44
jgriffithI didn't think we did21:44
jgriffithWe just had lots of arguing about the idea itself21:44
hemnayah I had it working in my first go around21:44
hemnabut ended up dying on the nova side21:44
jgriffithhemna: I know "you" had it working... but did it get submitted?21:44
jgriffithahhh... yeah21:44
jgriffithsurprise, surprise21:44
jgriffithso look... here's my thought21:44
jgriffithwe all want it, including folks on the Nova side (I think)21:45
jgriffithso don't abandon it by any means21:45
hemnaok thanks21:45
jgriffithlearn from the release data madness21:45
jgriffithHit it hard as top priority the day L opens up21:45
hemnaso I guess if anything what I've learned here is21:45
jgriffithuse the time between now and then to make sure everything is aligned and ready21:45
hemnaif cinder wants anything done in nova, we basically only have the 1st milestone to get it done21:46
*** e0ne is now known as e0ne_21:46
jgriffithhemna: well... yeah, that's a tough one21:46
hemna2nd milestone is very very iffy from the nova side and gets clamped down quickly21:46
hemnaalmost impossible after the 1st milestone to make nova changes for cinder21:46
jgriffithhemna: but you should jump in on that milestone thread and share the frustration21:46
jgriffithhemna: well... in general "any significant change"21:47
hemnayah21:47
jgriffithnot saying it's good bad or otherwise, but it is what it is21:47
jgriffithbut shouldn't stop us from moving forward IMO21:47
hemnato be fair though, the required changes on the nova side were far more that it really should have been for such a simple change.  but whatevs21:47
jgriffithespecially as hard as you've worked on it, and done "good things"21:47
hemnayah it is what it is.21:47
jgriffithhemna: yeah, that's a whole different problem21:48
hemnayuh21:48
jgriffithhemna: there's this scary theme of over-complication some times21:48
jgriffithbut hey... it could be worse... you could be working on that thing that starts with a V21:48
hemnaI'd like to have a discussion w/ the team at the conference about how nova interacts w/ our API21:48
hemna:P21:48
*** e0ne_ is now known as e0ne21:48
jgriffithThe Nova team?21:48
*** akerr has quit IRC21:48
jgriffithhemna: there's a solution here ya know....21:49
bswartzwhat starts with a V?21:49
jgriffithbswartz: Vegetables21:49
bswartzlol21:49
jgriffithVictory21:49
jgriffithVasquez21:49
jgriffithVaquero21:49
jgriffithNeed I continue :)21:49
hemnaI think we just need to talk about what we have, how nova is calling us, and the 'right' way to fix it.21:49
jgriffithhemna: agreed21:49
hemnait's kinda a mess right now.   it should be cleaner.   but that's an uphill battle21:49
hemnabut an important one for us I think.21:50
jgriffithhemna: meh, I don't think it's that tough a battle.  just have to have ducks in a row and a good plan21:50
jgriffithand communicate21:50
*** ronis_ has quit IRC21:50
jgriffithIt's a great big community21:50
hemnayah21:50
*** chlong has quit IRC21:51
hemnamaybe we can come up with a plan that we like and then present it to the nova folks21:51
*** julim has quit IRC21:51
*** shakamunyi_ has joined #openstack-cinder21:53
*** shakamunyi has quit IRC21:54
hemnaxyang, https://review.openstack.org/#/c/144561/  needs a rebase21:54
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add multiattach capabilities injection  https://review.openstack.org/15965021:54
*** bswartz has quit IRC21:55
*** aix has joined #openstack-cinder21:56
*** _afezekas|pub has joined #openstack-cinder21:56
hemnae0ne, so, fwiw, we should update the db-archiving spec to talk about the developer impact.   It currently says None, which is wrong IMHO.21:57
xyanghemna: thanks.  I'll rebase.  it is because the cinder object patch just landed21:58
e0nehemna: ok, sounds reasonable21:58
hemnaxyang, yah I had to rebase the multiattach stuff for the same reason21:58
e0nehemna: how should it be landed?21:58
e0nehemna: i mean should i somewhere specify changes to the spec?21:59
hemnalet me file a bug22:00
hemnathen you can reference that bug in the cinder-specs review that updates the existing spec ?22:00
hemnaso it's clear this isn't a new spec, but a 'bug fix' ?22:00
*** cebruns has quit IRC22:01
e0nei'm agree with you22:02
hemnae0ne, https://bugs.launchpad.net/cinder/+bug/142740222:04
openstackLaunchpad bug 1427402 in Cinder "db-archiving spec missing developer impact" [Undecided,New]22:04
*** dannywilson has quit IRC22:04
e0nehemna: thanks. i'll try to make patch tonight22:04
*** dustins has quit IRC22:04
hemnaI'll +3 the cinder code if you can update the cinder-spec to reflect the developer impact.22:04
e0ne:)22:05
*** _cjones_ has quit IRC22:06
hemnagreat, thanks e0ne22:06
e0nehemna: np22:06
e0nehemna: btw, i've started to continue work on my flask and keysthone auth lib today. i'll update it in a next 2 days22:07
hemnae0ne, sweet22:08
*** tshefi has quit IRC22:08
*** _afezekas|pub has quit IRC22:10
*** xyang has quit IRC22:10
openstackgerritxing-yang proposed openstack/cinder: Modify Consistency Group API  https://review.openstack.org/14456122:10
*** _cjones_ has joined #openstack-cinder22:12
openstackgerritxing-yang proposed openstack/cinder: Modify Consistency Group API  https://review.openstack.org/14456122:13
*** thingee has joined #openstack-cinder22:14
thingeejgriffith: sorry got disconnected (tethered to my phone) when you were talking to me here.22:15
*** Mandell has quit IRC22:15
thingeejgriffith, mtanino: yeah ok that makes sense wrt to IET being a bug.22:15
mtaninothingee: welcome back.22:16
thingeethangp: yay cinder object patches landed. I owe you a drink if you show up to the next summit :)22:18
thangp:D thanks22:18
thangpthere's more to do22:18
thangphopefully it doesnt cause a lot of bugs :P22:18
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Add 'Development Impact' section for DB Archiving  https://review.openstack.org/16053422:19
thingeethangp: that's why we're getting it in now to iron out any issues :)22:19
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Add 'Developer Impact' section for DB Archiving  https://review.openstack.org/16053422:20
thingeejgriffith, xyang1, DuncanT: thanks again for helping with the reviews and testing22:20
thangpjgriffith, xyang1, Duncan: thank you from me too!22:20
*** Lee1092 has quit IRC22:20
xyang1thingee: thangp you are welcome22:20
thingeethangp: now we'll have to migrate to the oslo stuff in liberty :)22:21
thangpthingee: Doug hasn't approved it for openstack/requirements yet22:21
thingeeah k22:21
thangpthingee: still waiting22:21
thangpbut there is a PoC I made for it: https://review.openstack.org/#/c/160209/22:22
thangpthingee: basically, just some cleanup...the objects stay pretty much the same22:23
*** e0ne has quit IRC22:23
thingeewanghao: can we get an update on this https://review.openstack.org/#/c/144590/ ?22:24
thingeehemna: what's up with brick :)22:24
mtaninoanish: Are you around today?22:24
thingeehemna: nova folks are asking me about it here at the magnum midcycle22:25
hemnahey22:25
hemnaoh?22:25
hemnawhat do you want to know?22:25
hemnaos-brick exists22:25
thingeeearly liberty at this point?22:25
hemnathingee, needs to do a release :)22:25
thingee;)22:25
hemna1) os-brick needs to release to pypi22:25
hemna2) cinder patch to remove cinder/brick -> os-brick22:25
hemna3) nova patch to remove internals of libvirt volume drivers -> os-brick22:26
hemnaL122:26
* hemna crosses fingers22:26
thingeehemna: ok!22:26
hemna(global requirements updates in between 1, 2) :)22:26
*** jungleboyj has joined #openstack-cinder22:26
*** e0ne has joined #openstack-cinder22:27
*** thangp has quit IRC22:30
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use oslo.log instead of oslo-incubator  https://review.openstack.org/15744122:32
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Add 'Developer Impact' section for DB Archiving  https://review.openstack.org/16053422:32
*** Mandell has joined #openstack-cinder22:33
*** e0ne has quit IRC22:34
*** diemt has quit IRC22:35
*** SergeyLukjanov has joined #openstack-cinder22:37
*** e0ne has joined #openstack-cinder22:38
*** mtanino has quit IRC22:39
*** patrickeast has quit IRC22:39
*** esker has joined #openstack-cinder22:40
*** asmith_brcd has quit IRC22:40
*** alexpilotti has quit IRC22:41
*** patrickeast__ has quit IRC22:44
*** patri875 has quit IRC22:44
*** patri928 has quit IRC22:44
openstackgerritTomoki Sekiyama proposed openstack/cinder: Failover to alternative iSCSI portals on login failure  https://review.openstack.org/14087722:44
*** patrickeast_ has quit IRC22:44
*** annegentle has quit IRC22:47
*** mtanino has joined #openstack-cinder22:47
*** e0ne has quit IRC22:49
*** e0ne has joined #openstack-cinder22:53
*** xyang has joined #openstack-cinder22:56
*** resker has joined #openstack-cinder22:58
*** annegentle has joined #openstack-cinder22:59
*** esker has quit IRC22:59
e0nejgriffith, xyang, smcginnis: about logging is oslo-log https://review.openstack.org/#/c/147312/223:00
*** tbarron has quit IRC23:04
*** tbarron has joined #openstack-cinder23:05
*** shakamunyi_ has quit IRC23:06
xyange0ne: oh, so the logging rules are changing again.  thanks for the note23:06
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Add 'Developer Impact' section for DB Archiving  https://review.openstack.org/16053423:08
*** ebalduf has quit IRC23:08
*** dannywilson has joined #openstack-cinder23:08
*** dannywilson has quit IRC23:09
*** tbarron has quit IRC23:09
*** dannywilson has joined #openstack-cinder23:09
openstackgerritAnthony Lee proposed openstack/cinder: Adding manage/unmanage support for LeftHand driver  https://review.openstack.org/15512723:10
*** cebruns has joined #openstack-cinder23:11
*** jcru has quit IRC23:12
*** ganso_ has quit IRC23:17
*** ebalduf has joined #openstack-cinder23:17
*** annegentle has quit IRC23:20
*** nellysmi_ has quit IRC23:21
*** annegentle has joined #openstack-cinder23:22
*** ebalduf has quit IRC23:23
*** nellysmitt has joined #openstack-cinder23:23
*** angela-s has joined #openstack-cinder23:25
*** git-harry has quit IRC23:26
*** nellysmitt has quit IRC23:28
openstackgerritAnthony Lee proposed openstack/cinder: HP 3par driver filter and evaluator function  https://review.openstack.org/15135323:29
*** dannywilson has quit IRC23:31
*** annegentle has quit IRC23:33
*** dannywilson has joined #openstack-cinder23:33
*** dannywil_ has joined #openstack-cinder23:33
*** chlong has joined #openstack-cinder23:33
*** dannywilson has quit IRC23:37
*** openstackgerrit has quit IRC23:38
*** openstackgerrit has joined #openstack-cinder23:39
*** kaufer has quit IRC23:43
*** mriedem is now known as mriedem_away23:46
*** _cjones_ has quit IRC23:47
*** jungleboyj has quit IRC23:47
*** annegentle has joined #openstack-cinder23:48
*** _cjones_ has joined #openstack-cinder23:51
*** erlon has quit IRC23:51
*** hemna is now known as hemnafk23:53
*** thingee has quit IRC23:56
*** tbarron has joined #openstack-cinder23:57
*** EmilienM is now known as EmilienM|afk23:58

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