| *** arif-ali has quit IRC | 00:00 | |
| *** vokt has quit IRC | 00:01 | |
| *** vokt has joined #openstack-cinder | 00:01 | |
| *** arif-ali has joined #openstack-cinder | 00:04 | |
| *** vokt has quit IRC | 00:06 | |
| *** yamada-h has joined #openstack-cinder | 00:07 | |
| *** yamada-h has quit IRC | 00:12 | |
| *** markvoelker has quit IRC | 00:12 | |
| asselin_ | patrickeast, ok...so my test now has 4 fc devices attached! | 00:15 |
|---|---|---|
| *** setmason has joined #openstack-cinder | 00:16 | |
| patrickeast | so much bandwidth! | 00:16 |
| patrickeast | were you guys testing with multipath enabled before? | 00:17 |
| asselin_ | patrickeast, yes, multipath enabled, but only one fc path | 00:19 |
| *** ebalduf has joined #openstack-cinder | 00:19 | |
| asselin_ | I just switched to the common fc script and it picked up all 4 | 00:19 |
| asselin_ | will see how to limit it to e.g. two | 00:19 |
| asselin_ | or 'n' | 00:19 |
| patrickeast | asselin_: yea i was thinking about that, starts to get hard since we need a way to know which ones to try for, and what to do if we don’t get them | 00:20 |
| asselin_ | my version would skip and keep going...i'll do a diff and see what else is missing | 00:20 |
| patrickeast | asselin_: like should it abort if it doesnt get n? or try for n and wait maybe up to some threshold and then give up | 00:20 |
| asselin_ | patrickeast, oh I see....good question | 00:21 |
| asselin_ | well...I'll start with the simple: get n or abort | 00:21 |
| patrickeast | sounds like a good plan to me | 00:21 |
| *** ebalduf has quit IRC | 00:23 | |
| nikeshm | patriceast: i want to check recheck myvendor is this correct in layout.yaml of zuul http://paste.openstack.org/show/243414/ | 00:24 |
| nikeshm | its not working for me in sandbox | 00:24 |
| *** emagana has quit IRC | 00:25 | |
| *** heyun has joined #openstack-cinder | 00:25 | |
| *** IlyaG has quit IRC | 00:26 | |
| patrickeast | nikeshm: i think you might need a space before DotHill | 00:30 |
| patrickeast | mines: comment: (?i)^(Patch Set [0-9]+:)?( [\w\\+-]*)*(\n\n)?\s*recheck(( (?:bug|lp)[\s#:]*(\d+))|( Pure Storage CI))\s*$ | 00:30 |
| patrickeast | which other than the space is the same as yours | 00:30 |
| * asselin_ checking | 00:30 | |
| asselin_ | I use (?i)^(Patch Set [0-9]+:)?( [\w\\+-]*)*(\n\n)?\s*recheck\s*$ | 00:32 |
| patrickeast | ive got that one too | 00:32 |
| asselin_ | but seems this is what upstream has http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n20 | 00:32 |
| asselin_ | comment: (?i)^(Patch Set [0-9]+:)?( [\w\\+-]*)*(\n\n)?\s*(recheck|reverify) | 00:33 |
| asselin_ | will probably change to that | 00:33 |
| *** setmason_ has joined #openstack-cinder | 00:39 | |
| *** setmason has quit IRC | 00:41 | |
| *** ociuhandu has quit IRC | 00:41 | |
| *** setmason_ is now known as setmason | 00:41 | |
| *** dims_ has quit IRC | 00:42 | |
| *** annegentle has joined #openstack-cinder | 00:45 | |
| *** dims_ has joined #openstack-cinder | 00:47 | |
| *** Longgeek has joined #openstack-cinder | 00:49 | |
| *** tobe has joined #openstack-cinder | 00:52 | |
| *** IlyaG has joined #openstack-cinder | 00:54 | |
| *** IlyaG has quit IRC | 00:55 | |
| *** _cjones_ has quit IRC | 01:00 | |
| *** annegentle has quit IRC | 01:03 | |
| *** jwcroppe has quit IRC | 01:03 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Use snapshot objects instead of snapshot dictionaries https://review.openstack.org/185495 | 01:03 |
| *** jwcroppe has joined #openstack-cinder | 01:03 | |
| *** manishg has left #openstack-cinder | 01:04 | |
| *** jwcroppe has quit IRC | 01:08 | |
| *** pboldin has quit IRC | 01:09 | |
| *** garthb_ has quit IRC | 01:12 | |
| *** garthb has quit IRC | 01:12 | |
| *** markvoelker has joined #openstack-cinder | 01:13 | |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Add Pure Storage FibreChannel driver https://review.openstack.org/186605 | 01:15 |
| *** vilobhmm has quit IRC | 01:17 | |
| *** markvoelker has quit IRC | 01:18 | |
| *** tobe has quit IRC | 01:22 | |
| *** Lee1092 has joined #openstack-cinder | 01:23 | |
| *** pboldin has joined #openstack-cinder | 01:25 | |
| *** alexpilotti has joined #openstack-cinder | 01:28 | |
| *** emagana has joined #openstack-cinder | 01:32 | |
| *** dmz has quit IRC | 01:33 | |
| *** dims_ has quit IRC | 01:35 | |
| *** setmason_ has joined #openstack-cinder | 01:36 | |
| *** alexpilotti has quit IRC | 01:38 | |
| *** setmason has quit IRC | 01:39 | |
| *** setmason_ is now known as setmason | 01:39 | |
| *** dims_ has joined #openstack-cinder | 01:39 | |
| *** pboldin has quit IRC | 01:45 | |
| *** dims_ has quit IRC | 01:45 | |
| *** tobe has joined #openstack-cinder | 01:52 | |
| *** patrickeast has quit IRC | 01:54 | |
| *** emagana has quit IRC | 01:57 | |
| *** emagana has joined #openstack-cinder | 01:58 | |
| *** yamada-h has joined #openstack-cinder | 02:09 | |
| *** Apoorva has quit IRC | 02:13 | |
| *** yamada-h has quit IRC | 02:14 | |
| *** Longgeek has quit IRC | 02:16 | |
| *** Apoorva has joined #openstack-cinder | 02:20 | |
| *** mriedem has quit IRC | 02:20 | |
| nikeshm | asselin: hi | 02:22 |
| *** Apoorva has quit IRC | 02:24 | |
| *** rwsu has quit IRC | 02:25 | |
| *** earlephilhower has quit IRC | 02:26 | |
| *** setmason has quit IRC | 02:31 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Use snapshot objects instead of snapshot dictionaries https://review.openstack.org/185495 | 02:35 |
| asselin_ | nikeshm, hi | 02:38 |
| *** jungleboyj has joined #openstack-cinder | 02:38 | |
| nikeshm | asselin_: looks like my gearman is not working,so after recheck its not going to jenkins to run job | 02:40 |
| asselin_ | nikeshm, double-check your regex & /var/log/zuul/debug.log file | 02:40 |
| *** rajinir has joined #openstack-cinder | 02:43 | |
| *** Yogi1 has joined #openstack-cinder | 02:47 | |
| *** yamada-h has joined #openstack-cinder | 02:49 | |
| *** markvoelker has joined #openstack-cinder | 03:02 | |
| *** lixiaoy1 has joined #openstack-cinder | 03:03 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Multiple pools support enhancement in VNX cinder driver https://review.openstack.org/184001 | 03:04 |
| *** _cjones_ has joined #openstack-cinder | 03:04 | |
| *** _cjones_ has quit IRC | 03:05 | |
| *** rajinir has quit IRC | 03:05 | |
| *** _cjones_ has joined #openstack-cinder | 03:05 | |
| *** rajinir has joined #openstack-cinder | 03:06 | |
| *** markvoelker has quit IRC | 03:06 | |
| *** yamada-h has quit IRC | 03:07 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove the no-parallel setting from pretty_tox.sh https://review.openstack.org/186586 | 03:08 |
| *** yamada-h has joined #openstack-cinder | 03:09 | |
| *** yamada-h has quit IRC | 03:09 | |
| *** yamada-h has joined #openstack-cinder | 03:10 | |
| *** yamada-h has quit IRC | 03:10 | |
| *** rajinir has quit IRC | 03:10 | |
| *** barra204 has quit IRC | 03:11 | |
| *** barra204 has joined #openstack-cinder | 03:13 | |
| *** dmz has joined #openstack-cinder | 03:15 | |
| *** vokt has joined #openstack-cinder | 03:18 | |
| *** vokt has quit IRC | 03:24 | |
| nikeshm | asselin: working now | 03:26 |
| asselin_ | great! | 03:26 |
| *** harlowja_at_home has joined #openstack-cinder | 03:35 | |
| *** Yogi1 has quit IRC | 03:35 | |
| *** lixiaoy1 has left #openstack-cinder | 03:35 | |
| *** ks_ne has joined #openstack-cinder | 03:37 | |
| *** _cjones_ has quit IRC | 03:40 | |
| *** pradipta has joined #openstack-cinder | 03:42 | |
| *** yamada-h has joined #openstack-cinder | 03:42 | |
| *** yamada-h has quit IRC | 03:43 | |
| *** yamada-h has joined #openstack-cinder | 03:44 | |
| openstackgerrit | wanghao proposed openstack/cinder: Remove un-used import at test_volume_transfer.py https://review.openstack.org/186246 | 03:44 |
| *** jwcroppe has joined #openstack-cinder | 03:46 | |
| *** yamada-h has quit IRC | 03:56 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Prophetstor driver needs to return snapshot objects for create_cgsnapshot and delete_cgsnapshot. https://review.openstack.org/185495 | 03:59 |
| openstackgerrit | Rick Chen proposed openstack/cinder: Complete switch to snapshot objects https://review.openstack.org/163910 | 03:59 |
| openstackgerrit | Rick Chen proposed openstack/cinder: Switch get_all_snapshots to use objects https://review.openstack.org/161077 | 03:59 |
| *** setmason has joined #openstack-cinder | 04:00 | |
| *** markvoelker has joined #openstack-cinder | 04:02 | |
| *** markvoelker has quit IRC | 04:07 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 04:09 |
| *** yamada-h has joined #openstack-cinder | 04:11 | |
| *** garthb_ has joined #openstack-cinder | 04:19 | |
| *** garthb has joined #openstack-cinder | 04:19 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Multiple pools support enhancement in VNX cinder driver https://review.openstack.org/184001 | 04:21 |
| *** dannywilson has quit IRC | 04:28 | |
| *** harlowja_at_home has quit IRC | 04:28 | |
| *** jdurgin has joined #openstack-cinder | 04:28 | |
| *** sks has joined #openstack-cinder | 04:29 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 04:31 |
| *** Longgeek has joined #openstack-cinder | 04:36 | |
| *** _cjones_ has joined #openstack-cinder | 04:37 | |
| *** Longgeek has quit IRC | 04:45 | |
| *** Longgeek has joined #openstack-cinder | 04:45 | |
| *** setmason has quit IRC | 04:48 | |
| *** setmason has joined #openstack-cinder | 04:50 | |
| *** tobe has quit IRC | 04:52 | |
| *** _cjones_ has quit IRC | 04:53 | |
| *** _cjones_ has joined #openstack-cinder | 04:53 | |
| *** _cjones_ has quit IRC | 04:54 | |
| *** avishay has joined #openstack-cinder | 05:10 | |
| *** vilobhmm has joined #openstack-cinder | 05:10 | |
| *** tobe has joined #openstack-cinder | 05:14 | |
| *** avishay has quit IRC | 05:22 | |
| *** avishay_ has joined #openstack-cinder | 05:22 | |
| *** avishay_ has quit IRC | 05:27 | |
| *** avishay_ has joined #openstack-cinder | 05:28 | |
| *** yamada-h has quit IRC | 05:29 | |
| *** IanGovett has joined #openstack-cinder | 05:39 | |
| *** setmason_ has joined #openstack-cinder | 05:40 | |
| *** garthb_ has quit IRC | 05:42 | |
| *** garthb has quit IRC | 05:42 | |
| *** vilobhmm has quit IRC | 05:43 | |
| *** setmason has quit IRC | 05:43 | |
| *** nkrinner has joined #openstack-cinder | 05:44 | |
| *** rushiagr_away is now known as rushiagr | 05:44 | |
| *** lixiaoy1 has joined #openstack-cinder | 05:45 | |
| *** setmason_ has quit IRC | 05:46 | |
| *** tobe has quit IRC | 05:47 | |
| *** avishay_ has quit IRC | 05:47 | |
| *** markvoelker has joined #openstack-cinder | 05:52 | |
| *** markvoelker has quit IRC | 05:56 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Prophetstor driver needs to return snapshot objects for create_cgsnapshot and delete_cgsnapshot. https://review.openstack.org/185495 | 06:04 |
| openstackgerrit | Peter Wang proposed openstack/cinder: Add 'source-id' and 'source-name' support in VNX driver https://review.openstack.org/178969 | 06:06 |
| *** Maike has joined #openstack-cinder | 06:10 | |
| *** Longgeek has quit IRC | 06:11 | |
| *** tobe has joined #openstack-cinder | 06:15 | |
| *** Longgeek has joined #openstack-cinder | 06:16 | |
| *** nihilifer has joined #openstack-cinder | 06:18 | |
| *** Miouge_ has joined #openstack-cinder | 06:18 | |
| *** Miouge has quit IRC | 06:20 | |
| *** Miouge_ is now known as Miouge | 06:20 | |
| *** anshul has joined #openstack-cinder | 06:22 | |
| openstackgerrit | yogeshprasad proposed openstack/cinder: Add chap support to CloudByte cinder driver https://review.openstack.org/173733 | 06:24 |
| *** IanGovett has quit IRC | 06:26 | |
| openstackgerrit | rakesh mishra proposed openstack/cinder: set-max-size-limit-for-per-volume https://review.openstack.org/185906 | 06:31 |
| *** sks has quit IRC | 06:33 | |
| *** lpetrut has joined #openstack-cinder | 06:38 | |
| *** kaisers1 has joined #openstack-cinder | 06:39 | |
| *** e0ne has joined #openstack-cinder | 06:42 | |
| *** sks has joined #openstack-cinder | 06:48 | |
| *** jdurgin has quit IRC | 06:48 | |
| *** e0ne has quit IRC | 06:49 | |
| *** alecv has joined #openstack-cinder | 06:54 | |
| *** Maike_ has joined #openstack-cinder | 06:57 | |
| *** Maike_ has quit IRC | 06:59 | |
| *** Maike_ has joined #openstack-cinder | 06:59 | |
| *** yamada-h has joined #openstack-cinder | 07:00 | |
| *** Maike has quit IRC | 07:00 | |
| *** Miouge_ has joined #openstack-cinder | 07:02 | |
| *** kaisers2 has joined #openstack-cinder | 07:03 | |
| *** Miouge has quit IRC | 07:03 | |
| *** Miouge_ is now known as Miouge | 07:03 | |
| *** leopoldj has joined #openstack-cinder | 07:04 | |
| *** kaisers2 has quit IRC | 07:06 | |
| *** markus_z has joined #openstack-cinder | 07:08 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Create volume backing in specific clusters https://review.openstack.org/186664 | 07:14 |
| *** dulek has joined #openstack-cinder | 07:18 | |
| *** ronis has joined #openstack-cinder | 07:23 | |
| *** yamada-h has quit IRC | 07:23 | |
| *** Miouge has quit IRC | 07:25 | |
| *** yamada-h has joined #openstack-cinder | 07:32 | |
| *** jwcroppe has quit IRC | 07:32 | |
| *** jwcroppe has joined #openstack-cinder | 07:33 | |
| *** Miouge has joined #openstack-cinder | 07:33 | |
| *** jwcroppe has quit IRC | 07:37 | |
| *** jistr has joined #openstack-cinder | 07:38 | |
| *** bkopilov has quit IRC | 07:39 | |
| *** markvoelker has joined #openstack-cinder | 07:40 | |
| *** markvoelker has quit IRC | 07:45 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Return 400 error instead of 500 for invalid body https://review.openstack.org/186669 | 07:46 |
| *** Maike_ has quit IRC | 07:52 | |
| *** Longgeek has quit IRC | 07:54 | |
| *** Maike has joined #openstack-cinder | 08:02 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 08:03 |
| *** Maike_ has joined #openstack-cinder | 08:10 | |
| *** dims_ has joined #openstack-cinder | 08:12 | |
| *** Maike has quit IRC | 08:14 | |
| *** Maike has joined #openstack-cinder | 08:14 | |
| *** kaisers1 has quit IRC | 08:15 | |
| *** Maike_ has quit IRC | 08:15 | |
| *** dims_ has quit IRC | 08:18 | |
| *** yamada-h has quit IRC | 08:18 | |
| *** tobe has quit IRC | 08:23 | |
| *** jordanP has joined #openstack-cinder | 08:27 | |
| *** tobe has joined #openstack-cinder | 08:29 | |
| *** alecv has quit IRC | 08:33 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Prophetstor driver needs to return snapshot objects for create_cgsnapshot and delete_cgsnapshot. https://review.openstack.org/185495 | 08:45 |
| *** lpetrut has quit IRC | 08:47 | |
| *** ndipanov has joined #openstack-cinder | 08:49 | |
| openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Adds friendly zone name support https://review.openstack.org/180518 | 08:54 |
| *** Miouge has quit IRC | 08:58 | |
| *** alecv has joined #openstack-cinder | 09:01 | |
| *** Miouge has joined #openstack-cinder | 09:01 | |
| *** Miouge has quit IRC | 09:01 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Made error message consistent across all apis https://review.openstack.org/186688 | 09:04 |
| *** ankit_ag has joined #openstack-cinder | 09:05 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Test https://review.openstack.org/186689 | 09:14 |
| openstackgerrit | wanghao proposed openstack/cinder: Notification with volume and snapshot metadata https://review.openstack.org/180400 | 09:17 |
| *** Maike has quit IRC | 09:19 | |
| wanghao | DuncanT: ping | 09:28 |
| wanghao | DuncanT: I'm very very sad to have to rebase Notification with volume and snapshot metadata again after Mike has workflow +1 for it. | 09:29 |
| *** markvoelker has joined #openstack-cinder | 09:29 | |
| wanghao | DuncanT: Cos the merge confilct. | 09:29 |
| DuncanT | wanghao: Sigh. This does happen often, unfortunately | 09:29 |
| DuncanT | wanghao: Once Jenkins has passed, I'll look again then maybe make it the first candidate for casual review friday | 09:30 |
| wanghao | DuncaT: That's great. Thank you a lot. | 09:31 |
| wanghao | DuncanT: For those days to bother you.... | 09:31 |
| DuncanT | wanghao: You're welcome. Thanks for sticking with it | 09:32 |
| DuncanT | https://etherpad.openstack.org/p/cinder-casual-review-friday | 09:32 |
| wanghao | DuncanT: Got it. | 09:33 |
| *** markvoelker has quit IRC | 09:34 | |
| *** alecv has quit IRC | 09:34 | |
| wanghao | DuncanT: I'm done for the day. Go home for Friday night. Have a good day:) | 09:36 |
| *** lpetrut has joined #openstack-cinder | 09:36 | |
| DuncanT | wanghao: And you, good luck! | 09:37 |
| *** lpetrut1 has joined #openstack-cinder | 09:39 | |
| DuncanT | Any netapp folks awake to take a look at https://review.openstack.org/#/c/186342/1/cinder/volume/drivers/netapp/eseries/client.py please? The 'fix' has changed the semantics of the client, even if the original semantics were nonsense, so I'd like somebody to take a quick look, please | 09:40 |
| *** lpetrut has quit IRC | 09:41 | |
| *** aix has joined #openstack-cinder | 09:41 | |
| *** pradipta has quit IRC | 09:45 | |
| *** lixiaoy1 has quit IRC | 09:45 | |
| openstackgerrit | Jessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage https://review.openstack.org/177113 | 09:47 |
| openstackgerrit | Tina Tang proposed openstack/cinder: Multiple pools support enhancement in VNX cinder driver https://review.openstack.org/184001 | 09:48 |
| *** dims_ has joined #openstack-cinder | 09:51 | |
| *** bkopilov has joined #openstack-cinder | 09:53 | |
| *** pboldin has joined #openstack-cinder | 10:06 | |
| *** dulek_ has joined #openstack-cinder | 10:08 | |
| *** dulek has quit IRC | 10:10 | |
| *** afazekas has joined #openstack-cinder | 10:12 | |
| *** Maike has joined #openstack-cinder | 10:14 | |
| *** heyun has quit IRC | 10:16 | |
| *** anshul has quit IRC | 10:35 | |
| *** pboldin has quit IRC | 10:36 | |
| nikeshm | DuncanT:hi | 10:38 |
| nikeshm | today i changed my CI to work on cinder patches | 10:39 |
| nikeshm | DuncanT: i done recheck on my cinder patch too https://review.openstack.org/#/c/177665/ | 10:43 |
| nikeshm | initially it failed then i figured out the reason and now it all tests should pass | 10:44 |
| nikeshm | tomorrow will ask walter boring to remove -2 and give =2 | 10:45 |
| nikeshm | +2 | 10:45 |
| *** anshul has joined #openstack-cinder | 10:46 | |
| nikeshm | if yu will be available tomorrow then had a look on patch and give some comments | 10:46 |
| *** dulek_ is now known as dulek | 10:47 | |
| *** pboldin has joined #openstack-cinder | 10:52 | |
| *** dims_ has quit IRC | 10:52 | |
| *** aix has quit IRC | 10:54 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 10:56 |
| *** zhenguo has quit IRC | 10:58 | |
| *** dulek has quit IRC | 10:58 | |
| *** Miouge has joined #openstack-cinder | 10:59 | |
| DuncanT | nikeshm: Looking at the patch now | 11:03 |
| DuncanT | nikeshm: Looks like it is still getting a -2 from your CI though? | 11:03 |
| *** dulek has joined #openstack-cinder | 11:06 | |
| nikeshm | DuncanT: in initial recheck it was,but that was due to some setup issue in my CI,i resolved that and after checking locally again rerun recheck | 11:07 |
| nikeshm | DuncanT: now every tests are passing | 11:08 |
| nikeshm | and sooner my CI will update on patch | 11:08 |
| DuncanT | Ah, ok | 11:08 |
| nikeshm | DuncanT: talked to walter today morning,he said as soom my CI will come on patch,he will remove -2 | 11:09 |
| nikeshm | DuncanT: tempest passed in my system,it will take 20 minutes to upload logs, | 11:11 |
| nikeshm | and then it will come in that patch | 11:11 |
| nikeshm | my log server is in INDIA | 11:11 |
| nikeshm | and this system running CI is in US | 11:11 |
| DuncanT | I'm looking at the patch, I'll leave comments or a +2 | 11:11 |
| *** lpetrut1 has quit IRC | 11:12 | |
| DuncanT | We can sell you a nice log server in the cloud in the US if that helps ;-) | 11:12 |
| *** mdbooth has quit IRC | 11:12 | |
| nikeshm | DuncanT:ha ha, sooner i have to upload patches for HPMSA,so will ask HP to provide | 11:13 |
| nikeshm | :) | 11:13 |
| *** merooney has joined #openstack-cinder | 11:13 | |
| nikeshm | i am in san jose,california for a short trip | 11:15 |
| nikeshm | but works in hyderabad India | 11:15 |
| nikeshm | for vedams | 11:15 |
| *** mdbooth has joined #openstack-cinder | 11:17 | |
| *** markvoelker has joined #openstack-cinder | 11:18 | |
| nikeshm | thanks for looking on the patch | 11:18 |
| nikeshm | going to sleep its 4:19 am of morning for me | 11:19 |
| nikeshm | :) | 11:19 |
| *** anshul has quit IRC | 11:19 | |
| nikeshm | and also this setup is for iscsi and we are working on FC too | 11:21 |
| nikeshm | and we are skipping four tests like we discussed | 11:22 |
| *** markvoelker has quit IRC | 11:22 | |
| *** merooney has quit IRC | 11:35 | |
| *** cdelatte has quit IRC | 11:38 | |
| *** tobe has quit IRC | 11:39 | |
| *** cdelatte has joined #openstack-cinder | 11:39 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Create volume backing in specific clusters https://review.openstack.org/186664 | 11:39 |
| *** aix has joined #openstack-cinder | 11:45 | |
| *** dims_ has joined #openstack-cinder | 11:50 | |
| *** rushiagr is now known as rushiagr_away | 11:50 | |
| *** Miouge has quit IRC | 11:52 | |
| *** sgotliv has quit IRC | 11:53 | |
| *** ganso_ has joined #openstack-cinder | 11:53 | |
| *** dulek has quit IRC | 11:54 | |
| *** dulek has joined #openstack-cinder | 11:55 | |
| *** markvoelker has joined #openstack-cinder | 11:59 | |
| *** markus_z has quit IRC | 12:00 | |
| *** nkrinner has quit IRC | 12:02 | |
| *** dulek has quit IRC | 12:03 | |
| *** markus_z has joined #openstack-cinder | 12:04 | |
| *** Yogi1 has joined #openstack-cinder | 12:05 | |
| *** dulek has joined #openstack-cinder | 12:05 | |
| *** nkrinner has joined #openstack-cinder | 12:05 | |
| *** aix has quit IRC | 12:06 | |
| *** aix has joined #openstack-cinder | 12:08 | |
| *** Yogi1 has quit IRC | 12:10 | |
| tbarron | DuncanT: thanks for calling our attention to https://review.openstack.org/#/c/186342 | 12:11 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Test https://review.openstack.org/186689 | 12:11 |
| *** tobe has joined #openstack-cinder | 12:13 | |
| *** tobe has quit IRC | 12:14 | |
| *** marcusvrn has joined #openstack-cinder | 12:14 | |
| tbarron | DuncanT: we'll update this review today (I'm just running git blame now and seeing who to talk to & may run the candidate change through our CI pre-merge) | 12:15 |
| DuncanT | tbarron: you're welcome. Thanks for jumping on it | 12:18 |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Made error message consistent across all apis https://review.openstack.org/186688 | 12:18 |
| tbarron | DuncanT: same author, no longer with us, for the nonsense dataontap/block_base.py and eseries/client.py | 12:23 |
| tbarron | I'm inclined just to endorse the change but since someone has asked cknight for his opinion in the review, let's get that as well | 12:23 |
| *** anshul has joined #openstack-cinder | 12:26 | |
| DuncanT | It looks like the change is sane, but it's your driver | 12:29 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Create volume backing in specific clusters https://review.openstack.org/186664 | 12:29 |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate name and description string https://review.openstack.org/182994 | 12:34 |
| *** lpetrut has joined #openstack-cinder | 12:38 | |
| *** bswartz has quit IRC | 12:39 | |
| *** marcusvrn1 has joined #openstack-cinder | 12:40 | |
| *** fthiagogv has joined #openstack-cinder | 12:41 | |
| *** Yogi11 has joined #openstack-cinder | 12:42 | |
| *** rushiagr_away is now known as rushiagr | 12:43 | |
| *** marcusvrn has quit IRC | 12:44 | |
| *** marcusvrn1 has quit IRC | 12:48 | |
| *** marcusvrn has joined #openstack-cinder | 12:48 | |
| *** sks has quit IRC | 12:51 | |
| *** Yogi11 has quit IRC | 12:53 | |
| *** med_ has joined #openstack-cinder | 12:54 | |
| *** med_ has quit IRC | 12:54 | |
| *** med_ has joined #openstack-cinder | 12:54 | |
| *** marcusvrn has quit IRC | 12:55 | |
| *** merooney has joined #openstack-cinder | 13:01 | |
| *** marcusvrn has joined #openstack-cinder | 13:08 | |
| *** primechuck has joined #openstack-cinder | 13:09 | |
| *** nkrinner has quit IRC | 13:10 | |
| *** xyang1 has joined #openstack-cinder | 13:11 | |
| *** Tross1 has quit IRC | 13:13 | |
| *** mriedem has joined #openstack-cinder | 13:14 | |
| *** akshai has joined #openstack-cinder | 13:15 | |
| *** merooney has quit IRC | 13:16 | |
| *** merooney has joined #openstack-cinder | 13:23 | |
| *** mtanino has joined #openstack-cinder | 13:26 | |
| *** annegentle has joined #openstack-cinder | 13:29 | |
| *** Yogi1 has joined #openstack-cinder | 13:29 | |
| *** sgotliv has joined #openstack-cinder | 13:30 | |
| *** nihilifer has quit IRC | 13:32 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Refactor scheduler's create_volume flow https://review.openstack.org/186439 | 13:33 |
| *** rushiagr is now known as rushiagr_away | 13:35 | |
| *** jungleboyj has quit IRC | 13:35 | |
| *** nihilifer has joined #openstack-cinder | 13:37 | |
| *** marcusw has joined #openstack-cinder | 13:39 | |
| *** merooney has quit IRC | 13:39 | |
| *** pboldin has quit IRC | 13:43 | |
| *** bswartz has joined #openstack-cinder | 13:44 | |
| *** annegent_ has joined #openstack-cinder | 13:44 | |
| *** annegentle has quit IRC | 13:45 | |
| *** Tross has joined #openstack-cinder | 13:45 | |
| *** aix has quit IRC | 13:49 | |
| *** aix has joined #openstack-cinder | 13:49 | |
| *** timcl has joined #openstack-cinder | 13:56 | |
| *** annegent_ has quit IRC | 13:57 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC VMAX Manage/Unmanage Volume https://review.openstack.org/182460 | 13:58 |
| *** merooney has joined #openstack-cinder | 13:59 | |
| *** krtaylor has quit IRC | 14:02 | |
| *** ankit_ag has quit IRC | 14:02 | |
| *** Maike_ has joined #openstack-cinder | 14:03 | |
| openstackgerrit | rakesh mishra proposed openstack/cinder: set-max-size-limit-for-volume https://review.openstack.org/185906 | 14:04 |
| *** Maike_ has quit IRC | 14:04 | |
| *** emagana has quit IRC | 14:04 | |
| *** Maike_ has joined #openstack-cinder | 14:04 | |
| *** mtanino has quit IRC | 14:05 | |
| *** rushil has joined #openstack-cinder | 14:05 | |
| *** anshul has quit IRC | 14:06 | |
| *** afazekas has quit IRC | 14:06 | |
| tbarron | DuncanT: you can remove your -1 from https://review.openstack.org/186342 now. Thanks again for the caution. | 14:06 |
| *** Maike has quit IRC | 14:07 | |
| *** annegentle has joined #openstack-cinder | 14:10 | |
| *** sgotliv has quit IRC | 14:11 | |
| openstackgerrit | wanghao proposed openstack/cinder: Notification with volume and snapshot metadata https://review.openstack.org/180400 | 14:12 |
| *** crose has joined #openstack-cinder | 14:13 | |
| *** krtaylor has joined #openstack-cinder | 14:13 | |
| *** lpetrut1 has joined #openstack-cinder | 14:15 | |
| *** lpetrut has quit IRC | 14:16 | |
| *** dims_ has quit IRC | 14:16 | |
| *** leopoldj has quit IRC | 14:17 | |
| *** breitz has quit IRC | 14:20 | |
| *** breitz has joined #openstack-cinder | 14:21 | |
| *** rushil has quit IRC | 14:21 | |
| *** ronis has quit IRC | 14:22 | |
| *** jwcroppe has joined #openstack-cinder | 14:24 | |
| *** jistr has quit IRC | 14:26 | |
| *** dulek has quit IRC | 14:27 | |
| *** rushil has joined #openstack-cinder | 14:30 | |
| *** esker has joined #openstack-cinder | 14:35 | |
| *** Maike_ has quit IRC | 14:39 | |
| *** ebalduf has joined #openstack-cinder | 14:40 | |
| *** kvidvans has joined #openstack-cinder | 14:40 | |
| *** s0nea has quit IRC | 14:40 | |
| *** ebalduf has quit IRC | 14:40 | |
| *** ebalduf has joined #openstack-cinder | 14:43 | |
| *** jungleboyj has joined #openstack-cinder | 14:46 | |
| *** merooney has quit IRC | 14:47 | |
| *** jistr has joined #openstack-cinder | 14:47 | |
| *** cbader has quit IRC | 14:52 | |
| *** merooney has joined #openstack-cinder | 14:53 | |
| *** buster_604 has joined #openstack-cinder | 14:58 | |
| jgriffith | mtreinish: any idea what this is about: http://logs.openstack.org/86/186586/2/check/gate-cinder-python27/907b00b/console.html#_2015-05-29_09_54_39_017 | 14:58 |
| jgriffith | mtreinish: I mean othe rthan "Length too long" | 14:58 |
| jgriffith | of course | 14:58 |
| *** esker has quit IRC | 14:59 | |
| *** s0nea has joined #openstack-cinder | 14:59 | |
| *** annegentle has quit IRC | 14:59 | |
| *** esker has joined #openstack-cinder | 15:00 | |
| mtreinish | jgriffith: my guess is that this attachment: http://logs.openstack.org/86/186586/2/check/gate-cinder-python27/907b00b/console.html#_2015-05-29_09_54_06_567 is so long that it's causing an issue with the 1to2 filter | 15:01 |
| mtreinish | but I've never seen that happen before | 15:01 |
| *** buster_604 has quit IRC | 15:01 | |
| jgriffith | mtreinish: I'm trying to figure out the coorelation between that and cranking up the number of processes in the test | 15:01 |
| mtreinish | it's probably a bug in 1to2, because I thought subunit should be able to chunk things properly for file attachments | 15:01 |
| *** emagana has joined #openstack-cinder | 15:02 | |
| jgriffith | mtreinish: I wonder... | 15:02 |
| *** buster_604 has joined #openstack-cinder | 15:02 | |
| mtreinish | it definitely seems like that test is loading all the migrations multiple times | 15:02 |
| jgriffith | mtreinish: yeah, I think that might be a problem | 15:03 |
| mtreinish | way more time then whatever the concurrency count is though | 15:03 |
| *** buster_604 has quit IRC | 15:04 | |
| *** cbader has joined #openstack-cinder | 15:05 | |
| aarefiev | jgriffith: hi, have a min? | 15:06 |
| mtreinish | jgriffith: man, that log file is hanging up my browser | 15:06 |
| mtreinish | but it didn't seem that big | 15:07 |
| openstackgerrit | xing-yang proposed openstack/cinder: Not to deduct thin volume size in capacity filter https://review.openstack.org/185764 | 15:09 |
| DuncanT | tbarron: +2/A | 15:12 |
| *** mtanino has joined #openstack-cinder | 15:12 | |
| *** ebalduf has quit IRC | 15:13 | |
| tbarron | DuncanT: cool | 15:13 |
| *** hemnafk is now known as hemna | 15:14 | |
| mtreinish | jgriffith: oh, figured it out I had zuul.o.o open in another tab that was causing the hangs | 15:15 |
| *** dims_ has joined #openstack-cinder | 15:16 | |
| *** dannywilson has joined #openstack-cinder | 15:18 | |
| *** jms has joined #openstack-cinder | 15:19 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Fix capacity filter to allow oversubscription https://review.openstack.org/185764 | 15:22 |
| *** dannywilson has quit IRC | 15:23 | |
| *** jordanP has quit IRC | 15:23 | |
| *** lpetrut1 has quit IRC | 15:24 | |
| *** jdurgin has joined #openstack-cinder | 15:24 | |
| *** dannywilson has joined #openstack-cinder | 15:25 | |
| *** agarciam has quit IRC | 15:25 | |
| *** nihilifer has quit IRC | 15:27 | |
| *** annegentle has joined #openstack-cinder | 15:30 | |
| *** tsekiyama has joined #openstack-cinder | 15:31 | |
| *** rmesta has joined #openstack-cinder | 15:36 | |
| jgriffith | sorry... had a visitor | 15:38 |
| jgriffith | WTF? "class TestSqliteMigrations...: pass" | 15:39 |
| *** ebalduf has joined #openstack-cinder | 15:39 | |
| jgriffith | nice.. the mixin bandit strikes again. sigh | 15:40 |
| jgriffith | mtreinish: so I think that kinda screws up the parallelism | 15:41 |
| mtanino | xyang1: Hi, I and Tomoki joined your session last night, it was very helpful to understand current and feature plan of Cinder :) | 15:41 |
| xyang1 | mtanino: Were you there? I didn't see you. Thanks for joining | 15:42 |
| *** annegentle has quit IRC | 15:42 | |
| xyang1 | mtanino: I didn't know you are in Boston area | 15:42 |
| mtanino | xyang1: we joined at little late | 15:43 |
| *** annegentle has joined #openstack-cinder | 15:43 | |
| xyang1 | mtanino: Ok | 15:43 |
| mtanino | xyang1: yes, we live in Boston area :) nice place | 15:43 |
| mtanino | xyang1: Are you still in Boston? | 15:44 |
| mtreinish | jgriffith: hmm, how does nova and others do the migration testing in parallel? | 15:44 |
| xyang1 | mtanino: Yes, I am in Boston area | 15:44 |
| mtreinish | I haven't looked at the cinder version of those tests, but they work fine in parallel for glance and subunit2sql (where I've added them) | 15:44 |
| *** dansmith is now known as superdan | 15:44 | |
| * mtreinish pulls up the code | 15:44 | |
| mtanino | xyang1: I hope you will enjoy Boston sightseeing :) | 15:44 |
| xyang1 | mtanino: I live here | 15:45 |
| mtanino | xyang1: Oh, I didn't know it | 15:45 |
| jgriffith | mtreinish: yeah, doesn't seem that's the problem | 15:46 |
| xyang1 | mtanino: my office is in Hopkinton | 15:46 |
| jgriffith | mtreinish: weird, and really weird I can't reproduce it locally either | 15:46 |
| *** jdurgin has quit IRC | 15:46 | |
| xyang1 | mtanino: Where is your office | 15:46 |
| mtanino | xyang1: our office is in Westford :) | 15:46 |
| xyang1 | mtanino: Ok | 15:47 |
| mtanino | xyang1: I know there is IBM's office along the route 495, but I didn't know also EMC's office is there | 15:49 |
| *** leeantho has joined #openstack-cinder | 15:49 | |
| xyang1 | mtanino: EMC has several buildings on south street in Hopkinton | 15:49 |
| *** dims_ is now known as dimsum__ | 15:49 | |
| xyang1 | mtanino: That is the headquarter | 15:49 |
| mtanino | xyang1: oh, I see | 15:50 |
| xyang1 | mtanino: There is also an office in Cambridge | 15:50 |
| mtanino | xyang1: good place :) | 15:50 |
| *** daneyon has joined #openstack-cinder | 15:50 | |
| xyang1 | mtanino: Though I don't like to go there, can't find parking:( | 15:50 |
| mtanino | xyang1: yup. I agree on it :( | 15:51 |
| *** lpetrut has joined #openstack-cinder | 15:51 | |
| mtanino | xyang1: If you are OK, can you share the presentation slide yesterday? | 15:52 |
| mtanino | that's very helpful :) | 15:52 |
| xyang1 | mtanino: Sure, I'll ask the organizer where to publish slides | 15:52 |
| mtanino | xyang1: thanks :) | 15:53 |
| *** daneyon_ has joined #openstack-cinder | 15:53 | |
| xyang1 | mtanino: Although I believe you should already know everything:). Nothing new there | 15:53 |
| mtreinish | jgriffith: oh if it's the db tests causing the issues, do you have the backends configured locally to reproduce? | 15:53 |
| mtreinish | IIRC it'll silently pass (may not even be a skip) if you don't have the backend db available | 15:54 |
| mtanino | xyang1: I found my SPEC for liberty was in your slide :) I'm glad about it. | 15:54 |
| jgriffith | mtreinish: OH, crud... I forgot we changed all that for db mmigrations | 15:54 |
| *** asselin_ has quit IRC | 15:54 | |
| jgriffith | mtreinish: no, I'm just running the default sqlite | 15:54 |
| xyang1 | mtanino: You should have anwsered questions for me:). Did you see my disclaimer? | 15:55 |
| jgriffith | mtreinish: but one other thing that bothers me | 15:55 |
| *** daneyon has quit IRC | 15:55 | |
| jgriffith | mtreinish: grep out on LOG in the cinder migrate directory :( | 15:56 |
| mtanino | xyang1: yes :) | 15:56 |
| mtreinish | jgriffith: I think there is a bug in the run script for unit tests, it doesn't need to use subunit-1to2 it can just do testr last --subunit (which is what we do on tempest jobs) | 15:56 |
| mtanino | xyang1: These feares are not promised to get in Liberty :( | 15:56 |
| jgriffith | mtreinish: oh... let me have a look at that. My run of subunit just hung it appears | 15:57 |
| mtanino | just examples! | 15:57 |
| *** timcl has quit IRC | 15:57 | |
| xyang1 | mtanino: I was just showing things that are already public, but can't guarantee anything:) | 15:57 |
| mtanino | xyang1: sure :) | 15:57 |
| jgriffith | mtreinish: well, that seems to work a bit better | 15:58 |
| jgriffith | Of course I'm not sure the output is whats wanted :) | 15:58 |
| *** jistr has quit IRC | 15:58 | |
| mtanino | xyang1: Everyone can't guarantee what's happen in next 6 months. | 15:58 |
| *** theanalyst has quit IRC | 15:58 | |
| xyang1 | mtanino: :) | 15:58 |
| *** rushil has quit IRC | 15:58 | |
| *** garthb has joined #openstack-cinder | 15:58 | |
| *** linkedinyou has quit IRC | 15:59 | |
| jgriffith | mtreinish: can we start a new repo? Name it "infra-things-griffith-might-look-for" | 15:59 |
| jgriffith | mtreinish: and folks can put all the stuff I might be interested in there :) | 15:59 |
| mtreinish | jgriffith: https://review.openstack.org/186809 :) | 16:01 |
| *** annegentle has quit IRC | 16:01 | |
| jgriffith | HA!!! | 16:01 |
| jgriffith | mtreinish: you did it again!!! | 16:01 |
| mtreinish | jgriffith: heh, fwiw I'd be perfectly fine renaming os-testr to things-griffith-might-look-for :) | 16:01 |
| jgriffith | mtreinish: one of these days I'm going to fix something for you!!! | 16:01 |
| jgriffith | LOL | 16:02 |
| *** theanalyst has joined #openstack-cinder | 16:02 | |
| jgriffith | mtreinish: that's next on my list... I promise you | 16:02 |
| jgriffith | mtreinish: and with the links you provided it shouldn't be hard to find the time to do it | 16:02 |
| mtreinish | heh, there isn't any rush, unless you want to beat me to being first to use it somewhere | 16:03 |
| jgriffith | mtreinish: Challenge Accepted!!! | 16:04 |
| mtreinish | ha, well I'll have to start bribing people for reviews now :) | 16:05 |
| *** jistr has joined #openstack-cinder | 16:06 | |
| *** krtaylor has quit IRC | 16:08 | |
| *** jistr has quit IRC | 16:09 | |
| jgriffith | mtreinish: testing an ostestr patch now | 16:09 |
| jgriffith | mtreinish: I think I'm going to remove all that debug nonsense in the migration tests | 16:12 |
| jgriffith | It's REALLY annoying | 16:12 |
| mtreinish | heh, yeah it is I think that's the default for running the migrate commands | 16:14 |
| jgriffith | mtreinish: so the kunundrum is... I have the ostestr change, but it opens the limiter, which leads me back to the stability issues | 16:14 |
| *** rwsu has joined #openstack-cinder | 16:15 | |
| jgriffith | mtreinish: I'm thinking maybe set a default in tox for 6 or nproc whichever is less | 16:15 |
| jgriffith | mtreinish: in .tox.ini that is | 16:15 |
| jgriffith | mtreinish: since 6 seems to be a magic number (without the test_gluster changes) | 16:15 |
| mtreinish | hmm, yeah if there is an issue with too much concurrency you'll have to set a default ceiling somewhere | 16:16 |
| mtreinish | I guess pretty_tox made that easy to do | 16:16 |
| jgriffith | mtreinish: hehe... if I just remove the test_glusterfs.py I can run 12 processes reliably no issues :) | 16:18 |
| jgriffith | mtreinish: ok, I'm going to stop blabbering and combine the two patches | 16:18 |
| mtreinish | jgriffith: heh, seems like a viable solution to me :) | 16:18 |
| *** markus_z has quit IRC | 16:22 | |
| openstackgerrit | Diem Tran proposed openstack/cinder: Re-integrate Oracle iSCSI Cinder driver https://review.openstack.org/178319 | 16:25 |
| *** timcl has joined #openstack-cinder | 16:28 | |
| *** Yogi1 has quit IRC | 16:31 | |
| *** rmesta has quit IRC | 16:31 | |
| *** rmesta has joined #openstack-cinder | 16:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 16:31 |
| *** _cjones_ has joined #openstack-cinder | 16:32 | |
| dhellmann | thingee: I would like to cut a release of os-brick on Monday to unblock the work on dropping the oslo namespace packages. The changes that will be released are listed in http://paste.openstack.org/show/245107/ -- Does that work for you? | 16:34 |
| thingee | dhellmann: I just did this for infra last week. :( | 16:36 |
| dhellmann | thingee: ? | 16:36 |
| dhellmann | thingee: I need "9247aa4 2015-04-28 22:25:57 +0000 Drop use of 'oslo' namespace package" | 16:36 |
| thingee | I had to do a release because of some requirement blockage | 16:36 |
| thingee | Just saying some coordination would be good so I don’t have to do this once a week. | 16:37 |
| *** esker has quit IRC | 16:38 | |
| dhellmann | thingee: well, I'm offering to do this one for you, I just want to make sure I'm not going to break anything by doing it | 16:38 |
| dhellmann | I don't know what the requirements thing was last week, did that have to do with pbr? | 16:39 |
| thingee | Nah it’s fine. I’m just raising this will be the second week in a row I’m doing this, just because of dependencies. | 16:40 |
| *** lpetrut has quit IRC | 16:40 | |
| jgriffith | FWIW it's a new project, it's going to be that way for a while likely | 16:42 |
| *** jungleboyj has quit IRC | 16:43 | |
| *** ociuhandu has joined #openstack-cinder | 16:44 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Fix capacity filter to allow oversubscription https://review.openstack.org/185764 | 16:48 |
| *** Tross has quit IRC | 16:51 | |
| *** rushil has joined #openstack-cinder | 16:52 | |
| openstackgerrit | Jon Bernard proposed openstack/os-brick: Add missing connectors to factory test https://review.openstack.org/186837 | 16:52 |
| *** vilobhmm has joined #openstack-cinder | 16:53 | |
| *** patrickeast has joined #openstack-cinder | 16:56 | |
| nikeshm | asselin: hi | 16:56 |
| asselin | nikeshm, hi | 16:56 |
| nikeshm | asselin: good morning,good comment regarding new lines, | 16:56 |
| *** raildo is now known as needmoresummit | 16:59 | |
| *** Apoorva has joined #openstack-cinder | 16:59 | |
| nikeshm | asselin:but not mentioned in http://docs.openstack.org/infra/system-config/third_party.html,how to do this? | 17:01 |
| asselin | nikeshm, I don't know either...mine just worked. | 17:02 |
| *** xyang1 has quit IRC | 17:02 | |
| asselin | nikeshm, however when you find the answer, we can update that url | 17:02 |
| nikeshm | asselin:ok | 17:03 |
| nikeshm | hemna: hi Ci is now showing on https://review.openstack.org/#/c/177665/ thanks for you and rammy for all support | 17:04 |
| *** daneyon_ has quit IRC | 17:05 | |
| *** dannywilson has quit IRC | 17:08 | |
| *** dannywilson has joined #openstack-cinder | 17:09 | |
| Swanson | Another day another devstack error. (sigh) | 17:10 |
| hemna | wee! | 17:10 |
| Swanson | settle | 17:11 |
| jgriffith | Swanson: what problem are you seeing? | 17:11 |
| *** patrickeast has quit IRC | 17:11 | |
| *** patrickeast has joined #openstack-cinder | 17:11 | |
| jgriffith | Swanson: odd, I've been going through logs the past few days you've been talking about devstack issues and I'm not seeing anything? | 17:11 |
| *** jdurgin has joined #openstack-cinder | 17:11 | |
| jgriffith | Swanson: deploying devstack on average 15x per day | 17:12 |
| jgriffith | Swanson: and then there's the gate, which is always a pretty good indicator | 17:12 |
| jgriffith | Swanson: git timeouts? | 17:12 |
| Swanson | jgriffith: this time it appears to be an oslo_config duplicate opt error today. That usually requires extra ajax to clean up but is usually my machine. | 17:12 |
| jgriffith | Swanson: are you "reusing" machines? | 17:13 |
| Swanson | yes | 17:13 |
| jgriffith | Swanson: yeah... don't do that :) | 17:13 |
| openstackgerrit | Jon Bernard proposed openstack/os-brick: Fix local connector test case inheritance https://review.openstack.org/186844 | 17:13 |
| jgriffith | Swanson: sersiously... your life would be soooo much easier | 17:13 |
| jgriffith | Swanson: get on board with the whole "cloud" philosophy | 17:13 |
| jgriffith | Swanson: make that crap disposable after use | 17:13 |
| *** rushil has quit IRC | 17:13 | |
| Swanson | I think that is about to happen. | 17:14 |
| jgriffith | Swanson: :) | 17:14 |
| Swanson | Typically hasn't been an issue, tho. | 17:14 |
| *** crose has quit IRC | 17:15 | |
| *** crose has joined #openstack-cinder | 17:15 | |
| jgriffith | Swanson: so I run things in OpenStack for "EVER" (one DB Instance has been running for a year now) | 17:15 |
| jgriffith | Swanson: but when it comes to things like deploying/redeploying devstack, it's not really meant to work that way | 17:15 |
| jgriffith | Swanson: unless you want to contribute some uber code to the devstack cleanup.sh | 17:16 |
| hemna | I restack frequently | 17:16 |
| hemna | it's usually worked for me | 17:16 |
| jgriffith | hemna: and you frequently ask in channel "anybody else seeing problems with devstack today" :) | 17:16 |
| hemna | unless I go long periods w/o updating devstack itself | 17:16 |
| jgriffith | just saying | 17:16 |
| hemna | hehe yah | 17:16 |
| jgriffith | hemna: Swanson ok, I don't know what I'm talking about, do whatever you like | 17:17 |
| Swanson | jgriffith: Fair enough. I think I'll keep restacking and then the second it fails me blow it away. | 17:17 |
| jgriffith | just trying to offer a suggestion | 17:17 |
| hemna | it's super easy for me to just spin up a new vagrant with the latest | 17:17 |
| hemna | and toss the old one | 17:17 |
| *** krtaylor has joined #openstack-cinder | 17:17 | |
| jgriffith | No matter what I say though I think someone here will contradict it | 17:17 |
| hemna | I do all my dev in vagrant VMs now anyway to combat those exact kinds of problems. | 17:17 |
| * jgriffith says "the sky is blue" | 17:17 | |
| Swanson | Should never have abdicated the throne. | 17:17 |
| jgriffith | someone will say "nope, it's azure" | 17:18 |
| jgriffith | Swanson: ? | 17:18 |
| hemna | I'll typically try to fix it for a few minutes, then put and vagrant up a new vm | 17:18 |
| dhellmann | thingee: ah, well, in this case it's actually a fix inside your lib that I need in order to be able to make some releases in oslo, so it's not quite the same thing | 17:18 |
| Swanson | jgriffith: like a joke only different at the end. | 17:19 |
| Swanson | It was | 17:19 |
| jgriffith | Swanson: oh... sorry, I didn't understand | 17:19 |
| *** rushil has joined #openstack-cinder | 17:19 | |
| jgriffith | Swanson: but I'll laugh anyway :) | 17:20 |
| asselin | patrickeast, fyi, i'm using fc passthrough from upstream now! | 17:20 |
| Swanson | jgriffith: Typically it is just best to laugh and then sidle away from me. That's what smcginnis does. | 17:20 |
| jgriffith | Swanson: LOL.. now that joke I got :) | 17:20 |
| jgriffith | Swanson: that also means you and I have something in common :) | 17:21 |
| Swanson | :) | 17:21 |
| *** buster_604 has joined #openstack-cinder | 17:22 | |
| *** buster_604 has quit IRC | 17:23 | |
| *** buster_604 has joined #openstack-cinder | 17:23 | |
| *** vivek_nandavanam has quit IRC | 17:23 | |
| *** barra204 has quit IRC | 17:26 | |
| *** amoturi has joined #openstack-cinder | 17:27 | |
| *** Tross has joined #openstack-cinder | 17:28 | |
| thingee | alright, so looks like no one opposed to smcginnis joining core. welcome smcginnis :) | 17:30 |
| hemna | :) | 17:30 |
| hemna | smcginnis, +2! | 17:30 |
| *** aix has quit IRC | 17:31 | |
| Swanson | Are you sure he is allowed in? I mean he took vacation somewhere without internet.. | 17:32 |
| * thingee needs to figure out how he's doing burning man this year with the release happening sep 3 | 17:32 | |
| *** madskier has joined #openstack-cinder | 17:32 | |
| thingee | "vacation" | 17:32 |
| *** Yogi1 has joined #openstack-cinder | 17:34 | |
| *** amoturi has quit IRC | 17:35 | |
| nikeshm | hemna : morning | 17:35 |
| hemna | hey | 17:35 |
| *** ebalduf has quit IRC | 17:36 | |
| nikeshm | hemna: now CI is showing with all tests paased on https://review.openstack.org/#/c/177665/ | 17:37 |
| *** madskier has quit IRC | 17:37 | |
| nikeshm | hemna: thanks for all your and rammy support | 17:37 |
| *** amoturi has joined #openstack-cinder | 17:37 | |
| hemna | sweet | 17:38 |
| hemna | thingee, what's a vacation? | 17:38 |
| hemna | I'm told an openstack summit is the same thing | 17:38 |
| nikeshm | hemna: can i expect to remove -2 :) | 17:40 |
| *** ronis has joined #openstack-cinder | 17:40 | |
| hemna | so the CI logs are completely unreadable :( | 17:40 |
| hemna | no line breaks | 17:41 |
| hemna | the logs are 1 long line :) | 17:41 |
| hemna | http://115.119.152.41/dothill-logs/logs/65/177665/5/check/dsvm-tempest-full/cdf861a/logs/screen-c-vol.txt.gz | 17:41 |
| hemna | looks like newlines are stripped ? | 17:42 |
| hemna | dunno | 17:42 |
| *** sgotliv has joined #openstack-cinder | 17:42 | |
| *** ronis has quit IRC | 17:42 | |
| *** sgotliv has quit IRC | 17:42 | |
| *** sgotliv has joined #openstack-cinder | 17:42 | |
| nikeshm | hemna: yes i will look into that,talk to rammy but he said he also donot know | 17:43 |
| nikeshm | how to do this | 17:43 |
| uberjay | thingee: a campmate completed and submitted his med-school application in the first 3 days on playa (note: this did not look like a fun time) | 17:44 |
| asselin | nikeshm, how did you setup your log server? | 17:44 |
| nikeshm | asselin: i setup manually | 17:44 |
| asselin | nikeshm, try setting up a new one with my scripts. might fix it for you | 17:44 |
| nikeshm | asselin: sure | 17:45 |
| asselin | https://github.com/rasselin/os-ext-testing/blob/master/puppet/install_log_server.sh | 17:45 |
| *** annegentle has joined #openstack-cinder | 17:45 | |
| asselin | instructions are int he readme | 17:45 |
| asselin | nikeshm, then you log server is guarenteed the same as mine & upstream | 17:45 |
| *** simondodsley has joined #openstack-cinder | 17:45 | |
| *** jungleboyj has joined #openstack-cinder | 17:46 | |
| uberjay | asselin: nikeshm that worked for me -- one gotcha were the assumptions about how your jenkins job is named (because of the directory name). this only controls what readme html gets attached to a particular directory, so it's not a huge deal, obviously. | 17:48 |
| uberjay | e.g., <Directory /srv/static/logs/*/*/*/*/*-tempest-dsvm*/*> | 17:49 |
| uberjay | ReadmeName /help/tempest-overview.html | 17:49 |
| uberjay | </Directory> | 17:49 |
| *** xyang1 has joined #openstack-cinder | 17:50 | |
| asselin | uberjay, what is your directory structure like? | 17:50 |
| uberjay | my problem was the jenkins job name didn't match (*-tempest-dsvm*) originally, but I've since renamed it. | 17:52 |
| uberjay | i had a confusing moment wondering why my CI logs didn't look like other CI logs, is all. :) | 17:53 |
| asselin | uberjay, ok interesting...perhaps an upstream patch to customize that would be nice to have. thanks for mentioning it | 17:53 |
| asselin | uberjay, that just affects the readme right? | 17:53 |
| uberjay | yep | 17:53 |
| uberjay | asselin: all of the work for unzipping and annotating the log files themselves still worked, the readme footers were just not attached to each directory | 17:54 |
| *** vokt has joined #openstack-cinder | 17:57 | |
| *** ebalduf has joined #openstack-cinder | 18:00 | |
| *** ebalduf has quit IRC | 18:00 | |
| *** ebalduf has joined #openstack-cinder | 18:00 | |
| *** ebalduf has quit IRC | 18:01 | |
| *** amoturi has quit IRC | 18:04 | |
| nikeshm | asselin: thanks working on it,btw whats your work location :) | 18:05 |
| asselin | nikeshm, california | 18:05 |
| nikeshm | asselin: great, i am too in san jose,californina | 18:06 |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove pretty_tox and use ostestr https://review.openstack.org/186859 | 18:07 |
| jgriffith | mtreinish: ^^ I think I finally beat you on one!!! | 18:08 |
| jgriffith | mtreinish: going to celebrate by having lunch outside ;) | 18:08 |
| *** annegentle has quit IRC | 18:08 | |
| *** barra204 has joined #openstack-cinder | 18:08 | |
| *** annegentle has joined #openstack-cinder | 18:09 | |
| *** lpetrut has joined #openstack-cinder | 18:12 | |
| uberjay | hemna: thanks for taking a look at our driver. as for the question about requests -- yes, I would actually prefer to use it over httplib. when I first looked into it, i incorrectly determined requests wasn't necessarily available. I'd be happy to change it over if you think it's worth doing before merging. | 18:14 |
| *** barra204 has quit IRC | 18:15 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/cinder: Implement Cinder Volume driver for HGST Solutions https://review.openstack.org/186580 | 18:16 |
| uberjay | hemna: the utilization question... I'm going to have to talk with some people to see if we can advertise a meaningful value. i was worried about depending on that for L1, and instead have it be an incremental change (hopefully for L1, or soon thereafter) | 18:17 |
| *** sgotliv has quit IRC | 18:18 | |
| *** barra204 has joined #openstack-cinder | 18:20 | |
| *** rushiagr_away is now known as rushiagr | 18:25 | |
| *** alexpilotti has joined #openstack-cinder | 18:26 | |
| *** barra204_ has joined #openstack-cinder | 18:28 | |
| dhellmann | thingee: thinking about what you said, was 0.1.0 the first release of os-brick? if you needed to release something to fix requirements, it looks like there is only one release now, so I'm confused | 18:29 |
| *** rasoto has quit IRC | 18:29 | |
| *** ebalduf has joined #openstack-cinder | 18:30 | |
| thingee | dhellmann: yes, 0.1.0 was the first. we're on 0.1.1 | 18:30 |
| dhellmann | hmm, I don't see that tag in my repo | 18:30 |
| thingee | dhellmann: https://pypi.python.org/pypi/os-brick https://github.com/openstack/os-brick/tree/0.1.1 | 18:31 |
| *** barra204 has quit IRC | 18:31 | |
| dhellmann | yeah, I see. I wonder why I don't have the tag | 18:31 |
| dhellmann | ok, a fresh checkout brings it in | 18:32 |
| *** patrickeast has quit IRC | 18:32 | |
| *** ebalduf has quit IRC | 18:34 | |
| *** annegentle has quit IRC | 18:41 | |
| *** patrickeast has joined #openstack-cinder | 18:42 | |
| *** patrickeast has quit IRC | 18:43 | |
| nikeshm | asselin: tried to set up log server as your scripts http://paste.openstack.org/show/245416/ | 18:51 |
| nikeshm | asselin: i got this in the end | 18:51 |
| asselin | nikeshm, strange...seems network issue or the server's down? | 18:51 |
| nikeshm | but able to ping that | 18:52 |
| asselin | try running it again | 18:52 |
| *** barra204_ is now known as shakamunyi | 18:52 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Fix capacity filter to allow oversubscription https://review.openstack.org/185764 | 18:53 |
| *** fthiagogv has quit IRC | 18:55 | |
| *** vokt has quit IRC | 18:55 | |
| *** buster_604 has quit IRC | 18:58 | |
| *** rushiagr is now known as rushiagr_away | 18:58 | |
| nikeshm | asselin: ok by mistake i gave private key now i changed it to public key http://paste.openstack.org/show/245436/ | 18:59 |
| nikeshm | in domain i gave public ip of my log server | 18:59 |
| openstackgerrit | John Griffith proposed openstack/cinder: Add volume ID to fake volumes in Gluster tests https://review.openstack.org/186876 | 19:00 |
| *** aix has joined #openstack-cinder | 19:00 | |
| asselin | nikeshm, yeah...domain can be anything you like | 19:00 |
| asselin | nikeshm, I haven't seen it breaking anything | 19:01 |
| hemna | uberjay, it's fine to get this one in with 'unknown', I just wanted to make you aware of the ramifications of it | 19:03 |
| mtreinish | jgriffith: couldn't you just have rebased when you pushed up the gluster patch instead of abandoning? | 19:06 |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove pretty_tox and use ostestr https://review.openstack.org/186879 | 19:07 |
| jgriffith | mtreinish: yeah, but every time I do that I screw something up | 19:07 |
| mtreinish | heh, ok | 19:07 |
| hemna | uberjay, so requests is available fwiw. | 19:07 |
| jgriffith | mtreinish: I don't know why, I just can't get that workflow down | 19:08 |
| jgriffith | guess I don't do it enough | 19:08 |
| mtreinish | jgriffith: I normally just do a 'git review -x review_id' with the new patch as head | 19:08 |
| *** crose has quit IRC | 19:08 | |
| mtreinish | that'll cherry-pick the review ontop | 19:08 |
| jgriffith | mtreinish: Oh... son of a | 19:08 |
| jgriffith | mtreinish: I was making it way harder than that | 19:08 |
| jgriffith | mtreinish: I always try and cherry-pick, merge etc | 19:09 |
| uberjay | hemna: ok, yeah, that makes sense. thanks! i think i'd rather get the base driver in, rather than hold it up for what will end up being a very small driver change when we sort out the backend details. i'm adding a task for the requests changeover, too. | 19:09 |
| uberjay | hemna: obviously, using requests would be nicer :) | 19:09 |
| hemna | yah you can switch to requests in a follow up | 19:09 |
| hemna | as well as change the reporting stats | 19:09 |
| hemna | no big. | 19:09 |
| *** ebalduf has joined #openstack-cinder | 19:10 | |
| uberjay | ok, great :) | 19:11 |
| jgriffith | mtreinish: the infra manual docs even spell it out for me :( | 19:11 |
| jgriffith | mtreinish: ok, filed that one away | 19:11 |
| mtreinish | jgriffith: heh, that's better than what I used to do when I first started using git. I'd use format patch and then git am to do a ghetto rebase | 19:11 |
| jgriffith | mtreinish: :) | 19:12 |
| jgriffith | sounds like my git-diff patch apply method :) | 19:12 |
| jgriffith | mtreinish: problem for me is "old habbits" die REALLY hard | 19:12 |
| *** Yogi1 has quit IRC | 19:22 | |
| *** alexpilotti has quit IRC | 19:23 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove pretty_tox and use ostestr https://review.openstack.org/186879 | 19:25 |
| openstackgerrit | John Griffith proposed openstack/cinder: Add volume ID to fake volumes in Gluster tests https://review.openstack.org/186876 | 19:26 |
| nikeshm | asselin: done,but do we have to copy logs in /srv/static/logs/ | 19:37 |
| *** jungleboyj has quit IRC | 19:39 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Remove pretty_tox and use ostestr https://review.openstack.org/186879 | 19:42 |
| *** dannywilson has quit IRC | 19:44 | |
| *** dannywilson has joined #openstack-cinder | 19:48 | |
| *** patrickeast has joined #openstack-cinder | 19:48 | |
| nikeshm | asselin:now on clicking on log,its giving "file not found" | 19:50 |
| mtanino | DuncanT: Hi, | 19:54 |
| *** IlyaG has joined #openstack-cinder | 19:57 | |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Non Disruptive Backup https://review.openstack.org/186897 | 19:59 |
| *** dannywilson has quit IRC | 20:00 | |
| *** kvidvans has quit IRC | 20:01 | |
| *** timcl has quit IRC | 20:02 | |
| *** bkopilov has quit IRC | 20:04 | |
| *** openstack has joined #openstack-cinder | 20:06 | |
| uberjay | hemna: is there anything else you think I should look at? | 20:08 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls https://review.openstack.org/182396 | 20:13 |
| *** merooney has quit IRC | 20:18 | |
| *** Lee1092 has quit IRC | 20:21 | |
| *** Apoorva has joined #openstack-cinder | 20:22 | |
| asselin | nikeshm, need more details | 20:22 |
| *** openstack has joined #openstack-cinder | 20:31 | |
| nikeshm | aaselin: now its working | 20:34 |
| nikeshm | can yu click on log link in https://review.openstack.org/#/c/177665/ and remove -1 :) | 20:34 |
| *** needmoresummit is now known as raildo | 20:38 | |
| nikeshm | hemna: done now please check | 20:39 |
| hemna | ok | 20:39 |
| asselin | nikeshm, looks good, thank you | 20:39 |
| hemna | cool | 20:40 |
| hemna | ok I'll review it now :) | 20:40 |
| *** alexpilotti has joined #openstack-cinder | 20:41 | |
| asselin | nikeshm, next is fc | 20:41 |
| nikeshm | asselin: sure | 20:42 |
| *** annegentle has joined #openstack-cinder | 20:42 | |
| *** ndipanov has quit IRC | 20:43 | |
| *** rushil has quit IRC | 20:43 | |
| *** dannywilson has joined #openstack-cinder | 20:47 | |
| *** jungleboyj has joined #openstack-cinder | 20:47 | |
| nikeshm | hemna: thanks,can we merge it? | 20:48 |
| Swanson | jgriffith: So, that starting over thing seems to, um, work. Thanks! | 20:48 |
| hemna | nikeshm, so your patch contains 2 drivers | 20:51 |
| hemna | but I think CI is only reporting 1 no ? | 20:51 |
| hemna | you'll need to report both iscsi and FC results | 20:51 |
| hemna | since there are 2 drivers | 20:51 |
| asselin | nikeshm, another possibility is to split your commit into two: one for each driver. But since you still have time to merge, just do the fc. You're almost there. | 20:52 |
| nikeshm | heman: sure | 20:52 |
| nikeshm | asselin: sure | 20:52 |
| hemna | yah | 20:52 |
| *** openstackgerrit has quit IRC | 20:59 | |
| *** openstackgerrit has joined #openstack-cinder | 21:00 | |
| *** raildo has quit IRC | 21:00 | |
| openstackgerrit | Mike Perez proposed openstack/cinder-specs: Standard Capabilities https://review.openstack.org/150511 | 21:01 |
| hemna | harlowja, ping | 21:02 |
| harlowja | hemna sup | 21:04 |
| hemna | hey | 21:04 |
| harlowja | yo yo | 21:04 |
| hemna | just curious why we do a local file lock around the create flow ? | 21:05 |
| hemna | just trying to understand it | 21:05 |
| hemna | looks like we are dynamically creating the lock name based upon some create params | 21:05 |
| hemna | to lock out the deletion of snapshots/volumes ? | 21:06 |
| harlowja | seems like thats why it was added | 21:06 |
| hemna | so to prevent someone from deleting a snapshot while we create a volume from it ? | 21:06 |
| harlowja | https://github.com/openstack/cinder/commit/4f6e5fcc25 | 21:07 |
| harlowja | seems like that one might explain why | 21:07 |
| hemna | ick ok | 21:07 |
| harlowja | and/or https://review.openstack.org/#/c/56442/ | 21:07 |
| *** julim has quit IRC | 21:07 | |
| hemna | *sigh* | 21:08 |
| hemna | yet another thing we have to fix | 21:08 |
| hemna | or HA won't ever work | 21:08 |
| hemna | it seems like we keep baking in HA problems | 21:08 |
| harlowja | locks at any level do that :-/ | 21:08 |
| harlowja | well not any level, but most levels | 21:08 |
| hemna | well these are local file locks | 21:08 |
| harlowja | right | 21:09 |
| hemna | distributed locking is bad, but doesn't preclude HA | 21:09 |
| hemna | :( | 21:09 |
| hemna | bleh | 21:09 |
| harlowja | i've seen https://review.openstack.org/#/c/183537/ | 21:09 |
| harlowja | which seems to try to switch those to distributed locking | 21:09 |
| harlowja | seems to be actively being workedo n | 21:10 |
| hemna | ah yah ok | 21:10 |
| hemna | cool | 21:10 |
| harlowja | another option/thought | 21:10 |
| *** bswartz has quit IRC | 21:10 | |
| harlowja | ironic has https://github.com/openstack/ironic/blob/master/ironic/common/states.py (which taskflow also has something like) | 21:11 |
| harlowja | they i think use that before/after each operation on a volume | 21:11 |
| harlowja | *on a BM i mean | 21:11 |
| hemna | you said tasflow | 21:11 |
| hemna | that's still ok in here isn't it? :P | 21:11 |
| harlowja | ha | 21:11 |
| harlowja | u said tasflow | 21:11 |
| hemna | doh! | 21:11 |
| harlowja | so i think that spelling is ok? | 21:11 |
| harlowja | thats a different secret project | 21:11 |
| harlowja | lol | 21:11 |
| harlowja | so the other option is to do something like that, pass that state-machine around in cinder context (?) and transition on it | 21:12 |
| *** simondodsley has quit IRC | 21:12 | |
| hemna | so I guess this locking is sort of special though | 21:12 |
| hemna | as the state change needs to happen on the parent object | 21:12 |
| harlowja | then u have a http://docs.openstack.org/developer/ironic/dev/states.html (and a nice picture to) | 21:12 |
| hemna | to prevent actions happening on them | 21:12 |
| harlowja | well its on the volume right? | 21:12 |
| harlowja | just like the ironic one i think is on the BM | 21:12 |
| hemna | the locks are being placed on the parent snapshot | 21:13 |
| hemna | and or parent volume | 21:13 |
| hemna | to prevent them from being deleted | 21:13 |
| hemna | while the new volume is being created from them. | 21:13 |
| hemna | that's the tricky sticky part | 21:13 |
| harlowja | right, gotcha | 21:13 |
| hemna | life is complicated | 21:14 |
| hemna | bleh | 21:14 |
| hemna | ok thanks | 21:14 |
| harlowja | parent_snapshot.transition(SNAPPING) | 21:14 |
| hemna | sorry to bug ya | 21:14 |
| harlowja | parent_volume.transition(SNAPPING) and then do stuff | 21:14 |
| harlowja | * https://github.com/openstack/ironic/blob/master/ironic/conductor/task_manager.py#L288 (which i think is the ironic code to do similar stuffs) | 21:14 |
| * hemna wonders if some of this magicness could be handled by objectifying everything | 21:14 | |
| harlowja | meh, no magic, just requires work :-P | 21:14 |
| hemna | turning the volume dicts into objects that can have their internal states and parents touched automatically | 21:15 |
| hemna | yah | 21:15 |
| harlowja | magic not usually solve much of anything, ha | 21:15 |
| harlowja | unless its the good kind, ha | 21:15 |
| harlowja | it might be intersting to poke the ironic folks, i only know a little bit about there state machien and how they use it | 21:15 |
| * harlowja be interested to know | 21:15 | |
| harlowja | maybe its applicable to u guys to | 21:15 |
| hemna | could be | 21:16 |
| hemna | it's a big can to open.... | 21:16 |
| harlowja | i know the code that is the state-machine (since i made it), but not exactly sure how they use it | 21:16 |
| hemna | I'm not sure I'm the right guy to do that | 21:16 |
| harlowja | :) | 21:16 |
| harlowja | engage can opener! | 21:16 |
| harlowja | engageeeeee | 21:16 |
| hemna | but it's a big problem for Cinder that we need some sort of plan to finally address it | 21:16 |
| harlowja | +2 | 21:16 |
| *** annegentle has quit IRC | 21:17 | |
| harlowja | so there are a few avenues, imho the explict state-machine is the best | 21:17 |
| harlowja | distributed locks will help, but will imho be hard to figure out when they fail | 21:17 |
| harlowja | and they will fail, lol | 21:17 |
| hemna | yah | 21:17 |
| harlowja | and they fail differently than local file locks | 21:17 |
| harlowja | *local file locks don't have servers diseapper, or expiries... | 21:18 |
| hemna | and those failures happen in very difficult scenarios to reproduce | 21:18 |
| harlowja | right | 21:18 |
| hemna | makes my brain hurt on a Friday | 21:18 |
| harlowja | :) | 21:19 |
| harlowja | thats why i was almost thinking, just having top-level state machine that is very strict at the API level might help, do big transitions and deny anything else | 21:20 |
| hemna | yah | 21:21 |
| hemna | we do a lot of this type of checking deep down in the manager | 21:21 |
| hemna | we should be doing more checks in the API and denying there. | 21:21 |
| harlowja | volume Z being created, do little stuff in init, transition parent_snapshot X, parent_volume Y to things that can not be acted on until that Z is done | 21:21 |
| harlowja | or if X or Y can not be transitioned, stop | 21:22 |
| hemna | yah | 21:22 |
| harlowja | and as final final action of Z being created, move X, Y back to 'useable' or whatever | 21:22 |
| hemna | VolumeIsBusyException | 21:22 |
| harlowja | ya | 21:22 |
| harlowja | something like that | 21:22 |
| harlowja | then as time goes on, releax that | 21:22 |
| hemna | nova shits all over that now though | 21:22 |
| harlowja | sucks for nova, lol | 21:22 |
| hemna | nova expects stuff to just always work or timeout | 21:22 |
| harlowja | should learn to put its diapears on | 21:22 |
| hemna | heh yah | 21:23 |
| *** jungleboyj has quit IRC | 21:23 | |
| hemna | I dunno | 21:23 |
| hemna | I think it's ok for Cinder to just simply say....sorry I'm busy right now. | 21:23 |
| harlowja | so u said timeout, ok, turn VolumeIsBusyException -> VolumeTimedOutWaitingtoBecomeUnBusyException | 21:23 |
| hemna | hehe | 21:23 |
| harlowja | if u can make the exception name longer, that'd be cool | 21:23 |
| hemna | if only we could put ! in a name | 21:24 |
| hemna | VolumeIsBusyException | 21:24 |
| hemna | or | 21:24 |
| hemna | VolumeIsBusyException!!!!!!!!! | 21:24 |
| hemna | yah, really busy! | 21:24 |
| harlowja | hemna i'd agree, i feel its easier to be strict, and later relax that strictness as u want, vs starting the other way around | 21:24 |
| harlowja | ll | 21:24 |
| harlowja | lol | 21:24 |
| hemna | VolumeIsBusyExceptionOMFGBBQ!!!!!! | 21:25 |
| harlowja | VolumeIsBusyNoShitOnYourselfExcepiton | 21:25 |
| hemna | :) | 21:25 |
| hemna | ok that's more like it on a Friday. | 21:25 |
| harlowja | lol | 21:25 |
| harlowja | once https://review.openstack.org/#/c/141961/ is finally all good to go, someone could think about that state-machine path to | 21:25 |
| * harlowja is waiting on a few other reviews for that | 21:26 | |
| harlowja | its another possiblity... | 21:26 |
| harlowja | u can not escape me cinder!!!! | 21:26 |
| harlowja | muhahaha | 21:26 |
| hemna | :) | 21:26 |
| harlowja | * https://review.openstack.org/#/c/185077/ (waiting on) | 21:26 |
| openstackgerrit | Yucong Feng proposed openstack/cinder-specs: Introducing parallel zoning https://review.openstack.org/186918 | 21:26 |
| *** shakamunyi has quit IRC | 21:26 | |
| harlowja | and we could figure out how its worked out for ironic | 21:27 |
| harlowja | dev does mention it in his emails, and stuff, and operators seem to like it, ha | 21:27 |
| harlowja | *deva | 21:27 |
| harlowja | * https://etherpad.openstack.org/p/liberty-ironic-state-machine-next-steps | 21:28 |
| hemna | automation | 21:28 |
| hemna | new lib ? | 21:28 |
| hemna | oh sorry | 21:28 |
| *** krtaylor has quit IRC | 21:28 | |
| *** lpetrut has left #openstack-cinder | 21:28 | |
| hemna | automaton | 21:28 |
| harlowja | ya, ironic copied out the state-machine from taskflow that i wrote, into there tree | 21:28 |
| harlowja | and then i was like, eck, not gonna work out long-term | 21:28 |
| harlowja | sooo, waited until they had stabalized a little, and were still using it | 21:29 |
| harlowja | and make new oslo library (really general purpose lib) | 21:29 |
| harlowja | and will eliminate both copies with that | 21:29 |
| *** lcurtis has joined #openstack-cinder | 21:29 | |
| hemna | heh, I like your examples | 21:30 |
| hemna | barks! | 21:30 |
| *** openstack has joined #openstack-cinder | 21:32 | |
| hemna | to get those checks happening up front | 21:32 |
| hemna | almost like a "can I do this thing I'm about to do on this object?" | 21:32 |
| harlowja | ya, should be possible to make them appear on the object itself | 21:32 |
| hemna | yes = go to volume manager and do it | 21:32 |
| * harlowja was thinking about having tasks (in tasflow) also have that same method ('transition') or something | 21:33 | |
| *** primechuck has quit IRC | 21:33 | |
| harlowja | right now its an external function call to do it | 21:33 |
| hemna | no = ResourceIsBusy or something along those lines | 21:33 |
| hemna | ask permission to do it | 21:33 |
| harlowja | hemna well the manager needs to do stuff for async things though right? | 21:33 |
| hemna | get approval, then go ahead | 21:33 |
| harlowja | if all api's were blocking, then i guess u could | 21:33 |
| hemna | if mom says no, then report no up the chain | 21:33 |
| harlowja | right, ask all the mom's that are needed to ask | 21:34 |
| hemna | yah | 21:34 |
| harlowja | *grandmom, mom... | 21:34 |
| harlowja | and then transition, and then do stuff | 21:34 |
| harlowja | and then transition mom and grandmom when done (async or other) | 21:34 |
| hemna | so... | 21:34 |
| hemna | volume.start_action("clone") | 21:35 |
| hemna | which asks the moms | 21:35 |
| hemna | or reports back failures | 21:35 |
| hemna | and then call manager to do the action | 21:35 |
| *** jungleboyj has joined #openstack-cinder | 21:35 | |
| hemna | and then volume.end_action | 21:35 |
| hemna | probably too simplistic | 21:35 |
| hemna | dunno | 21:35 |
| harlowja | ya, something like that i think | 21:35 |
| hemna | I do like the idea of encapsulating that logic into the volume object itself | 21:36 |
| hemna | then anyone can ask those same transitions | 21:36 |
| harlowja | or volume.transition('useable', 'clone') | 21:36 |
| harlowja | or hiding that, idk | 21:36 |
| hemna | I think the api into the volume object should be simple | 21:37 |
| harlowja | k | 21:37 |
| hemna | and sort of hide the transitions states | 21:37 |
| harlowja | ok dokie, thats fine | 21:37 |
| hemna | so the caller on the volume doesn't have to know any smarts about it | 21:37 |
| harlowja | sure | 21:37 |
| hemna | but simple asks, hey, can I clone you ? | 21:37 |
| hemna | or, can I delete you ? | 21:37 |
| harlowja | sure, makes sense | 21:37 |
| harlowja | hide the complexity in start_action or whatever | 21:37 |
| hemna | internals knows the current state, and locks it or whatnot | 21:38 |
| hemna | yah | 21:38 |
| hemna | at least it's all in 1 spot then | 21:38 |
| harlowja | ya | 21:38 |
| hemna | right now we have so many checks all over the place | 21:38 |
| harlowja | and just have each API call do volume.start_action() as its first thing, or die | 21:38 |
| hemna | it's damn near impossible to know what you can do on a volume now | 21:38 |
| *** primechuck has joined #openstack-cinder | 21:38 | |
| harlowja | and a volume.end_action at the end | 21:38 |
| hemna | yup! | 21:38 |
| harlowja | and thats all | 21:38 |
| hemna | exactly | 21:38 |
| hemna | so reporting to Nova or any caller is easy | 21:39 |
| hemna | and up front | 21:39 |
| harlowja | block everything else imho, until u want to make it possible to run states at the same time (but meh) | 21:39 |
| hemna | and for volume creation | 21:39 |
| hemna | the start_action | 21:39 |
| hemna | could put those distributed locks if needed on the parents | 21:39 |
| *** merooney has joined #openstack-cinder | 21:40 | |
| hemna | for create from snap, or prevent deletion, etc | 21:40 |
| hemna | and upon a failure | 21:40 |
| harlowja | just transition the parents to? | 21:40 |
| hemna | we might be able to get away with simple calling end_action on the same object to clean up | 21:40 |
| hemna | yup | 21:40 |
| hemna | transition and save inside of start_action | 21:40 |
| harlowja | avoid locks, just move parents into states that can not be touched | 21:40 |
| hemna | yup | 21:41 |
| harlowja | leave grandma alone goddamnit | 21:41 |
| harlowja | lol | 21:41 |
| harlowja | *thats the message u can use | 21:41 |
| hemna | and then the API calls into delete snapshot will puke because snapshot_start_action('delete') says no. | 21:41 |
| harlowja | yup | 21:41 |
| hemna | on the parent(s) | 21:41 |
| harlowja | yup | 21:41 |
| hemna | ok | 21:41 |
| hemna | make it so | 21:41 |
| hemna | :P | 21:42 |
| harlowja | hey, u make it so, lol | 21:42 |
| harlowja | vilobhmm make it so! | 21:42 |
| harlowja | lol | 21:42 |
| vilobhmm | state machine would be a better choice than distributed locks or any kind of locking isn't it | 21:42 |
| hemna | I'll be drinking beer all weekend and fishing. | 21:42 |
| *** primechuck has quit IRC | 21:42 | |
| harlowja | https://review.openstack.org/#/c/110434/28/cinder/volume/states.py ( hemna u can revive that ) | 21:42 |
| harlowja | or combine it with automaton | 21:42 |
| harlowja | orrrrr something, lol | 21:42 |
| hemna | the trick is you still need some sort of locks to make the transitions atomic | 21:42 |
| harlowja | yup | 21:42 |
| hemna | especially in the case of cascading those transitions to parent(s)/children(s) | 21:42 |
| hemna | yah something.... | 21:43 |
| *** merooney has quit IRC | 21:43 | |
| harlowja | few options for that, select for, tooz... | 21:43 |
| hemna | I'll forget all of this by Monday | 21:43 |
| harlowja | but at least the lock stuff is in one place | 21:43 |
| harlowja | *and is around a tiny set of methods | 21:43 |
| harlowja | hemna good things there are loggs, lol | 21:43 |
| hemna | :) | 21:43 |
| vilobhmm | state machine validation at API level locking while doing compare and set at the db level…. | 21:44 |
| harlowja | something like that | 21:44 |
| harlowja | and profit! | 21:44 |
| hemna | ooh yah profit! | 21:44 |
| vilobhmm | state machine validation at API level ; locking while doing compare and set at the db level….something of that sort was tried here https://review.openstack.org/#/c/124205/ disregard the micro-states part but the concept of "state machine validation at API level ; locking while doing compare and set at the db level" | 21:46 |
| harlowja | so hemna ya, there u go :-P | 21:46 |
| vilobhmm | hemna : ^^ | 21:46 |
| hemna | anyway, I'm a big fan of encapsulation and concentrating code | 21:46 |
| hemna | so it's not spread the hell around everywhere | 21:46 |
| harlowja | get er' done | 21:46 |
| *** annegentle has joined #openstack-cinder | 21:46 | |
| harlowja | just starting with the start_action and end_action and saying 'no' alot more than currently i'd think be a major win | 21:48 |
| harlowja | would allow u to remove a bunch of locks | 21:48 |
| harlowja | since the initial 'no' would make them not needed | 21:49 |
| harlowja | and people that are trying to do things concurrently on the same volume right now, are probably messing themselves up right now anyway | 21:49 |
| harlowja | *even with locks | 21:49 |
| *** IlyaG has quit IRC | 21:51 | |
| *** mriedem has quit IRC | 21:51 | |
| harlowja | my 2 cents :-P | 21:52 |
| *** marcusvrn has quit IRC | 21:53 | |
| *** dims_ has joined #openstack-cinder | 21:53 | |
| *** akshai has quit IRC | 21:53 | |
| hemna | harlowja, yah I agree | 21:53 |
| *** IlyaG_ has joined #openstack-cinder | 21:53 | |
| *** mdenny has quit IRC | 21:54 | |
| harlowja | cools, let me know when thats finished and i'll review it, lol | 21:54 |
| *** annashen has joined #openstack-cinder | 21:55 | |
| *** dimsum__ has quit IRC | 21:55 | |
| *** alexpilotti has quit IRC | 21:56 | |
| vilobhmm | hemna : May be we can start here https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L120 add the methods like start() end() and then invoke it whereever needed before doing a state validation ? | 21:57 |
| vilobhmm | since you said you would prefer it doing in an object…and have the obj expose these methods | 21:57 |
| hemna | yah I'd prefer that | 21:58 |
| hemna | that volume object is more of a DB abstraction object right? | 21:59 |
| vilobhmm | yes | 21:59 |
| *** jungleboyj has quit IRC | 21:59 | |
| hemna | hrmm yah I was thinking of something slightly higher I guess. | 22:00 |
| *** jms has quit IRC | 22:00 | |
| harlowja | objects <-> rpc <-> db orm <-> sql abstraction ? | 22:00 |
| vilobhmm | if you see https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L37 | 22:00 |
| harlowja | its a layered sandwich of goodness, lol | 22:00 |
| vilobhmm | :P | 22:00 |
| harlowja | objects <-> rpc <-> db orm <-> sql abstraction <-> sql <-> some btrees ? | 22:00 |
| harlowja | there | 22:00 |
| hemna | in my past life | 22:01 |
| hemna | I had something similar | 22:01 |
| hemna | I had a DB abstraction object | 22:01 |
| harlowja | a sandwich of goodness? | 22:01 |
| harlowja | oh | 22:01 |
| hemna | that knew how to do DB related things | 22:01 |
| hemna | and another 'business logic' style of container object that knew how to take actions and do 'work' on the db abstracted objects | 22:01 |
| harlowja | but it wasn't a seven layered burrito | 22:02 |
| harlowja | and how did u survive without that | 22:02 |
| vilobhmm | burrito haha | 22:02 |
| hemna | no sour guacamole! | 22:02 |
| harlowja | http://www.tacobell.com/food/menuitem/7+layer+burrito oh yaaa | 22:02 |
| harlowja | haha | 22:02 |
| hemna | yah TB's guac is awful | 22:02 |
| harlowja | lol | 22:02 |
| hemna | mostly lemons | 22:02 |
| hemna | and some green like liquid | 22:02 |
| harlowja | woot | 22:02 |
| harlowja | yummie | 22:03 |
| hemna | stuffs of champs baby! | 22:03 |
| harlowja | turns u into a man | 22:03 |
| harlowja | *and/or woman | 22:04 |
| harlowja | idk | 22:04 |
| harlowja | might turn a man into a woman? | 22:04 |
| harlowja | :-/ | 22:04 |
| hemna | girly man! | 22:04 |
| harlowja | lol | 22:04 |
| vilobhmm | ok hemna….i think we have a clear plan (which I feel we do after a long time) I can help you on this effort…may be we can discuss this in cinder weekly on 06/03 ? | 22:04 |
| harlowja | plan is to turn cinder into taco bell? | 22:04 |
| harlowja | cinderbell | 22:05 |
| vilobhmm | :P | 22:05 |
| hemna | vilobhmm, well, it's an idea | 22:05 |
| hemna | I'm not sure I'd go so far as calling it a plan :) | 22:05 |
| hemna | needs more beers | 22:05 |
| vilobhmm | ok idea .... | 22:05 |
| hemna | it's a boatloads of change | 22:05 |
| hemna | to implement | 22:05 |
| vilobhmm | i know | 22:05 |
| hemna | and affects nearly damn everything | 22:05 |
| vilobhmm | yup! | 22:05 |
| hemna | lets boil the ocean and break the world. | 22:05 |
| harlowja | u can call it the pyramid sub-project | 22:05 |
| hemna | who's ready? | 22:05 |
| harlowja | *pyramids built out of cinder bolks | 22:06 |
| vilobhmm | count me in | 22:06 |
| harlowja | *blocks | 22:06 |
| vilobhmm | but you have to do it someday …why not do it now :P | 22:06 |
| harlowja | we got a lively one, he might need more 7-layer burritos | 22:06 |
| hemna | harlowja, that might get us back into diapers though | 22:07 |
| harlowja | no might | 22:07 |
| harlowja | it will | 22:07 |
| hemna | kinda early in the weekend for that | 22:07 |
| harlowja | ok, on sunday then | 22:07 |
| harlowja | diaper night | 22:07 |
| harlowja | sounds like a very awkward party :-/ | 22:08 |
| hemna | yah, send invites to the managers and lets just go to the bar. | 22:08 |
| harlowja | lol | 22:09 |
| harlowja | and someone deliver 100 7-layer burritos | 22:09 |
| hemna | yah! party on! | 22:10 |
| harlowja | party will likely more be in those diapers | 22:11 |
| harlowja | :-/ | 22:11 |
| *** annashen has quit IRC | 22:11 | |
| * harlowja runs away, lol | 22:11 | |
| *** cbader has quit IRC | 22:12 | |
| hemna | :) | 22:12 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 22:15 |
| *** annegentle has quit IRC | 22:18 | |
| *** annegentle has joined #openstack-cinder | 22:20 | |
| *** annashen has joined #openstack-cinder | 22:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 22:30 |
| *** IlyaG_ has quit IRC | 22:34 | |
| *** IlyaG has joined #openstack-cinder | 22:36 | |
| *** hemna is now known as hemnabeer | 22:36 | |
| *** krtaylor has joined #openstack-cinder | 22:36 | |
| *** aix has quit IRC | 22:39 | |
| *** annegentle has quit IRC | 22:40 | |
| *** madskier has joined #openstack-cinder | 22:46 | |
| *** boris-42 has quit IRC | 22:48 | |
| jgriffith | Swanson: cool! | 22:49 |
| jgriffith | Swanson: your next hiccup will be clone errors :) | 22:49 |
| *** ebalduf has quit IRC | 22:50 | |
| *** madskier has quit IRC | 22:50 | |
| jgriffith | hemnabeer: harlowja That sounds like the simple object queue I drew up on the board in FTC last fall | 22:52 |
| *** emagana has quit IRC | 22:55 | |
| *** emagana has joined #openstack-cinder | 22:55 | |
| nikeshm | asselin: hi | 22:57 |
| asselin | hi | 22:58 |
| jgriffith | vilobhmm: hey... did you see my response to your -1 on my patch for os-testr? | 22:58 |
| jgriffith | vilobhmm: seems your "concern" is completely unrelated to my change | 22:59 |
| jgriffith | vilobhmm: unless I'm missing something you're trying to point out | 22:59 |
| vilobhmm | jgriffith : the patch seems ok to me..just wanted to make sure we dont have any code duplication… | 23:05 |
| vilobhmm | i did reply now | 23:05 |
| vilobhmm | was busy with meeting in afternoon sorry for late reply | 23:06 |
| *** sgotliv has joined #openstack-cinder | 23:06 | |
| nikeshm | asselin: trying to add in my job http://paste.openstack.org/show/245826/ | 23:07 |
| vilobhmm | jgriffith : https://review.openstack.org/#/c/186879/3/tox.ini | 23:07 |
| *** sgotliv has quit IRC | 23:07 | |
| asselin | nikeshm, ok...so what's th eissue? | 23:08 |
| *** annashen has quit IRC | 23:08 | |
| nikeshm | asselin: after install_master.sh,manuall copying manually copied invoke-fc-passthrough.sh in /opt/nodepool-scripts/invoke-fc-passthrough.sh,but when running jon its saying script not found | 23:09 |
| asselin | nikeshm, are you using stackforge/third-party-ci-tools or something else? | 23:09 |
| asselin | I recommend you git clone that repo in your pre_test_hook and then you have the file | 23:10 |
| asselin | no need for /opt/nodepool-scripts/ | 23:10 |
| jgriffith | vilobhmm: I get ya... but you gave it a -1 :( | 23:10 |
| vilobhmm | sorry :( | 23:11 |
| vilobhmm | must have been a 0 or so…will remove it | 23:11 |
| vilobhmm | bcz the code changes look good to me apart fromt hat concern | 23:11 |
| jgriffith | thanks | 23:11 |
| vilobhmm | i mentioned that beforehand as weell | 23:12 |
| jgriffith | vilobhmm: right, so you should've given it a +1 :) | 23:12 |
| jgriffith | vilobhmm: if the change set looks good, then that seems like +1 :) | 23:12 |
| vilobhmm | can't give a +1 like that :P had few concerns on it… | 23:12 |
| *** ganso_ has quit IRC | 23:14 | |
| jgriffith | vilobhmm: Hmm... ok | 23:15 |
| jgriffith | vilobhmm: well, whatever I guess | 23:15 |
| jgriffith | vilobhmm: thanks | 23:15 |
| vilobhmm | np | 23:15 |
| nikeshm | asselin : good idea,and what about private key file of FC provider | 23:15 |
| vilobhmm | jgriffith : you have few min | 23:16 |
| asselin | nikeshm, get that on the vm any way you like | 23:16 |
| *** severson_ has quit IRC | 23:24 | |
| *** rongze has joined #openstack-cinder | 23:26 | |
| *** rmesta has quit IRC | 23:30 | |
| *** lcurtis has quit IRC | 23:31 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri Cinder Volume driver https://review.openstack.org/185148 | 23:35 |
| *** rongze has quit IRC | 23:36 | |
| *** rongze has joined #openstack-cinder | 23:37 | |
| *** jdurgin has quit IRC | 23:39 | |
| winston-d | jgriffith: ping | 23:39 |
| *** rongze has quit IRC | 23:41 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Use SolidFire snapshots for Cinder snapshots https://review.openstack.org/186963 | 23:44 |
| jgriffith | winston-d: hey | 23:45 |
| *** xyang1 has quit IRC | 23:52 | |
| jgriffith | winston-d: sorry I missed ya, I'm heading home. Catch ya later? | 23:52 |
| *** bswartz has joined #openstack-cinder | 23:53 | |
| *** vilobhmm has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!