| *** _cjones_ has quit IRC | 00:06 | |
| *** scottda_ has quit IRC | 00:07 | |
| *** scottda_ has joined #openstack-cinder | 00:09 | |
| *** jaypipes has joined #openstack-cinder | 00:30 | |
| *** rushil has quit IRC | 00:33 | |
| *** rushil has joined #openstack-cinder | 00:34 | |
| *** david-lyle has joined #openstack-cinder | 00:35 | |
| *** Masahiro has joined #openstack-cinder | 00:35 | |
| *** Mandell has quit IRC | 00:37 | |
| *** ebalduf has joined #openstack-cinder | 00:37 | |
| *** rushil has quit IRC | 00:38 | |
| *** ryanpetrello has joined #openstack-cinder | 00:39 | |
| *** Yogi1 has joined #openstack-cinder | 00:39 | |
| *** Masahiro has quit IRC | 00:40 | |
| *** ebalduf has quit IRC | 00:42 | |
| *** scottda_ has quit IRC | 00:48 | |
| *** scottda_ has joined #openstack-cinder | 00:58 | |
| *** ryanpetrello has quit IRC | 00:58 | |
| *** ryanpetrello has joined #openstack-cinder | 01:17 | |
| *** scottda_ has quit IRC | 01:17 | |
| *** david-lyle has quit IRC | 01:18 | |
| *** david-lyle has joined #openstack-cinder | 01:25 | |
| *** david-lyle has quit IRC | 01:27 | |
| *** david-lyle has joined #openstack-cinder | 01:28 | |
| *** diegows has joined #openstack-cinder | 01:32 | |
| *** ebalduf has joined #openstack-cinder | 01:38 | |
| *** lpetrut has quit IRC | 01:40 | |
| openstackgerrit | Li Min Liu proposed openstack/cinder: Add secondary sshpool for san storage drivers https://review.openstack.org/146363 | 01:43 |
|---|---|---|
| *** ebalduf has quit IRC | 01:43 | |
| *** ryanpetrello has quit IRC | 01:48 | |
| *** Masahiro has joined #openstack-cinder | 01:51 | |
| *** Mandell has joined #openstack-cinder | 01:53 | |
| *** Masahiro has quit IRC | 01:56 | |
| *** david-lyle has quit IRC | 01:59 | |
| *** diegows has quit IRC | 02:01 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 02:14 |
| *** ling-yun has joined #openstack-cinder | 02:19 | |
| *** kaufer has joined #openstack-cinder | 02:19 | |
| *** Longgeek has joined #openstack-cinder | 02:21 | |
| *** ling-yun-1 has quit IRC | 02:22 | |
| *** ling-yun-1 has joined #openstack-cinder | 02:23 | |
| *** annashen has joined #openstack-cinder | 02:24 | |
| *** ling-yun has quit IRC | 02:25 | |
| *** mberlin1 has joined #openstack-cinder | 02:37 | |
| *** mberlin has quit IRC | 02:37 | |
| *** kaisers has joined #openstack-cinder | 02:38 | |
| *** kaisers1 has quit IRC | 02:38 | |
| *** ebalduf has joined #openstack-cinder | 02:39 | |
| *** BharatK has joined #openstack-cinder | 02:40 | |
| *** ling-yun has joined #openstack-cinder | 02:41 | |
| *** r-daneel has joined #openstack-cinder | 02:42 | |
| *** ebalduf has quit IRC | 02:43 | |
| *** ling-yun-1 has quit IRC | 02:45 | |
| *** ling-yun-1 has joined #openstack-cinder | 02:45 | |
| *** ling-yun has quit IRC | 02:49 | |
| *** zengyunling__ has joined #openstack-cinder | 02:50 | |
| *** Mandell has quit IRC | 02:50 | |
| *** Mandell has joined #openstack-cinder | 02:51 | |
| *** ling-yun-1 has quit IRC | 02:53 | |
| *** ling-yun has joined #openstack-cinder | 02:53 | |
| *** zengyunling__ has quit IRC | 02:56 | |
| *** ling-yun-1 has joined #openstack-cinder | 02:56 | |
| *** lcurtis has joined #openstack-cinder | 02:57 | |
| *** ling-yun has quit IRC | 02:59 | |
| *** zengyunling__ has joined #openstack-cinder | 03:03 | |
| *** ling-yun-1 has quit IRC | 03:06 | |
| *** lcurtis has quit IRC | 03:08 | |
| openstackgerrit | Li Min Liu proposed openstack/cinder: Add secondary sshpool for san storage drivers https://review.openstack.org/146363 | 03:15 |
| *** ling-yun-1 has joined #openstack-cinder | 03:16 | |
| *** zengyunling__ has quit IRC | 03:19 | |
| *** r-daneel has quit IRC | 03:22 | |
| *** r-daneel has joined #openstack-cinder | 03:23 | |
| *** julim has quit IRC | 03:24 | |
| *** julim has joined #openstack-cinder | 03:25 | |
| *** julim has quit IRC | 03:25 | |
| *** Yogi11 has joined #openstack-cinder | 03:31 | |
| *** annashen has quit IRC | 03:33 | |
| *** Yogi11 has quit IRC | 03:34 | |
| *** ebalduf has joined #openstack-cinder | 03:40 | |
| *** Masahiro has joined #openstack-cinder | 03:40 | |
| *** zengyunling__ has joined #openstack-cinder | 03:41 | |
| *** annashen has joined #openstack-cinder | 03:42 | |
| *** ling-yun-1 has quit IRC | 03:43 | |
| *** david-lyle has joined #openstack-cinder | 03:44 | |
| *** ebalduf has quit IRC | 03:45 | |
| *** Masahiro has quit IRC | 03:45 | |
| *** annashen has quit IRC | 03:52 | |
| *** haomaiwang has joined #openstack-cinder | 04:02 | |
| *** Longgeek has quit IRC | 04:14 | |
| *** Longgeek has joined #openstack-cinder | 04:21 | |
| *** hemna has joined #openstack-cinder | 04:23 | |
| *** coolsvap|afk is now known as coolsvap | 04:31 | |
| *** ebalduf has joined #openstack-cinder | 04:41 | |
| *** ebalduf has quit IRC | 04:45 | |
| *** david-lyle has quit IRC | 04:50 | |
| *** david-lyle has joined #openstack-cinder | 04:50 | |
| *** lpetrut has joined #openstack-cinder | 04:50 | |
| *** fischerw has quit IRC | 04:51 | |
| *** hemna has quit IRC | 05:06 | |
| *** david-lyle has quit IRC | 05:06 | |
| *** zengyunling__ has quit IRC | 05:06 | |
| *** ling-yun has joined #openstack-cinder | 05:08 | |
| *** ling-yun-1 has joined #openstack-cinder | 05:09 | |
| *** ling-yun has quit IRC | 05:12 | |
| *** avishay has joined #openstack-cinder | 05:15 | |
| *** teju has joined #openstack-cinder | 05:22 | |
| *** Longgeek has quit IRC | 05:24 | |
| *** hemna has joined #openstack-cinder | 05:28 | |
| openstackgerrit | TaoBai proposed openstack/cinder: IBM Storwize driver Consistency Group Implementation https://review.openstack.org/142683 | 05:29 |
| *** Masahiro has joined #openstack-cinder | 05:29 | |
| *** Longgeek has joined #openstack-cinder | 05:32 | |
| thingee | jgriffith: seems to make sense to me, re: your thoughts on the volume to image issue :) | 05:32 |
| thingee | jgriffith: I definitely missed that in everything else I was looking at | 05:32 |
| *** dulek has joined #openstack-cinder | 05:33 | |
| *** Masahiro has quit IRC | 05:33 | |
| *** lpetrut has quit IRC | 05:35 | |
| *** dulek_ has joined #openstack-cinder | 05:37 | |
| *** dulek has quit IRC | 05:37 | |
| *** RJ_ has joined #openstack-cinder | 05:37 | |
| *** pradipta_away has quit IRC | 05:39 | |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: Raise correct exception when validate_connector failed https://review.openstack.org/146374 | 05:42 |
| *** annashen has joined #openstack-cinder | 05:44 | |
| *** zengyunling__ has joined #openstack-cinder | 05:48 | |
| *** ling-yun-1 has quit IRC | 05:48 | |
| *** kaufer has quit IRC | 05:49 | |
| *** dulek_ has quit IRC | 05:51 | |
| *** Mandell has quit IRC | 05:58 | |
| *** rushil has joined #openstack-cinder | 06:00 | |
| *** pradipta has joined #openstack-cinder | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/144933 | 06:09 |
| *** zengyunling__ has quit IRC | 06:10 | |
| *** ling-yun-1 has joined #openstack-cinder | 06:12 | |
| *** anshul has joined #openstack-cinder | 06:13 | |
| *** nikesh_vedams has joined #openstack-cinder | 06:13 | |
| *** hemna has quit IRC | 06:15 | |
| *** annashen has quit IRC | 06:16 | |
| RJ_ | hey guys having some doubts ... | 06:17 |
| RJ_ | 1. it necessry to have the backend names differnt for both nodes ..?? | 06:17 |
| RJ_ | 2. it is reuired to enable_backend in cinder.conf file ..?? | 06:17 |
| RJ_ | 3. need to have Replication relationship db table: ..? | 06:17 |
| RJ_ | hey this is my code for driver | 06:17 |
| RJ_ | all in one file | 06:17 |
| RJ_ | http://paste.openstack.org/show/156400/ | 06:18 |
| RJ_ | can you ponit out mistakes and sugestions ... | 06:18 |
| RJ_ | please help ...? | 06:18 |
| RJ_ | :p | 06:18 |
| *** zengyunling__ has joined #openstack-cinder | 06:18 | |
| *** ling-yun-1 has quit IRC | 06:19 | |
| *** r-daneel has quit IRC | 06:25 | |
| *** nlevinki has joined #openstack-cinder | 06:26 | |
| *** ling-yun-1 has joined #openstack-cinder | 06:28 | |
| *** Masahiro has joined #openstack-cinder | 06:30 | |
| RJ_ | Error logs : | 06:30 |
| RJ_ | http://paste.openstack.org/show/156401/ | 06:30 |
| *** Mandell has joined #openstack-cinder | 06:31 | |
| anish | jgriffith: hemnafk can I get some feedback on this code ? https://review.openstack.org/#/c/146233/ | 06:32 |
| *** zengyunling__ has quit IRC | 06:32 | |
| *** Masahiro has quit IRC | 06:34 | |
| *** annashen has joined #openstack-cinder | 06:35 | |
| *** nshaikh has joined #openstack-cinder | 06:35 | |
| *** BharatK has quit IRC | 06:36 | |
| *** ling-yun-1 has quit IRC | 06:37 | |
| *** nshaikh has quit IRC | 06:39 | |
| *** ebalduf has joined #openstack-cinder | 06:42 | |
| *** fandi has joined #openstack-cinder | 06:44 | |
| *** fandi has quit IRC | 06:46 | |
| *** fandi has joined #openstack-cinder | 06:47 | |
| *** rushil has quit IRC | 06:47 | |
| *** ebalduf has quit IRC | 06:47 | |
| *** rushil has joined #openstack-cinder | 06:48 | |
| *** rushil has quit IRC | 06:48 | |
| *** BharatK has joined #openstack-cinder | 06:50 | |
| *** annashen has quit IRC | 06:55 | |
| *** nshaikh has joined #openstack-cinder | 06:56 | |
| nikesh_vedams | i m trying to create CI for cinder driver testing | 06:57 |
| nikesh_vedams | i deployed a kilo devstack on ubuntu 14.04 server. i launched a instance from dashboard,successfully launched but i am unable to open console from dashboard for instance Also instacne is unable to get ip | 06:57 |
| *** rushiagr_away is now known as rushiagr | 06:58 | |
| *** Longgeek has quit IRC | 06:59 | |
| *** fandi has quit IRC | 07:01 | |
| *** cdelatte has joined #openstack-cinder | 07:01 | |
| *** cdelatte has quit IRC | 07:02 | |
| thingee | nikesh_vedams: I recommend asking #openstack. This is not a support channel for openstack. | 07:02 |
| nikesh_vedams | ok | 07:03 |
| openstackgerrit | Johnson koil raj proposed openstack/cinder: VMware: driver not handling port other than 443 https://review.openstack.org/146384 | 07:05 |
| *** paul-- has quit IRC | 07:08 | |
| *** Longgeek has joined #openstack-cinder | 07:08 | |
| *** paul-- has joined #openstack-cinder | 07:09 | |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: Raise correct exception when validate_connector failed https://review.openstack.org/146374 | 07:13 |
| openstackgerrit | TaoBai proposed openstack/cinder: IBM Storwize driver Consistency Group Implementation https://review.openstack.org/142683 | 07:13 |
| *** avishay has quit IRC | 07:14 | |
| *** avishay_ has joined #openstack-cinder | 07:14 | |
| *** rakesh___510 has joined #openstack-cinder | 07:15 | |
| *** sgotliv has quit IRC | 07:18 | |
| *** coolsvap is now known as coolsvap|afk | 07:20 | |
| *** fandi has joined #openstack-cinder | 07:24 | |
| *** thingee has quit IRC | 07:25 | |
| *** coolsvap has joined #openstack-cinder | 07:29 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Use dict comprehensions instead of dict constructor https://review.openstack.org/143851 | 07:35 |
| *** Miouge has joined #openstack-cinder | 07:35 | |
| *** coolsvap has quit IRC | 07:38 | |
| *** ebalduf has joined #openstack-cinder | 07:43 | |
| *** ebalduf has quit IRC | 07:48 | |
| *** jamielennox is now known as jamielennox|away | 07:56 | |
| *** avishay_ has quit IRC | 07:56 | |
| *** paul-- has quit IRC | 08:00 | |
| *** paul-- has joined #openstack-cinder | 08:01 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Use dict comprehensions instead of dict constructor https://review.openstack.org/143851 | 08:05 |
| *** e0ne has joined #openstack-cinder | 08:07 | |
| *** e0ne has quit IRC | 08:10 | |
| *** chlong has quit IRC | 08:10 | |
| *** skraynev has joined #openstack-cinder | 08:16 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 08:16 |
| openstackgerrit | Johnson koil raj proposed openstack/cinder: VMware: driver not handling port other than 443 https://review.openstack.org/146384 | 08:18 |
| *** pwehrle has joined #openstack-cinder | 08:18 | |
| *** Masahiro has joined #openstack-cinder | 08:18 | |
| *** Masahiro has quit IRC | 08:23 | |
| *** fandi has quit IRC | 08:23 | |
| *** skraynev_ has joined #openstack-cinder | 08:28 | |
| *** skraynev_ has quit IRC | 08:28 | |
| *** nkrinner has joined #openstack-cinder | 08:32 | |
| *** tshefi has joined #openstack-cinder | 08:32 | |
| *** Miouge has quit IRC | 08:35 | |
| *** jkraj has joined #openstack-cinder | 08:39 | |
| *** afazekas has joined #openstack-cinder | 08:39 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Use dict comprehensions instead of dict constructor https://review.openstack.org/143851 | 08:40 |
| *** coolsvap|afk is now known as coolsvap | 08:42 | |
| *** jordanP has joined #openstack-cinder | 08:44 | |
| *** ebalduf has joined #openstack-cinder | 08:44 | |
| *** ebalduf has quit IRC | 08:49 | |
| *** annashen has joined #openstack-cinder | 08:55 | |
| *** annashen has quit IRC | 09:00 | |
| *** Mandell has quit IRC | 09:00 | |
| *** nlevinki has quit IRC | 09:01 | |
| *** Mandell has joined #openstack-cinder | 09:02 | |
| *** jpich has joined #openstack-cinder | 09:02 | |
| *** teju has quit IRC | 09:02 | |
| git-harry | Can some cores take a look at this devstack patch https://review.openstack.org/#/c/145858/ - a devstack core has asked for some cinder +1s | 09:04 |
| *** Mandell has quit IRC | 09:07 | |
| openstackgerrit | Trung Trinh proposed openstack/cinder: Delete volume attached to already-deleted VM https://review.openstack.org/142006 | 09:08 |
| *** nlevinki has joined #openstack-cinder | 09:17 | |
| *** aviramb has joined #openstack-cinder | 09:21 | |
| *** ndipanov has joined #openstack-cinder | 09:22 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 09:23 |
| *** karimb has joined #openstack-cinder | 09:29 | |
| *** RJ_ has quit IRC | 09:37 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: Give a meaningful error message if some library couldn't be loaded. https://review.openstack.org/146403 | 09:38 |
| *** RJ_ has joined #openstack-cinder | 09:39 | |
| flip214 | jgriffith: please look at ^^ | 09:41 |
| *** ebalduf has joined #openstack-cinder | 09:45 | |
| RJ_ | can any one look at | 09:45 |
| RJ_ | http://paste.openstack.org/show/156400/ | 09:45 |
| RJ_ | volume driver for cinder | 09:46 |
| RJ_ | i trying to add replication support ..?? | 09:46 |
| RJ_ | can any one point mistakes or suggesstions | 09:46 |
| RJ_ | http://paste.openstack.org/show/156401/ | 09:46 |
| RJ_ | these logs for errors . | 09:47 |
| *** ebalduf has quit IRC | 09:49 | |
| DuncanT | RJ_: Add some log messages to the capabilities scheduler to figure out what is going on | 09:50 |
| *** dalgaaf has joined #openstack-cinder | 09:54 | |
| *** jistr has joined #openstack-cinder | 09:58 | |
| RJ_ | DuncanT: capabilities scheduler is not there | 09:59 |
| RJ_ | in the code | 09:59 |
| DuncanT | I haven't got it in my checkout either. That log message is coming from somewhere though. Do you have a different version of the code running? | 10:00 |
| RJ_ | DuncanT: can you suggest me the file | 10:00 |
| RJ_ | I have juno lates | 10:00 |
| RJ_ | http://docs.openstack.org/juno/install-guide/install/yum/content/ch_basic_environment.html | 10:01 |
| RJ_ | this link i follwed for installation | 10:01 |
| *** nlevinki has quit IRC | 10:01 | |
| DuncanT | Juno? That's old. You want to be developing against head of trunk | 10:01 |
| RJ_ | DuncanT: can you check that | 10:01 |
| RJ_ | which one you have links of that ...?? | 10:01 |
| DuncanT | If you just install devstack, that will be trunk by default. That is what most people use for dev work | 10:02 |
| RJ_ | DuncanT: ohk | 10:02 |
| RJ_ | i have not installled that ..? | 10:02 |
| RJ_ | so ineed to install that ..?? | 10:03 |
| DuncanT | Google 'devstack howto' - there are loads of tutorials | 10:03 |
| DuncanT | It will clone the git trees of all of devstack in /opt/stack so you'll have the latest version of everything | 10:03 |
| RJ_ | https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py | 10:04 |
| RJ_ | DuncanT : these barnch also | 10:04 |
| RJ_ | not having capablities schedule | 10:04 |
| RJ_ | ..?? | 10:04 |
| *** nshaikh has left #openstack-cinder | 10:05 | |
| DuncanT | Yes, I think the capabilities scheduler got renamed, maybe moved into the main scheduler code - I really don't remember. | 10:05 |
| DuncanT | I haven't looked at the scheduler in a while | 10:05 |
| RJ_ | ohk | 10:05 |
| DuncanT | If you port your code to devstack, I'll be more able to help | 10:06 |
| *** nshaikh has joined #openstack-cinder | 10:06 | |
| RJ_ | DuncanT: ohk so need to remove all the installed part | 10:06 |
| RJ_ | of junoi | 10:06 |
| RJ_ | DuncanT: is it possible to merge juno with devstack | 10:07 |
| *** Masahiro has joined #openstack-cinder | 10:07 | |
| DuncanT | That is probably easiest to remove juno. Many people run devstack in a VM so that they can blow it away and start again easily, or have several different streams of development | 10:07 |
| RJ_ | DuncanT: then it is going to work with storage nodes ..? | 10:09 |
| RJ_ | comupute nodes ../ | 10:09 |
| RJ_ | DuncanT: or need to again install the all storage and compute nodes ..? | 10:10 |
| DuncanT | For test purposes, you can run compute and storage on the same node. Google 'multinode devstack' for getting it working with the two services on separate nodes | 10:10 |
| *** Masahiro has quit IRC | 10:11 | |
| RJ_ | what devstack having extra part ..?? | 10:13 |
| RJ_ | in that ../ | 10:13 |
| RJ_ | from juno ../ | 10:13 |
| DuncanT | Forget juno, juno is old. You should develop new patches against trunk | 10:17 |
| *** deepakcs has joined #openstack-cinder | 10:18 | |
| RJ_ | DuncanT: you have any links for follow that ..? | 10:20 |
| RJ_ | DuncanT: devstack .. | 10:21 |
| openstackgerrit | Bob proposed openstack/cinder: Add loopingcalls for Huawei storage system driver https://review.openstack.org/143765 | 10:22 |
| DuncanT | RJ_: http://docs.openstack.org/developer/devstack/guides/multinode-lab.html (top hit on a google search for "devstack multinode howto") seems like the logical place to start | 10:26 |
| *** e0ne has joined #openstack-cinder | 10:27 | |
| RJ_ | DuncanT: ohk | 10:30 |
| RJ_ | but avishay talking me that | 10:30 |
| RJ_ | DuncanT: this errors may be due to data[replication]= true | 10:31 |
| openstackgerrit | ustcdylan proposed openstack/cinder: Fix bug in sheepdog driver: the wrong volume size https://review.openstack.org/139804 | 10:31 |
| RJ_ | is not set ../ | 10:31 |
| *** marcusvrn has joined #openstack-cinder | 10:32 | |
| DuncanT | RJ_: I have no idea, which is why I suggested adding extra LOG lines to be sure. Given that you're using old code, you won't be able to submit your work for merging anyway - you need to rebase on top of trunk and test it on that code base | 10:33 |
| RJ_ | ohk .. | 10:33 |
| RJ_ | you seen the code of driver ..? | 10:34 |
| RJ_ | DuncanT: | 10:34 |
| DuncanT | I don't understand the question | 10:34 |
| RJ_ | http://paste.openstack.org/show/156400/ | 10:34 |
| RJ_ | DuncanT: what you feel about this code ..? | 10:35 |
| DuncanT | It doesn't look terrible, and when it is working and tested I'm happy to review it | 10:35 |
| *** BharatK has quit IRC | 10:36 | |
| DuncanT | Until it is rebased on trunk though, I've really very little interest in it | 10:36 |
| RJ_ | ohk | 10:37 |
| RJ_ | DuncanT: me too will be happy | 10:38 |
| RJ_ | DuncanT: buut not getting the exact knwldg regarding the replication | 10:38 |
| RJ_ | therse is no doc related for replication | 10:38 |
| RJ_ | DuncanT: what is pre required changes etc | 10:39 |
| DuncanT | Get your driver working on devstack. Add log lines to the scheduler until you understand what is going on. Write some docs on what you found. Currently we only have example code for replication, not good docs | 10:39 |
| RJ_ | DuncanT: what changes in cinder.conf or how to and second node etc ...?? | 10:40 |
| DuncanT | RJ_: Nobody has written those docs that I'm aware of. I'm not aware of anybody specifically working on those docs. You are very welcome to figure out how to do it and document it. As I've said, repeatedly, your first step should probably to get devstack set up rather than trying to base your work on juno, since the replication work was barely started by the | 10:41 |
| DuncanT | end of juno. | 10:41 |
| RJ_ | ohk | 10:45 |
| *** ebalduf has joined #openstack-cinder | 10:46 | |
| *** nlevinki has joined #openstack-cinder | 10:47 | |
| *** BharatK has joined #openstack-cinder | 10:49 | |
| *** nellysmitt has joined #openstack-cinder | 10:49 | |
| *** ebalduf has quit IRC | 10:50 | |
| flip214 | DuncanT: thanks a lot for the quick feedback, will redo. | 10:56 |
| *** annashen has joined #openstack-cinder | 10:56 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 10:58 |
| *** pcaruana has joined #openstack-cinder | 10:59 | |
| *** annashen has quit IRC | 11:01 | |
| *** pradipta has quit IRC | 11:05 | |
| RJ_ | DuncanT: hey one more | 11:08 |
| *** pradipta has joined #openstack-cinder | 11:08 | |
| *** Masahiro has joined #openstack-cinder | 11:08 | |
| RJ_ | DuncanT: replication relation table required in the db for replication suppport ..?? | 11:08 |
| DuncanT | I believe that is used by the manager to handle promote etc, so yes. I haven't looked at the code in a while though | 11:09 |
| DuncanT | It should all be taken care of for you, the driver shouldn't need to touch it | 11:09 |
| *** nshaikh has quit IRC | 11:12 | |
| *** Masahiro has quit IRC | 11:13 | |
| *** pradipta is now known as pradipta_away | 11:21 | |
| *** haomaiwang has quit IRC | 11:22 | |
| *** triggerz_ has quit IRC | 11:28 | |
| *** david-lyle has joined #openstack-cinder | 11:32 | |
| *** chlong has joined #openstack-cinder | 11:41 | |
| *** nshaikh has joined #openstack-cinder | 11:41 | |
| *** Miouge has joined #openstack-cinder | 11:43 | |
| *** chlong has quit IRC | 11:47 | |
| *** chlong has joined #openstack-cinder | 11:48 | |
| *** Miouge has quit IRC | 11:51 | |
| *** e0ne is now known as e0ne_ | 11:56 | |
| *** Miouge has joined #openstack-cinder | 11:58 | |
| *** lpabon has joined #openstack-cinder | 12:02 | |
| *** e0ne_ is now known as e0ne | 12:04 | |
| *** EmilienM|afk is now known as EmilienM | 12:10 | |
| erlon | DuncanT: hi Duncan, who is the guys/team responsible for backporting changes to stable/master? | 12:15 |
| *** rakesh___510 has quit IRC | 12:19 | |
| *** rakesh___510 has joined #openstack-cinder | 12:19 | |
| nikesh_vedams | hi | 12:24 |
| nikesh_vedams | we are trying to use sos_ci | 12:25 |
| nikesh_vedams | CI setup in cinder | 12:25 |
| nikesh_vedams | we delpoyed a openstack kilo cloud for this | 12:25 |
| nikesh_vedams | we updated sos-ci/sos-ci/ansible/vars.yml | 12:26 |
| nikesh_vedams | with our cloud credentials | 12:26 |
| nikesh_vedams | now what to run | 12:26 |
| nikesh_vedams | not getting | 12:26 |
| nikesh_vedams | can any one help please | 12:26 |
| *** rakesh___510 has quit IRC | 12:27 | |
| DuncanT | erlon: Anybody can submit backport reviews | 12:28 |
| *** sgotliv has joined #openstack-cinder | 12:30 | |
| *** jkraj has quit IRC | 12:31 | |
| DuncanT | nikesh_vedams: Not claiming any expertise, but "python os_ci.py" by the looks of it. You might want to look at testit.py first, it looks like it does a one-shot test of your config | 12:31 |
| *** IanGovett has joined #openstack-cinder | 12:33 | |
| *** diegows has joined #openstack-cinder | 12:39 | |
| *** alexpilotti has joined #openstack-cinder | 12:39 | |
| *** Longgeek has quit IRC | 12:39 | |
| erlon | DuncanT: my question is, who are the people that approve what changes will or won't be backported | 12:40 |
| DuncanT | Ah, jungleboyj, jgriffith and thingee are the people I know are active on that - there are probably others, check who voted on the last few stable reviews to be sure | 12:41 |
| *** alexpilotti has quit IRC | 12:44 | |
| *** ebalduf has joined #openstack-cinder | 12:48 | |
| nikesh_vedams | DuncanT:have you used sos-ci ? | 12:49 |
| nikesh_vedams | DuncanT:whatever readme suggested we provided but still i doubt some things missing | 12:50 |
| *** ebalduf has quit IRC | 12:52 | |
| *** RJ_ has quit IRC | 12:53 | |
| *** deepakcs has quit IRC | 12:54 | |
| *** alexpilotti has joined #openstack-cinder | 12:55 | |
| *** chlong has quit IRC | 12:55 | |
| *** rushiagr is now known as rushiagr_away | 12:56 | |
| *** Masahiro has joined #openstack-cinder | 12:57 | |
| *** annashen has joined #openstack-cinder | 12:57 | |
| DuncanT | nikesh_vedams: I've not used John's version, no. I've a hacked together (unpublished) thing of my own. I'm sure John would accept patches to improve the docs of sos-ci | 12:57 |
| *** Miouge has quit IRC | 12:58 | |
| nikesh_vedams | DuncanT:can you suggest how to easily setup CI? | 13:00 |
| nikesh_vedams | DuncanT: we have a patch "https://review.openstack.org/#/c/145778/" | 13:01 |
| nikesh_vedams | DuncanT: we want to set up a CI for it | 13:01 |
| *** annashen has quit IRC | 13:01 | |
| *** Masahiro has quit IRC | 13:01 | |
| DuncanT | nikesh_vedams: Most people see to have gone the jenkins/nodepool route, which is supported by #openstack-infra | 13:02 |
| *** Longgeek has joined #openstack-cinder | 13:02 | |
| *** Mandell has joined #openstack-cinder | 13:02 | |
| DuncanT | nikesh_vedams: sos-ci and similar are a lighter weight approach but assume a certain degree of understanding of the problem space currently | 13:02 |
| *** Miouge has joined #openstack-cinder | 13:02 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Update README.rst to current state https://review.openstack.org/145519 | 13:03 |
| DuncanT | nikesh_vedams: If you want to go the sos-ci route then I suggest you'll need to go through the code for it (there isn't much) and test the bits | 13:03 |
| DuncanT | nikesh_vedams: If you want to go the officially recommended route (jenkins) then #openstack-infra have people with more experience of helping | 13:04 |
| DuncanT | nikesh_vedams: There's a series of blog posts by Jaya Pipes that might be a good starting point to the jenkins approach | 13:05 |
| *** jinhui has joined #openstack-cinder | 13:06 | |
| *** Mandell has quit IRC | 13:06 | |
| *** e0ne is now known as e0ne_ | 13:11 | |
| nikesh_vedams | DuncanT:thanks | 13:12 |
| nikesh_vedams | why no one replies in #openstack-infra :( | 13:12 |
| *** bswartz has quit IRC | 13:14 | |
| *** ryanpetrello has joined #openstack-cinder | 13:17 | |
| *** sandywalsh has joined #openstack-cinder | 13:23 | |
| *** jkraj has joined #openstack-cinder | 13:25 | |
| *** rushiagr_away is now known as rushiagr | 13:33 | |
| *** kalango has joined #openstack-cinder | 13:35 | |
| DuncanT | nikesh_vedams: Timezones, I expect. Be patient | 13:45 |
| nikesh_vedams | :) | 13:45 |
| *** e0ne_ is now known as e0ne | 13:45 | |
| *** ebalduf has joined #openstack-cinder | 13:48 | |
| *** rakesh___510 has joined #openstack-cinder | 13:49 | |
| *** ebalduf has quit IRC | 13:53 | |
| *** alonmarx_ has joined #openstack-cinder | 13:53 | |
| *** alonmarx has quit IRC | 13:56 | |
| *** bswartz has joined #openstack-cinder | 13:57 | |
| *** akerr has joined #openstack-cinder | 13:57 | |
| *** sgotliv has quit IRC | 13:57 | |
| *** Miouge has quit IRC | 14:01 | |
| *** Miouge has joined #openstack-cinder | 14:02 | |
| *** e0ne is now known as e0ne_ | 14:03 | |
| *** rushil has joined #openstack-cinder | 14:04 | |
| *** kalango has left #openstack-cinder | 14:05 | |
| *** rakesh___510_ has joined #openstack-cinder | 14:06 | |
| *** rakesh___510 has quit IRC | 14:09 | |
| *** Yogi1 has joined #openstack-cinder | 14:09 | |
| *** dustins has joined #openstack-cinder | 14:10 | |
| *** e0ne_ is now known as e0ne | 14:11 | |
| *** sgotliv has joined #openstack-cinder | 14:13 | |
| *** jinhui has quit IRC | 14:13 | |
| *** julim has joined #openstack-cinder | 14:15 | |
| *** jcru has joined #openstack-cinder | 14:17 | |
| *** kaufer has joined #openstack-cinder | 14:18 | |
| *** rakesh___510_ has quit IRC | 14:21 | |
| *** mwichmann has joined #openstack-cinder | 14:22 | |
| *** anshul has quit IRC | 14:23 | |
| *** Miouge has quit IRC | 14:27 | |
| *** BharatK has quit IRC | 14:30 | |
| *** Miouge has joined #openstack-cinder | 14:30 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 14:32 |
| *** eharney has joined #openstack-cinder | 14:32 | |
| *** mriedem has joined #openstack-cinder | 14:36 | |
| *** anshul has joined #openstack-cinder | 14:36 | |
| *** Longgeek has quit IRC | 14:37 | |
| *** nshaikh has left #openstack-cinder | 14:37 | |
| *** e0ne is now known as e0ne_ | 14:39 | |
| *** tbarron has joined #openstack-cinder | 14:40 | |
| *** e0ne_ is now known as e0ne | 14:42 | |
| *** Longgeek has joined #openstack-cinder | 14:42 | |
| *** jwang__ has joined #openstack-cinder | 14:42 | |
| *** jwang_ has quit IRC | 14:43 | |
| *** Masahiro has joined #openstack-cinder | 14:45 | |
| *** EduardBiceriMate has joined #openstack-cinder | 14:48 | |
| *** jecarey has quit IRC | 14:49 | |
| *** ebalduf has joined #openstack-cinder | 14:49 | |
| *** Masahiro has quit IRC | 14:50 | |
| *** jkraj has quit IRC | 14:51 | |
| *** ebalduf has quit IRC | 14:54 | |
| *** thangp has joined #openstack-cinder | 14:55 | |
| *** annashen has joined #openstack-cinder | 14:58 | |
| *** ndipanov has quit IRC | 14:58 | |
| *** fifieldt__ has joined #openstack-cinder | 15:00 | |
| *** tbarron has quit IRC | 15:00 | |
| *** asselin_ has joined #openstack-cinder | 15:01 | |
| *** Yogi11 has joined #openstack-cinder | 15:01 | |
| *** annashen has quit IRC | 15:02 | |
| *** ndipanov has joined #openstack-cinder | 15:02 | |
| *** fifieldt_ has quit IRC | 15:02 | |
| *** nlevinki has quit IRC | 15:02 | |
| *** tbarron has joined #openstack-cinder | 15:03 | |
| *** dustins_ has joined #openstack-cinder | 15:03 | |
| *** julim_ has joined #openstack-cinder | 15:04 | |
| *** dustins has quit IRC | 15:04 | |
| *** Yogi1 has quit IRC | 15:04 | |
| *** jungleboyj has joined #openstack-cinder | 15:04 | |
| *** julim has quit IRC | 15:05 | |
| *** lcurtis has joined #openstack-cinder | 15:05 | |
| git-harry | Can some cores take a look at this devstack patch https://review.openstack.org/#/c/145858/Â - a devstack core has asked for some cinder +1s | 15:06 |
| *** mtanino has joined #openstack-cinder | 15:07 | |
| *** juzuluag has joined #openstack-cinder | 15:10 | |
| *** EduardBiceriMate has quit IRC | 15:12 | |
| *** superdan is now known as dansmith | 15:15 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Remove the useless next link for volumes, transfers and backups https://review.openstack.org/131915 | 15:15 |
| *** fischerw has joined #openstack-cinder | 15:15 | |
| *** beekneemech is now known as bnemec | 15:16 | |
| jgriffith | git-harry: thanks for putting that together | 15:16 |
| jgriffith | git-harry: I'd like to make sure thingee looks at it when he gets in | 15:17 |
| *** jecarey has joined #openstack-cinder | 15:17 | |
| *** nlevinki has joined #openstack-cinder | 15:17 | |
| git-harry | jgriffith: okay, sure. Thanks for looking. | 15:17 |
| *** ebalduf has joined #openstack-cinder | 15:19 | |
| *** Yogi11 has quit IRC | 15:21 | |
| *** rmesta has joined #openstack-cinder | 15:24 | |
| *** dustins_ has quit IRC | 15:26 | |
| nikesh_vedams | jgriffith:we are trying to create a CI with sos-ci,we deployed kilo devstack for sos-ci and we changed sos-ci/sos-ci/ansible/vars.yml | 15:26 |
| nikesh_vedams | jgriffith: we configured CI account too | 15:26 |
| jgriffith | nikesh_vedams: cool | 15:27 |
| nikesh_vedams | jgriffith:but seems like many hard-coding is in sos-ci with solidfire and local directories | 15:27 |
| nikesh_vedams | so need some help | 15:28 |
| jgriffith | nikesh_vedams: just the local.conf.template file | 15:28 |
| jgriffith | nikesh_vedams: and then variables in your ansible dir | 15:28 |
| jgriffith | nikesh_vedams: as far as directories, yes... I think you'll need to work that out | 15:29 |
| jgriffith | nikesh_vedams: thought I converted it to use relative paths but probably not | 15:29 |
| *** e0ne is now known as e0ne_ | 15:30 | |
| *** ndipanov has quit IRC | 15:31 | |
| *** kaisers has quit IRC | 15:31 | |
| nikesh_vedams | do we have to run os_ci.py to start the CI setup and also this script is taking CI environment variables | 15:31 |
| nikesh_vedams | jgriffith: do we have to run os_ci.py to start the CI setup and also this script is taking CI environment variables | 15:32 |
| nikesh_vedams | jgriffith: so i hope we have to set it explicitly right | 15:32 |
| jgriffith | nikesh_vedams: yes | 15:33 |
| *** tr4sh_heap has joined #openstack-cinder | 15:33 | |
| *** nlevinki has quit IRC | 15:35 | |
| nikesh_vedams | jgriffith: CINDER_BRANCH=master in local.conf.base but we want devstack setup with our scst changes so shall we give CINDER_BRANCH=refs/changes/78/145778/1 | 15:40 |
| *** e0ne_ has quit IRC | 15:41 | |
| *** ndipanov has joined #openstack-cinder | 15:44 | |
| *** aarefiev_ has joined #openstack-cinder | 15:46 | |
| *** Miouge has quit IRC | 15:47 | |
| *** kaisers has joined #openstack-cinder | 15:48 | |
| *** e0ne has joined #openstack-cinder | 15:51 | |
| *** EduardBiceriMate has joined #openstack-cinder | 15:54 | |
| *** tbarron has quit IRC | 15:55 | |
| *** ebalduf has quit IRC | 15:56 | |
| *** e0ne_ has joined #openstack-cinder | 15:59 | |
| *** tshefi has quit IRC | 15:59 | |
| *** tbarron has joined #openstack-cinder | 16:00 | |
| e0ne_ | hi cinder-devs! | 16:00 |
| *** ndipanov has quit IRC | 16:00 | |
| e0ne_ | i'm looking on cinderclient.. it's still uses v1 as default. should we change it to v2 or it v1 will be dropped in Kilo? | 16:01 |
| *** e0ne has quit IRC | 16:02 | |
| *** tbarron has quit IRC | 16:02 | |
| *** aarefiev_ has quit IRC | 16:03 | |
| *** aarefiev has quit IRC | 16:03 | |
| *** ndipanov has joined #openstack-cinder | 16:03 | |
| *** kaisers has quit IRC | 16:04 | |
| *** ebalduf has joined #openstack-cinder | 16:05 | |
| *** kaisers has joined #openstack-cinder | 16:05 | |
| *** Miouge has joined #openstack-cinder | 16:05 | |
| *** lcurtis has quit IRC | 16:05 | |
| *** rhagarty has quit IRC | 16:06 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Set default OS_VOLUME_API_VERSION to '2' https://review.openstack.org/146541 | 16:06 |
| *** rhagarty has joined #openstack-cinder | 16:06 | |
| *** dustins has joined #openstack-cinder | 16:07 | |
| *** MIDENN_ has joined #openstack-cinder | 16:08 | |
| *** ebalduf has quit IRC | 16:09 | |
| *** dustins has quit IRC | 16:09 | |
| *** tbarron has joined #openstack-cinder | 16:10 | |
| *** MIDENN_ has quit IRC | 16:10 | |
| *** hemnafk is now known as hemna | 16:11 | |
| hemna | mornin | 16:11 |
| *** dustins has joined #openstack-cinder | 16:12 | |
| e0ne_ | hi | 16:12 |
| *** e0ne_ is now known as e0ne | 16:12 | |
| *** tbarron1 has joined #openstack-cinder | 16:12 | |
| *** tbarron has quit IRC | 16:13 | |
| *** dustins has quit IRC | 16:13 | |
| *** dustins has joined #openstack-cinder | 16:14 | |
| *** asselin_ has quit IRC | 16:14 | |
| *** Yogi1 has joined #openstack-cinder | 16:17 | |
| *** tsekiyama has joined #openstack-cinder | 16:21 | |
| *** coolsvap is now known as coolsvap|afk | 16:25 | |
| *** TobiasE has joined #openstack-cinder | 16:26 | |
| *** ebalduf has joined #openstack-cinder | 16:26 | |
| *** Longgeek_ has joined #openstack-cinder | 16:29 | |
| *** rushiagr is now known as rushiagr_away | 16:29 | |
| *** Longgeek has quit IRC | 16:30 | |
| *** nkrinner has quit IRC | 16:30 | |
| *** jdurgin has joined #openstack-cinder | 16:30 | |
| smcginnis | jungleboyj: Morning Jay! What's the oslo patch related to dropping _lazy()? Your comment actually referred to the patch you said you would drop. | 16:31 |
| *** nkrinner has joined #openstack-cinder | 16:31 | |
| jungleboyj | Oye vey. | 16:31 |
| smcginnis | jungleboyj: Oh, nevermind. I found it. | 16:32 |
| jungleboyj | Just udpated. Sorry about that. | 16:33 |
| e0ne | jungleboyj: hi! could you please take a look on https://review.openstack.org/#/c/141414/ again? i'm be happy to get all migration-related tests merged:) | 16:33 |
| jungleboyj | e0ne: Sure. Have a coupel things goin on. | 16:34 |
| e0ne | jungleboyj: thanks, Jay | 16:34 |
| *** Masahiro has joined #openstack-cinder | 16:34 | |
| smcginnis | jungleboyj: Shoot. Requirement conflict with openstack/requirements. :[ | 16:35 |
| hemna | #openstack-infra | 16:35 |
| hemna | gah | 16:36 |
| jungleboyj | Argh! | 16:36 |
| smcginnis | jungleboyj: Welcome to Monday morning. :) | 16:36 |
| *** e0ne_ has joined #openstack-cinder | 16:37 | |
| *** Masahiro has quit IRC | 16:39 | |
| hemna | heh yah | 16:39 |
| hemna | can I buy a /j please ? | 16:39 |
| *** e0ne has quit IRC | 16:40 | |
| *** lcurtis has joined #openstack-cinder | 16:40 | |
| smcginnis | hemna: Depends what state you live in. (JK) | 16:41 |
| hemna | hehe | 16:41 |
| *** dhellmann has quit IRC | 16:42 | |
| *** dhellmann has joined #openstack-cinder | 16:42 | |
| *** coolsvap|afk has quit IRC | 16:42 | |
| *** akerr is now known as akerr_away | 16:44 | |
| *** ndipanov has quit IRC | 16:44 | |
| *** lpabon has quit IRC | 16:45 | |
| *** coolsvap has joined #openstack-cinder | 16:47 | |
| *** btran has quit IRC | 16:48 | |
| *** btran has joined #openstack-cinder | 16:49 | |
| *** ebalduf has quit IRC | 16:49 | |
| jungleboyj | smcginnis: Where did you see that? | 16:52 |
| *** r-daneel has joined #openstack-cinder | 16:52 | |
| jungleboyj | smcginnis: Ok, I see it now. | 16:53 |
| *** _cjones_ has joined #openstack-cinder | 16:53 | |
| jungleboyj | Oh crap, so I would have to make the same change to openstack/requirements for stable/juno. I am guessing I can't do that. | 16:53 |
| smcginnis | Probably more difficult. | 16:54 |
| smcginnis | Might be easier sticking with your first approach. | 16:54 |
| *** dannywilson has joined #openstack-cinder | 16:54 | |
| * jungleboyj sighs. | 16:54 | |
| *** ebalduf has joined #openstack-cinder | 16:55 | |
| *** ndipanov has joined #openstack-cinder | 16:57 | |
| *** lcurtis has quit IRC | 16:58 | |
| *** vozcelik has joined #openstack-cinder | 16:58 | |
| *** sgotliv has quit IRC | 16:58 | |
| jungleboyj | smcginnis: I am getting the expert over here. | 17:01 |
| *** mikedillion has joined #openstack-cinder | 17:02 | |
| *** Miouge has quit IRC | 17:02 | |
| *** jdurgin has quit IRC | 17:04 | |
| *** pwehrle has left #openstack-cinder | 17:05 | |
| *** dannywilson has quit IRC | 17:06 | |
| *** mikedillion has quit IRC | 17:06 | |
| *** dannywilson has joined #openstack-cinder | 17:06 | |
| *** rwsu has joined #openstack-cinder | 17:09 | |
| *** rwsu has quit IRC | 17:09 | |
| *** MIDENN_ has joined #openstack-cinder | 17:09 | |
| *** akerr_away is now known as akerr | 17:10 | |
| *** vozcelik has left #openstack-cinder | 17:10 | |
| *** MIDENN_ has quit IRC | 17:10 | |
| nikesh_vedams | jgriffith: there? | 17:14 |
| *** tellesnobrega has quit IRC | 17:15 | |
| *** lcurtis has joined #openstack-cinder | 17:18 | |
| *** Yogi1 has quit IRC | 17:19 | |
| *** emagana has joined #openstack-cinder | 17:21 | |
| *** jpich has quit IRC | 17:25 | |
| *** Yogi1 has joined #openstack-cinder | 17:25 | |
| *** pcaruana is now known as pcaruana|off| | 17:27 | |
| *** nkrinner has quit IRC | 17:27 | |
| *** rwsu has joined #openstack-cinder | 17:27 | |
| nikesh_vedams | jgriifith:hi we are getting below output after running os_ci.py http://paste.openstack.org/show/156498/ | 17:28 |
| *** afazekas has quit IRC | 17:35 | |
| *** e0ne_ has quit IRC | 17:36 | |
| *** dannywilson has quit IRC | 17:38 | |
| *** jecarey has quit IRC | 17:38 | |
| *** aarefiev has joined #openstack-cinder | 17:40 | |
| *** Longgeek_ has quit IRC | 17:42 | |
| *** juzuluag has quit IRC | 17:48 | |
| *** juzuluag has joined #openstack-cinder | 17:48 | |
| *** alonmarx_ has quit IRC | 17:49 | |
| *** alonmarx has joined #openstack-cinder | 17:50 | |
| *** patrickeast has joined #openstack-cinder | 17:54 | |
| *** xmltok has left #openstack-cinder | 17:59 | |
| *** jordanP has quit IRC | 17:59 | |
| *** tellesnobrega has joined #openstack-cinder | 18:00 | |
| *** sgotliv has joined #openstack-cinder | 18:01 | |
| *** leeantho has joined #openstack-cinder | 18:03 | |
| *** jistr has quit IRC | 18:03 | |
| jgriffith | nikesh_vedams: for starters you might want to install ansible :) | 18:04 |
| *** leeantho_ has joined #openstack-cinder | 18:04 | |
| jgriffith | nikesh_vedams: hard to run ansible playbooks without it | 18:04 |
| *** ebalduf has quit IRC | 18:05 | |
| jgriffith | nikesh_vedams: not sure what's up with the paramiko trace you're seeing there | 18:05 |
| nikesh_vedams | jgriffith:actuallly we installed ansible earlier with apt-get,so things went under /usr/bin | 18:06 |
| nikesh_vedams | jgriffith:after apt-get purge ansible and installing ansible and other things with pip,that error gone | 18:07 |
| jgriffith | yeah, pip is better | 18:07 |
| jgriffith | nikesh_vedams: pip and apt install in different locations IIRC | 18:07 |
| *** leeantho has quit IRC | 18:08 | |
| *** bswartz has quit IRC | 18:08 | |
| *** crose has joined #openstack-cinder | 18:08 | |
| nikesh_vedams | jgriffith: yes apt install in /usr/bin and pip install in /usr/local/bin | 18:09 |
| nikesh_vedams | that why error | 18:09 |
| nikesh_vedams | now we are continuosly getting "loop...debug" | 18:09 |
| nikesh_vedams | how can we know it is working or not | 18:10 |
| *** bswartz has joined #openstack-cinder | 18:12 | |
| *** annashen has joined #openstack-cinder | 18:15 | |
| *** Mandell has joined #openstack-cinder | 18:22 | |
| *** dannywilson has joined #openstack-cinder | 18:23 | |
| *** dannywilson has quit IRC | 18:23 | |
| *** Masahiro has joined #openstack-cinder | 18:23 | |
| *** dannywilson has joined #openstack-cinder | 18:23 | |
| jgriffith | nikesh_vedams: you should read the code :) | 18:25 |
| jgriffith | nikesh_vedams: loop...debug mean it's listening waiting for commits | 18:26 |
| jgriffith | nikesh_vedams: when commit comes in and gets processed you'll see it logged | 18:26 |
| nikesh_vedams | oh | 18:26 |
| nikesh_vedams | one more thing, CINDER_BRANCH=master in local.conf.base but we want devstack setup with our scst changes so we are giving CINDER_BRANCH=refs/changes/78/145778/1 | 18:28 |
| *** Masahiro has quit IRC | 18:28 | |
| nikesh_vedams | is it ok | 18:28 |
| jgriffith | the ansible scripts should set that for you based on the commit that is picked up by the listener | 18:29 |
| nikesh_vedams | ok so master is enough | 18:29 |
| nikesh_vedams | right | 18:29 |
| nikesh_vedams | and one more thing we have to install scst packages,so we have to update " sos-ci/sos-ci/ansible/tasks/install_devstack.yml" right | 18:30 |
| jgriffith | nikesh_vedams: yes, or better yet write a new ansible task and insert it | 18:31 |
| *** jdurgin has joined #openstack-cinder | 18:34 | |
| *** Mandell has quit IRC | 18:38 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-cinderclient: WIP Add volume multi attach support https://review.openstack.org/85856 | 18:39 |
| *** harlowja_away is now known as harlowja | 18:39 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support https://review.openstack.org/85856 | 18:41 |
| *** Mandell has joined #openstack-cinder | 18:42 | |
| *** anshul has quit IRC | 18:50 | |
| *** jecarey has joined #openstack-cinder | 18:51 | |
| *** dannywilson has quit IRC | 18:52 | |
| *** mudassirlatif has joined #openstack-cinder | 18:53 | |
| *** e0ne has joined #openstack-cinder | 18:54 | |
| *** harlowja has quit IRC | 19:00 | |
| *** harlowja has joined #openstack-cinder | 19:00 | |
| *** harlowja has quit IRC | 19:00 | |
| nikesh_vedams | jgriffith:i am new to yaml,do you any good link to understand yaml syntax | 19:00 |
| jgriffith | nikesh_vedams: nope, but that syntax is ansible yaml so you should consult the ansible docs | 19:02 |
| *** crose has quit IRC | 19:06 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Implement clone_image caching on SolidFire https://review.openstack.org/142859 | 19:09 |
| *** e0ne is now known as e0ne_ | 19:09 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Remove locks from LeftHand driver https://review.openstack.org/141475 | 19:11 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Snapshot object https://review.openstack.org/133566 | 19:12 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Cinder objects base https://review.openstack.org/131873 | 19:12 |
| *** corvus is now known as jeblair | 19:13 | |
| *** dannywilson has joined #openstack-cinder | 19:13 | |
| *** alecv has joined #openstack-cinder | 19:14 | |
| *** e0ne_ has quit IRC | 19:15 | |
| *** dalgaaf has quit IRC | 19:17 | |
| *** karimb has quit IRC | 19:19 | |
| *** e0ne has joined #openstack-cinder | 19:20 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add support to differential backups in cinder https://review.openstack.org/110068 | 19:21 |
| *** marcusvrn has quit IRC | 19:21 | |
| *** Mandell has quit IRC | 19:25 | |
| *** anshul has joined #openstack-cinder | 19:26 | |
| *** anshul has quit IRC | 19:26 | |
| *** Mandell has joined #openstack-cinder | 19:31 | |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Proof of concept for reset-state-with-driver https://review.openstack.org/140502 | 19:34 |
| jgriffith | DuncanT: around? | 19:35 |
| *** sgotliv has quit IRC | 19:37 | |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Proof of concept for reset-state-with-driver https://review.openstack.org/140502 | 19:38 |
| *** ebalduf has joined #openstack-cinder | 19:40 | |
| *** EduardBiceriMate has quit IRC | 19:42 | |
| *** _cjones_ has quit IRC | 19:44 | |
| *** _cjones_ has joined #openstack-cinder | 19:44 | |
| anish | jgriffith: hemna can I get some feedback on this WIP code ? https://review.openstack.org/#/c/146233/ | 19:47 |
| hemna | anish, sure | 19:47 |
| anish | sweet | 19:48 |
| anish | specifically, it also happens to implement code that was supposed to be done by someone else's bp | 19:49 |
| anish | so I'm not sure how to proceed with that | 19:49 |
| *** jnrao has joined #openstack-cinder | 19:50 | |
| *** harlowja has joined #openstack-cinder | 19:52 | |
| *** thingee has joined #openstack-cinder | 19:54 | |
| jgriffith | harlowja: ping :) | 19:55 |
| harlowja | jgriffith pong | 19:56 |
| jgriffith | harlowja: hey ya... got a second? | 19:56 |
| jgriffith | harlowja: or a few hundred :) | 19:56 |
| harlowja | sureee | 19:56 |
| harlowja | sup | 19:56 |
| jgriffith | harlowja: ok, so the concept I was looking at of putting that update in the run block won't work | 19:57 |
| harlowja | what about a new task then? | 19:57 |
| jgriffith | harlowja: becuase the flow actually calls the volume/manager.py create method each time / retry | 19:57 |
| harlowja | shove it at the end of the workflow? | 19:57 |
| jgriffith | harlowja: yeah, that's what I think has to be done | 19:57 |
| thingee | git-harry, jgriffith: responded to https://review.openstack.org/#/c/145858/ | 19:57 |
| jgriffith | harlowja: but I'm trying to better understand how this works | 19:57 |
| harlowja | k, which part confusing u | 19:58 |
| harlowja | ? | 19:58 |
| jgriffith | harlowja: so what I've gathered is.... | 19:58 |
| jgriffith | harlowja: once the command goes to the cinder.scheduler; | 19:58 |
| jgriffith | harlowja: the retrys are actually handled by the taskflow lib | 19:58 |
| jgriffith | harlowja: and sent *directly* to the host manager | 19:58 |
| jgriffith | harlowja: ie they don't go back through the cinder filter scheduler as new rpc calls? | 19:58 |
| harlowja | afaik they should be | 19:59 |
| jgriffith | but I'm uncertain... | 19:59 |
| harlowja | afaik u guys aren't using the taskflow retry stuffs | 19:59 |
| jgriffith | harlowja: we sure are | 19:59 |
| harlowja | well its sorta via that damn external loop that exists from nova days | 19:59 |
| jgriffith | harlowja: Ok, so tell me if this sounds right.... | 19:59 |
| jgriffith | haha | 20:00 |
| jgriffith | stupid nova days | 20:00 |
| *** e0ne has quit IRC | 20:00 | |
| harlowja | ;) | 20:00 |
| *** akerr has quit IRC | 20:00 | |
| harlowja | we are all young once | 20:00 |
| harlowja | didn't mean to inhale... | 20:00 |
| harlowja | *except its ok in colorado | 20:00 |
| harlowja | lol | 20:00 |
| jgriffith | LOL | 20:00 |
| *** akerr has joined #openstack-cinder | 20:00 | |
| jgriffith | We're even more liberal that CA now LOL | 20:00 |
| jgriffith | Ok... so the call to create.... | 20:00 |
| harlowja | ;) | 20:00 |
| jgriffith | goes from cinder.volume.api --> cinder.volume.flows.api.create_volume | 20:01 |
| anish | cinder sort means burnt wood yes ? a specific type of tree even. very apt | 20:01 |
| *** david-lyle has quit IRC | 20:02 | |
| jgriffith | which hits up cinder.volume.flows.manager.create_volume | 20:02 |
| harlowja | k, so right, thats the api stuffs, ya | 20:02 |
| jgriffith | and issues the call out to cinder.scheduler.**** | 20:02 |
| harlowja | right | 20:02 |
| *** juzuluag has quit IRC | 20:02 | |
| jgriffith | and there it bounces around between filter rpc module, taskflow module that's inside the scheduler etc | 20:02 |
| harlowja | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L766 (all that stuffs) | 20:02 |
| *** e0ne has joined #openstack-cinder | 20:03 | |
| harlowja | ok, ya, so that bouncing is sorta not in taskflow, but external (from the nova retry days) | 20:03 |
| harlowja | that loop is not so easy to know about, ha | 20:03 |
| jgriffith | harlowja: hmmm | 20:03 |
| jgriffith | harlowja: why the hell do we need that if we have the retry loop in tflow? | 20:03 |
| thingee | e0ne: with regards to your question about cinderclient defaulting to v2. have you seen https://review.openstack.org/#/c/145858/ | 20:03 |
| jgriffith | :) | 20:03 |
| thingee | e0ne: whoops | 20:03 |
| harlowja | jgriffith young days | 20:03 |
| harlowja | cinder inhaled.... | 20:03 |
| thingee | e0ne: this one https://review.openstack.org/#/c/145613/ | 20:03 |
| jgriffith | harlowja: haha | 20:03 |
| harlowja | jgriffith so that gets bounced via https://github.com/openstack/cinder/blob/master/cinder/scheduler/filter_scheduler.py#L223 | 20:04 |
| jgriffith | harlowja: so the problem with just adding a new task... | 20:04 |
| jgriffith | harlowja: I actually need to break the update piece out of that method in the volume manager | 20:04 |
| jgriffith | make it its own independent method | 20:04 |
| jgriffith | and call it on completion | 20:05 |
| e0ne | thingee: thanks. i've seen your request | 20:05 |
| jgriffith | that's not a terribly big deal I suppose | 20:05 |
| harlowja | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L88 (is where this gets sent back to the scheduler; and repeat...) | 20:05 |
| jgriffith | Oh... yay!!! that helps :) | 20:05 |
| harlowja | right, jgriffith so https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L754 should be a place u can add a new task | 20:06 |
| jgriffith | harlowja: the OnFailureReschedule class no? | 20:06 |
| harlowja | thats the thing that gets triggered when a failure happens; that causes the loop to go back to the scheduler | 20:06 |
| harlowja | which the filter scheduler then interprets (and thats why this loop hard to understand) | 20:06 |
| jgriffith | harlowja: that's where I was at, but honestly got a bit concerned when I see the taskflow.create_volume versions EVERYWHERE | 20:07 |
| harlowja | ya, perhaps that could be better named module | 20:07 |
| jgriffith | in scheduler, as well as volume/flows/manager and API | 20:07 |
| harlowja | since its really 'create_volume_api' | 20:07 |
| harlowja | or create_volume_scheduler | 20:07 |
| harlowja | or create_volume_manager | 20:07 |
| harlowja | workflows for each piece | 20:07 |
| jgriffith | harlowja: yeah, as it stands it's sort of hard to figure out by code inspection IMHO | 20:07 |
| jgriffith | harlowja: especially since EVERYTHING has the same module name | 20:08 |
| jgriffith | just diff locations | 20:08 |
| jgriffith | and they all have pretty common method names :( | 20:08 |
| harlowja | at least we are consistent ;) | 20:08 |
| jgriffith | harlowja: that is true! | 20:08 |
| jgriffith | harlowja: so if I add this calss to volume/flows/manager | 20:08 |
| harlowja | yup | 20:09 |
| harlowja | drop it in at the end of volume_flow.add(ExtractVolumeSpecTask(db), | 20:09 |
| harlowja | NotifyVolumeActionTask(db, "create.start"), | 20:09 |
| harlowja | CreateVolumeFromSpecTask(db, driver), | 20:09 |
| harlowja | CreateVolumeOnFinishTask(db, "create.end")) | 20:09 |
| jgriffith | harlowja: I will also need an rpc call back out to cinder.volume.manager no? | 20:09 |
| harlowja | volume flows manager is running in the manager afaik | 20:09 |
| jgriffith | harlowja: because keep in mind, this is an internal class variable that we're trying to update | 20:09 |
| jgriffith | harlowja: Ahhhh!!!! | 20:10 |
| jgriffith | harlowja: so I'm already across the boundary at this point! | 20:10 |
| jgriffith | harlowja: rpc boundary that is | 20:10 |
| harlowja | just inhale dude | 20:10 |
| harlowja | lol | 20:10 |
| harlowja | oh | 20:10 |
| harlowja | that boundary | 20:10 |
| jgriffith | :) | 20:10 |
| e0ne | thingee: i've got some interesting issue from out support guys. here is steps to reproduce | 20:10 |
| jgriffith | harlowja: see what I mean? I don't think it's that trivial | 20:10 |
| jgriffith | harlowja: because it needs to update the host the volume was scheduled to | 20:11 |
| *** tbarron1 has quit IRC | 20:11 | |
| git-harry | thingee: regarding https://review.openstack.org/#/c/145858/ - it's not for cinder endpoints | 20:11 |
| *** tbarron has joined #openstack-cinder | 20:11 | |
| e0ne | thingee: i should to create a launchpad bug before submitting https://review.openstack.org/#/c/146541/ | 20:11 |
| jgriffith | harlowja: and we want to update it regardelss if it's created in error or available | 20:11 |
| git-harry | thingee: it so that cinder properly references nova's | 20:12 |
| jgriffith | harlowja: but obviously only increment it once :) | 20:12 |
| *** Masahiro has joined #openstack-cinder | 20:12 | |
| jgriffith | harlowja: the other thing I thought of was some way of checking the exception to see if it's the "last" attempt | 20:12 |
| harlowja | ok, so then u almost need to know if the rescheduling got triggered right | 20:12 |
| jgriffith | and just simply be done | 20:12 |
| jgriffith | exactly | 20:12 |
| harlowja | soooo | 20:12 |
| harlowja | an idear | 20:12 |
| jgriffith | I need to know if it was rescheduled (that is what led to the call I'm looking at) | 20:13 |
| jgriffith | Or | 20:13 |
| harlowja | u can plug-in notification to an engine in taskflow; to get notified of all tasks running | 20:13 |
| jgriffith | the other way around know that the exception is the last time I'll see it ;) | 20:13 |
| harlowja | and u can then handle if OnFailureRescheduleTask got triggered or if it didn't | 20:13 |
| jgriffith | harlowja: easy for you to say :) | 20:13 |
| jgriffith | harlowja: so that sounds like an elegant solution | 20:13 |
| jgriffith | harlowja: but I would have no idea how to go about that | 20:14 |
| *** Mandell has quit IRC | 20:14 | |
| harlowja | wellll thats ok | 20:14 |
| jgriffith | harlowja: :) | 20:14 |
| harlowja | https://review.openstack.org/#/c/146245/2/taskflow/examples/echo_listener.py is one such example | 20:15 |
| harlowja | or https://github.com/openstack/taskflow/blob/master/taskflow/examples/simple_linear_listening.py | 20:15 |
| jgriffith | harlowja: mmm | 20:15 |
| harlowja | engine.task_notifier.register('*', task_watch) (basically registering a function that will get triggered on all state changes) | 20:16 |
| harlowja | then if 'OnFailureRescheduleTask' runs; don't do whatever u wanted to | 20:16 |
| harlowja | but if it doesn't; then run special update code | 20:16 |
| *** Masahiro has quit IRC | 20:16 | |
| *** ryanpetrello has quit IRC | 20:17 | |
| jgriffith | harlowja: hurumph | 20:17 |
| *** juzuluag has joined #openstack-cinder | 20:17 | |
| harlowja | an idear | 20:17 |
| *** ryanpetrello has joined #openstack-cinder | 20:17 | |
| harlowja | http://docs.openstack.org/developer/taskflow/states.html#task are states u should get notified about | 20:18 |
| jgriffith | harlowja: maybe I'll just scrap this approach.... don't increment on failed create | 20:18 |
| jgriffith | harlowja: and add a flag somewhere that indicates as such | 20:18 |
| jgriffith | and never decrement if it's not toggled | 20:18 |
| jgriffith | at this point it seems like that might be *easier* | 20:18 |
| jgriffith | harlowja: looking at your last link... | 20:19 |
| harlowja | agreed, then u can just add a 'finish' adjustment that does that increment | 20:19 |
| harlowja | or finish on error (but not rescheduled) | 20:19 |
| harlowja | stupid rescheduling loop | 20:19 |
| harlowja | imho that needs to die (to confusing and funky) | 20:19 |
| jgriffith | harlowja: FWIW, I think the rescheduling part is cool | 20:19 |
| jgriffith | harlowja: just in the wrong place ;) | 20:19 |
| harlowja | but u almost need a conductor (that does the rescheduling in 1 place) to fix it better | 20:20 |
| jgriffith | harlowja: yeah | 20:20 |
| jgriffith | harlowja: I think so | 20:20 |
| harlowja | instead of being across 2 places in bounce<->bounce loop | 20:20 |
| harlowja | but we were all young once, ha | 20:20 |
| jgriffith | harlowja: yeah, it's kind of a big problem IMO | 20:20 |
| jgriffith | harlowja: haha | 20:21 |
| jgriffith | harlowja: well, who would've thought :) | 20:21 |
| harlowja | i sorta came out of my mom full grown so i didn't think it | 20:21 |
| harlowja | lol | 20:21 |
| harlowja | *poor mom* | 20:21 |
| jgriffith | harlowja: indeed!!! | 20:21 |
| *** alonmarx has quit IRC | 20:21 | |
| *** alonmarx has joined #openstack-cinder | 20:22 | |
| harlowja | it'd be nice to just start figuring out a path to some type of conductor imho (not neccasrily a remoting/objects conductor; but one that manages the lifecycle after API) | 20:22 |
| jgriffith | harlowja: well that's "in progress" | 20:22 |
| harlowja | basically API -> conductor (conductor then talks with volume manager, scheduler, blah blah; manages rest of lifecycle) | 20:22 |
| harlowja | cool | 20:22 |
| jgriffith | harlowja: thangp is working on Cinder objects and conductor I believe | 20:22 |
| harlowja | ya, but thats the remoting/objects part of the conductor | 20:23 |
| jgriffith | harlowja: but that doesn't help with "fixing" bugs :( | 20:23 |
| harlowja | which imho is nice; but meh | 20:23 |
| *** jamielennox|away has quit IRC | 20:23 | |
| *** sweston has quit IRC | 20:23 | |
| harlowja | lifecycle conductor management imho is nicer | 20:23 |
| jgriffith | harlowja: ahhhhh.... your talking manager of managers sor to fthing | 20:23 |
| harlowja | ya | 20:23 |
| jgriffith | harlowja: oh... interesting | 20:23 |
| jgriffith | harlowja: well that would sure make this code a bit "simpler" to understand and work on I think | 20:23 |
| harlowja | basically other managers are 'dumb | 20:23 |
| harlowja | conductor/lifecycle manager is smart | 20:23 |
| jgriffith | harlowja: which as you know has been my only real complaint | 20:24 |
| jgriffith | harlowja: but I seem to be the only one having issues :( | 20:24 |
| *** dannywilson has quit IRC | 20:24 | |
| harlowja | conductor/lifecycle manager is the only thing that uses taskflow (possible) | 20:24 |
| jgriffith | harlowja: maybe what I should do is punt and let the smarter people fix the bug | 20:24 |
| harlowja | other managers just take orders | 20:24 |
| jgriffith | harlowja: wonder how long it will sit in LP | 20:24 |
| jgriffith | harlowja: I like the idea *I think* | 20:25 |
| harlowja | conductor/lifecycle manager then can use state-tracking of taskflow; to be able to crash and recover | 20:25 |
| harlowja | annnnd profit :-P | 20:25 |
| jgriffith | if I understand what your proposing correctly | 20:25 |
| *** DuncanT has quit IRC | 20:25 | |
| harlowja | maybe, whats your interpretation :) | 20:25 |
| *** krtaylor has quit IRC | 20:25 | |
| *** boris-42 has quit IRC | 20:25 | |
| *** russellb has quit IRC | 20:25 | |
| jgriffith | harlowja: and if you get that it eliminates the need for the other work being proposed quite honestly | 20:25 |
| harlowja | agreed | 20:25 |
| *** vishy has quit IRC | 20:26 | |
| *** jbernard has quit IRC | 20:26 | |
| jgriffith | harlowja: a "true" manager or conductor to deal with "object" or "tasks" | 20:26 |
| jgriffith | so you have a central manager that rules the create flow for example | 20:26 |
| jgriffith | and includes allowing/disallowing state transistions within that task | 20:26 |
| harlowja | ya (everything after APi stuff); it uses simple RPC to talk to other | 20:26 |
| harlowja | dumb managers | 20:26 |
| *** Guest58319 has quit IRC | 20:27 | |
| jgriffith | so when you run into things like I'm running into now... you have a very easy API to just say "add this thing" | 20:27 |
| *** sweston has joined #openstack-cinder | 20:27 | |
| jgriffith | or include this additional info in the response/exception | 20:27 |
| harlowja | ya | 20:27 |
| jgriffith | well code that shit up :) | 20:27 |
| harlowja | i think nova will get to this someday to | 20:27 |
| jgriffith | You have until Friday | 20:27 |
| harlowja | *someday* | 20:27 |
| harlowja | haha | 20:27 |
| jgriffith | hey... actually, that's another idea | 20:27 |
| *** DuncanT has joined #openstack-cinder | 20:28 | |
| jgriffith | is there a way for me to modify the exception and include "retry attempt of 'n' allowed" | 20:28 |
| harlowja | make HA conductor/lifecycle manager; scale that out as needed (sharding, each lifecycle manager/group owns certain buckets...) | 20:28 |
| *** barra204 has joined #openstack-cinder | 20:28 | |
| harlowja | jgriffith i think it has that info; thought it did | 20:28 |
| jgriffith | harlowja: ok, now you're talking crazy | 20:28 |
| harlowja | ha | 20:29 |
| jgriffith | harlowja: I don't think so.... but lemme check | 20:29 |
| harlowja | oh, u have num_attempts; but not max | 20:29 |
| *** mgagne has joined #openstack-cinder | 20:29 | |
| *** jbernard has joined #openstack-cinder | 20:29 | |
| *** russellb has joined #openstack-cinder | 20:29 | |
| harlowja | https://github.com/openstack/cinder/blob/master/cinder/scheduler/filter_scheduler.py#L223 has this number though | 20:30 |
| harlowja | so if its passed along i guess it could get used? | 20:30 |
| *** mgagne is now known as Guest36580 | 20:30 | |
| harlowja | 'self.max_attempts' | 20:30 |
| *** boris-42 has joined #openstack-cinder | 20:30 | |
| harlowja | boris-42 i think wants to refactor nova to a simple 'lifecycle conductor' and dumb managers | 20:30 |
| harlowja | so he'll have that done by friday | 20:30 |
| jgriffith | harlowja: so the problem is that flow_engine.run() just bubbles up the original exception :( | 20:30 |
| *** krtaylor has joined #openstack-cinder | 20:31 | |
| jgriffith | harlowja: no info regarding taskflow | 20:31 |
| harlowja | hmmm; ya; u get a log message though https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L102 | 20:31 |
| boris-42 | harlowja: ?) | 20:31 |
| boris-42 | harlowja: me? | 20:31 |
| harlowja | boris-42 no questions; just do it | 20:31 |
| harlowja | haha | 20:31 |
| *** jamielennox|away has joined #openstack-cinder | 20:32 | |
| boris-42 | harlowja: okay | 20:32 |
| *** annashen has quit IRC | 20:32 | |
| *** barra204_ has joined #openstack-cinder | 20:32 | |
| *** barra204 has quit IRC | 20:32 | |
| harlowja | boris-42 we are talking about how cinder could change to have a 'smart conductor that does all the smart workflow stuff' that then is made HA; it talks to dumb other managers/drivers over RPC | 20:32 |
| *** jamielennox|away is now known as jamielennox | 20:32 | |
| harlowja | similar to that conversation we had way back when about nova (which they might get to eventually) | 20:32 |
| boris-42 | harlowja: hehe | 20:33 |
| thingee | git-harry: that is correct. However, in the future, nova would just need to know the service_type name 'volume' and that could be used for v1 or v2 in older systems. | 20:33 |
| harlowja | jgriffith sooo u want to almost modify the exception that is propgating in taskflow? | 20:33 |
| harlowja | hmmm | 20:33 |
| harlowja | i wonder if this is easier than it seems; one sec | 20:33 |
| thingee | git-harry: now that I think about it though, this might be useful for the stable versions of devstack | 20:33 |
| jgriffith | harlowja: bet it is for you :) | 20:33 |
| thingee | just not master of the projects | 20:34 |
| harlowja | well u'll see def revert(self, context, result, flow_failures, **kwargs): gets flow_failures | 20:34 |
| harlowja | flow_failures does contain the propagating exception | 20:34 |
| harlowja | but i don't think its mutaable | 20:34 |
| *** Mandell has joined #openstack-cinder | 20:34 | |
| jgriffith | harlowja: where are you looking at? | 20:34 |
| harlowja | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L147 | 20:34 |
| jgriffith | harlowja: ie where does the actual try/except and retry occur | 20:34 |
| jgriffith | looking | 20:34 |
| *** vishy has joined #openstack-cinder | 20:34 | |
| harlowja | jgriffith that's deep in taskflow where that happens | 20:35 |
| harlowja | *deeper | 20:35 |
| *** _cjones_ has quit IRC | 20:35 | |
| jgriffith | harlowja: that's what I thought | 20:35 |
| harlowja | https://github.com/openstack/taskflow/blob/master/taskflow/engines/action_engine/executor.py#L62 if u really want to know | 20:35 |
| jgriffith | harlowja: so *where* in flows/manager/create_volume can I intercept? | 20:35 |
| *** elazro has quit IRC | 20:36 | |
| jgriffith | harlowja: because from what I can tell... I can't | 20:36 |
| *** elazro has joined #openstack-cinder | 20:36 | |
| anish | jgriffith: thingee is IET support removed now ? | 20:36 |
| harlowja | well if u could alter the message to include more data at revert() time would that be ok | 20:37 |
| anish | thingee: whoops, not you. sorry | 20:37 |
| jgriffith | anish: I never implemented it but will probably need to | 20:37 |
| jgriffith | anish: we never had a formal deprecation so it probably has to be there | 20:37 |
| thingee | git-harry: Oh I see, this is cinder to nova. my bad | 20:38 |
| jgriffith | harlowja: you know what would be "smart".... | 20:38 |
| jgriffith | harlowja: so currently if you're a single host, and create fails | 20:38 |
| jgriffith | harlowja: we just keep retrying on that same host | 20:39 |
| jgriffith | harlowja: maybe we should only call self.reschedule_context if you have more than one host | 20:39 |
| harlowja | that'd safe people some sanity | 20:39 |
| harlowja | *save | 20:39 |
| harlowja | lifecycle conductor with all this logic :-P | 20:39 |
| harlowja | get yer done | 20:39 |
| jgriffith | sigh | 20:40 |
| jgriffith | :) | 20:40 |
| jungleboyj | smcginnis: Been working on the issue and now understand it. The problem is that lazy is getting enabled in test cases and not turned off. Trying to figure out the right way to fix that. | 20:40 |
| harlowja | http://docs.openstack.org/developer/taskflow/types.html#module-taskflow.types.failure is what comes in that 'flow_failure' stuff that gets into task revert methods | 20:40 |
| jgriffith | alright, I'm going to punt on this bug and let somebody like thingee or DuncanT that fully understand taskflow take it | 20:40 |
| harlowja | but u sorta just want to modify the contained exceptions in that and add a message addition | 20:40 |
| harlowja | *except those are immutable currently, lol | 20:41 |
| jgriffith | harlowja: well that won't work then :) | 20:41 |
| *** alexpilotti has quit IRC | 20:41 | |
| harlowja | perhaps u could munge with http://docs.openstack.org/developer/taskflow/types.html#taskflow.types.failure.Failure.exc_info but i'm not sure that would work | 20:42 |
| harlowja | *thats the raw (exc_type, exception, backtrace) tuple | 20:42 |
| harlowja | but u'd need to modify that [1] member there; and do things to it | 20:43 |
| anish | jgriffith: I was referring to https://github.com/openstack/cinder/commit/3a62391dc12d6383831e971a60d33aa4227f5368 | 20:43 |
| harlowja | which might be possible, depending on the type | 20:43 |
| jgriffith | harlowja: yeah but all of this means modifying the lib :( | 20:43 |
| harlowja | jgriffith i don't think so | 20:43 |
| anish | maybe I am following the code wrong, but looks like all references to ietadm were removed | 20:43 |
| harlowja | let me see | 20:43 |
| jgriffith | anish: yeah, so was I :) | 20:43 |
| jgriffith | anish: I modified that code and did not implement the iet portion | 20:43 |
| anish | aah good, then I am following it correctly | 20:43 |
| jgriffith | anish: referenced as such in the commit messages somewhere along the way | 20:44 |
| jgriffith | anish: any interest in doing the iet work? :) | 20:44 |
| thingee | jgriffith: yeah that's fine, sorry to hear it's making things difficult for this problem. | 20:44 |
| jgriffith | thingee: what's that? | 20:44 |
| thingee | jgriffith: with regards to you saying punting the bug to DuncanT or I. | 20:44 |
| jgriffith | thingee: oh | 20:45 |
| jgriffith | thingee: ok... sold | 20:45 |
| anish | jgriffith: I have two k2 specs to implement, but sure | 20:45 |
| thingee | jgriffith: I know you have other things on your plate. I'll have to catch up on your convo with harlowja though. | 20:45 |
| jgriffith | anish: I was kinda joking, sounds like you have a full plate right now | 20:45 |
| anish | haha | 20:46 |
| jgriffith | anish: I'll see if I can gte it coded up this week | 20:46 |
| jgriffith | I think it should be pretty straight forward | 20:46 |
| anish | I was implementing an iscsi helper based on the old iet code, so I'm kinda in the middle of that code anyways :) | 20:46 |
| jgriffith | LOL | 20:46 |
| anish | but yes, a reference IET implementation would be 100% useful | 20:46 |
| jgriffith | anish: alright, I'll carve out some time for it this week then | 20:46 |
| anish | awesome, thanks | 20:47 |
| jgriffith | anish: sorry I yanked that out from under you | 20:47 |
| anish | no worries ! I started following tgt/lio instead. just a bit more complicated | 20:47 |
| thingee | git-harry: ok sorry for my mistake. gave it a +1 | 20:49 |
| thingee | e0ne: thanks, looks like tests pass | 20:50 |
| thingee | harlowja: I'll ping ya later about what jgriffith was working on. | 20:50 |
| e0ne | thingee: np. i'll try to don't be such lazy and file a bugs | 20:51 |
| harlowja | thingee k | 20:51 |
| *** nellysmitt has quit IRC | 20:51 | |
| harlowja | jgriffith ya, to mutate the exception like in http://paste.ubuntu.com/9720730/ this would need to get rid of the immutablity of that object in the failure class | 20:51 |
| harlowja | which needs http://paste.ubuntu.com/9720737/ at least | 20:51 |
| harlowja | so its not impossible | 20:52 |
| harlowja | and might just be a useful addition | 20:52 |
| *** nellysmitt has joined #openstack-cinder | 20:54 | |
| thingee | e0ne: I'm not sure what you're talking about with regards to bug here | 20:55 |
| e0ne | thingee: are you talking about my patch to cinderclient? | 20:56 |
| *** lpabon has joined #openstack-cinder | 20:56 | |
| thingee | e0ne: you keep talking about bugs. I'm not sure what you're talking about | 20:56 |
| *** _cjones_ has joined #openstack-cinder | 20:57 | |
| e0ne | thingee: i'm sorry, i confused you. i mean that it would be nice to have a bug fot such patches as my one | 20:57 |
| jgriffith | harlowja: nice | 20:57 |
| *** david-lyle has joined #openstack-cinder | 20:57 | |
| jgriffith | harlowja: for now I'm just doing it another way | 20:59 |
| jgriffith | harlowja: thingee adding an "allocated" column to DB object, and a call out to the driver to check if it was allocated or not | 20:59 |
| jgriffith | harlowja: thingee then check that field on delete... done | 21:00 |
| *** Guest36580 is now known as mgagne | 21:04 | |
| *** mgagne has joined #openstack-cinder | 21:04 | |
| thingee | jgriffith: do you know why project clients in devstack aren't being shown in /opt/stack anymore? | 21:04 |
| jgriffith | thingee: there was a change a while back to add that | 21:05 |
| e0ne | thingee: because of it setuped from pypi, not sources | 21:05 |
| jgriffith | thingee: they're now all pip installed | 21:05 |
| e0ne | :) | 21:05 |
| thingee | mriedem: ^ | 21:05 |
| jgriffith | thingee: LIBS_FROM_GIT=python-cinderclient | 21:05 |
| thingee | mriedem: would it be possible that glance fix is not released? | 21:05 |
| jgriffith | add that to your local.conf and you're good | 21:05 |
| e0ne | i've got the similar problem with clients last week | 21:06 |
| thingee | mriedem: http://logs.openstack.org/66/145066/7/gate/gate-tempest-dsvm-full/233cf6d/logs/screen-c-vol.txt.gz#_2015-01-09_22_02_00_432 | 21:06 |
| thingee | jgriffith: good to know, thanks | 21:06 |
| jgriffith | thingee: ahhh... nice!!! | 21:06 |
| mriedem | thingee: is there a specific fix on glanceclient? scrollback is failing me | 21:06 |
| thingee | mriedem: I might've misinterpreted what you were saying in #openstack-infra about this being an old issue in glanceclient | 21:07 |
| mriedem | thingee: i know swiftclient had a similar problem | 21:07 |
| mriedem | thingee: i'll try to dig up the bug i know about | 21:07 |
| thingee | mriedem: thanks | 21:07 |
| mriedem | something about the move to requests lib and blobs in the request | 21:07 |
| mriedem | killing the pools and kaboom | 21:07 |
| mriedem | ha, top bug https://bugs.launchpad.net/python-glanceclient/+bug/1341777 | 21:08 |
| *** bswartz has quit IRC | 21:08 | |
| jgriffith | hehe | 21:08 |
| jgriffith | mriedem: thingee which is EXACTLY what's in the cinder logs :) | 21:09 |
| thingee | jgriffith: yup! | 21:09 |
| mriedem | yeah, i mean http://goo.gl/2jMEQD | 21:09 |
| mriedem | 20+K hits in 7 days | 21:09 |
| mriedem | this isn't old | 21:09 |
| mriedem | if you check the glance logs i think you see the same for swiftclient | 21:09 |
| harlowja | jgriffith https://review.openstack.org/#/c/146642/ should help here | 21:09 |
| mriedem | which is this bug https://bugs.launchpad.net/python-swiftclient/+bug/1295812 | 21:09 |
| harlowja | make that kind of mutation easier (where its desired to use it) | 21:10 |
| jgriffith | harlowja: looking | 21:10 |
| jgriffith | harlowja: hmm | 21:11 |
| jgriffith | harlowja: what about just raising a taskflow exception? | 21:11 |
| thingee | mriedem: ok thanks for the information | 21:11 |
| harlowja | jgriffith replace it is basically equivalent | 21:11 |
| jgriffith | harlowja: and part of that exception class is "original exception name and message" | 21:11 |
| jgriffith | harlowja: similar to how we do putils exceptions | 21:12 |
| jgriffith | harlowja: and include by default things like tf retry counters etc | 21:12 |
| jgriffith | harlowja: which... is pretty close to what you have anyway I think | 21:13 |
| jgriffith | harlowja: although..... | 21:13 |
| jgriffith | harlowja: hmmm | 21:13 |
| harlowja | well this would be a task controlled mechanism | 21:13 |
| jgriffith | harlowja: I do kinda like the idea of the replace on revert | 21:13 |
| *** git-harry has quit IRC | 21:14 | |
| harlowja | well its either that or revert throws a new different exceptoion | 21:14 |
| harlowja | which it can | 21:14 |
| *** git-harry has joined #openstack-cinder | 21:14 | |
| *** bswartz has joined #openstack-cinder | 21:15 | |
| *** e0ne has quit IRC | 21:15 | |
| harlowja | although thats sorta more for catastrophic things, ha | 21:15 |
| *** git-harry has left #openstack-cinder | 21:15 | |
| *** david-lyle has quit IRC | 21:16 | |
| *** david-ly_ has joined #openstack-cinder | 21:16 | |
| nikesh_vedams | anish:hi | 21:17 |
| anish | hello | 21:17 |
| harlowja | jgriffith replace on revert would basically allow u to shove whatever information u want in to a new exception (that will then bubble up); throwing from revert sorta stops other reverting of predeccessors (so thats not going to end well) | 21:19 |
| jgriffith | harlowja: true | 21:19 |
| anish | nikesh_vedams: you pinged ? | 21:20 |
| harlowja | so this is more of a useful function for those that really know what they are doing and can safely do it | 21:20 |
| jgriffith | harlowja: which means I'm out :) | 21:20 |
| harlowja | lol | 21:21 |
| harlowja | bb | 21:21 |
| nikesh_vedams | anish: hi small doubt,we reviewed your patch,how will it deal with multiple-cards | 21:22 |
| anish | as in multiple transports all connected to the same remote target ? | 21:23 |
| anish | the current implementation as I have done it does not support multiple cards | 21:24 |
| *** git-harry has joined #openstack-cinder | 21:24 | |
| anish | the chief reason being with routing information you cannot figure out which transport would connect to which remote target | 21:25 |
| anish | so all you can do is a mix of tcp + iser/hardware | 21:25 |
| anish | *without routing information | 21:26 |
| jgriffith | thingee: if you target this to K2 I'm signed up to do it | 21:26 |
| anish | ** tcp + 1 iser/hardware, which is provided by the config file | 21:26 |
| *** julim_ has quit IRC | 21:26 | |
| *** bswartz has quit IRC | 21:30 | |
| thingee | jgriffith: ok. I need to do the LIO portion too | 21:31 |
| jgriffith | thingee: ? | 21:31 |
| jgriffith | thingee: lio is there? | 21:31 |
| thingee | enable iser with LIO, yea | 21:31 |
| jgriffith | thingee: not following? Did I miss something? | 21:31 |
| thingee | jgriffith: not sure. I do know that I want to allow enabling iser if LIO is the target helper | 21:32 |
| *** xyang1 has joined #openstack-cinder | 21:33 | |
| jgriffith | thingee: oh, use LIO for iser instead of defaulting to Tgt? | 21:33 |
| *** akerr has quit IRC | 21:34 | |
| thingee | yeah LIO can enable iser on the network portal. it's a simple bit switch. "simple" :) | 21:34 |
| *** mriedem has left #openstack-cinder | 21:34 | |
| thingee | jgriffith: I started on some clean up work in rtstool to switch to argparser and then I stopped, heh | 21:34 |
| jgriffith | thingee: distractions distractions :) | 21:35 |
| thingee | jgriffith: indeed. makes me wonder what I'm actually getting done | 21:35 |
| nikesh_vedams | anish: tcp + iser/hardware ? if CONF.libvirt.iscsi_transport=be2iscsi ,then how can i use tcp transport | 21:35 |
| anish | transport is not used for discovery | 21:36 |
| anish | if you obtain targets via discovery, then the transport used will be tcp | 21:36 |
| *** jungleboyj has quit IRC | 21:37 | |
| anish | I'm also planning to add a fallback case for login over transport, retry with tcp if login does not succeed | 21:38 |
| nikesh_vedams | anish: in our case,we have to use both software and hardware transport and multi-path too | 21:39 |
| anish | yes, I am handling that as well. code is still WIP | 21:40 |
| *** _cjones_ has quit IRC | 21:40 | |
| nikesh_vedams | anish: if we are using multi-path then it will do discovery with transport i guess | 21:40 |
| anish | however, there is no way to do multiple hardware transports | 21:40 |
| *** david-ly_ has quit IRC | 21:40 | |
| *** david-lyle has joined #openstack-cinder | 21:41 | |
| *** annashen has joined #openstack-cinder | 21:41 | |
| anish | I'm not sure about be2iscsi, but bnx2i,cxgbNi,qla4xxx are all iface bound, and iser/tcp are not bound | 21:42 |
| anish | so I will be adding iser to the discovery command in follow up code | 21:42 |
| anish | however there is no way to automatically handle multiple hardware transports, unless you have the target explicitly telling you which transport to pick | 21:43 |
| anish | information which again cannot automatically computed | 21:43 |
| *** Mandell has quit IRC | 21:48 | |
| *** david-lyle has quit IRC | 21:49 | |
| *** david-lyle has joined #openstack-cinder | 21:50 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/python-cinderclient: Add volume multi attach support https://review.openstack.org/85856 | 21:55 |
| *** Mandell has joined #openstack-cinder | 21:58 | |
| *** Masahiro has joined #openstack-cinder | 22:01 | |
| *** dustins has quit IRC | 22:05 | |
| *** ebalduf has quit IRC | 22:05 | |
| *** Masahiro has quit IRC | 22:05 | |
| nikesh_vedams | anish: so i think even if we use hardware transport,then multi=path will not work with your patch | 22:07 |
| nikesh_vedams | only hardware transport | 22:07 |
| *** Yogi1 has quit IRC | 22:08 | |
| *** ebalduf has joined #openstack-cinder | 22:08 | |
| nikesh_vedams | anish: am i right | 22:08 |
| *** bkopilov has quit IRC | 22:08 | |
| anish | multi path will work fine | 22:09 |
| anish | multipath uses discovery to get targets. discovery does not use transports | 22:09 |
| *** ebalduf has quit IRC | 22:10 | |
| anish | what will not work is trying to use multiple transports at once, as there is no way to specify this | 22:10 |
| *** mriedem has joined #openstack-cinder | 22:10 | |
| anish | though even this you can get around by manually creating nodes with the iface parameter specified | 22:10 |
| *** _cjones_ has joined #openstack-cinder | 22:11 | |
| *** rushil_ has joined #openstack-cinder | 22:12 | |
| *** Mandell has quit IRC | 22:12 | |
| *** thangp has quit IRC | 22:12 | |
| *** rushil has quit IRC | 22:14 | |
| *** ebalduf has joined #openstack-cinder | 22:14 | |
| *** e0ne has joined #openstack-cinder | 22:17 | |
| *** bkopilov has joined #openstack-cinder | 22:18 | |
| *** lpabon has quit IRC | 22:19 | |
| *** juzuluag has quit IRC | 22:20 | |
| *** jungleboyj has joined #openstack-cinder | 22:21 | |
| nikesh_vedams | anish: ok | 22:21 |
| harlowja | jgriffith soon enough we'll make a TF expert out of u ;) | 22:22 |
| jgriffith | harlowja: haha | 22:23 |
| *** r-daneel has quit IRC | 22:23 | |
| jungleboyj | jgriffith: If you have a chance can you take a look at the results of my investigation here: https://review.openstack.org/#/c/145642/ | 22:23 |
| harlowja | jgriffith why u laughing | 22:23 |
| harlowja | lol | 22:23 |
| jgriffith | harlowja: my understanding after the summit is that we already had several | 22:23 |
| jgriffith | harlowja: I'm game, but you're going to have to set aside a couple hours in Vancouver to educate me | 22:24 |
| anish | what pray tell is TF ? | 22:24 |
| jgriffith | harlowja: last few days has def helped | 22:24 |
| harlowja | team fortress | 22:24 |
| harlowja | lol | 22:24 |
| jgriffith | LMAO | 22:24 |
| anish | up your skills to TF2 at the very least | 22:24 |
| harlowja | def | 22:24 |
| harlowja | i'm game for TF session in vancouver | 22:27 |
| harlowja | TF2 | 22:27 |
| harlowja | ha | 22:27 |
| harlowja | to 'educate' | 22:27 |
| openstackgerrit | Rushil Chugh proposed openstack/cinder: Garbage Remains when Attached Volume is Migrated with NFS Driver https://review.openstack.org/143431 | 22:28 |
| *** nellysmitt has quit IRC | 22:28 | |
| nikesh_vedams | jgriffith:hi our CI is running for more than 1hr,i saw some patched uploaded in this channel but still loop...debug | 22:32 |
| nikesh_vedams | jgriffith:https://review.openstack.org/#/c/143431/ latest commited | 22:32 |
| jgriffith | nikesh_vedams: well then your listener isn't working it would appear | 22:33 |
| jgriffith | nikesh_vedams: add a logger in on the even listener | 22:34 |
| jgriffith | nikesh_vedams: you have your openstack CI account and ssh keys? | 22:34 |
| nikesh_vedams | yes | 22:34 |
| jgriffith | nikesh_vedams: write a little test and make sure you're receiving notifications | 22:34 |
| nikesh_vedams | just now its worked | 22:35 |
| jgriffith | nikesh_vedams: :) | 22:35 |
| nikesh_vedams | Adding review id 85856 to job queue... Adding event to queue... | 22:35 |
| jgriffith | nikesh_vedams: note that the version on my github has a filter | 22:36 |
| jgriffith | nikesh_vedams: HOPEFULLY you at least looked at the code before kicking this off | 22:36 |
| jgriffith | nikesh_vedams: it should be VERY OBVIOUS that the filter is set to only pass on events submitted by Jenkins that include a "verified +1" | 22:36 |
| *** Mandell has joined #openstack-cinder | 22:37 | |
| nikesh_vedams | jgriffith:in which file is that filter | 22:38 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add migration tests for PostgreSQL https://review.openstack.org/141414 | 22:39 |
| nikesh_vedams | jgriffith:ERROR: the playbook: ./ansible/sites.yml could not be found ERROR: the playbook: ./ansible/publish.yml could not be found ERROR: the playbook: ./ansible/teardown.yml could not be found Completed VEDAMS SCST CI-dsvm-full | 22:40 |
| jungleboyj | e0ne: Do you sleep? | 22:41 |
| e0ne | jungleboyj: it's difficult question | 22:41 |
| e0ne | jungleboyj: thanks for quick review | 22:42 |
| jungleboyj | e0ne: :-) You are welcome. Thanks for fixing. | 22:42 |
| jgriffith | he's views are a bit nikesh_vedams those errors seem pretty clear to me | 22:44 |
| jgriffith | nikesh_vedams: you might have to actually open the code in and editor and try to understand it a bit on your own | 22:45 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Snapshot object https://review.openstack.org/133566 | 22:45 |
| *** mriedem has quit IRC | 22:45 | |
| jgriffith | nikesh_vedams: you should be able to explain to me how it works | 22:45 |
| jgriffith | nikesh_vedams: it's about as simple as it gets | 22:45 |
| jgriffith | nikesh_vedams: thus the name "sort of simple" | 22:45 |
| nikesh_vedams | jgriffith:ok actually its 4.15 am for me | 22:46 |
| nikesh_vedams | yes i will understand it | 22:46 |
| nikesh_vedams | :) | 22:46 |
| *** dannywilson has joined #openstack-cinder | 22:47 | |
| *** _cjones_ has quit IRC | 22:48 | |
| *** _cjones_ has joined #openstack-cinder | 22:48 | |
| anish | jgriffith: review or a at least a quick set of eyes please :) https://review.openstack.org/#/c/146233/ | 22:50 |
| *** e0ne has quit IRC | 22:54 | |
| *** kaufer has quit IRC | 22:54 | |
| *** annashen has quit IRC | 22:55 | |
| jgriffith | anish: oh... I didn't realize "_get_all_block_devices" returned the full path | 22:55 |
| *** ryanpetrello_ has joined #openstack-cinder | 22:56 | |
| jgriffith | anish: seems pretty solid to me | 22:57 |
| *** annashen has joined #openstack-cinder | 22:57 | |
| anish | awesome ! | 22:57 |
| openstackgerrit | Aaron Rosen proposed openstack/python-cinderclient: Leverage openstack.common.importutils import_class https://review.openstack.org/118971 | 22:58 |
| anish | there is still the issue that nikesh_vedams mentioned, however there is no good way around that as of now | 22:58 |
| *** ryanpetrello has quit IRC | 22:59 | |
| *** ryanpetrello_ is now known as ryanpetrello | 22:59 | |
| anish | viz. for a multipath device you could have ever single path over a different hardware transport, but this cannot be automated in any reasonable way | 22:59 |
| anish | unless I get the target to request a transport | 22:59 |
| anish | which I am really not sure is even doable, without extensive knowledge of network topography | 23:00 |
| *** barra204__ has joined #openstack-cinder | 23:02 | |
| *** barra204_ has quit IRC | 23:02 | |
| *** tbarron has quit IRC | 23:03 | |
| jgriffith | if anybody has any silly nits or bike-shedding related requests for https://review.openstack.org/#/c/142859/ | 23:08 |
| jgriffith | please speak now or forever hold your peace | 23:08 |
| jgriffith | :) | 23:08 |
| openstackgerrit | John Griffith proposed openstack/cinder: Implement clone_image caching on SolidFire https://review.openstack.org/142859 | 23:09 |
| smcginnis | jgriffith: Looks like jenkins does. :/ | 23:09 |
| *** barra204__ is now known as shakamunyi | 23:09 | |
| jgriffith | smcginnis: yeah, that pep8 :) | 23:09 |
| jgriffith | smcginnis: stupid "and vs if:if" | 23:09 |
| jgriffith | smcginnis: but that's fixed :) | 23:10 |
| jgriffith | smcginnis: FWIW your comments were all valid | 23:10 |
| smcginnis | jgriffith: It's looked good to me. I think you've been nit picked enough for now. :) | 23:10 |
| jgriffith | smcginnis: gee thanks :) | 23:11 |
| *** bswartz has joined #openstack-cinder | 23:13 | |
| *** lcurtis has quit IRC | 23:15 | |
| *** david-lyle has quit IRC | 23:20 | |
| *** rlucio has joined #openstack-cinder | 23:21 | |
| mtanino | jgriffith: hi, Could you see this bug fix patch for target again? https://review.openstack.org/#/c/145381/ | 23:26 |
| mtanino | jgriffith: Most fixes are done in your patch. This adds some test codes and minor bug fixes. | 23:27 |
| *** chlong has joined #openstack-cinder | 23:29 | |
| *** fischerw has quit IRC | 23:29 | |
| jgriffith | mtanino: for sure... looking now | 23:30 |
| mtanino | jgriffith: Thank you :) | 23:30 |
| jgriffith | mtanino: ha!!! | 23:31 |
| jgriffith | mtanino: test_create_create_export :) | 23:31 |
| mtanino | jgriffith: yes... | 23:31 |
| jgriffith | mtanino: I actually did that on purpose | 23:31 |
| jgriffith | mtanino: although was supposed to be changed before submitting :) | 23:31 |
| jgriffith | mtanino: LGTM thanks! | 23:32 |
| mtanino | jgriffith: Thank you for the quick review. | 23:34 |
| openstackgerrit | Nagendra Rao Jaladanki proposed openstack/cinder-specs: Spec for Brocade Zone Driver Virtual Fabrics support https://review.openstack.org/144389 | 23:34 |
| *** shakamunyi has quit IRC | 23:35 | |
| tsekiyama | jgriffith: Could you review https://review.openstack.org/#/c/145391/ too...? It fixes another issue in TgtAdm that _get_target_chap_auth() is defined but not used. | 23:37 |
| tsekiyama | jgriffith: This make volume boot instances fail to live-migration. | 23:38 |
| *** kmartin has quit IRC | 23:38 | |
| *** shakamunyi has joined #openstack-cinder | 23:38 | |
| *** jungleboyj has quit IRC | 23:40 | |
| nikesh_vedams | jgriffith:that error fixed | 23:42 |
| nikesh_vedams | but again new error Unable to find an inventory file, specify one with -i ? | 23:42 |
| *** annashen has quit IRC | 23:44 | |
| nikesh_vedams | jgriffith:Unable to find an inventory file, specify one with -i ? | 23:46 |
| *** annashen has joined #openstack-cinder | 23:47 | |
| *** Masahiro has joined #openstack-cinder | 23:49 | |
| *** mriedem has joined #openstack-cinder | 23:52 | |
| *** Masahiro has quit IRC | 23:54 | |
| *** scottda_ has joined #openstack-cinder | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!