| jmccrory | mriedem : i'm seeing that in openstack-ansible's testing as well. http://logs.openstack.org/64/380464/2/check/gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty/09951c6/console.html#_2016-10-02_23_51_42_469350 | 00:03 |
|---|---|---|
| mriedem | hmm, i don't see any recent new packages that would blow this up | 00:03 |
| mriedem | no new releases of things | 00:03 |
| *** cknight has quit IRC | 00:04 | |
| mriedem | now it does look like pycparser was updated on pypi today https://pypi.python.org/pypi/pycparser | 00:04 |
| mriedem | for whatever reason | 00:04 |
| *** jwcroppe has joined #openstack-cinder | 00:05 | |
| *** cknight has joined #openstack-cinder | 00:07 | |
| *** jungleboyj has quit IRC | 00:15 | |
| jmccrory | mriedem: looks like that's it. i just tried installing pycparser 2.14 from source and the db sync succeeded | 00:17 |
| *** akshai has quit IRC | 00:23 | |
| *** jwcroppe has quit IRC | 00:33 | |
| *** jwcroppe has joined #openstack-cinder | 00:33 | |
| mriedem | hmm maybe the new version was recompiled against something different | 00:33 |
| mriedem | i don't know how to even open an issue against pycparser | 00:34 |
| mriedem | oh i see | 00:34 |
| mriedem | yup | 00:34 |
| mriedem | https://github.com/eliben/pycparser/issues/148 | 00:34 |
| mriedem | tracking in openstack with this bug https://bugs.launchpad.net/cinder/+bug/1629726 | 00:38 |
| openstack | Launchpad bug 1629726 in Cinder "cinder-manage db sync fails with recompiled pycparser 2.14" [Undecided,Confirmed] | 00:38 |
| mriedem | i'll get something in e-r | 00:38 |
| mriedem | it's at least reported upstream | 00:38 |
| *** ducttape_ has joined #openstack-cinder | 00:46 | |
| *** mriedem has quit IRC | 00:55 | |
| *** haplo37_ has quit IRC | 01:23 | |
| *** haplo37_ has joined #openstack-cinder | 01:25 | |
| *** kaisers_ has joined #openstack-cinder | 01:37 | |
| *** aohuanxuan has joined #openstack-cinder | 01:37 | |
| *** zul has joined #openstack-cinder | 01:40 | |
| *** kaisers_ has quit IRC | 01:41 | |
| *** alonma has joined #openstack-cinder | 01:46 | |
| *** alonma has quit IRC | 01:51 | |
| *** salv-orlando has joined #openstack-cinder | 02:04 | |
| *** salv-orlando has quit IRC | 02:08 | |
| *** Lee1092 has joined #openstack-cinder | 02:14 | |
| *** sdake has quit IRC | 02:29 | |
| *** mtanino has joined #openstack-cinder | 02:55 | |
| *** cknight has quit IRC | 02:58 | |
| *** salv-orlando has joined #openstack-cinder | 03:04 | |
| *** jungleboyj has joined #openstack-cinder | 03:07 | |
| *** salv-orlando has quit IRC | 03:09 | |
| *** akshai has joined #openstack-cinder | 03:18 | |
| *** sudipto has joined #openstack-cinder | 03:20 | |
| *** sudipto_ has joined #openstack-cinder | 03:20 | |
| *** sdake has joined #openstack-cinder | 03:21 | |
| *** akshai has quit IRC | 03:21 | |
| *** kaisers_ has joined #openstack-cinder | 03:26 | |
| *** kaisers_ has quit IRC | 03:31 | |
| *** bks has joined #openstack-cinder | 03:32 | |
| *** sdake_ has joined #openstack-cinder | 03:38 | |
| *** sdake has quit IRC | 03:40 | |
| *** coolsvap has joined #openstack-cinder | 03:41 | |
| *** laughterwym has joined #openstack-cinder | 03:42 | |
| *** zhengyin has joined #openstack-cinder | 03:43 | |
| *** laughterwym has quit IRC | 03:47 | |
| *** links has joined #openstack-cinder | 03:49 | |
| *** diablo_rojo has joined #openstack-cinder | 03:50 | |
| jungleboyj | The diablo is here! | 03:51 |
| diablo_rojo | jungleboyj, patrickeast So... Ibiza? | 03:51 |
| jungleboyj | Ahh, Ibiza. | 03:51 |
| jungleboyj | patrickeast: So, I looked and flights there are like 35 to 54 bucks. So, not a big deal. | 03:52 |
| jungleboyj | It is the off season but it looks like Pacha is still pretty active. | 03:52 |
| patrickeast | I'm game | 03:53 |
| jungleboyj | There is hiking and good food over there. | 03:53 |
| diablo_rojo | The hitch would be we would have to stay up all night and get a flight back to BCN at like 7 AM | 03:53 |
| jungleboyj | diablo_rojo: There is that. | 03:54 |
| patrickeast | Haha | 03:54 |
| jungleboyj | Or leave at 11 . | 03:54 |
| jungleboyj | Pm. | 03:54 |
| patrickeast | Well, there are worse things than staying up all night | 03:54 |
| diablo_rojo | Which is like before the party even starts | 03:54 |
| jungleboyj | diablo_rojo: Yeah. | 03:54 |
| diablo_rojo | Almost did it last night lol | 03:54 |
| *** alonma has joined #openstack-cinder | 03:55 | |
| patrickeast | lol | 03:55 |
| diablo_rojo | With proper caffination it should be easy. | 03:55 |
| patrickeast | What day were you guys thinking to go? | 03:55 |
| jungleboyj | A sweet balance of alcohol and caffeine. | 03:55 |
| diablo_rojo | Thats still up for a little debate | 03:57 |
| diablo_rojo | kinda depends on what your other plans are and what we are interested in going to see | 03:57 |
| jungleboyj | So, Sankey's has Unusual Suspects on Thursday night: http://www.ibiza-spotlight.com/night/promoters/unusual-suspects | 03:57 |
| jungleboyj | Minimal House and Techno. | 03:58 |
| jungleboyj | Or Sankeys has Tribal Sessions on Saturday but that is less likely to be up our alley Music wise. | 03:59 |
| *** alonma has quit IRC | 04:00 | |
| jungleboyj | Otherwise we could just head there Friday, do some hiking, etc and find clubs at night. | 04:00 |
| jungleboyj | See where it takes us. | 04:00 |
| patrickeast | I'm pretty open for those any of those dates, I had booked a tour on Saturday but can move that around whenever | 04:01 |
| jungleboyj | diablo_rojo: What do you think? | 04:02 |
| *** alonma has joined #openstack-cinder | 04:02 | |
| diablo_rojo | Friday or Saturday would be fine by me | 04:03 |
| *** oak has quit IRC | 04:03 | |
| jungleboyj | diablo_rojo patrickeast Ok, I will do a little more looking at what is available Friday/Saturday and come up with a plan. | 04:04 |
| *** salv-orlando has joined #openstack-cinder | 04:05 | |
| diablo_rojo | Sounds good to me | 04:05 |
| jungleboyj | patrickeast: I can book whatever once we agree and you can shoot me more money/ | 04:05 |
| *** haplo37_ has quit IRC | 04:05 | |
| patrickeast | jungleboyj: sounds good | 04:06 |
| *** alonma has quit IRC | 04:06 | |
| diablo_rojo | patrickeast, I am doing some introductory looking into going to Marrakesh (I was inspired by that episode of house hunters we watched) and its not looking cheap.. | 04:07 |
| *** haplo37_ has joined #openstack-cinder | 04:08 | |
| patrickeast | diablo_rojo: dang, like how not cheap? | 04:08 |
| *** pgadiya has joined #openstack-cinder | 04:09 | |
| *** salv-orlando has quit IRC | 04:10 | |
| diablo_rojo | patrickeast, Basically it will take a day to get there if you dont fly and flying is like 500ish | 04:10 |
| diablo_rojo | patrickeast, I can do some more research tomorrow and see if I can find something better, but I'm not holding my breath. Guess the foundation should hold an OSD event there so we can go lol | 04:11 |
| patrickeast | Ouch | 04:11 |
| diablo_rojo | Yeah. | 04:11 |
| patrickeast | diablo_rojo: haha yea, probably the best strategy | 04:11 |
| jungleboyj | :-) | 04:12 |
| patrickeast | OpenStack days Casablanca | 04:12 |
| diablo_rojo | YES | 04:12 |
| diablo_rojo | Can they do one in Egypt too? | 04:12 |
| diablo_rojo | So many places I want to travel. | 04:12 |
| jungleboyj | diablo_rojo: That would be awesome! | 04:12 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/cinder-specs: Fix typo: remove redundant 'the' https://review.openstack.org/380906 | 04:23 |
| *** diablo_rojo has quit IRC | 04:23 | |
| *** aohuanxuan has quit IRC | 04:28 | |
| *** bks has quit IRC | 04:35 | |
| *** laughterwym has joined #openstack-cinder | 04:41 | |
| *** laughterwym has quit IRC | 04:46 | |
| *** pdeore has joined #openstack-cinder | 04:49 | |
| *** shausy has joined #openstack-cinder | 04:49 | |
| *** gouthamr has quit IRC | 04:56 | |
| *** salv-orlando has joined #openstack-cinder | 05:04 | |
| *** kaisers_ has joined #openstack-cinder | 05:15 | |
| *** kaisers_ has quit IRC | 05:19 | |
| *** GB21 has joined #openstack-cinder | 05:30 | |
| *** links has quit IRC | 05:31 | |
| *** e0ne has joined #openstack-cinder | 05:31 | |
| *** e0ne has quit IRC | 05:35 | |
| *** raunak has joined #openstack-cinder | 05:37 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Switch service capabilities to ovo https://review.openstack.org/319040 | 05:38 |
| *** links has joined #openstack-cinder | 05:44 | |
| *** raunak has quit IRC | 05:45 | |
| *** mugsie has quit IRC | 05:50 | |
| *** mugsie has joined #openstack-cinder | 05:51 | |
| *** GB21 has quit IRC | 05:52 | |
| *** raunak has joined #openstack-cinder | 05:52 | |
| *** sandanar has joined #openstack-cinder | 05:53 | |
| *** raunak has quit IRC | 05:55 | |
| *** lpetrut has joined #openstack-cinder | 05:57 | |
| *** raunak has joined #openstack-cinder | 05:57 | |
| *** Poornima has joined #openstack-cinder | 06:00 | |
| *** salv-orlando has quit IRC | 06:00 | |
| *** GB21 has joined #openstack-cinder | 06:03 | |
| *** rcernin has joined #openstack-cinder | 06:04 | |
| *** GB21 has quit IRC | 06:08 | |
| *** raunak has quit IRC | 06:09 | |
| *** raunak has joined #openstack-cinder | 06:11 | |
| *** alonma has joined #openstack-cinder | 06:17 | |
| *** raunak has quit IRC | 06:20 | |
| *** alonma has quit IRC | 06:21 | |
| *** sdake has joined #openstack-cinder | 06:22 | |
| *** sdake_ has quit IRC | 06:23 | |
| *** links has quit IRC | 06:31 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/cinder: Fix typo: remove redundant 'that' https://review.openstack.org/380941 | 06:32 |
| *** GB21 has joined #openstack-cinder | 06:33 | |
| *** haplo37_ has quit IRC | 06:41 | |
| *** haplo37_ has joined #openstack-cinder | 06:43 | |
| *** links has joined #openstack-cinder | 06:44 | |
| *** salv-orlando has joined #openstack-cinder | 06:50 | |
| *** pcaruana has joined #openstack-cinder | 06:53 | |
| *** ldeptula has joined #openstack-cinder | 06:53 | |
| *** e0ne has joined #openstack-cinder | 06:54 | |
| *** GB21 has quit IRC | 06:59 | |
| *** lpetrut has quit IRC | 07:01 | |
| *** tesseract- has joined #openstack-cinder | 07:03 | |
| *** sudipto_ has quit IRC | 07:06 | |
| *** sudipto has quit IRC | 07:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/380812 | 07:09 |
| *** lpetrut has joined #openstack-cinder | 07:10 | |
| *** GB21 has joined #openstack-cinder | 07:11 | |
| *** sdake has quit IRC | 07:15 | |
| *** MohammadSayyed has joined #openstack-cinder | 07:17 | |
| *** GB21 has quit IRC | 07:17 | |
| *** MohammadSayyed is now known as msayyed | 07:20 | |
| *** sdake has joined #openstack-cinder | 07:26 | |
| *** GB21 has joined #openstack-cinder | 07:29 | |
| *** tgtanya has joined #openstack-cinder | 07:34 | |
| *** lpetrut has quit IRC | 07:35 | |
| *** bks has joined #openstack-cinder | 07:37 | |
| *** zigo has quit IRC | 07:39 | |
| openstackgerrit | Sharat Sharma proposed openstack/cinder: Stop adding ServiceAvailable group option https://review.openstack.org/380960 | 07:40 |
| *** alonmarx has joined #openstack-cinder | 07:41 | |
| *** zigo has joined #openstack-cinder | 07:42 | |
| *** zigo is now known as Guest24756 | 07:42 | |
| *** tgtanya has quit IRC | 07:42 | |
| *** salv-orl_ has joined #openstack-cinder | 07:44 | |
| *** kaisers_ has joined #openstack-cinder | 07:47 | |
| *** salv-orlando has quit IRC | 07:48 | |
| *** sdake has quit IRC | 07:58 | |
| *** gcb has joined #openstack-cinder | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| openstackgerrit | Sharat Sharma proposed openstack/cinder: Stop adding ServiceAvailable group option https://review.openstack.org/380960 | 08:05 |
| *** bks has quit IRC | 08:10 | |
| *** jordanP has joined #openstack-cinder | 08:22 | |
| *** gaurangt has joined #openstack-cinder | 08:24 | |
| *** sandanar_ has joined #openstack-cinder | 08:30 | |
| *** sudipto_ has joined #openstack-cinder | 08:31 | |
| *** sudipto has joined #openstack-cinder | 08:31 | |
| *** sandanar__ has joined #openstack-cinder | 08:31 | |
| *** alonma has joined #openstack-cinder | 08:32 | |
| *** sandanar has quit IRC | 08:34 | |
| *** sandanar_ has quit IRC | 08:35 | |
| *** sudipto_ has quit IRC | 08:36 | |
| *** sudipto has quit IRC | 08:36 | |
| *** alonma has quit IRC | 08:37 | |
| *** alonma has joined #openstack-cinder | 08:40 | |
| *** GB21 has quit IRC | 08:41 | |
| *** alonma has quit IRC | 08:44 | |
| *** GB21 has joined #openstack-cinder | 08:48 | |
| *** Guest24756 is now known as zigo | 08:54 | |
| GB21 | Hi, I am writing Searchlight plugin for consistency groups and I am using python client, I need to index the project and user id associated with that consistency group. | 09:00 |
| GB21 | apparently, the python-cinder client does not provide the user id or project id in to_dict() method for consistency group | 09:01 |
| GB21 | can anyone tell me, how can I extract user_id/project_id; thanks in advance | 09:02 |
| *** Poornima has quit IRC | 09:05 | |
| *** lpetrut has joined #openstack-cinder | 09:06 | |
| *** zhengyin has quit IRC | 09:08 | |
| *** ManishD has joined #openstack-cinder | 09:38 | |
| *** hoonetorg has quit IRC | 09:39 | |
| *** haplo37_ has quit IRC | 09:41 | |
| *** haplo37_ has joined #openstack-cinder | 09:43 | |
| *** sdake has joined #openstack-cinder | 09:44 | |
| *** hoonetorg has joined #openstack-cinder | 09:52 | |
| openstackgerrit | Sharat Sharma proposed openstack/cinder: Stop adding ServiceAvailable group option https://review.openstack.org/380960 | 09:54 |
| *** hoonetorg has quit IRC | 10:00 | |
| *** alonma has joined #openstack-cinder | 10:03 | |
| *** alonma has quit IRC | 10:07 | |
| *** alonma has joined #openstack-cinder | 10:10 | |
| *** hoonetorg has joined #openstack-cinder | 10:13 | |
| *** alonma has quit IRC | 10:15 | |
| *** sudipto has joined #openstack-cinder | 10:18 | |
| *** sudipto_ has joined #openstack-cinder | 10:18 | |
| *** Poornima has joined #openstack-cinder | 10:20 | |
| *** sudipto has quit IRC | 10:21 | |
| *** sudipto_ has quit IRC | 10:21 | |
| *** sudipto_ has joined #openstack-cinder | 10:21 | |
| *** sudipto has joined #openstack-cinder | 10:21 | |
| *** sudipto has quit IRC | 10:21 | |
| *** sudipto_ has quit IRC | 10:21 | |
| *** zhangguoqing has joined #openstack-cinder | 10:36 | |
| *** sudipto has joined #openstack-cinder | 10:43 | |
| *** sudipto_ has joined #openstack-cinder | 10:44 | |
| odyssey4me | anyone around who's seeing this error with a new install of stable/newton? http://logs.openstack.org/99/380999/1/check/gate-openstack-ansible-os_cinder-ansible-func-ubuntu-trusty/618d3d8/console.html#_2016-10-03_09_36_24_429481 | 10:47 |
| odyssey4me | ie cinder-manage db sync is erroring out with 'sorry, but this version only supports 100 named groups' | 10:47 |
| *** openstackgerrit has quit IRC | 10:48 | |
| odyssey4me | it may related to https://github.com/eliben/pycparser/issues/147 - busy confirming | 10:49 |
| *** openstackgerrit has joined #openstack-cinder | 10:49 | |
| e0ne | odyssey4me: here is a reported issue https://bugs.launchpad.net/cinder/+bug/1629726 | 10:50 |
| openstack | Launchpad bug 1629726 in OpenStack Compute (nova) "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed] | 10:50 |
| odyssey4me | ah, thanks e0ne | 10:51 |
| *** kaisers_ has quit IRC | 10:52 | |
| *** cdelatte has joined #openstack-cinder | 10:55 | |
| *** Poornima has quit IRC | 10:56 | |
| *** nikeshm has joined #openstack-cinder | 10:59 | |
| *** nicolasbock has joined #openstack-cinder | 11:04 | |
| nikeshm | hi | 11:07 |
| nikeshm | facing this issue in CI | 11:07 |
| nikeshm | from morning | 11:07 |
| nikeshm | http://paste.openstack.org/show/583974/ | 11:07 |
| nikeshm | any idea? | 11:07 |
| dulek | nikeshm: https://bugs.launchpad.net/cinder/+bug/1629726 | 11:08 |
| openstack | Launchpad bug 1629726 in OpenStack Compute (nova) "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed] | 11:08 |
| *** Poornima has joined #openstack-cinder | 11:08 | |
| dulek | nikeshm: And http://lists.openstack.org/pipermail/openstack-dev/2016-October/104909.html | 11:08 |
| nikeshm | dulek: thnks | 11:09 |
| *** sudipto has quit IRC | 11:12 | |
| *** sdake_ has joined #openstack-cinder | 11:12 | |
| *** sudipto has joined #openstack-cinder | 11:12 | |
| *** salv-orl_ has quit IRC | 11:14 | |
| *** sdake has quit IRC | 11:14 | |
| nikeshm | dulek: so we have to stop our CI | 11:16 |
| nikeshm | right | 11:16 |
| nikeshm | until the issue is resolved | 11:16 |
| nikeshm | or is there any work around | 11:17 |
| *** salv-orlando has joined #openstack-cinder | 11:18 | |
| *** Poornima has quit IRC | 11:23 | |
| dulek | nikeshm: A possible patch is here: https://review.openstack.org/#/c/381011/ | 11:25 |
| *** zhengyin has joined #openstack-cinder | 11:29 | |
| openstackgerrit | Bartek Żurawski proposed openstack/cinder: Add backup notification to cinder-volume-usage-audit https://review.openstack.org/379546 | 11:30 |
| nikeshm | dulek: thnks | 11:32 |
| *** jrollen has joined #openstack-cinder | 11:35 | |
| *** ManishD has quit IRC | 11:39 | |
| *** gouthamr has joined #openstack-cinder | 11:41 | |
| *** alyson_ has joined #openstack-cinder | 11:49 | |
| *** gaurangt has left #openstack-cinder | 11:51 | |
| *** sdake_ is now known as sdake | 11:52 | |
| *** alonma has joined #openstack-cinder | 11:56 | |
| *** alonma has quit IRC | 12:00 | |
| *** tpsilva has joined #openstack-cinder | 12:03 | |
| *** Poornima has joined #openstack-cinder | 12:03 | |
| *** alonma has joined #openstack-cinder | 12:04 | |
| *** alonma has quit IRC | 12:08 | |
| *** chlong has joined #openstack-cinder | 12:08 | |
| *** alonma has joined #openstack-cinder | 12:11 | |
| *** MohammadSayyed has joined #openstack-cinder | 12:13 | |
| *** chlong has quit IRC | 12:14 | |
| *** alonma has quit IRC | 12:16 | |
| *** msayyed has quit IRC | 12:16 | |
| *** pdeore has quit IRC | 12:16 | |
| *** jrollen has quit IRC | 12:18 | |
| *** alonma has joined #openstack-cinder | 12:19 | |
| *** jungleboyj has quit IRC | 12:19 | |
| *** edmondsw has joined #openstack-cinder | 12:23 | |
| *** alonmarx has quit IRC | 12:23 | |
| *** alonma has quit IRC | 12:24 | |
| *** chlong has joined #openstack-cinder | 12:26 | |
| *** alonma has joined #openstack-cinder | 12:27 | |
| *** alonma has quit IRC | 12:32 | |
| *** dave-mccowan has joined #openstack-cinder | 12:36 | |
| *** pgadiya has quit IRC | 12:37 | |
| *** erlon has joined #openstack-cinder | 12:40 | |
| *** kaisers_ has joined #openstack-cinder | 12:40 | |
| *** jrollen has joined #openstack-cinder | 12:41 | |
| *** kaisers_ has quit IRC | 12:45 | |
| *** jrollen has quit IRC | 12:45 | |
| *** jrollen has joined #openstack-cinder | 12:45 | |
| nikeshm | jgriffith: hi | 12:50 |
| *** sudipto has quit IRC | 12:51 | |
| *** sudipto_ has quit IRC | 12:51 | |
| *** porrua has joined #openstack-cinder | 12:53 | |
| *** jroll has quit IRC | 12:53 | |
| *** pgadiya has joined #openstack-cinder | 12:54 | |
| *** jrollen is now known as jroll | 12:55 | |
| *** kaisers_ has joined #openstack-cinder | 12:57 | |
| *** zhangguoqing has quit IRC | 13:00 | |
| *** mtanino has quit IRC | 13:02 | |
| *** lpetrut has quit IRC | 13:02 | |
| *** ckonstanski has quit IRC | 13:08 | |
| *** pgadiya has quit IRC | 13:12 | |
| *** ducttape_ has quit IRC | 13:12 | |
| *** rraja has joined #openstack-cinder | 13:16 | |
| *** venkat has joined #openstack-cinder | 13:16 | |
| *** adrianofr has joined #openstack-cinder | 13:17 | |
| *** pgadiya has joined #openstack-cinder | 13:24 | |
| *** akerr has joined #openstack-cinder | 13:25 | |
| *** akshai has joined #openstack-cinder | 13:26 | |
| *** venkat has quit IRC | 13:31 | |
| *** gaurangt has joined #openstack-cinder | 13:32 | |
| *** mvk has quit IRC | 13:35 | |
| *** ducttape_ has joined #openstack-cinder | 13:39 | |
| *** eharney has joined #openstack-cinder | 13:41 | |
| *** salv-orl_ has joined #openstack-cinder | 13:44 | |
| *** pdeore has joined #openstack-cinder | 13:44 | |
| *** bjolo has joined #openstack-cinder | 13:46 | |
| *** salv-orlando has quit IRC | 13:47 | |
| nikeshm | jgriffith: can you review these https://review.openstack.org/#/c/380737/ https://review.openstack.org/#/c/380694 | 13:48 |
| *** eharney has quit IRC | 13:49 | |
| *** khushbu has joined #openstack-cinder | 13:49 | |
| *** mtanino has joined #openstack-cinder | 13:52 | |
| *** enriquetaso has joined #openstack-cinder | 13:52 | |
| *** GB21 has quit IRC | 13:52 | |
| *** khushbu has quit IRC | 13:55 | |
| *** cknight has joined #openstack-cinder | 13:58 | |
| *** alonmarx has joined #openstack-cinder | 14:00 | |
| *** eharney has joined #openstack-cinder | 14:01 | |
| *** mvk has joined #openstack-cinder | 14:02 | |
| *** diablo_rojo has joined #openstack-cinder | 14:07 | |
| *** mriedem has joined #openstack-cinder | 14:07 | |
| *** shausy has quit IRC | 14:10 | |
| *** pdeore has quit IRC | 14:11 | |
| *** pdeore has joined #openstack-cinder | 14:26 | |
| *** zhengyin has quit IRC | 14:27 | |
| *** rajinir has joined #openstack-cinder | 14:29 | |
| *** oak has joined #openstack-cinder | 14:31 | |
| *** timcl has quit IRC | 14:31 | |
| *** laughterwym has joined #openstack-cinder | 14:31 | |
| *** timcl has joined #openstack-cinder | 14:32 | |
| *** rooneym has joined #openstack-cinder | 14:32 | |
| *** merooney has joined #openstack-cinder | 14:32 | |
| *** crose has joined #openstack-cinder | 14:32 | |
| *** alonma has joined #openstack-cinder | 14:35 | |
| *** pdeore has quit IRC | 14:36 | |
| *** pgadiya has quit IRC | 14:37 | |
| *** jungleboyj has joined #openstack-cinder | 14:38 | |
| *** xyang1 has joined #openstack-cinder | 14:39 | |
| *** alonma has quit IRC | 14:40 | |
| *** ldeptula has quit IRC | 14:44 | |
| *** alonma has joined #openstack-cinder | 14:45 | |
| *** alonma has quit IRC | 14:49 | |
| *** diogogmt has joined #openstack-cinder | 14:50 | |
| *** alonma has joined #openstack-cinder | 14:53 | |
| *** narayrak has joined #openstack-cinder | 14:54 | |
| *** alonma has quit IRC | 14:58 | |
| *** lprice has joined #openstack-cinder | 15:00 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Hacking: Remove C305 contextlib.nested check https://review.openstack.org/381173 | 15:02 |
| *** alonma has joined #openstack-cinder | 15:03 | |
| *** links has quit IRC | 15:04 | |
| *** mvk has quit IRC | 15:06 | |
| *** alonma has quit IRC | 15:07 | |
| openstackgerrit | Merged openstack/cinder: Speed up kaminario's drivers tests https://review.openstack.org/376329 | 15:12 |
| openstackgerrit | Merged openstack/cinder: Disable API v1 by default https://review.openstack.org/379657 | 15:14 |
| *** sandanar__ has quit IRC | 15:15 | |
| e0ne | smcginnis: hi. just to confirm: did you verify that nobody uses api v1? | 15:17 |
| smcginnis | e0ne: Absolutely not | 15:17 |
| jgriffith | e0ne: they don't now :) | 15:17 |
| *** pcaruana has quit IRC | 15:18 | |
| e0ne | smcginnis, jgriffith: I remember how was it painful to remove it last year | 15:18 |
| jgriffith | unless they surive the arduous task of configuring it :) | 15:18 |
| *** crose has quit IRC | 15:18 | |
| smcginnis | e0ne: Right, that's why I didn't try to remove it again. ;) | 15:18 |
| jgriffith | e0ne: yeah, that was mostly because Nova was using it | 15:18 |
| jgriffith | e0ne: it's not removed here though, just not the default | 15:18 |
| jgriffith | smcginnis: :) | 15:18 |
| e0ne | I hope, we won't re-enable it | 15:18 |
| *** narayrak has quit IRC | 15:19 | |
| *** mvk has joined #openstack-cinder | 15:19 | |
| jgriffith | e0ne: people can configure and run it forever AFAIC | 15:19 |
| jgriffith | e0ne: but we don't have to support it :) | 15:19 |
| smcginnis | This at least makes it a little more obvious that it's not supported. | 15:19 |
| jgriffith | e0ne: or better yet, we don't have to promote it any further by leaving it as default | 15:19 |
| smcginnis | Or at least not the preferred API. | 15:19 |
| *** rcernin has quit IRC | 15:19 | |
| smcginnis | jgriffith: +1 | 15:19 |
| jgriffith | smcginnis: +1 nice work | 15:19 |
| e0ne | I'm all for disabling it by default | 15:20 |
| smcginnis | jgriffith: Now if we can do that to v2 and just get down to only one API enabled by default. | 15:20 |
| smcginnis | Might be a little premature for that though. | 15:20 |
| e0ne | smcginnis: did you change devstack https://github.com/openstack-dev/devstack/blob/master/lib/cinder#L378 part too? | 15:20 |
| jgriffith | smcginnis: yeah, I was actually *thinking* about that while reviewing your patch | 15:20 |
| smcginnis | e0ne: No, I left that part. Do you think we should pull that out too? | 15:21 |
| smcginnis | It actually doesn't do any good now unless you enable it on the cinder side. | 15:21 |
| smcginnis | But you could. | 15:21 |
| e0ne | smcginnis: let's see what happens with your patch first | 15:22 |
| e0ne | smcginnis: if nobody uses v1 now - will be OK to disable it via devstack too | 15:22 |
| jgriffith | smcginnis: not sure but there is likely value in making sure we don't break creation of the endpoint | 15:22 |
| smcginnis | jgriffith: True | 15:22 |
| jgriffith | smcginnis: e0ne depends on if anything in dsvm full uses v1 in the grenade test maybe... I dunno | 15:23 |
| *** Guest92 has joined #openstack-cinder | 15:23 | |
| jgriffith | smcginnis: e0ne remove it and find out who yells :) | 15:23 |
| e0ne | :) | 15:23 |
| *** Guest92 is now known as rushil | 15:23 | |
| jgriffith | smcginnis: so I tried something interesting last night.... regarding the attach/detach stuff | 15:24 |
| e0ne | jgriffith, smcginnis: if you both are in TC, you could vote for removing deprecated APIs :) | 15:24 |
| smcginnis | e0ne: Hah! | 15:24 |
| smcginnis | jgriffith: What was that? | 15:24 |
| jgriffith | smcginnis: I've been messing with johnthetubaguy 's suggestions about an ack after making the connection call out to os-brick on the Nova side | 15:24 |
| smcginnis | Been meaning to update my test deployment but haven't had the chance yet. | 15:25 |
| jgriffith | smcginnis: that in some ways pushes us back to the model we have already with initialize--->attach | 15:25 |
| openstackgerrit | Merged openstack/cinder: Hacking: remove check for "tests/unit/integrated" https://review.openstack.org/376542 | 15:25 |
| jgriffith | smcginnis: so I rewrote those last night, but refactored everything to simplify them and make them use attachment ID's | 15:25 |
| smcginnis | jgriffith: So same basic calls, but simpler than what we have now? | 15:26 |
| jgriffith | smcginnis: so now you call initialize_connection (I renamed it for now) and it either does the refresh or creates a connection just like today, but it creates a place-holder attachment | 15:26 |
| jgriffith | smcginnis: yes | 15:26 |
| jgriffith | smcginnis: I'm not sure if there's advantages to one over the other or not | 15:26 |
| jgriffith | smcginnis: but I am taking my time and looking at varying options because I want to get this *right* | 15:27 |
| smcginnis | jgriffith: I really hate the naming if initialize_connection doesn't actually initialize anything. | 15:27 |
| smcginnis | jgriffith: But that could be fixed. | 15:27 |
| jgriffith | smcginnis: I changed it to "connection_create_or_update" | 15:27 |
| smcginnis | jgriffith: Good call on taking time and getting it right! | 15:27 |
| smcginnis | jgriffith: OK, I like that. That makes it obvious. | 15:27 |
| jgriffith | smcginnis: and I made the "attach" "finalize_attachment" | 15:27 |
| jgriffith | or something like that | 15:27 |
| smcginnis | +1 | 15:27 |
| jgriffith | anyway, I'm wondering if I should flush out both options a little bit and post them for people to compare? | 15:28 |
| smcginnis | I've noticed we've had issues in various areas just because of naming and then false assumptions down the road based on those unclear names. | 15:28 |
| jgriffith | smcginnis: indeed! | 15:28 |
| jgriffith | smcginnis: it turns out things like terminate and detach right now are even worse :( | 15:28 |
| jgriffith | smcginnis: there's some goofy stuff that has been hacked in there, and some of it doesn't actually even work I don't think | 15:29 |
| *** haplo37__ has joined #openstack-cinder | 15:29 | |
| smcginnis | jgriffith: Which is your take as a better approach? The only problem I have with doing both (other than the extra time spent) is then we don't have one clear path. | 15:29 |
| jgriffith | smcginnis: anyway... wanted to get your opinion on if there's value in posting two alternatives or not? | 15:29 |
| smcginnis | But maybe that's fine. | 15:29 |
| jgriffith | smcginnis: yep, that's the problem :( | 15:29 |
| *** jdurgin1 has joined #openstack-cinder | 15:29 | |
| smcginnis | I'd kind of rather all get behind one "this is what we think is right" approach, TBH. | 15:30 |
| jgriffith | smcginnis: I'm mixed honestly. So I don't like the create_attachment only because it still calls the shitty methods we already have, but I do plan to fix that in follow up work | 15:30 |
| jgriffith | smcginnis: the other thing is that adding in the ack for the connection kind of forces it back to looking like what the other approach anyway | 15:31 |
| jgriffith | smcginnis: the only difference is I made that optional so Nova could use it, but others could ignore that piece if they wanted | 15:31 |
| jgriffith | I don't know... there's pros/cons to each of them | 15:31 |
| smcginnis | jgriffith: I'm actually worried about what would happen if the ack gets lost. | 15:31 |
| jgriffith | hmmm... | 15:32 |
| smcginnis | But I guess at the same time we have issues because we have fire and forget calls that just assume things worked out. | 15:32 |
| jgriffith | smcginnis: then we're back to the infamous "hung in attaching" | 15:32 |
| jgriffith | smcginnis: well, we used to have all sorts of weird cases where we would get "stuck" there | 15:32 |
| jgriffith | smcginnis: I don't want to repeat that :( | 15:32 |
| smcginnis | Yeah, that'd be bad. | 15:33 |
| smcginnis | Distributed systems are hard. Let's go shopping. | 15:33 |
| jgriffith | :) | 15:33 |
| jgriffith | I'll work out a few more things on the alternate approach this morning and let people vote on things | 15:34 |
| smcginnis | jgriffith: Don't have a good answer for you. If you think putting up both will help get the input to find the best path, then maybe that is the right answer. | 15:34 |
| jgriffith | the one good thing is I don't have to carry around the mess that's in the existing attach and terminate; it's also probably less churn on the Caller side | 15:34 |
| smcginnis | +1 | 15:34 |
| jgriffith | smcginnis: alright... I'll give it a go and see | 15:34 |
| smcginnis | jgriffith: Thanks for working on that. Really hope we can end up with something cleaner than what we have now. | 15:35 |
| jgriffith | smcginnis: +10000000000000......................00000000 | 15:35 |
| smcginnis | :) | 15:35 |
| hemna | mornin | 15:40 |
| smcginnis | hemna: Howdy | 15:40 |
| jgriffith | hola | 15:40 |
| *** lprice1 has joined #openstack-cinder | 15:40 | |
| hemna | jgriffith, is there any merit to creating a new volume state | 15:41 |
| hemna | and then after initialize_connection, set the volume state to exported | 15:41 |
| hemna | if nova/anyone wants to ack that it's attached, they can | 15:41 |
| jgriffith | hemna: :) | 15:41 |
| hemna | Cinder doesn't really care | 15:41 |
| hemna | I dunno if this helps anything or makes it more complicated | 15:41 |
| jgriffith | hemna: so yes, I was putting that in to the attachment | 15:41 |
| *** e0ne has quit IRC | 15:42 | |
| jgriffith | hemna: but the problem is it leads us back to the problem of "if i have to receive and ack back from Nova, there's not a ton of value on condensing the calls" | 15:42 |
| jgriffith | hemna: so that's why I started messing with a new approach again | 15:42 |
| hemna | so, I guess my point was | 15:42 |
| hemna | does nova really even have to ack it ? | 15:42 |
| *** lprice has quit IRC | 15:42 | |
| *** mfisch` is now known as mfisch | 15:43 | |
| hemna | I dunno | 15:43 |
| hemna | I see what you are saying | 15:43 |
| jgriffith | hemna: my opinion was no, but folks that share connections do have a potential problem. I don't and LVM doesn't so... I wasn't worried about it. But for those that do, there is a a race there | 15:43 |
| *** mfisch is now known as Guest7223 | 15:43 | |
| jgriffith | and a potential issue | 15:43 |
| hemna | if we have to have the same workflow, then what are we accomplishing by this rewrite | 15:43 |
| jgriffith | hemna: multi-attach and simplicity | 15:44 |
| hemna | so for multi-attach we already have the flag for drivers to prevent it | 15:44 |
| hemna | if that backend can't do it | 15:44 |
| jgriffith | hemna: well, sure but that's not what I think needs solved | 15:45 |
| jgriffith | hemna: the problem IMO is that what's there is packed in and a bit brittle in terms of the logic. And there's still a big issue with disconnects (shared targets) | 15:45 |
| jgriffith | hemna: but if you think it's good like it is then maybe I don't need to do anything on this | 15:46 |
| hemna | the shared targets is definitely an issue | 15:46 |
| jgriffith | hemna: I might be missing something | 15:46 |
| jgriffith | ok... good :) | 15:46 |
| openstackgerrit | Merged openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/376131 | 15:46 |
| hemna | it's starting to seem like we are coming full circle on the existing api | 15:46 |
| jgriffith | hemna: maybe... thanks to shared targets :( | 15:47 |
| jgriffith | hemna: although I still argue that that sort of thing should be handled by the caller | 15:47 |
| jgriffith | hemna: targets should always be "dumb" | 15:47 |
| jgriffith | just do what you ask them to do, and if you asked them to do something stupid... well then you were wrong | 15:47 |
| hemna | it is possible for nova to find out if it's shared or not | 15:48 |
| jgriffith | hemna: wait... now you're talking about the terminate? | 15:48 |
| jgriffith | hemna: or are you talking about acking the connection piece? | 15:48 |
| hemna | sorry I was thinking about the problem on the nova side of calling brick or not to disconnect_volume | 15:48 |
| *** Guest7223 is now known as mfisch | 15:49 | |
| *** mfisch has quit IRC | 15:49 | |
| *** mfisch has joined #openstack-cinder | 15:49 | |
| hemna | re: multi-attach and a shared target vs individual target | 15:49 |
| jgriffith | hemna: yeah, you'd have thought that was an easy thing to figure out | 15:49 |
| jgriffith | hemna: but apparantly it's not | 15:49 |
| hemna | it is. nova has that data in it's bdm's | 15:50 |
| * smcginnis grumbles about nova keeping track of attachments... | 15:50 | |
| jgriffith | hemna: yeah, so why has this been such a cluster *&^$ all this time then? | 15:50 |
| hemna | I dunno. I raised this at the beginning. | 15:50 |
| *** bardia has joined #openstack-cinder | 15:51 | |
| jgriffith | I'm confused | 15:51 |
| hemna | I think there are 2 issues wrt to shared vs non shared volumes | 15:51 |
| hemna | 1) calling brick on the compute host or not | 15:51 |
| hemna | 2) calling terminate_connection or not | 15:51 |
| hemna | #2 is easy to me. if you call it, cinder terminates. if you don't mean to....don't. | 15:52 |
| hemna | #1 the data is there in the nova db. it should make sure it does the right thing....I guess it's just like #2 in that respect | 15:52 |
| jgriffith | hemna: well, I've been saying #2 for a couple years now but that doesn't seem to resonate | 15:53 |
| *** harlowja_at_home has joined #openstack-cinder | 15:53 | |
| jgriffith | hemna: an in fact you submitted a patch to Cinder that puts that in Cinders control | 15:53 |
| jgriffith | hemna: so I'm really kinda confused now | 15:53 |
| hemna | ok crap, I didn't mean to confuse anyone | 15:54 |
| jgriffith | haha | 15:54 |
| hemna | :( | 15:54 |
| jgriffith | hemna: no, don't worry about that | 15:54 |
| jgriffith | better to talk through this stuff | 15:54 |
| hemna | I do like the new API though | 15:54 |
| hemna | I think we should be saving the connector in the attachments | 15:55 |
| hemna | that's good stuffs | 15:55 |
| jgriffith | It's just I'm not getting the impression from you that this all a big waste of time :) | 15:55 |
| jgriffith | hemna: oh, well yeah for sure! | 15:55 |
| jgriffith | hemna: I'm doing that no matter what | 15:55 |
| hemna | and I do like being explicit and passing around the attachment id. | 15:55 |
| hemna | but honestly, if all we end up doing is saving the connector out of all of this | 15:56 |
| hemna | it wasn't a waste of time | 15:56 |
| jgriffith | hemna: so if nothing else then we'll just call this a "cleanup" | 15:56 |
| jgriffith | nothing more, nothing less :) | 15:56 |
| hemna | even an exercise that leads us to the same api....isn't a waste of time. | 15:56 |
| *** tesseract- has quit IRC | 15:56 | |
| hemna | but I think the new API is a better approach IMO | 15:57 |
| jgriffith | hemna: let me finish coding up this other option a bit and post it. I'd like to get your thoughts on it | 15:58 |
| *** mriedem1 has joined #openstack-cinder | 15:59 | |
| hemna | ok coolio | 15:59 |
| *** mriedem has quit IRC | 16:00 | |
| *** jungleboyj has quit IRC | 16:01 | |
| *** mriedem1 is now known as mriedem | 16:01 | |
| *** chlong has quit IRC | 16:06 | |
| *** sudipto has joined #openstack-cinder | 16:08 | |
| *** sudipto_ has joined #openstack-cinder | 16:08 | |
| *** chris_morrell has joined #openstack-cinder | 16:11 | |
| *** leeantho has joined #openstack-cinder | 16:15 | |
| *** alonmarx has quit IRC | 16:15 | |
| *** jungleboyj has joined #openstack-cinder | 16:17 | |
| *** chris_morrell has quit IRC | 16:22 | |
| *** chris_morrell has joined #openstack-cinder | 16:22 | |
| *** laughterwym has quit IRC | 16:25 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Removing deprecated Dell EqualLogic config options https://review.openstack.org/380869 | 16:26 |
| *** chris_morrell has quit IRC | 16:26 | |
| *** bardia has quit IRC | 16:28 | |
| *** bardia has joined #openstack-cinder | 16:29 | |
| *** sdake has quit IRC | 16:30 | |
| *** merooney has quit IRC | 16:33 | |
| *** sdake has joined #openstack-cinder | 16:34 | |
| *** merooney has joined #openstack-cinder | 16:34 | |
| *** mriedem has quit IRC | 16:35 | |
| *** mriedem has joined #openstack-cinder | 16:38 | |
| *** lpetrut has joined #openstack-cinder | 16:40 | |
| *** harlowja_still_a has joined #openstack-cinder | 16:40 | |
| *** sbezverk has quit IRC | 16:42 | |
| *** harlowja_at_home has quit IRC | 16:44 | |
| *** harlowja_still_a has quit IRC | 16:46 | |
| *** jungleboyj has quit IRC | 16:48 | |
| *** clenimar has joined #openstack-cinder | 16:51 | |
| *** crose has joined #openstack-cinder | 16:54 | |
| *** alonma has joined #openstack-cinder | 16:56 | |
| *** GB21 has joined #openstack-cinder | 16:58 | |
| *** crose has quit IRC | 16:59 | |
| *** jungleboyj has joined #openstack-cinder | 17:00 | |
| *** alonma has quit IRC | 17:00 | |
| *** Poornima has quit IRC | 17:00 | |
| gaurangt | jgriffith: hi | 17:01 |
| jgriffith | gaurangt: hey | 17:01 |
| ildikov | smcginnis: hemna: johnthetubaguy: meeting time if you're around :) | 17:02 |
| *** alonma has joined #openstack-cinder | 17:03 | |
| gaurangt | When a particular host is down (cinder service-list shows the c-vol service as down), one cannot run the failover-host command for that host. It expects c-vol service to be running, right? | 17:03 |
| gaurangt | The reason I'm asking is - GPFS runs c-vol service on multiple hosts which may be failover targets for each other. When one of them is down, c-vol will also be down. But with current implementation, we would not be able to failover the volumes to the other host. | 17:04 |
| jgriffith | gaurangt: correct | 17:05 |
| *** jdurgin1 has quit IRC | 17:05 | |
| jgriffith | gaurangt: well, that seems right to me | 17:05 |
| jgriffith | gaurangt: I mean, how can you fail-over to a service that is "down" ? | 17:05 |
| jgriffith | gaurangt: if you want a new state I guess that could be considered, but I don't see that it's particularly valuable | 17:06 |
| *** merooney_ has joined #openstack-cinder | 17:06 | |
| gaurangt | jgriffith: no, I want to failover to the host which is running. | 17:06 |
| jgriffith | gaurangt: oh | 17:06 |
| jgriffith | gaurangt: sure you can | 17:06 |
| jgriffith | gaurangt: at least you used to be able to, and you *should* be able to | 17:07 |
| *** sudipto has quit IRC | 17:07 | |
| *** sudipto_ has quit IRC | 17:07 | |
| jgriffith | gaurangt: the whole premise was that the initial source died in a fire | 17:07 |
| gaurangt | Let me explain with example. Consider two sites (site A and site B) and we have same GPFS filesystem across two sites and we have cinder configured with GPFS as storage backend. | 17:07 |
| *** sudipto_ has joined #openstack-cinder | 17:08 | |
| *** sudipto has joined #openstack-cinder | 17:08 | |
| gaurangt | Here, cinder volume service running on one of the nodes on each site. | 17:08 |
| *** alonma has quit IRC | 17:08 | |
| jgriffith | gaurangt: so you're not talking about a DR Replication scheme, you're doing a geo-rep | 17:08 |
| gaurangt | jgriffith: cinder service-list shows the c-vol on both the sites. | 17:08 |
| gaurangt | yeah. kind of. GPFS takes care of the replication here. | 17:09 |
| *** merooney has quit IRC | 17:09 | |
| jgriffith | gaurangt: go on... I'm listening :) | 17:10 |
| gaurangt | jgriffith: now, when one of the sites fail, c-vol on that particular site would also be down. The volumes which are managed by that host would no longer be manageable. | 17:10 |
| *** alyson_ has quit IRC | 17:10 | |
| *** alonma has joined #openstack-cinder | 17:10 | |
| jgriffith | yep | 17:11 |
| gaurangt | jgriffith: with failover, we want to change the host parameter for those volumes which are tagged with host on the failed site so that they become manageable. | 17:11 |
| jgriffith | gaurangt: yep | 17:11 |
| *** alyson_ has joined #openstack-cinder | 17:11 | |
| *** salv-orl_ has quit IRC | 17:12 | |
| gaurangt | jgriffith: but current implementation is such that we cannot run failover-host command if the service is down. It would just wait for the c-vol service to come up until driver runs the failover_host method code. | 17:12 |
| jgriffith | gaurangt: I don't think you're correct | 17:12 |
| jgriffith | gaurangt: if there's been something added that does that it's a blatant bug | 17:12 |
| jgriffith | gaurangt: the failover command should not require any interaction with the primary what so ever | 17:13 |
| jgriffith | gaurangt: it's all just db changes/updates | 17:13 |
| gaurangt | jgriffith: Last time when I talked with Patrick East on this, he mentioned that we would need c-vol service running always | 17:14 |
| gaurangt | I've tested it with GPFS. It doesn't work for me unless the host comes back up. | 17:14 |
| *** rushil has quit IRC | 17:14 | |
| jgriffith | gaurangt: than that's a bug IMO | 17:15 |
| *** alonma has quit IRC | 17:15 | |
| jgriffith | gaurangt: I'd have to test it out and make sure I follow exactly, unless I can see something specific in the code | 17:15 |
| gaurangt | jgriffith: yeah sure. | 17:15 |
| jgriffith | gaurangt: do you by chance know why/where it's trying to get to the primary driver and failing? | 17:15 |
| gaurangt | if I understand the code correct, the failover_host method in cinder/volume/api.py makes a call self.volume_rpcapi.failover_host(ctxt, host, secondary_id) | 17:16 |
| gaurangt | jgriffith: where the host is the actual host which is down and needs to be failed over, right? | 17:17 |
| jgriffith | gaurangt: hmm | 17:17 |
| jgriffith | gaurangt: well it's evident this is one of those *features* that has never actually been used apparantly :) | 17:18 |
| gaurangt | jgriffith: now, in cinder/volume/rpcapi.py it has a call "cctxt = self._get_cctxt(host, version)" in failover_host method. | 17:18 |
| jgriffith | gaurangt: I'll have a look at it here shortly and file a bug etc | 17:18 |
| gaurangt | jgriffith: this will only be run on the host which is already down | 17:19 |
| jgriffith | gaurangt: yeah, I see exactly what you're talking about, but that should pass through because we don't do the require_initialized check | 17:19 |
| jgriffith | gaurangt: but I'll have to look. We could/should route it to the secondary we are failing over to, that's how I thought I wrote it originally | 17:19 |
| jgriffith | but apparantly not | 17:19 |
| gaurangt | jgriffith: yeah.. It doesn't work with GPFS at least. I'm not sure if it's the case with other shared filesystem drivers. | 17:20 |
| *** e0ne has joined #openstack-cinder | 17:20 | |
| *** salv-orlando has joined #openstack-cinder | 17:20 | |
| jgriffith | gaurangt: not sure that it would work with anything :) | 17:20 |
| gaurangt | jgriffith: I've anyways put this as an agenda item in upcoming cinder meeting. We can discuss more in that. | 17:20 |
| jgriffith | gaurangt: sounds good, thanks! | 17:20 |
| gaurangt | jgriffith: thanks too. | 17:22 |
| smcginnis | ildikov: Sorry, just got back from another meeting. | 17:22 |
| *** tongli has joined #openstack-cinder | 17:22 | |
| GB21 | Hi everyone, I am working on searchlight and we are implementing cinder plugins, there are some attributes that are missing from certain objects. As requested by the searchlight team earlier, we really need them implemented so that we can advance our project; some blueprints were initiated but they were never finished | 17:27 |
| GB21 | this one is one of them | 17:28 |
| GB21 | https://review.openstack.org/#/c/289658/ | 17:28 |
| mriedem | does anyone know how conditional_updates in the cinder db api are translated to http exceptions in the cinder-api wsgi layer? | 17:28 |
| *** _ducttape_ has joined #openstack-cinder | 17:29 | |
| GB21 | I request someone to please take a look, or maybe guide me to work them out, thanks in advance | 17:29 |
| *** mvk has quit IRC | 17:29 | |
| *** rraja has quit IRC | 17:30 | |
| ildikov | smcginnis: no worries, we're still eagerly in it :) | 17:31 |
| *** Guest92 has joined #openstack-cinder | 17:32 | |
| *** raunak has joined #openstack-cinder | 17:32 | |
| *** ducttape_ has quit IRC | 17:32 | |
| eharney | mriedem: in the code i've seen, failure of conditional_update is manually checked by the caller which then usually raises some type of Invalid* exception which has an HTTP 400 attached to it, so nothing really special for that case | 17:33 |
| mriedem | eharney: ok so unreserve wouldn't actually fail if the update didn't happen https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L632 | 17:34 |
| mriedem | hemna: scottda: ^ | 17:34 |
| eharney | mriedem: right | 17:34 |
| *** gcb has quit IRC | 17:35 | |
| eharney | mriedem: that code should check for the result before logging that info message though | 17:35 |
| *** Guest92 has quit IRC | 17:37 | |
| *** sdake has quit IRC | 17:39 | |
| *** Apoorva has joined #openstack-cinder | 17:43 | |
| *** sdake has joined #openstack-cinder | 17:44 | |
| hemna | eharney, +1 | 17:49 |
| *** salv-orlando has quit IRC | 17:53 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Unreserve volume: fix log message for failure https://review.openstack.org/381262 | 17:54 |
| eharney | fixed there ^ | 17:54 |
| *** jordanP has quit IRC | 17:55 | |
| *** lpetrut has quit IRC | 17:59 | |
| *** sdake has quit IRC | 18:01 | |
| *** GB21 has quit IRC | 18:02 | |
| *** akerr has quit IRC | 18:02 | |
| jgriffith | mriedem: eharney but the unreserve *is* that update, that's all it is. So IIRC that means that it will just leave it at whatever state it's at no? | 18:08 |
| jgriffith | that was the whole point of a conditional-update, only update under conditions x, y, z | 18:09 |
| mriedem | jgriffith: yeah | 18:09 |
| mriedem | it's not an error | 18:09 |
| mriedem | which is why i -1ed | 18:09 |
| mriedem | to log it as an error | 18:09 |
| jgriffith | well, it is an error though because it's not doing what you asked :) | 18:09 |
| eharney | oh, yeah, i agree it shouldn't be logged at error level | 18:10 |
| jgriffith | mriedem: but I get your point | 18:10 |
| mriedem | it's not an error that an operator is going to do anything about though | 18:10 |
| mriedem | if it were an error, you'd return a 400 or 409 to the client | 18:10 |
| jgriffith | mriedem: Like I said, I get your point, not arguing | 18:11 |
| mriedem | i'm sorry i can't accept not arguing | 18:11 |
| * hemna gets out the popcorn. | 18:11 | |
| jgriffith | mriedem: yes you can!!! | 18:12 |
| jgriffith | mriedem: WRONG... WRONG... that's WRONG | 18:13 |
| hemna | you are both wrong | 18:13 |
| eharney | hemna: disagree, i wanted pretzels for this | 18:13 |
| jgriffith | hemna: right, but you're wrong | 18:13 |
| hemna | you are right, but it doesn't mean you aren't wrong | 18:14 |
| gaurangt | jgriffith: did you get a chance to test replication code in your setup? Any findings? | 18:16 |
| jgriffith | hemna: you're wrong, and that is certainly right | 18:16 |
| jgriffith | gaurangt: LOL | 18:16 |
| jgriffith | gaurangt: you don't know me very well do you :) | 18:16 |
| smcginnis | Quote of the day: "you are right, but it doesn't mean you aren't wrong" :D | 18:16 |
| gaurangt | jgriffith: :) | 18:17 |
| jgriffith | gaurangt: firing up a fresh stack now | 18:17 |
| gaurangt | jgriffith: okay | 18:17 |
| *** _ducttape_ has quit IRC | 18:20 | |
| patrickeast | jgriffith: gaurangt: i'm just catching up on irc backlog from this morning, but from the description i think that is the expected behavior, we send the failover call to the c-vol service that is being failed over | 18:20 |
| patrickeast | since cinder doesn't keep a like mapping from replication device -> cinder backend | 18:21 |
| jgriffith | patrickeast: we shouldn't require that though | 18:21 |
| *** ducttape_ has joined #openstack-cinder | 18:21 | |
| patrickeast | yea sure | 18:21 |
| patrickeast | but we don't support it today | 18:21 |
| *** sudipto_ has quit IRC | 18:21 | |
| *** sudipto has quit IRC | 18:21 | |
| jgriffith | patrickeast: well then I should fix that :) | 18:21 |
| patrickeast | i think right now its built on the assumption that you run HA c-vols (hopefully active-passive) | 18:21 |
| jgriffith | patrickeast: right, but the reason you *need* replication in our scenario is that if your backend takes a crap | 18:22 |
| *** gaurangt has quit IRC | 18:22 | |
| *** kaisers_ has quit IRC | 18:22 | |
| *** Guest92 has joined #openstack-cinder | 18:22 | |
| jgriffith | patrickeast: that was kinda the whole point, and the entire use case | 18:22 |
| jgriffith | patrickeast: there's nothing to keep a driver from acting on a call when a service is down | 18:23 |
| jgriffith | it's | 18:23 |
| jgriffith | patrickeast: the actual work in the driver (in my case at least) has nothing to do with that actual service actually (or the primary backend) | 18:23 |
| jgriffith | patrickeast: it just updates the driver to point to another backend, then the failover completes and the service should be back UP again | 18:24 |
| patrickeast | jgriffith: right | 18:25 |
| patrickeast | jgriffith: sorry got side tracked with someone else at my desk | 18:25 |
| patrickeast | jgriffith: so like the model we have now is for failing over/replicating remote backends with remote c-vol services managing them | 18:25 |
| *** laughterwym has joined #openstack-cinder | 18:25 | |
| *** Yogi1 has joined #openstack-cinder | 18:26 | |
| patrickeast | jgriffith: we just didn't add in code for the scenario of c-vol being the cinder service *and* storage backend you might want to failover | 18:26 |
| patrickeast | jgriffith: everyone who has implemented replication v2.x works on the assumption that c-vol is highly available and the driver for that cinder backend can reach both replicated devices *somehow* | 18:27 |
| *** porrua has quit IRC | 18:28 | |
| jgriffith | patrickeast: right so what I was getting at with gaurangt is that you just make your config different | 18:28 |
| jgriffith | patrickeast: so don't say "failover to another cinder", just failover to another backend | 18:28 |
| jgriffith | patrickeast: ignore the fact that it just so happens to be a cinder-service | 18:28 |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Removing deprecated QoS specs. https://review.openstack.org/370816 | 18:29 |
| *** pcaruana has joined #openstack-cinder | 18:29 | |
| patrickeast | jgriffith: not sure i follow that part | 18:30 |
| *** laughterwym has quit IRC | 18:30 | |
| patrickeast | jgriffith: you mean failover to another backend that is the same cinder backend? like ala A-P HA? | 18:30 |
| patrickeast | thats part of this that confuses me too, since iiuc that backend is basically an NFS share mounted on the c-vol host, so why you cant just mount it elsewhere and keep going when the cinder controller dies doesn't make sense to me | 18:31 |
| *** rcernin has joined #openstack-cinder | 18:31 | |
| *** alonma has joined #openstack-cinder | 18:32 | |
| jgriffith | patrickeast: yeah, I don't know | 18:32 |
| jgriffith | patrickeast: the whole gpfs/nfs model is completely unique and foreign to me | 18:32 |
| patrickeast | jgriffith: ditto | 18:32 |
| patrickeast | but i wanna help them replicate if they want to :D | 18:32 |
| jgriffith | patrickeast: in his/her case I don't even know what it means for the "bakcend to fail" | 18:32 |
| jgriffith | patrickeast: LOL | 18:33 |
| patrickeast | hah | 18:33 |
| hemna | patrickeast, but failover in that case isn't the same model as replication that we've been thinking of for Cinder | 18:33 |
| hemna | where the storage backend itself pukes | 18:33 |
| patrickeast | hemna: exactly | 18:33 |
| jgriffith | hemna: wrong! | 18:33 |
| jgriffith | hemna: no wait... *right* | 18:34 |
| hemna | damn | 18:34 |
| hemna | :P | 18:34 |
| *** alonma has quit IRC | 18:36 | |
| *** ducttape_ has quit IRC | 18:38 | |
| *** gaurangt has joined #openstack-cinder | 18:38 | |
| jgriffith | patrickeast: oh well... I just tested it on my side and it still works as I thought it did so I'm good at least :) | 18:38 |
| *** harlowja has quit IRC | 18:38 | |
| jgriffith | patrickeast: so I just "unplug" the primary backend, wait a while, then issue the failover command | 18:38 |
| gaurangt | jgriffith: sorry, I got disconnected. | 18:39 |
| *** harlowja has joined #openstack-cinder | 18:39 | |
| jgriffith | gaurangt: hey.. there you are | 18:39 |
| jgriffith | gaurangt: so it's quite possible I just don't understand what you're trying to achieve | 18:39 |
| jgriffith | gaurangt: I simulated my setup by setting things up, then going and shutting down the primary backend | 18:40 |
| jgriffith | gaurangt: waited a bit, then issued the failover cmd | 18:40 |
| jgriffith | gaurangt: it worked on my side | 18:40 |
| gaurangt | jgriffith: is c-vol service common for both the backends? | 18:40 |
| jgriffith | gaurangt: I don't know what that means | 18:41 |
| jgriffith | gaurangt: that's an a/a setup | 18:41 |
| jgriffith | gaurangt: completely different deal | 18:41 |
| gaurangt | jgriffith: yeah..I think for any clustered filesystem, it would be the same case (a/a setup). | 18:42 |
| jgriffith | gaurangt: I have a single backend and a config that looks like this: http://paste.openstack.org/show/584066/ | 18:42 |
| jgriffith | gaurangt: sure, but then what are you doing replication for? | 18:42 |
| gaurangt | the data is being replicated on both the sites from GPFS perspective. | 18:43 |
| jgriffith | gaurangt: well... what I mean is, you have an Active/Active setup doing replication between your GPFS clusters right? | 18:43 |
| jgriffith | actually, I guess I can look at your driver... just a sec | 18:44 |
| gaurangt | for GPFS, cinder volumes are nothing but files on the filesystem. | 18:44 |
| jgriffith | gaurangt: understood | 18:44 |
| jgriffith | gaurangt: oh, you don't have any code in github for that | 18:45 |
| jgriffith | never mind, I can't look there :) | 18:45 |
| *** abhi has joined #openstack-cinder | 18:45 | |
| gaurangt | jgriffith: GPFS takes care of replication if filesystem is set accordingly. | 18:45 |
| jgriffith | gaurangt: so why do you need cinders replcation then? | 18:46 |
| smcginnis | gaurangt: So is there even a need to use Cinder replication? | 18:46 |
| smcginnis | jgriffith: jynx | 18:46 |
| jgriffith | smcginnis: :) | 18:46 |
| jgriffith | gaurangt: are you trying to solve the problem of the c-vol node blowing up? | 18:46 |
| gaurangt | jgriffith: yep | 18:47 |
| jgriffith | gaurangt: ok that's different | 18:47 |
| jgriffith | gaurangt: cinders replication doesn't do anything for you | 18:48 |
| gaurangt | jgriffith: ok. So you mean the current replication use case is different then. | 18:48 |
| jgriffith | gaurangt: I guess maybe that's what patrickeast 's comment was and that he already know what you were doing | 18:48 |
| jgriffith | gaurangt: completely different | 18:48 |
| jgriffith | gaurangt: it's replicating a backend device | 18:48 |
| jgriffith | gaurangt: not the volume service | 18:48 |
| gaurangt | jgriffith: what might be the correct way to address the problem in my case then? | 18:49 |
| jgriffith | gaurangt: you can make that work as is today though, just don't use replication | 18:49 |
| jgriffith | gaurangt: just run another cvol-service with the same name/topic etc | 18:49 |
| gaurangt | same host name? | 18:49 |
| jgriffith | gaurangt: then whichever one is up/available will pull requests from the queue | 18:49 |
| jgriffith | gaurangt: yes, same host-name | 18:49 |
| jgriffith | gaurangt: spoof it in your c-vol settings | 18:49 |
| jgriffith | gaurangt: you basically just lie | 18:50 |
| jgriffith | gaurangt: make the controller think there's only one c-vol service out there | 18:50 |
| gaurangt | yeah.. have same host entry in both cinder.conf | 18:50 |
| jgriffith | gaurangt:yes | 18:50 |
| gaurangt | jgriffith: got you. | 18:50 |
| jgriffith | gaurangt: You will have some different challenges than I do or my customers do probably | 18:51 |
| gaurangt | jgriffith: challenges like? | 18:51 |
| jgriffith | gaurangt: in my case it sort of relies on making sure people aren't *trying* to break things | 18:51 |
| jgriffith | gaurangt: given you're in a public cloud (at least I assume) you have to be a bit more dilligent | 18:51 |
| gaurangt | jgriffith: can it lead to db corruption or so? | 18:51 |
| jgriffith | gaurangt: but I don't know that it even matters with gpfs | 18:52 |
| jgriffith | gaurangt: so there are some races | 18:52 |
| jgriffith | gaurangt: I don't know your device well enough to really advise you on any of that | 18:53 |
| jgriffith | gaurangt: you would have to think about scenarios where different backends respond to a request and how quickly they are synced up | 18:53 |
| gaurangt | jgriffith: well, I've tested gpfs driver with same host entry with multiple c-vol services. It had worked earlier without any problems. | 18:54 |
| jgriffith | gaurangt: yeah, I've talked to quite a few people using that trick over the last few years | 18:54 |
| jgriffith | gaurangt: for some devices it "just works" | 18:55 |
| jgriffith | gaurangt: it all depends though on how the backend works | 18:55 |
| gaurangt | jgriffith: what I was trying to propose is - have a backup for each c-vol service configured in cinder.conf and when we issue cinder failover_host, we change the host entry for the volumes on the failed host. | 18:55 |
| gaurangt | so, it changes the host attribute for the volumes which are tied to failed host. | 18:56 |
| gaurangt | jgriffith: that would do it gracefully, IMO. | 18:56 |
| gaurangt | do you agree? | 18:56 |
| *** raunak has quit IRC | 18:57 | |
| *** ducttape_ has joined #openstack-cinder | 18:57 | |
| jgriffith | gaurangt: well that's active/passive which there are already common solutions for using pacemaker, zmq etc | 18:58 |
| jgriffith | no? | 18:58 |
| gaurangt | jgriffith: hmm | 18:58 |
| gaurangt | jgriffith: consider a case where we have two controller nodes across two sites (FYI, we run all cinder services on controller node) accessing the same common database. | 19:00 |
| jgriffith | gaurangt: If you want to talk A/A you should hit up geguileo, he's the expert there and in the process of doing a ton of work to make that more robust | 19:00 |
| *** hoonetorg has quit IRC | 19:00 | |
| gaurangt | so, users on each site would access cinder endpoint on their site respectively. | 19:00 |
| jgriffith | geguileo: yes, I have customers that do that today | 19:00 |
| jgriffith | geguileo: but I do it by replicating in the background, without Cinder or any OpenStack knowledge/involvement | 19:01 |
| jgriffith | errr.. sorry geguileo | 19:01 |
| geguileo | jgriffith: np | 19:01 |
| jgriffith | I meant that ^^ for gaurangt | 19:01 |
| gaurangt | jgriffith: if site A goes down, then the volumes created from Site A host would not be manageable, right? | 19:02 |
| jgriffith | gaurangt: frankly it's MUCH better that way | 19:02 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Error message for image conversion failure https://review.openstack.org/381272 | 19:02 |
| jgriffith | gaurangt: incorrect | 19:02 |
| gaurangt | jgriffith: how? | 19:02 |
| jgriffith | gaurangt: site A creates a volume.... in the background that volume is duplicated/presented/replicated at site B | 19:02 |
| jgriffith | gaurangt: you can't access site-b from site-a's cinder though if that's what you mean | 19:03 |
| gaurangt | jgriffith: can I fire a command from site B and delete the volume then? | 19:03 |
| jgriffith | gaurangt: yes | 19:03 |
| jgriffith | gaurangt: it's deleted at site-b, then the backend syncing deletes it at site-a | 19:03 |
| jgriffith | gaurangt: BUT | 19:04 |
| gaurangt | jgriffith: I don't think so. It waits till the c-vol at site A comes back up. | 19:04 |
| jgriffith | gaurangt: you don't think what? | 19:04 |
| gaurangt | jgriffith: cinder list will show the volume state as deleting. | 19:04 |
| jgriffith | gaurangt: and? | 19:05 |
| jgriffith | gaurangt: I'm honestly unclear what you're looking for here, sorry. | 19:05 |
| gaurangt | jgriffith: unless c-vol service comes back up on site A, the volume doesn't get deleted. | 19:05 |
| jgriffith | gaurangt: I'm just saying I've helped people build mirroed sites | 19:05 |
| jgriffith | gaurangt: they run a daemon that detects changes in cinder and issues replication commands to the remote site | 19:06 |
| gaurangt | jgriffith: I'm talking from GPFS point of view here. | 19:06 |
| jgriffith | gaurangt: I don't know shit about GPFS | 19:06 |
| jgriffith | gaurangt: so I'm not really going to be very helpful in that context probably | 19:06 |
| *** haplo37__ has quit IRC | 19:06 | |
| jgriffith | gaurangt: but I don't think the principles are that much different | 19:07 |
| jgriffith | anyway | 19:07 |
| jgriffith | sounds like you have a unique use case, that we don't currently cover | 19:07 |
| jgriffith | gaurangt: you should write up a spec and propose it perhaps | 19:07 |
| gaurangt | jgriffith: yeah, but I think this might be the case with all shared filesystems. | 19:08 |
| jgriffith | gaurangt: or figure out how to write your own custom daemon if it's not something that will translate to anything but gpfs | 19:08 |
| jgriffith | gaurangt: you should use manila then :) | 19:08 |
| jgriffith | gaurangt: sorry... that was a joke | 19:08 |
| jgriffith | gaurangt: I realize that's a different problem to solve | 19:08 |
| gaurangt | jgriffith: :) | 19:08 |
| *** lpetrut has joined #openstack-cinder | 19:08 | |
| jgriffith | gouthamr: or I should say "solves a different problem" | 19:08 |
| jgriffith | grrrrr | 19:08 |
| jgriffith | too many people with nics starting with 'g' | 19:09 |
| gaurangt | jgriffith: haha | 19:09 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Unreserve volume: fix log message for failure https://review.openstack.org/381262 | 19:09 |
| *** merooney_ has quit IRC | 19:10 | |
| gaurangt | jgriffith: I think for now faking host identity with same hostname solves the problem for GPFS. We can improve it a bit with what I am thinking. | 19:10 |
| *** gaurangt has left #openstack-cinder | 19:14 | |
| *** gaurangt has joined #openstack-cinder | 19:14 | |
| jgriffith | gaurangt: yeah... FWIW in the replication work we did initially have an idea/approach to try and do the failover to another service that would work well in your case | 19:15 |
| jgriffith | gaurangt: we decided to start simple though, and cut that out. I believe it was part of rep V2 (not V2.1) if you wanted to dig it up and have a look | 19:15 |
| jgriffith | and on that note, I'm going to lunch, pretty much blew my morning :( | 19:16 |
| jgriffith | too many phone calls and meetings, and I talk too much :) | 19:16 |
| gaurangt | jgriffith: sorry about that !! | 19:16 |
| jgriffith | gaurangt: not at all... wasn't referring to you | 19:17 |
| jgriffith | I've had people at my desk, asked to go to meeting and had a couple phone calls | 19:17 |
| jgriffith | gaurangt: at least this is a topic I enjoy :) | 19:17 |
| jgriffith | gaurangt: the other ones... not so much | 19:17 |
| gaurangt | jgriffith: Oh ok.. But it was good discussion ..at least I won't too much time supporting current replication spec with GPFS :) | 19:18 |
| gaurangt | spend* | 19:18 |
| *** catintheroof has joined #openstack-cinder | 19:22 | |
| *** merooney has joined #openstack-cinder | 19:24 | |
| *** kfarr has joined #openstack-cinder | 19:25 | |
| *** gaurangt has left #openstack-cinder | 19:26 | |
| *** hoonetorg has joined #openstack-cinder | 19:27 | |
| *** Guest92 has quit IRC | 19:27 | |
| *** Guest92 has joined #openstack-cinder | 19:27 | |
| *** Guest92 has quit IRC | 19:27 | |
| *** salv-orlando has joined #openstack-cinder | 19:30 | |
| * gouthamr thinks jgriffith though of manila and replication and tagged the right guy | 19:41 | |
| gouthamr | s/though/thought | 19:41 |
| *** salv-orl_ has joined #openstack-cinder | 19:44 | |
| *** Lee1092 has quit IRC | 19:45 | |
| *** david-lyle has joined #openstack-cinder | 19:47 | |
| *** salv-orlando has quit IRC | 19:47 | |
| jgriffith | gouthamr: ha... lucky coincidence :) | 19:47 |
| gouthamr | jgriffith: :) | 19:48 |
| *** jungleboyj has quit IRC | 19:50 | |
| *** bjolo has quit IRC | 19:51 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: WIP: New test for long volume names https://review.openstack.org/381287 | 19:57 |
| *** raunak has joined #openstack-cinder | 19:58 | |
| *** enriquetaso has quit IRC | 20:01 | |
| *** Cibo has joined #openstack-cinder | 20:03 | |
| *** e0ne has quit IRC | 20:05 | |
| *** sdake has joined #openstack-cinder | 20:05 | |
| openstackgerrit | Merged openstack/cinder: Fix pep8 E501 line too long https://review.openstack.org/375248 | 20:05 |
| *** Suyi has joined #openstack-cinder | 20:05 | |
| *** kaisers_ has joined #openstack-cinder | 20:11 | |
| *** Cibo has quit IRC | 20:15 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Cleanup RCP API versioning https://review.openstack.org/375137 | 20:15 |
| *** Apoorva has quit IRC | 20:15 | |
| *** kaisers_ has quit IRC | 20:15 | |
| openstackgerrit | Merged openstack/cinder: Fix a typo in manager.py,test_common.py and emc_vmax_utils.py https://review.openstack.org/379048 | 20:20 |
| openstackgerrit | Merged openstack/cinder: Fix typo: remove redundant 'that' https://review.openstack.org/380941 | 20:21 |
| *** Yogi1 has quit IRC | 20:22 | |
| *** Apoorva has joined #openstack-cinder | 20:23 | |
| *** lpetrut has quit IRC | 20:23 | |
| openstackgerrit | Merged openstack/cinder: Fix backup NFS share mount with default backup_mount_options https://review.openstack.org/351890 | 20:26 |
| *** Apoorva has quit IRC | 20:28 | |
| *** catintheroof has quit IRC | 20:29 | |
| *** david-lyle has quit IRC | 20:30 | |
| *** asselin__ has joined #openstack-cinder | 20:34 | |
| *** david-lyle has joined #openstack-cinder | 20:35 | |
| *** asselin_ has quit IRC | 20:35 | |
| *** alonmarx has joined #openstack-cinder | 20:36 | |
| *** alonma has joined #openstack-cinder | 20:40 | |
| *** alonma has quit IRC | 20:44 | |
| *** david-lyle has quit IRC | 20:44 | |
| *** pcaruana has quit IRC | 20:45 | |
| *** alonma has joined #openstack-cinder | 20:47 | |
| *** ccesario has quit IRC | 20:48 | |
| *** alonma has quit IRC | 20:51 | |
| *** jwcroppe has quit IRC | 20:58 | |
| *** jwcroppe has joined #openstack-cinder | 20:59 | |
| *** jwcroppe has quit IRC | 20:59 | |
| jgriffith | :q | 20:59 |
| *** jwcroppe has joined #openstack-cinder | 20:59 | |
| hemna | has anyone converted their driver to use the ovo volume object? | 21:04 |
| hemna | I'm having a problem with unit tests for fake volumes | 21:04 |
| smcginnis | jgriffith: You can't quit. | 21:04 |
| jgriffith | smcginnis: apparently not :) | 21:05 |
| jgriffith | smcginnis: at least not in IRC window | 21:05 |
| smcginnis | jgriffith: Trust me, I've tried many times. | 21:05 |
| jgriffith | smcginnis: haha | 21:05 |
| smcginnis | jgriffith: And chromium tabs don't work either. | 21:05 |
| jgriffith | smcginnis: pro-tip of the day! | 21:05 |
| * smcginnis wonders how long it would take to write an extension... | 21:05 | |
| jgriffith | smcginnis: if you want to write a really super useful extension, make the gerrit review page searchable again for me :) | 21:06 |
| jgriffith | smcginnis: I'll pay you! | 21:06 |
| smcginnis | Hah! | 21:06 |
| smcginnis | Now that would be funny, and extension that scrapes the page and shows it in the old format! :) | 21:06 |
| jgriffith | smcginnis: well I don't care how it's implemented :) I just want to be able to search again | 21:07 |
| smcginnis | jgriffith: You mean in the side by side diff window? | 21:07 |
| jgriffith | smcginnis: it's rather annoying that to do a proper review I have to download the patch | 21:07 |
| jgriffith | smcginnis: yeah... is there a trick maybe that I'm missing? | 21:07 |
| smcginnis | jgriffith: You might like this :) | 21:07 |
| smcginnis | jgriffith: Click on the gear in the top left corner. | 21:08 |
| jgriffith | don't say gerrty, don't say gertty.... | 21:08 |
| smcginnis | On the Diff Preferences window, change Render to Slow. | 21:08 |
| smcginnis | No! :D | 21:08 |
| jgriffith | smcginnis: get the heck out... changing render to slow fixes it? | 21:08 |
| eharney | doesn't hitting ctrl-F twice do it too? | 21:08 |
| smcginnis | jgriffith: Cuz why wouldn't you want to render something fast. | 21:08 |
| jgriffith | eharney: I dunno but I'm about to find out | 21:08 |
| smcginnis | eharney: Haven't tried that, but I think it only loads portions of the page at a time so it might not. | 21:09 |
| smcginnis | In fast mode that is. | 21:09 |
| smcginnis | Slow mode also gets rid of the annoying jumping around that you get sometimes when you scroll through the diff. | 21:09 |
| jgriffith | eharney: smcginnis you both win a prize from me | 21:09 |
| smcginnis | Woot | 21:09 |
| eharney | i wrote this thing just so i could read the reviewers list: https://userstyles.org/styles/125355/ | 21:09 |
| jgriffith | eharney: oh nice! | 21:10 |
| jgriffith | eharney: you've been holding out all this time! | 21:10 |
| smcginnis | eharney: I still have your spec on installed. Not sure if that even does anything now. | 21:11 |
| eharney | smcginnis: i think not | 21:11 |
| smcginnis | eharney: Ooo, I like | 21:12 |
| smcginnis | It's at least greying out some of the CIs. :) | 21:12 |
| eharney | smcginnis: yeah, i was hesitant to blacklist people who used their own email address for them, but i'm probably going to eventually | 21:12 |
| smcginnis | eharney: Does green mean core? | 21:13 |
| eharney | smcginnis: well, it's supposed to mean they have +W rights (that's what the CSS checks) | 21:13 |
| smcginnis | Ah, OK. | 21:13 |
| eharney | smcginnis: but it occasionally turns other random people green for people that make no sense to me (silly web nonsense) | 21:13 |
| eharney | for reasons that make no sense to me* | 21:13 |
| *** edmondsw has quit IRC | 21:14 | |
| eharney | it's like 95% correct though, good enough :P | 21:14 |
| jgriffith | eharney: smcginnis well you've both made me very happy today FWIW | 21:14 |
| jgriffith | eharney: smcginnis I know not much, but still | 21:14 |
| jgriffith | good for me anyway | 21:14 |
| smcginnis | jgriffith: It's the little things... | 21:14 |
| jgriffith | smcginnis: it truly is | 21:15 |
| jgriffith | hemna: so it looks like brick is doing it's own magic to determine if/when to delete an entry for an attachment on it's own already? | 21:15 |
| jgriffith | hemna: or am I just not finding the right code somewhere? | 21:16 |
| jgriffith | hemna: ahh... wait, I see nova is checking the lun numbers for shared target case | 21:16 |
| eharney | jgriffith: good :) | 21:17 |
| hemna | brick doesn't know anything about shared vs non shared | 21:18 |
| *** lpetrut has joined #openstack-cinder | 21:18 | |
| hemna | it just tries to remove whatever is passed into it | 21:18 |
| *** openstackgerrit has quit IRC | 21:19 | |
| *** david-lyle has joined #openstack-cinder | 21:19 | |
| *** openstackgerrit has joined #openstack-cinder | 21:20 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Removing deprecated QoS specs. https://review.openstack.org/370816 | 21:20 |
| *** auggy has joined #openstack-cinder | 21:23 | |
| *** eharney has quit IRC | 21:25 | |
| *** Apoorva has joined #openstack-cinder | 21:25 | |
| *** hoonetorg has quit IRC | 21:26 | |
| *** Apoorva has quit IRC | 21:30 | |
| *** diogogmt has quit IRC | 21:32 | |
| *** hoonetorg has joined #openstack-cinder | 21:32 | |
| *** mvk has joined #openstack-cinder | 21:32 | |
| *** Apoorva has joined #openstack-cinder | 21:34 | |
| *** akshai has quit IRC | 21:36 | |
| *** cknight has quit IRC | 21:39 | |
| *** lprice1 has quit IRC | 21:42 | |
| *** dave-mccowan has quit IRC | 21:45 | |
| *** alonmarx has quit IRC | 21:48 | |
| *** ducttape_ has quit IRC | 21:49 | |
| *** gouthamr has quit IRC | 21:49 | |
| *** leeantho has quit IRC | 21:56 | |
| *** xyang1 has quit IRC | 22:00 | |
| *** kaisers_ has joined #openstack-cinder | 22:00 | |
| *** kaisers_ has quit IRC | 22:05 | |
| *** lpetrut has quit IRC | 22:05 | |
| *** david-lyle has quit IRC | 22:09 | |
| *** jwcroppe has quit IRC | 22:10 | |
| *** akshai has joined #openstack-cinder | 22:12 | |
| *** mriedem has quit IRC | 22:13 | |
| *** jwcroppe has joined #openstack-cinder | 22:14 | |
| *** sdake has quit IRC | 22:16 | |
| *** jwcroppe has quit IRC | 22:19 | |
| *** rcernin has quit IRC | 22:25 | |
| *** gcb has joined #openstack-cinder | 22:27 | |
| *** salv-orlando has joined #openstack-cinder | 22:28 | |
| *** salv-orl_ has quit IRC | 22:28 | |
| *** oak has quit IRC | 22:31 | |
| *** oak has joined #openstack-cinder | 22:32 | |
| *** kfarr has quit IRC | 22:33 | |
| *** sdake has joined #openstack-cinder | 22:34 | |
| *** akshai has quit IRC | 22:36 | |
| *** david-lyle has joined #openstack-cinder | 22:36 | |
| *** asselin__ has quit IRC | 22:42 | |
| *** lprice has joined #openstack-cinder | 22:46 | |
| *** lprice has quit IRC | 22:51 | |
| *** salv-orlando has quit IRC | 22:55 | |
| *** alonma has joined #openstack-cinder | 22:55 | |
| *** tongli has quit IRC | 22:57 | |
| *** alyson_ has quit IRC | 22:59 | |
| *** alonma has quit IRC | 23:00 | |
| *** Suyi has quit IRC | 23:01 | |
| *** asselin__ has joined #openstack-cinder | 23:02 | |
| *** lprice has joined #openstack-cinder | 23:14 | |
| *** tpsilva has quit IRC | 23:14 | |
| *** diablo_rojo has quit IRC | 23:19 | |
| *** mtanino has quit IRC | 23:22 | |
| *** rajinir has quit IRC | 23:25 | |
| *** laughterwym has joined #openstack-cinder | 23:27 | |
| *** kaisers1 has quit IRC | 23:29 | |
| *** kaisers has quit IRC | 23:29 | |
| *** markvoelker has quit IRC | 23:29 | |
| *** kaisers has joined #openstack-cinder | 23:29 | |
| *** kaisers1 has joined #openstack-cinder | 23:30 | |
| *** laughterwym has quit IRC | 23:31 | |
| openstackgerrit | Abhilash Divakaran proposed openstack/cinder: Fix for Tegile driver failing to establish volume connection https://review.openstack.org/381341 | 23:34 |
| *** jamielennox is now known as jamielennox|away | 23:35 | |
| *** merooney has quit IRC | 23:41 | |
| *** david-lyle has quit IRC | 23:44 | |
| *** cj has joined #openstack-cinder | 23:46 | |
| cj | o/ | 23:46 |
| *** jamielennox|away is now known as jamielennox | 23:48 | |
| *** kaisers_ has joined #openstack-cinder | 23:49 | |
| *** kaisers_ has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!