| *** markvoelker has quit IRC | 00:03 | |
| *** ho has joined #openstack-cinder | 00:17 | |
| *** emagana has joined #openstack-cinder | 00:23 | |
| *** emagana has quit IRC | 00:29 | |
| *** emagana has joined #openstack-cinder | 00:29 | |
| *** emagana has quit IRC | 00:35 | |
| *** rongze has joined #openstack-cinder | 00:37 | |
| *** rongze has quit IRC | 00:42 | |
| *** markvoelker has joined #openstack-cinder | 01:00 | |
| *** markvoelker has quit IRC | 01:04 | |
| *** annashen has joined #openstack-cinder | 01:05 | |
| *** diegows has quit IRC | 01:15 | |
| *** emagana has joined #openstack-cinder | 01:18 | |
| *** ebalduf has joined #openstack-cinder | 01:20 | |
| *** emagana has quit IRC | 01:23 | |
| *** ebalduf has quit IRC | 01:24 | |
| *** annashen has quit IRC | 01:25 | |
| *** ebalduf has joined #openstack-cinder | 01:29 | |
| *** rongze has joined #openstack-cinder | 01:38 | |
| *** paul-- has quit IRC | 01:39 | |
| *** ebalduf has quit IRC | 01:40 | |
| *** rongze has quit IRC | 01:43 | |
| *** dimsum__ has quit IRC | 01:46 | |
| *** Mandell has joined #openstack-cinder | 01:52 | |
| *** julim_ has quit IRC | 01:55 | |
| *** markvoelker has joined #openstack-cinder | 02:00 | |
| *** archers has joined #openstack-cinder | 02:01 | |
| *** markvoelker has quit IRC | 02:05 | |
| *** archers has quit IRC | 02:08 | |
| *** emagana has joined #openstack-cinder | 02:12 | |
| *** Mandell has quit IRC | 02:12 | |
| *** emagana has quit IRC | 02:17 | |
| Liu | . | 02:34 |
|---|---|---|
| *** rongze has joined #openstack-cinder | 02:39 | |
| *** rongze has quit IRC | 02:44 | |
| *** dimsum__ has joined #openstack-cinder | 02:46 | |
| *** annashen has joined #openstack-cinder | 02:51 | |
| *** dimsum__ has quit IRC | 02:52 | |
| *** emagana has joined #openstack-cinder | 02:54 | |
| *** emagana has quit IRC | 02:59 | |
| *** chlong has joined #openstack-cinder | 03:00 | |
| *** vilobhmm1 has joined #openstack-cinder | 03:01 | |
| *** markvoelker has joined #openstack-cinder | 03:01 | |
| *** markvoelker has quit IRC | 03:06 | |
| *** rongze has joined #openstack-cinder | 03:07 | |
| *** vilobhmm11 has joined #openstack-cinder | 03:15 | |
| *** Mandell has joined #openstack-cinder | 03:16 | |
| *** vilobhmm1 has quit IRC | 03:18 | |
| *** emagana has joined #openstack-cinder | 03:23 | |
| jbernard | jgriffith: sort of | 03:28 |
| jbernard | jgriffith: im still not sure where things are going wrong, but cinder appears to be functioning correctly (or at least consistently) | 03:28 |
| *** annashen has quit IRC | 03:29 | |
| *** coolsvap|afk is now known as coolsvap | 03:29 | |
| *** emagana has quit IRC | 03:32 | |
| *** changbl has joined #openstack-cinder | 03:32 | |
| *** annashen has joined #openstack-cinder | 03:34 | |
| *** archers has joined #openstack-cinder | 03:45 | |
| *** rongze has quit IRC | 03:47 | |
| *** archers has quit IRC | 03:47 | |
| *** rongze has joined #openstack-cinder | 03:55 | |
| *** annashen has quit IRC | 04:01 | |
| *** markvoelker has joined #openstack-cinder | 04:02 | |
| *** markvoelker has quit IRC | 04:07 | |
| *** rongze has quit IRC | 04:18 | |
| *** changbl has quit IRC | 04:40 | |
| *** rongze has joined #openstack-cinder | 04:52 | |
| *** annashen has joined #openstack-cinder | 05:02 | |
| *** markvoelker has joined #openstack-cinder | 05:03 | |
| *** zerda has joined #openstack-cinder | 05:03 | |
| *** anshul has joined #openstack-cinder | 05:04 | |
| zerda | hemnafk, hello, I wonder is there any plan to resurrect HP MSA driver for cinder removed in Kilo? | 05:05 |
| *** changbl has joined #openstack-cinder | 05:05 | |
| thingee | DuncanT: is the huawei ci log links working for you? | 05:06 |
| thingee | DuncanT: I'm trying to verify things, but none of the links have been loading for me | 05:06 |
| Liu | I am the for the Huawei Volume CI | 05:07 |
| thingee | Hi Liu none of the links are currently loading for me. | 05:07 |
| thingee | I've tried some of the reviews that have been reviewed by the huawei ci | 05:07 |
| Liu | Please make difference with huawei ci and huawei-volume-ci | 05:07 |
| Liu | I just for huawei-volume-ci not for huawei-vi | 05:08 |
| *** annashen has quit IRC | 05:08 | |
| Liu | hauwei-volume-ci is ok | 05:08 |
| thingee | Liu: right, that's not the problem. | 05:08 |
| thingee | Liu: none of the logs posted to these reviews are working for me http://lists.openstack.org/pipermail/openstack-dev/2015-April/060630.html | 05:08 |
| thingee | they just don't load...they eventually time out | 05:09 |
| Liu | Let me have a look at it | 05:09 |
| Liu | thingee: May be some of them are abandoned? | 05:13 |
| Liu | thingee:https://review.openstack.org/169812 is ok | 05:13 |
| Liu | thingee: or you can check it at https://review.openstack.org/#/q/project:openstack/cinder,n,z | 05:14 |
| thingee | Liu: please figure out why these are being removed. | 05:14 |
| *** Longgeek has joined #openstack-cinder | 05:14 | |
| thingee | the logs need to be kept around for 30 days | 05:14 |
| Liu | thingee: ok, I send this email at 1/4 | 05:15 |
| Liu | thingee: sorry, it is 4/4 | 05:15 |
| Liu | thingee: yes, we really keeped this logs | 05:16 |
| Liu | thingee: you can just check it at https://review.openstack.org/#/q/project:openstack/cinder,n,z | 05:17 |
| thingee | Liu: I'm looking at http://lists.openstack.org/pipermail/openstack-dev/2015-April/060630.html | 05:18 |
| thingee | for example https://review.openstack.org/#/c/169153/ | 05:18 |
| thingee | the links don't work | 05:18 |
| thingee | 182.138.104.27/huawei-18000-iscsi-dsvm-tempest-full/97 | 05:18 |
| thingee | 182.138.104.27/huawei-18000-fc-dsvm-tempest-full/103 | 05:18 |
| Liu | thingee: let me see | 05:19 |
| Liu | thingee: I have checked | 05:19 |
| Liu | it really works | 05:19 |
| Liu | can I paste this page at somewhere? | 05:20 |
| Liu | I have opened these pages | 05:20 |
| Liu | And for the links at http://lists.openstack.org/pipermail/openstack-dev/2015-April/060630.html | 05:21 |
| thingee | Liu: the point of the CI is for the public to be able to access them to verify why a patch breaks your driver's integration. | 05:21 |
| thingee | Liu: If I can't access them, it defeats the purpose. | 05:21 |
| thingee | Liu: do you understand that? | 05:21 |
| Liu | I know this thingee, and I have tested them from the public | 05:22 |
| Liu | I will have a see now from the public | 05:22 |
| thingee | Liu: Ok, well there appears to be some problems. Did we not try using some third party to host the logs temporarily until you guys can sort things internally? | 05:22 |
| *** rongze has quit IRC | 05:23 | |
| thingee | Liu: once you sort things out, please reply to my message on the openstack dev ML | 05:23 |
| Liu | thingee: I just have checked it , and it really can be accessed form the public | 05:24 |
| thingee | Liu: I'm still having issues access it =/ | 05:24 |
| *** deepakcs has joined #openstack-cinder | 05:25 | |
| Liu | thingee: may be I should check it at home, not just at my company | 05:25 |
| Liu | thinged: the network in my home is at the public | 05:26 |
| *** anshul has quit IRC | 05:26 | |
| Liu | if I can access them at home, I think it can be accessed form the public | 05:26 |
| *** anshul has joined #openstack-cinder | 05:26 | |
| *** mriedem1 has joined #openstack-cinder | 05:28 | |
| thingee | Liu: I think I'm being blocked according to traceroutes | 05:28 |
| Liu | thingee: I really applied the ip 182.138.104.27 to publish logs | 05:28 |
| Liu | thingee: may be we should have somebody else to check them? | 05:29 |
| Liu | thingee: and for the links in http://lists.openstack.org/pipermail/openstack-dev/2015-April/060630.html | 05:30 |
| *** nlevinki has joined #openstack-cinder | 05:30 | |
| thingee | Liu: how would I verify the logs on my own later if something fails from a proposed patch? | 05:30 |
| thingee | Liu: this is going to be problematic if it's blocked for some people | 05:30 |
| thingee | I pretty much get stopped at 202.97.33.185 | 05:31 |
| thingee | chinanet backbone | 05:31 |
| *** mriedem has quit IRC | 05:31 | |
| *** anshul has quit IRC | 05:32 | |
| Liu | thingee: I just received a email from the ML, saying that I should change the apache conf to not to download the log files | 05:32 |
| thingee | Liu: that's a separate issue | 05:33 |
| Liu | thingee: so I think he really have seen my log | 05:33 |
| *** sks has joined #openstack-cinder | 05:33 | |
| *** coolsvap is now known as coolsvap|afk | 05:33 | |
| thingee | Liu: yea, but I can view some of your logs | 05:33 |
| thingee | Liu: that's the problem though | 05:33 |
| thingee | Liu: I should be able to see them all | 05:33 |
| openstackgerrit | Mike Perez proposed openstack/cinder: Revert "Removing ZFSSA driver for no reported CI" https://review.openstack.org/170770 | 05:34 |
| *** mriedem has joined #openstack-cinder | 05:34 | |
| *** changbl has quit IRC | 05:34 | |
| Liu | thingee: may be some of your request have been filtered? | 05:35 |
| *** mriedem1 has quit IRC | 05:36 | |
| Liu | thingee: can you give me a little time to very this? | 05:38 |
| Liu | thingee: can you give me a little time to verify this? | 05:40 |
| thingee | Liu: sure | 05:44 |
| Liu | thingee: thanks | 05:44 |
| Liu | thingee:and for the links in http://lists.openstack.org/pipermail/openstack-dev/2015-April/060630.html | 05:44 |
| thingee | Liu: I'm going to bed, but please talk to DuncanT for verifying things. I'll be back in 9 hours. | 05:45 |
| Liu | you just add a #/c/ fefore the review number | 05:45 |
| thingee | DuncanT: please work with Liu on any verification of the Huawei CI. So far I'm having an issue where majority of the log links aren't loading for me. Think it might be a blocking issue. | 05:45 |
| Liu | you can just add a #/c/ before the review number | 05:45 |
| thingee | Liu: yeah I got that. | 05:46 |
| *** coolsvap has joined #openstack-cinder | 05:47 | |
| *** Longgeek has quit IRC | 05:47 | |
| *** rongze has joined #openstack-cinder | 05:48 | |
| *** rongze has quit IRC | 05:48 | |
| *** anshul has joined #openstack-cinder | 05:50 | |
| openstackgerrit | Mike Perez proposed openstack/python-cinderclient: Enable version discovery https://review.openstack.org/145613 | 05:55 |
| *** lpetrut has joined #openstack-cinder | 05:57 | |
| *** anshul has quit IRC | 06:01 | |
| *** emagana has joined #openstack-cinder | 06:06 | |
| *** anshul has joined #openstack-cinder | 06:06 | |
| *** bkopilov has quit IRC | 06:09 | |
| *** emagana has quit IRC | 06:11 | |
| *** anshul has quit IRC | 06:12 | |
| *** anshul has joined #openstack-cinder | 06:14 | |
| *** sgotliv has quit IRC | 06:18 | |
| *** annashen has joined #openstack-cinder | 06:19 | |
| *** nshaikh has joined #openstack-cinder | 06:23 | |
| *** Longgeek has joined #openstack-cinder | 06:24 | |
| *** avishay has joined #openstack-cinder | 06:24 | |
| *** anshul has quit IRC | 06:24 | |
| *** coolsvap has quit IRC | 06:25 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Revert "Removing ZFSSA driver for no reported CI" https://review.openstack.org/170770 | 06:26 |
| *** Mandell has quit IRC | 06:27 | |
| *** lpetrut has quit IRC | 06:27 | |
| *** bkopilov has joined #openstack-cinder | 06:27 | |
| *** rongze has joined #openstack-cinder | 06:28 | |
| Liu | DuncanT: I have found the problem, if you change to access the port 8088 instead of 80, you can access the page | 06:30 |
| Liu | thingee: can you please change the port to 8088, like 182.138.104.27:8088/huawei-18000-iscsi-dsvm-tempest-full/97 | 06:31 |
| *** yrabl__ has joined #openstack-cinder | 06:32 | |
| Liu | thingee: Then it can works | 06:32 |
| *** elonden has joined #openstack-cinder | 06:36 | |
| *** annashen has quit IRC | 06:37 | |
| *** annashen has joined #openstack-cinder | 06:37 | |
| *** tshefi has joined #openstack-cinder | 06:38 | |
| *** nikesh has quit IRC | 06:38 | |
| *** coolsvap has joined #openstack-cinder | 06:38 | |
| *** elonden has quit IRC | 06:39 | |
| *** elonden has joined #openstack-cinder | 06:41 | |
| *** Mandell has joined #openstack-cinder | 06:42 | |
| *** rongze has quit IRC | 06:46 | |
| *** anshul has joined #openstack-cinder | 06:48 | |
| *** jamielennox|away is now known as jamielennox | 06:48 | |
| *** annashen has quit IRC | 06:50 | |
| *** anshul has quit IRC | 06:52 | |
| *** vilobhmm11 has quit IRC | 06:57 | |
| *** markvoelker has quit IRC | 07:04 | |
| *** anshul has joined #openstack-cinder | 07:05 | |
| *** nikesh_vedams has joined #openstack-cinder | 07:07 | |
| *** sgotliv has joined #openstack-cinder | 07:08 | |
| *** elonden has quit IRC | 07:08 | |
| *** lpetrut has joined #openstack-cinder | 07:11 | |
| *** anshul has quit IRC | 07:12 | |
| *** anshul has joined #openstack-cinder | 07:12 | |
| *** anshul has quit IRC | 07:17 | |
| *** Lee1092 has joined #openstack-cinder | 07:21 | |
| *** annashen has joined #openstack-cinder | 07:26 | |
| *** yrabl__ has quit IRC | 07:30 | |
| *** yrabl_ has quit IRC | 07:31 | |
| *** yrabl has quit IRC | 07:31 | |
| *** yrabl has joined #openstack-cinder | 07:31 | |
| *** e0ne has joined #openstack-cinder | 07:35 | |
| thingee | e0ne: https://review.openstack.org/#/c/145613/ | 07:36 |
| e0ne | thingee: hi! thanks for update! i'll be at office in 1.5 hours and take a look/test it | 07:37 |
| thingee | e0ne: ok, it still has issues | 07:37 |
| e0ne | i see | 07:37 |
| thingee | e0ne: just wanted to post something to get early eyes on it | 07:37 |
| e0ne | ok. i test it once i'll get my env for it | 07:38 |
| *** anshul has joined #openstack-cinder | 07:38 | |
| e0ne | thingee: what timezone are you in now? | 07:39 |
| *** sgotliv has quit IRC | 07:39 | |
| e0ne | it's abaou 0:40am pst | 07:39 |
| thingee | e0ne: yeah was trying to organize stuff for the RC. I sent you all an email for priorities | 07:40 |
| thingee | https://etherpad.openstack.org/p/cinder-kilo-rc-priorities | 07:40 |
| thingee | alright heading to bed. | 07:40 |
| *** anshul has quit IRC | 07:41 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Revert "Removing ZFSSA driver for no reported CI" https://review.openstack.org/170770 | 07:41 |
| *** anshul has joined #openstack-cinder | 07:41 | |
| thingee | e0ne: also readding drivers | 07:41 |
| e0ne | thingee: it's pretty short list for these three days | 07:42 |
| e0ne | i mean only etherpad's list | 07:42 |
| thingee | e0ne: that means we're doing something right or we're missing bugs that should be targeted ;) | 07:42 |
| thingee | Liu: can you change your log web server to report on port 80? | 07:43 |
| thingee | Liu: so that the report result link on reviews works? | 07:43 |
| thingee | Liu: I verified 8088 works | 07:43 |
| Liu | thingee: May be the 80 port is blocked from the public | 07:44 |
| thingee | Liu: if that's so, can you change your CI to post the result link with port 8088 then? | 07:44 |
| Liu | thingee: May be I should use 8088 from now on | 07:44 |
| Liu | thingee: yes, I have changed the report link just now | 07:45 |
| Liu | thingee: the newly report will use 8088 port | 07:45 |
| thingee | Liu: thanks, I'll look at some of reviews it posts to later. | 07:45 |
| e0ne | thingee: one wish fro me if a can ask it: i want to rebout high-priority bug about unit tests once huawei drivers will be added. imo, those tests are unsupproted | 07:45 |
| Liu | thingee: thanks :) | 07:45 |
| *** annashen has quit IRC | 07:46 | |
| thingee | e0ne: what? | 07:47 |
| e0ne | thingee: imo, https://github.com/openstack/cinder/blob/stable/juno/cinder/tests/test_huawei_t_dorado.py it must be refactored if added to cinder again. | 07:48 |
| *** Miouge has joined #openstack-cinder | 07:48 | |
| e0ne | a lot off global variables, etc | 07:48 |
| *** anshul has quit IRC | 07:48 | |
| Liu | e0ne: huawei_t and huawei_dorado need not to be readded | 07:49 |
| e0ne | i spent almost 2 days to make it work in concurrency, but these tests should be re-writed | 07:49 |
| Liu | e0ne:just need to readd huawei_18000 | 07:49 |
| thingee | e0ne: yeah I won't be readding that one. just the 18000 | 07:49 |
| Liu | yes | 07:50 |
| thingee | alright heading to bed this time. | 07:50 |
| thingee | bbl | 07:50 |
| e0ne | thingee, Liu: ok, i just wanted to notice that we need to get better unit tests for drivers | 07:50 |
| Liu | e0ne: thanks:) | 07:51 |
| Liu | e0ne, thingee: I will make some improment for our drivers | 07:51 |
| Liu | e0ne,thingee: I know there is somthing should be improment | 07:52 |
| e0ne | Liu: thank you! | 07:53 |
| *** ronis has joined #openstack-cinder | 07:53 | |
| Liu | e0ne: :) | 07:54 |
| *** emagana has joined #openstack-cinder | 07:54 | |
| *** emagana has quit IRC | 07:59 | |
| *** markvoelker has joined #openstack-cinder | 08:03 | |
| *** nlevinki has quit IRC | 08:05 | |
| *** anshul has joined #openstack-cinder | 08:05 | |
| *** markvoelker has quit IRC | 08:08 | |
| *** e0ne has quit IRC | 08:12 | |
| *** anshul has quit IRC | 08:13 | |
| *** shyama has quit IRC | 08:16 | |
| *** e0ne has joined #openstack-cinder | 08:17 | |
| Liu | thingee: please check https://review.openstack.org/#/c/170770/ | 08:18 |
| Liu | thingee: it has been reported to 8088 port | 08:19 |
| *** yamada-h has joined #openstack-cinder | 08:19 | |
| *** nlevinki has joined #openstack-cinder | 08:20 | |
| *** e0ne has quit IRC | 08:20 | |
| *** pradipta has joined #openstack-cinder | 08:22 | |
| *** anshul has joined #openstack-cinder | 08:26 | |
| *** yrabl has quit IRC | 08:27 | |
| *** yrabl has joined #openstack-cinder | 08:28 | |
| *** yrabl has joined #openstack-cinder | 08:29 | |
| *** Miouge has quit IRC | 08:31 | |
| *** Mandell has quit IRC | 08:37 | |
| *** nshaikh has quit IRC | 08:47 | |
| *** emagana has joined #openstack-cinder | 08:49 | |
| *** shyama has joined #openstack-cinder | 08:50 | |
| *** nlevinki has quit IRC | 08:51 | |
| *** emagana has quit IRC | 08:53 | |
| *** chlong_ has joined #openstack-cinder | 08:54 | |
| *** coolsvap has quit IRC | 08:55 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 08:57 |
| *** chlong has quit IRC | 08:57 | |
| *** yrabl has quit IRC | 08:57 | |
| *** alecv_ has joined #openstack-cinder | 08:57 | |
| *** coolsvap has joined #openstack-cinder | 08:57 | |
| *** yrabl has joined #openstack-cinder | 08:58 | |
| *** e0ne has joined #openstack-cinder | 09:00 | |
| *** anshul has quit IRC | 09:01 | |
| *** e0ne is now known as e0ne_ | 09:01 | |
| *** nshaikh has joined #openstack-cinder | 09:01 | |
| *** markvoelker has joined #openstack-cinder | 09:04 | |
| *** nlevinki has joined #openstack-cinder | 09:04 | |
| *** e0ne_ is now known as e0ne | 09:08 | |
| *** markvoelker has quit IRC | 09:08 | |
| *** alecv has joined #openstack-cinder | 09:12 | |
| *** alecv_ has quit IRC | 09:12 | |
| DuncanT | thingee: The logs work with the 8088 in the link, and new reviews are getting the correct links | 09:12 |
| *** anshul has joined #openstack-cinder | 09:13 | |
| *** anshul has quit IRC | 09:14 | |
| *** anshul has joined #openstack-cinder | 09:15 | |
| *** Miouge has joined #openstack-cinder | 09:17 | |
| *** abhijeetm has joined #openstack-cinder | 09:19 | |
| *** pradipta has quit IRC | 09:24 | |
| *** nshaikh has quit IRC | 09:25 | |
| *** sks has quit IRC | 09:26 | |
| *** pradipta has joined #openstack-cinder | 09:27 | |
| *** anshul has quit IRC | 09:28 | |
| *** coolsvap has quit IRC | 09:30 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Lock on a per-volume basis https://review.openstack.org/169858 | 09:31 |
| *** anshul has joined #openstack-cinder | 09:40 | |
| *** sks has joined #openstack-cinder | 09:40 | |
| *** sgotliv has joined #openstack-cinder | 09:42 | |
| *** emagana has joined #openstack-cinder | 09:43 | |
| *** Longgeek has quit IRC | 09:43 | |
| *** Longgeek has joined #openstack-cinder | 09:43 | |
| *** coolsvap has joined #openstack-cinder | 09:45 | |
| *** nikesh_vedams has quit IRC | 09:47 | |
| *** anshul has quit IRC | 09:47 | |
| *** emagana has quit IRC | 09:47 | |
| *** lpetrut has quit IRC | 09:47 | |
| *** coolsvap has quit IRC | 09:55 | |
| *** dimsum__ has joined #openstack-cinder | 09:56 | |
| *** anshul has joined #openstack-cinder | 10:00 | |
| *** markvoelker has joined #openstack-cinder | 10:04 | |
| *** anshul has quit IRC | 10:05 | |
| *** coolsvap has joined #openstack-cinder | 10:08 | |
| *** lpetrut has joined #openstack-cinder | 10:09 | |
| *** markvoelker has quit IRC | 10:09 | |
| *** yamada-h has quit IRC | 10:12 | |
| *** sks has quit IRC | 10:12 | |
| *** anshul has joined #openstack-cinder | 10:19 | |
| *** yrabl has quit IRC | 10:19 | |
| *** yrabl has joined #openstack-cinder | 10:20 | |
| *** sks has joined #openstack-cinder | 10:28 | |
| *** anshul has quit IRC | 10:29 | |
| *** IanGovett has joined #openstack-cinder | 10:33 | |
| *** ronis has quit IRC | 10:35 | |
| *** lpetrut has quit IRC | 10:37 | |
| *** chlong_ has quit IRC | 10:41 | |
| e0ne | DuncanT: hi! I'm not sure that I understand your coment about SQL query right for https://review.openstack.org/#/c/151567/ patch | 10:46 |
| *** anshul has joined #openstack-cinder | 10:47 | |
| e0ne | DuncanT: could you explain what I missed, please? | 10:47 |
| DuncanT | e0ne: I suspect this check is one of the (probably many) races we have in our DB accesses. The comment about SQL is slightly misleading, sorry, I mean I want to understand the atomicity of each of our DB accesses, and their collision behaviour (which is slightly different between stock mysql and galera), as well as our general approach to accessing the db | 10:49 |
| DuncanT | via check-and-set without being wrapped in a transaction. | 10:49 |
| DuncanT | e0ne: It shouldn't hold up this patch, I was thinking out loud | 10:50 |
| *** lpetrut has joined #openstack-cinder | 10:52 | |
| e0ne | DuncanT: we use single transaction for SELECT and UPDATE there. are you talking about using select-for-update? | 10:52 |
| DuncanT | e0ne: I can't remember which one behaves weirdly on galera, I just remember there being an issue | 10:54 |
| *** anshul has quit IRC | 10:57 | |
| e0ne | DuncanT: afaik, Galera doesn't support 'select for update' statement | 10:57 |
| *** chlong_ has joined #openstack-cinder | 10:58 | |
| DuncanT | So in this case we're ok? Thanks for confirming. I'll go any put a comment on the patch to the effect that I'm happy. We have a larger problem to solve, but I don't think it affects this patch | 10:58 |
| e0ne | yes, it looks good for me | 10:59 |
| *** anshul has joined #openstack-cinder | 10:59 | |
| DuncanT | Ok, changed my comment/vote | 11:00 |
| e0ne | we use with_lockmode('update') ('select for update stamement) for quotas and allocation ascsi target only | 11:00 |
| e0ne | i'll put my comments to gerrit too | 11:01 |
| *** chlong_ has quit IRC | 11:05 | |
| *** markvoelker has joined #openstack-cinder | 11:05 | |
| *** markvoelker has quit IRC | 11:10 | |
| *** marcusvrn has joined #openstack-cinder | 11:11 | |
| *** yrabl has quit IRC | 11:11 | |
| *** yrabl has joined #openstack-cinder | 11:11 | |
| *** sgotliv has quit IRC | 11:11 | |
| *** yamada-h has joined #openstack-cinder | 11:13 | |
| *** anshul has quit IRC | 11:13 | |
| *** e0ne is now known as e0ne_ | 11:14 | |
| *** e0ne_ is now known as e0ne | 11:16 | |
| *** diegows has joined #openstack-cinder | 11:17 | |
| *** yamada-h has quit IRC | 11:18 | |
| *** Longgeek has quit IRC | 11:20 | |
| *** chlong has joined #openstack-cinder | 11:24 | |
| *** anshul has joined #openstack-cinder | 11:26 | |
| *** pradipta has quit IRC | 11:31 | |
| *** emagana has joined #openstack-cinder | 11:31 | |
| *** Miouge has quit IRC | 11:32 | |
| *** anshul has quit IRC | 11:35 | |
| *** emagana has quit IRC | 11:35 | |
| *** anuragpalsule has joined #openstack-cinder | 11:37 | |
| *** anuragpalsule has quit IRC | 11:38 | |
| -openstackstatus- NOTICE: gerrit has been restarted to restore event streaming. any change events missed by zuul (between 10:56 and 11:37 utc) will need to be rechecked or have new approval votes set | 11:38 | |
| *** Longgeek has joined #openstack-cinder | 11:39 | |
| *** ho has quit IRC | 11:40 | |
| *** Lee1092 has quit IRC | 11:40 | |
| *** anshul has joined #openstack-cinder | 11:48 | |
| *** jamielennox is now known as jamielennox|away | 11:49 | |
| *** sgotliv has joined #openstack-cinder | 11:52 | |
| *** diegows has quit IRC | 11:53 | |
| *** ronis has joined #openstack-cinder | 11:53 | |
| *** marcusvrn has quit IRC | 11:56 | |
| *** marcusvrn has joined #openstack-cinder | 11:56 | |
| *** Lee1092 has joined #openstack-cinder | 12:02 | |
| *** avishay_ has joined #openstack-cinder | 12:06 | |
| *** markvoelker has joined #openstack-cinder | 12:06 | |
| *** anshul has quit IRC | 12:06 | |
| *** avishay has quit IRC | 12:06 | |
| *** sgotliv has quit IRC | 12:10 | |
| *** markvoelker has quit IRC | 12:10 | |
| *** yamada-h has joined #openstack-cinder | 12:14 | |
| *** anshul has joined #openstack-cinder | 12:18 | |
| *** bswartz has quit IRC | 12:19 | |
| *** yamada-h has quit IRC | 12:19 | |
| *** xyang1 has joined #openstack-cinder | 12:19 | |
| *** markvoelker has joined #openstack-cinder | 12:24 | |
| *** coolsvap has quit IRC | 12:25 | |
| *** emagana has joined #openstack-cinder | 12:25 | |
| *** dimsum__ has quit IRC | 12:25 | |
| *** theanalyst has quit IRC | 12:28 | |
| *** emagana has quit IRC | 12:30 | |
| *** theanalyst has joined #openstack-cinder | 12:31 | |
| *** akerr has joined #openstack-cinder | 12:33 | |
| *** rhefner has joined #openstack-cinder | 12:36 | |
| *** Mandell has joined #openstack-cinder | 12:37 | |
| *** deepakcs has quit IRC | 12:38 | |
| *** marcusvrn has quit IRC | 12:41 | |
| *** marcusvrn1 has joined #openstack-cinder | 12:41 | |
| *** Mandell has quit IRC | 12:43 | |
| *** zerda has quit IRC | 12:43 | |
| *** anuragpalsule has joined #openstack-cinder | 12:45 | |
| *** e0ne is now known as e0ne_ | 12:46 | |
| *** anuragpalsule1 has joined #openstack-cinder | 12:46 | |
| *** e0ne_ is now known as e0ne | 12:49 | |
| *** anuragpalsule has quit IRC | 12:50 | |
| *** annegentle has joined #openstack-cinder | 12:50 | |
| *** dimsum__ has joined #openstack-cinder | 12:51 | |
| *** annegentle has quit IRC | 12:52 | |
| *** annegentle has joined #openstack-cinder | 12:52 | |
| *** xyang has joined #openstack-cinder | 12:55 | |
| *** anuragpalsule1 has quit IRC | 12:55 | |
| *** Yogi1 has joined #openstack-cinder | 12:55 | |
| *** bswartz has joined #openstack-cinder | 12:57 | |
| *** mriedem1 has joined #openstack-cinder | 13:00 | |
| *** anuragpalsule has joined #openstack-cinder | 13:00 | |
| *** mriedem has quit IRC | 13:02 | |
| *** dimsum__ is now known as dims | 13:03 | |
| *** mriedem1 has quit IRC | 13:06 | |
| *** xyang has quit IRC | 13:08 | |
| *** sgotliv has joined #openstack-cinder | 13:09 | |
| *** julim has joined #openstack-cinder | 13:09 | |
| *** julim has quit IRC | 13:11 | |
| *** avishay_ has quit IRC | 13:15 | |
| *** cbader has joined #openstack-cinder | 13:17 | |
| *** avishay_ has joined #openstack-cinder | 13:18 | |
| *** emagana has joined #openstack-cinder | 13:20 | |
| *** Yogi1 has quit IRC | 13:22 | |
| *** annegentle has quit IRC | 13:22 | |
| *** emagana has quit IRC | 13:24 | |
| *** anuragpalsule has quit IRC | 13:24 | |
| *** anuragpalsule1 has joined #openstack-cinder | 13:25 | |
| *** Yogi1 has joined #openstack-cinder | 13:26 | |
| *** Yogi1 has quit IRC | 13:28 | |
| *** dustins has joined #openstack-cinder | 13:29 | |
| *** Mandell has joined #openstack-cinder | 13:29 | |
| *** timcl has joined #openstack-cinder | 13:30 | |
| *** e0ne is now known as e0ne_ | 13:31 | |
| openstackgerrit | yogeshprasad proposed openstack/cinder: Fixes snapshot creation failure in CloudByte driver. https://review.openstack.org/167978 | 13:32 |
| *** Longgeek has quit IRC | 13:33 | |
| *** rongze has joined #openstack-cinder | 13:37 | |
| *** mriedem has joined #openstack-cinder | 13:37 | |
| *** svasheka has joined #openstack-cinder | 13:41 | |
| *** thangp has joined #openstack-cinder | 13:45 | |
| *** dustins_ has joined #openstack-cinder | 13:47 | |
| *** Mandell has quit IRC | 13:48 | |
| *** eharney has joined #openstack-cinder | 13:48 | |
| *** esker has joined #openstack-cinder | 13:49 | |
| *** dustins has quit IRC | 13:49 | |
| *** rmesta has joined #openstack-cinder | 13:51 | |
| *** annegentle has joined #openstack-cinder | 13:52 | |
| *** e0ne_ is now known as e0ne | 13:53 | |
| *** anshul has quit IRC | 13:53 | |
| *** mriedem has quit IRC | 13:53 | |
| *** marcusvrn1 has quit IRC | 13:53 | |
| *** rhefner has quit IRC | 13:53 | |
| *** markvoelker has quit IRC | 13:53 | |
| *** IanGovett has quit IRC | 13:53 | |
| *** sdague has quit IRC | 13:53 | |
| *** briancline has quit IRC | 13:53 | |
| *** alecv has quit IRC | 13:53 | |
| *** e0ne has quit IRC | 13:53 | |
| *** bkopilov has quit IRC | 13:53 | |
| *** thingee has quit IRC | 13:53 | |
| *** wanghao has quit IRC | 13:53 | |
| *** david-lyle has quit IRC | 13:53 | |
| *** mdbooth has quit IRC | 13:53 | |
| *** mdenny has quit IRC | 13:53 | |
| *** jwang_ has quit IRC | 13:53 | |
| *** hflai has quit IRC | 13:53 | |
| *** leseb- has quit IRC | 13:53 | |
| *** eikke has quit IRC | 13:53 | |
| *** d0ugal has quit IRC | 13:53 | |
| *** mkoderer has quit IRC | 13:53 | |
| *** avishay_ has quit IRC | 13:53 | |
| *** krtaylor has quit IRC | 13:53 | |
| *** mitz has quit IRC | 13:53 | |
| *** fanyaohong has quit IRC | 13:53 | |
| *** mgagne has quit IRC | 13:53 | |
| *** adam_g_out has quit IRC | 13:53 | |
| *** ekarlso has quit IRC | 13:53 | |
| *** sparr has quit IRC | 13:53 | |
| *** coolsvap|afk has quit IRC | 13:53 | |
| *** Trozz has quit IRC | 13:53 | |
| *** jbernard has quit IRC | 13:53 | |
| *** SergeyLukjanov has quit IRC | 13:53 | |
| *** timcl has quit IRC | 13:53 | |
| *** dims has quit IRC | 13:53 | |
| *** sks has quit IRC | 13:53 | |
| *** tshefi has quit IRC | 13:53 | |
| *** strictlyb has quit IRC | 13:53 | |
| *** frakt has quit IRC | 13:53 | |
| *** jgriffith has quit IRC | 13:53 | |
| *** Anticimex has quit IRC | 13:53 | |
| *** danieru has quit IRC | 13:53 | |
| *** beecee has quit IRC | 13:53 | |
| *** rmk has quit IRC | 13:53 | |
| *** larsks has quit IRC | 13:53 | |
| *** rongze has quit IRC | 13:53 | |
| *** abhijeetm has quit IRC | 13:53 | |
| *** harlowja_away has quit IRC | 13:53 | |
| *** boichev has quit IRC | 13:53 | |
| *** skraynev has quit IRC | 13:53 | |
| *** pck has quit IRC | 13:53 | |
| *** junhongl has quit IRC | 13:53 | |
| *** smcginnis has quit IRC | 13:53 | |
| *** sc has quit IRC | 13:53 | |
| *** ondergetekende has quit IRC | 13:53 | |
| *** tellesnobrega has quit IRC | 13:54 | |
| *** tellesnobrega has joined #openstack-cinder | 13:55 | |
| *** avishay_ has joined #openstack-cinder | 13:55 | |
| *** krtaylor has joined #openstack-cinder | 13:55 | |
| *** mitz has joined #openstack-cinder | 13:55 | |
| *** fanyaohong has joined #openstack-cinder | 13:55 | |
| *** mgagne has joined #openstack-cinder | 13:55 | |
| *** adam_g_out has joined #openstack-cinder | 13:55 | |
| *** ekarlso has joined #openstack-cinder | 13:55 | |
| *** sparr has joined #openstack-cinder | 13:55 | |
| *** coolsvap|afk has joined #openstack-cinder | 13:55 | |
| *** Trozz has joined #openstack-cinder | 13:55 | |
| *** jbernard has joined #openstack-cinder | 13:55 | |
| *** SergeyLukjanov has joined #openstack-cinder | 13:55 | |
| *** sparr has quit IRC | 13:55 | |
| *** timcl has joined #openstack-cinder | 13:56 | |
| *** dims has joined #openstack-cinder | 13:56 | |
| *** sks has joined #openstack-cinder | 13:56 | |
| *** tshefi has joined #openstack-cinder | 13:56 | |
| *** strictlyb has joined #openstack-cinder | 13:56 | |
| *** frakt has joined #openstack-cinder | 13:56 | |
| *** jgriffith has joined #openstack-cinder | 13:56 | |
| *** Anticimex has joined #openstack-cinder | 13:56 | |
| *** danieru has joined #openstack-cinder | 13:56 | |
| *** beecee has joined #openstack-cinder | 13:56 | |
| *** rmk has joined #openstack-cinder | 13:56 | |
| *** larsks has joined #openstack-cinder | 13:56 | |
| *** Mandell_ has joined #openstack-cinder | 13:58 | |
| *** mriedem has joined #openstack-cinder | 13:58 | |
| *** marcusvrn1 has joined #openstack-cinder | 13:58 | |
| *** rhefner has joined #openstack-cinder | 13:58 | |
| *** markvoelker has joined #openstack-cinder | 13:58 | |
| *** IanGovett has joined #openstack-cinder | 13:58 | |
| *** sdague has joined #openstack-cinder | 13:58 | |
| *** briancline has joined #openstack-cinder | 13:58 | |
| *** sparr has joined #openstack-cinder | 13:58 | |
| *** alecv has joined #openstack-cinder | 13:58 | |
| *** e0ne has joined #openstack-cinder | 13:58 | |
| *** bkopilov has joined #openstack-cinder | 13:58 | |
| *** thingee has joined #openstack-cinder | 13:58 | |
| *** wanghao has joined #openstack-cinder | 13:58 | |
| *** david-lyle has joined #openstack-cinder | 13:58 | |
| *** mdbooth has joined #openstack-cinder | 13:58 | |
| *** mdenny has joined #openstack-cinder | 13:58 | |
| *** jwang_ has joined #openstack-cinder | 13:58 | |
| *** hflai has joined #openstack-cinder | 13:58 | |
| *** leseb- has joined #openstack-cinder | 13:58 | |
| *** eikke has joined #openstack-cinder | 13:58 | |
| *** d0ugal has joined #openstack-cinder | 13:58 | |
| *** mkoderer has joined #openstack-cinder | 13:58 | |
| *** rongze has joined #openstack-cinder | 13:58 | |
| *** abhijeetm has joined #openstack-cinder | 13:58 | |
| *** harlowja_away has joined #openstack-cinder | 13:58 | |
| *** boichev has joined #openstack-cinder | 13:58 | |
| *** skraynev has joined #openstack-cinder | 13:58 | |
| *** pck has joined #openstack-cinder | 13:58 | |
| *** junhongl has joined #openstack-cinder | 13:58 | |
| *** smcginnis has joined #openstack-cinder | 13:58 | |
| *** sc has joined #openstack-cinder | 13:58 | |
| *** ondergetekende has joined #openstack-cinder | 13:58 | |
| *** sc has quit IRC | 13:59 | |
| *** Mandell_ has quit IRC | 14:00 | |
| *** mriedem has quit IRC | 14:00 | |
| *** marcusvrn1 has quit IRC | 14:00 | |
| *** rhefner has quit IRC | 14:00 | |
| *** markvoelker has quit IRC | 14:00 | |
| *** IanGovett has quit IRC | 14:00 | |
| *** sdague has quit IRC | 14:00 | |
| *** briancline has quit IRC | 14:00 | |
| *** Mandell_ has joined #openstack-cinder | 14:01 | |
| *** mriedem has joined #openstack-cinder | 14:01 | |
| *** marcusvrn1 has joined #openstack-cinder | 14:01 | |
| *** rhefner has joined #openstack-cinder | 14:01 | |
| *** markvoelker has joined #openstack-cinder | 14:01 | |
| *** IanGovett has joined #openstack-cinder | 14:01 | |
| *** sdague has joined #openstack-cinder | 14:01 | |
| *** briancline has joined #openstack-cinder | 14:01 | |
| *** sc has joined #openstack-cinder | 14:01 | |
| *** lpetrut has quit IRC | 14:01 | |
| *** avishay_ has quit IRC | 14:03 | |
| *** avishay__ has joined #openstack-cinder | 14:03 | |
| *** anshul has joined #openstack-cinder | 14:03 | |
| *** Mandell_ has quit IRC | 14:03 | |
| *** annegentle has quit IRC | 14:05 | |
| *** anshul has quit IRC | 14:08 | |
| openstackgerrit | John Haan proposed openstack/python-cinderclient: Adding search options to backup list https://review.openstack.org/78233 | 14:15 |
| *** emagana has joined #openstack-cinder | 14:18 | |
| *** emagana has quit IRC | 14:18 | |
| *** emagana has joined #openstack-cinder | 14:19 | |
| *** rushil has joined #openstack-cinder | 14:21 | |
| openstackgerrit | Peter Hamilton proposed openstack/python-cinderclient: Add encryption-type-update to cinderclient https://review.openstack.org/58782 | 14:22 |
| *** mtanino has joined #openstack-cinder | 14:23 | |
| ameade | hey folks, I need a cinder core to redo the workflow +1 on this patch: https://review.openstack.org/#/c/169781/ | 14:27 |
| ameade | due to a gerrit restart it was lost in the queue | 14:27 |
| *** mtanino has quit IRC | 14:27 | |
| ameade | and rechecks will not solve the issue | 14:27 |
| *** mtanino has joined #openstack-cinder | 14:27 | |
| ameade | as per a convo with anteaya and fungi in #openstack-infra | 14:28 |
| ameade | thingee, xyang1 ^^ | 14:28 |
| xyang1 | ameade: It has +A, what else do we need to do? | 14:30 |
| ameade | xyang1: I think remove it and then add it back, they had to restart gerrit earlier and this fell within the problem window | 14:30 |
| *** sgotliv has quit IRC | 14:30 | |
| xyang1 | ameade: Ok, let's see if it works now | 14:32 |
| e0ne | ameade, xyang: this patch is in the gate queue | 14:32 |
| *** deepakcs has joined #openstack-cinder | 14:32 | |
| ameade | xyang1: tyvm | 14:32 |
| ameade | e0ne: it is now :p | 14:32 |
| e0ne | ameade: i think that recheck helped | 14:33 |
| anteaya | xyang1: hi yes ameade is correct, due to the gerrit restart the queues were lost | 14:34 |
| anteaya | since the +1 workflow was already there, zuul doesn't see it as a new event | 14:34 |
| xyang1 | anteaya: Thanks. I removed +A and added back | 14:34 |
| anteaya | and since it already had a +1 from jenkins rechecking it would just give it a +1 which would not be new so no new event would be broadcast as a result | 14:35 |
| anteaya | so zuul still wouldn't see it | 14:35 |
| anteaya | sure thank you | 14:35 |
| *** sk69 has joined #openstack-cinder | 14:36 | |
| sk69 | Hi, does volume migration need 2 block nodes? | 14:36 |
| akerr | sk69: just 2 backends / pools | 14:39 |
| mtanino | sk69: If you set up 2 volume backends on the same node, you can migrate a volume within one node. | 14:39 |
| sk69 | akerr, mtanio, thanks alot for ur replies, how can i create backends/pools and see them? | 14:41 |
| sk69 | with cinder create command? | 14:41 |
| *** Yogi1 has joined #openstack-cinder | 14:41 | |
| mtanino | like this. http://docs.openstack.org/admin-guide-cloud/content/multi_backend.html | 14:43 |
| deepakcs | hi all, I have a q on the test_volume_boot_pattern tempest test, that was enabled recently, and its causing the glusterFS CI job to fail | 14:45 |
| deepakcs | for glusterfs case, i debugged using the logs, its passing test_volume_boot_pattern, except for the last scneario (where it boots with the volume created from snap) | 14:45 |
| deepakcs | eharney, ^^ | 14:45 |
| deepakcs | I looked at the nova logs and all it says is "Build Error"... unable to prep block device | 14:45 |
| deepakcs | I am wondering why it would fail only for the last scenario (booting from a volume created from snap). | 14:46 |
| deepakcs | what coould be the reason for gettign a build error for the above case ? any hints/clues appreacited :) | 14:46 |
| guitarzan | deepakcs: is nova timing out? | 14:47 |
| *** ebalduf has joined #openstack-cinder | 14:47 | |
| deepakcs | guitarzan, no, its not timing out, create server (aka nova instance), the build tries to prep the block devices and it fails there | 14:47 |
| *** dflorea has joined #openstack-cinder | 14:47 | |
| guitarzan | usually the real error is hidden behind the nova fault | 14:47 |
| eharney | deepakcs: i thought jbernard had chased this toward a general Nova problem that wasn't specific to glusterfs | 14:47 |
| deepakcs | eharney, i am not sure yet, but glusterfs failure is diff than ceph | 14:48 |
| jbernard | that's correct, gluster and ceph happen to be failing and they appear to be unrelated | 14:48 |
| jbernard | yes, distinctly different failures | 14:48 |
| deepakcs | eharney, for me most part of test case is passing, except for the last section (where it tries to create server using voluem created from snap) | 14:48 |
| deepakcs | eharney, so i am doubting if that can happen if the image metadata was persisted/copied across the create-snap-then-create-vol-from-snap operations ? | 14:49 |
| *** annegentle has joined #openstack-cinder | 14:49 | |
| *** tsekiyama has joined #openstack-cinder | 14:49 | |
| deepakcs | jbernard, not totally unrelated, the same test case is failing for both gluster and ceph, just that the step inside the TC are different | 14:50 |
| eharney | deepakcs: doubting if what can happen | 14:50 |
| jbernard | deepakcs: gluster is failing for a differetn reason than ceph | 14:50 |
| jbernard | deepakcs: nova fails to generate a correct xml for the instance on first boot | 14:50 |
| jbernard | deepakcs: i think you're getting nearly to the end of the test and failing to boot from the snap | 14:51 |
| deepakcs | eharney, bcos the TC creates 2 instances and they both work for glusterfs, it only fails when it creates a new volume from a snap and then boots form that vol | 14:51 |
| deepakcs | jbernard, right | 14:51 |
| *** ebalduf has quit IRC | 14:51 | |
| *** julim has joined #openstack-cinder | 14:52 | |
| deepakcs | eharney, Also its not clear what the 'general Nova problem' is.. bcos the nova LP bug is duped to cinder LP, so stil not clear if its a nova issue or cinder driver specific | 14:52 |
| deepakcs | guitarzan, ^^ | 14:53 |
| *** dflorea has quit IRC | 14:54 | |
| *** dflorea has joined #openstack-cinder | 14:54 | |
| jungleboyj | hemnafk: jgriffith xyang1 thingee Thoughts on getting this merged? https://review.openstack.org/#/c/166351/ | 14:55 |
| eharney | deepakcs: jon just indicated that they are distinct failures so ditch that. i was just going based on what i thought someone said last week. | 14:56 |
| *** jaypipes has joined #openstack-cinder | 14:56 | |
| *** ociuhandu has joined #openstack-cinder | 14:56 | |
| deepakcs | eharney, ok, np, I was trying to see if someone had any ideas on the distinct failure seen in glusterfs case | 14:57 |
| guitarzan | deepakcs: people always pass the buck :) but you should actually see an exception on nova's side before it sets the state to "failed prepping..." | 14:57 |
| deepakcs | guitarzan, i don't much understand the _prepare_block_devices code of Nova, 'guess will have to spend time getting inside it! | 14:58 |
| guitarzan | deepakcs: there should be a stack in the logs at least | 14:58 |
| *** dflorea has quit IRC | 14:59 | |
| *** esker has quit IRC | 15:00 | |
| *** chlong has quit IRC | 15:00 | |
| deepakcs | guitarzan, eharney http://paste.fedoraproject.org/207620/42833246 | 15:01 |
| sk69 | mtanino thax for the link | 15:01 |
| mtanino | sk69: sure. | 15:01 |
| *** lpetrut has joined #openstack-cinder | 15:01 | |
| guitarzan | deepakcs: that's the beginning eh? | 15:01 |
| *** Mandell has joined #openstack-cinder | 15:01 | |
| deepakcs | guitarzan, u mean u want more verbose log ? (incl the debug/info in between ) ? | 15:02 |
| *** e0ne is now known as e0ne_ | 15:02 | |
| guitarzan | deepakcs: I'm not sure...but there should be a reason for "Instance failed block device setup" | 15:02 |
| *** asselin has joined #openstack-cinder | 15:02 | |
| *** e0ne_ is now known as e0ne | 15:03 | |
| guitarzan | although looking at that code path.. maybe not :( | 15:04 |
| guitarzan | except Exception: | 15:04 |
| guitarzan | createa new exception and don't tell what the orignal was! | 15:04 |
| guitarzan | thanks guys | 15:04 |
| *** esker has joined #openstack-cinder | 15:04 | |
| deepakcs | guitarzan, eharney http://paste.fedoraproject.org/207622/14283327 | 15:05 |
| guitarzan | ah, there ya go | 15:05 |
| guitarzan | 500 | 15:05 |
| guitarzan | I don't know really :) | 15:05 |
| guitarzan | the post returned a 500? | 15:06 |
| deepakcs | guitarzan, yeah i saw that too, but wasn't clear how that could be related | 15:06 |
| guitarzan | oh ya, LOG.exception, that's the right thing :) | 15:07 |
| deepakcs | guitarzan, line no ? | 15:07 |
| guitarzan | the attach is throwing a 500, so you should see that on the cinder side | 15:08 |
| eharney | wait, /opt/stack/new/, this is a grenade failure? | 15:09 |
| *** dannywilson has joined #openstack-cinder | 15:14 | |
| *** nlevinki has quit IRC | 15:16 | |
| *** dannywilson has quit IRC | 15:16 | |
| deepakcs | guitarzan, eharney I see this (http://paste.fedoraproject.org/207626/33337514) on the cinder side, few mins before the nova failure, does that raise any doubts ? | 15:16 |
| *** dannywilson has joined #openstack-cinder | 15:17 | |
| guitarzan | deepakcs: hmm, that's not from the attach call? but earlier? | 15:18 |
| *** marcusvrn has joined #openstack-cinder | 15:19 | |
| deepakcs | guitarzan, i am havign a hard time co-relate the nova/cinder logs, but the cinder log does seems to preclude the nova excp , looking at the timestamps | 15:20 |
| *** diemt has joined #openstack-cinder | 15:21 | |
| guitarzan | deepakcs: look for the initialize_connection call | 15:21 |
| guitarzan | that's the one that blew up for nova | 15:21 |
| *** sk69 has quit IRC | 15:21 | |
| deepakcs | guitarzan, yeah i see that it resulted in 500, and it does reach all the way to Cinder, so u r saying since it shows 500 it never reached cinder is it ? | 15:22 |
| *** sparr has quit IRC | 15:22 | |
| *** sparr has joined #openstack-cinder | 15:22 | |
| guitarzan | no, I'm saying it *is* in cinder | 15:23 |
| guitarzan | so find out why it failed in cinder :) | 15:23 |
| deepakcs | guitarzan, then how do u justify the 500 ? | 15:23 |
| guitarzan | I don't understand the question | 15:23 |
| deepakcs | guitarzan, on nova side, the initalize_conn resulted in internal server error, right ? | 15:23 |
| guitarzan | no, initialize_connection is a cinder call... | 15:24 |
| guitarzan | I'm very confused by what you're asking :) | 15:24 |
| guitarzan | cinder returned a 500 | 15:24 |
| guitarzan | find out why :) | 15:24 |
| deepakcs | guitarzan, ok, i see ur point now | 15:24 |
| *** Apoorva has joined #openstack-cinder | 15:25 | |
| *** deepakcs has quit IRC | 15:27 | |
| *** Mandell has quit IRC | 15:27 | |
| *** lpabon has joined #openstack-cinder | 15:28 | |
| *** jdurgin has joined #openstack-cinder | 15:31 | |
| *** sks has quit IRC | 15:32 | |
| *** rwsu has joined #openstack-cinder | 15:37 | |
| *** Miouge has joined #openstack-cinder | 15:41 | |
| *** ebalduf has joined #openstack-cinder | 15:48 | |
| *** sk69 has joined #openstack-cinder | 15:49 | |
| sk69 | Hi, for volume migration, which node should i enable the "enabled_backends" for the cinder.conf, block storage or controller or both? | 15:51 |
| sk69 | using http://docs.openstack.org/admin-guide-cloud/content/multi_backend.html | 15:51 |
| *** ebalduf has quit IRC | 15:52 | |
| *** _cjones_ has joined #openstack-cinder | 15:56 | |
| *** emagana has quit IRC | 15:56 | |
| jgriffith | jungleboyj: seems like a good fix up... | 15:57 |
| jgriffith | jungleboyj: is there ever a chance that kwargs will be empty? | 15:57 |
| * jgriffith sadly hasn't looked closely or tested | 15:57 | |
| jgriffith | jungleboyj: ahh.. ok | 15:58 |
| jgriffith | jungleboyj: we're good there I think | 15:58 |
| *** anuragpalsule1 has quit IRC | 15:59 | |
| jungleboyj | jgriffith: Yeah, I think it is safe. It is similar to what is being done in Nova and helps with a couple of bugs. | 15:59 |
| jungleboyj | Wasn't sure about it at first, but have had some people ask about it. Wanted to get it some consideration. | 16:00 |
| openstackgerrit | yatin proposed openstack/python-cinderclient: changed --force parameter into boolean https://review.openstack.org/170724 | 16:00 |
| *** emagana has joined #openstack-cinder | 16:00 | |
| *** _cjones_ has quit IRC | 16:01 | |
| jgriffith | jungleboyj: yeah, I put it in the debugger and figured out exactly how that works :) | 16:01 |
| jgriffith | jungleboyj: and indeed, you're correct... it's the exact pattern novaclient uses | 16:01 |
| jgriffith | jungleboyj: thanks for highlighting it | 16:01 |
| *** asselin has quit IRC | 16:02 | |
| jungleboyj | jgriffith: Welcome. Thanks for looking. | 16:02 |
| *** annegentle has quit IRC | 16:03 | |
| Liu | thingee:Duncan have confirmed our CI results in the ML and IRC | 16:03 |
| *** avishay__ has quit IRC | 16:06 | |
| *** marcusvrn2 has joined #openstack-cinder | 16:11 | |
| *** adam_g_out is now known as adam_g | 16:12 | |
| *** marcusvrn1 has quit IRC | 16:13 | |
| *** e0ne has quit IRC | 16:13 | |
| guitarzan | hey folks, have they said much about the design summit schedule? | 16:13 |
| *** crose has joined #openstack-cinder | 16:18 | |
| *** tshefi has quit IRC | 16:22 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Add hacking check for print() statements https://review.openstack.org/166066 | 16:23 |
| *** alexpilotti has joined #openstack-cinder | 16:23 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Add hacking check for print() statements https://review.openstack.org/166066 | 16:26 |
| *** btran has joined #openstack-cinder | 16:28 | |
| *** leeantho has joined #openstack-cinder | 16:28 | |
| *** vilobhmm1 has joined #openstack-cinder | 16:29 | |
| *** vilobhmm11 has joined #openstack-cinder | 16:30 | |
| *** vilobhmm1 has quit IRC | 16:30 | |
| *** vilobhmm11 has quit IRC | 16:30 | |
| *** vilobhmm1 has joined #openstack-cinder | 16:31 | |
| mriedem | this change would make the ceph job voting on cinder changes (and nova and glance changes), so fyi for any cinder peeps https://review.openstack.org/#/c/170913/ | 16:33 |
| mriedem | not the marshmallow kind | 16:33 |
| *** bkopilov has quit IRC | 16:33 | |
| *** bkopilov has joined #openstack-cinder | 16:35 | |
| *** thingee has quit IRC | 16:37 | |
| *** jdurgin has quit IRC | 16:40 | |
| *** thingee has joined #openstack-cinder | 16:40 | |
| *** bkopilov has quit IRC | 16:40 | |
| vilobhmm1 | thangp : ping | 16:44 |
| openstackgerrit | Mike Perez proposed openstack/cinder: Partial Revert "Removing ZFSSA driver" https://review.openstack.org/170770 | 16:45 |
| *** abhijeetm has quit IRC | 16:45 | |
| *** anuragpalsule has joined #openstack-cinder | 16:47 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Revert "Removing Huawei drivers for no reported CI" https://review.openstack.org/170923 | 16:48 |
| *** anuragpalsule1 has joined #openstack-cinder | 16:48 | |
| *** leeantho has quit IRC | 16:48 | |
| *** leeantho has joined #openstack-cinder | 16:48 | |
| *** ebalduf has joined #openstack-cinder | 16:48 | |
| asselin_ | thingee, can you look at this https://bugs.launchpad.net/tempest/+bug/1440227 | 16:49 |
| openstack | Launchpad bug 1440227 in tempest "encrypted volume tests don't check if a volume is actually encrypted" [Undecided,New] | 16:49 |
| *** anuragpalsule has quit IRC | 16:51 | |
| *** rongze has quit IRC | 16:52 | |
| asselin_ | mtreinish, ^^ | 16:52 |
| *** alecv has quit IRC | 16:52 | |
| mtreinish | asselin_: what does tempest check exactly? Like does it just make the request and not verify the api response says it's encrypted? | 16:53 |
| *** ebalduf has quit IRC | 16:53 | |
| *** virmitio has joined #openstack-cinder | 16:54 | |
| *** annegentle has joined #openstack-cinder | 16:55 | |
| asselin_ | mtreinish, looking it up | 16:55 |
| asselin_ | mtreinish, just checks attach & detach https://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_encrypted_cinder_volumes.py#n58 | 16:57 |
| asselin_ | mtreinish, which doesn't verify this code path is hit: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1053 | 16:58 |
| *** timcl has quit IRC | 16:59 | |
| *** timcl1 has joined #openstack-cinder | 16:59 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Partial Revert "Removing Huawei drivers" https://review.openstack.org/170923 | 16:59 |
| *** anuragpalsule has joined #openstack-cinder | 17:00 | |
| thangp | vilobhmm1: hey | 17:01 |
| mriedem | asselin_: is it possible to get some metadata about the attached volume from cinder after the operation and compare to before the attach to see if it changed? | 17:02 |
| vilobhmm1 | thangp : for https://review.openstack.org/#/c/163641/ | 17:03 |
| mtreinish | mriedem: heh, now you're just trying to confuse people :) | 17:03 |
| *** anuragpalsule1 has quit IRC | 17:03 | |
| vilobhmm1 | i would like to do a peicemeal appraoch and check in the basic infrastructure changes first | 17:03 |
| mriedem | well it looks like in the luks case a header is written to the volume | 17:04 |
| mriedem | but tempest isn't going to want to try and read that | 17:04 |
| * mtreinish disappears to get food | 17:04 | |
| vilobhmm1 | and they go with the chnages related to where these methods are called and the corresponding unit test | 17:04 |
| asselin_ | mriedem, maybe... | 17:04 |
| vilobhmm1 | does it seem ok to you ? thangp : ^^^ | 17:04 |
| *** Mandell has joined #openstack-cinder | 17:07 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Fixed encrypted property for 3PAR FC and iSCSI drivers https://review.openstack.org/170346 | 17:07 |
| thangp | vilobhmm1: you should be changing the code that calls CG, and should not really introduce code that is not used | 17:09 |
| thangp | vilobhmm1: that way, there is a better chance to find bugs and make changes more effectively | 17:09 |
| thangp | vilobhmm1: fwiw, i do agree with michal | 17:10 |
| thangp | vilobhmm1: the object version should not be changed unless there were changes to the db | 17:10 |
| *** Miouge has quit IRC | 17:11 | |
| vilobhmm1 | hmm ok thangp will submit a new patch set then…agreed regarding object version ….also regarding quota objects https://review.openstack.org/#/c/163232/ | 17:11 |
| thangp | vilobhmm1: you could start with the common cases in one patch, then do follow up patches to handle more specific ones | 17:11 |
| thangp | vilobhmm1: in a dependency chain | 17:12 |
| vilobhmm1 | sounds good | 17:12 |
| vilobhmm1 | thanks | 17:12 |
| *** anuragpalsule has quit IRC | 17:14 | |
| openstackgerrit | Bob Callaway proposed openstack/cinder: Fix LUN misalignment issue with NetApp iSCSI drivers https://review.openstack.org/162376 | 17:22 |
| *** sk69 has quit IRC | 17:24 | |
| *** patrickeast has joined #openstack-cinder | 17:28 | |
| *** Mandell has quit IRC | 17:29 | |
| *** julim has quit IRC | 17:34 | |
| *** julim has joined #openstack-cinder | 17:37 | |
| *** marcusvrn has quit IRC | 17:39 | |
| *** annashen has joined #openstack-cinder | 17:47 | |
| *** ebalduf has joined #openstack-cinder | 17:50 | |
| *** Mandell has joined #openstack-cinder | 17:51 | |
| *** tshefi has joined #openstack-cinder | 17:51 | |
| *** jdurgin has joined #openstack-cinder | 17:53 | |
| *** ebalduf has quit IRC | 17:54 | |
| *** rongze has joined #openstack-cinder | 18:03 | |
| *** timcl1 has quit IRC | 18:05 | |
| *** timcl has joined #openstack-cinder | 18:06 | |
| jbernard | jgriffith: i think i have an understanding of the ceph issue now | 18:06 |
| *** timcl1 has joined #openstack-cinder | 18:07 | |
| *** esker has quit IRC | 18:09 | |
| *** timcl has quit IRC | 18:10 | |
| *** harlowja_away is now known as harlowja | 18:11 | |
| mtreinish | asselin_: so do you have a proposal to make the tempest test better, or add an additional test to catch that issue? | 18:14 |
| asselin_ | mtreinish, no, not yet... | 18:14 |
| *** crose has quit IRC | 18:18 | |
| *** sgotliv has joined #openstack-cinder | 18:19 | |
| *** e0ne has joined #openstack-cinder | 18:26 | |
| *** theanalyst has quit IRC | 18:26 | |
| *** Mandell has quit IRC | 18:28 | |
| *** Mandell has joined #openstack-cinder | 18:30 | |
| *** theanalyst has joined #openstack-cinder | 18:30 | |
| *** Mandell has quit IRC | 18:31 | |
| jgriffith | jbernard: oh? Awesome! | 18:32 |
| jbernard | jgriffith: when tempest runs on lvm, the volume created from the image contains kernel_id and ramdisk_id metadata in the 'cinder show' output | 18:32 |
| jbernard | jgriffith: and this metadata is missing from a similarly created ceph volume | 18:33 |
| jbernard | jgriffith: and i *believe* this difference is causing the boot failures | 18:33 |
| jgriffith | jbernard: makes snese | 18:33 |
| jgriffith | sense | 18:33 |
| jgriffith | even :) | 18:33 |
| jbernard | jgriffith: however :) i just created manually two volumes from a UEC image, one ceph and one lvm, and neither have the metadata | 18:33 |
| jbernard | jgriffith: so i may just be losing my mind finally | 18:33 |
| jgriffith | jbernard: haha... don't do that :) | 18:34 |
| jbernard | it's probably too late :) | 18:34 |
| jgriffith | jbernard: you want some help on this? | 18:34 |
| jgriffith | jbernard: I can fire it up tonight when I land | 18:34 |
| jgriffith | happy to see if I can help | 18:34 |
| jbernard | jgriffith: are you able to confirm the presense of kernel_id and ramdisk_id in a volume created from a UEC image? | 18:34 |
| jbernard | jgriffith: in the 'show' output | 18:35 |
| jgriffith | jbernard: well... I run it in my CI tests | 18:35 |
| jgriffith | jbernard: so I'll have to see if I can figure out what "different | 18:35 |
| jbernard | the differece im seeing is this: | 18:35 |
| jgriffith | jbernard: so I could use the fancy devstack/ceph scripts I would imagine | 18:35 |
| jbernard | when tempest runs agains lvm, it creates a volume from a glance iamge | 18:36 |
| jbernard | and then tells nova to boot from it | 18:36 |
| jbernard | and this volume contains the correct kernel and ramdisk metadata | 18:36 |
| jgriffith | jbernard: right | 18:36 |
| jbernard | the same test with ceph, and the kernel and ramdisk are missing | 18:36 |
| jgriffith | jbernard: oh.. the "volume" contains it | 18:36 |
| jbernard | the volume_image_metadata | 18:37 |
| jgriffith | jbernard: which goes back to my suspicion about not using the standard image tools | 18:37 |
| jgriffith | jbernard: but I don't know if you've already squashed that | 18:37 |
| jbernard | yeah, the rbd driver skips some of the routines called by the lvm driver | 18:37 |
| jgriffith | jbernard: right... and my driver uses said routines as well and works | 18:38 |
| jbernard | but ive got a lot of digging and testing to do before i really follow where it gets left out | 18:38 |
| jbernard | but this is the best theory ive got so far :) | 18:38 |
| jbernard | so im going with it! | 18:38 |
| jgriffith | jbernard: :) | 18:38 |
| *** shyama has quit IRC | 18:39 | |
| jbernard | the bit that confuses me now is on volume creation from an image with cinderclient - it appears to still be missing even on lvm | 18:39 |
| jbernard | which is not consistent with my understanding | 18:39 |
| *** bkopilov has joined #openstack-cinder | 18:40 | |
| jbernard | jgriffith: tempest does not use the cinderclient interface to communicate with cinder | 18:41 |
| jbernard | jgriffith: could there also be a bug in cinderclient? | 18:41 |
| e0ne | thingee: hi. looking on your patch for cinderclient. it broke my env:(. i'll take a look on it closer later today | 18:41 |
| jgriffith | jbernard: not following... | 18:41 |
| jbernard | jgriffith: when i create a volume manually on lvm from a UEC image, the metadata appears to be missing | 18:42 |
| jgriffith | jbernard: you're saying "cinder create --image-id xxx" doesn't work? | 18:42 |
| *** abhijeetm has joined #openstack-cinder | 18:42 | |
| *** Apoorva has quit IRC | 18:42 | |
| jbernard | jgriffith: im not sure how tempest managed to have it there | 18:42 |
| jbernard | jgriffith: right | 18:42 |
| jgriffith | jbernard: but Tempest calls via client | 18:42 |
| jbernard | ok, so im really missing something then | 18:43 |
| *** annashen has quit IRC | 18:43 | |
| jgriffith | jbernard: although may be a different version? | 18:43 |
| jbernard | anything's possible, im hanging on by a very thin thread | 18:43 |
| jbernard | what i do know, is that tempest seems to succeed when kernel_id and ramdisk_id are present in the cinder metadata | 18:43 |
| jbernard | and this seems to be the case for lvm volumes | 18:43 |
| jbernard | and not for ceph volumes | 18:43 |
| *** crose has joined #openstack-cinder | 18:44 | |
| jungleboyj | jgriffith: Other than Nova and Horizon, are there other components that use Cinder's APIs? | 18:45 |
| jbernard | jungleboyj: glance for copy volume to image? | 18:46 |
| *** abhijeetm has quit IRC | 18:46 | |
| jgriffith | jungleboyj: they don't use them directly either | 18:47 |
| jgriffith | jungleboyj: they go through keystone client endpoint etc | 18:47 |
| e0ne | jungleboy: heat? | 18:47 |
| jbernard | jungleboyj: i think devstack uses them to break everything before i run my tests also | 18:47 |
| jungleboyj | Wondering, as we try to remove V1 what all will break. | 18:48 |
| jgriffith | jungleboyj: jbernard hmm... that would be weird for them go direct in some cases like that | 18:50 |
| e0ne | jungleboyj: cinderclient by default use api v1 | 18:50 |
| e0ne | may be it is your case | 18:50 |
| jgriffith | e0ne: that should be updated now | 18:50 |
| *** ebalduf has joined #openstack-cinder | 18:50 | |
| jgriffith | FWIW, the default should have finally switched to V2 after over a year :( | 18:50 |
| jungleboyj | e0ne: Yeah, I am hoping that is changed by now. | 18:51 |
| e0ne | jgriffith: agree. we've got two patches for it: thingee's with version autodiscovering and my | 18:51 |
| jungleboyj | jgriffith: I think it has as we had some issues when that change initially went in. | 18:51 |
| jgriffith | jungleboyj: so I know it did... just trying to be "polite" and use the IIRC :) | 18:51 |
| jgriffith | jungleboyj: and teh "should have" | 18:52 |
| jgriffith | :) | 18:52 |
| e0ne | https://github.com/openstack/python-cinderclient/blob/master/cinderclient/shell.py#L55 | 18:52 |
| jgriffith | e0ne: that sucks... but good thing is somebody did half the work at least | 18:53 |
| jgriffith | e0ne: and updated it in devstack/nova etc | 18:53 |
| e0ne | jgriffith: https://review.openstack.org/146541 :) | 18:53 |
| jgriffith | e0ne: but that's messed up that was missed there | 18:53 |
| *** rhefner has quit IRC | 18:54 | |
| e0ne | jgriffith: we've got the same issue with our distro few months ago | 18:54 |
| jgriffith | e0ne: Oh... well, ok then I guess someobdy needs to take it up with thingee | 18:54 |
| jungleboyj | Yeah, looks like we should get that changed. | 18:55 |
| jgriffith | e0ne: seems REALLY bad to have this part in part out things propagating in all of the Cinder code at this point | 18:55 |
| e0ne | jgriffith: here is thingee's patch: https://review.openstack.org/145613 | 18:55 |
| *** ebalduf has quit IRC | 18:55 | |
| e0ne | jgriffith: btw, rally use api v1 :( //cc boris-42 | 18:55 |
| jungleboyj | Unfortunately, it appears we are out of luck until L at this point. | 18:56 |
| *** Mandell has joined #openstack-cinder | 18:56 | |
| jgriffith | e0ne: I'm trying to bring up client.py | 18:56 |
| jgriffith | but I think I just lost wifi again | 18:56 |
| jgriffith | hehe.. I'm back | 18:56 |
| e0ne | welcome back:) | 18:56 |
| jgriffith | e0ne: :) | 18:57 |
| jgriffith | well that's crap https://github.com/openstack/python-cinderclient/blob/master/cinderclient/client.py#L149 | 18:57 |
| jgriffith | Maybe we should just abandon the whole V2 thing at this point :) | 18:58 |
| jgriffith | Just kidding obvoulsy | 18:58 |
| jgriffith | but I think that mess needs straightened out before Kilo goes out the door | 18:59 |
| thingee | ugh, yeah this recent patch land is going to make things difficult for version discovery | 18:59 |
| jungleboyj | It is scary how complicated this is. | 18:59 |
| e0ne | jgriffith: looking throw code. #L149 is it used for keystone version? | 18:59 |
| *** avishay__ has joined #openstack-cinder | 19:00 | |
| jgriffith | e0ne: nah... we set it up in keystone IIRC. Keystone has "both" | 19:00 |
| jgriffith | e0ne: but it's funky for packagers now | 19:00 |
| jungleboyj | jgriffith: Yeah, having both is kind-of a workaround that we need to fix in L | 19:01 |
| jgriffith | jungleboyj: yeah... but I think we just forced another reset on removal | 19:01 |
| jgriffith | jungleboyj: in other words deprecation periods don't count if you couldn't use them IMO :) | 19:02 |
| jungleboyj | :-) | 19:02 |
| thingee | jungleboyj: the patch I have posted right now allows volume, volumev2 legacy endpoints to still work, plus just using volume service pointed to x.x.x.x:8776 for discovery | 19:03 |
| *** Rockyg has joined #openstack-cinder | 19:03 | |
| thingee | jungleboyj: I'm trying to work through the issues of this recent patch merge that broke it | 19:03 |
| thingee | e0ne: ^ | 19:03 |
| thingee | jungleboyj, e0ne: see the unit tests | 19:03 |
| thingee | jungleboyj, e0ne: there also appears to be some breakage on the nova side that I need to figure out | 19:04 |
| thingee | according to tempest run | 19:04 |
| jungleboyj | thingee: You are talking about: https://review.openstack.org/145613 | 19:05 |
| thingee | yes | 19:05 |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Correct cinder hacking check numbering https://review.openstack.org/170959 | 19:06 |
| e0ne | thingee: it's the same error as i've got on my env | 19:06 |
| thingee | e0ne: yea | 19:06 |
| thingee | e0ne: do you have any env variables set? | 19:07 |
| thingee | in my tests this should be defaulting to version 1 | 19:07 |
| thingee | which if that endpoint is enabled, should be discovered | 19:07 |
| e0ne | thingee: afaik, no. but it's from rally output. so, i'm not sure | 19:08 |
| thingee | e0ne: yeah I get that. Just trying to understand what's different that rally is doing. | 19:08 |
| e0ne | thingee: rally uses v1 api | 19:08 |
| thingee | or nova's api | 19:08 |
| thingee | I get that | 19:08 |
| thingee | this supports v1 and v2 legacy endpoints and discovery endpoint | 19:09 |
| thingee | just trying to understand what's different that's causing this to trigger | 19:09 |
| e0ne | i did'nt add any endpoints myself | 19:09 |
| thingee | Having trouble reproducing from manual tests. I guess I can drop in a breakpoint and catch it for that tempest test | 19:09 |
| *** avishay__ has quit IRC | 19:09 | |
| thingee | e0ne: I get that as well..it's default devstack endpoints | 19:09 |
| thingee | e0ne: I'm just trying to understand what these things do differently in issuing requests to the client | 19:10 |
| e0ne | thingee: few more warmings: http://paste.openstack.org/show/198742/ | 19:11 |
| thingee | e0ne: that's normal unfortunately | 19:11 |
| e0ne | ok | 19:11 |
| e0ne | thingee: i've got an idea | 19:12 |
| e0ne | thingee: you changed https://review.openstack.org/#/c/145613/4/cinderclient/client.py Client arguments | 19:13 |
| e0ne | line #588 | 19:13 |
| e0ne | thingee: rally doesn't pass 'version as named argument - https://github.com/stackforge/rally/blob/master/rally/osclients.py#L226 | 19:14 |
| *** rongze has quit IRC | 19:15 | |
| thingee | e0ne: ah ha | 19:15 |
| thingee | e0ne: thanks | 19:15 |
| e0ne | thingee: np. | 19:15 |
| thingee | jungleboyj: yeah I mentioned this work was a mess at the midcycle meetup. Unfortunately we'll have to live with this code until we can feel comfortable with removing volumev2 support. | 19:16 |
| thingee | jgriffith: unfortunately you weren't present during the midcycle meetup when I went over this patch. It didn't really help anyways to go over it, because I think I confused people more. | 19:17 |
| thingee | I believe it ended with DuncanT saying "well thank god you're looking into it" | 19:17 |
| jungleboyj | thingee: So, we are going to leave this as is for K and try to get it fixed early in L? | 19:17 |
| thingee | I kind of have to though, since I introduced volumev2 | 19:17 |
| thingee | does the client matter with the timeline? | 19:18 |
| *** boris-42 has quit IRC | 19:18 | |
| thingee | jgriffith: ^ | 19:18 |
| *** uschreiber_ has joined #openstack-cinder | 19:18 | |
| e0ne | thingee: it doesn't. but the last relaes was before Juno, afaik | 19:20 |
| e0ne | https://review.openstack.org/#/c/117436/ - one more feature, that will be great to have in a next release | 19:21 |
| *** uschreiber_ has quit IRC | 19:23 | |
| *** eharney has quit IRC | 19:24 | |
| alexpilotti | thingee: hi there | 19:29 |
| alexpilotti | thingee: we got some pretty good results with the SMFS CI | 19:30 |
| alexpilotti | s/SMFS/SMBFS/ | 19:30 |
| thingee | alexpilotti: I was looking at the logs last night and was noticing only console was being posted. Doesn't seem to meet my initial request out through april 1 to now | 19:30 |
| alexpilotti | thingee: during the weekend there was a network outage | 19:31 |
| alexpilotti | thingee: unfortunately we had to wait for Monday for getting that fixed | 19:32 |
| thingee | alexpilotti: I'll have to review things again. I just did a round of readding some drivers earlier. a little busy at the moment | 19:32 |
| alexpilotti | understood, just wanted to let you know we are ready for getting “reviewed” | 19:33 |
| thingee | e0ne: it would probably be good to get things cached on discovery | 19:37 |
| thingee | e0ne: I don't know how to do that yet | 19:37 |
| thingee | and to rediscover if the cached stuff fails | 19:37 |
| e0ne | thingee: maby we should discuss it in the design summit? it it the second place where we need caching | 19:38 |
| e0ne | and i'm pretty sure that it's not the last place | 19:38 |
| e0ne | thingee: we need some caching for json schema for glance api usage | 19:39 |
| thingee | e0ne: will worse case for this though is four keystone hits to do discovery | 19:39 |
| thingee | that's only if things are really screwy though on server side | 19:40 |
| thingee | like v1 being only available on volumev2 :) | 19:40 |
| thingee | not sure why someone would do that, but I didn't rule it out | 19:40 |
| e0ne | :) | 19:40 |
| thingee | writing this really broke my head a few times | 19:40 |
| thingee | and I'm not proud of it, but willing to take refactor suggestions | 19:41 |
| e0ne | "if user coud do something, he _will_ do it" | 19:41 |
| thingee | he or she :) | 19:41 |
| e0ne | :) | 19:41 |
| thingee | e0ne: ok got tests passing again with resolving merge conflicts | 19:43 |
| *** Apoorva has joined #openstack-cinder | 19:43 | |
| *** annashen has joined #openstack-cinder | 19:43 | |
| thingee | e0ne: now to have client.Client default to a version and we should be good! | 19:43 |
| e0ne | thingee: cool! i'll test it once new patch will be available | 19:43 |
| thingee | e0ne: are you coming to the summit? | 19:44 |
| e0ne | thingee: i hope. need to get visa first | 19:45 |
| *** alexpilotti has quit IRC | 19:48 | |
| *** annashen has quit IRC | 19:49 | |
| *** ebalduf has joined #openstack-cinder | 19:51 | |
| *** ebalduf has quit IRC | 19:55 | |
| *** annashen has joined #openstack-cinder | 19:57 | |
| thingee | e0ne: apparently you can't use imports in default kwargs | 19:58 |
| thingee | e0ne: I wanted to do def Client(self, version=shell.DEFAULT_OS_VOLUME_API_VERSION, **kwargs) | 19:59 |
| thingee | guess i can I just do the check inside the function | 19:59 |
| *** annegentle has quit IRC | 20:01 | |
| e0ne | thingee: it should work | 20:02 |
| e0ne | thingee: but in your case, it will be circular dependency on modules | 20:03 |
| thingee | I guess then we need to break the default into another module | 20:04 |
| e0ne | so as i see, it won't work in current case | 20:04 |
| thingee | defaults | 20:04 |
| *** alexpilotti has joined #openstack-cinder | 20:04 | |
| thingee | default constants | 20:04 |
| * e0ne bb tomorrow (in ~12 hours) | 20:13 | |
| *** Apoorva has quit IRC | 20:15 | |
| *** lpabon has quit IRC | 20:15 | |
| *** Apoorva has joined #openstack-cinder | 20:15 | |
| *** rongze has joined #openstack-cinder | 20:15 | |
| *** alexpilotti has quit IRC | 20:16 | |
| *** ociuhandu has quit IRC | 20:16 | |
| *** tshefi has quit IRC | 20:16 | |
| *** bkopilov has quit IRC | 20:17 | |
| *** emagana has quit IRC | 20:19 | |
| *** rongze has quit IRC | 20:21 | |
| *** bkopilov has joined #openstack-cinder | 20:22 | |
| jungleboyj | thingee: You know anything about the plans going forward to get Cinder to work with Docker? | 20:23 |
| anteaya | jungleboyj: :( | 20:23 |
| jungleboyj | anteaya: Yes? | 20:24 |
| anteaya | jungleboyj: you posted your CI status to the mailing list | 20:24 |
| anteaya | and so are others following your example | 20:24 |
| *** emagana has joined #openstack-cinder | 20:24 | |
| anteaya | :( | 20:24 |
| jungleboyj | anteaya: What is wrong with that? | 20:25 |
| anteaya | this is the page for status updates for CI systems | 20:25 |
| anteaya | https://wiki.openstack.org/wiki/ThirdPartySystems | 20:25 |
| anteaya | look how many CI systems there are | 20:25 |
| anteaya | I have worked hard to have them update their status in one place that people can find | 20:25 |
| anteaya | the wikipage | 20:25 |
| anteaya | with a link to individual account pages | 20:26 |
| anteaya | you can put anything you want in your individual account page | 20:26 |
| anteaya | that is what it is for | 20:26 |
| mtanino | jungleboyj: Hi, I'm trying to reproduce this https://bugs.launchpad.net/cinder/+bug/1440201, but I couldn't.. | 20:26 |
| openstack | Launchpad bug 1440201 in Cinder "reboot of a nova instance appears to leave attached volumes in a bad state" [Medium,New] | 20:26 |
| anteaya | thingee: might I suggest a place in the cinder weekly meeting for ci operators who would like to report system status to do so? | 20:27 |
| thingee | anteaya: that might be my fault too. I didn't instruct people at the beginning to not post anything to the ML. So I think people are just doing it to get my attention so their driver gets readded | 20:27 |
| anteaya | yes | 20:27 |
| anteaya | I get that | 20:27 |
| anteaya | and I understand | 20:27 |
| anteaya | folks are frantic and confused | 20:27 |
| anteaya | but having everyone post every change to the ml isn't going to work for folks | 20:27 |
| anteaya | and ci folks don't take instruction well | 20:27 |
| anteaya | as we all have learned | 20:28 |
| *** krtaylor has quit IRC | 20:28 | |
| anteaya | so given the state of panic | 20:28 |
| mtanino | jungleboyj: How did you reboot the VM? from Horizon(soft / hard) or reboot command inside the vm or etc | 20:28 |
| anteaya | how do we work together and get them using their wikipage? | 20:28 |
| thingee | anteaya: I can contact people individually like I have been. seems to get their attention | 20:29 |
| thingee | anteaya: and mention in the meeting | 20:29 |
| jungleboyj | anteaya: My apologies. As I just responded I was using the ML since that was where thingee started the discussion. | 20:29 |
| jungleboyj | Can save other discussion for the weekly meeting. | 20:30 |
| anteaya | thingee: okay the contacting people individually needs to be acknowledged that noone else does this | 20:30 |
| anteaya | so if you want to do that, that is your call, but I don't do that | 20:30 |
| anteaya | so if they are surprised that I don't and won't do that, then they will be surprised | 20:31 |
| jungleboyj | anteaya: I have been yelled at in the past for not communicating publically. Therefor the ML message. | 20:31 |
| anteaya | jungleboyj: I understanding, you were communicating | 20:31 |
| anteaya | jungleboyj: and I appreciate that | 20:31 |
| anteaya | so yeah, the having updates at the weekly meeting would be good, if that would work for you | 20:31 |
| anteaya | and please let me know if anyone is having difficulty understanding how to create and update a wikipage for their status | 20:32 |
| jungleboyj | anteaya: Ok, will do. | 20:32 |
| anteaya | jungleboyj: thank you | 20:32 |
| jungleboyj | anteaya: Welcome. | 20:32 |
| anteaya | jungleboyj: and I hope you know I'm not upset with you, as you can listen | 20:32 |
| jungleboyj | mtanino: it was a 'nova reboot <vm>' | 20:32 |
| anteaya | jungleboyj: it is the hoard of others that tip me over the edge | 20:33 |
| mtanino | jungleboyj: Thank you. let me try | 20:33 |
| *** krtaylor has joined #openstack-cinder | 20:33 | |
| thingee | e0ne: sorry taking longer than I expected | 20:33 |
| asselin_ | thingee, what is you opiion on this bug w.r.t. other drivers passing it? https://bugs.launchpad.net/tempest/+bug/1440227 | 20:34 |
| openstack | Launchpad bug 1440227 in tempest "encrypted volume tests don't check if a volume is actually encrypted" [Undecided,New] | 20:34 |
| thingee | asselin_: busy atm | 20:35 |
| asselin_ | ok | 20:35 |
| thingee | asselin_: I'll get back to you, promise | 20:35 |
| jungleboyj | The suggest workaround was to use 'nova reboot --hard' The system where we saw it, however, hasn't been available for me to try again. | 20:35 |
| lpetrut | smb | 20:43 |
| *** Maike has joined #openstack-cinder | 20:43 | |
| *** lpetrut has left #openstack-cinder | 20:44 | |
| e0ne | thingee: i can take a look and test your patch tomorrow morning. | 20:45 |
| *** marcusvrn2 has quit IRC | 20:49 | |
| *** ebalduf has joined #openstack-cinder | 20:52 | |
| *** akerr has quit IRC | 20:54 | |
| *** Rockyg has quit IRC | 20:56 | |
| *** ebalduf has quit IRC | 20:56 | |
| *** diemt has quit IRC | 20:58 | |
| tbarron | jungleboyj: the Docker question above intrigues me. Today we rely on the hypervisor for tenant isolation, right? With containers, do we lose the isolation? | 21:01 |
| tbarron | ^^^ Newbie questions above. If y'all can point to some place that is discussed/resolved, I'll look. Thanks. | 21:02 |
| *** bswartz has quit IRC | 21:05 | |
| *** thangp has quit IRC | 21:09 | |
| *** emagana has quit IRC | 21:10 | |
| *** emagana has joined #openstack-cinder | 21:13 | |
| *** e0ne has quit IRC | 21:14 | |
| *** Bsony has joined #openstack-cinder | 21:14 | |
| *** julim has quit IRC | 21:14 | |
| *** Yogi1 has quit IRC | 21:16 | |
| *** rongze has joined #openstack-cinder | 21:16 | |
| *** dustins_ has quit IRC | 21:19 | |
| *** rongze has quit IRC | 21:21 | |
| *** ronis has quit IRC | 21:25 | |
| *** Bsony has quit IRC | 21:28 | |
| *** Maike has quit IRC | 21:30 | |
| *** annegentle has joined #openstack-cinder | 21:35 | |
| jungleboyj | tbarron: Don't fully understand your question, but I don't fully understand Docker either. | 21:36 |
| tbarron | jungleboyj: well, I may be confused :-) | 21:37 |
| jungleboyj | tbarron: I know I am. | 21:37 |
| tbarron | I'm thinking that in Nova the hypervisor maps a device, say sdg, into an instance so that it shows up as, say, vdc. | 21:38 |
| tbarron | So it's visible in that instance and not in another. | 21:38 |
| jungleboyj | Right. | 21:38 |
| jungleboyj | Correct. | 21:38 |
| tbarron | That gives "tenant isolation". | 21:38 |
| tbarron | If we have containers instead of genuine VMs, then there may be a challenge to make sure I in my instance can't see your block device in your instance. | 21:39 |
| tbarron | Maybe that problem is already solved. | 21:39 |
| smcginnis | tbarron: If I remember right from the little bit I've played around with containers you can still control what local storage gets exposed through. | 21:39 |
| jungleboyj | tbarron: It sounds like it currently is not. From what I can find. | 21:39 |
| *** crose has quit IRC | 21:40 | |
| smcginnis | tbarron: But I know very little about it. | 21:40 |
| tbarron | smginnis: if that problem is solved, then I would think we'd be most of the way there. | 21:40 |
| jungleboyj | It sounds like the closest thing they have to support for Cinder is mapping a volume to the Docker host and then manually setting up containers to make use of that volume though. As far as tenant isolation, I have no idea. | 21:40 |
| openstackgerrit | Alex Meade proposed openstack/cinder: Add tracing facility to NetApp ONTAP drivers https://review.openstack.org/157439 | 21:40 |
| tbarron | jungleboyj: smcginnis: anyway, I'm just registering interest. If you folks bump into resources, etc. on this, I'd be interested in pointers. | 21:42 |
| jungleboyj | I have been doing more googling around. Seems like people either have many approaches or things like Flocker have their own 'volume' concept. Otherwise, there are some in process patches out there to improve Docker support for Cinder Volumes. | 21:43 |
| *** melwitt has joined #openstack-cinder | 21:43 | |
| openstackgerrit | Mike Perez proposed openstack/python-cinderclient: Enable version discovery https://review.openstack.org/145613 | 21:45 |
| thingee | jungleboyj: ^ | 21:45 |
| *** annegentle has quit IRC | 21:46 | |
| jungleboyj | thingee: looking. | 21:46 |
| thingee | jungleboyj: recommend just waiting for tempest first :) | 21:47 |
| thingee | rebased though | 21:47 |
| thingee | jungleboyj: also removed the sig changes to client.Client() .. that would've been disaster and I'm sure that's why things failed in the gate. | 21:48 |
| jungleboyj | thingee: Ok. | 21:48 |
| thingee | jungleboyj: this though https://review.openstack.org/#/c/145613/5/cinderclient/tests/unit/test_shell.py | 21:49 |
| thingee | jungleboyj: 127.0.0.1:8776 and one volume service | 21:49 |
| thingee | exciting times | 21:49 |
| openstackgerrit | xing-yang proposed openstack/cinder: Create initiator id if not exist in VMAX driver https://review.openstack.org/170689 | 21:50 |
| *** mriedem is now known as mriedem_away | 21:50 | |
| jungleboyj | thingee: That will be good. | 21:51 |
| thingee | jungleboyj: if you take a look at my comment to smcginnis here https://review.openstack.org/#/c/145613/3/cinderclient/client.py | 21:51 |
| thingee | jungleboyj: it explains the reason for so many workarounds | 21:52 |
| *** dims has quit IRC | 21:52 | |
| jungleboyj | thingee: This code to try the other API version baffles me. | 21:52 |
| thingee | it's to work with the volumev2 being set a decorator on somethings which forces service_type to be volumev2 | 21:52 |
| jungleboyj | thingee: Can't that cause issues? | 21:52 |
| thingee | jungleboyj: according to the ops mailing list, they wanted fallback support | 21:53 |
| thingee | jungleboyj: not sure if you follow that | 21:53 |
| *** ebalduf has joined #openstack-cinder | 21:53 | |
| jungleboyj | thingee: Ok. | 21:53 |
| thingee | jungleboyj: if not version input is given | 21:53 |
| thingee | not=no | 21:53 |
| thingee | jungleboyj: as I mentioned earlier, instead of people complaining, would appreciate feedback on how to better refactor things for the different situations. | 21:54 |
| *** dims has joined #openstack-cinder | 21:54 | |
| *** dims has quit IRC | 21:54 | |
| thingee | volume set to v1 w/ volumev2 set to v2, volumev2 set to v1, volume set to v1, etc | 21:55 |
| thingee | people are doing weird stuff | 21:55 |
| *** dims has joined #openstack-cinder | 21:55 | |
| thingee | maybe we should just break stuff if they do weird things | 21:55 |
| jungleboyj | thingee: I don't think they are doing it all intentionally. I think some of it results from people just not moving to V2 like they were supposed to. | 21:56 |
| *** ebalduf has quit IRC | 21:57 | |
| thingee | jungleboyj: for the cases where people move to the discovery url, they won't have so many keystone api hits | 21:57 |
| thingee | at the very leat. | 21:57 |
| thingee | least* | 21:57 |
| smcginnis | Yeah, that version discovery was a little confusing. | 22:02 |
| smcginnis | A couple times through and it made sense though | 22:02 |
| thingee | smcginnis: :( | 22:02 |
| smcginnis | No idea how to make it any more clear. | 22:02 |
| *** alexpilotti has joined #openstack-cinder | 22:02 | |
| smcginnis | thingee: I think you did as good as could be done. :) | 22:02 |
| thingee | jamielennox|away: https://review.openstack.org/#/c/145613/5 | 22:03 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Fix instance live-migration with attached volumes https://review.openstack.org/170990 | 22:07 |
| *** scottda has quit IRC | 22:12 | |
| thingee | jungleboyj: so far it passes icehouse tempest | 22:13 |
| thingee | \o/ | 22:13 |
| jungleboyj | That is good. | 22:13 |
| thingee | I'll feel good when we can remove the volumev2 stuff out completely | 22:14 |
| thingee | :) | 22:14 |
| *** alexpilotti has quit IRC | 22:14 | |
| thingee | I would do the patch now and have it on standby, but I'm sure it'll be useless after rebase in the future | 22:14 |
| *** alexpilotti has joined #openstack-cinder | 22:14 | |
| jungleboyj | Agreed. So confusing. | 22:14 |
| thingee | jungleboyj: it's nice that we have the discovery stuff *now* | 22:15 |
| thingee | would've been good in the past | 22:15 |
| thingee | to avoid this all together | 22:15 |
| *** annashen has quit IRC | 22:15 | |
| *** akerr has joined #openstack-cinder | 22:15 | |
| jungleboyj | Agreed. | 22:16 |
| *** rongze has joined #openstack-cinder | 22:17 | |
| *** akerr_ has joined #openstack-cinder | 22:18 | |
| *** alexpilotti has quit IRC | 22:19 | |
| *** alexpilotti has joined #openstack-cinder | 22:20 | |
| *** akerr has quit IRC | 22:21 | |
| *** rongze has quit IRC | 22:22 | |
| *** annashen has joined #openstack-cinder | 22:24 | |
| *** alexpilotti has quit IRC | 22:25 | |
| *** alexpilotti has joined #openstack-cinder | 22:26 | |
| *** alexpilotti has quit IRC | 22:29 | |
| *** ociuhandu has joined #openstack-cinder | 22:29 | |
| *** _cjones_ has joined #openstack-cinder | 22:30 | |
| *** Mandell has quit IRC | 22:33 | |
| *** rushil has quit IRC | 22:33 | |
| *** Bsony has joined #openstack-cinder | 22:34 | |
| *** akerr_ has quit IRC | 22:38 | |
| openstackgerrit | Mike Perez proposed openstack/python-cinderclient: Enable version discovery https://review.openstack.org/145613 | 22:39 |
| *** __afazekas has quit IRC | 22:39 | |
| *** Bsony has quit IRC | 22:39 | |
| *** annashen has quit IRC | 22:40 | |
| *** annashen has joined #openstack-cinder | 22:43 | |
| *** akerr has joined #openstack-cinder | 22:44 | |
| *** annegentle has joined #openstack-cinder | 22:44 | |
| *** bswartz has joined #openstack-cinder | 22:48 | |
| *** Mandell has joined #openstack-cinder | 22:48 | |
| *** markvoelker has quit IRC | 22:52 | |
| *** ebalduf has joined #openstack-cinder | 22:54 | |
| *** ebalduf has quit IRC | 22:58 | |
| *** devlaps has joined #openstack-cinder | 22:59 | |
| *** annegentle has quit IRC | 22:59 | |
| *** annegent_ has joined #openstack-cinder | 23:00 | |
| thingee | smcginnis: https://etherpad.openstack.org/p/cinder-kilo-rc-priorities | 23:03 |
| thingee | smcginnis: just so you know | 23:04 |
| *** jamielennox|away is now known as jamielennox | 23:07 | |
| *** virmitio has quit IRC | 23:09 | |
| *** chlong has joined #openstack-cinder | 23:10 | |
| *** Lee1092 has quit IRC | 23:10 | |
| *** annegent_ has quit IRC | 23:13 | |
| *** _cjones_ has quit IRC | 23:13 | |
| *** akerr has quit IRC | 23:15 | |
| *** jaypipes has quit IRC | 23:16 | |
| *** rongze has joined #openstack-cinder | 23:18 | |
| *** rmesta has quit IRC | 23:19 | |
| *** rhefner has joined #openstack-cinder | 23:22 | |
| *** rongze has quit IRC | 23:23 | |
| *** _cjones_ has joined #openstack-cinder | 23:24 | |
| *** markvoelker has joined #openstack-cinder | 23:27 | |
| asselin_ | thingee, please take a look at https://bugs.launchpad.net/tempest/+bug/1440227 since it could affect other drivers | 23:31 |
| openstack | Launchpad bug 1440227 in tempest "encrypted volume tests don't check if a volume is actually encrypted" [Undecided,New] | 23:31 |
| asselin_ | thingee, have to go. you can chat with leeantho or kmartin about it | 23:36 |
| *** annegent_ has joined #openstack-cinder | 23:36 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Switch to oslo_versionedobjects https://review.openstack.org/160209 | 23:37 |
| *** IanGovett has quit IRC | 23:40 | |
| *** IanGovett has joined #openstack-cinder | 23:40 | |
| *** annashen has quit IRC | 23:44 | |
| *** annegent_ has quit IRC | 23:48 | |
| *** Mandell has quit IRC | 23:50 | |
| *** markvoelker has quit IRC | 23:53 | |
| *** ebalduf has joined #openstack-cinder | 23:54 | |
| *** emagana has quit IRC | 23:54 | |
| *** Mandell has joined #openstack-cinder | 23:55 | |
| *** emagana has joined #openstack-cinder | 23:55 | |
| *** dannywilson has quit IRC | 23:59 | |
| *** ebalduf has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!