| *** harlowja has quit IRC | 00:02 | |
| *** harlowja has joined #openstack-cinder | 00:03 | |
| *** vilobhmm11 has joined #openstack-cinder | 00:16 | |
| *** emagana has quit IRC | 00:19 | |
| *** sseverson has quit IRC | 00:19 | |
| *** vilobhmm1 has quit IRC | 00:19 | |
| *** annashen has quit IRC | 00:19 | |
| *** dflorea has joined #openstack-cinder | 00:30 | |
| *** cpg is now known as cpg|away | 00:32 | |
| *** vilobhmm11 has quit IRC | 00:32 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Add missing qos_specs_id to volume type show https://review.openstack.org/169153 | 00:36 |
|---|---|---|
| *** rushil has quit IRC | 00:48 | |
| *** dflorea has quit IRC | 00:49 | |
| *** dflorea has joined #openstack-cinder | 00:50 | |
| *** mwichmann has quit IRC | 00:51 | |
| *** _cjones_ has quit IRC | 00:54 | |
| *** dflorea has quit IRC | 00:54 | |
| *** leeantho has quit IRC | 01:00 | |
| *** vilobhmm1 has joined #openstack-cinder | 01:00 | |
| *** MentalRay has quit IRC | 01:01 | |
| *** MentalRay_ has joined #openstack-cinder | 01:01 | |
| *** Lee1092 has joined #openstack-cinder | 01:05 | |
| *** akerr has joined #openstack-cinder | 01:10 | |
| *** xyang1 has quit IRC | 01:16 | |
| *** annegentle has joined #openstack-cinder | 01:18 | |
| *** annashen has joined #openstack-cinder | 01:20 | |
| *** haigang has joined #openstack-cinder | 01:22 | |
| *** diegows has quit IRC | 01:25 | |
| *** annashen has quit IRC | 01:26 | |
| *** MentalRay_ has quit IRC | 01:26 | |
| *** casusbel_ has joined #openstack-cinder | 01:28 | |
| *** haomaiwang has quit IRC | 01:29 | |
| *** kaisers1 has joined #openstack-cinder | 01:31 | |
| *** casusbelli has quit IRC | 01:31 | |
| *** kaisers has quit IRC | 01:33 | |
| *** Apoorva has quit IRC | 01:36 | |
| *** ebalduf has joined #openstack-cinder | 01:39 | |
| *** julim has quit IRC | 01:39 | |
| *** annegentle has quit IRC | 01:43 | |
| *** ebalduf has quit IRC | 01:43 | |
| *** haigang has quit IRC | 01:48 | |
| *** haigang has joined #openstack-cinder | 01:58 | |
| *** haomaiwang has joined #openstack-cinder | 02:00 | |
| *** patrickeast has quit IRC | 02:09 | |
| *** vilobhmm1 has quit IRC | 02:10 | |
| *** mtanino has quit IRC | 02:10 | |
| *** annegentle has joined #openstack-cinder | 02:19 | |
| *** annegentle has quit IRC | 02:24 | |
| *** akerr is now known as akerr_away | 02:25 | |
| *** harlowja is now known as harlowja_away | 02:29 | |
| *** jdurgin1 has quit IRC | 02:35 | |
| *** _cjones_ has joined #openstack-cinder | 02:36 | |
| *** ebalduf has joined #openstack-cinder | 02:40 | |
| *** ebalduf has quit IRC | 02:44 | |
| *** akerr_away is now known as akerr | 02:47 | |
| *** akerr has quit IRC | 02:47 | |
| *** _cjones_ has quit IRC | 02:52 | |
| *** _cjones_ has joined #openstack-cinder | 02:53 | |
| *** dims_ has quit IRC | 02:55 | |
| *** zongliang has quit IRC | 02:56 | |
| *** annegentle has joined #openstack-cinder | 02:57 | |
| *** MentalRay has joined #openstack-cinder | 03:01 | |
| *** Longgeek_ has joined #openstack-cinder | 03:03 | |
| *** rhefner has quit IRC | 03:04 | |
| *** Longgeek has quit IRC | 03:05 | |
| *** david-lyle has joined #openstack-cinder | 03:13 | |
| *** vilobhmm1 has joined #openstack-cinder | 03:16 | |
| *** annashen has joined #openstack-cinder | 03:23 | |
| *** vilobhmm1 has quit IRC | 03:26 | |
| *** annashen has quit IRC | 03:27 | |
| *** changbl_ has joined #openstack-cinder | 03:36 | |
| *** changbl has quit IRC | 03:38 | |
| *** _cjones_ has quit IRC | 03:47 | |
| *** thingee has quit IRC | 03:51 | |
| *** rushiagr_away is now known as rushiagr | 03:54 | |
| *** avishay has joined #openstack-cinder | 03:57 | |
| *** heyun has joined #openstack-cinder | 03:59 | |
| *** MentalRay has quit IRC | 04:05 | |
| *** MentalRay has joined #openstack-cinder | 04:05 | |
| *** emagana has joined #openstack-cinder | 04:07 | |
| *** lpetrut has joined #openstack-cinder | 04:13 | |
| *** shilpa_ has joined #openstack-cinder | 04:14 | |
| *** rushiagr is now known as rushiagr_away | 04:22 | |
| *** annegentle has quit IRC | 04:26 | |
| *** dflorea has joined #openstack-cinder | 04:36 | |
| *** haigang has quit IRC | 04:38 | |
| *** Mandell has joined #openstack-cinder | 04:41 | |
| *** haigang has joined #openstack-cinder | 04:44 | |
| *** eharney has quit IRC | 04:47 | |
| *** krtaylor has quit IRC | 04:48 | |
| *** eharney has joined #openstack-cinder | 04:49 | |
| *** haigang has quit IRC | 04:50 | |
| *** krtaylor has joined #openstack-cinder | 04:50 | |
| *** deepakcs has joined #openstack-cinder | 04:50 | |
| *** BharatK has joined #openstack-cinder | 04:58 | |
| *** emagana has quit IRC | 05:04 | |
| *** emagana has joined #openstack-cinder | 05:05 | |
| *** emagana has quit IRC | 05:10 | |
| *** dflorea has quit IRC | 05:13 | |
| *** dflorea has joined #openstack-cinder | 05:14 | |
| *** rushiagr_away is now known as rushiagr | 05:14 | |
| *** shilpa_ has quit IRC | 05:15 | |
| *** dflorea has quit IRC | 05:19 | |
| *** rushiagr is now known as rushiagr_away | 05:23 | |
| *** reed has quit IRC | 05:29 | |
| *** lpetrut has quit IRC | 05:29 | |
| *** haigang has joined #openstack-cinder | 05:35 | |
| *** Longgeek_ has quit IRC | 05:38 | |
| *** Longgeek has joined #openstack-cinder | 05:39 | |
| *** tshefi has joined #openstack-cinder | 05:39 | |
| *** vilobhmm1 has joined #openstack-cinder | 05:40 | |
| *** haigang has joined #openstack-cinder | 05:45 | |
| *** Maike has joined #openstack-cinder | 05:48 | |
| *** cpg|away is now known as cpg | 05:49 | |
| *** Maike_ has joined #openstack-cinder | 05:50 | |
| *** Maike has quit IRC | 05:53 | |
| *** spradeepv has joined #openstack-cinder | 06:03 | |
| *** pradipta has joined #openstack-cinder | 06:05 | |
| *** haigang has quit IRC | 06:07 | |
| spradeepv | hemnafk: I am from brocade. Can you provide me the logs for this bug - https://bugs.launchpad.net/cinder/+bug/1438722 | 06:08 |
| openstack | Launchpad bug 1438722 in Cinder "Brocade Zone Manager does not wait for zoning transactions to complete" [Medium,Triaged] - Assigned to Angela Smith (aallen-m) | 06:08 |
| *** haigang has joined #openstack-cinder | 06:09 | |
| *** haigang has quit IRC | 06:10 | |
| *** nlevinki has joined #openstack-cinder | 06:12 | |
| *** haigang has joined #openstack-cinder | 06:13 | |
| *** spradeepv is now known as spradeepv_ | 06:20 | |
| *** afazekas has joined #openstack-cinder | 06:21 | |
| *** spradeepv_ is now known as spradeepv | 06:23 | |
| *** haigang has quit IRC | 06:23 | |
| *** Liu has joined #openstack-cinder | 06:23 | |
| *** annashen has joined #openstack-cinder | 06:25 | |
| *** haigang has joined #openstack-cinder | 06:25 | |
| *** ankit_ag has joined #openstack-cinder | 06:25 | |
| *** annashen has quit IRC | 06:29 | |
| *** abehl has joined #openstack-cinder | 06:29 | |
| *** ronis has joined #openstack-cinder | 06:33 | |
| openstackgerrit | git-harry proposed openstack/python-cinderclient: Fix bypass_url option, required by session object https://review.openstack.org/145493 | 06:38 |
| *** shilpa_ has joined #openstack-cinder | 06:40 | |
| *** ishant has joined #openstack-cinder | 06:41 | |
| openstackgerrit | Tushar Patil proposed openstack/python-cinderclient: Allow cinderclient to handle exception response https://review.openstack.org/166351 | 06:43 |
| *** Raj_ has joined #openstack-cinder | 06:43 | |
| *** MentalRay has quit IRC | 06:44 | |
| *** dulek has joined #openstack-cinder | 06:45 | |
| *** vilobhmm1 has quit IRC | 06:45 | |
| *** TobiasE has joined #openstack-cinder | 06:48 | |
| openstackgerrit | YAMADA Hideki proposed openstack/cinder: Fix missing clone_image API support for sheepdog driver. https://review.openstack.org/159856 | 06:52 |
| openstackgerrit | git-harry proposed openstack/python-cinderclient: Allow manually specified auth token https://review.openstack.org/111051 | 06:56 |
| openstackgerrit | Julien Danjou proposed openstack/cinder: tests: replace mox by mox3, clean out mox usage https://review.openstack.org/147476 | 07:05 |
| openstackgerrit | Julien Danjou proposed openstack/cinder: Leverage timeutils, drop strtime() usage https://review.openstack.org/165798 | 07:08 |
| *** markvoelker has quit IRC | 07:09 | |
| *** jordanP has joined #openstack-cinder | 07:09 | |
| *** geguileo has joined #openstack-cinder | 07:13 | |
| *** jistr has joined #openstack-cinder | 07:14 | |
| *** heyun has quit IRC | 07:19 | |
| *** casusbel_ has quit IRC | 07:25 | |
| *** casusbelli has joined #openstack-cinder | 07:26 | |
| *** rushiagr_away is now known as rushiagr | 07:30 | |
| *** sgotliv has quit IRC | 07:34 | |
| *** sgotliv has joined #openstack-cinder | 07:34 | |
| *** rushiagr is now known as rushiagr_away | 07:37 | |
| *** markus_z has joined #openstack-cinder | 07:44 | |
| *** markus_z has quit IRC | 07:47 | |
| *** abhijeetm has joined #openstack-cinder | 07:48 | |
| *** dims_ has joined #openstack-cinder | 07:50 | |
| *** dims_ has quit IRC | 07:55 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Set volume_attachment to [] for the temporary volume creation https://review.openstack.org/167134 | 07:56 |
| openstackgerrit | Rick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver https://review.openstack.org/169653 | 07:58 |
| openstackgerrit | Rick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver https://review.openstack.org/169653 | 08:00 |
| openstackgerrit | Rick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver https://review.openstack.org/169653 | 08:03 |
| *** ndipanov has joined #openstack-cinder | 08:07 | |
| *** Longgeek has quit IRC | 08:13 | |
| *** Longgeek has joined #openstack-cinder | 08:13 | |
| *** e0ne has joined #openstack-cinder | 08:16 | |
| *** lpetrut has joined #openstack-cinder | 08:18 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 08:23 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Delete the temporary volume if migration fails https://review.openstack.org/160682 | 08:26 |
| openstackgerrit | Rick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver https://review.openstack.org/169653 | 08:26 |
| *** e0ne is now known as e0ne_ | 08:29 | |
| *** kashyap has joined #openstack-cinder | 08:30 | |
| *** alecv_ has joined #openstack-cinder | 08:40 | |
| *** ho has quit IRC | 08:43 | |
| *** e0ne_ is now known as e0ne | 08:43 | |
| *** yuriy_n17 has joined #openstack-cinder | 08:55 | |
| *** deepakcs has quit IRC | 08:58 | |
| *** Longgeek has quit IRC | 08:59 | |
| *** Miouge has quit IRC | 09:00 | |
| *** rushiagr_away is now known as rushiagr | 09:01 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver https://review.openstack.org/169653 | 09:08 |
| anish | heh, gerrit has clippy today | 09:09 |
| *** kashyap has left #openstack-cinder | 09:13 | |
| *** markus_z has joined #openstack-cinder | 09:15 | |
| *** Longgeek has joined #openstack-cinder | 09:27 | |
| *** annashen has joined #openstack-cinder | 09:27 | |
| *** ociuhandu has quit IRC | 09:28 | |
| *** annashen has quit IRC | 09:33 | |
| *** lpetrut has quit IRC | 09:39 | |
| *** Miouge has joined #openstack-cinder | 09:41 | |
| *** Raj_ has quit IRC | 09:51 | |
| *** haomaiwang has quit IRC | 09:51 | |
| *** heyun has joined #openstack-cinder | 09:54 | |
| *** dims_ has joined #openstack-cinder | 10:00 | |
| *** dims__ has joined #openstack-cinder | 10:01 | |
| *** alexpilotti has joined #openstack-cinder | 10:03 | |
| *** dims_ has quit IRC | 10:04 | |
| *** Miouge has quit IRC | 10:07 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Add patch for consistency group update in ProphetStor driver https://review.openstack.org/169653 | 10:10 |
| dulek | DuncanT: hi! | 10:14 |
| DuncanT | Hi | 10:14 |
| openstackgerrit | Jeegn Chen proposed openstack/cinder: More error handling on VNX migration failure. https://review.openstack.org/159704 | 10:15 |
| dulek | DuncanT: Working on bug 1438744 I've run into a problem. | 10:15 |
| openstack | bug 1438744 in Cinder "AttributeError from _update_allocated_capacity upon create failure" [Undecided,Confirmed] https://launchpad.net/bugs/1438744 - Assigned to Michal Dulko (michal-dulko-f) | 10:15 |
| DuncanT | Ok... | 10:15 |
| dulek | DuncanT: Bug is there because in VolumeManager I'm catching CinderException | 10:15 |
| dulek | DuncanT: And comes out the from the create_volume flow also ProcessUtilException can be raised. | 10:16 |
| dulek | DuncanT: This is raised from the driver directly. | 10:16 |
| dulek | DuncanT: I wonder if I should repack the exception in create_volume flow or just catch also ProcessUtilException in the VolumeManager. | 10:17 |
| dulek | DuncanT: I also wonder if there aren't other exception types raised from the drivers. | 10:17 |
| dulek | DuncanT: Any opinion? | 10:17 |
| DuncanT | I suspect there are drivers that raise all sorts of things in corner cases | 10:18 |
| *** Miouge has joined #openstack-cinder | 10:19 | |
| *** rushiagr is now known as rushiagr_away | 10:19 | |
| dulek | DuncanT: So the question is - add try-except to places where create_volume flow is calling the driver and reraise VolumeDriverException | 10:20 |
| dulek | DuncanT: Or just catch Exception in VolumeManager | 10:20 |
| *** lpetrut has joined #openstack-cinder | 10:21 | |
| DuncanT | Sorry, I'm working through code as fast as I can here, catching up. Can you point me at the catch block you mean in VolumeManager? | 10:21 |
| dulek | DuncanT: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L470 | 10:21 |
| DuncanT | Thanks | 10:22 |
| *** rushiagr_away is now known as rushiagr | 10:22 | |
| *** Muntaner has joined #openstack-cinder | 10:22 | |
| Muntaner | hi guys | 10:22 |
| Muntaner | I'm trying to add a NAS (buffalo) to my OpenStack all-in-one installation | 10:22 |
| Muntaner | I'd like to use it with iscsi | 10:22 |
| *** jistr has quit IRC | 10:23 | |
| Muntaner | it is in a LAN that I use for services, 10.10.10.0/24, and can ping it via the server @10.10.10.52 | 10:23 |
| Muntaner | any advices? | 10:23 |
| dulek | DuncanT: And here's where exception is raised. | 10:23 |
| dulek | DuncanT: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L637 | 10:23 |
| DuncanT | Yeah, I think catching Exception there is fine | 10:23 |
| *** haigang has quit IRC | 10:24 | |
| dulek | DuncanT: Okay, thank you for your opinion. :) | 10:25 |
| *** aarefiev has quit IRC | 10:25 | |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: Using 'fallocate' instead of 'dd' to create raw volumes https://review.openstack.org/164527 | 10:25 |
| DuncanT | Drivers might raise all sorts of exceptions, and all you're doing is cleaning up then reraising | 10:25 |
| DuncanT | Now I've got the code in front of me, I'm intrigued by https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L475-L479 - we're going back to the db just to get the volume id, which we already have and we know is immutable | 10:25 |
| DuncanT | Oh, wait, I see it, ignore me | 10:26 |
| DuncanT | Sorry | 10:26 |
| DuncanT | Being slow | 10:26 |
| DuncanT | Though really we only need to do that in the 'if not rescheduled:' case, right? | 10:27 |
| dulek | Hm... | 10:27 |
| DuncanT | (It's a minor performance optimisation, but something we need to start looking at to improve out DB scaling. Don't feel you need to care now) | 10:28 |
| dulek | DuncanT: Yes, you're right I think. | 10:28 |
| dulek | I really hope this code is only nasty workaround we'll be able to get rid of in L. | 10:29 |
| DuncanT | :-) We've said that about lots of code | 10:29 |
| dulek | Aww, I'm naive. ;) | 10:30 |
| dulek | I'll add that optimization to my TODO list. Thanks for pointing it out. | 10:30 |
| openstackgerrit | Tina Tang proposed openstack/cinder: Create consistgroup from cgsnapshot support in VNX driver https://review.openstack.org/163706 | 10:35 |
| *** aarefiev has joined #openstack-cinder | 10:36 | |
| *** jistr has joined #openstack-cinder | 10:36 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Attach/detach batch processing in VNX driver https://review.openstack.org/154350 | 10:36 |
| Muntaner | nobody? I'm pretty lost right now | 10:40 |
| *** Raj_ has joined #openstack-cinder | 10:41 | |
| *** IanGovett has joined #openstack-cinder | 10:41 | |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Remove duplicate find request in find_resource https://review.openstack.org/162130 | 10:41 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add cache to Managers' find/findall methods https://review.openstack.org/162139 | 10:41 |
| *** ociuhandu has joined #openstack-cinder | 10:44 | |
| *** ishant has quit IRC | 10:44 | |
| *** nellysmitt has joined #openstack-cinder | 10:50 | |
| *** e0ne is now known as e0ne_ | 10:53 | |
| *** haigang has joined #openstack-cinder | 10:54 | |
| *** iwao has quit IRC | 10:56 | |
| *** e0ne_ is now known as e0ne | 10:57 | |
| *** heyun has quit IRC | 10:58 | |
| *** haigang has quit IRC | 11:10 | |
| *** jistr is now known as jistr|demo | 11:11 | |
| *** Muntaner has quit IRC | 11:22 | |
| *** rushiagr is now known as rushiagr_away | 11:28 | |
| *** rushiagr_away is now known as rushiagr | 11:28 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Catch more general exception in manager's create_volume https://review.openstack.org/169707 | 11:29 |
| *** annashen has joined #openstack-cinder | 11:29 | |
| *** annashen has quit IRC | 11:34 | |
| openstackgerrit | Bharat Kumar Kobagana proposed openstack/cinder: Using 'fallocate' instead of 'dd' to create raw volumes https://review.openstack.org/164527 | 11:36 |
| *** cpg is now known as cpg|away | 11:37 | |
| *** dulek has quit IRC | 11:40 | |
| *** pradipta has quit IRC | 11:45 | |
| *** lpetrut has quit IRC | 11:53 | |
| *** shilpa_ has quit IRC | 11:54 | |
| abhijeetm | Hi all, can someome please review patch : https://review.openstack.org/#/c/144409/ | 11:55 |
| abhijeetm | I already got one +2 | 11:55 |
| *** BharatK has quit IRC | 11:58 | |
| *** Lee1092 has quit IRC | 12:00 | |
| *** e0ne is now known as e0ne_ | 12:01 | |
| openstackgerrit | ZhiQiang Fan proposed openstack/cinder: Switch to oslo.policy 0.3.0 https://review.openstack.org/165688 | 12:01 |
| openstackgerrit | ZhiQiang Fan proposed openstack/cinder: Switch to oslo.policy 0.3.0 https://review.openstack.org/165688 | 12:02 |
| *** bswartz has quit IRC | 12:09 | |
| *** BharatK has joined #openstack-cinder | 12:13 | |
| *** MentalRay has joined #openstack-cinder | 12:14 | |
| *** markvoelker has joined #openstack-cinder | 12:14 | |
| *** akerr has joined #openstack-cinder | 12:18 | |
| *** jungleboyj has joined #openstack-cinder | 12:29 | |
| *** Miouge has quit IRC | 12:34 | |
| *** dulek has joined #openstack-cinder | 12:39 | |
| *** MentalRay has quit IRC | 12:41 | |
| *** Miouge has joined #openstack-cinder | 12:42 | |
| *** e0ne_ is now known as e0ne | 12:43 | |
| *** diegows has joined #openstack-cinder | 12:46 | |
| *** timcl has joined #openstack-cinder | 12:48 | |
| *** BharatK has quit IRC | 12:48 | |
| *** diegows has quit IRC | 12:51 | |
| *** Yogi1 has joined #openstack-cinder | 12:53 | |
| *** ebalduf has joined #openstack-cinder | 12:55 | |
| *** lpabon has joined #openstack-cinder | 12:59 | |
| *** ebalduf has quit IRC | 13:00 | |
| *** timcl1 has joined #openstack-cinder | 13:02 | |
| *** BharatK has joined #openstack-cinder | 13:02 | |
| *** timcl has quit IRC | 13:05 | |
| *** annegentle has joined #openstack-cinder | 13:05 | |
| avishay | abhijeetm: checking | 13:07 |
| *** julim has joined #openstack-cinder | 13:07 | |
| abhijeetm | avishay: ok, thanks | 13:08 |
| *** mwichmann has joined #openstack-cinder | 13:09 | |
| *** rhefner has joined #openstack-cinder | 13:16 | |
| *** Raj_ has quit IRC | 13:23 | |
| *** mriedem_away is now known as mriedem | 13:24 | |
| avishay | abhijeetm: the review site just went down for me :( | 13:26 |
| avishay | abhijeetm: looks OK, except the comment that you added in manager.py is very confusing IMO | 13:26 |
| avishay | abhijeetm: i would prefer it were removed | 13:26 |
| avishay | abhijeetm: oh, the site is back... | 13:27 |
| -openstackstatus- NOTICE: gerrit has been restarted to restore event streaming. any change events missed by zuul (between 12:48 and 13:28 utc) will need to be rechecked or have new approval votes set | 13:28 | |
| *** haigang has joined #openstack-cinder | 13:31 | |
| abhijeetm | avishay: While upload is in progress we can delete instance or detach volume from instance | 13:35 |
| *** alexpilotti has quit IRC | 13:35 | |
| abhijeetm | avishay: and in that case volume will remain in 'in-use' state | 13:35 |
| avishay | abhijeetm: i understand, but the code under the comment updates the state atomically, which is the correct thing to do in any case - it is not something specific to that issue | 13:36 |
| avishay | abhijeetm: if someone who isn't familiar with this bug looks at the code and they see the comment, they won't understand | 13:37 |
| *** timcl1 has quit IRC | 13:37 | |
| abhijeetm | avishay: ok, I will remove comment and upload new patch set | 13:37 |
| avishay | abhijeetm: thanks! | 13:38 |
| *** Yogi1 has quit IRC | 13:38 | |
| *** Liu has quit IRC | 13:42 | |
| *** Liu has joined #openstack-cinder | 13:42 | |
| *** lpetrut has joined #openstack-cinder | 13:42 | |
| *** nlevinki has quit IRC | 13:44 | |
| *** diegows has joined #openstack-cinder | 13:46 | |
| *** abehl has quit IRC | 13:46 | |
| openstackgerrit | Abhijeet Malawade proposed openstack/cinder: Get volume from db again before updating it's status https://review.openstack.org/144409 | 13:46 |
| *** anshul has joined #openstack-cinder | 13:46 | |
| scottda | Is it just me, or do the review.openstack.org pages have an annoying Windows Office paperclip animation on them? | 13:51 |
| *** anshul has quit IRC | 13:51 | |
| *** haigang has quit IRC | 13:52 | |
| *** timcl has joined #openstack-cinder | 13:53 | |
| avishay | scottda: i assume it's for april fools | 13:56 |
| jungleboyj | scottda: I just noticed that too. I am on a Linux box. Was very confused for a minute. | 13:57 |
| jungleboyj | Man, he shows up on the review pages too. | 13:59 |
| *** bswartz has joined #openstack-cinder | 13:59 | |
| *** dustins has joined #openstack-cinder | 14:00 | |
| *** crose has joined #openstack-cinder | 14:00 | |
| *** MentalRay has joined #openstack-cinder | 14:00 | |
| *** Yogi11 has joined #openstack-cinder | 14:00 | |
| *** anshul has joined #openstack-cinder | 14:02 | |
| *** Lee1092 has joined #openstack-cinder | 14:04 | |
| scottda | I'm on linux as well... | 14:05 |
| avishay | april fools..... | 14:05 |
| scottda | avishay: I think you are right! Actually, that's quite funny! | 14:05 |
| *** anshul has quit IRC | 14:07 | |
| *** Maike_ has quit IRC | 14:08 | |
| *** annegentle has quit IRC | 14:09 | |
| *** tshefi has quit IRC | 14:11 | |
| *** ebalduf has joined #openstack-cinder | 14:11 | |
| *** rushiagr is now known as rushiagr_away | 14:14 | |
| *** shilpa_ has joined #openstack-cinder | 14:15 | |
| *** ebalduf has quit IRC | 14:16 | |
| *** thangp has joined #openstack-cinder | 14:16 | |
| *** e0ne is now known as e0ne_ | 14:17 | |
| *** mtanino has joined #openstack-cinder | 14:18 | |
| *** Muntaner has joined #openstack-cinder | 14:22 | |
| Muntaner | hey guys | 14:22 |
| Muntaner | I know I'm repetitive | 14:22 |
| Muntaner | I need some help in connecting my NAS to my Openstack cinder module :) | 14:23 |
| *** e0ne_ is now known as e0ne | 14:23 | |
| thangp | did someone put clippy on gerrit reviews as an april fool's joke? | 14:23 |
| *** pradipta has joined #openstack-cinder | 14:24 | |
| *** Yogi11 has quit IRC | 14:26 | |
| jbernard | yep | 14:26 |
| scottda | thangp: no, we are all required to use Windows for OpenStack dev now. | 14:27 |
| jbernard | sadly, it doesn't help me navigate the UI any better | 14:27 |
| thangp | haha | 14:27 |
| thangp | but clippy comes from aws :) | 14:27 |
| *** bkopilov has quit IRC | 14:27 | |
| thangp | at least that's what the html says | 14:28 |
| scottda | hehe | 14:28 |
| thangp | https://s3.amazonaws.com/clippy.js | 14:28 |
| *** bkopilov has joined #openstack-cinder | 14:28 | |
| thangp | i like the pacman on google maps though | 14:28 |
| *** annegentle has joined #openstack-cinder | 14:36 | |
| *** alexpilotti has joined #openstack-cinder | 14:38 | |
| avishay | Muntaner: do you have a specific problem? which driver? | 14:40 |
| *** diegows has quit IRC | 14:43 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: Revert "Removing Netapp FC drivers for no reported CI" https://review.openstack.org/169781 | 14:44 |
| *** patrickeast has joined #openstack-cinder | 14:45 | |
| *** Yogi1 has joined #openstack-cinder | 14:52 | |
| *** Mandell has quit IRC | 14:52 | |
| *** rmesta has joined #openstack-cinder | 14:53 | |
| *** nlevinki has joined #openstack-cinder | 14:54 | |
| *** tsekiyama has joined #openstack-cinder | 14:54 | |
| Muntaner | avishay, I simply dunno how to start the configuration | 14:56 |
| Muntaner | actually I have this NAS, can ping it, etc, but dunno what to do exactly | 14:56 |
| avishay | Muntaner: you need to specify the appropriate configuration options in cinder.conf , each backend has its own | 14:56 |
| avishay | Muntaner: which driver is it? most have documentation | 14:57 |
| Muntaner | avishay, I'd like to use iscsi | 14:57 |
| Muntaner | and I think this nas is set up with it (it's the first time see it) | 14:57 |
| Muntaner | the problem is I'm not expert in this | 14:57 |
| avishay | Muntaner: which vendor, model? | 14:57 |
| Muntaner | it is a buffalo terastation | 14:57 |
| *** emagana has joined #openstack-cinder | 14:58 | |
| Muntaner | avishay, do you need the exact model? | 14:58 |
| Muntaner | TS3400D | 14:58 |
| *** Bsony has joined #openstack-cinder | 15:00 | |
| *** guitarzan has quit IRC | 15:00 | |
| avishay | Muntaner: is that based on Windows Server 2012? | 15:00 |
| *** guitarzan has joined #openstack-cinder | 15:00 | |
| Muntaner | avishay, dunno :( | 15:01 |
| Muntaner | but it has ssh, so I think there's some linux inside | 15:01 |
| *** AlkaD has joined #openstack-cinder | 15:02 | |
| *** asselin has joined #openstack-cinder | 15:02 | |
| *** kaisers has joined #openstack-cinder | 15:02 | |
| *** annegent_ has joined #openstack-cinder | 15:02 | |
| avishay | Muntaner: I'm not familiar with it...there is a driver for Windows Server 2012 based boxes, and many others, but not for buffalo specifically as far as i know | 15:02 |
| avishay | Muntaner: i would suggest contacting the vendor | 15:02 |
| *** nellysmi_ has joined #openstack-cinder | 15:03 | |
| *** nellysmitt has quit IRC | 15:04 | |
| *** annegentle has quit IRC | 15:04 | |
| *** nellysmitt has joined #openstack-cinder | 15:04 | |
| *** nellysmitt has quit IRC | 15:05 | |
| Muntaner | avishay, the problem is: to say what? | 15:06 |
| Muntaner | how do I tell cinder to use this NAS? these are my problems | 15:06 |
| *** kaisers has quit IRC | 15:06 | |
| avishay | Muntaner: Ask if the storage is supported by Cinder, and if so, do they have any documentation on configuring Cinder to work with it | 15:06 |
| avishay | Muntaner: If they don't know, ask if it is based on Windows Server 2012, because that has a driver | 15:07 |
| Muntaner | avishay, with NFS or iscsi shouldn't I be able to use it without knowing the vendor or model? | 15:07 |
| jgriffith | No meeting today? | 15:07 |
| Muntaner | if it runs as a NFS server, for example? | 15:08 |
| Muntaner | just asking, I'm not expert | 15:08 |
| *** nellysmi_ has quit IRC | 15:08 | |
| *** kaisers has joined #openstack-cinder | 15:08 | |
| rhe00_ | jgriffith: 10am mountain time. :) | 15:08 |
| jgriffith | hehe :) | 15:08 |
| jgriffith | DST strikes | 15:08 |
| *** ankit_ag has quit IRC | 15:08 | |
| rhe00_ | yes, and my outlook is not handling that well. | 15:08 |
| avishay | jgriffith: yea i checked google for "time now in UTC" :) | 15:09 |
| Muntaner | avishay, I look noobish, I know :) | 15:10 |
| *** rushiagr_away is now known as rushiagr | 15:11 | |
| *** kaisers has quit IRC | 15:11 | |
| *** rhe00_ is now known as rhe00 | 15:11 | |
| *** Yogi1 has quit IRC | 15:15 | |
| *** Muntaner has quit IRC | 15:16 | |
| mtreinish | jgriffith: btw, not sure you saw this: https://github.com/j-griffith/sos-ci/pull/10 but I'm going for #4 :) | 15:17 |
| *** sks has joined #openstack-cinder | 15:17 | |
| *** abhijeetm has quit IRC | 15:17 | |
| mtreinish | probably worth testing locally before you merge though... | 15:17 |
| *** kaisers has joined #openstack-cinder | 15:17 | |
| *** shyama has quit IRC | 15:20 | |
| *** kaisers has quit IRC | 15:22 | |
| jgriffith | mtreinish: hehe... no I didn't see it. Looking... | 15:24 |
| jgriffith | mtreinish: oh yay!! | 15:24 |
| *** thingee has joined #openstack-cinder | 15:24 | |
| jgriffith | mtreinish: I'll load it up in my test env then accept the PR | 15:24 |
| jgriffith | mtreinish: the shell.CONF.set_override is a nice touch as well BTW | 15:25 |
| mtreinish | jgriffith: heh, I really need to kill the oslo.db usage in there and drop using oslo.config | 15:26 |
| jgriffith | mtreinish: and this leads in to the next thing I wanted to add... | 15:26 |
| jgriffith | mtreinish: associating some meta with the run entry | 15:26 |
| jgriffith | mtreinish: keep track of patch-id, CI-name etc | 15:27 |
| mtreinish | oh, yeah I had no idea if the metadata storage I added there was useful/correct or not. I took a guess | 15:27 |
| *** bkopilov has quit IRC | 15:27 | |
| jgriffith | mtreinish: yeah, :) | 15:27 |
| jgriffith | mtreinish: although now you stoll my commit :) | 15:27 |
| jgriffith | mtreinish: if I'm not careful you'll be the top contributor LOL | 15:27 |
| *** hemnafk is now known as hemna | 15:28 | |
| *** TobiasE has quit IRC | 15:28 | |
| jgriffith | mtreinish: something else I'm considering depending on uptake... | 15:28 |
| jgriffith | mtreinish: just publishing on PyPi | 15:28 |
| jgriffith | mtreinish: it's already relatively "simple" to get up and running, but that would sort of make it a no brainer | 15:29 |
| mtreinish | so you can pip install sos-ci? Yeah, that would probably be useful for people | 15:29 |
| mtreinish | heh, although I'm probably the wrong person to ask because I've never run it :) | 15:29 |
| jgriffith | LOL | 15:29 |
| jgriffith | beauty of open source collaboration | 15:30 |
| *** jdurgin1 has joined #openstack-cinder | 15:31 | |
| *** kaisers has joined #openstack-cinder | 15:31 | |
| *** annashen has joined #openstack-cinder | 15:33 | |
| *** bswartz has quit IRC | 15:34 | |
| *** kaisers has quit IRC | 15:35 | |
| *** reed has joined #openstack-cinder | 15:35 | |
| *** hemna has quit IRC | 15:36 | |
| *** e0ne is now known as e0ne_ | 15:37 | |
| *** annashen has quit IRC | 15:37 | |
| *** e0ne_ is now known as e0ne | 15:38 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Only use operational LIFs for iscsi target details https://review.openstack.org/169812 | 15:38 |
| *** kaisers has joined #openstack-cinder | 15:40 | |
| *** hemna has joined #openstack-cinder | 15:42 | |
| *** lpabon has quit IRC | 15:44 | |
| *** hemna has quit IRC | 15:45 | |
| *** rushil has joined #openstack-cinder | 15:45 | |
| *** bkopilov has joined #openstack-cinder | 15:46 | |
| *** bkopilov has quit IRC | 15:46 | |
| *** bswartz has joined #openstack-cinder | 15:47 | |
| *** _cjones_ has joined #openstack-cinder | 15:47 | |
| *** kaisers has quit IRC | 15:48 | |
| *** xyang1 has joined #openstack-cinder | 15:49 | |
| *** hemna has joined #openstack-cinder | 15:49 | |
| *** shilpa_ has quit IRC | 15:54 | |
| *** dannywilson has joined #openstack-cinder | 15:56 | |
| *** ebalduf has joined #openstack-cinder | 15:57 | |
| *** deepakcs has joined #openstack-cinder | 15:57 | |
| *** rmesta has quit IRC | 15:58 | |
| *** rmesta has joined #openstack-cinder | 15:58 | |
| *** rmesta has quit IRC | 15:59 | |
| *** rmesta has joined #openstack-cinder | 15:59 | |
| e0ne | looks like we'll get a short meeting today | 15:59 |
| *** Yogi1 has joined #openstack-cinder | 16:00 | |
| jgriffith | e0ne: don't jynx it :) | 16:01 |
| *** sks has quit IRC | 16:01 | |
| e0ne | :) | 16:01 |
| *** ebalduf has quit IRC | 16:02 | |
| jgriffith | e0ne: so short we won't even have one :) | 16:02 |
| *** HoloIRCUser5 has joined #openstack-cinder | 16:04 | |
| *** timcl1 has joined #openstack-cinder | 16:05 | |
| *** MentalRay has quit IRC | 16:06 | |
| *** timcl has quit IRC | 16:07 | |
| *** Yogi1 has quit IRC | 16:07 | |
| *** vbala has joined #openstack-cinder | 16:08 | |
| *** Yogi1 has joined #openstack-cinder | 16:08 | |
| *** rongze has joined #openstack-cinder | 16:08 | |
| *** vbala has quit IRC | 16:08 | |
| *** jdurgin1 has quit IRC | 16:09 | |
| *** patrickeast has quit IRC | 16:09 | |
| *** avishay has quit IRC | 16:09 | |
| *** alecv has joined #openstack-cinder | 16:10 | |
| *** gary-smith_ has quit IRC | 16:10 | |
| *** gary-smith_ has joined #openstack-cinder | 16:11 | |
| *** dustins has quit IRC | 16:14 | |
| *** HoloIRCUser5 has quit IRC | 16:14 | |
| *** rushil has quit IRC | 16:15 | |
| *** alexpilotti has quit IRC | 16:16 | |
| *** enterprisedc has joined #openstack-cinder | 16:18 | |
| *** dulek has quit IRC | 16:19 | |
| jgriffith | mtreinish: boooo http://paste.openstack.org/show/197806/ | 16:19 |
| * e0ne bb in ~2hours | 16:19 | |
| *** annashen has joined #openstack-cinder | 16:20 | |
| *** dustins has joined #openstack-cinder | 16:20 | |
| *** e0ne has quit IRC | 16:20 | |
| mtreinish | jgriffith: heh, oops | 16:21 |
| jgriffith | :) | 16:22 |
| *** annegent_ has quit IRC | 16:22 | |
| *** deepakcs has quit IRC | 16:23 | |
| *** MentalRay has joined #openstack-cinder | 16:23 | |
| *** kaisers has joined #openstack-cinder | 16:23 | |
| *** asselin has quit IRC | 16:24 | |
| *** rhagarty has joined #openstack-cinder | 16:24 | |
| *** bswartz has quit IRC | 16:24 | |
| *** rhagarty_ has joined #openstack-cinder | 16:25 | |
| *** AlkaD has quit IRC | 16:25 | |
| jgriffith | boris-42: WTF.. haha, I just saw Clippy on my dashboard | 16:26 |
| *** kaisers has quit IRC | 16:27 | |
| winston-d | jgriffith: yeah, saw that too | 16:27 |
| hemna | lol | 16:27 |
| hemna | man, I saw that earlier in a review, and thought...crap...what extension in chrome did I screw up now..... | 16:28 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix always false condition in glance wrapper https://review.openstack.org/169831 | 16:28 |
| jgriffith | haha | 16:28 |
| jgriffith | boris-42: asked me about it yesterday and I told him I thought he was drunk :) | 16:28 |
| thingee | hemna: LOL | 16:28 |
| hemna | I was getting pissed. | 16:29 |
| winston-d | jgriffith: M$ revealed the secret behind Azure and Office 365, they are all on OpenStack | 16:29 |
| hemna | friggin clippy thing...wtf did I do? | 16:29 |
| hemna | oh...shit..wait.....*sigh* | 16:29 |
| jgriffith | winston-d: LMAO | 16:30 |
| jgriffith | winston-d: That's funny!!!! | 16:30 |
| boris-42 | jgriffith: LOL | 16:30 |
| kmartin | winston-d, awesome | 16:30 |
| *** leeantho has joined #openstack-cinder | 16:30 | |
| thingee | have you all looked at google maps today? | 16:30 |
| hemna | uh oh...no | 16:31 |
| winston-d | hemna: did you try left or right click the clippy? I tried many times... | 16:31 |
| hemna | I tried right clicking on it, dragging it.. | 16:31 |
| hemna | wth, this damn thing...... | 16:31 |
| thingee | I wish clippy would start nitpicking at code | 16:31 |
| thingee | looks like that comment is missing a space and period | 16:31 |
| hemna | lol yah | 16:31 |
| thingee | looks like you could use a mock decorator instead of as a nested context managers | 16:32 |
| *** jogo has left #openstack-cinder | 16:32 | |
| jgriffith | thingee: that would be AWESOME! | 16:33 |
| winston-d | a gift from M$ office team, 108 yrs of professional nitpicking experience | 16:34 |
| jgriffith | LOL | 16:34 |
| *** mdbooth has quit IRC | 16:35 | |
| *** jaypipes has quit IRC | 16:37 | |
| *** esker has joined #openstack-cinder | 16:37 | |
| *** shilpa_ has joined #openstack-cinder | 16:38 | |
| *** markus_z has quit IRC | 16:40 | |
| *** mdbooth has joined #openstack-cinder | 16:42 | |
| *** kaisers has joined #openstack-cinder | 16:42 | |
| mtreinish | jgriffith: ok, I updated the pr | 16:44 |
| *** kaisers has quit IRC | 16:45 | |
| *** jistr|demo has quit IRC | 16:47 | |
| *** winston-d is now known as winston-d_zZZ | 16:48 | |
| jgriffith | mtreinish: there's an extra blank space in there | 16:49 |
| jgriffith | mtreinish: "cfg.DataBase. database_connection_string" | 16:49 |
| *** yuriy_n17 has quit IRC | 16:50 | |
| *** jordanP has quit IRC | 16:51 | |
| *** timcl1 has quit IRC | 16:51 | |
| *** dims__ has quit IRC | 16:53 | |
| *** dims_ has joined #openstack-cinder | 16:53 | |
| *** Mandell has joined #openstack-cinder | 16:54 | |
| *** crose has quit IRC | 16:55 | |
| *** Mandell_ has joined #openstack-cinder | 16:57 | |
| *** rongze has quit IRC | 16:57 | |
| *** Yogi11 has joined #openstack-cinder | 16:58 | |
| *** lpetrut has quit IRC | 16:59 | |
| *** timcl has joined #openstack-cinder | 16:59 | |
| *** alexpilotti has joined #openstack-cinder | 17:00 | |
| *** Mandell has quit IRC | 17:00 | |
| *** Yogi1 has quit IRC | 17:01 | |
| *** kaisers has joined #openstack-cinder | 17:01 | |
| *** dustins has quit IRC | 17:02 | |
| *** harlowja_away is now known as harlowja | 17:02 | |
| *** dustins_ has joined #openstack-cinder | 17:02 | |
| *** emagana has quit IRC | 17:04 | |
| *** kaisers has quit IRC | 17:04 | |
| *** _cjones_ has quit IRC | 17:06 | |
| *** e0ne has joined #openstack-cinder | 17:07 | |
| *** dannywilson has quit IRC | 17:08 | |
| *** alecv_ has quit IRC | 17:11 | |
| *** annegentle has joined #openstack-cinder | 17:12 | |
| *** Apoorva has joined #openstack-cinder | 17:12 | |
| *** _cjones_ has joined #openstack-cinder | 17:13 | |
| *** patrickeast has joined #openstack-cinder | 17:14 | |
| *** Mandell_ has quit IRC | 17:18 | |
| mtreinish | jgriffith: ugh, seriously | 17:20 |
| mtreinish | one sec I'll respin it right now | 17:20 |
| *** e0ne is now known as e0ne_ | 17:21 | |
| mtreinish | jgriffith: ok pushed it again, now I hope it's typo free | 17:22 |
| *** mwichmann has quit IRC | 17:22 | |
| *** timcl has quit IRC | 17:22 | |
| *** pradipta has quit IRC | 17:24 | |
| *** sks has joined #openstack-cinder | 17:26 | |
| *** shilpa_ has quit IRC | 17:26 | |
| *** e0ne_ has quit IRC | 17:27 | |
| *** timcl has joined #openstack-cinder | 17:27 | |
| jgriffith | mtreinish: it pains me to EVER note a typo | 17:27 |
| mtreinish | jgriffith: heh, except when I make them things no work :) | 17:28 |
| jgriffith | :) | 17:28 |
| jgriffith | mtreinish: I hosed ya.. can you rebase my bug fix on top or that? | 17:29 |
| jgriffith | mtreinish: never midn | 17:29 |
| jgriffith | mind | 17:29 |
| jgriffith | mtreinish: github figured it out | 17:29 |
| *** thingee has quit IRC | 17:29 | |
| *** Mandell has joined #openstack-cinder | 17:30 | |
| jgriffith | mtreinish: thanks!!! | 17:30 |
| kmartin | https://redhat.bluejeans.com/7819293200 or dial-in US: +1 800-451-8679 or +1 212 729 5016, IL: 809462557, IN: 02030291605, UK: 08445790676, CZ: 800-451-8679, AU: 0730870178, Conf id: 781 929 3200# | 17:30 |
| *** BharatK has quit IRC | 17:30 | |
| hemna | lol | 17:31 |
| kmartin | my bad, wrong window | 17:31 |
| mtreinish | jgriffith: awesome, I guess we're just used to jgit screwing everything up in gerrit :) | 17:31 |
| *** emagana has joined #openstack-cinder | 17:31 | |
| jgriffith | kmartin: I'm sooo tempted to eavesdrop now :) | 17:33 |
| *** e0ne has joined #openstack-cinder | 17:34 | |
| *** mwichmann has joined #openstack-cinder | 17:36 | |
| *** annegentle has quit IRC | 17:36 | |
| *** diegows has joined #openstack-cinder | 17:36 | |
| *** thingee has joined #openstack-cinder | 17:42 | |
| *** vilobhmm1 has joined #openstack-cinder | 17:43 | |
| *** vilobhmm1 has quit IRC | 17:44 | |
| *** vilobhmm1 has joined #openstack-cinder | 17:44 | |
| *** vilobhmm1 has quit IRC | 17:45 | |
| *** vilobhmm1 has joined #openstack-cinder | 17:45 | |
| sks | I am getting the error: [demo] cinder list | 17:46 |
| sks | ERROR: Unable to establish connection to http://x.y.z.t:9856/v1/461cd6404b54428099d916291c7bf467/volumes/detail | 17:46 |
| sks | [stack@dhcp mytools]$ [demo] exit | 17:46 |
| *** vilobhmm11 has joined #openstack-cinder | 17:46 | |
| sks | Does anybody know the issue? | 17:46 |
| *** Yogi11 has quit IRC | 17:46 | |
| *** vilobhmm11 has quit IRC | 17:47 | |
| sks | I could able to do nova list successfully | 17:47 |
| *** vilobhmm11 has joined #openstack-cinder | 17:47 | |
| DuncanT | sks: cinder-api service not running? Firewall issues? | 17:47 |
| sks | DuncanT, c-api was running | 17:48 |
| *** ndipanov has quit IRC | 17:49 | |
| DuncanT | sks: Firewall? Wrong port in the catalogue? | 17:49 |
| sks | DuncanT, If it is firewall issue then the nova list also should have been failed | 17:49 |
| sks | DuncanT, ok | 17:49 |
| DuncanT | sks: Nova and cinder are on different ports. Is this a devstack install? If so, unlikely to be firewall | 17:50 |
| *** vilobhmm1 has quit IRC | 17:50 | |
| sks | DuncanT, yes it is devstack install | 17:50 |
| DuncanT | sks: Do 'nova --debug list' and see if the connection string looks similar | 17:50 |
| sks | DuncanT, sure | 17:50 |
| dansmith | can someone answer a question about volume creation where an image is involved (i.e. a BFV request) | 17:51 |
| dansmith | ? | 17:51 |
| *** e0ne is now known as e0ne_ | 17:52 | |
| DuncanT | dansmith: Just ask, don't ask to ask. We'll answer if we know | 17:52 |
| *** afazekas has quit IRC | 17:53 | |
| dansmith | DuncanT: was finishing the -nova side of this, was planning to just ask :) | 17:53 |
| DuncanT | :-) | 17:53 |
| dansmith | when we create a volume and have the image laid down, what is the expected behavior with a uec image? does the rootfs get laid down and that's it? or is the kernel/ramdisk supposed to be injected and a bootloader put on somehow? | 17:54 |
| *** dustins has joined #openstack-cinder | 17:55 | |
| DuncanT | dansmith: I'm not certain, but I think only the rootfs gets laid down. Nomally I avoid uec images like the plague | 17:55 |
| *** dustins_ has quit IRC | 17:55 | |
| dansmith | DuncanT: hence why I don't know the answer to this :) | 17:55 |
| DuncanT | dansmith: Somebody might be able to tell you for sure | 17:55 |
| DuncanT | dansmith: But I remember I couldn't create working bootable volumes from them when I tried | 17:56 |
| jbernard | dansmith: with boot-from-volume, the image linking metadata is lost, no? | 17:56 |
| dansmith | well, the volume boot scenario test was just re-enabled and it claims it should work :) | 17:56 |
| dansmith | jbernard: I don't know | 17:56 |
| dansmith | jbernard: if so, this should be failing for non-ceph cases too | 17:56 |
| jbernard | true | 17:57 |
| jbernard | dansmith: i may retry with lvm and see if anything looks different | 17:57 |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 17:58 |
| *** ndipanov has joined #openstack-cinder | 17:58 | |
| dansmith | DuncanT: does cinder import and store the image metadata with the volume? | 17:59 |
| DuncanT | dansmith: Yes. glance-metadata table | 17:59 |
| dansmith | okay | 17:59 |
| *** e0ne has joined #openstack-cinder | 17:59 | |
| jgriffith | dansmith: I believe the problem is that since Ceph typically *assumes* everything is on it's backend it doesn't do the convert | 18:00 |
| jgriffith | dansmith: if you look in the base drive you'll notice we download to a temp dir and run qemu-convert | 18:00 |
| jgriffith | dansmith: Ceph doesn't do that IIRC because it has "internal" knowledge of the image | 18:00 |
| vilobhmm11 | jgriffith : qemu-img convert to a RAW format you mean | 18:01 |
| jgriffith | vilobhmm11: yes | 18:01 |
| jgriffith | among other things... | 18:01 |
| dansmith | jgriffith: so in the normal case qemu-img convert should stuff the kernel and ramdisk into the image? | 18:01 |
| jgriffith | dansmith: I believe that's where the magic is yes | 18:02 |
| dansmith | jgriffith: okay, jbernard is going to try with lvm to see what a "normal" one looks like | 18:02 |
| jgriffith | dansmith: just a sec... looking in github | 18:02 |
| *** diegows has quit IRC | 18:02 | |
| dansmith | jgriffith: however, I'm not sure I get why this should really be different for ceph | 18:02 |
| jgriffith | dansmith: ok... so actually it does have the fetch_to_raw call | 18:02 |
| jgriffith | dansmith: yeah... agreed | 18:03 |
| *** Yogi1 has joined #openstack-cinder | 18:03 | |
| jgriffith | dansmith: in this case I don't think it should | 18:03 |
| dansmith | okay | 18:03 |
| *** dannywilson has joined #openstack-cinder | 18:04 | |
| jbernard | i am able to boot from volume on the command line, is something different about that? | 18:04 |
| jgriffith | jbernard: are you using a UEC? | 18:04 |
| jbernard | i did, yes | 18:05 |
| *** e0ne is now known as e0ne_ | 18:05 | |
| jbernard | a day or so ago | 18:05 |
| dansmith | jbernard: the same image that tempest is configured with? | 18:05 |
| dansmith | oh | 18:05 |
| jgriffith | jbernard: hmm | 18:05 |
| jbernard | yep, as far as I know | 18:05 |
| jgriffith | jbernard: dansmith BTW, we should probably remove the duplicate code in the ceph driver for copy_image_to_volume and just use the base driver | 18:05 |
| jbernard | jgriffith: yeah, ill fix that up once i get this sorted | 18:06 |
| jgriffith | jbernard: dansmith haha... actually I lied :) | 18:06 |
| jgriffith | looking at wrong tab | 18:06 |
| dansmith | *gasp* | 18:06 |
| jgriffith | so I don't see any of the convert calls in the rbd driver | 18:07 |
| dansmith | okay, that would explain it | 18:07 |
| dansmith | jgriffith: so we have three options it sounds like: | 18:08 |
| dansmith | 1. Make the driver do the conversion | 18:08 |
| dansmith | 2. Skip this in tempest for ceph as "uec BFV not supported" | 18:08 |
| *** nkrinner has joined #openstack-cinder | 18:08 | |
| dansmith | 3. Try to get the ceph job to use the disk-based cirros image | 18:08 |
| dansmith | sound right? | 18:08 |
| *** nkrinner has quit IRC | 18:08 | |
| *** e0ne_ has quit IRC | 18:10 | |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 18:10 |
| *** timcl has quit IRC | 18:12 | |
| dansmith | sdague: is #3 above doable? | 18:14 |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 18:14 |
| *** xyang1 has quit IRC | 18:15 | |
| *** ndipanov has quit IRC | 18:16 | |
| *** lpetrut has joined #openstack-cinder | 18:17 | |
| *** MentalRay has quit IRC | 18:18 | |
| *** diegows has joined #openstack-cinder | 18:21 | |
| *** timcl has joined #openstack-cinder | 18:21 | |
| *** ndipanov has joined #openstack-cinder | 18:21 | |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 18:23 |
| *** bkopilov has joined #openstack-cinder | 18:23 | |
| *** nkrinner has joined #openstack-cinder | 18:24 | |
| sks | DuncanT, The connection string of nova and cinder seems to be different as the port for cinde ris diff | 18:27 |
| *** Bsony_ has joined #openstack-cinder | 18:27 | |
| DuncanT | sks: The port should be the only difference | 18:27 |
| sks | DuncanT, yes | 18:27 |
| DuncanT | sks: I'm not sure what else it can be then. You can try connecting to the port with nc or something to see if it is really open? | 18:27 |
| *** Bsony has quit IRC | 18:27 | |
| sks | DuncanT, I wonder I tried in a different system with f20 devstack and there was no issue | 18:28 |
| *** e0ne has joined #openstack-cinder | 18:28 | |
| *** e0ne is now known as e0ne_ | 18:29 | |
| sks | DuncanT, I want to know the rootcause or its a bug | 18:29 |
| *** e0ne_ is now known as e0ne | 18:29 | |
| sks | DuncantT, connecting to port with nc? | 18:31 |
| *** ndipanov has quit IRC | 18:31 | |
| sks | ahh not connected ? | 18:31 |
| jgriffith | dansmith: sorry... had to step away | 18:34 |
| dansmith | jgriffith: np | 18:34 |
| jgriffith | dansmith: I'd at least test that the conversion is the issue | 18:34 |
| jgriffith | dansmith: I'm curious about jbernard 's testing though | 18:34 |
| *** timcl has quit IRC | 18:35 | |
| dansmith | jgriffith: yep, we're doing that now | 18:35 |
| *** erlon has joined #openstack-cinder | 18:35 | |
| dansmith | jgriffith: the steps above are assuming that's the issue | 18:35 |
| jgriffith | dansmith: ahh... cool | 18:35 |
| jgriffith | got ya | 18:35 |
| dansmith | jgriffith: is #1 an option? Assume it's an L thing if so | 18:35 |
| DuncanT | sks: nc = netcat, a simple tcp tool | 18:35 |
| *** Bsony_ has quit IRC | 18:36 | |
| jgriffith | dansmith: that where I'd start and see if it "works" | 18:36 |
| DuncanT | sks: run 'nc a.b.c.d <port>' and see if it connects | 18:36 |
| jgriffith | dansmith: I'd actually consider in L reworking that whole code path | 18:36 |
| jgriffith | dansmith: use the default method from the base driver if/when possible | 18:36 |
| *** dflorea has joined #openstack-cinder | 18:36 | |
| dansmith | jgriffith: what I mean is... is there any reason this is not done specifically for ceph (that you know of) or is this just an oversight? | 18:36 |
| jgriffith | and only do the backend Ceph magic when applicable (although that's likely to be 99% of the time in real world) | 18:37 |
| sks | DuncanT, ohh ok it is similar to ethereal | 18:37 |
| dansmith | cool, sounds like that'd be good | 18:37 |
| jgriffith | dansmith: yeah... to clarify I think it's NOT done now because typically Ceph is the Glance store and the Cinder backend | 18:37 |
| jgriffith | dansmith: so there's some 'magic' that it can do to make things more effecient and better | 18:37 |
| dansmith | jgriffith: sounds like it's that combined with "UECs are weird", but yeah | 18:38 |
| *** bswartz has joined #openstack-cinder | 18:38 | |
| jgriffith | dansmith: Yeah, that part is completely diff | 18:38 |
| jgriffith | dansmith: I don't know enough about the inner workings of Ceph to be of much help there | 18:38 |
| jgriffith | thingee: and jdurgin would know though probably | 18:38 |
| jgriffith | I think jdurgin wrote that code initially | 18:39 |
| dansmith | yep, I think jbernard is probably on that fix, if that's the fix | 18:39 |
| *** timcl has joined #openstack-cinder | 18:47 | |
| *** timcl has quit IRC | 18:50 | |
| *** emagana has quit IRC | 18:50 | |
| *** ho has joined #openstack-cinder | 18:52 | |
| jgriffith | mtreinish: :( the introduction of oslo_config raises some problems | 18:55 |
| *** dflorea has quit IRC | 18:56 | |
| *** bkopilov has quit IRC | 18:56 | |
| mtreinish | it conflicts with argparse or ConfigParser? | 18:57 |
| mtreinish | I was afraid of that | 18:57 |
| *** dflorea has joined #openstack-cinder | 18:57 | |
| mtreinish | jgriffith: we can revert that patch and make it a todo, and we can look at ripping oslo.db out of subunit2sql | 18:58 |
| jgriffith | mtreinish: well... neither :) | 18:58 |
| *** timcl has joined #openstack-cinder | 18:58 | |
| mtreinish | oh, then what did it break> | 18:58 |
| jgriffith | mtreinish: double register: http://goo.gl/wbYvmz | 18:58 |
| mtreinish | oh, because we're initing calling the register opts each time run is called | 18:59 |
| jgriffith | mtreinish: indeed | 18:59 |
| mtreinish | we can move the parse call outside of run and that should work I think | 18:59 |
| jgriffith | mtreinish: those are threads | 18:59 |
| jgriffith | yeah, that's what I'm looking at right now | 19:00 |
| jgriffith | mtreinish: just wanted to keep you in the loop :) | 19:00 |
| jgriffith | mtreinish: and make sure I wasn't missing something | 19:00 |
| mtreinish | still think I need to look at killing oslo.db there. There is no reason for me to be using it | 19:00 |
| *** dflorea has quit IRC | 19:01 | |
| *** MentalRay has joined #openstack-cinder | 19:05 | |
| marcusvrn | thingee: ping | 19:06 |
| *** tellesnobrega_ has joined #openstack-cinder | 19:07 | |
| jgriffith | mtreinish: looking at a quick hack to avoid revert | 19:08 |
| *** tellesnobrega_ has quit IRC | 19:08 | |
| jgriffith | mtreinish: but yeah.. maybe since I just got a ping "hey.. your sos-ci stuff doesn't work" LOL | 19:08 |
| jgriffith | mtreinish: I *think* could just set those config options when I init the threads | 19:09 |
| *** emagana has joined #openstack-cinder | 19:09 | |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 19:09 |
| mtreinish | jgriffith: yeah that should work, since it should only be needed once per process space | 19:10 |
| *** lifeless has quit IRC | 19:11 | |
| jgriffith | mtreinish: yeah, going to try it now | 19:11 |
| *** rushil has joined #openstack-cinder | 19:13 | |
| *** geguileo has quit IRC | 19:13 | |
| jgriffith | mtreinish: I've always wanted to try passing a method to the thread constructor.. now's my chance :) | 19:13 |
| jgriffith | mtreinish: meh... I'll just make an init | 19:15 |
| sdague | dansmith: so.... there was some reason why we are using euc images in devstack by default, but I can't tell you why now | 19:15 |
| mtreinish | jgriffith: heh, come on push the boundaries man | 19:15 |
| *** Apoorva has quit IRC | 19:15 | |
| dansmith | sdague: okay, we just confirmed that the problem is ceph+uec images | 19:15 |
| jgriffith | sdague: wasn't it because of size and compat for Xen at one ponit? | 19:15 |
| sdague | I do agree using really img for boot from volume is probably more sensible | 19:15 |
| jgriffith | *point* | 19:16 |
| dansmith | sdague: okay, so should we convert devstack to that, or just make it optional and enable for the ceph job? | 19:16 |
| sdague | jgriffith: maybe, I honestly dtroyer might know | 19:16 |
| jgriffith | dansmith: sdague so just my two cents... I'm not completely comfortable with the notion of changing how we do things because Ceph doesn't work | 19:16 |
| jgriffith | dansmith: not necessarily saying that's what you're proposing | 19:17 |
| *** Rockyg has joined #openstack-cinder | 19:17 | |
| dansmith | jgriffith: you mean the ceph cinder driver... :) | 19:17 |
| sdague | dansmith: we need to get the img downloaded and installed ... probably as a second type. And I expect that tempest will need a config for 'use this other image' | 19:17 |
| jgriffith | dansmith: just kinda "icky" | 19:17 |
| dansmith | sdague: right, okay | 19:17 |
| dansmith | I'm not sure that using a uec image really mirrors how most people use openstack (without ec2 at least) | 19:18 |
| dansmith | so image-based seems more correct-er to me anyway, but adding it to glance as another option seems like a good plan | 19:18 |
| jgriffith | dansmith: maybe not.. BUT I would argue thatt given every other Cinder backend supports this Ceph shoudl as well | 19:18 |
| dansmith | jgriffith: no argument there | 19:18 |
| jgriffith | unless we're going to say OpenStack no longer supports UEC images? | 19:18 |
| jgriffith | dansmith: got ya | 19:18 |
| sdague | jgriffith: any idea if it's ceph, or the cinder driver? | 19:19 |
| jgriffith | sdague: soryy.. I should clarify it's likely the ceph driver | 19:19 |
| jgriffith | sdague: in this case at least | 19:19 |
| dansmith | yep | 19:19 |
| jgriffith | sdague: Oh...wait | 19:19 |
| jgriffith | sdague: dansmith the Ceph devstack run uses Ceph as image store as well no? | 19:19 |
| dansmith | jgriffith: yeah | 19:19 |
| jgriffith | that explains the differing results form jbernard maybe | 19:20 |
| dansmith | it really doesn't matter.. if the ceph cinder driver turns a UEC into what we're seeing, then it's broken | 19:20 |
| jgriffith | anyway... I don't want to confuse things. I could be completely off base | 19:20 |
| dansmith | it either needs to fail that or do it right | 19:20 |
| sdague | still seems like there should be a bug on the cinder side though to fix the driver | 19:20 |
| dansmith | agreed | 19:20 |
| jgriffith | dansmith: agree.. I'm just talking about how to figure out if it's *ceph* or a simple driver fix | 19:20 |
| jgriffith | sdague: +1 | 19:21 |
| dansmith | jgriffith: I don't think ceph knows anything about any of this | 19:21 |
| *** rushiagr is now known as rushiagr_away | 19:21 | |
| dansmith | this is at least a layer above what ceph actually knows about, AFAIK | 19:21 |
| dansmith | it knows how to do the magic clone of the glance image to the thing cinder uses, | 19:21 |
| jgriffith | dansmith: perhaps, it's been a very long time since i've dug aroudn in there... but it used to be they had some internal knowledge for moving things from object to block to file etc | 19:22 |
| dansmith | but it doesn't know about combining the UEC parts into a bootable linux image I don't think | 19:22 |
| jgriffith | but I could be compeltely wrong | 19:22 |
| dansmith | sure, but that's not the problem here | 19:22 |
| dansmith | it's combining three things into one in the right way | 19:22 |
| jgriffith | and you're right, the UEC stuff in particular I wouldn't expect it to have any knowledge of at all | 19:22 |
| jgriffith | dansmith: yeah... I'm with ya. | 19:22 |
| jgriffith | dansmith: you want to file the bug or shall I? | 19:22 |
| jgriffith | dansmith: I def agree it's a driver bug | 19:23 |
| dansmith | jbernard might want the bug glory, I dunno :) | 19:23 |
| jgriffith | lol... I'll give him a chance to file it | 19:23 |
| dansmith | heh | 19:23 |
| jgriffith | if nothing there tonight I'll post some jibberish :) | 19:23 |
| jgriffith | window 43 | 19:24 |
| smcginnis | 43 windows? Dang. | 19:25 |
| jgriffith | smcginnis: I need to close my session and start over at some point | 19:26 |
| jgriffith | smcginnis: this byobu session has been up for almost a year | 19:26 |
| smcginnis | jgriffith: You know you can close some of those, right? :) | 19:26 |
| jgriffith | smcginnis: I know... I'm lazy | 19:26 |
| smcginnis | :) | 19:26 |
| jgriffith | smcginnis: and irssi doesn't always "fix" up the numbers | 19:26 |
| smcginnis | jgriffith: Yeah, I've noticed that. Works good most of the time, but not always. | 19:27 |
| jgriffith | smcginnis: so my count goes like 10,11,12,40,41,42,50 | 19:27 |
| *** dflorea has joined #openstack-cinder | 19:27 | |
| *** nkrinner has quit IRC | 19:28 | |
| smcginnis | byobu looks interesting. I've only ever used screen. | 19:28 |
| jbernard | jgriffith: ill file it, i have my own particular brand of jibberish :) | 19:29 |
| jgriffith | jbernard: :) | 19:30 |
| jgriffith | I find clippy being on my gerrit dashboard very disturbing | 19:31 |
| jbernard | i installed no-script specifically for those shenanigans | 19:31 |
| jgriffith | especially when he/she is sleeping | 19:31 |
| jgriffith | jbernard: ahh.. nice! | 19:31 |
| *** dflorea has quit IRC | 19:32 | |
| jgriffith | jbernard: and the beauty is if I block something I don't want to, I'll never know :) | 19:32 |
| jbernard | exactly@ | 19:32 |
| jgriffith | :) | 19:32 |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 19:34 |
| *** lifeless has joined #openstack-cinder | 19:40 | |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 19:40 |
| *** dflorea has joined #openstack-cinder | 19:40 | |
| *** reed has quit IRC | 19:40 | |
| *** dflorea has quit IRC | 19:42 | |
| *** dflorea has joined #openstack-cinder | 19:42 | |
| *** dflorea has quit IRC | 19:42 | |
| *** dflorea has joined #openstack-cinder | 19:43 | |
| *** sks has quit IRC | 19:47 | |
| *** cpg|away is now known as cpg | 19:48 | |
| *** nlevinki has quit IRC | 19:51 | |
| *** dflorea has quit IRC | 20:00 | |
| *** annashen has quit IRC | 20:00 | |
| openstackgerrit | Alessandro Pilotti proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 20:01 |
| *** dflorea has joined #openstack-cinder | 20:01 | |
| *** lcurtis has joined #openstack-cinder | 20:04 | |
| *** dflorea has quit IRC | 20:05 | |
| *** annegent_ has joined #openstack-cinder | 20:06 | |
| jbernard | jgriffith: https://bugs.launchpad.net/cinder/+bug/1439371 | 20:09 |
| openstack | Launchpad bug 1439371 in Cinder "Volume creation from image fails for UEC+Ceph" [Undecided,New] - Assigned to Jon Bernard (jbernard) | 20:09 |
| jbernard | jgriffith: milestone should be L, correct? | 20:09 |
| thingee | jbernard: bug fixes right now are ok | 20:16 |
| *** vilobhmm11 has quit IRC | 20:16 | |
| *** vilobhmm1 has joined #openstack-cinder | 20:16 | |
| *** vilobhmm1 has quit IRC | 20:17 | |
| *** emagana has quit IRC | 20:17 | |
| jbernard | thingee: ok, i still have a few gaps in my understanding, but ill try to get it sorted asap | 20:17 |
| *** vilobhmm1 has joined #openstack-cinder | 20:17 | |
| *** vilobhmm1 has quit IRC | 20:18 | |
| casusbelli | jgriffith: Hi! Quick Q: Regarding the localconf.base in sos-ci, is the network config in there the config that you're really using? If not could you pls give me a paste or info on how you have that configured? | 20:18 |
| *** vilobhmm1 has joined #openstack-cinder | 20:18 | |
| jbernard | thingee: i didn't check the -initrd and -kernel options to qemu | 20:18 |
| *** vilobhmm1 has quit IRC | 20:18 | |
| *** vilobhmm1 has joined #openstack-cinder | 20:18 | |
| jbernard | thingee: presumably, if rbd only extracts the uec rootfs, libvirt could still boot the instance from a volume if it also fetched the remaining pieces, no? | 20:18 |
| jbernard | thingee: can you clarify what is /supposed/ to happen in that case? | 20:19 |
| thingee | marcusvrn: pong | 20:22 |
| jgriffith | casusbelli: it is | 20:22 |
| casusbelli | jgriffith: thanks! | 20:23 |
| jgriffith | casusbelli: np | 20:24 |
| jgriffith | casusbelli: is the /20 throwing you off maybe? | 20:24 |
| casusbelli | jgriffith: /20 ?? | 20:24 |
| marcusvrn | thingee: does CI need to run tempest.thirdparty.boto tests, too? | 20:24 |
| jgriffith | casusbelli: never mind then :) | 20:24 |
| jgriffith | bb in a hour | 20:24 |
| casusbelli | jgriffith: :-D | 20:24 |
| *** sseverson has joined #openstack-cinder | 20:24 | |
| *** vilobhmm1 has quit IRC | 20:27 | |
| casusbelli | jgriffith: ah, now got it, no different issue... :) | 20:28 |
| *** Yogi1 has quit IRC | 20:30 | |
| *** emagana has joined #openstack-cinder | 20:30 | |
| *** annegent_ has quit IRC | 20:31 | |
| *** lpetrut has quit IRC | 20:32 | |
| thingee | marcusvrn: in my testing this is not skipped by default devstack install | 20:32 |
| thingee | I think there is one skipped | 20:32 |
| thingee | marcusvrn: setUpClass (tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest) ... SKIPPED: EC2 InstanceRunTest: requires ami/aki/ari manifest | 20:32 |
| thingee | marcusvrn: but these aren't: | 20:33 |
| thingee | tempest.thirdparty.boto.test_ec2_volumes.EC2VolumesTest.test_create_get_delete [13.925216s] ... ok | 20:33 |
| thingee | tempest.thirdparty.boto.test_ec2_volumes.EC2VolumesTest.test_create_volume_from_snapshot [4.306667s] ... ok | 20:33 |
| mtreinish | thingee: heh, I forgot those boto tests would match a regex of 'volume' | 20:33 |
| marcusvrn | thingee: mine is not skipping and it is failing (not all time) | 20:34 |
| marcusvrn | thingee: xyang's CI is not skipping also (http://publiclogs.emc.com/vnx_ostack/EMC_VNX_ISCSI/1906/console.html) | 20:35 |
| *** annashen has joined #openstack-cinder | 20:35 | |
| thingee | jbernard: I don't have experience of trying that with a UEC image, but worth a shot. | 20:36 |
| thingee | jbernard: I don't have any experience with uec really | 20:36 |
| marcusvrn | thingee: solidfire's CI is skipping it: http://54.164.167.86/solidfire-ci-logs/refs-changes-10-163910-6/console.log.out | 20:36 |
| thingee | marcusvrn: just that one that I already listed here, yes | 20:37 |
| thingee | the other two arent | 20:37 |
| marcusvrn | so....CIs are not running the same tests.... | 20:37 |
| thingee | marcusvrn: that was established on the list and pointed out | 20:37 |
| thingee | marcusvrn: As I mentioned those two boto tests by default aren't skipped | 20:38 |
| *** vilobhmm1 has joined #openstack-cinder | 20:39 | |
| thingee | marcusvrn: solidfire is doing the correct thing, only skipping the one because the aki manifest dne | 20:39 |
| thingee | marcusvrn: so I'm not sure what you're a sking | 20:39 |
| thingee | marcusvrn: the two I just pointed out in chat are running though and not being skipped | 20:39 |
| marcusvrn | thingee: how can I skip those boto tests while I'm trying to find out what is the problem with them? | 20:40 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add volume:update rule to policy.json https://review.openstack.org/169915 | 20:40 |
| thingee | mtreinish: ^ | 20:40 |
| thingee | mtreinish: not sure if we figured out how to allow ci's to skip stuff that's being fixed up | 20:41 |
| mtreinish | thingee: well the only method right now is to create a regex that excludes the tests you don't want to run | 20:42 |
| mtreinish | and use that instead of 'volume' | 20:42 |
| mtreinish | thingee: in the near term once I get os-testr released on pypi and tempest switched over to it there will be an exclude file people can use | 20:42 |
| mtreinish | so it'll make things a bit nicer | 20:43 |
| thingee | mtreinish: that sounds great long term, thanks | 20:43 |
| *** melwitt has joined #openstack-cinder | 20:43 | |
| thingee | marcusvrn: so it's following https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#What_tests_do_I_use.3F | 20:43 |
| thingee | marcusvrn: but instead of volume, a regex to include volume tests, but exclude ones that contain boto | 20:43 |
| mtreinish | thingee: heh, well long term is as soon as I can get a +A on my project-config patch which already has 2 +2s :) | 20:44 |
| thingee | marcusvrn: for now | 20:44 |
| mtreinish | thingee: I think something like: '(?!boto)(volume)' would work | 20:44 |
| mtreinish | but I suck at regex so no guarantees | 20:44 |
| marcusvrn | thingee: I'm doing it, but it will be not acceptable after the deadline, will it? | 20:45 |
| thingee | mtreinish: rent a perl monk? | 20:45 |
| thingee | I don't think these tests really have much to do with your driver itself failing, so I think it'll be fine...just get it fixed soon. | 20:46 |
| thingee | marcusvrn: ^ | 20:46 |
| thingee | mtreinish might correct me on that though | 20:47 |
| mtreinish | thingee: well, they might depending on which tests are failing they could just be tickling a bug in a feature | 20:47 |
| mtreinish | I'd have to see an example failure to really judge | 20:47 |
| marcusvrn | mtreinish: http://177.84.241.119:10000/41/163641/17/silent/hds-hnas-nfs/eb563e0/console.html | 20:48 |
| marcusvrn | grep for tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest.test_compute_with_volumes[id-ab836c29-737b-4101-9fb9-87045eaf89e9] | 20:48 |
| thingee | bbl | 20:49 |
| marcusvrn | it seems like an SSH issue | 20:49 |
| mtreinish | marcusvrn: well I found: http://177.84.241.119:10000/41/163641/17/silent/hds-hnas-nfs/eb563e0/logs/screen-c-api.txt.gz?level=TRACE | 20:51 |
| *** e0ne has quit IRC | 20:51 | |
| mtreinish | you might also want to switch to using config drive by default, because http://177.84.241.119:10000/41/163641/17/silent/hds-hnas-nfs/eb563e0/logs/screen-n-api.txt.gz?level=TRACE might mean that the metatadata service might be having issues | 20:51 |
| *** e0ne has joined #openstack-cinder | 20:52 | |
| mtreinish | (although it almost never works right on a good day) | 20:52 |
| *** thingee has quit IRC | 20:53 | |
| mtreinish | marcusvrn: but yeah that particular failure might not be related to your driver, just a general ci config issue | 20:54 |
| *** e0ne has quit IRC | 20:55 | |
| *** bkopilov has joined #openstack-cinder | 20:57 | |
| marcusvrn | mtreinish: hmmm nice, so can I remove those tests for now and try to fix it up later, right? :) | 20:57 |
| *** e0ne has joined #openstack-cinder | 20:58 | |
| *** rushil has quit IRC | 20:59 | |
| marcusvrn | mtreinish: I have another issue, maybe you can help me... http://177.84.241.119:10000/83/169283/1/silent/hitachi-hbsd-fc/a02efa9/console.html | 20:59 |
| mtreinish | marcusvrn: that's not my call. :) But, fwiw I think using the metadata service is probably the cause of your ssh failures and switching to config drive would help | 20:59 |
| *** emagana has quit IRC | 21:00 | |
| *** emagana has joined #openstack-cinder | 21:00 | |
| marcusvrn | mtreinish: that issue (minimum_basic_scenario) is failing on ssh authentication...It does all volume stuff, but fails on ssh authentication | 21:01 |
| mtreinish | marcusvrn: in that case the test took too long, we kill any test which exceeds 500 secs | 21:01 |
| *** dustins has quit IRC | 21:02 | |
| mtreinish | the traceback shows the fixture raised a timeout exception | 21:02 |
| *** timcl has quit IRC | 21:03 | |
| mtreinish | marcusvrn: you can increase the timeout by setting: OS_TEST_TIMEOUT in the env that calls testr | 21:03 |
| mtreinish | marcusvrn: if your backend or the ci environment is a bit slower than what we gate with, 500 secs might not be sufficient | 21:04 |
| marcusvrn | mtreinish: and about switching driver configurations,... I tried it out, but it didn't work... may be I didn't change the right config.... | 21:04 |
| *** emagana_ has joined #openstack-cinder | 21:04 | |
| *** emagana has quit IRC | 21:04 | |
| mtreinish | marcusvrn: you mean setting to default to config drive? | 21:05 |
| *** e0ne has quit IRC | 21:05 | |
| mtreinish | marcusvrn: if so you should just need to set FORCE_CONFIG_DRIVE=True in your localrc/local.conf | 21:06 |
| *** akerr has quit IRC | 21:06 | |
| marcusvrn | mtreinish: ah... so that's the option that I need to set... | 21:06 |
| marcusvrn | the OS_TEST_TIMEOUT, have I to set it in tempest.conf file? | 21:07 |
| mtreinish | marcusvrn: no it's an env variable | 21:07 |
| marcusvrn | mtreinish: ok, I will export it | 21:08 |
| *** thangp has quit IRC | 21:08 | |
| marcusvrn | mtreinish: the FORCE_CONFIG_DRIVE is already set as True... | 21:10 |
| *** diegows has quit IRC | 21:10 | |
| *** Lee1092 has quit IRC | 21:10 | |
| marcusvrn | mtreinish: http://paste.openstack.org/show/197833/ | 21:11 |
| smcginnis | mtreinish: Do you have a reference to FORCE_CONFIG_DRIVE that explains what that entails? | 21:12 |
| *** thingee has joined #openstack-cinder | 21:13 | |
| smcginnis | mtreinish: nm, found it: http://docs.openstack.org/user-guide/content/enable_config_drive.html | 21:14 |
| mtreinish | marcusvrn: hmm, yeah I see that in the logs now. I wonder what is querying the metadata server then? | 21:14 |
| *** _cjones_ has quit IRC | 21:14 | |
| mtreinish | because it should be attaching a config drive the guest and cloud-init uses that instead of asking the metadata server for anything | 21:14 |
| jgriffith | marcusvrn: what? | 21:15 |
| *** sgotliv has quit IRC | 21:15 | |
| mtreinish | oh I wonder if the ec2 stuff does things differently | 21:16 |
| *** julim has quit IRC | 21:16 | |
| marcusvrn | mtreinish:hmmm | 21:16 |
| marcusvrn | jgriffith: what what? | 21:17 |
| mtreinish | yeah looking at a gate run, that's what it looks like, ec2 api calls are always querying the metadata server | 21:17 |
| jgriffith | marcusvrn: what test is SolidFire skipping that others aren't? | 21:17 |
| *** Liu has quit IRC | 21:18 | |
| marcusvrn | jgriffith: ah, this one: tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest | 21:18 |
| *** MentalRay has quit IRC | 21:18 | |
| smcginnis | How would that be run if you are filtering on 'volume'? | 21:18 |
| *** MentalRay has joined #openstack-cinder | 21:19 | |
| jgriffith | marcusvrn: I'll check my enabled services, thanks | 21:19 |
| jgriffith | let the stone throwing begin :) | 21:19 |
| *** dims_ has quit IRC | 21:19 | |
| marcusvrn | mtreinish: if I use the default metadata, Should it solve that problem? | 21:19 |
| *** _cjones_ has joined #openstack-cinder | 21:20 | |
| *** dflorea has joined #openstack-cinder | 21:21 | |
| marcusvrn | jgriffith: nooo, it's not my intention...I'm trying to fix this test in my driver and I saw that EMC CI is passing but yours is skipping it...so I was trying to find why you guys are skipping it, because it can be the same problem that I have... | 21:22 |
| marcusvrn | jgriffith: I'm just trying to find out people with the same problem to try to solve it | 21:23 |
| mtreinish | marcusvrn: what do you mean by default metadata? I think your error is caused by the nova metadata api service not working correctly which I thought default to always use config drive would fix | 21:23 |
| *** Liu has joined #openstack-cinder | 21:24 | |
| mtreinish | but with ec2 requests it always queries the metadata api service so I'm not sure what to tell you next | 21:24 |
| jgriffith | marcusvrn: so the secret lies in the local.conf | 21:24 |
| jgriffith | marcusvrn: the different services that are enabled etc | 21:24 |
| mtreinish | jgriffith: heh, are you just disabling ec2? | 21:24 |
| jgriffith | marcusvrn: and, not sure but I *think* EMC was running all tests, no filter | 21:24 |
| mtreinish | jgriffith: I wish I could just do that in tempest | 21:25 |
| jgriffith | mtreinish: marcusvrn nahh... just enabling the minimum: http://goo.gl/do6WtJ | 21:25 |
| marcusvrn | mtreinish: hmm I see | 21:25 |
| jgriffith | mtreinish: and using "tox -eall -- volume" | 21:25 |
| *** dflorea has quit IRC | 21:25 | |
| jgriffith | to filter only on Volume tagged tests | 21:25 |
| *** dflorea has joined #openstack-cinder | 21:25 | |
| jgriffith | marcusvrn: what's your tempst command look like? | 21:25 |
| marcusvrn | jgriffith: I'm using devstack gate, so I'm using 'export DEVSTACK_GATE_TEMPEST_REGEX="volume"' | 21:26 |
| jgriffith | marcusvrn: hmm... that should essentially be the same I think | 21:27 |
| jgriffith | marcusvrn: well... other than the "other" config options | 21:27 |
| jgriffith | marcusvrn: suppose I could modfiy my setup to install "everything" and duplicate what you've got going on | 21:27 |
| *** boris-42 has quit IRC | 21:28 | |
| marcusvrn | jgriffith: and I have the same services enabled as you have: http://paste.openstack.org/show/197833/ | 21:28 |
| jgriffith | indeed | 21:29 |
| mtreinish | marcusvrn: do you have an example of jgriffith's solidfire runs that has the ec2 test your failing on skipped? | 21:29 |
| *** cbader has quit IRC | 21:30 | |
| marcusvrn | mtreinish: hm.... one sec | 21:30 |
| jgriffith | http://54.164.167.86/solidfire-ci-logs/refs-changes-98-165798-3/testr_results.html | 21:30 |
| marcusvrn | http://54.164.167.86/solidfire-ci-logs/refs-changes-10-163910-6/console.log.out | 21:30 |
| jgriffith | marcusvrn: forgot the test in question already :) | 21:30 |
| marcusvrn | jgriffith: tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest | 21:30 |
| mtreinish | marcusvrn: ah it's because he isn't setting up the ec2 images so it's getting skipped in setup class | 21:31 |
| jgriffith | :) | 21:32 |
| jgriffith | mtreinish: correct good sir | 21:32 |
| mtreinish | {2} setUpClass (tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest) ... SKIPPED: EC2 InstanceRunTest: requires ami/aki/ari manifest | 21:32 |
| *** dflorea has quit IRC | 21:32 | |
| *** dflorea has joined #openstack-cinder | 21:33 | |
| mtreinish | man I hate the boto test code... | 21:33 |
| jgriffith | mtreinish: I hate boto | 21:33 |
| mtreinish | man is this clear: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/thirdparty/boto/test.py#n42 | 21:34 |
| jgriffith | LOL | 21:34 |
| marcusvrn | lol | 21:36 |
| marcusvrn | mriedem: jgriffith: the question is...Is that test necessary? because I think what are been tested in that test, are tested in another tests, correct me if I'm wrong.. | 21:36 |
| marcusvrn | ops.. mtreinish** | 21:36 |
| mriedem | mtreinish: it's the decider, what's not clear about it? | 21:37 |
| *** dflorea has quit IRC | 21:37 | |
| marcusvrn | mriedem: sry, I tagged you wrongly | 21:39 |
| jgriffith | marcusvrn: So I think this might be the same *problem* dansmith is working on with uec images on the Ceph driver | 21:40 |
| mriedem | marcusvrn: i know :) | 21:40 |
| mriedem | happens a lot | 21:40 |
| jgriffith | mriedem: :) | 21:40 |
| mriedem | but now i'm here and you're all going to have to deal with it | 21:40 |
| jgriffith | LOL | 21:40 |
| mtreinish | marcusvrn: I can't really judge whether it's necessary in your use case (personally I want to nuke the ec2 tests in tempest) | 21:41 |
| marcusvrn | mriedem: lol :) | 21:41 |
| mtreinish | I think the test in question does overlap, if at least partially, with one of the volume scenario tests | 21:41 |
| jgriffith | marcusvrn: any indication why it fails? | 21:41 |
| mtreinish | but I'd have to compare the code to be sure | 21:41 |
| mtreinish | jgriffith: he's getting an error from nova metadata when ec2 tries to talk to it and he's never able to login because of that | 21:42 |
| jgriffith | oh | 21:42 |
| marcusvrn | yep | 21:42 |
| jgriffith | not what I thought | 21:42 |
| mtreinish | of course the nova metadata logging doesn't really say anything: http://177.84.241.119:10000/41/163641/17/silent/hds-hnas-nfs/eb563e0/logs/screen-n-api.txt.gz?#_2015-04-01_10_57_50_894 | 21:42 |
| *** Rockyg has quit IRC | 21:43 | |
| mtreinish | oh, wait that's the 3rd failure I wonder if it's just a cascade issue from the first one | 21:43 |
| mtreinish | marcusvrn: http://177.84.241.119:10000/41/163641/17/silent/hds-hnas-nfs/eb563e0/logs/screen-n-cpu.txt.gz?level=TRACE | 21:44 |
| mtreinish | that's probably the real reason your stuff is failing | 21:44 |
| jgriffith | can you have metadata service without config drive? | 21:44 |
| mtreinish | jgriffith: yeah, they're not mutual exclusive | 21:45 |
| jgriffith | k... I'm painfully unfamiliar :( | 21:45 |
| jgriffith | marcusvrn: FWIW, I'll spin it up and see if I get the same problem | 21:45 |
| mtreinish | force_config_drive is just a way to tell nova to boot guests assuming all incoming api requests have the use config drive api param set | 21:46 |
| jgriffith | mtreinish: Ahh... ok | 21:46 |
| mtreinish | which would bypass having cloud init query the metadata server for anything | 21:46 |
| mtreinish | (of course not with ec2 requests because it always goes to metadata for those) | 21:46 |
| jgriffith | Ohhh.... I remember this | 21:47 |
| *** xyang1 has joined #openstack-cinder | 21:47 | |
| marcusvrn | jgriffith: ok | 21:47 |
| mtreinish | jgriffith: we do it on purpose because using the metadata service (especially with neutron) goes wrong more often than not | 21:47 |
| jgriffith | marcusvrn: http://goo.gl/dVpMuJ | 21:48 |
| jgriffith | ^^ I'll be of no help to you here | 21:48 |
| mtreinish | because it's a terrible idea to hard code an ip for things to query to get the metadata | 21:48 |
| jgriffith | :) | 21:48 |
| marcusvrn | this IP is my local repository server | 21:50 |
| *** mriedem is now known as mriedem_away | 21:51 | |
| marcusvrn | Is this not the right way to point to the local repository? | 21:52 |
| jgriffith | Ummm... stupid question, but; why are you setting that anywa marcusvrn ? | 21:55 |
| jgriffith | looking at logs form a recent dsv-full run by the gate: http://logs.openstack.org/24/169124/3/check/check-tempest-dsvm-full/7576a66/logs/localrc.txt.gz | 21:55 |
| *** vilobhmm1 has quit IRC | 21:57 | |
| jgriffith | marcusvrn: http://goo.gl/p2Jfdc | 21:57 |
| jgriffith | marcusvrn: we don't even run it in the gate | 21:57 |
| *** boris-42 has joined #openstack-cinder | 21:57 | |
| *** thingee has quit IRC | 21:59 | |
| marcusvrn | jgriffith: to try to make builds faster, because, initially, heat was enabled and it uses fedora image, which is a little bit huge to download every time....then as I had to setup local repo to fedora image, I put all images needed, locally | 21:59 |
| marcusvrn | jgriffith: ah...you're talking about the test...lol | 22:00 |
| *** emagana_ has quit IRC | 22:01 | |
| marcusvrn | jgriffith: actually, I'm running this test because I'm using the wiki cinder regex and it runs that test | 22:02 |
| *** dflorea has joined #openstack-cinder | 22:04 | |
| marcusvrn | I can exclude that test, ... but I don't know if it is acceptable, once I'm excluding a test manually... | 22:04 |
| jgriffith | marcusvrn: it's the local.conf | 22:04 |
| *** ekarlso has quit IRC | 22:04 | |
| *** ekarlso has joined #openstack-cinder | 22:04 | |
| jgriffith | not the regex | 22:04 |
| *** vilobhmm1 has joined #openstack-cinder | 22:07 | |
| marcusvrn | but I have the same SKIP_EXERCISES -> http://paste.openstack.org/show/197833/ | 22:08 |
| *** dflorea has quit IRC | 22:08 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix missing translations for log messages https://review.openstack.org/165402 | 22:11 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Logging not using oslo.i18n guidelines https://review.openstack.org/165276 | 22:12 |
| smcginnis | hemna: Thanks! | 22:12 |
| hemna | :) | 22:13 |
| *** 77CAAKTHH has joined #openstack-cinder | 22:13 | |
| *** scottda_ has joined #openstack-cinder | 22:16 | |
| *** thingee has joined #openstack-cinder | 22:16 | |
| marcusvrn | jgriffith: well....I have to... tomorrow morning I'll continue on it.... | 22:17 |
| marcusvrn | have to go* | 22:17 |
| marcusvrn | 22:18 | |
| marcusvrn | mtreinish: jgriffith thanks a lot :) | 22:18 |
| *** asselin has joined #openstack-cinder | 22:20 | |
| *** scottda_ has quit IRC | 22:21 | |
| *** reed has joined #openstack-cinder | 22:21 | |
| jgriffith | marcusvrn: hehe.. what I'm saying is remove the uec image entries from your local.conf | 22:22 |
| jgriffith | marcusvrn: we don't run those in dsvm-full in the gate anyway | 22:22 |
| jgriffith | marcusvrn: unless you just want to be an over-achiever :) | 22:22 |
| *** zhipeng has joined #openstack-cinder | 22:23 | |
| *** 77CAAKTHH has quit IRC | 22:23 | |
| *** asselin has quit IRC | 22:25 | |
| *** esker has quit IRC | 22:29 | |
| *** scottda_ has joined #openstack-cinder | 22:29 | |
| *** lcurtis has quit IRC | 22:30 | |
| *** emagana has joined #openstack-cinder | 22:35 | |
| *** thingee has quit IRC | 22:42 | |
| *** ebalduf has joined #openstack-cinder | 22:43 | |
| *** thingee has joined #openstack-cinder | 22:43 | |
| *** emagana has quit IRC | 22:43 | |
| *** mwichmann has quit IRC | 22:45 | |
| *** ebalduf has quit IRC | 22:47 | |
| *** annegentle has joined #openstack-cinder | 22:48 | |
| *** annegentle has quit IRC | 22:48 | |
| *** mwichmann has joined #openstack-cinder | 22:57 | |
| *** annegentle has joined #openstack-cinder | 22:58 | |
| *** hemna is now known as hemnafk | 22:59 | |
| *** mwichmann has quit IRC | 23:00 | |
| *** mwichmann has joined #openstack-cinder | 23:05 | |
| *** scottda_ has quit IRC | 23:06 | |
| *** chlong has joined #openstack-cinder | 23:07 | |
| *** zhipeng has quit IRC | 23:13 | |
| *** annashen has quit IRC | 23:14 | |
| *** annegentle has quit IRC | 23:15 | |
| *** iwao has joined #openstack-cinder | 23:16 | |
| *** dflorea has joined #openstack-cinder | 23:16 | |
| *** scottda_ has joined #openstack-cinder | 23:19 | |
| *** annegentle has joined #openstack-cinder | 23:29 | |
| *** harlowja has quit IRC | 23:31 | |
| *** harlowja has joined #openstack-cinder | 23:32 | |
| *** annegentle has quit IRC | 23:37 | |
| *** dflorea has quit IRC | 23:39 | |
| *** dflorea has joined #openstack-cinder | 23:40 | |
| *** mwichmann has quit IRC | 23:42 | |
| *** scottda_ has quit IRC | 23:43 | |
| *** dflorea has quit IRC | 23:44 | |
| *** harlowja has quit IRC | 23:49 | |
| *** harlowja has joined #openstack-cinder | 23:50 | |
| *** zongliang has joined #openstack-cinder | 23:54 | |
| openstackgerrit | Hahyun proposed openstack/os-brick: Fix wrong command for _rescan_multipath The _run_multipath command is wrong, it should be 'multipath -r' not 'multipath - r'. https://review.openstack.org/169951 | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!