| *** gouthamr has quit IRC | 00:01 | |
| *** julim has joined #openstack-cinder | 00:09 | |
| *** dave-mccowan has quit IRC | 00:12 | |
| *** dave-mccowan has joined #openstack-cinder | 00:15 | |
| *** dims_ has quit IRC | 00:17 | |
| *** wilson1 has joined #openstack-cinder | 00:26 | |
| *** wilson-1 has quit IRC | 00:29 | |
| *** bswartz has quit IRC | 01:01 | |
| *** abehl has quit IRC | 01:02 | |
| *** bswartz has joined #openstack-cinder | 01:03 | |
| *** david-lyle has quit IRC | 01:13 | |
| *** lcurtis has joined #openstack-cinder | 01:21 | |
| *** julim has quit IRC | 01:22 | |
| *** Lee1092 has joined #openstack-cinder | 01:31 | |
| *** ronis has quit IRC | 01:31 | |
| *** david-lyle has joined #openstack-cinder | 01:44 | |
| openstackgerrit | xing-yang proposed openstack/cinder: CG driver function should not access db https://review.openstack.org/229580 | 01:58 |
|---|---|---|
| *** gouthamr has joined #openstack-cinder | 02:07 | |
| *** gouthamr_ has quit IRC | 02:10 | |
| *** gouthamr has quit IRC | 02:12 | |
| openstackgerrit | xing-yang proposed openstack/cinder: VMAX Target iSCSI IP Address https://review.openstack.org/229712 | 02:13 |
| *** gouthamr has joined #openstack-cinder | 02:20 | |
| *** bswartz has quit IRC | 02:20 | |
| *** dimsum__ has joined #openstack-cinder | 02:21 | |
| *** gouthamr has quit IRC | 02:30 | |
| openstackgerrit | Zhang Jinnan proposed openstack/cinder: Volume extend error not catch exception https://review.openstack.org/227971 | 02:32 |
| *** bswartz has joined #openstack-cinder | 02:39 | |
| *** dimsum__ has quit IRC | 02:40 | |
| *** lcurtis has quit IRC | 02:42 | |
| *** asselin__ has quit IRC | 02:51 | |
| *** julim has joined #openstack-cinder | 03:03 | |
| *** dave-mccowan has quit IRC | 03:07 | |
| *** julim has quit IRC | 03:09 | |
| *** david-lyle has quit IRC | 03:32 | |
| *** smoriya has joined #openstack-cinder | 03:35 | |
| *** asselin_ has joined #openstack-cinder | 03:43 | |
| *** david-lyle has joined #openstack-cinder | 03:43 | |
| *** david-lyle has quit IRC | 03:48 | |
| *** haomaiwang has joined #openstack-cinder | 03:52 | |
| *** david-lyle has joined #openstack-cinder | 03:53 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** david-lyle has quit IRC | 04:02 | |
| *** david-lyle has joined #openstack-cinder | 04:06 | |
| *** david-lyle has quit IRC | 04:12 | |
| *** dimsum__ has joined #openstack-cinder | 04:21 | |
| *** david-lyle has joined #openstack-cinder | 04:22 | |
| *** amit213 has quit IRC | 04:28 | |
| *** amit213 has joined #openstack-cinder | 04:28 | |
| *** david-lyle has quit IRC | 04:29 | |
| *** edtubill has joined #openstack-cinder | 04:32 | |
| *** fifieldt has joined #openstack-cinder | 04:46 | |
| *** merooney has joined #openstack-cinder | 04:49 | |
| *** merooeny_ has joined #openstack-cinder | 04:51 | |
| *** merooney has quit IRC | 04:54 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| *** merooeny_ has quit IRC | 05:03 | |
| *** pradipta has joined #openstack-cinder | 05:09 | |
| *** stevemar_ has quit IRC | 05:20 | |
| *** lpetrut has joined #openstack-cinder | 05:23 | |
| *** shausy has joined #openstack-cinder | 05:34 | |
| *** BharatK has joined #openstack-cinder | 05:34 | |
| *** edtubill has quit IRC | 05:34 | |
| *** nkrinner has joined #openstack-cinder | 05:35 | |
| *** et10man has joined #openstack-cinder | 05:46 | |
| *** lpetrut has quit IRC | 05:47 | |
| openstackgerrit | watanabe isao proposed openstack/cinder: Don't merge https://review.openstack.org/230880 | 05:51 |
| *** BharatK has quit IRC | 05:53 | |
| *** lpetrut has joined #openstack-cinder | 05:58 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| *** ankit_ag has joined #openstack-cinder | 06:09 | |
| *** BharatK has joined #openstack-cinder | 06:09 | |
| *** deepakcs has joined #openstack-cinder | 06:11 | |
| *** lpetrut has quit IRC | 06:14 | |
| *** ishant has joined #openstack-cinder | 06:19 | |
| *** ankit_ag has quit IRC | 06:24 | |
| *** deepakcs has quit IRC | 06:26 | |
| *** aarefiev_ has quit IRC | 06:28 | |
| *** aarefiev has joined #openstack-cinder | 06:30 | |
| *** vlaza has joined #openstack-cinder | 06:31 | |
| *** links has joined #openstack-cinder | 06:35 | |
| *** vgridnev has joined #openstack-cinder | 06:37 | |
| *** deepakcs has joined #openstack-cinder | 06:41 | |
| *** alexschm has joined #openstack-cinder | 06:45 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Add option for user to disable fast format at thick-provisioned volume creation in IBM Storwize cinder https://review.openstack.org/230825 | 07:14 |
| openstackgerrit | xiaoqin proposed openstack/cinder: Add option to disable fast format for thick volume creation https://review.openstack.org/230825 | 07:26 |
| openstackgerrit | xiaoqin proposed openstack/cinder: Add option to disable fast format for thick volume creation https://review.openstack.org/230825 | 07:26 |
| *** fifieldt has quit IRC | 07:28 | |
| *** geguileo has joined #openstack-cinder | 07:39 | |
| openstackgerrit | Sergey Gotliv proposed openstack/python-cinderclient: Adding pagination to snapshots and backups lists https://review.openstack.org/225582 | 07:40 |
| *** aix has quit IRC | 07:45 | |
| *** aix has joined #openstack-cinder | 07:45 | |
| *** jordanP has joined #openstack-cinder | 07:48 | |
| openstackgerrit | oliver-leahy-l proposed openstack/cinder: encryption_api_url requires a version https://review.openstack.org/230031 | 07:49 |
| *** et10man has quit IRC | 07:52 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:01 | |
| *** arif-ali has quit IRC | 08:05 | |
| *** arif-ali has joined #openstack-cinder | 08:09 | |
| *** e0ne has joined #openstack-cinder | 08:10 | |
| *** arif-ali has quit IRC | 08:16 | |
| *** arif-ali has joined #openstack-cinder | 08:19 | |
| *** nkrinner_ has joined #openstack-cinder | 08:22 | |
| *** lpetrut has joined #openstack-cinder | 08:22 | |
| *** nkrinner_ has quit IRC | 08:23 | |
| *** david-lyle has joined #openstack-cinder | 08:28 | |
| *** arif-ali has quit IRC | 08:34 | |
| *** jistr has joined #openstack-cinder | 08:38 | |
| *** arif-ali has joined #openstack-cinder | 08:39 | |
| *** arif-ali has quit IRC | 08:47 | |
| *** david-lyle has quit IRC | 08:51 | |
| *** arif-ali has joined #openstack-cinder | 08:51 | |
| *** ankit_ag has joined #openstack-cinder | 08:58 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Add fast format option for thick volume creation https://review.openstack.org/230825 | 08:58 |
| *** jistr has quit IRC | 09:00 | |
| *** david-lyle has joined #openstack-cinder | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:01 | |
| *** jistr has joined #openstack-cinder | 09:01 | |
| *** BharatK has quit IRC | 09:01 | |
| *** markvoelker has joined #openstack-cinder | 09:05 | |
| *** markvoelker_ has joined #openstack-cinder | 09:06 | |
| *** david-lyle has quit IRC | 09:07 | |
| *** abehl has joined #openstack-cinder | 09:08 | |
| *** markvoelker has quit IRC | 09:09 | |
| *** BharatK has joined #openstack-cinder | 09:13 | |
| *** ronis has joined #openstack-cinder | 09:16 | |
| *** vgridnev has quit IRC | 09:19 | |
| *** vgridnev_ has joined #openstack-cinder | 09:19 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: emc vmax driver: use integer division for Python 3 https://review.openstack.org/229998 | 09:20 |
| *** markvoelker_ has quit IRC | 09:20 | |
| *** david-lyle has joined #openstack-cinder | 09:23 | |
| *** david-lyle has quit IRC | 09:27 | |
| *** david-lyle has joined #openstack-cinder | 09:30 | |
| *** jordan_ has joined #openstack-cinder | 09:35 | |
| *** ankit_ag has quit IRC | 09:36 | |
| *** jordanP has quit IRC | 09:39 | |
| *** ociuhandu has joined #openstack-cinder | 09:44 | |
| *** asselin__ has joined #openstack-cinder | 09:44 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Fix capacity report error in Huawei driver https://review.openstack.org/226308 | 09:44 |
| *** asselin_ has quit IRC | 09:48 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Encode username and password in Huawei driver https://review.openstack.org/216085 | 09:52 |
| *** david-lyle has quit IRC | 09:54 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** david-lyle has joined #openstack-cinder | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** e0ne has quit IRC | 10:10 | |
| *** e0ne has joined #openstack-cinder | 10:14 | |
| *** IanGovett has joined #openstack-cinder | 10:14 | |
| *** david-lyle has quit IRC | 10:16 | |
| *** david-lyle has joined #openstack-cinder | 10:19 | |
| *** IanGovett has quit IRC | 10:22 | |
| *** shausy has quit IRC | 10:27 | |
| *** shausy has joined #openstack-cinder | 10:27 | |
| *** david-lyle has quit IRC | 10:29 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 10:29 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/222292 | 10:30 |
| *** david-lyle has joined #openstack-cinder | 10:32 | |
| *** IanGovett has joined #openstack-cinder | 10:41 | |
| *** david-lyle has quit IRC | 10:42 | |
| *** david-lyle has joined #openstack-cinder | 10:49 | |
| *** BharatK has quit IRC | 10:53 | |
| *** david-lyle has quit IRC | 10:53 | |
| *** david-lyle has joined #openstack-cinder | 10:56 | |
| *** smoriya has quit IRC | 10:57 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| *** aix has quit IRC | 11:03 | |
| *** haomaiwang has quit IRC | 11:05 | |
| *** haomaiwa_ has joined #openstack-cinder | 11:05 | |
| *** BharatK has joined #openstack-cinder | 11:06 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Cleanup for cinder tests with CGSnapshot https://review.openstack.org/207866 | 11:07 |
| *** david-lyle has quit IRC | 11:07 | |
| *** david-lyle has joined #openstack-cinder | 11:08 | |
| *** haomaiwa_ has quit IRC | 11:09 | |
| *** jordan_ has quit IRC | 11:14 | |
| *** Yogi1 has joined #openstack-cinder | 11:18 | |
| *** david-lyle has quit IRC | 11:18 | |
| haypo | hello. any progress on the failures of the python 3 gate? | 11:24 |
| *** david-lyle has joined #openstack-cinder | 11:25 | |
| *** timcl has joined #openstack-cinder | 11:25 | |
| *** jordan_ has joined #openstack-cinder | 11:27 | |
| *** aix has joined #openstack-cinder | 11:35 | |
| *** kevincarr1991 has joined #openstack-cinder | 12:00 | |
| *** e0ne has quit IRC | 12:01 | |
| *** bswartz has quit IRC | 12:05 | |
| *** sb has quit IRC | 12:08 | |
| *** strictlyb has joined #openstack-cinder | 12:08 | |
| *** e0ne has joined #openstack-cinder | 12:13 | |
| *** dzamboni has joined #openstack-cinder | 12:14 | |
| e0ne | haypo: hi. afaik, no:( | 12:16 |
| *** raildo-afk is now known as raildo | 12:16 | |
| e0ne | haypo: I'm going to take a look on it again a bit later | 12:16 |
| *** BharatK has quit IRC | 12:18 | |
| *** Yogi1 has quit IRC | 12:19 | |
| *** bill_az has joined #openstack-cinder | 12:22 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Convert Retry-After header parameter value to string https://review.openstack.org/230977 | 12:25 |
| *** martyturner has joined #openstack-cinder | 12:27 | |
| *** bill_az has quit IRC | 12:28 | |
| *** ishant has quit IRC | 12:29 | |
| *** diablo_rojo has joined #openstack-cinder | 12:32 | |
| haypo | e0ne: any clue? last time i checked, i was still unable to reproduce, even with "tox -e py34" on a fresh clone | 12:33 |
| haypo | e0ne: should i have to run an external service like a MySQL server, RabbitMQ server or something else? | 12:33 |
| e0ne | haypo: I'm checking it with a frech tox env | 12:33 |
| *** lpetrut has quit IRC | 12:35 | |
| kevincarr1991 | Good Morning! I have cinder installed on my controller node and I have cinder using the NFS drivers. I am able to create volumes from Horizon, but when I do they have improper permissions which restricts me from being able to attach the volumes to instances. | 12:37 |
| kevincarr1991 | Has anyone has this issue or something like it before? | 12:37 |
| *** edmondsw has joined #openstack-cinder | 12:37 | |
| *** dave-mccowan has joined #openstack-cinder | 12:38 | |
| *** bswartz has joined #openstack-cinder | 12:40 | |
| *** gouthamr has joined #openstack-cinder | 12:43 | |
| *** Lee1092 has quit IRC | 12:43 | |
| *** takedakn has joined #openstack-cinder | 12:45 | |
| *** porrua has joined #openstack-cinder | 12:48 | |
| e0ne | haypo: what is your's concurrency param value in test run? | 12:50 |
| e0ne | haypo: "By default ostestr will set concurrency to ncpu, to specify something else use the concurrency=<n> option call ie: 'tox -epy27 -- --concurrency=4'" | 12:50 |
| *** dustins has joined #openstack-cinder | 12:52 | |
| *** kevincarr1991 has quit IRC | 12:53 | |
| *** marcusvrn_ has joined #openstack-cinder | 12:53 | |
| *** akerr has joined #openstack-cinder | 12:53 | |
| *** julim has joined #openstack-cinder | 12:56 | |
| *** Yogi1 has joined #openstack-cinder | 12:57 | |
| *** akerr has quit IRC | 12:57 | |
| *** akerr has joined #openstack-cinder | 12:58 | |
| *** takedakn has quit IRC | 13:03 | |
| e0ne | haypo: I've got something strange on my env. cinder.tests.unit.test_cmd failed even with python27 | 13:03 |
| *** dimsum__ is now known as dims | 13:08 | |
| *** julim_ has joined #openstack-cinder | 13:09 | |
| *** jaypipes has joined #openstack-cinder | 13:09 | |
| *** julim has quit IRC | 13:12 | |
| diablo_rojo | smcginnis: DuncanT hemna_ When you all get a sec, would you re review https://review.openstack.org/#/c/223375/ Eric has voiced some concerns with the approach and I am curious to know if you agree or if you think this is an acceptable approach. Thanks! | 13:14 |
| *** willsama has joined #openstack-cinder | 13:19 | |
| *** changbl has quit IRC | 13:22 | |
| *** links has quit IRC | 13:22 | |
| *** bill_az has joined #openstack-cinder | 13:23 | |
| *** chlong has joined #openstack-cinder | 13:26 | |
| *** lpetrut has joined #openstack-cinder | 13:27 | |
| *** erhudy has joined #openstack-cinder | 13:32 | |
| *** jasondotstar|afk is now known as jasondotstar | 13:32 | |
| *** superdan is now known as dansmith | 13:34 | |
| tbarron | kevincarr1991: permissions on NFS exported filesystems are a frequent issue, quite apart form cinder and nova. | 13:36 |
| tbarron | kevincarr1991: you need to check how you are doing exports, mounts, whether root is squashed, etc. | 13:37 |
| *** bill_az has quit IRC | 13:37 | |
| tbarron | kevincarr1991: as I've suggested when you asked about other issues with NFS, determine the specific commands being run | 13:38 |
| *** diogogmt has quit IRC | 13:39 | |
| tbarron | kevincarr1991: by looking at logs (in this case nova cpu log). Run them by hand while logged in as the nova user, after unmounting,, re-exporting from the NFS server, remounting, etd. | 13:39 |
| tbarron | etc. | 13:39 |
| tbarron | kevincarr1991: cinder and nova do not control how your NFS filesystem is exported, and that (plus any nfs mount options that you supply) | 13:40 |
| tbarron | kevincarr1991: is what determines permissions for the cinder user or nova user when they interact with that filesystem. | 13:41 |
| *** deepakcs has quit IRC | 13:41 | |
| *** jungleboyj has joined #openstack-cinder | 13:41 | |
| *** xyang1 has joined #openstack-cinder | 13:45 | |
| *** mc_nair has joined #openstack-cinder | 13:45 | |
| *** vgridnev_ has quit IRC | 13:52 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Detach volume on device validation failure https://review.openstack.org/230429 | 13:52 |
| *** haomaiwang has joined #openstack-cinder | 13:54 | |
| *** bswartz has quit IRC | 13:55 | |
| *** openstackstatus has joined #openstack-cinder | 13:56 | |
| *** ChanServ sets mode: +v openstackstatus | 13:56 | |
| *** mriedem has joined #openstack-cinder | 13:56 | |
| *** diablo_rojo has quit IRC | 13:57 | |
| *** diablo_rojo has joined #openstack-cinder | 13:58 | |
| *** jungleboyj has quit IRC | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** dave-mccowan has quit IRC | 14:01 | |
| *** stevemar_ has joined #openstack-cinder | 14:01 | |
| *** links has joined #openstack-cinder | 14:05 | |
| xyang1 | smcginnis: ping | 14:05 |
| *** haomaiwa_ has joined #openstack-cinder | 14:06 | |
| smcginnis | xyang1: Hey! | 14:07 |
| xyang1 | smcginnis: I was trying to backport, but got some issues | 14:08 |
| xyang1 | smcginnis: did you get my email | 14:08 |
| smcginnis | xyang1: Just looking at it now. | 14:08 |
| *** xyang has joined #openstack-cinder | 14:08 | |
| smcginnis | xyang1: Not sure why that would be. | 14:08 |
| *** xyang has quit IRC | 14:08 | |
| *** shausy has quit IRC | 14:09 | |
| xyang1 | smcginnis: very odd, the patch looks clean | 14:09 |
| smcginnis | xyang1: Yeah, I thought it looked ok. Hmm... | 14:09 |
| xyang1 | smcginnis: but the commit id does not bring you to the original | 14:09 |
| xyang1 | smcginnis: i wonder if I can just remove wip | 14:10 |
| xyang1 | smcginnis: should I manually change the commit msg | 14:10 |
| *** changbl has joined #openstack-cinder | 14:10 | |
| smcginnis | xyang1: I suppose. Unless someone else here has more experience eith cherry pick issues, that might have to be what to do. | 14:11 |
| smcginnis | xyang1: I know jungleboyj has had to do quite a few. Maybe wait until he's online and check with him. | 14:11 |
| xyang1 | smcginnis: ok, I'll update the commit msg then | 14:11 |
| *** thangp has joined #openstack-cinder | 14:11 | |
| smcginnis | Unless someone else chimes in. | 14:11 |
| xyang1 | smcginnis: sure | 14:11 |
| smcginnis | xyang1: We've got a little time yet. | 14:12 |
| xyang1 | smcginnis: when do you have to make the cut | 14:12 |
| *** jgregor has joined #openstack-cinder | 14:13 | |
| smcginnis | xyang1: I think Wednesday at the latest. | 14:14 |
| *** dave-mccowan has joined #openstack-cinder | 14:14 | |
| *** mdenny has joined #openstack-cinder | 14:14 | |
| xyang1 | smcginnis: ok, thanks | 14:14 |
| smcginnis | xyang1: Do you know - are there any string changes in your patch? Translations could be an issue. | 14:14 |
| *** jungleboyj has joined #openstack-cinder | 14:15 | |
| xyang1 | smcginnis: good point, let me check | 14:15 |
| xyang1 | smcginnis: there is new warn msg | 14:16 |
| *** raildo is now known as raildo-afk | 14:16 | |
| smcginnis | xyang1: I think that's fine. | 14:16 |
| smcginnis | xyang1: But that probably means we should get it in sooner rather than later to let the translators have some time to translate it. | 14:16 |
| *** nkrinner has quit IRC | 14:17 | |
| *** Lee1092 has joined #openstack-cinder | 14:17 | |
| xyang1 | smcginnis: i can change to debug if necessary | 14:17 |
| smcginnis | xyang1: OK. Let's wait a bit and see. | 14:19 |
| *** lcurtis has joined #openstack-cinder | 14:19 | |
| xyang1 | smcginnis: sure. let me add some comments there so people know what issue I ran into | 14:19 |
| smcginnis | xyang1: Good plan. | 14:19 |
| *** merooney has joined #openstack-cinder | 14:20 | |
| *** salv-orl_ has joined #openstack-cinder | 14:22 | |
| *** eharney has joined #openstack-cinder | 14:23 | |
| *** salv-orlando has quit IRC | 14:24 | |
| *** raildo-afk is now known as raildo | 14:25 | |
| *** lcurtis has quit IRC | 14:25 | |
| *** lcurtis has joined #openstack-cinder | 14:26 | |
| *** mtanino has joined #openstack-cinder | 14:26 | |
| *** jgregor has quit IRC | 14:29 | |
| xyang1 | smcginnis: I added comments there to explain the problem: https://review.openstack.org/#/c/230861/ | 14:30 |
| smcginnis | xyang1: +1 | 14:31 |
| xyang1 | smcginnis: thanks | 14:31 |
| *** vivekd has joined #openstack-cinder | 14:32 | |
| *** vlaza has left #openstack-cinder | 14:35 | |
| *** changbl has quit IRC | 14:36 | |
| *** chlong has quit IRC | 14:38 | |
| *** garthb has joined #openstack-cinder | 14:38 | |
| *** CBADER has joined #openstack-cinder | 14:39 | |
| *** diogogmt has joined #openstack-cinder | 14:42 | |
| smcginnis | xyang1: Looks like eharney knows what's going on (as usual). :) | 14:46 |
| *** lpetrut has quit IRC | 14:46 | |
| xyang1 | smcginnis: let me see what he says:) | 14:46 |
| xyang1 | eharney: thanks! Let me try that | 14:47 |
| *** baumann has joined #openstack-cinder | 14:49 | |
| *** edtubill has joined #openstack-cinder | 14:50 | |
| *** tsekiyama has joined #openstack-cinder | 14:50 | |
| *** jgregor has joined #openstack-cinder | 14:51 | |
| *** EmilienM has quit IRC | 14:55 | |
| *** EmilienM has joined #openstack-cinder | 14:55 | |
| *** ntpttr has joined #openstack-cinder | 14:55 | |
| *** david-ly_ has joined #openstack-cinder | 14:56 | |
| *** david-lyle has quit IRC | 14:57 | |
| *** stevemar_ has quit IRC | 14:57 | |
| *** david-ly_ is now known as david-lyle | 14:57 | |
| *** stevemar_ has joined #openstack-cinder | 14:57 | |
| *** haomaiwa_ has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** stevemar_ has quit IRC | 15:02 | |
| *** aix has quit IRC | 15:03 | |
| *** david-ly_ has joined #openstack-cinder | 15:03 | |
| *** david-lyle has quit IRC | 15:03 | |
| *** harlowja_at_home has joined #openstack-cinder | 15:04 | |
| *** david-ly_ is now known as david-lyle | 15:04 | |
| *** vivekd has quit IRC | 15:05 | |
| *** asselin_ has joined #openstack-cinder | 15:09 | |
| *** jungleboyj has quit IRC | 15:09 | |
| *** setmason has joined #openstack-cinder | 15:10 | |
| *** asselin__ has quit IRC | 15:12 | |
| *** diogogmt_ has joined #openstack-cinder | 15:13 | |
| *** shausy has joined #openstack-cinder | 15:13 | |
| *** diogogmt has quit IRC | 15:13 | |
| *** diogogmt_ is now known as diogogmt | 15:13 | |
| *** hemna_ is now known as hemna | 15:17 | |
| *** stevemar_ has joined #openstack-cinder | 15:18 | |
| *** thingee has joined #openstack-cinder | 15:19 | |
| *** alejandrito has joined #openstack-cinder | 15:23 | |
| hemna | diablo_rojo, so I'm fine with the current approach. I think adding a cinder.utils.register wrapper is probably not something we want to do. It could lead to other issues | 15:25 |
| hemna | diablo_rojo, I do think Gorka has some good points in his review that need addressing fwiw. | 15:25 |
| *** mc_nair has quit IRC | 15:28 | |
| *** lcurtis has quit IRC | 15:29 | |
| *** jungleboyj has joined #openstack-cinder | 15:29 | |
| *** merooney has quit IRC | 15:29 | |
| diablo_rojo | hemna: Yeah I just saw Gorka's comments and should have a new patch up as soon as pep8 passes :) | 15:30 |
| diablo_rojo | hemna: Thanks for looking at it. I will give you a heads up when I get the new patch up. | 15:30 |
| hemna | ok coolio | 15:30 |
| *** jdurgin1 has joined #openstack-cinder | 15:30 | |
| *** sparr has quit IRC | 15:32 | |
| *** tbarron has quit IRC | 15:32 | |
| *** dsariel_ has joined #openstack-cinder | 15:32 | |
| *** asselin__ has joined #openstack-cinder | 15:32 | |
| *** tbarron has joined #openstack-cinder | 15:33 | |
| *** sparr has joined #openstack-cinder | 15:34 | |
| *** changbl has joined #openstack-cinder | 15:35 | |
| *** asselin_ has quit IRC | 15:36 | |
| *** Guest43458 is now known as med_ | 15:36 | |
| *** med_ has quit IRC | 15:36 | |
| *** med_ has joined #openstack-cinder | 15:36 | |
| *** dustins is now known as dustins|afk | 15:36 | |
| *** med_ is now known as med | 15:37 | |
| *** med is now known as med_ | 15:38 | |
| *** dims has quit IRC | 15:40 | |
| *** Yogi1 has quit IRC | 15:40 | |
| *** mc_nair has joined #openstack-cinder | 15:42 | |
| *** dims has joined #openstack-cinder | 15:47 | |
| *** dims has quit IRC | 15:47 | |
| *** Guest96374 has joined #openstack-cinder | 15:47 | |
| *** Guest96374 has quit IRC | 15:48 | |
| *** david-ly_ has joined #openstack-cinder | 15:48 | |
| *** david-lyle has quit IRC | 15:48 | |
| *** dims_ has joined #openstack-cinder | 15:48 | |
| *** david-ly_ is now known as david-lyle | 15:49 | |
| *** mriedem has quit IRC | 15:50 | |
| *** haomaiwang has quit IRC | 15:50 | |
| *** haomaiwang has joined #openstack-cinder | 15:51 | |
| *** alexschm has quit IRC | 15:52 | |
| *** alejandrito has quit IRC | 15:55 | |
| *** shausy has quit IRC | 15:59 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Hacking check for opt name registration https://review.openstack.org/223375 | 16:03 |
| *** hemna_ has joined #openstack-cinder | 16:04 | |
| *** haypo has left #openstack-cinder | 16:04 | |
| diablo_rojo | hemna geguileo ^^ | 16:04 |
| geguileo | diablo_rojo: Ok, reviewing | 16:05 |
| diablo_rojo | geguileo: Thank you! | 16:06 |
| *** CBADER has quit IRC | 16:07 | |
| *** cbader has joined #openstack-cinder | 16:07 | |
| *** akerr is now known as akerr_away | 16:07 | |
| *** raildo is now known as raildo-afk | 16:10 | |
| *** baumann1 has joined #openstack-cinder | 16:11 | |
| *** bnemec has joined #openstack-cinder | 16:11 | |
| *** jdurgin1 has quit IRC | 16:12 | |
| *** baumann has quit IRC | 16:12 | |
| *** raildo-afk is now known as raildo | 16:12 | |
| *** chris_morrell has joined #openstack-cinder | 16:13 | |
| *** mriedem has joined #openstack-cinder | 16:13 | |
| *** _cjones_ has joined #openstack-cinder | 16:14 | |
| Swanson | Anyone know when (if) there is going to be a kilo bug fix release? | 16:14 |
| *** alejandrito has joined #openstack-cinder | 16:16 | |
| *** hemna_ has quit IRC | 16:16 | |
| *** stevemar_ has quit IRC | 16:17 | |
| *** leeantho has joined #openstack-cinder | 16:17 | |
| *** angela-s has joined #openstack-cinder | 16:17 | |
| *** stevemar_ has joined #openstack-cinder | 16:18 | |
| *** pradipta has quit IRC | 16:22 | |
| *** jistr has quit IRC | 16:26 | |
| *** e0ne has quit IRC | 16:28 | |
| *** dustins|afk has quit IRC | 16:28 | |
| *** timcl has quit IRC | 16:31 | |
| *** harlowja_at_home has quit IRC | 16:33 | |
| *** alejandrito has quit IRC | 16:37 | |
| *** raildo is now known as raildo-afk | 16:38 | |
| *** raildo-afk is now known as raildo | 16:41 | |
| *** salv-orl_ has quit IRC | 16:42 | |
| *** apoorvad has joined #openstack-cinder | 16:44 | |
| *** haomaiw__ has joined #openstack-cinder | 16:45 | |
| *** haomaiwang has quit IRC | 16:46 | |
| *** crose has joined #openstack-cinder | 16:53 | |
| *** baumann1 has quit IRC | 17:00 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Fix broken format string in vol mgr log https://review.openstack.org/231093 | 17:00 |
| *** haomaiw__ has quit IRC | 17:01 | |
| *** 32NAAH197 has joined #openstack-cinder | 17:01 | |
| hemna | lots of CI failures | 17:02 |
| hemna | bleh | 17:02 |
| smcginnis | hemna: I've still been get some residual stacking failures. | 17:05 |
| smcginnis | Mostly shaken out, but some occasional ones yet. | 17:05 |
| hemna | smcginnis, I was just looking at this guy https://review.openstack.org/#/c/223375 | 17:05 |
| hemna | and seeing the initial list of failures right away | 17:05 |
| smcginnis | Dang | 17:05 |
| hemna | haven't looked into them to see if the patch itself is causing it or not | 17:05 |
| hemna | just seems odd | 17:06 |
| *** Yogi1 has joined #openstack-cinder | 17:07 | |
| *** merooney has joined #openstack-cinder | 17:07 | |
| *** subscope has joined #openstack-cinder | 17:09 | |
| *** bswartz has joined #openstack-cinder | 17:10 | |
| *** timcl has joined #openstack-cinder | 17:10 | |
| *** aix has joined #openstack-cinder | 17:11 | |
| *** lcurtis has joined #openstack-cinder | 17:13 | |
| *** tsekiyama has quit IRC | 17:13 | |
| *** akerr_away is now known as akerr | 17:15 | |
| *** e0ne has joined #openstack-cinder | 17:17 | |
| *** markstur has joined #openstack-cinder | 17:17 | |
| *** raildo is now known as raildo-afk | 17:18 | |
| *** stevemar_ has quit IRC | 17:21 | |
| *** kjelly has quit IRC | 17:24 | |
| *** kjelly has joined #openstack-cinder | 17:25 | |
| *** earlephilhower has joined #openstack-cinder | 17:28 | |
| *** raildo-afk is now known as raildo | 17:30 | |
| *** ociuhandu has quit IRC | 17:30 | |
| *** openstackgerrit has quit IRC | 17:31 | |
| *** openstackgerrit has joined #openstack-cinder | 17:32 | |
| *** alejandrito has joined #openstack-cinder | 17:34 | |
| *** stevemar_ has joined #openstack-cinder | 17:36 | |
| *** alejandrito has quit IRC | 17:38 | |
| *** salv-orlando has joined #openstack-cinder | 17:42 | |
| *** 32NAAH197 has quit IRC | 17:44 | |
| *** salv-orl_ has joined #openstack-cinder | 17:45 | |
| *** ociuhandu has joined #openstack-cinder | 17:46 | |
| xyang1 | smcginnis: eharney jungleboyj : I re-submitted the backport patch: https://review.openstack.org/#/c/231111/ | 17:46 |
| *** salv-orlando has quit IRC | 17:46 | |
| *** raildo is now known as raildo-afk | 17:48 | |
| *** timcl has quit IRC | 17:54 | |
| *** crose has quit IRC | 17:57 | |
| *** ccesario_ has quit IRC | 17:58 | |
| smcginnis | xyang1: Great, thanks. Just waiting for CI. | 17:58 |
| xyang1 | smcginnis: thanks! We are only running CI against the master though | 17:59 |
| smcginnis | xyang1: Oh yeah, I just meant Jenkins check. | 18:00 |
| xyang1 | smcginnis: Ok, yes | 18:00 |
| smcginnis | :) | 18:00 |
| *** openstackgerrit has quit IRC | 18:01 | |
| *** openstackgerrit has joined #openstack-cinder | 18:02 | |
| *** dsariel_ has quit IRC | 18:02 | |
| *** apoorvad_ has joined #openstack-cinder | 18:02 | |
| *** timcl has joined #openstack-cinder | 18:03 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Do not merge: Debugging bug 1501745 https://review.openstack.org/231122 | 18:03 |
| openstack | bug 1501745 in Cinder "gate-cinder-python34 failed" [High,Confirmed] https://launchpad.net/bugs/1501745 - Assigned to Eric Harney (eharney) | 18:03 |
| *** jaypipes has quit IRC | 18:03 | |
| *** apoorvad has quit IRC | 18:05 | |
| ntpttr | Hey everyone, I'm pretty fresh at contributing to Cinder, and I'm wondering if there are any resources people would recommend for learning the architecture of the project and how to get to know my way around the code. Is the wiki the best place? Is it good to just look around the github repo for a while feeling your way around? Thanks | 18:06 |
| *** merooney has quit IRC | 18:12 | |
| *** aix has quit IRC | 18:14 | |
| *** raildo-afk is now known as raildo | 18:14 | |
| *** tsekiyama has joined #openstack-cinder | 18:14 | |
| *** dustins has joined #openstack-cinder | 18:19 | |
| *** salv-orl_ has quit IRC | 18:25 | |
| scottda | ntpttr: One way to learn is to follow what's going on here, in cinder-irc... | 18:26 |
| scottda | click on some of the links ^^^^ and read the code. | 18:26 |
| scottda | If someone is discussing something you don't understand, grep the code, look in google, or ask here in IRC | 18:26 |
| ntpttr | scottda: Sounds good, thanks you | 18:27 |
| *** dims_ has quit IRC | 18:27 | |
| scottda | sure. Welcome to Cinder | 18:27 |
| ntpttr | scottda: Thanks :) | 18:28 |
| scottda | We've plenty of bugs you could look at: https://bugs.launchpad.net/cinder/+bugs?orderby=status&start=0 | 18:29 |
| *** abehl has quit IRC | 18:31 | |
| *** salv-orlando has joined #openstack-cinder | 18:32 | |
| *** IanGovett has quit IRC | 18:32 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix iSCSI multipath cleanup https://review.openstack.org/231134 | 18:35 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix FC multipath cleanup https://review.openstack.org/231135 | 18:36 |
| *** IanGovett has joined #openstack-cinder | 18:38 | |
| *** links has quit IRC | 18:44 | |
| *** raildo is now known as raildo-afk | 18:45 | |
| *** raildo-afk is now known as raildo | 18:45 | |
| *** diegows has joined #openstack-cinder | 18:48 | |
| diablo_rojo | geguileo: I see you just +2ed my patch which I greatly appreciate. I had just gotten done addressing your replies and was going to push a new patch if you'd like. Otherwise we can just leave it. | 18:49 |
| diablo_rojo | geguileo: Thoughts? | 18:49 |
| geguileo | diablo_rojo: If you've already done the work I'd hate it go to waste | 18:50 |
| *** stevemar_ has quit IRC | 18:50 | |
| geguileo | diablo_rojo: Just push it and I'll review it before closing for shop for today | 18:50 |
| *** stevemar_ has joined #openstack-cinder | 18:50 | |
| diablo_rojo | geguileo: Ha okay I will push it now. One sec. | 18:51 |
| geguileo | diablo_rojo: No rush | 18:51 |
| *** merooney has joined #openstack-cinder | 18:51 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Hacking check for opt name registration https://review.openstack.org/223375 | 18:52 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Remove hardcoded releases list from unit tests https://review.openstack.org/218226 | 18:54 |
| *** stevemar_ has quit IRC | 18:55 | |
| *** kevincarr1991 has joined #openstack-cinder | 18:56 | |
| *** dims_ has joined #openstack-cinder | 18:58 | |
| *** dims_ has quit IRC | 18:58 | |
| *** dims_ has joined #openstack-cinder | 18:59 | |
| *** apoorvad_ has quit IRC | 19:01 | |
| *** apoorvad has joined #openstack-cinder | 19:01 | |
| *** dzamboni has quit IRC | 19:03 | |
| *** dims_ has quit IRC | 19:03 | |
| *** akerr has quit IRC | 19:05 | |
| *** e0ne has quit IRC | 19:06 | |
| *** pv_ has joined #openstack-cinder | 19:10 | |
| pv_ | hey quick question other than run_tests.sh script is there another way to run unit test? i wanna test the output of my unit tests but when i do ./run_tests.sh my_test_file it runs the unit tests for emc_vmax? and errors out | 19:11 |
| lifeless | pv_: tox | 19:12 |
| *** wolsen_ is now known as wolsen | 19:13 | |
| pv_ | ooooo for sure thanks | 19:13 |
| pv_ | i just looked it up http://docs.openstack.org/developer/cinder/devref/unit_tests.html lol shoulda checked there first instead of asking here | 19:13 |
| pv_ | thanks | 19:13 |
| *** Lee1092 has quit IRC | 19:13 | |
| *** kevincar_ has joined #openstack-cinder | 19:15 | |
| smcginnis | pv_: We are considering removing run_tests.sh partly to avoid this confusion of multiple ways to do this. | 19:16 |
| *** stevemar_ has joined #openstack-cinder | 19:16 | |
| *** kevincarr1991 has quit IRC | 19:18 | |
| *** dsariel_ has joined #openstack-cinder | 19:20 | |
| *** thingee has quit IRC | 19:27 | |
| *** thingee has joined #openstack-cinder | 19:28 | |
| pv_ | oh yeah i see that now on the page for unit tests thanks | 19:28 |
| *** akerr has joined #openstack-cinder | 19:33 | |
| *** kevincar_ has quit IRC | 19:35 | |
| *** akerr_ has joined #openstack-cinder | 19:38 | |
| *** kevincarr1991 has joined #openstack-cinder | 19:40 | |
| *** rhe00 has quit IRC | 19:40 | |
| *** akerr has quit IRC | 19:41 | |
| kevincarr1991 | I have setup cinder on my controller node and use a nfs share for the volumes. I am able to create volumes using horizon but I am unable to attach the volumes to instances using horizon | 19:44 |
| *** ronis has quit IRC | 19:50 | |
| *** thingee has quit IRC | 19:50 | |
| *** dims_ has joined #openstack-cinder | 19:53 | |
| *** thingee has joined #openstack-cinder | 19:56 | |
| *** thingee has quit IRC | 19:56 | |
| *** thingee has joined #openstack-cinder | 19:56 | |
| *** timcl has quit IRC | 19:57 | |
| *** dims_ has quit IRC | 19:57 | |
| *** apoorvad_ has joined #openstack-cinder | 20:00 | |
| *** DericHorn-HP has joined #openstack-cinder | 20:03 | |
| *** apoorvad has quit IRC | 20:03 | |
| *** merooney has quit IRC | 20:04 | |
| *** markvoelker has joined #openstack-cinder | 20:12 | |
| *** raildo is now known as raildo-afk | 20:13 | |
| *** e0ne has joined #openstack-cinder | 20:15 | |
| *** dims_ has joined #openstack-cinder | 20:16 | |
| *** dims_ has quit IRC | 20:16 | |
| *** dims_ has joined #openstack-cinder | 20:16 | |
| *** raildo-afk is now known as raildo | 20:18 | |
| *** lcurtis_ has joined #openstack-cinder | 20:19 | |
| *** raildo is now known as raildo-afk | 20:23 | |
| *** thangp has quit IRC | 20:24 | |
| *** jordan_ has quit IRC | 20:25 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Remove hardcoded releases list from unit tests https://review.openstack.org/218226 | 20:31 |
| *** diegows has quit IRC | 20:35 | |
| *** anshul has joined #openstack-cinder | 20:37 | |
| *** erhudy has quit IRC | 20:39 | |
| *** dustins has quit IRC | 20:42 | |
| *** bswartz has quit IRC | 20:48 | |
| *** Yogi1 has quit IRC | 20:48 | |
| *** aix has joined #openstack-cinder | 20:49 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: This patch adds "over subscription in thin provisioning" support to Dell Equallogic driver. The following changes are proposed: * Dell eqlx driver reports the following new capacities and ratios: * provisioned_capacity * max_over_subscription_ratio https://review.openstack.org/231186 | 21:02 |
| openstackgerrit | Rajini Ram proposed openstack/cinder: This patch adds "over subscription in thin provisioning" support to Dell Equallogic driver. The following changes are proposed: * Dell eqlx driver reports the following new capacities and ratios: * provisioned_capacity * max_over_subscription_ratio https://review.openstack.org/231186 | 21:03 |
| *** lcurtis_ has quit IRC | 21:04 | |
| *** baumann has joined #openstack-cinder | 21:05 | |
| *** baumann has left #openstack-cinder | 21:06 | |
| *** zigo has quit IRC | 21:09 | |
| *** zigo has joined #openstack-cinder | 21:10 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Dell Eqlx: Support over subscription in thin provisioning https://review.openstack.org/231186 | 21:12 |
| diablo_rojo | geg | 21:14 |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Hacking check for opt name registration https://review.openstack.org/223375 | 21:15 |
| *** apoorvad_ has quit IRC | 21:16 | |
| *** apoorvad has joined #openstack-cinder | 21:17 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Dell Eqlx: Support over subscription in thin provisioning https://review.openstack.org/231186 | 21:17 |
| *** martyturner has quit IRC | 21:21 | |
| *** diablo_rojo has quit IRC | 21:22 | |
| *** thingee has quit IRC | 21:22 | |
| *** ntpttr has quit IRC | 21:25 | |
| *** mriedem is now known as mriedem_away | 21:26 | |
| *** kevincarr1991 has quit IRC | 21:26 | |
| *** kevincarr1991 has joined #openstack-cinder | 21:27 | |
| *** asselin__ has quit IRC | 21:30 | |
| *** dave-mccowan has quit IRC | 21:31 | |
| *** eharney has quit IRC | 21:31 | |
| *** subscope has quit IRC | 21:33 | |
| *** kevincarr1991 has quit IRC | 21:33 | |
| *** kevincarr1991 has joined #openstack-cinder | 21:34 | |
| Swanson | jgriffith, patrickeast: what exactly is expected to happen with replication_failover? | 21:37 |
| *** kevincarr1991 has quit IRC | 21:38 | |
| patrickeast | Swanson: it should switch over to the replicated volume for the one being failed over | 21:38 |
| patrickeast | Swanson: so like for Pure arrays we start re-routing management commands for that volume to the secondary array (if possible), in theory subsequent attaches or whatever then are using the other array with the replicated volume | 21:40 |
| *** jgregor has quit IRC | 21:40 | |
| Swanson | patrickeast: At the server level? I should connect the former replication target to any servers that were attached to the source volume? | 21:40 |
| patrickeast | Swanson: heh yea, so i'm not sure exactly how that is supposed to work for the initiators | 21:41 |
| Swanson | patrickeast: Or just make the target the primary and let nova attach? | 21:41 |
| patrickeast | Swanson: that was my interpretation of it | 21:42 |
| Swanson | patrickeast: That simplifies things. | 21:42 |
| patrickeast | jgriffith: ^ does that line up with how you envisioned the failover to go? | 21:44 |
| *** edmondsw has quit IRC | 21:44 | |
| Swanson | patrickeast, jgriffith: a concern would be that the repl destination (at least for Dell) is actually another volume with the same data but not the same id. That's fine so long as initialize_connection and export are called on the new volume. | 21:46 |
| *** julim_ has quit IRC | 21:46 | |
| Swanson | patrickeast, jgriffith: And that would still be an issue even if the id were the same as the repl destination would have different iscsi ports and whatnot. | 21:46 |
| patrickeast | Swanson: yea, i would imagine in your drivers failover_replication (or replication_failover... whatever we called it) you swap the provider_id or something so you can reference the new volume | 21:47 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 21:47 |
| patrickeast | Swanson: yea same here, the connection info is different for it after the failover | 21:47 |
| patrickeast | Swanson: so at least for Pure (and sounds like Dell) it requires being reattached through the normal flow | 21:47 |
| Swanson | patrickeast: Okay. This is making some sense. A little bit magical but it does make sense. | 21:49 |
| *** jwang_ has joined #openstack-cinder | 21:50 | |
| Swanson | patrickeast: Is replication_failover the only way to kick off a failover? Or is there some expectation the driver should do it? Cause if the driver just up and does it I'm assuming we need to poke things above us somehow. | 21:51 |
| *** e0ne has quit IRC | 21:51 | |
| Swanson | jgriffith: ^ | 21:52 |
| patrickeast | Swanson: I think the API is the only way it gets kicked off, at one point there was a cool idea of doing one if the backend went down and stopped responding but that got dropped from the initial v2 implementation | 21:52 |
| hemna | patrickeast, it depends on if you are implementing 'managed' or 'unmanaged' | 21:54 |
| *** e0ne has joined #openstack-cinder | 21:54 | |
| patrickeast | hemna: which part? | 21:54 |
| hemna | what to do inside of failover_replication | 21:54 |
| patrickeast | ahh yea | 21:55 |
| patrickeast | well | 21:55 |
| patrickeast | kinda | 21:55 |
| patrickeast | depends on the way the backend does replication i suppose | 21:55 |
| hemna | for managed, you basically do whatever you need to do on the array, and then return a model update. | 21:55 |
| Swanson | I went straight to unmanaged. | 21:55 |
| hemna | switching the host to the new driver host instance | 21:55 |
| hemna | for unmanaged, it's more complicated | 21:55 |
| patrickeast | for us it works pretty much the same way whether its managed or unmanaged | 21:56 |
| hemna | we are doing 2 separate patches that basically first implement managed, and then a follow up (dep) for unmanaged | 21:56 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 21:56 |
| patrickeast | ^ good timing :D | 21:56 |
| hemna | patrickeast, :) | 21:56 |
| *** edtubill has quit IRC | 21:57 | |
| apoorvad | kevincarr1991: did you try with command line? Whats the error? | 21:58 |
| Swanson | hemna, so if I'm doing managed I just do whatever I need to do to break the replication and return a model update with a new volume_type/whatever... | 21:58 |
| *** apoorvad_ has joined #openstack-cinder | 21:59 | |
| *** harlowja has quit IRC | 21:59 | |
| *** abhi has joined #openstack-cinder | 21:59 | |
| hemna | Swanson, yup, basically | 22:00 |
| hemna | Swanson, if you look in the solidfire driver, that's what they are doing. | 22:00 |
| patrickeast | Swanson: so for managed you need to update the host field in the model update along with the other stuff so that after the failover it is the other c-vol backend handling the request | 22:00 |
| patrickeast | Swanson: unmanaged is more like the same driver managing both targets | 22:00 |
| hemna | patrickeast, I still had a few issues that I wanted to chat with you about | 22:00 |
| hemna | but it's not a huge hurry | 22:00 |
| *** setmason has quit IRC | 22:01 | |
| hemna | but I think the list replication targets needs to be standardized in it's response from the drivers | 22:01 |
| *** e0ne has quit IRC | 22:01 | |
| patrickeast | hemna: yea it seems like for consumption by someone calling the API we should have a standard format | 22:01 |
| *** stevemar_ has quit IRC | 22:02 | |
| hemna | I was thinking at least having a standard key for the target_name | 22:02 |
| *** setmason has joined #openstack-cinder | 22:02 | |
| hemna | so when the cinderclient lists the available targets, the thing that folks copy/paste for failover is the same key | 22:02 |
| *** stevemar_ has joined #openstack-cinder | 22:02 | |
| *** apoorvad has quit IRC | 22:02 | |
| hemna | or we'll end up with different things for a target between solidfire, 3par, pure, etc | 22:02 |
| patrickeast | ah yea that would be great | 22:02 |
| hemna | that would suxors for users. | 22:03 |
| patrickeast | but maybe i wanna be a beautiful snowflake!! | 22:03 |
| hemna | :) | 22:03 |
| jgriffith | hemna: patrickeast that sort of defeats the whole purpose if I follow what you two are saying | 22:03 |
| hemna | we are even allowing mappings from primary pool name, to secondary pool name, for where to replication the volumes. | 22:03 |
| patrickeast | jgriffith: for the standarized keys in the response? | 22:04 |
| hemna | well,just 1 standard key really | 22:04 |
| hemna | and the value can be anything | 22:04 |
| jgriffith | You're talking about the admin show targets cmd? | 22:04 |
| patrickeast | jgriffith: it would make it easier to figure out what field you need to use for calling the failover and specifying a target, right? | 22:04 |
| hemna | patrickeast, exactly | 22:04 |
| patrickeast | yea so hemna is right, just the one key | 22:04 |
| patrickeast | the rest can be free-form | 22:04 |
| jgriffith | patrickeast: sure, so long as everybody can use the "same" thing :) | 22:04 |
| patrickeast | haha yea | 22:05 |
| hemna | jgriffith, yah, that's the idea.....hopefully. | 22:05 |
| jgriffith | patrickeast: I thought that's how I set it already... maybe not | 22:05 |
| hemna | target_name | 22:05 |
| *** jungleboyj has quit IRC | 22:05 | |
| patrickeast | jgriffith: easy, we add a functional test and validate the return field ;) | 22:05 |
| *** stevemar_ has quit IRC | 22:05 | |
| *** stevemar_ has joined #openstack-cinder | 22:05 | |
| jgriffith | :) | 22:05 |
| patrickeast | jgriffith: its very possible that is already there and i missed it | 22:06 |
| patrickeast | iirc when implementing it we just made up the format loosly following the comments in the code and may have done it wrong | 22:06 |
| jgriffith | patrickeast: hemna https://github.com/openstack/cinder/blob/master/doc/source/devref/replication.rst | 22:06 |
| *** haigang has joined #openstack-cinder | 22:07 | |
| jgriffith | patrickeast: hemna managed is "EZ", but I don't know how/if you'd want to do something different on unmanaged? | 22:07 |
| *** setmason has quit IRC | 22:07 | |
| jgriffith | patrickeast: hemna the idea was to just spit out what they set in the config file | 22:08 |
| hemna | yah, unmanaged is the more complex case | 22:08 |
| patrickeast | jgriffith: yea they end up pretty much the same for us, but depends on how the arrays work | 22:08 |
| jgriffith | hemna: and you can't "force" some structure there IMO | 22:08 |
| hemna | and the case I was referring to in returning the standardized 'target_name' for the replication targets list | 22:09 |
| jgriffith | hemna: you mean "list_replication_targets" ? | 22:09 |
| hemna | yes | 22:09 |
| hemna | sorry | 22:09 |
| hemna | I'm terrible at communicating what I'm thinking | 22:09 |
| jgriffith | hemna: which is what I'm talking about too :) And is the last item in that doc | 22:09 |
| jgriffith | haha | 22:09 |
| jgriffith | I'll work on my telepathy :) | 22:09 |
| hemna | hehe | 22:10 |
| hemna | so, what I was thinking was adding a target_name in there | 22:10 |
| *** chlong has joined #openstack-cinder | 22:10 | |
| patrickeast | jgriffith: yea so in the doc we have different keys for the manage vs unmanaged, but imo it makes sense to have a standard one that is the 'secondary' string you copy pasta into the api command to failover | 22:10 |
| hemna | that everyone will add to each of the dict entries | 22:10 |
| patrickeast | at least one common one | 22:10 |
| hemna | patrickeast, +1 | 22:11 |
| jgriffith | patrickeast: hemna sure if it works, I'd need to see what is proposed though | 22:12 |
| jgriffith | patrickeast: hemna when I looked through the code that was out there everybody needed some pretty "different" things | 22:12 |
| patrickeast | yea def | 22:12 |
| hemna | maybe I'll put together a quick json dict to show | 22:13 |
| jgriffith | cool | 22:13 |
| hemna | http://pastebin.com/WcQDpewB | 22:15 |
| *** openstackgerrit has quit IRC | 22:16 | |
| *** openstackgerrit has joined #openstack-cinder | 22:17 | |
| *** akerr_ has quit IRC | 22:17 | |
| *** jungleboyj has joined #openstack-cinder | 22:19 | |
| *** porrua has quit IRC | 22:19 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Add a standard key for replication target dicts https://review.openstack.org/231210 | 22:20 |
| patrickeast | jgriffith: hemna: ^ i put it into the doc hopefully for more context | 22:21 |
| * patrickeast should mark that was a do not merge | 22:21 | |
| hemna | patrickeast, nice. much better than what I did. | 22:21 |
| hemna | that's basically what I had in mind as well | 22:21 |
| hemna | we could bikeshed for a few hours on 'name' vs. 'target_name' | 22:21 |
| patrickeast | haha yea | 22:22 |
| hemna | :) | 22:22 |
| hemna | or roll some dice and be done with it. | 22:22 |
| hemna | either way, I'd be happy with it. | 22:22 |
| patrickeast | my reasoning is, and i can see the for-loop now, you would end up doing for target in targets: target['target_name'] | 22:22 |
| patrickeast | id be like psh i just want target['name'] | 22:22 |
| *** tsekiyama has quit IRC | 22:23 | |
| hemna | cool | 22:23 |
| hemna | sold | 22:23 |
| hemna | heh | 22:23 |
| hemna | I'm not married to 'target_name' | 22:23 |
| hemna | just as long as the name idea is there so we can all output something that is there for the copy/paste for failover | 22:24 |
| patrickeast | settle it with http://justflipacoin.com/ :D | 22:24 |
| *** anshul has quit IRC | 22:24 | |
| patrickeast | yep totally agree | 22:24 |
| jgriffith | Oh... sure, you just want/need a label | 22:25 |
| jgriffith | that makes sense | 22:25 |
| *** tsekiyama has joined #openstack-cinder | 22:25 | |
| patrickeast | yea should be simple for everyone to do it | 22:25 |
| hemna | yah, it's just a label | 22:25 |
| jgriffith | replication_target_name | 22:26 |
| patrickeast | i dunno man, we've only got 80 chars per line to work with... | 22:26 |
| * patrickeast is kidding... mostly | 22:26 | |
| hemna | :) | 22:26 |
| hemna | the_common_name_that_everyone_hated | 22:27 |
| patrickeast | haha | 22:27 |
| hemna | :) | 22:27 |
| hemna | literally, that key | 22:27 |
| hemna | have to do something to make it through Monday | 22:28 |
| *** jamielennox|away is now known as jamielennox | 22:31 | |
| *** akerr has joined #openstack-cinder | 22:32 | |
| *** setmason has joined #openstack-cinder | 22:33 | |
| patrickeast | gah this cisco fczm is annoying, can't run the tempest tests in parallel with it puking all over | 22:33 |
| hemna | ? | 22:35 |
| patrickeast | basically this bug https://bugs.launchpad.net/cinder/+bug/1482398 | 22:35 |
| openstack | Launchpad bug 1482398 in Cinder "cisco zone manager fails to delete zone" [Undecided,New] - Assigned to Al Lau (alau2) | 22:35 |
| *** kevincarr1991 has joined #openstack-cinder | 22:35 | |
| patrickeast | the zone manager (or i guess maybe the switch?) can't handle doing parallel management requests to modify the zone | 22:36 |
| hemna | don't they have a local file lock in there? | 22:36 |
| *** akerr has quit IRC | 22:36 | |
| patrickeast | dunno | 22:36 |
| * patrickeast checks | 22:36 | |
| hemna | patrickeast, https://github.com/openstack/cinder/blob/master/cinder/zonemanager/drivers/cisco/cisco_fc_zone_driver.py#L111 | 22:37 |
| hemna | they also have one on delete_connection | 22:38 |
| patrickeast | hmm that is interesting | 22:38 |
| patrickeast | just now on a test i'm seeing http://paste.openstack.org/show/475379/ | 22:39 |
| patrickeast | errors deleting from a zone and adding to one | 22:39 |
| patrickeast | maybe there is something asynchronous on the switch? so even if its locked in the cinder code it isn't ready yet.. | 22:40 |
| *** mc_nair has quit IRC | 22:40 | |
| hemna | FC just sucks | 22:40 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 22:40 |
| *** kevincarr1991 has quit IRC | 22:40 | |
| *** akerr has joined #openstack-cinder | 22:40 | |
| patrickeast | i might just go add a retry with like 10 tries on it | 22:40 |
| hemna | so | 22:40 |
| hemna | I wonder | 22:40 |
| hemna | if a previous delete command wasn't done on the switch | 22:40 |
| hemna | if they fire off the command and expect it to finish async at some later time | 22:41 |
| hemna | and they get a follow up delete before the async finishes, and so it pukes | 22:41 |
| patrickeast | yea : / thats probably exactly whats happening | 22:41 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 22:41 |
| hemna | patrickeast, https://supportforums.cisco.com/discussion/11089316/nexus-7000-copy-run-start-config-change-already-progress | 22:45 |
| *** raildo-afk is now known as raildo | 22:45 | |
| *** wilson-1 has joined #openstack-cinder | 22:46 | |
| *** harlowja has joined #openstack-cinder | 22:48 | |
| *** wilson1 has quit IRC | 22:49 | |
| *** dims__ has joined #openstack-cinder | 22:49 | |
| *** dims__ has quit IRC | 22:49 | |
| *** merooney has joined #openstack-cinder | 22:50 | |
| *** lcurtis has quit IRC | 22:53 | |
| patrickeast | hemna: mmm so it looks like there is already some retry logic built in, i'm going to test out just setting this from 1 to 5 or something https://github.com/openstack/cinder/blob/master/cinder/zonemanager/drivers/cisco/cisco_fc_zone_client_cli.py#L318 | 22:53 |
| *** merooeny_ has joined #openstack-cinder | 22:53 | |
| *** dims_ has quit IRC | 22:53 | |
| *** willsama has quit IRC | 22:54 | |
| hemna | ok | 22:55 |
| *** merooney has quit IRC | 22:56 | |
| hemna | oh the attempts=1 | 22:56 |
| hemna | interesting | 22:56 |
| patrickeast | from what i'm seeing, it recovers fine so hopefully it isn't the bug mentioned in that link you sent | 22:56 |
| patrickeast | yea | 22:56 |
| patrickeast | it will keep trying with a sleep between tries | 22:56 |
| hemna | that's quite odd they have a retry built in to executing ssh commands | 22:58 |
| patrickeast | haha yea | 22:58 |
| patrickeast | more so in that i don't see anywhere with it being >1 | 22:58 |
| hemna | yah, we know ssh sucks, so just try a bunch | 22:58 |
| patrickeast | its really going to suck with A-A if we don't have a distributed lock | 22:59 |
| patrickeast | no way this thing is going to work correctly | 22:59 |
| hemna | yah | 22:59 |
| hemna | but we are pushing the boundaries of what some of these devices were intended | 22:59 |
| patrickeast | haha true | 22:59 |
| hemna | a device that was designed to be updated by a single dude | 22:59 |
| patrickeast | so we just have to properly emulate that dude | 23:00 |
| hemna | put a REST interface on everything | 23:00 |
| patrickeast | haha depends how they implement it though... easy enough to slap a REST API on top of whatever handled those CLI commands and still have the same problem | 23:01 |
| hemna | no comment! | 23:01 |
| hemna | :P | 23:01 |
| patrickeast | lol | 23:01 |
| *** merooeny_ has quit IRC | 23:01 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 23:02 |
| *** dave-mccowan has joined #openstack-cinder | 23:02 | |
| *** david-ly_ has joined #openstack-cinder | 23:04 | |
| *** david-lyle has quit IRC | 23:04 | |
| patrickeast | hemna: oh, so this fc stuff was another side track along poking at that permission denied/unable to write bug... i added some code to check what /sys/block/xxx/ro is... but what i'm wondering is if that is actually definitive, like for sure if its 1 the read-only, do you know if a 0 means for sure it is writable? | 23:06 |
| *** kevincarr1991 has joined #openstack-cinder | 23:06 | |
| *** wilson1 has joined #openstack-cinder | 23:07 | |
| patrickeast | i'm concerned it might be more of a best guess | 23:07 |
| patrickeast | by the kernel | 23:07 |
| hemna | you mean the return/result code 1 ? | 23:08 |
| *** raildo is now known as raildo-afk | 23:08 | |
| patrickeast | just whatever the value is, so like 'cat /sys/block/sdb/ro' is either 0 or 1 | 23:08 |
| patrickeast | or it shows up in the output of lsblk | 23:09 |
| hemna | hrmm | 23:09 |
| *** gouthamr has quit IRC | 23:10 | |
| hemna | have you tried dumping out the perms on the device prior to the qemu-img convert call ? | 23:10 |
| *** wilson-1 has quit IRC | 23:10 | |
| patrickeast | yea the file system permissions are always the same between ones that work and ones that fail | 23:10 |
| hemna | wth | 23:10 |
| hemna | maybe disable directio ? | 23:11 |
| *** kevincarr1991 has quit IRC | 23:11 | |
| patrickeast | so im going the next level down to the block device read-only info | 23:11 |
| hemna | the test that pukes is right after it thinks it can do directio on it | 23:11 |
| hemna | leaving out the device buffers | 23:11 |
| *** apoorvad_ has quit IRC | 23:11 | |
| hemna | crap...off to soccer practice | 23:12 |
| *** hemna is now known as hemnafk | 23:12 | |
| patrickeast | hmm thats worth a shot | 23:12 |
| patrickeast | i'll try it out | 23:12 |
| patrickeast | hemnafk: thanks | 23:12 |
| *** apoorvad has joined #openstack-cinder | 23:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/231225 | 23:13 |
| *** earlephilhower has quit IRC | 23:14 | |
| *** chris_morrell has quit IRC | 23:15 | |
| *** dims_ has joined #openstack-cinder | 23:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/231239 | 23:17 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (unmanaged) https://review.openstack.org/231245 | 23:19 |
| *** tsekiyama has quit IRC | 23:23 | |
| patrickeast | :o | 23:23 |
| patrickeast | hemnafk: finally got one to repro... they are showing up as read-only at that level http://paste.openstack.org/show/475384/ | 23:23 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (unmanaged) https://review.openstack.org/231245 | 23:24 |
| *** zhiyan has quit IRC | 23:24 | |
| *** xyang1 has quit IRC | 23:25 | |
| *** zhiyan has joined #openstack-cinder | 23:26 | |
| patrickeast | jungleboyj: hey, just saw your comment on https://review.openstack.org/#/c/230781/ | 23:27 |
| patrickeast | jungleboyj: so uh... the short answer is 'its complicated' | 23:28 |
| patrickeast | jungleboyj: i have a list of them over in the Nova side of things https://review.openstack.org/#/c/229152/ | 23:28 |
| *** angela-s has quit IRC | 23:29 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add ability to set prefix in SolidFire Volume name https://review.openstack.org/231250 | 23:30 |
| *** IanGovett has quit IRC | 23:32 | |
| *** gouthamr has joined #openstack-cinder | 23:32 | |
| *** gouthamr_ has joined #openstack-cinder | 23:34 | |
| *** tsekiyama has joined #openstack-cinder | 23:34 | |
| *** gouthamr has quit IRC | 23:37 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add ability to set prefix in SolidFire Volume name https://review.openstack.org/231250 | 23:41 |
| *** stevemar_ has quit IRC | 23:42 | |
| *** setmason has quit IRC | 23:42 | |
| *** stevemar_ has joined #openstack-cinder | 23:45 | |
| patrickeast | jgriffith: hey, just curious for that change with the prefix ^ any reason in particular why you aren't using the volume_name_template and volume['name'] instead of volume['id'] | 23:47 |
| patrickeast | jgriffith: i ask because i wasn't sure what exactly the intended use was for it... but my driver was implemented with it, just curious if there are any downsides i should know about | 23:48 |
| *** dims_ has quit IRC | 23:50 | |
| *** DericHorn-HP has quit IRC | 23:51 | |
| jgriffith | patrickeast: Name is optional... but I could just use the existing conf option I guess | 23:51 |
| jgriffith | patrickeast: ie volume[name] is optional | 23:52 |
| jgriffith | wanted guaranteed unique also | 23:52 |
| jgriffith | patrickeast: template_name might be a good idea though.... | 23:53 |
| patrickeast | jgriffith: eh other name... this confused me for a while, https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L127 | 23:53 |
| *** haigang has quit IRC | 23:53 | |
| patrickeast | jgriffith: volume['name'] will be the UUID passed into the template | 23:53 |
| jgriffith | patrickeast: Ohhh.. that name :) | 23:53 |
| patrickeast | jgriffith: and does the "right" thing after migrations | 23:53 |
| jgriffith | patrickeast: well lookie there :) | 23:54 |
| *** gouthamr has joined #openstack-cinder | 23:55 | |
| jgriffith | patrickeast: cool... I marked it as wip and noted your suggestion | 23:55 |
| jgriffith | I'll take a look at that tomorrow | 23:55 |
| jgriffith | great idea, thanks! | 23:55 |
| patrickeast | jgriffith: ah ok, cool, so i guess the answer was 'no' there wasn't any downside to it i should be concerned about with my driver ;) | 23:55 |
| jgriffith | patrickeast: No way | 23:56 |
| *** gouthamr_ has quit IRC | 23:56 | |
| *** I has joined #openstack-cinder | 23:56 | |
| * patrickeast was accidentally helpful | 23:56 | |
| jgriffith | patrickeast: haha | 23:56 |
| *** I is now known as Guest71307 | 23:56 | |
| jgriffith | patrickeast: I have to look at it, it doesn't help the problem of people not having it set or if I introduce it with something other than UUID- | 23:56 |
| jgriffith | patrickeast: because existing folks will upgrade and while it works on the Cinder side, the lookups on the SolidFire side will all fail :( | 23:57 |
| patrickeast | jgriffith: yea : ( that does suck | 23:57 |
| jgriffith | which is why I went the route I did.... but I'm now thinking there may be a way around it | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!