*** haomaiwang has quit IRC | 00:01 | |
*** haomaiwang has joined #openstack-cinder | 00:01 | |
*** akerr has joined #openstack-cinder | 00:05 | |
*** akerr_ has joined #openstack-cinder | 00:06 | |
*** salv-orl_ has quit IRC | 00:08 | |
*** akerr has quit IRC | 00:09 | |
*** dims has quit IRC | 00:10 | |
hemna | patrickeast, ping | 00:11 |
---|---|---|
*** sheel has joined #openstack-cinder | 00:17 | |
patrickeast | hemna: pong | 00:18 |
hemna | patrickeast, was looking at this one https://review.openstack.org/#/c/214825/ | 00:18 |
hemna | but I don't see pure CI reporting :( | 00:18 |
hemna | otherwise I'd shove it in | 00:18 |
patrickeast | Yah Im fixing the ci now | 00:19 |
hemna | ok | 00:19 |
hemna | didn't mean to pester you on it | 00:19 |
hemna | patrickeast, https://review.openstack.org/#/c/236143/ when you have time? :) | 00:20 |
*** mtanino_ has joined #openstack-cinder | 00:22 | |
*** diablo_rojo has quit IRC | 00:23 | |
*** mtanino has quit IRC | 00:24 | |
*** raildo-afk is now known as raildo | 00:25 | |
patrickeast | hemna: will do :D | 00:26 |
hemna | patrickeast, thanks | 00:27 |
*** mudassirlatif has quit IRC | 00:29 | |
*** mudassirlatif has joined #openstack-cinder | 00:32 | |
*** mudassirlatif has quit IRC | 00:32 | |
*** akerr_ has quit IRC | 00:34 | |
openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Create consistency group from source CG https://review.openstack.org/274908 | 00:35 |
*** arch-nemesis has quit IRC | 00:37 | |
*** geguileo has quit IRC | 00:47 | |
*** liewegas has quit IRC | 00:49 | |
*** liewegas has joined #openstack-cinder | 00:50 | |
*** alonma has joined #openstack-cinder | 00:52 | |
*** qeelee has joined #openstack-cinder | 00:55 | |
*** jamielennox|away is now known as jamielennox | 00:55 | |
*** alonma has quit IRC | 00:57 | |
openstackgerrit | abhiram moturi proposed openstack/cinder: Zfssaiscsi: Implement replication v2 https://review.openstack.org/274954 | 01:00 |
*** haomaiwang has quit IRC | 01:01 | |
*** haomaiwang has joined #openstack-cinder | 01:01 | |
*** haomaiwang has quit IRC | 01:04 | |
*** [1]Thelo has joined #openstack-cinder | 01:04 | |
*** qeelee has quit IRC | 01:06 | |
*** Thelo has quit IRC | 01:06 | |
*** [1]Thelo is now known as Thelo | 01:06 | |
openstackgerrit | Alex Meade proposed openstack/cinder-specs: User facing error messages https://review.openstack.org/274958 | 01:13 |
*** davechen has joined #openstack-cinder | 01:14 | |
*** markvoelker has joined #openstack-cinder | 01:15 | |
*** diogogmt has joined #openstack-cinder | 01:18 | |
*** markvoelker has quit IRC | 01:20 | |
*** qeelee has joined #openstack-cinder | 01:21 | |
*** jamielennox is now known as jamielennox|away | 01:23 | |
*** bill_az has quit IRC | 01:23 | |
*** sheel has quit IRC | 01:27 | |
*** qeelee has quit IRC | 01:27 | |
*** qwebirc92791 has joined #openstack-cinder | 01:27 | |
*** mtanino_ has quit IRC | 01:27 | |
*** Lee1092 has joined #openstack-cinder | 01:28 | |
*** cheneydc has joined #openstack-cinder | 01:29 | |
*** garthb has quit IRC | 01:34 | |
*** daneyon_ has quit IRC | 01:36 | |
*** daneyon has joined #openstack-cinder | 01:36 | |
*** leeantho has quit IRC | 01:41 | |
*** zhangjn has joined #openstack-cinder | 01:43 | |
*** julim has joined #openstack-cinder | 01:49 | |
*** jgriffith is now known as jgriffith_away | 01:50 | |
*** alonma has joined #openstack-cinder | 01:52 | |
*** Apoorva has quit IRC | 01:52 | |
*** jgriffith_away is now known as jgriffith | 01:53 | |
*** haomaiwang has joined #openstack-cinder | 01:55 | |
*** alonma has quit IRC | 01:57 | |
*** raildo is now known as raildo-afk | 01:59 | |
*** haomaiwang has quit IRC | 02:01 | |
*** haomaiwang has joined #openstack-cinder | 02:01 | |
*** jgriffith is now known as jgriffith_away | 02:04 | |
*** qeelee has joined #openstack-cinder | 02:05 | |
*** itzdilip has joined #openstack-cinder | 02:09 | |
*** dims has joined #openstack-cinder | 02:11 | |
*** markvoelker has joined #openstack-cinder | 02:16 | |
*** thurloat is now known as thurloat_isgone | 02:20 | |
*** markvoelker has quit IRC | 02:20 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix service-list filter https://review.openstack.org/263257 | 02:22 |
*** zaze89 has joined #openstack-cinder | 02:25 | |
*** coolsvap|away is now known as coolsvap | 02:26 | |
*** davechen1 has joined #openstack-cinder | 02:31 | |
*** davechen1 has quit IRC | 02:31 | |
*** davechen1 has joined #openstack-cinder | 02:32 | |
*** davechen has quit IRC | 02:33 | |
*** houming has joined #openstack-cinder | 02:38 | |
*** jamielennox|away is now known as jamielennox | 02:44 | |
*** haomaiwa_ has joined #openstack-cinder | 02:44 | |
*** haomaiwang has quit IRC | 02:45 | |
*** davechen1 is now known as davechen | 02:46 | |
*** qeelee has quit IRC | 02:46 | |
*** laughterwym has joined #openstack-cinder | 02:48 | |
*** sheel has joined #openstack-cinder | 02:50 | |
*** alonma has joined #openstack-cinder | 02:53 | |
*** laughterwym has quit IRC | 02:53 | |
*** zhangjn has quit IRC | 02:53 | |
*** zhangjn has joined #openstack-cinder | 02:54 | |
*** alonma has quit IRC | 02:57 | |
*** bardia has quit IRC | 02:59 | |
*** coolsvap is now known as coolsvap|away | 02:59 | |
*** baojg_ has quit IRC | 02:59 | |
*** haomaiwa_ has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** gouthamr_ has quit IRC | 03:04 | |
*** qeelee has joined #openstack-cinder | 03:05 | |
*** zaze89 has quit IRC | 03:07 | |
*** haomaiwang has quit IRC | 03:12 | |
*** haomaiwang has joined #openstack-cinder | 03:13 | |
*** links has joined #openstack-cinder | 03:15 | |
*** davechen1 has joined #openstack-cinder | 03:16 | |
*** davechen has quit IRC | 03:18 | |
*** jgriffith_away is now known as jgriffith | 03:26 | |
*** itzdilip has quit IRC | 03:30 | |
*** ianbrown_ has joined #openstack-cinder | 03:31 | |
*** ianbrown has quit IRC | 03:34 | |
*** alonma has joined #openstack-cinder | 03:34 | |
*** haomaiwang has quit IRC | 03:37 | |
*** laughterwym has joined #openstack-cinder | 03:39 | |
*** alonma has quit IRC | 03:39 | |
*** itzdilip has joined #openstack-cinder | 03:40 | |
*** alonma has joined #openstack-cinder | 03:42 | |
*** haomaiwang has joined #openstack-cinder | 03:45 | |
*** alonma has quit IRC | 03:46 | |
*** gouthamr has joined #openstack-cinder | 03:47 | |
*** alonma has joined #openstack-cinder | 03:48 | |
*** gouthamr_ has joined #openstack-cinder | 03:49 | |
*** qeelee has quit IRC | 03:50 | |
*** EinstCra_ has joined #openstack-cinder | 03:51 | |
*** gouthamr has quit IRC | 03:51 | |
*** dims has quit IRC | 03:53 | |
*** laughterwym has quit IRC | 03:53 | |
*** alonma has quit IRC | 03:54 | |
*** laughterwym has joined #openstack-cinder | 03:54 | |
*** EinstCrazy has quit IRC | 03:55 | |
*** alonma has joined #openstack-cinder | 03:55 | |
*** laughterwym has quit IRC | 03:58 | |
*** alonma has quit IRC | 03:59 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-cinder | 04:01 | |
*** gouthamr_ has quit IRC | 04:04 | |
*** haomaiw__ has joined #openstack-cinder | 04:06 | |
*** coolsvap|away is now known as coolsvap | 04:07 | |
*** haomaiwang has quit IRC | 04:08 | |
*** alonma has joined #openstack-cinder | 04:09 | |
*** jamielennox is now known as jamielennox|away | 04:10 | |
*** winston-d_ has joined #openstack-cinder | 04:10 | |
*** davechen has joined #openstack-cinder | 04:10 | |
*** davechen1 has quit IRC | 04:12 | |
*** alonma has quit IRC | 04:13 | |
*** mriedem has left #openstack-cinder | 04:15 | |
*** markvoelker has joined #openstack-cinder | 04:17 | |
*** markvoelker has quit IRC | 04:21 | |
*** isaacb has joined #openstack-cinder | 04:23 | |
*** grumpycatt has quit IRC | 04:23 | |
*** grumpycatt has joined #openstack-cinder | 04:26 | |
*** shyama has joined #openstack-cinder | 04:29 | |
*** laughterwym has joined #openstack-cinder | 04:29 | |
*** jwcroppe has joined #openstack-cinder | 04:36 | |
*** jamielennox|away is now known as jamielennox | 04:39 | |
*** zhangjn has quit IRC | 04:46 | |
*** zhangjn has joined #openstack-cinder | 04:47 | |
*** isaacb has quit IRC | 04:49 | |
*** baojg has joined #openstack-cinder | 04:49 | |
*** shyama_ has joined #openstack-cinder | 04:57 | |
*** shyama has quit IRC | 04:58 | |
*** shyama_ is now known as shyama | 04:58 | |
*** haomaiw__ has quit IRC | 05:01 | |
*** shyama has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-cinder | 05:01 | |
*** ankit_ag has joined #openstack-cinder | 05:05 | |
*** alonma has joined #openstack-cinder | 05:09 | |
*** laughterwym has quit IRC | 05:12 | |
*** laughterwym has joined #openstack-cinder | 05:12 | |
*** jamielennox is now known as jamielennox|away | 05:13 | |
*** alonma has quit IRC | 05:13 | |
*** alonma has joined #openstack-cinder | 05:16 | |
*** laughterwym has quit IRC | 05:17 | |
*** laughterwym has joined #openstack-cinder | 05:18 | |
*** alonma has quit IRC | 05:20 | |
*** alonma has joined #openstack-cinder | 05:22 | |
*** jwcroppe_ has joined #openstack-cinder | 05:25 | |
*** alonma has quit IRC | 05:26 | |
*** cheneydc has quit IRC | 05:26 | |
*** jwcroppe has quit IRC | 05:27 | |
*** baojg has quit IRC | 05:28 | |
*** alonma has joined #openstack-cinder | 05:28 | |
*** baojg has joined #openstack-cinder | 05:30 | |
*** alonma has quit IRC | 05:33 | |
*** alonma has joined #openstack-cinder | 05:35 | |
*** alonma has quit IRC | 05:39 | |
*** alonma has joined #openstack-cinder | 05:41 | |
*** slunkad has quit IRC | 05:41 | |
*** sileht has quit IRC | 05:44 | |
*** alonma has quit IRC | 05:45 | |
*** alonma has joined #openstack-cinder | 05:47 | |
*** Guest51478 has quit IRC | 05:48 | |
sheel | ameade: hi there | 05:52 |
*** alonma has quit IRC | 05:52 | |
*** cheneydc has joined #openstack-cinder | 05:52 | |
*** alonma has joined #openstack-cinder | 05:53 | |
*** slunkad has joined #openstack-cinder | 05:53 | |
*** changbl has quit IRC | 05:56 | |
*** vivekd has joined #openstack-cinder | 05:57 | |
*** alonma has quit IRC | 05:58 | |
*** alonma has joined #openstack-cinder | 06:00 | |
*** haomaiwang has quit IRC | 06:01 | |
*** haomaiwa_ has joined #openstack-cinder | 06:01 | |
*** alonma has quit IRC | 06:04 | |
*** alonma has joined #openstack-cinder | 06:06 | |
*** vgridnev has joined #openstack-cinder | 06:06 | |
*** alonma has quit IRC | 06:11 | |
*** alonma has joined #openstack-cinder | 06:12 | |
*** ChubYann has quit IRC | 06:13 | |
*** alonma has quit IRC | 06:16 | |
*** markvoelker has joined #openstack-cinder | 06:18 | |
*** alonma has joined #openstack-cinder | 06:18 | |
*** qeelee has joined #openstack-cinder | 06:22 | |
*** markvoelker has quit IRC | 06:22 | |
*** rcernin has joined #openstack-cinder | 06:22 | |
*** alonma has quit IRC | 06:23 | |
*** bkumar has joined #openstack-cinder | 06:23 | |
*** boris-42 has quit IRC | 06:23 | |
*** alonma has joined #openstack-cinder | 06:24 | |
openstackgerrit | Merged openstack/cinder: Improve logging for volume detach https://review.openstack.org/274630 | 06:25 |
bkumar | https://bugs.launchpad.net/cinder/+bug/1540761 | 06:25 |
openstack | Launchpad bug 1540761 in Cinder "Image Volume Cache Feature is failing in master branch" [Undecided,New] | 06:25 |
*** bardia has joined #openstack-cinder | 06:27 | |
bkumar | is anyone facing the same problem? | 06:28 |
*** alonma has quit IRC | 06:28 | |
*** alonma has joined #openstack-cinder | 06:30 | |
patrickeast | bkumar: i haven't seen it, but also haven't tested the image cache with lvm in a while | 06:34 |
*** alonma has quit IRC | 06:35 | |
*** alonma has joined #openstack-cinder | 06:36 | |
bkumar | @<patrickeast> i removed some line of code in the program , after that im seeing 3 copies of the cached image | 06:40 |
*** vgridnev has quit IRC | 06:40 | |
patrickeast | bkumar: uh, which lines did you remove? | 06:40 |
*** vgridnev has joined #openstack-cinder | 06:40 | |
*** vgridnev has quit IRC | 06:41 | |
*** alonma has quit IRC | 06:41 | |
*** alonma has joined #openstack-cinder | 06:43 | |
*** rcernin has quit IRC | 06:46 | |
bkumar | line number 552 in /opt/stack/cinder/cinder/brick/local_dev/lvm.py | 06:47 |
*** alonma has quit IRC | 06:47 | |
*** haomaiw__ has joined #openstack-cinder | 06:47 | |
bkumar | this line : | 06:48 |
bkumar | #try: | 06:48 |
bkumar | # self._execute(*cmd, | 06:48 |
bkumar | # root_helper=self._root_helper, | 06:48 |
bkumar | # run_as_root=True) | 06:48 |
bkumar | #except putils.ProcessExecutionError as err: | 06:48 |
bkumar | # LOG.exception(_LE('Error creating Volume')) | 06:48 |
*** haomaiwa_ has quit IRC | 06:48 | |
bkumar | # LOG.error(_LE('Cmd :%s'), err.cmd) | 06:48 |
bkumar | # LOG.error(_LE('StdOut :%s'), err.stdout) | 06:48 |
bkumar | # LOG.error(_LE('StdErr :%s'), err.stderr) | 06:48 |
bkumar | # LOG.error(_LE('Error is from here')) | 06:48 |
bkumar | # raise | 06:48 |
*** alonma has joined #openstack-cinder | 06:49 | |
*** dalgaaf has joined #openstack-cinder | 06:51 | |
*** alonma has quit IRC | 06:53 | |
*** zhangjn has quit IRC | 06:54 | |
bkumar | patrickeast : the error was Logical volume is already exists , and after commenting the exception im getting 3 copies of image , Somehow it is creating 3 copies which is throwing the error | 06:54 |
*** alonma has joined #openstack-cinder | 06:55 | |
*** avishay has joined #openstack-cinder | 06:57 | |
lixiaoy1 | DuncanT: Hi Duncan, may I ask you review https://review.openstack.org/#/c/262395/? Chinese Spring festival is comming soon, and I will take vacation. Hope you review it earlier? | 06:58 |
*** bardia has quit IRC | 06:59 | |
*** alonma has quit IRC | 07:00 | |
*** haomaiw__ has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-cinder | 07:01 | |
*** alonma has joined #openstack-cinder | 07:01 | |
*** cfriesen has quit IRC | 07:03 | |
*** alonma has quit IRC | 07:06 | |
*** alonma has joined #openstack-cinder | 07:07 | |
*** jwcroppe_ has quit IRC | 07:09 | |
*** alonma has quit IRC | 07:12 | |
*** alonma has joined #openstack-cinder | 07:14 | |
*** coolsvap is now known as coolsvap|away | 07:16 | |
avishay | alonma: Hey. To connect to XIV properly, do I need to install the host tools? | 07:17 |
*** alonma has quit IRC | 07:18 | |
*** liverpooler has joined #openstack-cinder | 07:20 | |
*** alonma has joined #openstack-cinder | 07:22 | |
*** nkrinner has joined #openstack-cinder | 07:23 | |
*** anshul has joined #openstack-cinder | 07:25 | |
*** alonma has quit IRC | 07:26 | |
*** alonma has joined #openstack-cinder | 07:28 | |
*** sileht_ has joined #openstack-cinder | 07:29 | |
*** zul has joined #openstack-cinder | 07:29 | |
*** alonma has quit IRC | 07:33 | |
*** alonma has joined #openstack-cinder | 07:34 | |
openstackgerrit | wuyuting proposed openstack/cinder: Add index for reservations on (deleted, uuid) https://review.openstack.org/275047 | 07:34 |
*** janice has joined #openstack-cinder | 07:34 | |
*** zhonghua-lee has quit IRC | 07:35 | |
*** sileht_ is now known as sileht | 07:35 | |
*** zhonghua-lee has joined #openstack-cinder | 07:35 | |
*** alonma has quit IRC | 07:38 | |
*** alonma has joined #openstack-cinder | 07:40 | |
*** zul has quit IRC | 07:41 | |
*** belmoreira has joined #openstack-cinder | 07:44 | |
openstackgerrit | LisaLi proposed openstack/cinder: Update quota when volume type renames. https://review.openstack.org/267999 | 07:44 |
*** alonma has quit IRC | 07:45 | |
*** alonma has joined #openstack-cinder | 07:46 | |
openstackgerrit | wanghao proposed openstack/cinder: Uploading volume to specific image https://review.openstack.org/270073 | 07:49 |
*** sileht has quit IRC | 07:50 | |
*** sileht has joined #openstack-cinder | 07:50 | |
*** alonma has quit IRC | 07:50 | |
*** alonma has joined #openstack-cinder | 07:52 | |
*** alonma has quit IRC | 07:57 | |
*** alonma has joined #openstack-cinder | 07:58 | |
patrickeast | bkumar: i just tested the image cache with lvm and pure backends and had no issues on the latest master | 08:00 |
*** haomaiwang has quit IRC | 08:01 | |
patrickeast | bkumar: whats the sha id for the revision you are seeing the problem on? | 08:01 |
*** haomaiwang has joined #openstack-cinder | 08:02 | |
*** alonma has quit IRC | 08:03 | |
*** zhonghua-lee has quit IRC | 08:04 | |
*** zhonghua-lee has joined #openstack-cinder | 08:05 | |
*** alonma has joined #openstack-cinder | 08:07 | |
*** jistr has joined #openstack-cinder | 08:11 | |
*** alonma has quit IRC | 08:11 | |
*** ildikov has quit IRC | 08:13 | |
*** alonma has joined #openstack-cinder | 08:13 | |
*** vivekd has quit IRC | 08:16 | |
*** boris-42 has joined #openstack-cinder | 08:17 | |
*** alonma has quit IRC | 08:17 | |
*** markvoelker has joined #openstack-cinder | 08:18 | |
*** alonma has joined #openstack-cinder | 08:19 | |
*** vivekd has joined #openstack-cinder | 08:22 | |
*** markvoelker has quit IRC | 08:22 | |
*** sinese has joined #openstack-cinder | 08:25 | |
*** arecknag has joined #openstack-cinder | 08:26 | |
*** tobe has joined #openstack-cinder | 08:28 | |
*** e0ne has joined #openstack-cinder | 08:38 | |
*** chhavi has joined #openstack-cinder | 08:39 | |
*** yhayashi has quit IRC | 08:43 | |
*** vivekd has quit IRC | 08:43 | |
*** sgotliv has joined #openstack-cinder | 08:45 | |
*** vivekd has joined #openstack-cinder | 08:48 | |
*** zul has joined #openstack-cinder | 08:56 | |
*** ndipanov has joined #openstack-cinder | 08:58 | |
*** ndipanov has quit IRC | 08:59 | |
openstackgerrit | wanghao proposed openstack/cinder-specs: Uploading volume to specific image https://review.openstack.org/258310 | 09:00 |
*** haomaiwang has quit IRC | 09:01 | |
*** Rust has joined #openstack-cinder | 09:01 | |
*** haomaiwa_ has joined #openstack-cinder | 09:01 | |
*** Rust is now known as Guest6527 | 09:02 | |
*** zhangjn has joined #openstack-cinder | 09:04 | |
*** zhangjn has quit IRC | 09:04 | |
openstackgerrit | Sheel Rana proposed openstack/python-cinderclient: is_public=N/A in output of cinder type-update https://review.openstack.org/274429 | 09:05 |
*** Guest6527 has quit IRC | 09:06 | |
*** avishay has quit IRC | 09:07 | |
*** lpetrut has joined #openstack-cinder | 09:07 | |
*** zhangjn has joined #openstack-cinder | 09:09 | |
*** vgridnev has joined #openstack-cinder | 09:10 | |
*** houming has quit IRC | 09:13 | |
*** jordanP has joined #openstack-cinder | 09:15 | |
*** janice has quit IRC | 09:15 | |
*** tobe has quit IRC | 09:16 | |
*** qwebirc92791 has quit IRC | 09:20 | |
*** ildikov has joined #openstack-cinder | 09:20 | |
*** xiaohui has quit IRC | 09:23 | |
*** xiaohui has joined #openstack-cinder | 09:23 | |
openstackgerrit | Jordan Pittier proposed openstack/cinder: Workaround for python-novaclient not supporting Keystone v3 https://review.openstack.org/274790 | 09:25 |
*** qwebirc6054 has joined #openstack-cinder | 09:26 | |
*** EinstCra_ has quit IRC | 09:28 | |
*** dalgaaf has quit IRC | 09:28 | |
*** EinstCrazy has joined #openstack-cinder | 09:28 | |
*** vivekd has quit IRC | 09:33 | |
*** vivekd_ has joined #openstack-cinder | 09:33 | |
*** vivekd_ is now known as vivekd | 09:33 | |
*** andymaier has joined #openstack-cinder | 09:34 | |
*** avishay has joined #openstack-cinder | 09:35 | |
*** markd_ has joined #openstack-cinder | 09:40 | |
*** mdavidson has quit IRC | 09:41 | |
*** grumpycatt has quit IRC | 09:41 | |
*** markd_ has quit IRC | 09:41 | |
*** mdavidson has joined #openstack-cinder | 09:41 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix service-list filter https://review.openstack.org/263257 | 09:43 |
*** baojg has quit IRC | 09:44 | |
*** baojg has joined #openstack-cinder | 09:45 | |
*** grumpycatt has joined #openstack-cinder | 09:47 | |
*** baojg has quit IRC | 09:49 | |
*** qeelee has quit IRC | 09:50 | |
*** davechen has left #openstack-cinder | 09:54 | |
*** geguileo has joined #openstack-cinder | 09:57 | |
*** zhangjn has quit IRC | 09:57 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add description when rename LUN in Huawei driver https://review.openstack.org/271603 | 09:57 |
*** haomaiwa_ has quit IRC | 10:01 | |
*** cheneydc has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-cinder | 10:01 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Don't fail when port group does not exist https://review.openstack.org/271599 | 10:01 |
*** vivekd has quit IRC | 10:03 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add description when rename LUN in Huawei driver https://review.openstack.org/271603 | 10:03 |
*** mvk has joined #openstack-cinder | 10:05 | |
*** shyh has joined #openstack-cinder | 10:07 | |
*** zhangjn has joined #openstack-cinder | 10:11 | |
*** tpsilva has joined #openstack-cinder | 10:12 | |
*** grumpycatt has quit IRC | 10:14 | |
sheel | Hello All, please find some time to review https://review.openstack.org/#/c/274429/ | 10:14 |
*** openstackgerrit has quit IRC | 10:17 | |
*** openstackgerrit has joined #openstack-cinder | 10:18 | |
*** markvoelker has joined #openstack-cinder | 10:19 | |
*** markvoelker has quit IRC | 10:23 | |
*** huangtianhua has quit IRC | 10:27 | |
*** tiantian has joined #openstack-cinder | 10:29 | |
*** grumpycatt has joined #openstack-cinder | 10:31 | |
*** vivekd has joined #openstack-cinder | 10:33 | |
*** sgotliv has quit IRC | 10:33 | |
*** EinstCrazy has quit IRC | 10:41 | |
*** EinstCrazy has joined #openstack-cinder | 10:42 | |
*** sgotliv has joined #openstack-cinder | 10:48 | |
*** EinstCrazy has quit IRC | 10:50 | |
*** ildikov has quit IRC | 10:51 | |
*** alonma has quit IRC | 10:53 | |
*** alonma has joined #openstack-cinder | 10:53 | |
*** zhangjn has quit IRC | 10:57 | |
*** EinstCrazy has joined #openstack-cinder | 10:57 | |
*** alonma has quit IRC | 10:57 | |
*** ociuhandu has quit IRC | 10:58 | |
*** zhangjn has joined #openstack-cinder | 10:58 | |
*** zhangjn has quit IRC | 10:59 | |
*** alonma has joined #openstack-cinder | 10:59 | |
*** zhangjn has joined #openstack-cinder | 10:59 | |
*** zhangjn has quit IRC | 11:00 | |
*** zhangjn has joined #openstack-cinder | 11:00 | |
*** zhangjn has quit IRC | 11:01 | |
*** haomaiwang has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-cinder | 11:01 | |
*** wilson-1 has joined #openstack-cinder | 11:02 | |
*** dims_ has joined #openstack-cinder | 11:02 | |
*** wilson-liu has quit IRC | 11:03 | |
*** alonma has quit IRC | 11:03 | |
*** [1]Thelo has joined #openstack-cinder | 11:04 | |
*** zhangjn has joined #openstack-cinder | 11:06 | |
*** zhangjn has quit IRC | 11:06 | |
*** Thelo has quit IRC | 11:07 | |
*** [1]Thelo is now known as Thelo | 11:07 | |
*** lixiaoy1 has quit IRC | 11:07 | |
*** alonma has joined #openstack-cinder | 11:08 | |
*** fthiagogv has quit IRC | 11:09 | |
*** alonma has quit IRC | 11:13 | |
*** zhangjn has joined #openstack-cinder | 11:13 | |
*** zhangjn has quit IRC | 11:13 | |
*** zhangjn has joined #openstack-cinder | 11:14 | |
*** zhangjn has quit IRC | 11:14 | |
*** alonma has joined #openstack-cinder | 11:15 | |
*** zhangjn has joined #openstack-cinder | 11:15 | |
*** zhangjn has quit IRC | 11:15 | |
*** zhangjn has joined #openstack-cinder | 11:16 | |
*** zhangjn has quit IRC | 11:16 | |
*** zhangjn has joined #openstack-cinder | 11:17 | |
*** zhangjn has quit IRC | 11:17 | |
*** lprice1 has quit IRC | 11:18 | |
*** lprice has joined #openstack-cinder | 11:19 | |
*** ildikov has joined #openstack-cinder | 11:19 | |
*** zhangjn has joined #openstack-cinder | 11:19 | |
*** alonma has quit IRC | 11:19 | |
*** zhangjn has quit IRC | 11:19 | |
*** zhangjn has joined #openstack-cinder | 11:20 | |
*** zhangjn has quit IRC | 11:20 | |
*** alonma has joined #openstack-cinder | 11:21 | |
*** zhangjn has joined #openstack-cinder | 11:21 | |
*** zhangjn has quit IRC | 11:21 | |
*** alonma has quit IRC | 11:22 | |
*** jistr has quit IRC | 11:22 | |
*** alonma has joined #openstack-cinder | 11:22 | |
*** zhangjn has joined #openstack-cinder | 11:24 | |
*** zhangjn has quit IRC | 11:24 | |
*** zhangjn has joined #openstack-cinder | 11:25 | |
*** qwebirc6054 has quit IRC | 11:26 | |
*** fthiagogv has joined #openstack-cinder | 11:33 | |
*** sheel has quit IRC | 11:33 | |
*** ociuhandu has joined #openstack-cinder | 11:34 | |
*** esp has joined #openstack-cinder | 11:46 | |
*** gus_ is now known as gus | 11:47 | |
*** esp has quit IRC | 11:53 | |
*** dongwenjuan has joined #openstack-cinder | 12:00 | |
*** jwcroppe has joined #openstack-cinder | 12:00 | |
*** haomaiwang has quit IRC | 12:01 | |
*** haomaiwang has joined #openstack-cinder | 12:01 | |
*** geguileo has quit IRC | 12:04 | |
*** avishay has quit IRC | 12:05 | |
*** dalgaaf has joined #openstack-cinder | 12:08 | |
*** raildo-afk is now known as raildo | 12:12 | |
*** andymaier has quit IRC | 12:15 | |
*** smoriya_ has quit IRC | 12:15 | |
*** avishay has joined #openstack-cinder | 12:18 | |
*** markvoelker has joined #openstack-cinder | 12:20 | |
*** markvoelker has quit IRC | 12:24 | |
*** fthiagogv has quit IRC | 12:26 | |
*** fthiagogv has joined #openstack-cinder | 12:27 | |
*** julim_ has joined #openstack-cinder | 12:34 | |
*** j_king_ has joined #openstack-cinder | 12:35 | |
*** nug has quit IRC | 12:36 | |
*** j_king has quit IRC | 12:36 | |
*** cFouts is now known as nug | 12:36 | |
*** julim has quit IRC | 12:36 | |
*** DuncanT has quit IRC | 12:36 | |
*** links has quit IRC | 12:36 | |
*** nug is now known as Guest67795 | 12:36 | |
*** bswartz has quit IRC | 12:36 | |
*** alonma has quit IRC | 12:37 | |
*** kmartin__ has quit IRC | 12:37 | |
*** DuncanT has joined #openstack-cinder | 12:38 | |
*** gnarld_ has joined #openstack-cinder | 12:38 | |
*** kmartin__ has joined #openstack-cinder | 12:38 | |
*** links has joined #openstack-cinder | 12:39 | |
*** boris-42 has quit IRC | 12:43 | |
*** alonma has joined #openstack-cinder | 12:44 | |
*** cheneydc has joined #openstack-cinder | 12:47 | |
*** alonma has quit IRC | 12:48 | |
*** alonma has joined #openstack-cinder | 12:50 | |
*** esp has joined #openstack-cinder | 12:50 | |
*** andymaier has joined #openstack-cinder | 12:53 | |
*** geguileo has joined #openstack-cinder | 12:54 | |
*** jgregor has joined #openstack-cinder | 12:54 | |
*** alonma has quit IRC | 12:55 | |
*** alonma has joined #openstack-cinder | 12:56 | |
*** dongwenjuan_ has joined #openstack-cinder | 12:56 | |
*** erlon has joined #openstack-cinder | 12:57 | |
*** esp has quit IRC | 12:57 | |
*** tiantian has quit IRC | 12:57 | |
*** dongwenjuan has quit IRC | 12:58 | |
*** tiantian has joined #openstack-cinder | 12:58 | |
*** anteaya has joined #openstack-cinder | 12:59 | |
*** logan- has quit IRC | 13:00 | |
*** logan- has joined #openstack-cinder | 13:00 | |
*** alonma has quit IRC | 13:00 | |
*** dongwenjuan has joined #openstack-cinder | 13:01 | |
*** haomaiwang has quit IRC | 13:01 | |
*** haomaiwa_ has joined #openstack-cinder | 13:01 | |
*** dongwenjuan_ has quit IRC | 13:02 | |
*** alonma has joined #openstack-cinder | 13:02 | |
*** alonma has quit IRC | 13:03 | |
*** alonma has joined #openstack-cinder | 13:03 | |
*** avishay has quit IRC | 13:05 | |
*** anteaya has quit IRC | 13:06 | |
*** takedakn has joined #openstack-cinder | 13:06 | |
*** EinstCrazy has quit IRC | 13:07 | |
*** dongwenjuan_ has joined #openstack-cinder | 13:09 | |
*** EinstCrazy has joined #openstack-cinder | 13:11 | |
*** dongwenjuan has quit IRC | 13:11 | |
*** dongwenjuan_ has quit IRC | 13:12 | |
*** timcl has joined #openstack-cinder | 13:13 | |
*** dongwenjuan has joined #openstack-cinder | 13:13 | |
openstackgerrit | abhiram moturi proposed openstack/cinder: Zfssaiscsi: Implement replication v2 https://review.openstack.org/274954 | 13:15 |
*** dongwenjuan_ has joined #openstack-cinder | 13:16 | |
openstackgerrit | Matan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver https://review.openstack.org/269624 | 13:16 |
*** dongwenjuan has quit IRC | 13:18 | |
*** dongwenjuan__ has joined #openstack-cinder | 13:18 | |
*** grumpycatt has quit IRC | 13:18 | |
*** haomaiwa_ has quit IRC | 13:19 | |
*** adrianofr has joined #openstack-cinder | 13:20 | |
*** dongwenjuan_ has quit IRC | 13:20 | |
*** markvoelker has joined #openstack-cinder | 13:21 | |
*** gouthamr has joined #openstack-cinder | 13:24 | |
*** grumpycatt has joined #openstack-cinder | 13:24 | |
*** dongwenjuan__ has quit IRC | 13:24 | |
*** xyang1 has joined #openstack-cinder | 13:25 | |
*** markvoelker has quit IRC | 13:26 | |
*** markvoelker_ has joined #openstack-cinder | 13:26 | |
*** andrei__ has joined #openstack-cinder | 13:27 | |
*** dongwenjuan_ has joined #openstack-cinder | 13:31 | |
*** vivekd has quit IRC | 13:31 | |
*** vivekd has joined #openstack-cinder | 13:33 | |
*** gnarld_ is now known as nug | 13:37 | |
*** edmondsw has joined #openstack-cinder | 13:37 | |
*** dongwenjuan__ has joined #openstack-cinder | 13:38 | |
*** bkumar has quit IRC | 13:39 | |
*** bill_az has joined #openstack-cinder | 13:41 | |
*** dongwenjuan_ has quit IRC | 13:41 | |
*** andymaier has quit IRC | 13:42 | |
*** zul has quit IRC | 13:45 | |
*** lpetrut has quit IRC | 13:49 | |
*** lpetrut has joined #openstack-cinder | 13:49 | |
*** haomaiwang has joined #openstack-cinder | 13:50 | |
*** laughterwym has quit IRC | 13:51 | |
*** jwcroppe has quit IRC | 13:51 | |
*** krtaylor has quit IRC | 13:51 | |
*** laughterwym has joined #openstack-cinder | 13:51 | |
*** jwcroppe has joined #openstack-cinder | 13:52 | |
*** jwcroppe has quit IRC | 13:56 | |
*** laughterwym has quit IRC | 13:56 | |
*** jgriffith is now known as jgriffith_away | 13:58 | |
*** zul has joined #openstack-cinder | 13:59 | |
*** Julien-zte has joined #openstack-cinder | 13:59 | |
*** baumann has joined #openstack-cinder | 13:59 | |
*** itzdilip has quit IRC | 14:00 | |
*** haomaiwang has quit IRC | 14:01 | |
*** cknight has joined #openstack-cinder | 14:01 | |
*** salv-orlando has joined #openstack-cinder | 14:01 | |
*** haomaiwang has joined #openstack-cinder | 14:01 | |
*** krtaylor has joined #openstack-cinder | 14:05 | |
*** rlrossit has joined #openstack-cinder | 14:07 | |
*** changbl has joined #openstack-cinder | 14:07 | |
*** thurloat_isgone is now known as thurloat | 14:08 | |
*** shyh has quit IRC | 14:09 | |
*** martyturner has joined #openstack-cinder | 14:11 | |
*** akerr has joined #openstack-cinder | 14:16 | |
openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder: Support https keystone in volume quotas https://review.openstack.org/272437 | 14:18 |
*** edtubill has joined #openstack-cinder | 14:24 | |
*** jwcroppe has joined #openstack-cinder | 14:25 | |
*** diogogmt has quit IRC | 14:26 | |
*** cheneydc has quit IRC | 14:26 | |
*** sheel has joined #openstack-cinder | 14:28 | |
*** EinstCrazy has quit IRC | 14:29 | |
*** grumpycatt has quit IRC | 14:30 | |
*** dongwenjuan_ has joined #openstack-cinder | 14:30 | |
*** jistr has joined #openstack-cinder | 14:31 | |
sheel | DuncanT: Hi | 14:31 |
*** kmartin__ has quit IRC | 14:31 | |
sheel | DuncanT: are you there? | 14:32 |
*** amoturi has joined #openstack-cinder | 14:33 | |
*** zul has quit IRC | 14:33 | |
sheel | smcginnis: Hi there | 14:33 |
scottda | dulek: Hi | 14:34 |
*** dongwenjuan__ has quit IRC | 14:34 | |
scottda | dulek: I'm looking at https://bugs.launchpad.net/cinder/+bug/1518513 and seeing it on our stable/liberty based branch | 14:34 |
openstack | Launchpad bug 1518513 in Cinder "Quota update call fails in Liberty" [Medium,In progress] - Assigned to Dan Nguyen (daniel-a-nguyen) | 14:34 |
*** thangp has joined #openstack-cinder | 14:35 | |
*** grumpycatt has joined #openstack-cinder | 14:35 | |
scottda | dulek: I'm curious if your repro was stable/liberty for both Cinder and Keystone? | 14:36 |
*** dustins has joined #openstack-cinder | 14:36 | |
*** cfriesen has joined #openstack-cinder | 14:36 | |
smcginnis | sheel: Hey, here now. | 14:40 |
yuriy_n17 | geguileo: Sorry for disturbing. Please review https://review.openstack.org/#/c/263257/ whenever you find free time. Thanks a lot for advance. | 14:40 |
*** mc_nair has quit IRC | 14:41 | |
sheel | smcginnis: you always use to be there for all calls.. really great lead.. | 14:41 |
amoturi | Hi what is the latest for replication? i sent out a patch yesterday(https://review.openstack.org/#/c/274954/) But it seems there is another implementation. | 14:42 |
smcginnis | sheel: Hah, thanks. | 14:42 |
*** mriedem has joined #openstack-cinder | 14:43 | |
sheel | smcginnis: thanks to you.. | 14:43 |
sheel | smcginnis: actually I wanted your some time for review on "https://review.openstack.org/#/c/274429/" | 14:43 |
smcginnis | amoturi: Yeah, I need to apologize for that. At the midcycle we had a lot of discussion on replication. | 14:43 |
smcginnis | amoturi: We decided to change course before it's too late. | 14:43 |
smcginnis | amoturi: And simplify things first. | 14:43 |
sheel | and https://review.openstack.org/#/c/272229/ | 14:43 |
smcginnis | amoturi: So I think what you have now probably won't need to change too much. | 14:43 |
smcginnis | amoturi: But there will be some change. | 14:44 |
smcginnis | sheel: Got it, open in a tab now, so I'll try to get to reviewing it a little later. Just need to catch up on some things first. | 14:44 |
sheel | smcginnis: sure, please take your time... | 14:44 |
smcginnis | amoturi: jgriffith_away is working on some updates. | 14:44 |
amoturi | smcginnis: Sure. i wouldn't mind making changes to make it simpler. But i see that the blueprint for the new implementation was just approved 4 days ago. when i can expect it to go in? | 14:45 |
*** ankit_ag has quit IRC | 14:45 | |
smcginnis | amoturi: I plan on bringing it up in tomorrow's meeting if you are around to help spread the word and awareness that things are changing. | 14:45 |
amoturi | smcginnis: also since the deadline is march 19th (i think?) can my patch make it by then? | 14:45 |
smcginnis | amoturi: I know he is working on some code right now. Hopefully this week. | 14:45 |
smcginnis | amoturi: Yes, I think you should have enough time. You'll have support from the other vendors that need to re-reimplement replication. ;)\ | 14:46 |
*** lprice has quit IRC | 14:46 | |
*** wanghao_ has joined #openstack-cinder | 14:47 | |
*** dongwenjuan__ has joined #openstack-cinder | 14:47 | |
amoturi | smcginnis: nope i was wrong mitaka 3 is march 1st. Ok perfect. | 14:47 |
*** dongwenjuan_ has quit IRC | 14:47 | |
amoturi | smcginnis: and since i can help review theirs if they review mine :) | 14:47 |
smcginnis | amoturi: Yeah, we'll have to work together on this one. I don't want anyone left behind because of this. | 14:49 |
amoturi | smcginnis: great. thanks for the support. I'll attend tomorrow's meeting as well | 14:50 |
*** cheneydc has joined #openstack-cinder | 14:52 | |
*** mvk has quit IRC | 14:56 | |
*** zul has joined #openstack-cinder | 14:57 | |
*** mdavidson has quit IRC | 14:59 | |
*** jwcroppe has quit IRC | 14:59 | |
*** zhangjn has quit IRC | 15:00 | |
*** esker has joined #openstack-cinder | 15:00 | |
*** haomaiwang has quit IRC | 15:01 | |
*** haomaiwa_ has joined #openstack-cinder | 15:01 | |
*** krtaylor has quit IRC | 15:03 | |
*** merooney has joined #openstack-cinder | 15:03 | |
*** Julien-z_ has joined #openstack-cinder | 15:03 | |
*** Julien-zte has quit IRC | 15:06 | |
*** fthiagogv has quit IRC | 15:06 | |
*** fthiagogv has joined #openstack-cinder | 15:07 | |
*** jwcroppe has joined #openstack-cinder | 15:09 | |
*** zul has quit IRC | 15:10 | |
*** sheel has quit IRC | 15:11 | |
*** rajinir has joined #openstack-cinder | 15:11 | |
*** links has quit IRC | 15:11 | |
*** zul has joined #openstack-cinder | 15:12 | |
*** laurent\_ is now known as laurent\ | 15:12 | |
*** wilson-liu has joined #openstack-cinder | 15:13 | |
*** dongwenjuan__ has quit IRC | 15:14 | |
*** krtaylor has joined #openstack-cinder | 15:15 | |
*** eharney has joined #openstack-cinder | 15:15 | |
*** lprice has joined #openstack-cinder | 15:16 | |
*** wilson-1 has quit IRC | 15:17 | |
*** mc_nair has joined #openstack-cinder | 15:17 | |
*** zul has quit IRC | 15:18 | |
*** bswartz has joined #openstack-cinder | 15:20 | |
*** merooney has quit IRC | 15:21 | |
*** sheel has joined #openstack-cinder | 15:22 | |
*** alonma has quit IRC | 15:22 | |
*** takedakn has quit IRC | 15:24 | |
*** merooney has joined #openstack-cinder | 15:26 | |
*** mtanino has joined #openstack-cinder | 15:27 | |
mtanino | geguileo: hello, | 15:28 |
*** alonma has joined #openstack-cinder | 15:29 | |
mtanino | geguileo: Can we continue to discuss LIO target problem fix? https://review.openstack.org/#/c/271424/ | 15:29 |
*** EinstCrazy has joined #openstack-cinder | 15:29 | |
geguileo | mtanino: This week I'm in a training | 15:29 |
mtanino | geguileo: Ah, ok. see you next week :) | 15:30 |
*** zul has joined #openstack-cinder | 15:30 | |
*** alonma has quit IRC | 15:33 | |
*** chhavi has quit IRC | 15:34 | |
*** alonma has joined #openstack-cinder | 15:35 | |
*** EinstCrazy has quit IRC | 15:35 | |
*** zul has quit IRC | 15:36 | |
*** crose has joined #openstack-cinder | 15:37 | |
*** alonma has quit IRC | 15:39 | |
*** alonma has joined #openstack-cinder | 15:41 | |
*** JoseMello has joined #openstack-cinder | 15:43 | |
*** alonma has quit IRC | 15:45 | |
mc_nair | DuncanT: can you send a link to the approach you mentioned at the midcycle where a user's encryption key was getting passed with every attach/detach call as part of the header so Cinder wouldn't need to manage it (Google had prototyped it)? | 15:46 |
*** esp has joined #openstack-cinder | 15:46 | |
openstackgerrit | Alex Meade proposed openstack/cinder-specs: User facing error messages https://review.openstack.org/274958 | 15:46 |
*** bill_az has quit IRC | 15:47 | |
*** alonma has joined #openstack-cinder | 15:47 | |
sheel | ameade:hi | 15:50 |
*** esp has quit IRC | 15:50 | |
sheel | ameade: please let me know when can we discuss about midcycle points for asynchronous error messages? | 15:51 |
*** alonma has quit IRC | 15:52 | |
*** bill_az has joined #openstack-cinder | 15:52 | |
*** alonma has joined #openstack-cinder | 15:53 | |
*** dims has joined #openstack-cinder | 15:54 | |
*** dims_ has quit IRC | 15:54 | |
*** amoturi has quit IRC | 15:56 | |
*** amoturi has joined #openstack-cinder | 15:57 | |
*** mgarza has joined #openstack-cinder | 15:57 | |
*** alonma has quit IRC | 15:58 | |
*** cebruns has quit IRC | 15:59 | |
*** garthb has joined #openstack-cinder | 15:59 | |
*** arch-nemesis has joined #openstack-cinder | 15:59 | |
*** cebruns has joined #openstack-cinder | 16:00 | |
*** alonma has joined #openstack-cinder | 16:00 | |
*** haomaiwa_ has quit IRC | 16:01 | |
*** 6JTAAC8GZ has joined #openstack-cinder | 16:01 | |
*** david-lyle has joined #openstack-cinder | 16:02 | |
*** diablo_rojo has joined #openstack-cinder | 16:03 | |
*** nkrinner has quit IRC | 16:04 | |
*** garthb has quit IRC | 16:04 | |
*** alonma has quit IRC | 16:05 | |
*** merooney has quit IRC | 16:06 | |
*** geguileo has quit IRC | 16:06 | |
*** anshul has quit IRC | 16:06 | |
*** sgotliv has quit IRC | 16:07 | |
*** cheneydc has quit IRC | 16:08 | |
*** Julien-z_ has quit IRC | 16:10 | |
*** alonma has joined #openstack-cinder | 16:11 | |
*** dustins has quit IRC | 16:13 | |
*** sinese has quit IRC | 16:13 | |
*** vgridnev has quit IRC | 16:14 | |
*** alonma has quit IRC | 16:16 | |
*** alonma has joined #openstack-cinder | 16:19 | |
*** pradipm has joined #openstack-cinder | 16:19 | |
*** alonma has quit IRC | 16:23 | |
*** dustins has joined #openstack-cinder | 16:24 | |
*** belmoreira has quit IRC | 16:25 | |
*** alonma has joined #openstack-cinder | 16:25 | |
*** alonma has quit IRC | 16:29 | |
*** mdenny has quit IRC | 16:29 | |
*** daneyon_ has joined #openstack-cinder | 16:29 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Permit volume type operations for policy authorized users https://review.openstack.org/274204 | 16:31 |
*** alonma has joined #openstack-cinder | 16:31 | |
*** garthb has joined #openstack-cinder | 16:32 | |
*** edtubill has quit IRC | 16:32 | |
*** daneyon has quit IRC | 16:33 | |
*** tiagogomes_ has left #openstack-cinder | 16:34 | |
*** alonma has quit IRC | 16:36 | |
*** gouthamr has quit IRC | 16:36 | |
*** daneyon has joined #openstack-cinder | 16:37 | |
*** dustins_ has joined #openstack-cinder | 16:37 | |
*** kmartin has quit IRC | 16:37 | |
*** alonma has joined #openstack-cinder | 16:38 | |
*** dustins has quit IRC | 16:38 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Revert "Allow host and instance_uuid at attach" https://review.openstack.org/275316 | 16:40 |
*** daneyon_ has quit IRC | 16:40 | |
*** timcl has quit IRC | 16:41 | |
*** alonma has quit IRC | 16:42 | |
*** esker has quit IRC | 16:42 | |
*** gouthamr has joined #openstack-cinder | 16:43 | |
*** timcl has joined #openstack-cinder | 16:44 | |
*** mrrowden has joined #openstack-cinder | 16:46 | |
*** dims_ has joined #openstack-cinder | 16:47 | |
*** lpetrut has quit IRC | 16:48 | |
*** dims has quit IRC | 16:49 | |
*** jistr has quit IRC | 16:53 | |
*** arecknag has quit IRC | 16:54 | |
*** mvk has joined #openstack-cinder | 16:54 | |
*** e0ne has quit IRC | 16:55 | |
*** jdurgin1 has joined #openstack-cinder | 16:56 | |
*** wanghao_ has quit IRC | 16:57 | |
*** vivekd has quit IRC | 17:00 | |
*** rlrossit_ has joined #openstack-cinder | 17:00 | |
*** 6JTAAC8GZ has quit IRC | 17:01 | |
*** haomaiwa_ has joined #openstack-cinder | 17:01 | |
*** vivekd has joined #openstack-cinder | 17:02 | |
*** rlrossit has quit IRC | 17:03 | |
*** sheel has quit IRC | 17:12 | |
*** salv-orlando has quit IRC | 17:14 | |
*** sinese has joined #openstack-cinder | 17:17 | |
*** ntpttr has joined #openstack-cinder | 17:17 | |
*** mdenny has joined #openstack-cinder | 17:17 | |
*** dalgaaf has quit IRC | 17:18 | |
*** zul has joined #openstack-cinder | 17:19 | |
*** bkumar has joined #openstack-cinder | 17:25 | |
*** bkumar has quit IRC | 17:26 | |
*** salv-orlando has joined #openstack-cinder | 17:26 | |
*** pradipm1 has joined #openstack-cinder | 17:26 | |
*** pradipm has quit IRC | 17:27 | |
*** pradipm1 has quit IRC | 17:28 | |
*** mgarza has quit IRC | 17:30 | |
*** martyturner has quit IRC | 17:30 | |
*** openstackgerrit has quit IRC | 17:32 | |
*** mgarza_ has joined #openstack-cinder | 17:33 | |
*** EinstCrazy has joined #openstack-cinder | 17:33 | |
*** openstackgerrit has joined #openstack-cinder | 17:33 | |
*** jdurgin1 has quit IRC | 17:33 | |
*** dalgaaf has joined #openstack-cinder | 17:34 | |
*** martyturner has joined #openstack-cinder | 17:35 | |
*** Apoorva has joined #openstack-cinder | 17:35 | |
*** leeantho has joined #openstack-cinder | 17:36 | |
*** alonma has joined #openstack-cinder | 17:38 | |
*** EinstCrazy has quit IRC | 17:38 | |
*** shyama has joined #openstack-cinder | 17:39 | |
*** porrua has quit IRC | 17:41 | |
*** leeantho has quit IRC | 17:41 | |
*** alonma has quit IRC | 17:43 | |
*** alonma has joined #openstack-cinder | 17:44 | |
*** sinese has quit IRC | 17:44 | |
*** sinese has joined #openstack-cinder | 17:44 | |
*** alonma has quit IRC | 17:48 | |
*** wilson-liu has quit IRC | 17:48 | |
*** wilson-liu has joined #openstack-cinder | 17:49 | |
*** salv-orl_ has joined #openstack-cinder | 17:50 | |
*** alonma has joined #openstack-cinder | 17:50 | |
*** salv-orlando has quit IRC | 17:54 | |
*** edtubill has joined #openstack-cinder | 17:55 | |
*** alonma has quit IRC | 17:55 | |
*** zul has quit IRC | 17:56 | |
*** alonma has joined #openstack-cinder | 17:57 | |
*** haomaiwa_ has quit IRC | 18:01 | |
*** alonma has quit IRC | 18:01 | |
*** merooney has joined #openstack-cinder | 18:01 | |
*** haomaiwang has joined #openstack-cinder | 18:01 | |
*** simondodsley has joined #openstack-cinder | 18:03 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 18:03 |
*** alonma has joined #openstack-cinder | 18:06 | |
*** porrua has joined #openstack-cinder | 18:09 | |
*** alonma has quit IRC | 18:10 | |
vivekd | DuncanT: could you please approve this? https://review.openstack.org/271848 | 18:10 |
*** bwallis has quit IRC | 18:10 | |
*** alonma has joined #openstack-cinder | 18:12 | |
*** chris_morrell has joined #openstack-cinder | 18:12 | |
*** jordanP has quit IRC | 18:12 | |
*** anteaya has joined #openstack-cinder | 18:14 | |
openstackgerrit | Alex Meade proposed openstack/cinder-specs: User facing error messages https://review.openstack.org/274958 | 18:15 |
ameade | eharney, DuncanT: ^^ | 18:15 |
ameade | fairly short spec | 18:15 |
ameade | i'll get an implementation up in a couple days | 18:16 |
smcginnis | ameade: That's to address the API wg spec, right? | 18:16 |
smcginnis | ameade: Oh, nevermind. | 18:16 |
smcginnis | Just saw the other activity, then this one. | 18:16 |
ameade | smcginnis: i was SO CONFUSED, lol | 18:16 |
smcginnis | Thought you were really quick on that. :) | 18:16 |
*** alonma has quit IRC | 18:17 | |
*** merooney has quit IRC | 18:17 | |
*** shyama has quit IRC | 18:26 | |
*** salv-orl_ has quit IRC | 18:28 | |
*** salv-orlando has joined #openstack-cinder | 18:28 | |
*** kfarr has joined #openstack-cinder | 18:28 | |
*** vivekd has quit IRC | 18:29 | |
*** merooney has joined #openstack-cinder | 18:29 | |
*** ChubYann has joined #openstack-cinder | 18:30 | |
*** bwallis has joined #openstack-cinder | 18:30 | |
*** salv-orlando has quit IRC | 18:31 | |
*** salv-orlando has joined #openstack-cinder | 18:32 | |
*** bardia has joined #openstack-cinder | 18:38 | |
*** haomaiwang has quit IRC | 18:40 | |
*** salv-orlando has quit IRC | 18:43 | |
*** zul has joined #openstack-cinder | 18:45 | |
*** salv-orlando has joined #openstack-cinder | 18:45 | |
*** garthb_ has joined #openstack-cinder | 18:52 | |
*** garthb has quit IRC | 18:52 | |
*** martyturner has quit IRC | 18:53 | |
*** changbl has quit IRC | 18:54 | |
*** changbl has joined #openstack-cinder | 18:55 | |
*** garthb_ has quit IRC | 18:57 | |
*** martyturner has joined #openstack-cinder | 18:58 | |
*** merooney has quit IRC | 18:58 | |
*** garthb_ has joined #openstack-cinder | 19:00 | |
*** merooney has joined #openstack-cinder | 19:01 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume https://review.openstack.org/250220 | 19:03 |
*** whydidyoustealmy has joined #openstack-cinder | 19:03 | |
*** jdurgin has quit IRC | 19:05 | |
*** esker has joined #openstack-cinder | 19:05 | |
*** asselin_ has joined #openstack-cinder | 19:05 | |
*** jdurgin has joined #openstack-cinder | 19:05 | |
*** thangp_ has joined #openstack-cinder | 19:05 | |
*** doffm_ has joined #openstack-cinder | 19:07 | |
*** jdurgin has quit IRC | 19:09 | |
*** leeantho has joined #openstack-cinder | 19:10 | |
*** sb- has joined #openstack-cinder | 19:11 | |
*** bswartz has quit IRC | 19:12 | |
*** thangp has quit IRC | 19:12 | |
*** Kimmo_ has quit IRC | 19:12 | |
*** wanghao has quit IRC | 19:12 | |
*** hogepodge has quit IRC | 19:12 | |
*** strictlyb has quit IRC | 19:12 | |
*** barra204 has quit IRC | 19:12 | |
*** asselin has quit IRC | 19:12 | |
*** kbyrne has quit IRC | 19:12 | |
*** mordred has quit IRC | 19:12 | |
*** doffm has quit IRC | 19:12 | |
*** thingee has quit IRC | 19:12 | |
*** thangp_ is now known as thangp | 19:12 | |
*** alonma has joined #openstack-cinder | 19:12 | |
*** wanghao has joined #openstack-cinder | 19:12 | |
*** kbyrne has joined #openstack-cinder | 19:15 | |
*** mordred has joined #openstack-cinder | 19:15 | |
*** alonma has quit IRC | 19:17 | |
*** boris-42 has joined #openstack-cinder | 19:19 | |
*** angela-s has joined #openstack-cinder | 19:21 | |
*** hogepodge has joined #openstack-cinder | 19:24 | |
*** salv-orlando has quit IRC | 19:25 | |
*** thingee has joined #openstack-cinder | 19:26 | |
*** alonma has joined #openstack-cinder | 19:26 | |
*** esp_ has joined #openstack-cinder | 19:27 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror) https://review.openstack.org/237387 | 19:29 |
*** bswartz has joined #openstack-cinder | 19:30 | |
*** jgregor has left #openstack-cinder | 19:30 | |
*** alonma has quit IRC | 19:30 | |
ildikov | hemna: smcginnis: hi. are you around? | 19:30 |
*** jgregor has joined #openstack-cinder | 19:30 | |
*** mgarza_ has quit IRC | 19:31 | |
*** jdurgin has joined #openstack-cinder | 19:31 | |
hemna | hey | 19:34 |
*** esp_ has quit IRC | 19:34 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication https://review.openstack.org/237387 | 19:35 |
*** vgridnev has joined #openstack-cinder | 19:36 | |
ildikov | hemna: hey, just wanted to check whether we have a plan for multiattach regarding addressing the issues we identified so far | 19:36 |
ildikov | I hoped we can merge more patches, but I still need ot improve my karma it seems... | 19:38 |
hemna | so, I wasn't sure where we ended up really | 19:43 |
*** EinstCrazy has joined #openstack-cinder | 19:43 | |
hemna | I think I had to revert my host cinder patch that allowed the host name along with instance_uuid | 19:43 |
hemna | and everything was going to be disabled | 19:43 |
hemna | ? | 19:44 |
hemna | I was kinda confused by it all TBH | 19:44 |
smcginnis | I'm back... | 19:44 |
*** thurloat is now known as thurloat_isgone | 19:44 | |
smcginnis | ildikov, hemna: So I think the nova patch needed to be reverted. | 19:44 |
smcginnis | The cinder one could be, but I don't think that part was critical. | 19:44 |
smcginnis | The main problem was nova sending both to an older cinder service that enforced NOT having both. | 19:45 |
ildikov | smcginnis: no, the Cinder part is not critical | 19:45 |
hemna | ok I wasn't sure, we have it in our notes that I was going to revert the cinder patch | 19:45 |
*** huangtianhua has joined #openstack-cinder | 19:45 | |
smcginnis | Making that a backwards incompatible thing. | 19:45 |
smcginnis | hemna: Yeah, we did talk about that. | 19:45 |
ildikov | I don't know why that check was introduced there, but for now the Cinder side patch does not cause any issue | 19:45 |
smcginnis | But thinking more, not sure if we absolutely have to. | 19:45 |
hemna | which is why I posted this one : https://review.openstack.org/#/c/275316/ | 19:45 |
hemna | we could re-add it after as a microversion | 19:46 |
ildikov | and the Nova patch caused issue "only" for upgrade scenarios and mixed environments (new Nova - old Cinder) | 19:46 |
*** merooeny_ has joined #openstack-cinder | 19:46 | |
ildikov | does Nova have to pass the host_name to Cinder? | 19:46 |
ildikov | I think that's the fundamental question here as if yes, then we either need to keep that Cinder patch or figure out something else | 19:47 |
hemna | so... | 19:47 |
ildikov | my concern still is that how to make detach work? | 19:47 |
hemna | it's nice to have that host | 19:47 |
*** bwallis_ has joined #openstack-cinder | 19:47 | |
hemna | because then cinder can determine how many volumes are attached to that host | 19:48 |
smcginnis | ildikov: So I think if it's just possible to change that nova handling to try both, and if that fails, try failing back to just one, that would at least get by the backwards incompatible piece. | 19:48 |
hemna | as well as how many times a volume is attached to a host | 19:48 |
*** EinstCrazy has quit IRC | 19:48 | |
*** mvk_ has joined #openstack-cinder | 19:48 | |
ildikov | smcginnis: you mean Nova should check which Cinder it talks to? | 19:48 |
smcginnis | ildikov: Either check, or instead of failing out when it gets an error back from sending both host and instance_uuid, retry with just one or the other. Whichever it sent before it was changed. | 19:49 |
ildikov | hemna: live migration will mess up things | 19:49 |
hemna | it always does :( | 19:49 |
ildikov | hemna: although if we disable it for multiattach volume this info still can be used | 19:49 |
*** simondodsley_ has joined #openstack-cinder | 19:50 | |
hemna | I think the problem was | 19:50 |
hemna | that if we get a new nova (Mitaka), before upgrading Cinder, Cinder will puke | 19:51 |
*** bwallis has quit IRC | 19:51 | |
*** simondodsley has quit IRC | 19:51 | |
*** mvk has quit IRC | 19:51 | |
*** tiantian has quit IRC | 19:51 | |
*** merooney has quit IRC | 19:51 | |
ildikov | smcginnis: yeah, we can give that a try | 19:51 |
*** simondodsley_ is now known as simondodsley | 19:51 | |
hemna | because Liberty Cinder can't handle host and instance_uuid at the same time. | 19:51 |
*** esker has quit IRC | 19:51 | |
ildikov | hemna: we're wondering to do a second attach call without a host_name in that case | 19:51 |
smcginnis | ildikov: I may be over simplifying things, but it seems like that should make Liberty Cinder happy without losing any functionality. | 19:52 |
hemna | so | 19:52 |
hemna | weren't we going to 'fix' this by adding a new microversion in Cinder that supports it ? | 19:52 |
ildikov | although as we will not enable multiattach in Mitaka Nova, we can analyse more what needs to be fixed in Cinder to be able to detach the multiattach volumes as well | 19:53 |
*** whydidyoustealmy is now known as barra204 | 19:53 | |
ildikov | smcginnis: yeah, although with Liberty Cinder we should not enable multiattach as the detach will fail with certain drivers :( | 19:54 |
*** thurloat_isgone is now known as thurloat | 19:54 | |
*** resker has joined #openstack-cinder | 19:54 | |
*** dustins_ has quit IRC | 19:54 | |
*** jungleboyj has joined #openstack-cinder | 19:55 | |
*** ircuser-1 has quit IRC | 19:55 | |
smcginnis | ildikov: We can't enable multiattach with Liberty, right? | 19:55 |
ildikov | no, we can't | 19:55 |
smcginnis | hemna: I forget, was everything in there for Liberty? | 19:55 |
hemna | ? | 19:56 |
scottda | I think Nova would need a try: block around the call that passes both host and instance_id to catch the exception from Liberty Cinder, then retry... | 19:56 |
smcginnis | hemna: Was everything on the cinder side for multiattach landed in liberty? | 19:56 |
scottda | microversions won't help that case | 19:56 |
smcginnis | hemna: And yeah, I remember microversions being discussed as a preferred way to know if this is supported from the nova side. | 19:56 |
ildikov | smcginnis: this patch contains comments with the logs from the tempest test regarding which parts of the detach fail: https://review.openstack.org/#/c/266605 | 19:56 |
hemna | I think we landed multiattach stuff in Kilo for Cinder | 19:57 |
smcginnis | Maybe overkill though if we just need exception handling around this in nova. | 19:57 |
smcginnis | hemna: Shoot, that's right. Wow, time flies. | 19:57 |
scottda | We should keep the Cinder-side fix that hemna has put up to revert...the sooner that is in Cinder, the sooner it won't blow up when NOva passes it in. | 19:57 |
ildikov | so the host_name handling is a side-track here | 19:57 |
smcginnis | scottda: Yeah, that's kind of what I'm thinking. | 19:58 |
ildikov | scottda: that part I agree | 19:58 |
smcginnis | I hate to remove it knowing that we actually do need it. | 19:58 |
ildikov | scottda: in general having proper versioning can help in a few things | 19:58 |
hemna | we do need it | 19:58 |
hemna | but I think it should be microversioned | 19:58 |
scottda | It does not hurt Cinder to have it, I don't think | 19:58 |
hemna | so nova can ask for that microversion client | 19:58 |
hemna | the trick is, I think that's done at cinderclient object creation | 19:58 |
hemna | not per request | 19:58 |
hemna | so nova has to know when to pass in host and not. | 19:58 |
hemna | that part of microversions is confusing to me. | 19:59 |
scottda | Yeah, not sure how that is going to work. We'll need some help from NOva folks to figure that out, and it might be too late for that code to get in Nova mitaks | 19:59 |
scottda | s/mitaks/mitaka | 19:59 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication https://review.openstack.org/237387 | 19:59 |
*** merooeny_ has quit IRC | 20:00 | |
*** martyturner has quit IRC | 20:00 | |
smcginnis | Isn't it pythonic to just pass it in and handle if it's not supported? | 20:00 |
scottda | I think we might be able to put some logic in the cinderclient to rip out the host if the cinder (micro)version does not support it. | 20:00 |
ildikov | in Nova the plan is to enable multiattach when all parts of Nova is upgraded to Newton | 20:00 |
scottda | the cinderclient will know what version of cinder it is talking to. | 20:00 |
smcginnis | scottda: Oh, that's an interesting idea, whether microversioned or not. | 20:00 |
*** dustins_ has joined #openstack-cinder | 20:01 | |
*** jaypipes has joined #openstack-cinder | 20:01 | |
hemna | scottda, ok makes sense | 20:01 |
hemna | I guess cinderclient itself will know which version allows the host and instance_uuid to be sent over the wire | 20:02 |
scottda | In fact, that might be the path to reconciling newer Nova stuffs with older cinder | 20:02 |
scottda | I'd want to verify that line of thinking with others, i.e. mriedem , but it seems like that might be the best place to do any translations of parameters, etc. | 20:03 |
scottda | BTW, I've been working on the changes to add new V3 endpoint for microversions. It passes tests on my workstation, but is having issues with Jenkins. | 20:04 |
scottda | which makes me nervous about the timing of getting it landed. | 20:04 |
hemna | coolsvap|away, that's a good first step :) | 20:04 |
hemna | bah | 20:04 |
smcginnis | scottda: Thanks, just looking through the notes from the midcycle on that and realizing we had identified a lot more work. :/ | 20:05 |
scottda | Yup. That's not including the need for testing that jgriffith_away had asked for (I agree with him). | 20:06 |
smcginnis | Yep | 20:06 |
ildikov | Now you only need to update me that how we will solve the remove_export and the disconnect_volume issues :) | 20:08 |
*** ociuhandu has quit IRC | 20:10 | |
*** merooney has joined #openstack-cinder | 20:10 | |
*** merooney has quit IRC | 20:12 | |
hemna | lunch....bbiab | 20:13 |
*** martyturner has joined #openstack-cinder | 20:14 | |
scottda | ildikov: And am I correct that we need to have these issues resolved in Newton? In Mitaka the only things we can do are #1: land microversions (possible, but getting risky) #2: Do not revert hemna patch to accept both host and instance_id (may not be needed if we change directions) | 20:17 |
scottda | #3: Figure out if we can get all drivers to create a new export per attach, regardless of if they are on the same host and/or | 20:17 |
hemna | scottda, I don't think all drivers can do a new export per attach | 20:18 |
scottda | #4: Create a cinder API to give a list of attachements, which might allow nova to have the logic as to when to detach | 20:18 |
ildikov | scottda: I think microversion is good to have and hemna's patch can be useful if we disable live migration for multiattach | 20:18 |
ildikov | scottda: as for #3 that would be crucial if we think that it's the solution for the detach problems | 20:19 |
*** merooney has joined #openstack-cinder | 20:19 | |
scottda | hemna: Yeah, I thought that too, but I just read it in the Nova etherpad, it's an #Action item for jgriffith_away | 20:19 |
hemna | so, should get the revert to land | 20:19 |
ildikov | scottda: and I think it would be best in Mitaka | 20:19 |
hemna | and then I can try working on a microversion follow up patch to re-enable the host + instance_uuid ? | 20:19 |
ildikov | hemna: I mean we can use to have Cinder being able to accept both host and instance_id | 20:20 |
scottda | hemna: I'm not sure about that. What's the original problem with Cinder getting both host and instance_uuid? | 20:20 |
ildikov | hemna: yeah, if you would like to do that with microversion, then in that way | 20:20 |
hemna | the problem is nova is upgraded to M, and Cinder is still at L | 20:20 |
hemna | microversioned change helps that | 20:20 |
hemna | I thinks | 20:20 |
jgriffith_away | scottda: it breaks anybody running an older version of Cinder | 20:21 |
scottda | hemna: Yes, I think so as well. Seems easy if done in cinderclient | 20:21 |
ildikov | we can add a try block to the Nova call and fall back to the original call without host | 20:21 |
ildikov | hemna: ^ | 20:21 |
jgriffith_away | hemna: +1 | 20:21 |
*** jgriffith_away is now known as jgriffith | 20:21 | |
hemna | ok, I'll see if I can hack on that later | 20:22 |
scottda | jgriffith_away: yes, but I'm wondering why it was added in the first place. That is, would it be safe to allow both now, for Mitaka Cinder ? Yes, older Cinder would fail with newer Nova. | 20:22 |
hemna | lets get the revert in for the time being then. | 20:22 |
jgriffith | ildikov: scottda you can't do that | 20:22 |
ildikov | jgriffith: what's up with the new export per attach plans? | 20:22 |
jgriffith | ildikov: scottda particularly you can't rely on a specific version of the client for anything | 20:23 |
jgriffith | ildikov: scottda the proper way to do what I *think* you want would be to have Nova catch the exception and retry with the old method | 20:23 |
scottda | Couldn't Mitaka nova pin to the newest cinderclient or later, which would have the right logic? | 20:23 |
ildikov | jgriffith: yeah, I was wondering to do a retry as smcginnis suggested above | 20:23 |
smcginnis | That's my preference - catch and retry. | 20:23 |
jgriffith | ildikov: scottda but when I mentioned that at the meetup someobdy said it might cause sdague's head to explode | 20:23 |
scottda | we dont' want that. | 20:24 |
jgriffith | scottda: no, certainly not | 20:24 |
jgriffith | he's a super valuable guy... people would hate us :) | 20:24 |
ildikov | jgriffith: there are many parts of the multiattach work which causes similar reactions in Nova :( | 20:24 |
*** sgotliv has joined #openstack-cinder | 20:24 | |
jgriffith | ildikov: I'm just relaying what I was told :) | 20:24 |
ildikov | jgriffith: lol :) | 20:24 |
scottda | So, that leaves #4) Add Cinder API to return all attachements and let Nova figure out when to call terminate_conn and detach | 20:25 |
jgriffith | scottda: but that doesn't solve the problem | 20:25 |
ildikov | jgriffith: to be honest my intention with starting this discussion was to figure out how to safely detach multiattached volumes | 20:25 |
jgriffith | scottda: if you can assume they're on the right version of Cinder and Cinder API you wouldn't have a problem to begin with :) | 20:25 |
ildikov | jgriffith: I'm not sure that this host_name discussion will lead us to the solution | 20:25 |
jgriffith | ildikov: safest way IMHO is to give each their own attachment/target record in the db | 20:26 |
jgriffith | ildikov: and have a ref-counter *somewhere* | 20:26 |
*** alonma has joined #openstack-cinder | 20:26 | |
smcginnis | Or as scottda stated as above. That's my ultimate preferred approach. | 20:27 |
ildikov | jgriffith: you mean even if we have one target for multiple attachments Cinder would remove the target when detaching the last one? | 20:27 |
jgriffith | smcginnis: which? | 20:27 |
smcginnis | Nova needs to know whether to tell us to detach or not. | 20:27 |
jgriffith | smcginnis: ahh.. yeah that :) | 20:27 |
smcginnis | If nova is still using it, don't tell us to get rid of it. | 20:27 |
jgriffith | smcginnis: scottda agreed, and in that model, nova gets a ref counter to manage | 20:27 |
ildikov | smcginnis: in my understanding the behavior depends on the driver that Cinder is using | 20:27 |
*** salv-orlando has joined #openstack-cinder | 20:28 | |
scottda | Or can get those attachment/target records and figure out ref count from that | 20:28 |
jgriffith | smcginnis: scottda that does seem like the *right* place | 20:28 |
jgriffith | ildikov: true, but we'll never have this problem if they're not using a multi-attach version of Cinder :) | 20:28 |
ildikov | smcginnis: if Cinder is deterministic regardless of the driver than Nova could do this, but it seems more to be Cinder's responsibility | 20:28 |
scottda | ildikov: I think that's a big problem. It will not be determinitstic. | 20:29 |
ildikov | jgriffith: will you be the one who tells hemna that the multiattach feature will never be implemented? ;) | 20:29 |
smcginnis | Lol | 20:29 |
ildikov | scottda: that's my thought exactly, this is why Nova cannot handle this part on its own | 20:29 |
jgriffith | ildikov: I'm certainly not saying that at all | 20:30 |
ildikov | jgriffith: sorry, I was just kidding, but I have an odd sense of humor | 20:30 |
jgriffith | ildikov: :) NO, I get it... I do the same thing | 20:30 |
jgriffith | ildikov: I took it as a joke | 20:30 |
*** alonma has quit IRC | 20:30 | |
scottda | It's how we all survive | 20:31 |
ildikov | jgriffith: especially that I worked on the Nova patches over Christmas, this feature can now be killed only throug my dead body ;) | 20:31 |
jgriffith | ildikov: but wanted to be clear before hemna saw it and *his* head exploded :) | 20:31 |
ildikov | jgriffith: haha, got me :D | 20:31 |
ildikov | jgriffith: lol :) | 20:31 |
ildikov | ok, back to the point | 20:32 |
ildikov | having the attachment/target record in the DB sounds like a good direction for me | 20:33 |
ildikov | I mean that way it could be checked in the DB whether the target needs to be removed or not if I'm not mistaken | 20:34 |
ildikov | jgriffith: ^^ | 20:34 |
jgriffith | ildikov: yes, exactly | 20:34 |
jgriffith | ildikov: I'm not sure, but I seem to recall a ref counter could be added to BDM maybe | 20:34 |
jgriffith | but I need to look closer at that | 20:34 |
ildikov | jgriffith: you mean in Nova? | 20:34 |
*** alonma has joined #openstack-cinder | 20:35 | |
jgriffith | ildikov: correct... back to the "don't call detach if you don't really mean it" | 20:35 |
jgriffith | as scottda and smcginnis pointed out | 20:35 |
ildikov | jgriffith: I think we need to find the solution for the target removal in Cinder | 20:35 |
jgriffith | ildikov: perhaps | 20:36 |
ildikov | jgriffith: as how the target is exported depends on the driver in Cinder | 20:36 |
ildikov | jgriffith: in Nova we will need to handle disconnect_volume | 20:36 |
ildikov | jgriffith: as in Nova we call out to os-brick to also remove the initiator | 20:37 |
*** simondodsley has quit IRC | 20:38 | |
smcginnis | Well... if Nova would be changed to not detach if it's still in use, couldn't it also be changed to not ask for a new export for every time it uses it for a different instance? | 20:39 |
smcginnis | I guess I don't get why multiple are needed. | 20:39 |
smcginnis | Expose when needed, do whatever is wanted on the nova side with it, detach when done. | 20:39 |
scottda | jgriffith: So, in your model for an attachment/target record in the DB, would it be 1 record if the driver exports for each attachment on a host, and multiple records if the driver multiplexes the target for multiple attachments on the same host? | 20:39 |
ildikov | jgriffith: also the BDM is the last thing in Nova I would want to get involved in this, especially that it is always said that Cinder is the ulimate source of truth | 20:39 |
*** alonma has quit IRC | 20:40 | |
ildikov | smcginnis: AFAIR hemna said that 3PAR will export new for each attachment | 20:40 |
ildikov | smcginnis: or he said something very similar to this :) | 20:40 |
smcginnis | ildikov: But if Nova never asks for a new attachement... | 20:40 |
ildikov | smcginnis: I don't know that driver enough to know why it is needed or not needed there | 20:41 |
jgriffith | scottda: in my model it would be 1:1 | 20:41 |
smcginnis | jgriffith: +1 | 20:42 |
*** alonma has joined #openstack-cinder | 20:42 | |
*** liverpoo1er has quit IRC | 20:43 | |
*** liverpooler has quit IRC | 20:43 | |
ildikov | but if it's 1:1 it means that each attachment has its own target, right? | 20:43 |
smcginnis | jgriffith: You meant 1:1 as in nova compute host to cinder volume, right? | 20:44 |
scottda | I'm wondering if we could add a new record for each target the driver exposes. Then we could remove a record for each call to terminate_conn, and when the last record is gone we could remove_export | 20:45 |
scottda | That would be the ref counting. | 20:45 |
scottda | Sorry, should not be "each target" but "each attatchment" | 20:45 |
jgriffith | I mean 1:1 as in even if you have a single volume attached multiple times to a single compute-node you have an entry for each attach | 20:46 |
jgriffith | brb | 20:46 |
*** alonma has quit IRC | 20:46 | |
scottda | If SolidFire exposes a new target for each attach in a multi-attach, there would only be 1 record and when detach is called, the export is removed... | 20:46 |
scottda | If 3par uses the same target for many attaches to a single host, there'd be 1 record for each. When the last one is gone, the export is removed. | 20:47 |
*** edtubill has quit IRC | 20:47 | |
scottda | Just an idea as to how to manage this... | 20:48 |
ildikov | scottda: yeah, I was thinking about the same | 20:48 |
*** resker has quit IRC | 20:48 | |
*** alonma has joined #openstack-cinder | 20:48 | |
ildikov | scottda: pushing this to Nova and BDM could but I think it's rather would mess things up | 20:48 |
*** geguileo has joined #openstack-cinder | 20:50 | |
*** Lee1092 has quit IRC | 20:52 | |
*** alonma has quit IRC | 20:52 | |
*** mriedem is now known as mriedem_machell | 20:53 | |
*** thurloat is now known as thurloat_isgone | 20:54 | |
*** alonma has joined #openstack-cinder | 20:54 | |
*** raildo is now known as raildo-afk | 20:54 | |
*** dustins_ is now known as dustins | 20:56 | |
*** jgriffith is now known as jgriffith_away | 20:56 | |
*** garthb_ has quit IRC | 20:57 | |
*** garthb__ has joined #openstack-cinder | 20:57 | |
*** alonma has quit IRC | 20:58 | |
*** merooney has quit IRC | 21:00 | |
*** j_king_ is now known as j_king | 21:01 | |
*** alonma has joined #openstack-cinder | 21:01 | |
openstackgerrit | Angela Smith proposed openstack/cinder: Adds support for configuring zoning in a virtual fabric https://review.openstack.org/275407 | 21:01 |
*** Kimmo_ has joined #openstack-cinder | 21:02 | |
*** mrrowden has quit IRC | 21:03 | |
*** vgridnev has quit IRC | 21:04 | |
*** salv-orl_ has joined #openstack-cinder | 21:04 | |
*** salv-orlando has quit IRC | 21:04 | |
openstackgerrit | Michael Rowden proposed openstack/cinder: Correcting thin provisioning for LVM https://review.openstack.org/275408 | 21:04 |
*** [1]Thelo has joined #openstack-cinder | 21:04 | |
*** alonma has quit IRC | 21:05 | |
*** Thelo has quit IRC | 21:07 | |
*** [1]Thelo is now known as Thelo | 21:07 | |
*** alonma has joined #openstack-cinder | 21:07 | |
*** salv-orl_ has quit IRC | 21:08 | |
*** geguileo has quit IRC | 21:08 | |
*** dalgaaf has quit IRC | 21:08 | |
*** salv-orlando has joined #openstack-cinder | 21:08 | |
*** Apoorva has quit IRC | 21:09 | |
*** amoturi has left #openstack-cinder | 21:09 | |
*** alonma has quit IRC | 21:11 | |
*** alonma has joined #openstack-cinder | 21:13 | |
angela-s | smcginnis: what should i do about this jenkins failure on this spec? https://review.openstack.org/#/c/144384/ | 21:15 |
angela-s | smcginnis: i do not understand what the issue is | 21:15 |
smcginnis | angela-s: Yeah, that's weird. | 21:16 |
smcginnis | angela-s: It looks like some additional checks were added after the first submission. | 21:16 |
*** akerr has quit IRC | 21:17 | |
smcginnis | angela-s: It looks like it needs to be updated to address the errors and warning now pointed out by the test. | 21:17 |
*** alonma has quit IRC | 21:17 | |
*** alonma has joined #openstack-cinder | 21:19 | |
mtanino | jungleboyj: hello, are you there? | 21:20 |
*** esker has joined #openstack-cinder | 21:22 | |
mtanino | jungleboyj: I have posted a bug fix which you marked as high. If you'll have a chance, please give me a feedback. thanks. | 21:22 |
mtanino | https://review.openstack.org/#/c/274204/2 | 21:22 |
angela-s | smcginnis: argh, this sucks. | 21:22 |
angela-s | smcginnis: anyway to just float this one through? :) | 21:22 |
*** ubuntu has joined #openstack-cinder | 21:23 | |
*** ubuntu is now known as Guest40848 | 21:23 | |
* angela-s crossing fingers, but knowing it's of no use | 21:23 | |
*** alonma has quit IRC | 21:24 | |
smcginnis | angela-s: Hah, I considered it too. But since it does change the FCZM driver interface that others would potentially need to implement to, it should probably be updated and merged. | 21:25 |
smcginnis | angela-s: If you can make the changes to make the test happy, I can push that one through as soon as Jenkins is happy with the revisions. | 21:25 |
openstackgerrit | Merged openstack/python-cinderclient: is_public=N/A in output of cinder type-update https://review.openstack.org/274429 | 21:25 |
smcginnis | angela-s: Sorry, I know it's a pain. | 21:25 |
*** alonma has joined #openstack-cinder | 21:26 | |
smcginnis | Be back online later. Need to drive home through the snow storm. Yay. :| | 21:26 |
mtanino | smcginnis: take care. | 21:27 |
*** esker has quit IRC | 21:28 | |
hemna | angela-s, what's up ? | 21:29 |
angela-s | hemna: the sky | 21:29 |
hemna | :) | 21:30 |
*** alonma has quit IRC | 21:30 | |
angela-s | hemna: i noticed you tried to do recheck on that spec failure, thanks, looks like i'm gonna have to work on the formatting though | 21:31 |
angela-s | and probably on the other spec too, argh... | 21:32 |
hemna | ah ok. yah I wasn't sure why it puked | 21:32 |
hemna | I figured a recheck would 'fix' it :P | 21:32 |
*** alonma has joined #openstack-cinder | 21:32 | |
*** Apoorva has joined #openstack-cinder | 21:32 | |
angela-s | would've been nice, but not my luck | 21:32 |
*** tiantian has joined #openstack-cinder | 21:33 | |
*** zul has quit IRC | 21:35 | |
*** huangtianhua has quit IRC | 21:36 | |
*** alonma has quit IRC | 21:36 | |
scottda | angela-s: It looks like you removed the title for "use cases" . That might be why that fails. | 21:36 |
*** alonma has joined #openstack-cinder | 21:38 | |
patrickeast | hemna: hey, got a min for https://review.openstack.org/#/c/236143/14/cinder/volume/drivers/hpe/hpe_3par_common.py ? | 21:38 |
*** porrua has quit IRC | 21:38 | |
*** timcl has quit IRC | 21:40 | |
hemna | hey | 21:40 |
hemna | I'm probably missing something, or stupid today | 21:40 |
hemna | or both! | 21:40 |
patrickeast | haha | 21:40 |
patrickeast | so what i was suggestion is that you *could* catch the not found error | 21:41 |
patrickeast | and ignore it | 21:41 |
hemna | oh | 21:41 |
hemna | lolz | 21:41 |
hemna | yah | 21:41 |
hemna | duh | 21:41 |
hemna | that would be odd that happened, but sure | 21:41 |
hemna | in the case that someone else is deleting at the same time I suppose | 21:41 |
patrickeast | yea i only suggest it since it looks like those temp snapshots expire | 21:41 |
patrickeast | i dunno exactly what that means | 21:41 |
hemna | yah they can expire | 21:42 |
patrickeast | but if they are deleted when time runs out, its a chance for them to be deleted under cinder | 21:42 |
hemna | and they can vanish, depending on the setting and how busy the array is | 21:42 |
patrickeast | gotcha | 21:42 |
hemna | yah good call. | 21:42 |
hemna | ok I wasn't following that in the chatter in the review. | 21:42 |
hemna | thanks, I'll look for that and push up another one. | 21:42 |
*** alonma has quit IRC | 21:42 | |
hemna | like I said....missing something and stupid | 21:42 |
hemna | :P | 21:42 |
patrickeast | haha, nah looking at the comments in gerrit i was having trouble even following | 21:43 |
*** haomaiwang has joined #openstack-cinder | 21:44 | |
*** alonma has joined #openstack-cinder | 21:44 | |
*** cknight has quit IRC | 21:45 | |
hemna | man gerrit seems like a POS today | 21:48 |
hemna | slow, and I just now got a random 503 | 21:48 |
*** alonma has quit IRC | 21:48 | |
*** haomaiwang has quit IRC | 21:49 | |
hemna | and a 502 | 21:49 |
hemna | *sigh* | 21:49 |
*** alonma has joined #openstack-cinder | 21:50 | |
*** thurloat_isgone is now known as thurloat | 21:52 | |
*** esker has joined #openstack-cinder | 21:53 | |
jbernard | hemna: i like my gerrit the same way i like my coffee: ground up and in the freezer | 21:53 |
hemna | :) | 21:53 |
jbernard | that one needs some more work | 21:54 |
*** alonma has quit IRC | 21:55 | |
*** erlon has quit IRC | 21:56 | |
*** alonma has joined #openstack-cinder | 21:57 | |
mc_nair | anyone know of a way to set configure the minimum volume size for Cinder? I see that Rackspace has minimum volume size requirements, but haven't been able to find any code on it so guessing that's custom code... | 22:00 |
*** doffm_ is now known as doffm | 22:01 | |
*** alonma has quit IRC | 22:01 | |
*** esker has quit IRC | 22:01 | |
*** esker has joined #openstack-cinder | 22:02 | |
*** alonma has joined #openstack-cinder | 22:03 | |
*** edmondsw has quit IRC | 22:05 | |
*** thangp has left #openstack-cinder | 22:05 | |
*** gouthamr has quit IRC | 22:06 | |
guitarzan | mc_nair: our backend does it itself | 22:06 |
*** alonma has quit IRC | 22:08 | |
mc_nair | guitarzan: cool, thanks. Wasn't sure if there was a global Cinder config or just up to backend if they care about that sort of thing (think I've got my answer though) | 22:08 |
angela-s | scottda: thanks! i'll try that first :) | 22:08 |
guitarzan | adding it to cinder proper was suggested and people didn't seem to think it was too crazy of an idea | 22:09 |
*** alonma has joined #openstack-cinder | 22:13 | |
*** edtubill has joined #openstack-cinder | 22:14 | |
*** sgotliv has quit IRC | 22:15 | |
*** esker has quit IRC | 22:16 | |
*** jwcroppe has quit IRC | 22:17 | |
*** jgregor has quit IRC | 22:17 | |
*** jwcroppe has joined #openstack-cinder | 22:17 | |
*** alonma has quit IRC | 22:17 | |
*** alonma has joined #openstack-cinder | 22:22 | |
*** itlinux has quit IRC | 22:22 | |
*** itlinux has joined #openstack-cinder | 22:24 | |
*** lprice has quit IRC | 22:24 | |
*** alonma has quit IRC | 22:26 | |
jungleboyj | guitarzan: Thanks. I thought we had discussed it in the past. So it is just something that hasn't been implemented? | 22:28 |
*** alonma has joined #openstack-cinder | 22:28 | |
*** zul has joined #openstack-cinder | 22:29 | |
guitarzan | yeah, no one (me?) has taken the time to do it | 22:30 |
openstackgerrit | Angela Smith proposed openstack/cinder-specs: Spec for FCZM Zone Driver Friendly Zone names https://review.openstack.org/144384 | 22:30 |
*** e0ne has joined #openstack-cinder | 22:31 | |
angela-s | smcginnis: ^^^ spec is fixed. scottda: thank you! | 22:31 |
*** jwcroppe has quit IRC | 22:31 | |
*** alonma has quit IRC | 22:32 | |
*** dustins has quit IRC | 22:34 | |
*** alonma has joined #openstack-cinder | 22:34 | |
*** eharney has quit IRC | 22:35 | |
*** tpsilva has quit IRC | 22:36 | |
*** alonma has quit IRC | 22:38 | |
*** authority_ has quit IRC | 22:39 | |
openstackgerrit | Angela Smith proposed openstack/cinder-specs: Spec for Brocade Zone Driver Virtual Fabrics support https://review.openstack.org/144389 | 22:39 |
*** jamielennox|away is now known as jamielennox | 22:41 | |
*** winston-d_ has quit IRC | 22:42 | |
*** zul has quit IRC | 22:42 | |
*** e0ne has quit IRC | 22:43 | |
*** baumann has quit IRC | 22:44 | |
*** itlinux_ has joined #openstack-cinder | 22:44 | |
*** edtubill has quit IRC | 22:47 | |
*** itlinux has quit IRC | 22:47 | |
*** EinstCrazy has joined #openstack-cinder | 22:48 | |
*** EinstCrazy has quit IRC | 22:52 | |
*** ntpttr has left #openstack-cinder | 22:53 | |
*** itlinux_ has quit IRC | 22:54 | |
*** itlinux has joined #openstack-cinder | 22:55 | |
*** bwallis_ has quit IRC | 22:59 | |
*** sinese has quit IRC | 23:00 | |
*** jwcroppe has joined #openstack-cinder | 23:03 | |
smcginnis | angela-s: Thanks. Will take a look once Jenkins blesses it. | 23:05 |
*** jamielennox is now known as jamielennox|away | 23:05 | |
scottda | angela-s: cool! | 23:11 |
*** salv-orlando has quit IRC | 23:13 | |
*** salv-orlando has joined #openstack-cinder | 23:13 | |
*** kfarr has quit IRC | 23:16 | |
*** e0ne has joined #openstack-cinder | 23:19 | |
*** rlrossit_ has quit IRC | 23:22 | |
*** thurloat is now known as thurloat_isgone | 23:23 | |
*** xyang1 has quit IRC | 23:25 | |
*** e0ne has quit IRC | 23:28 | |
*** haomaiwang has joined #openstack-cinder | 23:33 | |
*** haomaiwang has quit IRC | 23:37 | |
openstackgerrit | Merged openstack/cinder-specs: Spec for FCZM Zone Driver Friendly Zone names https://review.openstack.org/144384 | 23:45 |
*** boris-42 has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!