Thursday, 2015-04-23

*** mtanino has quit IRC00:00
asselinpatrickeast, yes...discussion on that going on in -infra00:01
*** markvoelker has joined #openstack-cinder00:01
*** tsekiyam_ has quit IRC00:01
*** Apoorva has quit IRC00:06
*** akerr_ has quit IRC00:06
*** Apoorva has joined #openstack-cinder00:06
*** ebalduf has joined #openstack-cinder00:07
*** rongze has joined #openstack-cinder00:10
*** ebalduf has quit IRC00:12
*** akerr has joined #openstack-cinder00:13
*** rongze has quit IRC00:15
*** akerr has quit IRC00:27
*** heyun has joined #openstack-cinder00:36
*** _cjones_ has quit IRC00:38
*** _cjones_ has joined #openstack-cinder00:39
*** Apoorva has quit IRC00:39
*** vilobhmm1 has quit IRC00:41
*** annashen has joined #openstack-cinder00:45
*** leeantho has quit IRC00:49
*** mtanino has joined #openstack-cinder00:58
*** annashen has quit IRC01:01
*** annashen has joined #openstack-cinder01:06
openstackgerritJohn Griffith proposed openstack/cinder: Remove force check from copy_volume_to_image
*** annashen has quit IRC01:06
jgriffithzigo: sorry, I was away:
jgriffithzigo: I generate one nightly and publish it to that web-site01:08
*** rongze has joined #openstack-cinder01:11
*** _cjones_ has quit IRC01:15
*** rongze has quit IRC01:15
*** akerr has joined #openstack-cinder01:24
*** akerr_ has joined #openstack-cinder01:26
openstackgerritJohn Griffith proposed openstack/cinder: Add resource tag to logging in
jgriffithhemnafk: tbarron ^^  I hope that satisfies the two of you.  Not worth arguing, but I would appreciate you at least read the comments and look at the logging guidelines.01:28
*** Lee1092 has joined #openstack-cinder01:28
*** akerr has quit IRC01:29
jgriffithhemnafk: tbarron particularly comment from me at 19:17, and the fact that we duplicate a single Info log message 376 times in a little more than a half hour period01:29
*** kaisers has quit IRC01:31
*** kaisers has joined #openstack-cinder01:32
*** markvoelker has quit IRC01:33
*** xyang has quit IRC01:37
openstackgerritThang Pham proposed openstack/cinder: Switch get_all_snapshots to use objects
*** haigang has joined #openstack-cinder01:41
*** IanGovett has quit IRC01:44
*** Longgeek has joined #openstack-cinder01:46
openstackgerritThang Pham proposed openstack/cinder: Complete switch to snapshot objects
openstackgerritDave Chen proposed openstack/python-cinderclient: Add commands for modifying image metadata
*** Longgeek has quit IRC01:56
*** Longgeek has joined #openstack-cinder01:56
*** davechen1 has joined #openstack-cinder01:57
*** david-lyle has quit IRC01:59
openstackgerritThang Pham proposed openstack/cinder: Switch to oslo_versionedobjects
*** annashen has joined #openstack-cinder02:07
*** ebalduf has joined #openstack-cinder02:09
*** nikesh has quit IRC02:11
*** annashen has quit IRC02:12
*** haigang has quit IRC02:12
*** harlowja is now known as harlowja_away02:12
*** annegentle has joined #openstack-cinder02:14
*** ebalduf has quit IRC02:14
*** _cjones_ has joined #openstack-cinder02:15
*** patrickeast has quit IRC02:18
*** adurbin_ has quit IRC02:19
*** annegentle has quit IRC02:19
*** markvoelker has joined #openstack-cinder02:22
*** lcurtis has quit IRC02:23
openstackgerritThang Pham proposed openstack/cinder: Switch to oslo_versionedobjects
*** haigang has joined #openstack-cinder02:39
*** dannywilson has joined #openstack-cinder02:40
*** dannywilson has quit IRC02:44
tbarronjgriffith: I think the overall principle is good, but not quite precise enough to fit this particular init-time-only, very-useful log msg.02:45
tbarronjgriffith: thanks for your flexibility on this.  You have my +1 on this now fwiw.02:46
tbarronWe've got to find some way to get the log messages coralled properly without diverting so much energy/resources on this.02:47
tbarronI want to move on the sp3llieng 3rr0rs02:47
tbarrongood night02:47
thingeejgriffith: yeah, I might have to add a condition to not do discovery if the endpoint contains a version already02:57
thingeeso it doesn't give those warnings02:57
*** rongze has joined #openstack-cinder03:02
*** akerr_ has quit IRC03:03
*** _cjones_ has quit IRC03:04
openstackgerritJeegn Chen proposed openstack/cinder: Remove redundant code from VNX Cinder Driver
*** vilobhmm1 has joined #openstack-cinder03:10
*** lan_ is now known as lan03:10
openstackgerritJohn Griffith proposed openstack/cinder: Add resource tag to logging in
*** rongze has quit IRC03:15
*** annashen has joined #openstack-cinder03:30
*** rongze has joined #openstack-cinder03:30
*** ebalduf has joined #openstack-cinder03:31
*** xyang has joined #openstack-cinder03:39
*** xyang has quit IRC03:42
*** esker has joined #openstack-cinder03:49
*** rongze has quit IRC03:58
*** rongze has joined #openstack-cinder04:01
*** breitz has quit IRC04:01
*** breitz1 has joined #openstack-cinder04:01
*** rongze has quit IRC04:04
*** annashen has quit IRC04:04
*** TobiasE has quit IRC04:04
*** rongze has joined #openstack-cinder04:06
*** rongze has quit IRC04:07
*** krtaylor has quit IRC04:08
*** krtaylor has joined #openstack-cinder04:11
*** esker has quit IRC04:16
*** esker has joined #openstack-cinder04:17
*** coolsvap|afk is now known as coolsvap04:31
*** coolsvap is now known as coolsvap|afk04:32
*** annashen has joined #openstack-cinder04:35
*** annashen has quit IRC04:48
*** esker has quit IRC04:51
*** david-lyle has joined #openstack-cinder04:59
*** sgotliv has joined #openstack-cinder05:00
*** mtanino has quit IRC05:01
*** pradipta has joined #openstack-cinder05:04
*** sks has joined #openstack-cinder05:05
*** deepakcs has joined #openstack-cinder05:10
*** bill_az has quit IRC05:10
*** nkrinner has joined #openstack-cinder05:12
*** bill_az has joined #openstack-cinder05:14
*** vilobhmm1 has quit IRC05:15
*** vilobhmm1 has joined #openstack-cinder05:17
*** krtaylor has quit IRC05:19
*** vilobhmm11 has joined #openstack-cinder05:19
*** annashen has joined #openstack-cinder05:20
*** markvoelker_ has joined #openstack-cinder05:21
*** markvoelker has quit IRC05:21
*** bswartz has quit IRC05:21
*** rwsu has quit IRC05:21
*** vilobhmm1 has quit IRC05:22
*** med_ has quit IRC05:24
*** ebalduf has quit IRC05:27
*** krtaylor has joined #openstack-cinder05:33
*** bswartz has joined #openstack-cinder05:35
*** med_ has joined #openstack-cinder05:36
*** med_ has quit IRC05:36
*** med_ has joined #openstack-cinder05:36
*** Longgeek has quit IRC05:44
*** _cjones_ has joined #openstack-cinder05:48
openstackgerrityogeshprasad proposed openstack/cinder: Add chap support to CloudByte cinder driver
*** xyang1 has quit IRC05:48
*** lpetrut has joined #openstack-cinder05:50
*** _cjones_ has quit IRC05:53
*** haomaiwang has quit IRC05:55
*** Longgeek has joined #openstack-cinder05:57
*** nikesh has joined #openstack-cinder05:57
*** emagana has joined #openstack-cinder05:59
*** Maike has joined #openstack-cinder06:01
*** BharatK has joined #openstack-cinder06:04
*** ankit_ag has joined #openstack-cinder06:08
*** afazekas_ has joined #openstack-cinder06:11
*** Longgeek has quit IRC06:16
*** anshul has quit IRC06:20
*** rongze has joined #openstack-cinder06:22
*** lpetrut has quit IRC06:23
*** Longgeek has joined #openstack-cinder06:26
*** ebalduf has joined #openstack-cinder06:28
*** ebalduf has quit IRC06:32
*** rongze has quit IRC06:32
*** anshul has joined #openstack-cinder06:33
*** Longgeek_ has joined #openstack-cinder06:43
*** Longgeek has quit IRC06:46
*** annashen has quit IRC06:46
*** sileht has quit IRC06:48
*** sgotliv has quit IRC06:50
*** haomaiwang has joined #openstack-cinder06:54
*** vilobhmm11 has quit IRC06:54
zigojgriffith: Well, see by yourself,
zigothere's lots of directives missing in the keystone_authtoken, as per: which I reported.07:06
openstackLaunchpad bug 1447380 in Cinder "wrong cinder.conf.sample generation: missing directives for keystone_authtoken (at least)" [Undecided,New]07:06
*** Maike has quit IRC07:06
openstackgerritTina Tang proposed openstack/cinder: Create consistgroup from cgsnapshot support in VNX driver
*** Longgeek_ has quit IRC07:09
*** sileht has joined #openstack-cinder07:10
*** Longgeek has joined #openstack-cinder07:11
*** ronis has joined #openstack-cinder07:11
*** rushiagr_away is now known as rushiagr07:12
*** davechen has joined #openstack-cinder07:14
*** davechen1 has quit IRC07:16
*** lpetrut has joined #openstack-cinder07:17
*** davechen1 has joined #openstack-cinder07:22
*** davechen has quit IRC07:24
*** bkopilov has quit IRC07:26
*** bill_az has quit IRC07:26
*** _cjones_ has joined #openstack-cinder07:37
*** jistr has joined #openstack-cinder07:38
openstackgerritPetrut Lucian proposed openstack/cinder: Windows SMBFS: Fix image resize errors during volume creation
openstackgerritPetrut Lucian proposed openstack/cinder: SMBFS: Fix retrieving total allocated size
*** e0ne has joined #openstack-cinder07:41
openstackgerritPetrut Lucian proposed openstack/cinder: SMBFS: Add minimum qemu-img version requirement
*** _cjones_ has quit IRC07:42
*** bkopilov has joined #openstack-cinder07:44
openstackgerritPetrut Lucian proposed openstack/cinder: Windows: Improve vhdutils error messages
openstackgerritJeegn Chen proposed openstack/cinder: Make VNX Cinder Driver aware of VNX Pool Full Threshold
openstackgerritPetrut Lucian proposed openstack/cinder: Fix fetch_to_volume_format if vhd is requested
*** coolsvap|afk is now known as coolsvap07:52
*** coolsvap is now known as coolsvap|afk07:53
*** Maike has joined #openstack-cinder07:53
*** e0ne is now known as e0ne_07:53
*** jordanP has joined #openstack-cinder08:11
*** e0ne_ is now known as e0ne08:14
*** bkopilov has quit IRC08:19
*** jordanP has quit IRC08:19
openstackgerritTina Tang proposed openstack/cinder: Attach/detach batch processing in VNX driver
*** e0ne is now known as e0ne_08:25
openstackgerritJulien Danjou proposed openstack/cinder: tests: replace mox by mox3, clean out mox usage
*** e0ne_ is now known as e0ne08:26
*** ebalduf has joined #openstack-cinder08:29
*** Longgeek has quit IRC08:31
*** Longgeek has joined #openstack-cinder08:31
*** e0ne has quit IRC08:32
*** ebalduf has quit IRC08:34
openstackgerritJulien Danjou proposed openstack/cinder: Leverage timeutils, drop strtime() usage
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Using mount method in RemoteFsClient
*** jordanP has joined #openstack-cinder08:43
*** vincepii has joined #openstack-cinder08:44
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Handle concurrent inventory folder create
*** alexpilotti has joined #openstack-cinder08:52
*** jwang has joined #openstack-cinder08:54
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: Using 'fallocate' instead of 'dd' to create raw volumes
*** jwang_ has quit IRC08:57
*** btran has quit IRC08:58
*** jwang_ has joined #openstack-cinder08:59
openstackgerritRick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver
*** jwang has quit IRC09:01
*** haomaiwang has quit IRC09:05
openstackgerritGorka Eguileor proposed openstack/cinder: Create iSCSI lio portals with right IPs and port
*** e0ne has joined #openstack-cinder09:13
openstackgerritYAMADA Hideki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver
openstackgerritBharat Kumar Kobagana proposed openstack/cinder: GlusterFS: Support over subscription in thin provisioning
*** ndipanov has joined #openstack-cinder09:18
*** kaisers has quit IRC09:20
*** kaisers has joined #openstack-cinder09:23
*** alecv has joined #openstack-cinder09:24
openstackgerritAnton Arefiev proposed openstack/cinder: Add missing backups entry to default quota class
*** d0ugal has quit IRC09:29
*** d0ugal has joined #openstack-cinder09:29
*** d0ugal is now known as Guest8147209:29
*** e0ne is now known as e0ne_09:31
*** haigang has quit IRC09:31
*** anshul has quit IRC09:32
*** aix has joined #openstack-cinder09:35
*** e0ne_ is now known as e0ne09:35
*** Guest81472 is now known as d0ugal209:40
*** anshul has joined #openstack-cinder09:44
*** annashen has joined #openstack-cinder09:47
*** d0ugal2 is now known as d0ugal09:47
*** d0ugal has quit IRC09:47
*** d0ugal has joined #openstack-cinder09:47
*** annashen has quit IRC09:51
*** e0ne is now known as e0ne_09:53
openstackgerritRushi Agrawal proposed openstack/cinder-specs: Snapshot sharing
*** Longgeek has quit IRC09:57
*** yuriy_n17 has joined #openstack-cinder09:59
*** Longgeek has joined #openstack-cinder09:59
*** davechen1 has quit IRC09:59
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement blueprint: check-volume-exists
*** e0ne_ has quit IRC10:03
*** e0ne has joined #openstack-cinder10:09
*** IanGovett has joined #openstack-cinder10:13
*** afazekas_ has quit IRC10:14
*** yamada-h has joined #openstack-cinder10:23
*** yamada-h has quit IRC10:23
*** afazekas has joined #openstack-cinder10:32
*** haomaiwa_ has joined #openstack-cinder10:38
*** rushiagr is now known as rushiagr_away10:40
*** pratyu has joined #openstack-cinder10:45
*** pratyu has quit IRC10:47
openstackgerritGorka Eguileor proposed openstack/cinder: Preserve usage and reservations on quota deletion
openstackgerritGorka Eguileor proposed openstack/cinder: Preserve usage and reservations on quota deletion
*** heyun has quit IRC10:50
*** dalgaaf has joined #openstack-cinder10:53
*** Maike has quit IRC11:04
*** _cjones_ has joined #openstack-cinder11:14
*** timcl has joined #openstack-cinder11:18
*** _cjones_ has quit IRC11:19
*** aix has quit IRC11:22
*** yamada-h has joined #openstack-cinder11:23
*** yamada-h has quit IRC11:28
*** david-lyle has quit IRC11:29
*** ebalduf has joined #openstack-cinder11:32
*** deepakcs has quit IRC11:36
*** ebalduf has quit IRC11:36
*** Maike has joined #openstack-cinder11:43
*** akerr has joined #openstack-cinder11:47
*** BharatK has quit IRC11:58
*** ociuhandu has joined #openstack-cinder11:59
*** aix has joined #openstack-cinder12:00
*** aix has quit IRC12:00
*** aix has joined #openstack-cinder12:00
*** yamada-h has joined #openstack-cinder12:02
*** dencaval has joined #openstack-cinder12:02
*** e0ne is now known as e0ne_12:04
*** david-lyle has joined #openstack-cinder12:05
*** yamada-h has quit IRC12:06
*** zhenguo has quit IRC12:12
openstackgerritwanghao proposed openstack/cinder: Implement function to import/export snapshots
*** rooneym has quit IRC12:14
openstackgerritJoel Coffman proposed openstack/cinder: Add test case for volume_encryption_metadata_get
openstackgerritwanghao proposed openstack/cinder: Implement function to import/export snapshots
openstackgerritwanghao proposed openstack/cinder: query volume detail support volume_glance_metadata
*** e0ne_ is now known as e0ne12:24
rmstarhi guys.  is thre a way to increase the download speed when i create a volume from an image?12:28
*** david-lyle has quit IRC12:31
*** kaisers has quit IRC12:33
openstackgerritJoel Coffman proposed openstack/cinder: Remove unnecessary checks for encrypted types
*** rooneym has joined #openstack-cinder12:37
*** kaisers has joined #openstack-cinder12:37
*** bswartz has quit IRC12:40
*** xyang1 has joined #openstack-cinder12:45
*** Longgeek has quit IRC12:47
*** erlon has joined #openstack-cinder12:47
*** Longgeek has joined #openstack-cinder12:48
*** pradipta has quit IRC12:49
*** Adriano_ has joined #openstack-cinder12:49
*** bill_az has joined #openstack-cinder12:51
openstackgerritwanghao proposed openstack/cinder-specs: Adds support to force-delete backups
*** Longgeek_ has joined #openstack-cinder12:55
*** Longgeek has quit IRC12:56
openstackgerritwanghao proposed openstack/cinder: non-admin user to query volume filter by az
*** vincepii has quit IRC12:57
*** rushil has joined #openstack-cinder13:00
openstackgerritwanghao proposed openstack/cinder: Fix response when query host detail by host name
*** Yogi1 has joined #openstack-cinder13:01
*** _cjones_ has joined #openstack-cinder13:03
*** yamada-h has joined #openstack-cinder13:03
*** davechen has joined #openstack-cinder13:05
*** zhipeng has joined #openstack-cinder13:06
*** _cjones_ has quit IRC13:09
*** yamada-h has quit IRC13:10
*** bswartz has joined #openstack-cinder13:15
*** davechen1 has joined #openstack-cinder13:16
*** davechen has quit IRC13:19
*** mriedem_away has quit IRC13:21
*** dustins has joined #openstack-cinder13:21
*** eduardo has joined #openstack-cinder13:21
*** annegentle has joined #openstack-cinder13:22
*** e0ne has quit IRC13:25
*** sks has quit IRC13:25
*** davechen1 has left #openstack-cinder13:26
*** e0ne has joined #openstack-cinder13:27
*** aswadr has joined #openstack-cinder13:30
smcginnisrmstar: Not really. I know at SolidFire has added some interesting image caching to their driver.13:32
*** ebalduf has joined #openstack-cinder13:34
*** mwichmann has joined #openstack-cinder13:35
*** jungleboyj has quit IRC13:36
*** ebalduf has quit IRC13:38
*** mriedem has joined #openstack-cinder13:47
*** kaisers has quit IRC14:00
*** Svedrin has quit IRC14:01
*** kaisers has joined #openstack-cinder14:02
*** ganso_ has joined #openstack-cinder14:03
*** ebalduf has joined #openstack-cinder14:07
*** eharney has joined #openstack-cinder14:08
jgriffithzigo: That's just awesome!14:12
*** esker has joined #openstack-cinder14:12
jgriffithzigo: sorry, I wasn't aware yesterday you found this problem14:13
jgriffithzigo: I thought you were just asking if there was a reference available :)14:13
zigojgriffith: Do you know what's the solution?14:13
jgriffithzigo: not at the moment... looking14:13
zigoThe generation seems quite broken to me! :)14:13
jgriffithzigo: this is the first I've been aware of it14:13
jgriffithzigo: well, dependins on your point of view14:14
jgriffithzigo: keystone/generation or something in oslo.config14:14
zigojgriffith: I'm using genrated config file other projects (like Ceilometer for example), with the exact same lib versions, and it works.14:14
jgriffithzigo: This is why I was not a fan of not gating on config14:14
*** lcurtis has joined #openstack-cinder14:14
jgriffithzigo: curious about Nova... bu that's a good data point14:15
zigoThough it's using oslo-config-generator directly, not the <project>/common stuff.14:15
zigoLet me check.14:15
jgriffithzigo: yeah, the project/common stuff may very well be a problem on our side14:15
zigoFor Nova, I use my own baked nova.conf, and just provide the sample as a doc.14:15
*** lpetrut has quit IRC14:15
jgriffithzigo: :)14:15
*** nkrinner has quit IRC14:16
jgriffithzigo: Dang... thought I may have found somebody that's been able to build a nova.conf14:16
jgriffithzigo: I've had bad luck with that14:16
zigojgriffith: Oh, fun, I have the same issue in Nova !!! :)14:16
jgriffithzigo: Yeah, that was what I suspected14:17
zigoWhich I didn't see, because of my custom nova.conf.14:17
zigoSo, the issue may well be simply in oslo-incubator.14:17
jgriffithzigo: It may be intentional.14:18
zigojgriffith: To remove some directives for the authtoken?14:18
jgriffithdhellmann: might have some insite ^^14:18
zigoThis would be a weird intention, no?14:18
jgriffithdhellmann: zigo noticed all the keystone directives are no longer in cinder/nova.conf files14:18
openstackLaunchpad bug 1447380 in Cinder "wrong cinder.conf.sample generation: missing directives for keystone_authtoken (at least)" [Undecided,Confirmed]14:18
dhellmannjgriffith: how is that file being generated?14:19
zigodhellmann: More precisely, the 2nd half of all the directives, including login/pass/tenant/hostname.14:19
jgriffithdhellmann: the old tox genconfig14:19
zigodhellmann: No problem with packages using oslo-config-generator directly, though there's the issue for nova and cinder which are using tools/config/
jgriffithzigo: dhellmann maybe I should just move us to use the generator directly14:20
jgriffithat least see what happens :)14:20
zigojgriffith: If you know the full command line to it, that's enough for me!14:20
zigojgriffith: I'd need a --namespace list, and that's it ...14:20
jgriffithis what we use currently14:20
dhellmannjgriffith: where are the missing options defined?14:20
zigodhellmann: In oslo.messaging.14:21
zigoBut that's the only one I *saw*.14:21
zigoMaybe there's more?14:21
dhellmannok, I think the old generator had support for the libraries through the entry point system, so the invocation may just need to be updated to add the right libs14:22
*** haomaiw__ has joined #openstack-cinder14:22
jgriffithdhellmann: thanks... so honestly I literally *just* saw zigo 's bug so I have no info other than that yet14:22
*** haomaiwa_ has quit IRC14:22
jgriffithdhellmann: thanks for the quick look/suggestion14:22
jgriffithI'll see if I can determine what's actually missing and if it *matters*14:23
dhellmannjgriffith: sure, and let me know if you have trouble making it work. today is going to be busy, but I think I'll have time tomorrow14:23
jgriffithzigo: there's a new keystone discovery thingy-magigy that might eliminate the need here depending on what's missing14:23
jgriffithdhellmann: awesome, thanks14:23
jgriffithzigo: first thing we need to verify is those opts aren't just dead as a result of auto discovery14:24
*** rwsu has joined #openstack-cinder14:24
*** anshul has quit IRC14:24
*** zhipeng has quit IRC14:25
e0nejgriffith: morning.14:28
*** mtanino has joined #openstack-cinder14:28
*** jungleboyj has joined #openstack-cinder14:30
*** haomaiw__ has quit IRC14:32
*** haomaiwa_ has joined #openstack-cinder14:32
zigoe0ne: Hi there!14:33
e0neit it ok that cinder-backup use swift if it failed to initialize other driver? ceph in my case14:33
e0nezigo: hi14:34
jgriffithe0ne: hello14:35
*** jistr is now known as jistr|mtg14:35
*** anshul has joined #openstack-cinder14:37
*** rmesta has joined #openstack-cinder14:37
e0nejgriffith: i'm trying to understans how cinder-backup should works14:38
tbarrone0ne: since only one backup service can be confiured (no scheduler, no multi-backend) I would think that we should hard-fail in that case rather than falling-back to Swift.14:38
*** haomaiwa_ has quit IRC14:38
tbarronDuncanT  ^^^^^14:38
e0nejgriffith: in my case i've got not well-configured cinder-backup service:
*** haomaiwang has joined #openstack-cinder14:39
e0netbarron: i'm agree with you. but it is not what we have now14:39
*** garysmith_ has joined #openstack-cinder14:40
tbarrone0ne: sounds like a bug to me14:40
*** thangp has joined #openstack-cinder14:41
*** emagana has quit IRC14:41
jgriffithe0ne: so I"m not sure I follow, but it should work "just like" cinder-volume backends, sans multi-backend14:42
*** emagana has joined #openstack-cinder14:42
tbarrone0ne: that paste is short, but are you sure it's not coming from a volume driver?  Backup manager loads the volume driver for each configured volume backend and runs its backup/restore method.  That's independent of whether Swift, Ceph, NFS, etc. is configured for the service itself.14:43
e0netbarron: i configured rbd backend for cinder-volume but didn't set backup_driver14:44
tbarron(I consider it a serious scaling issue that the backup service has to do this today.)14:44
tbarrone0ne: then you should get Swift14:45
tbarronthat's the default.14:45
e0netbarron: but it tries to connect to ceph14:45
tbarronbackup service is orthogonal to the cinder volume backends14:45
e0netbarron: and works with ceph if it is available14:45
tbarrone0ne: it should connect with Ceph for Ceph backend volumes, but back them up to Swift unless you have configured backup service to other than default.14:46
e0netbarron: if ceph is not available, it uses swift because it set as default14:46
*** emagana has quit IRC14:46
e0netbarron: thanks. your're right14:47
e0nei've missed it:(14:47
e0nei'm sorry for confuse14:47
tbarronI was very confuseed by the same thing when I started working on backup.14:48
tbarronbackup manager loads volume drivers.  They in turn use the backup service.  It's not intuitive.14:48
*** ankit_ag has quit IRC14:49
*** haomaiwang has quit IRC14:49
e0netbarron: thanks for help!14:49
akerrthat sounds like circular dependency...14:50
*** haomaiwang has joined #openstack-cinder14:50
openstackgerritJohn Griffith proposed openstack/cinder: Add resource tag to logging in
e0netbarron: we need more user friendly logs there14:50
tbarronHaving to load the backup drivers locally rather than doing a remote attach implies that the backup service node has to run all volume backends too.14:50
tbarronThat's a scaling limitation.14:51
akerrtbarron: no one backs up anyway, so not a problem :)14:51
tbarronInstead of scaling cinder services "out" one has to scale them "up" with a big honking single node.14:51
tbarronakerr: :-)14:51
jgriffithtbarron: I'd agree, TBH I haven't quite understood that architecture.14:52
jgriffithtbarron: assumed my ignorance is the problem, as DuncanT is typically very cognisant of things like scale14:53
tbarronjgriffith: I think Duncan told me that when we get remote attach for LVM we can address the issue to which I point.14:53
jgriffithtbarron: hmm... well, we already have remote attach for LVM :)14:54
jgriffithtbarron: its' called iSCSI14:54
tbarronjgriffith: so I'm probably confused then.14:54
DuncanTtbarron: The backup service just does an attach...14:54
DuncanTtbarron: Except for LVM14:54
tbarronhere's the man14:54
DuncanTtbarron: It needs some fixing up, but the principle is, I think, sound14:54
DuncanTtbarron: The service is usually CPU bound, particularly if you use SSL to swift14:55
*** xyang has joined #openstack-cinder14:55
DuncanT(just got an alert on my phone, not read scrollback yet)14:55
akerrDuncanT: what does the attaching?  The backup node?14:55
*** alexpilotti has quit IRC14:55
jgriffithsdague: you working that same push to Cinder?14:56
* tbarron stays tuned ...14:56
DuncanTakerr: The backup service. It uses the backup method in the driver, because different drivers can do it differently14:56
DuncanTakerr: But most will just do a iSCSI attach14:56
akerrso wouldn't you still be bound by a single node having to perform all backups?14:56
DuncanTakerr: No, the idea is you can have as many as you like14:57
akerrso you'd have multiple backup services?14:57
openstackgerritAlex Meade proposed openstack/cinder: NetApp E-Series: Fix instance live-migration with attached volumes
DuncanTakerr: Yes, they should be active/active/active/... scalable14:57
akerrand whoever gets the rabbit message first wins?14:58
DuncanTakerr: Correct14:58
*** thangp has quit IRC14:58
DuncanTakerr: At the moment it sends it to the hostname of the hosting volume node14:58
DuncanTakerr: That is because of the LVM driver14:58
akerrok, Thanks! I'll have to take another hard look at the code in my abundant free time15:00
DuncanTakerr: As I said, at the moment it goes to a named node, that needs fixing15:00
DuncanTakerr: It got changed at the last minute for LVM, and never got fixed up15:01
sdaguejgriffith: I thought someone else had that for cinder15:01
akerrDuncanT: ok, thanks15:01
jgriffithsdague: I'm doing it now15:02
jgriffithsdague: I'll look and make sure it's not already in the pipeline15:03
*** lpetrut has joined #openstack-cinder15:08
openstackgerritJohn Griffith proposed openstack/cinder: Sync oslo service module for SIGTERM exit
jgriffithsdague: thanks for tracking that down15:09
*** adurbin_ has joined #openstack-cinder15:09
*** ronis has quit IRC15:14
jgriffithzigo: I think I see the problem with config15:20
zigoCool! :)15:20
zigojgriffith: Add me to the review! :)15:20
jgriffithzigo: will do, assuming I'm correct :)15:20
smcginnisjgriffith: Could have sworn someone else had just submitted a patch for the SIGTERM issue. Is this related?15:22
jgriffithsmcginnis: maybe I missed it, sean thought the same thing15:22
jgriffithsmcginnis: I didn't see it though15:23
smcginnisjgriffith: Ah, I see why I couldn't find it. Merged last night.15:23
smcginnisjgriffith: Haven't looked close enough to see if they are different issues.15:23
jgriffithsmcginnis: umm, that's weird, if it merged why is the file still out of date?15:23
jgriffithsmcginnis: OH, he just cherry picked the line of the fix.  Didn't do a proper sync15:24
jgriffithsmcginnis: Ok, have to leave judgement on mine to thingee jungleboyj and dhellmann15:25
jgriffithtypically we used to discourage cherry picking rather than full syncs, guess that's not valid any longer15:25
smcginnisjgriffith: In my two second review, looks like what you are adding is needed/good.15:25
smcginnisjgriffith: Need to walk through it more to make sure I understand.15:25
jgriffithsmcginnis: seems like a sync is worthwhile IMHO, but I should change the commit message probably15:26
*** IanGovett has quit IRC15:26
jgriffithzigo: fixed15:27
jgriffithzigo: patch incoming15:27
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
dhellmannjgriffith: did you sync from master, or a specific commit?15:28
jgriffithdhellmann: I synced from master15:29
jungleboyjjgriffith: Need to run to a meeting.  Will look.15:29
*** david-lyle has joined #openstack-cinder15:29
dhellmannjgriffith: ok, I'm not seeing some changes I expected, but let me make sure I have my local copies up to date15:29
jgriffithdhellmann: hmmm15:29
dhellmannjgriffith: oh, I have the juno copy of cinder checked out, ignore me15:30
*** thangp has joined #openstack-cinder15:30
jgriffithdhellmann: phewww :)15:30
jungleboyjWe shouldn't be doing single line fixes of incubator code.15:30
jgriffithjungleboyj: umm, you gave it a +215:30
jungleboyjDoh!  And I plus 2'd that.  I mis interpreted what was going on there.15:30
jungleboyjjgriffith: Yeah, I see that.  :-(15:31
jgriffithand then thingee approved/merged it15:31
jungleboyjYesterday was one of those days.15:31
*** jdurgin1 has joined #openstack-cinder15:31
thingeeyeah was a mistake. we can still revert it15:32
thingeejgriffith: checked with ttx, we're making an exception on this one15:34
thingeecheck openstack-relmgr-office15:34
jgriffiththingee: cool15:34
jgriffiththingee: would've been good to communicate that for idiots like me :)15:35
jgriffiththingee: maybe a note in the commit mesg since it goes against usual process15:35
*** sandywalsh has quit IRC15:37
*** sandywalsh has joined #openstack-cinder15:38
thingeeso unless people start screaming really loud about blockers in Cinder, we'll be having an rc2 tag soon15:39
openstackgerritJohn Griffith proposed openstack/cinder: Remove incubator config and update genconfig
jgriffiththingee: you may want to consider adding that one ^^15:40
e0nethingee: what dou you think about getting to Kilo?15:43
openstackgerritJohn Griffith proposed openstack/cinder: Sync oslo service module
thingeee0ne: too late15:43
e0nethingee: ok, thanks15:44
jgriffithzigo: try that out15:44
jgriffithzigo: I kinda rushed/multi-tasked, may have missed something... lemme look15:45
*** bkopilov has joined #openstack-cinder15:46
*** akerr has quit IRC15:50
*** tsekiyama has joined #openstack-cinder15:52
*** _cjones_ has joined #openstack-cinder15:53
openstackgerritTom Barron proposed openstack/cinder: Fix range check for NFS used ratio
*** david-lyle has quit IRC15:56
* jungleboyj is back.15:56
*** _cjones_ has quit IRC15:57
jungleboyjjgriffith: Just added a link to one of my reviews with the format we have been using for Oslo syncs.15:58
openstackgerritEric Harney proposed openstack/python-cinderclient: Remove "OPTIONAL:" from optional argument help text (v2)
*** _cjones_ has joined #openstack-cinder16:01
xyangthingee: question for you one cinderclient release16:01
* jungleboyj is so confused. 16:01
thingeexyang: ok16:01
xyangthingee: s/one/on16:01
xyangthingee: is it released already for liberty?16:01
xyangthingee: is it released for kilo?16:02
xyangthingee: kmartin found a link yesterday about a release16:02
xyangthingee: that's good16:02
xyangthingee: as I have two patches that I want to get in16:02
xyangthingee: do you have a release date in mind?16:03
xyangthingee: that's easy:)  no for all questions16:03
thingeeworking on RC2 of cinder atm16:03
xyangthingee: sure.16:04
jgriffithjungleboyj: ?16:04
jgriffithjungleboyj: oh16:04
jungleboyjjgriffith: I am looking at
jgriffithjungleboyj: yeah, I'm with ya now16:05
jungleboyjI had started working on moving to the new config generator but that work was decided to be pushed off to Liberty.16:05
jungleboyjI didn't think we could just change which tool we were pointing to and go.  Otherwise I would have done that long ago.16:06
thingeecan I get an ok on that?16:06
jungleboyjI am trying your patch right now.16:06
thingeeoh heh16:07
*** garysmith_ has quit IRC16:07
jungleboyjthingee: That is what I am trying right now.  I am totally baffled that it works.16:07
*** hemnafk is now known as hemna16:07
thingeejungleboyj: blocking rc2 on this right now16:07
jungleboyjthingee: Ok, I am looking at it as we speak.16:07
jungleboyjOk, that was what I was afraid of.16:08
jungleboyjOh wait.16:08
*** emagana has joined #openstack-cinder16:08
jungleboyjOk, I am not nuts.  Thank god.16:10
thingeeno, you're still nuts16:10
jungleboyjthingee: :p16:10
hemnajgriffith, ping16:11
jungleboyjjgriffith: I am looking at the bug.16:13
*** emagana has quit IRC16:13
*** emagana has joined #openstack-cinder16:16
*** aix has quit IRC16:17
openstackLaunchpad bug 1447697 in Cinder "3PAR drivers are logging version numbers a lot" [Undecided,New]16:18
*** jistr|mtg is now known as jistr16:18
thingeejungleboyj: thanks. I would be a bit nervous at this time with your initial patch. just because of size16:20
jungleboyjthingee: Totally agree.16:21
*** Apoorva has joined #openstack-cinder16:21
jungleboyjthingee: I have an idea.16:21
jungleboyjthingee: I am thinking that we can use the old genconfig to create the cinder options and use the new one to pull in the options from the libraries.16:22
jungleboyjA temporary fix until we work out how best to use the new one.  Give me a few minutes to see if I can get something that looks right.16:23
*** vilobhmm1 has joined #openstack-cinder16:25
*** crose has joined #openstack-cinder16:28
*** _cjones_ has quit IRC16:29
*** vilobhmm1 has quit IRC16:29
*** _cjones_ has joined #openstack-cinder16:31
*** patrickeast has joined #openstack-cinder16:31
jungleboyjthingee: I need to go get my son to daycare.  Will be back in 30 min or so.  I am working on a solution that will get us by until I can get us fully moved to the new genconfig.16:34
jungleboyjI think we can use the old config generator to get Cinder's options pulled in and then combine that with the output from the new generator for the libraries.16:35
*** dannywilson has joined #openstack-cinder16:35
*** e0ne has quit IRC16:36
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR don't log version numbers every stats update
*** nestorf has joined #openstack-cinder16:38
*** alecv has quit IRC16:40
*** Yogi1 has quit IRC16:40
*** akerr has joined #openstack-cinder16:40
*** leeantho has joined #openstack-cinder16:43
*** jistr has quit IRC16:43
nestorfis it possible to set logging level for just my driver?16:44
openstackgerritPatrick East proposed openstack/cinder: Enable use of filter_function in PureISCIDriver
hemnanestorf, we do something like that for the 3PAR client16:45
hemnawe have a cinder.conf entry that allows you to turn on hp3par_debug = True16:45
hemnawhich bascially creates a new logger with that new level.16:46
hemnanestorf, so yes, it's possible.  You just have to do it all yourself in your own driver.16:46
hemnaI wouldn't really recommend it though.16:46
*** ronis has joined #openstack-cinder16:47
*** jdurgin1 has quit IRC16:47
*** yuriy_n17 has quit IRC16:50
*** jordanP has quit IRC16:52
*** dustins_ has joined #openstack-cinder16:56
*** vilobhmm11 has joined #openstack-cinder16:57
*** vilobhmm12 has joined #openstack-cinder16:58
*** Yogi1 has joined #openstack-cinder16:58
*** dustins has quit IRC16:59
*** vilobhmm11 has quit IRC17:01
*** rushil has quit IRC17:02
*** cbader has quit IRC17:03
*** cbader has joined #openstack-cinder17:03
*** rushil has joined #openstack-cinder17:04
*** alexpilotti has joined #openstack-cinder17:05
*** rushiagr_away is now known as rushiagr17:07
*** Longgeek_ has quit IRC17:07
jgriffithhemna: thanks17:11
jgriffithjungleboyj: thingee wait... what about genconfig?17:12
*** garthb has joined #openstack-cinder17:12
*** garthb_ has joined #openstack-cinder17:12
*** dannywilson has quit IRC17:12
jgriffithgood lord!17:13
*** harlowja_away is now known as harlowja17:13
*** alexpilotti has quit IRC17:16
*** marcusvrn has quit IRC17:20
*** marcusvrn has joined #openstack-cinder17:20
*** Maike has quit IRC17:23
*** ndipanov has quit IRC17:23
jungleboyjjgriffith: I am back.17:23
jungleboyjjgriffith: I am working that.17:23
jgriffithjungleboyj: I abandoned my change, but I think you should reconsider the direction on your patch17:24
jgriffithjungleboyj: rather than the crazy deepcopy +.....17:24
jgriffithuse an iterator and just build a list17:24
jgriffithnot sure how/if that can be automated, should be able to17:24
jungleboyjjgriffith: Yeah, I am open to doing that one differently.  That isn't going to happen for Kilo though.17:24
jgriffithjust walk all the files17:25
jungleboyjjgriffith: That patch needs to be completely rethought.17:25
jgriffithjungleboyj: yeah, we pretty much F'd up there17:25
*** marcusvrn1 has joined #openstack-cinder17:26
jungleboyjThat was the result of discussion with the Oslo community.  They sent me off to see what I could do.17:26
jungleboyjSo, we get to set the approach there and others will follow.17:27
jungleboyjThat isn't going to happen today though.17:27
*** marcusvrn has quit IRC17:27
jgriffithjungleboyj: so actually, it's not hard I don't think17:27
jgriffithjungleboyj: you can walk the files and gather the register_opts files dynamically17:27
*** jwcroppe has joined #openstack-cinder17:28
jgriffithbuild a list from them17:28
jungleboyjOk, that would be better than what I did.17:28
jungleboyjjgriffith: Wasn't sure if it was valid to build that file up dynamically though.17:28
jgriffithjungleboyj: ummm... why not?17:28
jgriffithjungleboyj: considering that's how it's worked for *years* until this change?17:29
jungleboyjWell right, but we built up a list in the script now building another .py file.17:29
jungleboyjIf you are ok with it though, I can look into it.17:30
jgriffithjungleboyj: so you're kinda screwed as you still have to build up the import list17:31
jgriffithjungleboyj: but you could then walk each of the imports dynamically at least17:31
jgriffithjungleboyj: and get rid of that deppcopy + stuff17:32
jgriffiththat's kinda fugly17:32
jgriffithmight be a more clever way... gimmie a minute17:32
jungleboyjjgriffith: Hold on.17:33
jungleboyjjgriffith: I don't think we want to go that direction last minute.17:33
jgriffithjungleboyj: yeah, much better to just leave it broken :)17:33
jgriffithand useless :)17:33
jungleboyjjgriffith: No no.  :-)17:33
jgriffithjungleboyj: I know, your two staged diff idea17:34
jungleboyjLet me try something here that is less of a departure from the current approach thatwas working.17:34
jgriffiththat's fine for kilo17:34
jungleboyjjgriffith: Right.17:34
*** AlkaD has joined #openstack-cinder17:34
jgriffithIt works17:34
jgriffithI have it running17:34
jungleboyjjgriffith: What works?17:35
jgriffithI run genconfig-global out using the libs to get external stuff....17:35
jgriffithThen run the incubator config gen for local stuff17:35
jgriffithand merge them together into a single file17:35
jungleboyjRight.  That was what I was going to put together.17:35
jungleboyjYou already have that written?17:36
*** patrickeast_ has joined #openstack-cinder17:36
jgriffithYeah, but it's not anything I can submit right now17:36
jungleboyjOk, that was what I was trying to put together for submission.17:36
jgriffithIt's two scripts17:36
jungleboyjDo you want to keep working that or do you want me to get something together?17:37
*** patrickeast has quit IRC17:37
*** patrickeast_ is now known as patrickeast17:37
jgriffithjungleboyj: I can do it if you guys won't object and -1 it to death :)17:41
jgriffithjungleboyj: so I just modified the script...17:42
jgriffithoslo-config-generator --namespace keystonemiddleware.auth_token >> etc/cinder/cinder.conf.sample17:42
jungleboyjOk, that is similar to what I was just looking at doing.17:43
jungleboyjjgriffith: If you want to push that up I will give it a try.17:44
*** sileht has quit IRC17:44
*** angela-s has joined #openstack-cinder17:45
*** lpetrut has quit IRC17:45
*** annegent_ has joined #openstack-cinder17:46
*** jwcroppe has quit IRC17:48
*** jwcroppe has joined #openstack-cinder17:48
*** annegentle has quit IRC17:50
jgriffithjungleboyj: quite frankly we should probably just stick with something like this anyway17:52
jgriffithjungleboyj: hybrid of the old incubator code and the new oslo.confg tool17:52
jgriffithjungleboyj: maybe... beats the hell out of harcoding all that nonsense IMO17:53
*** AlkaD has quit IRC17:53
*** e0ne has joined #openstack-cinder17:53
*** sileht has joined #openstack-cinder17:55
*** crose has quit IRC17:55
jungleboyjjgriffith: Agreed.  I still think we should move to using the new newer generator as others have done that.  I just need to come up with a better approach than I currently have that doesn't require a big hacking check.17:58
jungleboyjjgriffith: Just haven't gotten back to that yet.17:58
jgriffithjungleboyj: yeah, so what I'm saying is use both :)17:58
jgriffithjungleboyj: but modify the methods in incubator a bit17:59
jgriffithI'll push that change up when I get back... 5 minutes or so17:59
jgriffithtoo many interruptions17:59
*** jwcroppe has quit IRC17:59
jungleboyjTell me about it.17:59
*** aswadr has quit IRC18:01
*** e0ne is now known as e0ne_18:01
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements
*** jwcroppe has joined #openstack-cinder18:05
*** Yogi11 has joined #openstack-cinder18:05
*** e0ne_ has quit IRC18:06
*** david-lyle has joined #openstack-cinder18:07
*** Yogi1 has quit IRC18:07
openstackgerritJohn Griffith proposed openstack/cinder: Add external genconfig calls
*** melwitt has joined #openstack-cinder18:11
*** e0ne has joined #openstack-cinder18:13
*** breitz1 is now known as breitz18:21
thingeejgriffith, jungleboyj disappearing for a bit, bb in an hour18:23
*** morganfainberg is now known as grebniafnagrom18:24
thingeegrebniafnagrom: it's not casual nick friday :)18:24
*** grebniafnagrom is now known as morganfainberg18:24
thingeeunless you're going incognito18:24
morganfainbergsomeone made some "reflection" joke18:25
*** bswartz has quit IRC18:26
*** bswartz has joined #openstack-cinder18:27
*** alexpilotti has joined #openstack-cinder18:27
*** e0ne is now known as e0ne_18:28
*** e0ne_ is now known as e0ne18:29
ganso_Hello guys. I am having difficulties finding the CLI command for volume retype... could anyone help me?18:33
mtaninojungleboyj: hi,18:34
mtaninojungleboyj: May I ask a question related to this fix?
jungleboyjmtanino: Sure.18:35
mtaninojungleboyj: so the fix is backporting partial patch from Kilo tree. In this case, we also need to add cherry-pick comment?18:36
*** dalgaaf has quit IRC18:37
mtaninothank you :)18:38
mtaninopartialy or fully is not matter for adidng cherry-pick comment?18:39
*** e0ne is now known as e0ne_18:40
jungleboyjLooking again, I probably made that comment too quickly.18:41
mtaninojust a confirmation for the policy.18:41
jungleboyjYeah, this is ok since they couldn't bring the whole thing over.18:43
*** e0ne_ is now known as e0ne18:44
mtaninoyes. whole patch can't backport to Juno/stable, this bug need just one method to back-port.18:44
jungleboyjmtanino: Thanks for pointing that out.18:45
*** alexpilotti_ has joined #openstack-cinder18:46
mtaninojungleboyj: thank you for updating :)18:47
jungleboyjmtanino: Welcome.18:47
*** alexpilotti has quit IRC18:49
*** alexpilotti_ is now known as alexpilotti18:49
jungleboyjjgriffith: Patch was what I was thinking of.  Just need to change the namespace stuff to oslo_db instead of oslo.db, etc.18:49
jungleboyjjgriffith: Just in case we keep using this approach for a while.18:49
*** esker has quit IRC18:50
*** xyang has quit IRC18:51
*** AlkaD has joined #openstack-cinder18:54
*** jwcroppe has quit IRC18:54
jungleboyjIt is a known issue that trying to attach using 'auto' will some times result in an already used mount point being attempted.  Right?18:55
*** jwcroppe_ has joined #openstack-cinder18:58
*** kaisers has quit IRC19:05
*** kaisers has joined #openstack-cinder19:06
*** patrickeast has quit IRC19:10
*** _cjones_ has quit IRC19:12
*** rushiagr is now known as rushiagr_away19:14
*** jgravel has left #openstack-cinder19:18
*** _cjones_ has joined #openstack-cinder19:19
*** esker has joined #openstack-cinder19:29
*** bill_az has quit IRC19:32
*** liusheng has quit IRC19:34
*** ociuhandu has quit IRC19:38
*** annegent_ has quit IRC19:40
*** _cjones_ has quit IRC19:40
*** annegentle has joined #openstack-cinder19:41
*** lpetrut has joined #openstack-cinder19:41
*** ronis has quit IRC19:42
openstackgerritJay Bryant proposed openstack/cinder: Sync oslo service module
jungleboyjjgriffith: eharney ^^^ I fixed up the commit message since I am generally the one using those to make updates.  :-)19:49
*** dannywilson has joined #openstack-cinder19:49
*** dannywilson has quit IRC19:49
*** dannywilson has joined #openstack-cinder19:50
*** annashen has joined #openstack-cinder19:58
*** mtanino has quit IRC19:58
*** patrickeast has joined #openstack-cinder19:59
openstackgerritJohn Griffith proposed openstack/cinder: Add external genconfig calls
jgriffithjungleboyj: thanks!19:59
jgriffithjungleboyj: long meeting then lunch20:00
jungleboyjWelcome.  I have the process down and figured you wouldn't mind me doing the busy work.20:00
ameadeDuncanT: had to rebase since the unittests moved,
jgriffithjungleboyj: do we have that documented anywhere?20:00
jgriffithwe == openstack20:00
jungleboyjjgriffith: Good question.  I just always go back to the last one I did and use that as my template.20:01
jgriffithjungleboyj: hehe20:01
jgriffithsounds familiar20:01
e0nejgriffith, jungleboyj: is it that you ask?20:03
jgriffithe0ne: Perfect20:04
*** dencaval has quit IRC20:04
jungleboyjjgriffith: Yeah, that looks like the place to update.20:04
*** annashen has quit IRC20:04
jgriffithexcept it doesn't have the req for the commit messages20:04
jgriffithjungleboyj: e0ne syncing is easy, I do that just fine on my own thanks20:05
jgriffithjungleboyj: e0ne the "requirement" for the commit/change ID's is the part that I got -1'd on20:05
jgriffiththat I don't know that we've ever really documented20:05
jgriffithor why we don't automate that in the tool perahps20:06
*** annashen has joined #openstack-cinder20:06
e0nejgriffith: good question:)20:06
jungleboyjAutomating it has been discussed and deemed too hard.20:06
jungleboyjMaybe that isn't the right answer though.20:06
jungleboyj+2 on the genconfig change.20:06
jungleboyjI need to drop off for a while.20:07
*** Apoorva has quit IRC20:08
*** _cjones_ has joined #openstack-cinder20:09
*** annashen has quit IRC20:12
*** mtanino has joined #openstack-cinder20:12
vilobhmm12Unable to update stats, RBDDriver -1.1.0  driver is uninitialized mostly we get this when the volume_driver is not specified in cinder.conf right i am seeing this consistently on 2014.1.3 tag any suggestions ?20:13
*** jungleboyj has quit IRC20:13
vilobhmm12i think i have made the needed changes in cinder.conf20:13
vilobhmm12to enable it20:13
e0nejgriffith: looks like this requirement is cinder only20:14
e0nejgriffith: i found request for heat
kmartinjgriffith, asselin checked and saw that nova is running a live migration test (tempest.api.compute.admin.test_live_migration.LiveBlockMigrationTestJSON.test_live_block_migration) as part of dsvm multinode full see
e0neand for glance:20:15
kmartinjgriffith, cinder is currently skipping the test20:15
jgriffithkmartin: because cinder doesn't run "multi-node"20:17
jgriffithkmartin: if it is, you or asselin should enable the test :)20:17
jgriffithkmartin: btw, tested again last night, works fine for LVM and SolidFire20:17
jgriffithkmartin: dind't hear if xyang1 had a chance to try it or not20:18
asselinjgriffith, it's easier said than done as it requires multi-node setup.20:18
kmartinjgriffith, yep, we are aware of the problem with 3PAR and working on fixing20:18
xyang1jgriffith: Live migration? Not yet20:18
asselinjgriffith, but we'll get to it20:18
jgriffithasselin: I'm not sure if my text is garbled or what...20:18
jgriffithasselin: but yes, that's correct, and that's what I've been saying since yesterday morning :)20:19
jgriffithasselin: I wasn't saying you should enable multinode for Cinder today either :)20:19
jgriffithasselin: I was just telling kmartin that that's why it's "skipped" for Cinder20:19
jgriffithasselin: and that if I was out of the loop and the multi-node devstack for Cinder was in place somewhere then by all means we shoudl enable it.  That's all20:19
jgriffithasselin: sorry for the confusion there20:20
asselinjgriffith, ok...I missed yesterday's conversation....20:20
jgriffithasselin: No worries20:20
*** Apoorva has joined #openstack-cinder20:20
jgriffithI just don't want confusion/fud from statements like: 20:22 < kmartin> jgriffith, cinder is currently skipping the test20:20
asselinjgriffith, well we can enable it for cinder easily enough in upstream. probably 1 line needs to be added20:21
jgriffithasselin: right, but until we do the node config for dsvm-mutli it's irrelevant :)20:21
asselinok..dsvm full is skipping the test. dsvm multinode full runs it20:21
jgriffithasselin: :)20:21
jgriffithasselin: so that's perfect20:21
jgriffithas it should be20:22
*** AlkaD has quit IRC20:23
smcginnispatrickeast: ping20:24
patrickeastsmcginnis: hola20:24
smcginnispatrickeast: Hey! Did you see my response here:
smcginnispatrickeast: Wondering if you are cool with that or if you'd prefer I change some of those back.20:24
smcginnispatrickeast: And earlier reviewer wanted me to change the two step process.20:25
smcginnispatrickeast: I was picking my battles. :)20:25
patrickeastsmcginnis: ah yea, sry, forgot to change my review, i’m fine with it20:25
smcginnispatrickeast: OK, cool.20:25
patrickeastsmcginnis: i figured i would at least point it out, but its not like it breaks anything20:25
smcginnispatrickeast: Good to know of others opinions though. I will use that as backing next time.20:25
smcginnispatrickeast: Yeah, my thoughts too. I appreciate it.20:26
smcginnispatrickeast: Thanks!20:26
zigojgriffith: Running the command that you wrote in the tox.ini worked for me.20:26
*** e0ne is now known as e0ne_20:26
zigojgriffith: I mean, the keystone_authtoken was right.20:26
zigojgriffith: I'll now run a full install and tempest run with it...20:27
*** e0ne has joined #openstack-cinder20:27
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR don't log version numbers every stats update
*** dustins_ has quit IRC20:40
thingeejgriffith: hey back. what's new with genconfig?20:42
*** annashen has joined #openstack-cinder20:42
thingeeah found
*** akerr has quit IRC20:47
*** e0ne has quit IRC20:47
*** Lee1092 has quit IRC20:51
*** Svedrin has joined #openstack-cinder20:57
thingeejgriffith: thanks
*** lpetrut has quit IRC21:07
*** bswartz has quit IRC21:08
jgriffiththingee: awesome21:14
jgriffithzigo: to the problem with that was it only did the external bits.21:15
*** annegentle has quit IRC21:15
jgriffithzigo: should be fine for what was "missing" and what you needed.21:15
jgriffithzigo: I've updatd that patch now to just do all of that in the gen-config script21:15
*** thangp has quit IRC21:15
zigojgriffith: I thought you abandonned it?21:16
jgriffithhemna: yeah, read that... always wondered how much they "really" made21:16
hemnayah interesting21:16
jgriffithzigo: I did.. but added:
jgriffithhemna: Nice to know a big chunk of that comes from me :)21:17
jgriffithLike 10 bucks!!!21:17
*** Yogi11 has quit IRC21:17
zigojgriffith: That's the only bits missing?21:17
jgriffithzigo: yep, that should be it21:17
jgriffithzigo: unless we're using other libs that weren't included21:18
jgriffithzigo: which is certainly possible :)21:18
openstackgerritJulie Gravel proposed openstack/cinder: Validate volume type extra specs
*** jgravel has joined #openstack-cinder21:18
zigojgriffith: I have my own tooling to do the requirements.txt -> debian/control Depends:, which I use to for double checking, so I'm quite sure I have it correct ! :)21:19
zigojgriffith: Thanks a lot for you help btw.21:19
jgriffithzigo: absolutely21:20
*** annegentle has joined #openstack-cinder21:20
*** timcl has quit IRC21:22
*** cbader has quit IRC21:26
*** eharney has quit IRC21:28
*** marcusvrn1 has quit IRC21:29
*** marcusvrn has joined #openstack-cinder21:29
thingeehemna: updated os-brick jobs...that explains why pypi publish didn't work21:36
thingeemissed that when I was going through the docs verifying things21:36
hemnaah ok.21:36
hemnasorry about that.   I just followed the docs at the time21:37
thingeehemna: yeah I think they were updated after you did things21:37
thingeealso now we have docs being gated21:37
hemna:)  awesome21:37
thingeehemna: someone from infra will manually trigger the publish so we don't have to retag21:38
hemnaok cool21:38
*** akerr has joined #openstack-cinder21:43
*** annegentle has quit IRC21:44
*** akerr has quit IRC21:48
hemnathingee, jenkins failed21:55
*** jungleboyj has joined #openstack-cinder21:55
jungleboyjAnyone have a concern with me merging this:
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Sync oslo service module
hemnaneeded a rebase21:56
jungleboyjAh, thank you.21:59
jungleboyjThat should probably also get into RC2.  Do you agree thingee ?22:00
thingeehemna: yeah I think I fixed it...just reup'd22:00
thingeejungleboyj: why?22:01
hemnajungleboyj, dunno about putting that into RC222:02
jungleboyjWell, because it fixes namespace deprecation warnings and is the change that should have been pulled in and backported for RC.22:02
hemnathat's not really needed unless something is broken no ?22:02
*** rushil has quit IRC22:02
*** Guest47750 has joined #openstack-cinder22:03
thingeeunless things are really broken, I don't want to hold things up anymore if possible22:03
hemnajungleboyj, are deprecation warnings the worst of it ?22:04
hemnato me, that's a 'meh'.22:04
jungleboyjhemna: Yes.22:04
hemnacompared to the risk.22:04
*** mriedem is now known as mriedem_away22:04
jungleboyjhemna: I agree but people here, internally are grumbling at me about it.22:04
jungleboyjI can just fix it internally for now.22:04
hemnathere I fixed it.22:05
hemnajust add a #22:05
thingeejungleboyj: need to get rid of the warnings from the version discovery patch in cinderclient.22:05
thingeejungleboyj: jgriffith reminded me about it yesterday22:05
thingeeif a version is passed into the endpoint, we should avoid discovery22:05
thingeeotherwise keystone complains22:05
morganfainbergthingee: this is the version discovery warning thing in logS?22:06
thingeemorganfainberg: it's a valid warning from keystone
morganfainbergah yeah22:07
morganfainbergthat is valid22:08
morganfainbergeither you fix the versioned endpoint so discovery still works or you avoid it22:08
morganfainbergi think the former is better.22:08
morganfainbergbut i defer to your choice on attacking that22:08
morganfainbergthough discovery with the replaced values is weird22:09
*** openstackstatus has quit IRC22:09
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR don't log version numbers every stats update
*** Apoorva has quit IRC22:20
*** jwcroppe_ has quit IRC22:30
vilobhmm12thingee : ping22:31
vilobhmm12hemna : ping22:31
hemnavilobhmm12, yough22:31
vilobhmm12quick question when trying to create volume on ceph backend22:32
vilobhmm12getting "Unable to update stats, RBDDriver -1.1.0  driver is uninitialized"22:32
vilobhmm12so from my understanding22:32
*** Apoorva has joined #openstack-cinder22:32
hemnasomething happened at driver startup22:32
vilobhmm12volume manager->>librbd->librados22:32
vilobhmm12and that talks to ceph backend22:32
vilobhmm12cinder-volume is running on api node22:32
vilobhmm12so during volume creation the compute should not come into picture neither qemu etc stuff22:33
jungleboyjhemna: Thanks for adding that comment in the code.  Thought there was something I didn't know going on there.22:33
vilobhmm12its simply betwn the node runing cinder volume service and the backend on which to create volume righ ?22:33
hemnajungleboyj, np.  I figured after jdg asked about it, I should say something in the code to mention it.22:34
jungleboyjI had pinged you asking about it too.  :-)  Don't know if you saw that.22:34
hemnajungleboyj, ah crap, I didn't see that sorry22:34
jungleboyjThat is ok.  I was going to harass you again at some point.22:35
jungleboyjI feel a lot better now that I know I wasn't the only one baffled.22:35
vilobhmm12hemna : ^^22:35
hemnavilobhmm12, if you are getting that message above, the create_volume is never being called.22:36
*** alexpilotti has quit IRC22:36
hemnathe volume manager won't call create_volume if the driver isn't marked as initialized.22:36
hemnaor it shouldn't22:36
vilobhmm12and what are the reason for which the driver isn't marked as initialized ?22:38
vilobhmm12can see anything obvious here
hemnasomething in the driver failed at volume manager init_host time.22:39
hemnamisconfiguration ?22:39
*** lcurtis has quit IRC22:39
hemnaand in fact init_host has to do a bunch of stuff, before marking the driver 'initialized'22:40
hemnathat happens here:
hemnavilobhmm12, that is the only time the driver gets a chance to get marked as initialized.22:40
vilobhmm12sure…i did have a look at the second link22:40
vilobhmm12thanks will check the conf again22:41
vilobhmm12and see if i am missing something22:41
*** nestorf has quit IRC22:41
hemnaif you can, restart cinder and look at the log output right before all the conf entries start to get dumped22:41
*** nestorf has joined #openstack-cinder22:41
hemnathat's where you'll most likely see an exception for init_host time failures.22:41
vilobhmm12oh okay22:42
vilobhmm12lemme try it22:42
*** Guest47750 has quit IRC22:44
vilobhmm12hemna : you were right it pukes here
vilobhmm12looks like some access permission will fix it22:46
hemnayup.  there ya go.22:46
hemnanp.  glad I could help!22:48
*** erlon has quit IRC22:51
*** rhagarty has quit IRC22:51
*** rhagarty has joined #openstack-cinder22:52
*** angela-s has quit IRC22:57
*** annashen has quit IRC23:00
*** annashen has joined #openstack-cinder23:00
*** melwitt has quit IRC23:02
*** annegentle has joined #openstack-cinder23:03
*** rmesta has quit IRC23:05
*** annashen has quit IRC23:05
thingeevilobhmm12: hi23:05
*** hemna is now known as hemnafk23:07
*** annegentle has quit IRC23:14
vilobhmm12thingee : never mind got it resolved after talking to hemna…some weird accept permission was blocking me was not obvious after looking at code23:15
*** ganso_ has quit IRC23:17
*** julim has quit IRC23:28
uberjayexception raised from volume drivers: are there any rules or guidelines about what exceptions to raise? is any meaning derived from what exception class is used, or is it just dumped to the log? (i.e., should I be raising VolumeBackendAPIException, or does it not matter?) along the same lines, if there's a failure in create_volume (for example), should I bother logging something to the tune of "Create volume failed for reason [X]", or should I23:28
uberjay just log details of the underlying error condition and assume that a higher layer will make it clear what context the failure occurred in?23:28
uberjayI've been watching all of the logging related churn, and would like to save time and try to get it right before submitting for review23:30
thingeeuberjay: so the manager most has some catchall exception in place. It'll take care of the logging.23:32
thingeeuberjay: example
thingeeuberjay: if you look at other drivers, they'll define their own exceptions inheriting VolumeDriverException
*** adurbin_ has quit IRC23:33
uberjaythingee: ok, so there's really no reason to log it myself, unless there is additional diagnostic information worth recording that doesn't really fit into an exception message?23:34
*** zhenguo has joined #openstack-cinder23:34
uberjaythingee: ah, gotcha23:35
thingeeif there are things specific to your driver that warrants additional logging you're more than welcome to add what you feel is necessary. You can find guidelines for logging here
* thingee adds this to
uberjaymm, yeah, I've been watching the recent logging-related commits and that's what got me down this path. i'd like to make sure the driver is doing the right thing and save everyone's reviewing time :)23:36
thingeeuberjay: you're welcome :)23:37
*** ebalduf has quit IRC23:41
*** vilobhmm12 has quit IRC23:42
*** vilobhmm1 has joined #openstack-cinder23:42
*** vilobhmm11 has joined #openstack-cinder23:46
*** vilobhmm1 has quit IRC23:47
*** garthb_ has quit IRC23:51
*** garthb has quit IRC23:51
*** bswartz has joined #openstack-cinder23:54
*** atan8 has joined #openstack-cinder23:58

Generated by 2.14.0 by Marius Gedminas - find it at!